Re: [PATCH v4 0/8] GICv3 LPI and ITS feature implementation

[Peter Maydell]

On Wed, 2 Jun 2021 at 19:00, Shashi Mallela <shashi.mallela@linaro.org> wrote:
>
> This patchset implements qemu device model for enabling physical
> LPI support and ITS functionality in GIC as per GICv3 specification.
> Both flat table and 2 level tables are implemented.The ITS commands
> for adding/deleting ITS table entries,trigerring LPI interrupts are
> implemented.Translated LPI interrupt ids are processed by redistributor
> to determine priority and set pending state appropriately before
> forwarding the same to cpu interface.
> The ITS feature support has been added to sbsa-ref platform as well as
> virt platform,wherein the emulated functionality co-exists with kvm
> kernel functionality.
>
> Changes in v4:
>  - review comments addressed
>  - redesigned the lpi pending priority determination logic to scan
>    LPI pending table and config table right after lpi pending
>    statechanges(SET/RESET) through gicv3_redist_update_lpi() call to
>    determinethe highest priority lpi among the active lpis and save
>    the details.The high priority interrupt determination logic in
>    redistributor now usesthe saved high priority lpi details
>    (alongside other interrupt types) instead of calling
>    gicv3_redist_update_lpi() everytime(as in v3).This
>    significantly reduces the call overhead associated with
>    address_space_read of lpi config and pending tables.
>    Testing with this new design showed no boot delays.
>  - profiled execution of gicv3_redist_update_lpi() using perf and
>    framegraph to confirm execution is within normal limits.
>    Also,specifically measured execution time to be an average of 175us
>    with linux distro testing.
>  - All kvm_unit_tests PASS

This still fails to build with clang, in the same way as v3 failed.
Also (as noted in my other email, you need to integrate the updates
to the ACPI table test data into this series; 'make' and 'make check'
should work at every step in the patch series.

thanks
-- PMM