mirror of
https://gitlab.com/libvirt/libvirt.git
synced 2024-12-22 13:45:38 +00:00
de3289e2b5
There was one attempt a year ago done by me to drop HAL [1] but it was never resolved. There was another time when Dan suggested to drop HAL driver [2] but it was decided to keep it around in case device assignment will be implemented for FreeBSD and the fact that virt-manager uses node device driver [3]. I checked git history and code and it doesn't look like bhyve supports device assignment so from that POV it should not block removing HAL. The argument about virt-manager is not strong as well because libvirt installed from FreeBSD packages doesn't have HAL support so it will not affect these users as well [4]. The only users affected by this change would be the ones compiling libvirt from GIT on FreeBSD. I looked into alternatives and there is libudev-devd package on FreeBSD but unfortunately it doesn't work as it doesn't list any devices when used with libvirt. It provides libudev APIs using devd. I also looked into devd directly and it provides some APIs but there are no APIs for device monitoring and events so that would have to be somehow done by libvirt. Main motivation for dropping HAL support is to replace libdbus with GLib dbus implementation and it cannot be done with HAL driver present in libvirt because HAL APIs heavily depends on symbols provided by libdbus. [1] <https://www.redhat.com/archives/libvir-list/2019-May/msg00203.html> [2] <https://www.redhat.com/archives/libvir-list/2016-April/msg00992.html> [3] <https://www.redhat.com/archives/libvir-list/2016-April/msg00994.html> [4] <https://svnweb.freebsd.org/ports/head/devel/libvirt/Makefile?view=markup> Signed-off-by: Pavel Hrdina <phrdina@redhat.com> Reviewed-by: Michal Privoznik <mprivozn@redhat.com> |
||
|---|---|---|
| .. | ||
| apibuild.py | ||
| augeas-gentest.py | ||
| check-aclperms.py | ||
| check-aclrules.py | ||
| check-driverimpls.py | ||
| check-drivername.py | ||
| check-file-access.py | ||
| check-remote-protocol.py | ||
| check-symfile.py | ||
| check-symsorting.py | ||
| cocci-macro-file.h | ||
| dtrace2systemtap.py | ||
| esx_vi_generator.py | ||
| genaclperms.py | ||
| genpolkit.py | ||
| gensystemtap.py | ||
| group-qemu-caps.py | ||
| header-ifdef.py | ||
| hvsupport.py | ||
| hyperv_wmi_generator.py | ||
| meson-dist.py | ||
| meson-gen-authors.py | ||
| meson-gen-def.py | ||
| meson-gen-sym.py | ||
| meson-html-gen.py | ||
| meson-install-dirs.py | ||
| meson-install-symlink.py | ||
| meson-install-web.py | ||
| meson-python.sh | ||
| meson-timestamp.py | ||
| meson.build | ||
| mock-noinline.py | ||
| prohibit-duplicate-header.py | ||
| test-wrap-argv.py | ||