Re: [PATCH v2 0/6] Resolve TYPE_PIIX3_XEN_DEVICE

[Bernhard Beschow <shentey@xxxxxxxxx>]

Am 12. März 2023 21:02:03 UTC schrieb Chuck Zmudzinski <brchuckz@xxxxxxx>:
>On 3/12/23 5:22 AM, Bernhard Beschow wrote:
>> 
>> 
>> Am 11. März 2023 22:20:29 UTC schrieb Chuck Zmudzinski <brchuckz@xxxxxxx>:
>>>On 2/9/2023 4:53 PM, Bernhard Beschow wrote:
>>>> Am 1. Februar 2023 08:11:10 UTC schrieb Bernhard Beschow 
>>>> <shentey@xxxxxxxxx>:
>>>> >
>>>> >
>>>> >Am 24. Januar 2023 17:07:30 UTC schrieb Bernhard Beschow 
>>>> ><shentey@xxxxxxxxx>:
>>>> >>
>>>> >>
>>>> >>Am 24. Januar 2023 16:11:47 UTC schrieb Anthony PERARD 
>>>> >><anthony.perard@xxxxxxxxxx>:
>>>> >>>On Wed, Jan 18, 2023 at 05:13:03AM -0500, Michael S. Tsirkin wrote:
>>>> >>>> On Wed, Jan 04, 2023 at 03:44:31PM +0100, Bernhard Beschow wrote:
>>>> >>>> > This series first renders TYPE_PIIX3_XEN_DEVICE redundant and 
>>>> >>>> > finally removes
>>>> >>>> > it. The motivation is to 1/ decouple PIIX from Xen and 2/ to make 
>>>> >>>> > Xen in the PC
>>>> >>>> > machine agnostic to the precise southbridge being used. 2/ will 
>>>> >>>> > become
>>>> >>>> > particularily interesting once PIIX4 becomes usable in the PC 
>>>> >>>> > machine, avoiding
>>>> >>>> > the "Frankenstein" use of PIIX4_ACPI in PIIX3.
>>>> >>>> 
>>>> >>>> Looks ok to me.
>>>> >>>> Reviewed-by: Michael S. Tsirkin <mst@xxxxxxxxxx>
>>>> >>>> 
>>>> >>>> Feel free to merge through Xen tree.
>>>> >>>
>>>> >>>Hi Bernhard,
>>>> >>
>>>> >>Hi Anthony,
>>>> >>
>>>> >>>The series currently doesn't apply on master. And a quick try at
>>>> >>>applying the series it is based on also failed. Could you rebase it , or
>>>> >>>maybe you would prefer to wait until the other series "Consolidate
>>>> >>>PIIX..." is fully applied?
>>>> >>
>>>> >>Thanks for looking into it!
>>>> >>
>>>> >>You can get the compilable series from 
>>>> >>https://patchew.org/QEMU/20230104144437.27479-1-shentey@xxxxxxxxx/ . If 
>>>> >>it doesn't work for you let me know, then I can rebase onto master. All 
>>>> >>necessary dependencies for the series are upstreamed meanwhile.
>>>> >
>>>> >Ping
>>>>
>>>> Ping^2
>>>
>>>Hi Bernhard,
>> 
>> Hi Chuck,
>> 
>>>I took a look at this today to see why it cannot be applied.
>> 
>> Thanks for looking at it!
>> 
>>>I can see clearly that
>>>all the prerequisite patches have *not* been applied to master yet, so I can
>>>understand why Anthony cannot pull this up yet. Specifically, the series that
>>>consolidates PIIX3 and PIIX4 south bridges is not yet applied, and that is 
>>>one of
>>>the prerequisites. I think you said it was reviewed, but it apparently never 
>>>got
>>>pulled up into master.
>> 
>> Correct, the PIIX consolidation series isn't merged yet. This series 
>> currently depends on it to avoid merge conflicts but doesn't need it 
>> otherwise. Back then I anticipated that the consolidation series would land 
>> in master soon since it was fully reviewed before this one. But that turned 
>> out not to be the case.
>> 
>> This series depends on some necessary refactoring [1] which I did in the 
>> context of PIIX consolidation which is already in master. So this series can 
>> easily be rebased onto master and it even simplifies the consolidation 
>> series a bit. I'll take this route now and I'll post a v3.
>
>Thanks for posting v3, I was at a loss trying to figure out how to merge the 
>30-patch piix
>consolidation series into the master branch.

Yeah, the code suffered from bit rod...

>I just tested your recent v3 (all 6 patches)
>on top of the current master branch and it works well on my Xen guests, so you 
>can keep my
>Tested-by tag on that patch series!

Thanks!

Best regards,
Bernhard

>
>Kind regards,
>
>Chuck