On Fri, 23 Dec 2022 at 09:01, Alexander Graf <ag...@csgraf.de> wrote: > > The finalize_gic_version() function tries to determine which GIC version > the current accelerator / host combination supports. During the initial > HVF porting efforts, I didn't realize that I also had to touch this > function. Then Zenghui brought up this function as reply to my HVF GICv3 > enablement patch - and boy it is a mess. > > This patch set cleans up all of the GIC finalization so that we can > easily plug HVF in and also hopefully will have a better time extending > it in the future. As second step, it explicitly adds HVF support and > fails loudly for any unsupported accelerators. > > Alex > > v1 -> v2: > > - Leave VIRT_GIC_VERSION defines intact, we need them for MADT generation > - Include TCG header for tcg_enabled() > > v2 -> v3: > > - Fix comment > - Flip kvm-enabled logic for host around > > Alexander Graf (2): > hw/arm/virt: Consolidate GIC finalize logic > hw/arm/virt: Make accels in GIC finalize logic explicit
Since AIUI these patches depend on "hvf: arm: Add support for GICv3", would you mind including these when you respin that one (ie make a series-of-3-patches)? That way I don't need to keep this series on my to-review queue just because it's blocked on another patch :-) thanks -- PMM
