Am 11. März 2023 22:20:29 UTC schrieb Chuck Zmudzinski <[email protected]>:
>On 2/9/2023 4:53 PM, Bernhard Beschow wrote:
>> Am 1. Februar 2023 08:11:10 UTC schrieb Bernhard Beschow <[email protected]>:
>> >
>> >
>> >Am 24. Januar 2023 17:07:30 UTC schrieb Bernhard Beschow
>> ><[email protected]>:
>> >>
>> >>
>> >>Am 24. Januar 2023 16:11:47 UTC schrieb Anthony PERARD
>> >><[email protected]>:
>> >>>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 <[email protected]>
>> >>>>
>> >>>> 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/[email protected]/ . 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.
Best regards,
Bernhard
[1] https://lore.kernel.org/qemu-devel/[email protected]/
>
>For reference, here is the link to the prerequisite patch set I tested with
>this patch set:
>
>https://lore.kernel.org/qemu-devel/[email protected]/
>
>The patch set I tested is a 30-patch series, and I don't know if it has
>been partially applied. The title of that patch set is:
>
>This series consolidates the implementations of the PIIX3 and PIIX4 south
>
>So before this patch set to resolve the TYPE_PIIX3_XEN_DEVICE can be
>applied, the patch set to consolidate the PIIX3 and PIIX4 south bridges
>needs to be applied.
>
>I don't know if the feature freeze means these patches that do not add any
>new features still need to wait until the next development cycle.
>
>Kind regards,
>
>Chuck
>
>> >
>> >>
>> >>Thanks,
>> >>Bernhard
>> >>>
>> >>>Thanks.
>> >>>
>> >>>> > Testing done:
>> >>>> > None, because I don't know how to conduct this properly :(
>> >>>> >
>> >>>> > Based-on: <[email protected]>
>> >>>> > "[PATCH v4 00/30] Consolidate PIIX south bridges"
>> >>>
>
