>On 5/19/25 08:59, edmund.raile wrote:
>>> Hello,
>>>
>>> On 5/19/25 02:01, edmund.raile wrote:
>>>>>> Restore SR-IOV Intel iGPU VF passthrough capability:
>>>>>> Check x-igd-opregion=off parameter in vfio_pci_igd_config_quirk and
>>>>>> vfio_pci_kvmgt_config_quirk to ensure x-igd-opregion=off is
>>>>>> respected despite subsequent attempt of automatic
>>>>>> IGD opregion detection.
>>>>>>
>>>>>> Fixes: 7be29f2f1a3f ("Merge tag 'pull-vfio-20250509' of 
>>>>>> https://github.com/legoater/qemu into staging")
>>>>>
>>>>> Can you bisect to a specific commit from the merge?  The merge may be
>>>>> where the issue was introduced to mainline, but the merge commit itself
>>>>> is more than likely not the source of the issue.  Thanks,
>>>>>
>>>>> Alex
>>>>
>>>> I have to admit, I have no idea how to pluck apart
>>>> that merge pull using git.
>>>
>>> Please use git-bisect. The manual contains enough information to get
>>> you on the right track.
>>
>> I'm quite impressed that git bisect in main repo has the power
>> to bisect into commits on a branch of a fork repo
>
>I wouldn't used a fork repo to isolate the issue.
>
>vfio-10.1 is a dev branch. patches come and go.
>
>vfio-next contains candidates for upstream. The tree should be stable
>enough but patches are force pushed on this branch to keep it in sync
>with upstream.
>
>Please use the main QEMU repo instead.

That is what I did and did not expect to work.
The commit ID is from the main repo.

>
>> despite
>> being summarized into one commit on the main repo.
>> I did not think this was possible which is why I said I don't know
>> how to do this.
>>
>> c0273e77f2d7aab3312eb557b49332da528ff66b is the first bad commit
>>
>> So merging it into the main qemu repo changed the commit number
>> but it's the very commit I found; it adds the automatic IGD
>> opregion detection.
>>
>> Should I send in a patch v2 and correct the Fixes tag or can you do
>> this while committing?
>>
>> This would then be the correct tag:
>> Fixes: c0273e77f2d7 ("vfio/igd: Detect IGD device by OpRegion")
>
>
>Please send a v2.

Will do that.

>Use ./scripts/get_maintainer.pl to get the email recipients.

I did that originally but included Stefan Hajnoczi
<stefa...@redhat.com> since he did the original merge pull.

>
>
>Thanks,
>
>C.

Reply via email to