[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] hw/vfio/pci-quirks: Fix broken legacy IGD passthrough
From: |
Thomas Huth |
Subject: |
Re: [PATCH] hw/vfio/pci-quirks: Fix broken legacy IGD passthrough |
Date: |
Wed, 10 Jun 2020 09:50:46 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.9.0 |
On 10/06/2020 09.31, Philippe Mathieu-Daudé wrote:
> On 6/10/20 5:51 AM, Thomas Huth wrote:
>> The #ifdef CONFIG_VFIO_IGD in pci-quirks.c is not working since the
>> required header config-devices.h is not included, so that the legacy
>> IGD passthrough is currently broken. Let's include the right header
>> to fix this issue.
>>
>> Buglink: https://bugs.launchpad.net/qemu/+bug/1882784
>> Fixes: 29d62771c81d8fd244a67c14a1d968c268d3fb19
>> ("hw/vfio: Move the IGD quirk code to a separate file")
>
> What about shorter tag?
>
> Fixes: 29d62771c81 ("vfio: Move the IGD quirk code to a separate file")
I always forget whether to use the short or the long version for
"Fixes:" ... this can hopefully be fixed (if necessary) when the patch
gets picked up.
>> Signed-off-by: Thomas Huth <thuth@redhat.com>
>> ---
>> hw/vfio/pci-quirks.c | 1 +
>> 1 file changed, 1 insertion(+)
>>
>> diff --git a/hw/vfio/pci-quirks.c b/hw/vfio/pci-quirks.c
>> index f2155ddb1d..3158390db1 100644
>> --- a/hw/vfio/pci-quirks.c
>> +++ b/hw/vfio/pci-quirks.c
>> @@ -11,6 +11,7 @@
>> */
>>
>> #include "qemu/osdep.h"
>> +#include "config-devices.h"
>
> I've been wondering how we can avoid that mistake in the
> future, but can find anything beside human review.
I think in the long term, we should include config-devices.h in osdep.h,
just like config-host.h and config-target.h is already included there.
Everything else is just too confusing. But then we should also add a
mechanism to poison the switches from config-devices.h in common code...
thus this likely needs some work and discussion of the patch first, so I
think we should go with this change to pci-quirks.c here first to get
the regression fixed ASAP.
Thomas
- [PATCH] hw/vfio/pci-quirks: Fix broken legacy IGD passthrough, Thomas Huth, 2020/06/09
- Re: [PATCH] hw/vfio/pci-quirks: Fix broken legacy IGD passthrough, Philippe Mathieu-Daudé, 2020/06/10
- Re: [PATCH] hw/vfio/pci-quirks: Fix broken legacy IGD passthrough,
Thomas Huth <=
- Re: [PATCH] hw/vfio/pci-quirks: Fix broken legacy IGD passthrough, Philippe Mathieu-Daudé, 2020/06/10
- Re: [PATCH] hw/vfio/pci-quirks: Fix broken legacy IGD passthrough, Thomas Huth, 2020/06/10
- Re: [PATCH] hw/vfio/pci-quirks: Fix broken legacy IGD passthrough, Philippe Mathieu-Daudé, 2020/06/10
- Re: [PATCH] hw/vfio/pci-quirks: Fix broken legacy IGD passthrough, Thomas Huth, 2020/06/10
- Re: [PATCH] hw/vfio/pci-quirks: Fix broken legacy IGD passthrough, Philippe Mathieu-Daudé, 2020/06/10
- Re: [PATCH] hw/vfio/pci-quirks: Fix broken legacy IGD passthrough, Alex Williamson, 2020/06/11
- Re: [PATCH] hw/vfio/pci-quirks: Fix broken legacy IGD passthrough, Laurent Vivier, 2020/06/10
- Re: [PATCH] hw/vfio/pci-quirks: Fix broken legacy IGD passthrough, Thomas Huth, 2020/06/10