Am 12. März 2023 21:02:03 UTC schrieb Chuck Zmudzinski <brchu...@aol.com>:
>On 3/12/23 5:22 AM, Bernhard Beschow wrote:
>>
>>
>> Am 11. März 2023 22:20:29 UTC schrieb Chuck Zmudzinski <brchu...@aol.com>:
>>>On 2/9/2023 4:53 PM, Bernhard Beschow wrote:
>>>> Am 1. Februar 2023 08:11:10 UTC schrieb Bernhard Beschow
>>>> <shen...@gmail.com>:
>>>> >
>>>> >
>>>> >Am 24. Januar 2023 17:07:30 UTC schrieb Bernhard Beschow
>>>> ><shen...@gmail.com>:
>>>> >>
>>>> >>
>>>> >>Am 24. Januar 2023 16:11:47 UTC schrieb Anthony PERARD
>>>> >><anthony.per...@citrix.com>:
>>>> >>>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 <m...@redhat.com>
>>>> >>>>
>>>> >>>> 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-shen...@gmail.com/ . 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
