Displaying 4 results from an estimated 4 matches for "virpcidevicelistfind".
2020 Aug 09
3
multiple vms with same PCI passthrough
...ot the failing test cases)
would be:
diff --git a/src/util/virpci.c b/src/util/virpci.c
index 47c671daa0..a00c5e6f44 100644
--- a/src/util/virpci.c
+++ b/src/util/virpci.c
@@ -1597,7 +1597,7 @@ int
virPCIDeviceListAdd(virPCIDeviceListPtr list,
virPCIDevicePtr dev)
{
- if (virPCIDeviceListFind(list, dev)) {
+ if (virPCIDeviceBusContainsActiveDevices(dev, list)) {
virReportError(VIR_ERR_INTERNAL_ERROR,
_("Device %s is already in use"), dev->name);
return -1;
Is this too simplistic or undesirable a feature request/implementation?...
2020 Aug 16
0
multiple vms with same PCI passthrough
...ot the failing test cases)
would be:
diff --git a/src/util/virpci.c b/src/util/virpci.c
index 47c671daa0..a00c5e6f44 100644
--- a/src/util/virpci.c
+++ b/src/util/virpci.c
@@ -1597,7 +1597,7 @@ int
virPCIDeviceListAdd(virPCIDeviceListPtr list,
virPCIDevicePtr dev)
{
- if (virPCIDeviceListFind(list, dev)) {
+ if (virPCIDeviceBusContainsActiveDevices(dev, list)) {
virReportError(VIR_ERR_INTERNAL_ERROR,
_("Device %s is already in use"), dev->name);
return -1;
Is this too simplistic or undesirable a feature request/implementation?...
2020 Aug 17
0
Re: multiple vms with same PCI passthrough
...--git a/src/util/virpci.c b/src/util/virpci.c
> index 47c671daa0..a00c5e6f44 100644
> --- a/src/util/virpci.c
> +++ b/src/util/virpci.c
> @@ -1597,7 +1597,7 @@ int
> virPCIDeviceListAdd(virPCIDeviceListPtr list,
> virPCIDevicePtr dev)
> {
> - if (virPCIDeviceListFind(list, dev)) {
> + if (virPCIDeviceBusContainsActiveDevices(dev, list)) {
> virReportError(VIR_ERR_INTERNAL_ERROR,
> _("Device %s is already in use"), dev->name);
> return -1;
>
> Is this too simplistic or undesirable a...
2020 Aug 18
2
Re: multiple vms with same PCI passthrough
...pci.c
> > index 47c671daa0..a00c5e6f44 100644
> > --- a/src/util/virpci.c
> > +++ b/src/util/virpci.c
> > @@ -1597,7 +1597,7 @@ int
> > virPCIDeviceListAdd(virPCIDeviceListPtr list,
> > virPCIDevicePtr dev)
> > {
> > - if (virPCIDeviceListFind(list, dev)) {
> > + if (virPCIDeviceBusContainsActiveDevices(dev, list)) {
> > virReportError(VIR_ERR_INTERNAL_ERROR,
> > _("Device %s is already in use"), dev->name);
> > return -1;
> >
> > Is this too...