[libvirt] [PATCH 1/1] util/virhostdev: consolidate duplicated KVM support code
Michal Privoznik
mprivozn at redhat.com
Thu May 2 09:43:36 UTC 2019
On 4/30/19 11:51 PM, Daniel Henrique Barboza wrote:
> tests/virhostdevtest.c implements a function called
> 'virHostdevHostSupportsPassthroughKVM', that is equal to
> 'qemuHostdevHostSupportsPassthroughLegacy' that is declared
> inside qemu/qemu_hostdev.c.
>
> This patch removes the duplicated code from both files and
> and puts it inside util/virhostdev.c, under the name
> virHostdev...KVM, which represents what the code does better
> than using 'Legacy'.
In fact, I'd rather see the function dropped from the tests completely.
It's checking if the host the test us run on supports device assignment,
but we're using pcimock to emulate all PCI detach/attach. The test
should not be host dependent. It can still be architecture/OS dependent
though (e.g. tehre's no sysfs on FreeBSD).
Michal
More information about the libvir-list
mailing list