[PATCH] hw/intc: cannot clear GICv3 ITS CTLR[Enabled] bit

2021-11-24 Thread Shashi Mallela
When Enabled bit is cleared in GITS_CTLR,ITS feature continues to be enabled.This patch fixes the issue. Signed-off-by: Shashi Mallela --- hw/intc/arm_gicv3_its.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/hw/intc/arm_gicv3_its.c b/hw/intc/arm_gicv3_its.c index

RE: [for-6.2] hw/intc/arm_gicv3: Update cached state after acknowledging LPI

2021-11-23 Thread Shashi Mallela
Since LPIs do not have an active or active and pending state,the current implementation only clears the LPI pending state from the pending table once ICC_IAR1_EL1 acknowledges the interrupt.But, as part of gicv3_lpi_pending() processing, cs->hpplpi is updated with the next best priotiy lpi (only

RE: [PATCH v8 07/10] hw/arm/sbsa-ref: add ITS support in SBSA GIC

2021-11-11 Thread Shashi Mallela
From: Leif LindholmSent: November 11, 2021 1:21 PMTo: Peter MaydellCc: Shashi Mallela; Radoslaw Biernacki; Michael S. Tsirkin; Igor Mammedov; qemu-arm; QEMU Developers; Eric Auger; narmstr...@baylibre.com; Alex Bennée; Marcin JuszkiewiczSubject: Re: [PATCH v8 07/10] hw/arm/sbsa-ref: add ITS

[PATCH] hw/intc: GIC maintenance interrupt not triggered

2021-09-15 Thread Shashi Mallela
level to 1 if any of the maintenance interrupt attributes are set. Confirmed that the GIC maintanence interrupts are triggered and sbsa acs test cases passed with this change. Signed-off-by: Shashi Mallela --- hw/intc/arm_gicv3_cpuif.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff

Re: SMMU Stage 2 translation in QEMU

2021-09-14 Thread shashi . mallela
Hi Eric, On Mon, 2021-09-13 at 10:19 +0200, Eric Auger wrote: > Hi Shashi, > > On 9/10/21 3:32 PM, shashi.mall...@linaro.org wrote: > > So that would be the driver code running in guest OS because i see > > tables being setup by arm-smmu driver code in linux,which is > > similar to > > what

[PATCH v9 9/9] tests/data/acpi/virt: Update IORT files for ITS

2021-09-10 Thread Shashi Mallela
0 00 00 00 00 00 00 00 00 00 FF FF 00 00 // 0070: 00 00 00 00 30 00 00 00 00 00 00 00 // ....0... Signed-off-by: Shashi Mallela Acked-by: Igor Mammedov Reviewed-by: Peter Maydell --- tests/data/acpi/virt/IORT | Bin 0 -> 124 bytes tests/data/acpi/virt/IORT.me

[PATCH v9 8/9] hw/arm/virt: add ITS support in virt GIC

2021-09-10 Thread Shashi Mallela
Included creation of ITS as part of virt platform GIC initialization. This Emulated ITS model now co-exists with kvm ITS and is enabled in absence of kvm irq kernel support in a platform. Signed-off-by: Shashi Mallela Reviewed-by: Peter Maydell --- hw/arm/virt.c | 29

[PATCH v9 7/9] tests/data/acpi/virt: Add IORT files for ITS

2021-09-10 Thread Shashi Mallela
Added expected IORT files applicable with latest GICv3 ITS changes.Temporarily differences in these files are okay. Signed-off-by: Shashi Mallela Acked-by: Igor Mammedov Reviewed-by: Peter Maydell --- tests/data/acpi/virt/IORT | 0 tests/data/acpi/virt/IORT.memhp

[PATCH v9 6/9] hw/intc: GICv3 redistributor ITS processing

2021-09-10 Thread Shashi Mallela
of ITS INT,CLEAR,DISCARD command and GITS_TRANSLATER processing. Signed-off-by: Shashi Mallela Tested-by: Neil Armstrong Reviewed-by: Peter Maydell --- hw/intc/arm_gicv3.c| 14 +++ hw/intc/arm_gicv3_common.c | 1 + hw/intc/arm_gicv3_cpuif.c | 7 +- hw/intc

[PATCH v9 3/9] hw/intc: GICv3 ITS command queue framework

2021-09-10 Thread Shashi Mallela
Added functionality to trigger ITS command queue processing on write to CWRITE register and process each command queue entry to identify the command type and handle commands like MAPD,MAPC,SYNC. Signed-off-by: Shashi Mallela Reviewed-by: Peter Maydell Reviewed-by: Eric Auger Tested-by: Neil

[PATCH v9 4/9] hw/intc: GICv3 ITS Command processing

2021-09-10 Thread Shashi Mallela
of these commands make use of other functionalities implemented to get device table entry,collection table entry or interrupt translation table entry required for their processing. Signed-off-by: Shashi Mallela Reviewed-by: Peter Maydell --- hw/intc/arm_gicv3_its.c| 365

[PATCH v9 5/9] hw/intc: GICv3 ITS Feature enablement

2021-09-10 Thread Shashi Mallela
Added properties to enable ITS feature and define qemu system address space memory in gicv3 common,setup distributor and redistributor registers to indicate LPI support. Signed-off-by: Shashi Mallela Reviewed-by: Peter Maydell Tested-by: Neil Armstrong --- hw/intc/arm_gicv3_common.c

[PATCH v9 2/9] hw/intc: GICv3 ITS register definitions added

2021-09-10 Thread Shashi Mallela
for subsequent ITS processing) and initialize the same in ITS device. Signed-off-by: Shashi Mallela Reviewed-by: Peter Maydell Reviewed-by: Eric Auger Tested-by: Neil Armstrong --- hw/intc/arm_gicv3_its.c| 376 + hw/intc/gicv3_internal.h | 29

[PATCH v9 0/9] GICv3 LPI and ITS feature implementation

2021-09-10 Thread Shashi Mallela
kvm_unit_tests PASS - Verified Linux Boot functionality Shashi Mallela (9): hw/intc: GICv3 ITS initial framework hw/intc: GICv3 ITS register definitions added hw/intc: GICv3 ITS command queue framework hw/intc: GICv3 ITS Command processing hw/intc: GICv3 ITS Feature enablement hw/intc: GICv3

[PATCH v9 1/9] hw/intc: GICv3 ITS initial framework

2021-09-10 Thread Shashi Mallela
Added register definitions relevant to ITS,implemented overall ITS device framework with stubs for ITS control and translater regions read/write,extended ITS common to handle mmio init between existing kvm device and newer qemu device. Signed-off-by: Shashi Mallela Reviewed-by: Peter Maydell

Re: SMMU Stage 2 translation in QEMU

2021-09-10 Thread shashi . mallela
So that would be the driver code running in guest OS because i see tables being setup by arm-smmu driver code in linux,which is similar to what happens with ITS(table base addresses programmed in registers by linux driver). On Fri, 2021-09-10 at 13:54 +0100, Peter Maydell wrote: > On Fri, 10 Sept

Re: SMMU Stage 2 translation in QEMU

2021-09-10 Thread shashi . mallela
I am referring to the latter,"purely emulated QEMU with an emulated SMMU that handles accesses to emulated devices" Thanks Shashi On Fri, 2021-09-10 at 10:25 +0100, Peter Maydell wrote: > On Thu, 9 Sept 2021 at 21:18, wrote: > > I am trying to understand the approach required for an emulated >

SMMU Stage 2 translation in QEMU

2021-09-09 Thread shashi . mallela
Hi All, I am trying to understand the approach required for an emulated SMMU to convert IPAs(from each qemu guest) to PAs(respective host addresses) using stage 2 tables. The questions i have are:- 1) Since SMMU stage 2 tables are expected to be created and managed by a hypervisor,if there is

Re: [PATCH v8 07/10] hw/arm/sbsa-ref: add ITS support in SBSA GIC

2021-08-23 Thread shashi . mallela
On Thu, 2021-08-19 at 14:27 +0100, Peter Maydell wrote: > On Thu, 12 Aug 2021 at 17:53, Shashi Mallela < > shashi.mall...@linaro.org> wrote: > > Included creation of ITS as part of SBSA platform GIC > > initialization. > > > > Signed-off-by: Shashi Mallela

Re: [PATCH v7 09/10] hw/arm/virt: add ITS support in virt GIC

2021-08-12 Thread shashi . mallela
On Fri, 2021-08-06 at 13:09 +0200, Igor Mammedov wrote: > On Thu, 5 Aug 2021 18:30:01 -0400 > Shashi Mallela wrote: > > > Included creation of ITS as part of virt platform GIC > > initialization. This Emulated ITS model now co-exists with kvm > > ITS and is enabled i

[PATCH v8 09/10] hw/arm/virt: add ITS support in virt GIC

2021-08-12 Thread Shashi Mallela
Included creation of ITS as part of virt platform GIC initialization. This Emulated ITS model now co-exists with kvm ITS and is enabled in absence of kvm irq kernel support in a platform. Signed-off-by: Shashi Mallela Reviewed-by: Peter Maydell --- hw/arm/virt.c | 28

[PATCH v8 07/10] hw/arm/sbsa-ref: add ITS support in SBSA GIC

2021-08-12 Thread Shashi Mallela
Included creation of ITS as part of SBSA platform GIC initialization. Signed-off-by: Shashi Mallela --- hw/arm/sbsa-ref.c | 79 --- 1 file changed, 75 insertions(+), 4 deletions(-) diff --git a/hw/arm/sbsa-ref.c b/hw/arm/sbsa-ref.c index c1629df603

[PATCH v8 10/10] tests/data/acpi/virt: Update IORT files for ITS

2021-08-12 Thread Shashi Mallela
0 00 00 00 00 00 00 00 00 00 FF FF 00 00 // 0070: 00 00 00 00 30 00 00 00 00 00 00 00 // ....0... Signed-off-by: Shashi Mallela Acked-by: Igor Mammedov --- tests/data/acpi/virt/IORT | Bin 0 -> 124 bytes tests/data/acpi/virt/IORT.memhp | Bin 0 ->

[PATCH v8 08/10] tests/data/acpi/virt: Add IORT files for ITS

2021-08-12 Thread Shashi Mallela
Added expected IORT files applicable with latest GICv3 ITS changes.Temporarily differences in these files are okay. Signed-off-by: Shashi Mallela Acked-by: Igor Mammedov --- tests/data/acpi/virt/IORT | 0 tests/data/acpi/virt/IORT.memhp | 0 tests/data/acpi/virt

[PATCH v8 04/10] hw/intc: GICv3 ITS Command processing

2021-08-12 Thread Shashi Mallela
of these commands make use of other functionalities implemented to get device table entry,collection table entry or interrupt translation table entry required for their processing. Signed-off-by: Shashi Mallela Reviewed-by: Peter Maydell --- hw/intc/arm_gicv3_its.c| 351

[PATCH v8 06/10] hw/intc: GICv3 redistributor ITS processing

2021-08-12 Thread Shashi Mallela
of ITS INT,CLEAR,DISCARD command and GITS_TRANSLATER processing. Signed-off-by: Shashi Mallela Tested-by: Neil Armstrong --- hw/intc/arm_gicv3.c| 14 +++ hw/intc/arm_gicv3_common.c | 1 + hw/intc/arm_gicv3_cpuif.c | 7 +- hw/intc/arm_gicv3_its.c

[PATCH v8 05/10] hw/intc: GICv3 ITS Feature enablement

2021-08-12 Thread Shashi Mallela
Added properties to enable ITS feature and define qemu system address space memory in gicv3 common,setup distributor and redistributor registers to indicate LPI support. Signed-off-by: Shashi Mallela Reviewed-by: Peter Maydell Tested-by: Neil Armstrong --- hw/intc/arm_gicv3_common.c

[PATCH v8 02/10] hw/intc: GICv3 ITS register definitions added

2021-08-12 Thread Shashi Mallela
for subsequent ITS processing) and initialize the same in ITS device. Signed-off-by: Shashi Mallela Reviewed-by: Peter Maydell Reviewed-by: Eric Auger Tested-by: Neil Armstrong --- hw/intc/arm_gicv3_its.c| 376 + hw/intc/gicv3_internal.h | 29

[PATCH v8 03/10] hw/intc: GICv3 ITS command queue framework

2021-08-12 Thread Shashi Mallela
Added functionality to trigger ITS command queue processing on write to CWRITE register and process each command queue entry to identify the command type and handle commands like MAPD,MAPC,SYNC. Signed-off-by: Shashi Mallela Reviewed-by: Peter Maydell Reviewed-by: Eric Auger Tested-by: Neil

[PATCH v8 01/10] hw/intc: GICv3 ITS initial framework

2021-08-12 Thread Shashi Mallela
Added register definitions relevant to ITS,implemented overall ITS device framework with stubs for ITS control and translater regions read/write,extended ITS common to handle mmio init between existing kvm device and newer qemu device. Signed-off-by: Shashi Mallela Reviewed-by: Peter Maydell

[PATCH v8 00/10] GICv3 LPI and ITS feature implementation

2021-08-12 Thread Shashi Mallela
functionality. Changes in v8: - moved tcg ITS support to virt platform 6.1 options (since it will be supported in 6.2) - modified the ite entry access offset based on latest test results - All kvm_unit_tests PASS - Verified Linux Boot functionality Shashi Mallela (10): hw/intc: GICv3 ITS

Re: [PATCH v7 04/10] hw/intc: GICv3 ITS Command processing

2021-08-06 Thread shashi . mallela
Thanks for sharing the observations Neil. Will take care of it in the next patch-set. On Fri, 2021-08-06 at 10:58 +0200, Neil Armstrong wrote: > Hi, > > On 06/08/2021 00:29, Shashi Mallela wrote: > > Added ITS command queue handling for MAPTI,MAPI commands,handled > > ITS

[PATCH v7 09/10] hw/arm/virt: add ITS support in virt GIC

2021-08-05 Thread Shashi Mallela
Included creation of ITS as part of virt platform GIC initialization. This Emulated ITS model now co-exists with kvm ITS and is enabled in absence of kvm irq kernel support in a platform. Signed-off-by: Shashi Mallela Reviewed-by: Peter Maydell --- hw/arm/virt.c | 28

[PATCH v7 10/10] tests/data/acpi/virt: Update IORT files for ITS

2021-08-05 Thread Shashi Mallela
0 00 00 00 00 00 00 00 00 00 FF FF 00 00 // 0070: 00 00 00 00 30 00 00 00 00 00 00 00 // ....0... Signed-off-by: Shashi Mallela --- tests/data/acpi/virt/IORT | Bin 0 -> 124 bytes tests/data/acpi/virt/IORT.memhp | Bin 0 -> 124 bytes tests/data/a

[PATCH v7 06/10] hw/intc: GICv3 redistributor ITS processing

2021-08-05 Thread Shashi Mallela
of ITS INT,CLEAR,DISCARD command and GITS_TRANSLATER processing. Signed-off-by: Shashi Mallela --- hw/intc/arm_gicv3.c| 14 +++ hw/intc/arm_gicv3_common.c | 1 + hw/intc/arm_gicv3_cpuif.c | 7 +- hw/intc/arm_gicv3_its.c| 23 + hw/intc

[PATCH v7 04/10] hw/intc: GICv3 ITS Command processing

2021-08-05 Thread Shashi Mallela
of these commands make use of other functionalities implemented to get device table entry,collection table entry or interrupt translation table entry required for their processing. Signed-off-by: Shashi Mallela Reviewed-by: Peter Maydell --- hw/intc/arm_gicv3_its.c| 348

[PATCH v7 08/10] tests/data/acpi/virt: Add IORT files for ITS

2021-08-05 Thread Shashi Mallela
Added expected IORT files applicable with latest GICv3 ITS changes.Temporarily differences in these files are okay. Signed-off-by: Shashi Mallela --- tests/data/acpi/virt/IORT | 0 tests/data/acpi/virt/IORT.memhp | 0 tests/data/acpi/virt/IORT.numamem | 0

[PATCH v7 02/10] hw/intc: GICv3 ITS register definitions added

2021-08-05 Thread Shashi Mallela
for subsequent ITS processing) and initialize the same in ITS device. Signed-off-by: Shashi Mallela Reviewed-by: Peter Maydell Reviewed-by: Eric Auger --- hw/intc/arm_gicv3_its.c| 376 + hw/intc/gicv3_internal.h | 29 ++ include/hw/intc

[PATCH v7 03/10] hw/intc: GICv3 ITS command queue framework

2021-08-05 Thread Shashi Mallela
Added functionality to trigger ITS command queue processing on write to CWRITE register and process each command queue entry to identify the command type and handle commands like MAPD,MAPC,SYNC. Signed-off-by: Shashi Mallela Reviewed-by: Peter Maydell Reviewed-by: Eric Auger --- hw/intc

[PATCH v7 05/10] hw/intc: GICv3 ITS Feature enablement

2021-08-05 Thread Shashi Mallela
Added properties to enable ITS feature and define qemu system address space memory in gicv3 common,setup distributor and redistributor registers to indicate LPI support. Signed-off-by: Shashi Mallela Reviewed-by: Peter Maydell --- hw/intc/arm_gicv3_common.c | 12 hw/intc

[PATCH v7 07/10] hw/arm/sbsa-ref: add ITS support in SBSA GIC

2021-08-05 Thread Shashi Mallela
Included creation of ITS as part of SBSA platform GIC initialization. Signed-off-by: Shashi Mallela --- hw/arm/sbsa-ref.c | 79 --- 1 file changed, 75 insertions(+), 4 deletions(-) diff --git a/hw/arm/sbsa-ref.c b/hw/arm/sbsa-ref.c index c1629df603

[PATCH v7 01/10] hw/intc: GICv3 ITS initial framework

2021-08-05 Thread Shashi Mallela
Added register definitions relevant to ITS,implemented overall ITS device framework with stubs for ITS control and translater regions read/write,extended ITS common to handle mmio init between existing kvm device and newer qemu device. Signed-off-by: Shashi Mallela Reviewed-by: Peter Maydell

[PATCH v7 00/10] GICv3 LPI and ITS feature implementation

2021-08-05 Thread Shashi Mallela
-ref versioning to reflect the latest its memory map change and subsequent future use for helper functions - All kvm_unit_tests PASS - Verified Linux Boot functionality Shashi Mallela (10): hw/intc: GICv3 ITS initial framework hw/intc: GICv3 ITS register definitions added hw/intc: GICv3 ITS com

Re: [PATCH v5 02/10] hw/intc: GICv3 ITS register definitions added

2021-08-05 Thread shashi . mallela
On Thu, 2021-07-08 at 19:27 +0200, Eric Auger wrote: > Hi Shashi, > > On 7/6/21 11:29 AM, Eric Auger wrote: > > Hi, > > > > On 6/30/21 5:31 PM, Shashi Mallela wrote: > > > Defined descriptors for ITS device table,collection table and ITS > > > com

Re: [PATCH v4 7/8] hw/arm/sbsa-ref: add ITS support in SBSA GIC

2021-08-05 Thread shashi . mallela
On Thu, 2021-07-08 at 23:05 +0100, Leif Lindholm wrote: > On Thu, Jul 08, 2021 at 21:05:02 +0100, Peter Maydell wrote: > > On Thu, 8 Jul 2021 at 20:40, Leif Lindholm > > wrote: > > > I think my summary-summary would be: > > > - I think we will need to introduce a compatiblity-breaking > > >

[PATCH v6 10/10] tests/data/acpi/virt: Update IORT files for ITS

2021-07-06 Thread Shashi Mallela
0 00 00 00 00 00 00 00 00 00 FF FF 00 00 // 0070: 00 00 00 00 30 00 00 00 00 00 00 00 // ....0... Signed-off-by: Shashi Mallela --- tests/data/acpi/virt/IORT | Bin 0 -> 124 bytes tests/data/acpi/virt/IORT.memhp | Bin 0 -> 124 bytes tests/data/a

[PATCH v6 08/10] tests/data/acpi/virt: Add IORT files for ITS

2021-07-06 Thread Shashi Mallela
Added expected IORT files applicable with latest GICv3 ITS changes.Temporarily differences in these files are okay. Signed-off-by: Shashi Mallela --- tests/data/acpi/virt/IORT | 0 tests/data/acpi/virt/IORT.memhp | 0 tests/data/acpi/virt/IORT.numamem | 0

[PATCH v6 07/10] hw/arm/sbsa-ref: add ITS support in SBSA GIC

2021-07-06 Thread Shashi Mallela
Included creation of ITS as part of SBSA platform GIC initialization. Signed-off-by: Shashi Mallela --- hw/arm/sbsa-ref.c | 26 +++--- 1 file changed, 23 insertions(+), 3 deletions(-) diff --git a/hw/arm/sbsa-ref.c b/hw/arm/sbsa-ref.c index 43c19b4923..3d9c073636 100644

[PATCH v6 06/10] hw/intc: GICv3 redistributor ITS processing

2021-07-06 Thread Shashi Mallela
of ITS INT,CLEAR,DISCARD command and GITS_TRANSLATER processing. Signed-off-by: Shashi Mallela --- hw/intc/arm_gicv3.c| 14 +++ hw/intc/arm_gicv3_common.c | 1 + hw/intc/arm_gicv3_cpuif.c | 7 +- hw/intc/arm_gicv3_its.c| 23 + hw/intc

[PATCH v6 05/10] hw/intc: GICv3 ITS Feature enablement

2021-07-06 Thread Shashi Mallela
Added properties to enable ITS feature and define qemu system address space memory in gicv3 common,setup distributor and redistributor registers to indicate LPI support. Signed-off-by: Shashi Mallela Reviewed-by: Peter Maydell --- hw/intc/arm_gicv3_common.c | 12 hw/intc

[PATCH v6 01/10] hw/intc: GICv3 ITS initial framework

2021-07-06 Thread Shashi Mallela
Added register definitions relevant to ITS,implemented overall ITS device framework with stubs for ITS control and translater regions read/write,extended ITS common to handle mmio init between existing kvm device and newer qemu device. Signed-off-by: Shashi Mallela Reviewed-by: Peter Maydell

[PATCH v6 03/10] hw/intc: GICv3 ITS command queue framework

2021-07-06 Thread Shashi Mallela
Added functionality to trigger ITS command queue processing on write to CWRITE register and process each command queue entry to identify the command type and handle commands like MAPD,MAPC,SYNC. Signed-off-by: Shashi Mallela Reviewed-by: Peter Maydell Reviewed-by: Eric Auger --- hw/intc

[PATCH v6 04/10] hw/intc: GICv3 ITS Command processing

2021-07-06 Thread Shashi Mallela
of these commands make use of other functionalities implemented to get device table entry,collection table entry or interrupt translation table entry required for their processing. Signed-off-by: Shashi Mallela --- hw/intc/arm_gicv3_its.c| 354 - hw/intc/gicv3_internal.h

[PATCH v6 09/10] hw/arm/virt: add ITS support in virt GIC

2021-07-06 Thread Shashi Mallela
Included creation of ITS as part of virt platform GIC initialization. This Emulated ITS model now co-exists with kvm ITS and is enabled in absence of kvm irq kernel support in a platform. Signed-off-by: Shashi Mallela Reviewed-by: Peter Maydell --- hw/arm/virt.c | 28

[PATCH v6 00/10] GICv3 LPI and ITS feature implementation

2021-07-06 Thread Shashi Mallela
and gicv3_internal.h #defines - All kvm_unit_tests PASS Shashi Mallela (10): hw/intc: GICv3 ITS initial framework hw/intc: GICv3 ITS register definitions added hw/intc: GICv3 ITS command queue framework hw/intc: GICv3 ITS Command processing hw/intc: GICv3 ITS Feature enablement hw

[PATCH v6 02/10] hw/intc: GICv3 ITS register definitions added

2021-07-06 Thread Shashi Mallela
for subsequent ITS processing) and initialize the same in ITS device. Signed-off-by: Shashi Mallela Reviewed-by: Peter Maydell Reviewed-by: Eric Auger --- hw/intc/arm_gicv3_its.c| 376 + hw/intc/gicv3_internal.h | 29 ++ include/hw/intc

Re: [PATCH v5 01/10] hw/intc: GICv3 ITS initial framework

2021-07-06 Thread shashi . mallela
> > > > On Wed, 30 Jun 2021 at 16:32, Shashi Mallela < > > > > > shashi.mall...@linaro.org> wrote: > > > > > > Added register definitions relevant to ITS,implemented > > > > > > overall > > > > > > ITS device fram

Re: [PATCH v5 04/10] hw/intc: GICv3 ITS Command processing

2021-07-06 Thread shashi . mallela
On Tue, 2021-07-06 at 14:27 +0100, Peter Maydell wrote: > On Tue, 6 Jul 2021 at 13:46, wrote: > > On Tue, 2021-07-06 at 10:19 +0100, Peter Maydell wrote: > > > On Tue, 6 Jul 2021 at 04:25, wrote: > > > > > > But the pseudocode for MAPTI does not say anywhere that we should > > > be checking the

Re: [PATCH v5 01/10] hw/intc: GICv3 ITS initial framework

2021-07-06 Thread shashi . mallela
On Tue, 2021-07-06 at 09:44 +0200, Eric Auger wrote: > Hi, > > On 6/30/21 5:31 PM, Shashi Mallela wrote: > > Added register definitions relevant to ITS,implemented overall > > ITS device framework with stubs for ITS control and translater > > regions read/write,extende

Re: [PATCH v5 04/10] hw/intc: GICv3 ITS Command processing

2021-07-06 Thread shashi . mallela
On Tue, 2021-07-06 at 11:27 +0200, Eric Auger wrote: > Hi, > > On 7/5/21 4:07 PM, Peter Maydell wrote: > > On Wed, 30 Jun 2021 at 16:32, Shashi Mallela < > > shashi.mall...@linaro.org> wrote: > > > Added ITS command queue handling for MAPTI,MAPI commands,handl

Re: [PATCH v4 1/8] hw/intc: GICv3 ITS initial framework

2021-07-06 Thread shashi . mallela
; > Hi, > > > > > > On 6/11/21 6:21 PM, Eric Auger wrote: > > > > Hi, > > > > > > > > On 6/2/21 8:00 PM, Shashi Mallela wrote: > > > > > Added register definitions relevant to ITS,implemented > > > > > overall

Re: [PATCH v4 1/8] hw/intc: GICv3 ITS initial framework

2021-07-06 Thread shashi . mallela
Hi Eric, Please find my response inline(below):- On Tue, 2021-07-06 at 09:38 +0200, Eric Auger wrote: > Hi, > > On 6/11/21 6:21 PM, Eric Auger wrote: > > Hi, > > > > On 6/2/21 8:00 PM, Shashi Mallela wrote: > > > Added register definitions relevant to ITS,

Re: [PATCH v5 04/10] hw/intc: GICv3 ITS Command processing

2021-07-06 Thread shashi . mallela
On Tue, 2021-07-06 at 10:19 +0100, Peter Maydell wrote: > On Tue, 6 Jul 2021 at 04:25, wrote: > > On Mon, 2021-07-05 at 20:47 -0400, shashi.mall...@linaro.org wrote: > > > On Mon, 2021-07-05 at 15:54 +0100, Peter Maydell wrote: > > > > I missed this the first time around, but I don't think this

Re: [PATCH v5 04/10] hw/intc: GICv3 ITS Command processing

2021-07-05 Thread shashi . mallela
On Mon, 2021-07-05 at 20:47 -0400, shashi.mall...@linaro.org wrote: > On Mon, 2021-07-05 at 15:54 +0100, Peter Maydell wrote: > > On Wed, 30 Jun 2021 at 16:32, Shashi Mallela < > > shashi.mall...@linaro.org> wrote: > > > Added ITS command queue handling for MAPTI,M

Re: [PATCH v5 04/10] hw/intc: GICv3 ITS Command processing

2021-07-05 Thread shashi . mallela
On Mon, 2021-07-05 at 15:54 +0100, Peter Maydell wrote: > On Wed, 30 Jun 2021 at 16:32, Shashi Mallela < > shashi.mall...@linaro.org> wrote: > > Added ITS command queue handling for MAPTI,MAPI commands,handled > > ITS > > translation which triggers an LPI vi

Re: [PATCH v5 01/10] hw/intc: GICv3 ITS initial framework

2021-07-05 Thread shashi . mallela
On Mon, 2021-07-05 at 17:25 +0100, Peter Maydell wrote: > On Mon, 5 Jul 2021 at 16:55, wrote: > > On Mon, 2021-07-05 at 15:58 +0100, Peter Maydell wrote: > > > On Wed, 30 Jun 2021 at 16:32, Shashi Mallela < > > > shashi.mall...@linaro.org> wrote: > > > &g

Re: [PATCH v5 01/10] hw/intc: GICv3 ITS initial framework

2021-07-05 Thread shashi . mallela
On Mon, 2021-07-05 at 15:58 +0100, Peter Maydell wrote: > On Wed, 30 Jun 2021 at 16:32, Shashi Mallela < > shashi.mall...@linaro.org> wrote: > > Added register definitions relevant to ITS,implemented overall > > ITS device framework with stubs for ITS control and translate

[PATCH v5 10/10] tests/data/acpi/virt: Update IORT files for ITS

2021-06-30 Thread Shashi Mallela
Updated expected IORT files applicable with latest GICv3 ITS changes. Signed-off-by: Shashi Mallela --- tests/data/acpi/virt/IORT | Bin 0 -> 124 bytes tests/data/acpi/virt/IORT.memhp | Bin 0 -> 124 bytes tests/data/acpi/virt/IORT.numamem | Bin 0

[PATCH v5 09/10] hw/arm/virt: add ITS support in virt GIC

2021-06-30 Thread Shashi Mallela
Included creation of ITS as part of virt platform GIC initialization. This Emulated ITS model now co-exists with kvm ITS and is enabled in absence of kvm irq kernel support in a platform. Signed-off-by: Shashi Mallela Reviewed-by: Peter Maydell --- hw/arm/virt.c | 28

[PATCH v5 08/10] tests/data/acpi/virt: Add IORT files for ITS

2021-06-30 Thread Shashi Mallela
Added expected IORT files applicable with latest GICv3 ITS changes.Temporarily differences in these files are okay. Signed-off-by: Shashi Mallela --- tests/data/acpi/virt/IORT | 0 tests/data/acpi/virt/IORT.memhp | 0 tests/data/acpi/virt/IORT.numamem | 0

[PATCH v5 07/10] hw/arm/sbsa-ref: add ITS support in SBSA GIC

2021-06-30 Thread Shashi Mallela
Included creation of ITS as part of SBSA platform GIC initialization. Signed-off-by: Shashi Mallela --- hw/arm/sbsa-ref.c | 26 +++--- 1 file changed, 23 insertions(+), 3 deletions(-) diff --git a/hw/arm/sbsa-ref.c b/hw/arm/sbsa-ref.c index 43c19b4923..3d9c073636 100644

[PATCH v5 05/10] hw/intc: GICv3 ITS Feature enablement

2021-06-30 Thread Shashi Mallela
Added properties to enable ITS feature and define qemu system address space memory in gicv3 common,setup distributor and redistributor registers to indicate LPI support. Signed-off-by: Shashi Mallela --- hw/intc/arm_gicv3_common.c | 12 hw/intc/arm_gicv3_dist.c

[PATCH v5 06/10] hw/intc: GICv3 redistributor ITS processing

2021-06-30 Thread Shashi Mallela
of ITS INT,CLEAR,DISCARD command and GITS_TRANSLATER processing. Signed-off-by: Shashi Mallela --- hw/intc/arm_gicv3.c| 14 +++ hw/intc/arm_gicv3_common.c | 1 + hw/intc/arm_gicv3_cpuif.c | 7 +- hw/intc/arm_gicv3_its.c| 24 - hw/intc

[PATCH v5 04/10] hw/intc: GICv3 ITS Command processing

2021-06-30 Thread Shashi Mallela
of these commands make use of other functionalities implemented to get device table entry,collection table entry or interrupt translation table entry required for their processing. Signed-off-by: Shashi Mallela --- hw/intc/arm_gicv3_its.c| 361 - hw/intc/gicv3_internal.h

[PATCH v5 02/10] hw/intc: GICv3 ITS register definitions added

2021-06-30 Thread Shashi Mallela
for subsequent ITS processing) and initialize the same in ITS device. Signed-off-by: Shashi Mallela Reviewed-by: Peter Maydell --- hw/intc/arm_gicv3_its.c| 376 + hw/intc/gicv3_internal.h | 31 +- include/hw/intc/arm_gicv3_common.h | 3

[PATCH v5 01/10] hw/intc: GICv3 ITS initial framework

2021-06-30 Thread Shashi Mallela
Added register definitions relevant to ITS,implemented overall ITS device framework with stubs for ITS control and translater regions read/write,extended ITS common to handle mmio init between existing kvm device and newer qemu device. Signed-off-by: Shashi Mallela Reviewed-by: Peter Maydell

[PATCH v5 00/10] GICv3 LPI and ITS feature implementation

2021-06-30 Thread Shashi Mallela
functionality. Changes in v5: - v4 review comments addressed - All kvm_unit_tests PASS Shashi Mallela (10): hw/intc: GICv3 ITS initial framework hw/intc: GICv3 ITS register definitions added hw/intc: GICv3 ITS command queue framework hw/intc: GICv3 ITS Command processing hw/intc: GICv3 ITS

[PATCH v5 03/10] hw/intc: GICv3 ITS command queue framework

2021-06-30 Thread Shashi Mallela
Added functionality to trigger ITS command queue processing on write to CWRITE register and process each command queue entry to identify the command type and handle commands like MAPD,MAPC,SYNC. Signed-off-by: Shashi Mallela Reviewed-by: Peter Maydell --- hw/intc/arm_gicv3_its.c | 305

Re: [PATCH v4 4/8] hw/intc: GICv3 ITS Command processing

2021-06-28 Thread shashi . mallela
(below):- > > > > On Sun, 2021-06-13 at 17:55 +0200, Eric Auger wrote: > > > Hi Shashi, > > > > > > On 6/2/21 8:00 PM, Shashi Mallela wrote: > > > > Added ITS command queue handling for MAPTI,MAPI > > > > commands,h

Re: [PATCH v4 3/8] hw/intc: GICv3 ITS command queue framework

2021-06-28 Thread shashi . mallela
gt; > On Sun, 2021-06-13 at 16:13 +0200, Eric Auger wrote: > > > Hi Sashi, > > > > > > On 6/2/21 8:00 PM, Shashi Mallela wrote: > > > > Added functionality to trigger ITS command queue processing on > > > > write to CWRITE register and process ea

Re: [PATCH v4 2/8] hw/intc: GICv3 ITS register definitions added

2021-06-28 Thread shashi . mallela
> > > > On Sat, 2021-06-12 at 08:08 +0200, Eric Auger wrote: > > > On 6/2/21 8:00 PM, Shashi Mallela wrote: > > > > Defined descriptors for ITS device table,collection table and > > > > ITS > > > > command queue entities.Implemented register

Re: [PATCH v4 3/8] hw/intc: GICv3 ITS command queue framework

2021-06-28 Thread shashi . mallela
Hi Eric, Had missed this comment earlier.Please find my response (inline)below:- On Sun, 2021-06-13 at 16:39 +0200, Eric Auger wrote: > Hi, > > On 6/2/21 8:00 PM, Shashi Mallela wrote: > > Added functionality to trigger ITS command queue processing on > > write to CWRITE

Re: GICv3 for MTTCG

2021-06-17 Thread shashi . mallela
-xhci,p2=8,p3=8,id=usb,bus=pci.2,addr=0x0 -device > virtio-scsi-pci,id=scsi0,bus=pci.3,addr=0x0 -drive > file=/var/lib/libvirt/images/EulerOS-2.8- > Rich.qcow2,format=qcow2,if=none,id=drive-scsi0-0-0-0 > -device > scsi-hd,bus=scsi0.0,channel=0,scsi-id=0,lun=0,drive=drive-scsi0-

Re: [PATCH v4 6/8] hw/intc: GICv3 redistributor ITS processing

2021-06-16 Thread shashi . mallela
Hi Eric, Have accepted all comments with responses inline (below):- On Sun, 2021-06-13 at 18:26 +0200, Eric Auger wrote: > Hi Shashi, > > On 6/2/21 8:00 PM, Shashi Mallela wrote: > > Implemented lpi processing at redistributor to get lpi config info > s/Implemented/Implement

Re: [PATCH v4 3/8] hw/intc: GICv3 ITS command queue framework

2021-06-16 Thread shashi . mallela
Hi Eric, Please find my responses inline (below):- On Sun, 2021-06-13 at 16:13 +0200, Eric Auger wrote: > Hi Sashi, > > On 6/2/21 8:00 PM, Shashi Mallela wrote: > > Added functionality to trigger ITS command queue processing on > > write to CWRITE register and process eac

Re: [PATCH v4 4/8] hw/intc: GICv3 ITS Command processing

2021-06-16 Thread shashi . mallela
Hi Eric, Please find my responses inline (below):- On Sun, 2021-06-13 at 17:55 +0200, Eric Auger wrote: > Hi Shashi, > > On 6/2/21 8:00 PM, Shashi Mallela wrote: > > Added ITS command queue handling for MAPTI,MAPI commands,handled > > ITS > > translation which trig

Re: [PATCH v4 2/8] hw/intc: GICv3 ITS register definitions added

2021-06-16 Thread shashi . mallela
Hi Eric, Please find my responses inline (below):- On Sat, 2021-06-12 at 08:08 +0200, Eric Auger wrote: > > On 6/2/21 8:00 PM, Shashi Mallela wrote: > > Defined descriptors for ITS device table,collection table and ITS > > command queue entities.Implemented register re

Re: [PATCH v4 6/8] hw/intc: GICv3 redistributor ITS processing

2021-06-14 Thread Shashi Mallela
On Jun 11 2021, at 4:30 am, Peter Maydell wrote: > On Fri, 11 Jun 2021 at 00:39, Shashi Mallela > wrote: > > > > Have addressed all comments except the ones with responses(inline) below:- > > > > On Jun 8 2021, at 9:57 am, Peter Maydell wrote: > > >

Re: [PATCH v4 1/8] hw/intc: GICv3 ITS initial framework

2021-06-11 Thread Shashi Mallela
On Jun 11 2021, at 12:21 pm, Eric Auger wrote: > Hi, > > On 6/2/21 8:00 PM, Shashi Mallela wrote: > > Added register definitions relevant to ITS,implemented overall > > ITS device framework with stubs for ITS control and translater > > regions read/write,extended ITS

Re: [PATCH v4 6/8] hw/intc: GICv3 redistributor ITS processing

2021-06-10 Thread Shashi Mallela
Have addressed all comments except the ones with responses(inline) below:- On Jun 8 2021, at 9:57 am, Peter Maydell wrote: > On Wed, 2 Jun 2021 at 19:00, Shashi Mallela wrote: > > > > Implemented lpi processing at redistributor to get lpi config info > > from lpi configur

Re: [PATCH v4 7/8] hw/arm/sbsa-ref: add ITS support in SBSA GIC

2021-06-04 Thread shashi . mallela
On Fri, 2021-06-04 at 11:42 +0100, Leif Lindholm wrote: > On Thu, Jun 03, 2021 at 11:31:21 -0400, shashi.mall...@linaro.org > wrote: > > On Thu, 2021-06-03 at 12:42 +0100, Leif Lindholm wrote: > > > On Wed, Jun 02, 2021 at 14:00:41 -0400, Shashi Mallela wrote: > > &

Re: [PATCH v4 7/8] hw/arm/sbsa-ref: add ITS support in SBSA GIC

2021-06-03 Thread shashi . mallela
On Thu, 2021-06-03 at 12:42 +0100, Leif Lindholm wrote: > On Wed, Jun 02, 2021 at 14:00:41 -0400, Shashi Mallela wrote: > > Included creation of ITS as part of SBSA platform GIC > > initialization. > > > > Signed-off-by: Shashi Mallela > >

Re: [PATCH] hw/intc/arm_gicv3_cpuif: Tolerate spurious EOIR writes

2021-06-03 Thread Shashi Mallela
Yes it does. Thanks Shashi On Jun 3 2021, at 8:56 am, Peter Maydell wrote: > On Thu, 3 Jun 2021 at 12:01, Jean-Philippe Brucker > wrote: > > > > Commit 382c7160d1cd ("hw/intc/arm_gicv3_cpuif: Fix EOIR write access > > check logic") added an assert_not_reached() if the guest writes the EOIR > >

[PATCH v1 1/1] tests/data/acpi/virt: add IORT files for ITS

2021-06-03 Thread Shashi Mallela
Added expected IORT files applicable with latest GICv3 ITS changes. Signed-off-by: Shashi Mallela --- tests/data/acpi/virt/IORT | Bin 0 -> 124 bytes tests/data/acpi/virt/IORT.memhp | Bin 0 -> 124 bytes tests/data/acpi/virt/IORT.numamem | Bin 0 -> 124 bytes tests/data/

[PATCH v4 8/8] hw/arm/virt: add ITS support in virt GIC

2021-06-02 Thread Shashi Mallela
Included creation of ITS as part of virt platform GIC initialization.This Emulated ITS model now co-exists with kvm ITS and is enabled in absence of kvm irq kernel support in a platform. Signed-off-by: Shashi Mallela --- hw/arm/virt.c | 27 +-- include/hw/arm

[PATCH v4 7/8] hw/arm/sbsa-ref: add ITS support in SBSA GIC

2021-06-02 Thread Shashi Mallela
Included creation of ITS as part of SBSA platform GIC initialization. Signed-off-by: Shashi Mallela --- hw/arm/sbsa-ref.c | 26 +++--- 1 file changed, 23 insertions(+), 3 deletions(-) diff --git a/hw/arm/sbsa-ref.c b/hw/arm/sbsa-ref.c index 43c19b4923..3d9c073636 100644

[PATCH v4 6/8] hw/intc: GICv3 redistributor ITS processing

2021-06-02 Thread Shashi Mallela
of ITS INT,CLEAR,DISCARD command and GITS_TRANSLATER processing. Signed-off-by: Shashi Mallela --- hw/intc/arm_gicv3.c| 9 ++ hw/intc/arm_gicv3_common.c | 1 + hw/intc/arm_gicv3_cpuif.c | 7 +- hw/intc/arm_gicv3_its.c| 14 ++- hw/intc

[PATCH v4 5/8] hw/intc: GICv3 ITS Feature enablement

2021-06-02 Thread Shashi Mallela
Added properties to enable ITS feature and define qemu system address space memory in gicv3 common,setup distributor and redistributor registers to indicate LPI support. Signed-off-by: Shashi Mallela --- hw/intc/arm_gicv3_common.c | 12 hw/intc/arm_gicv3_dist.c

[PATCH v4 4/8] hw/intc: GICv3 ITS Command processing

2021-06-02 Thread Shashi Mallela
of these commands make use of other functionalities implemented to get device table entry,collection table entry or interrupt translation table entry required for their processing. Signed-off-by: Shashi Mallela --- hw/intc/arm_gicv3_its.c| 334 + hw/intc/gicv3_internal.h

[PATCH v4 3/8] hw/intc: GICv3 ITS command queue framework

2021-06-02 Thread Shashi Mallela
Added functionality to trigger ITS command queue processing on write to CWRITE register and process each command queue entry to identify the command type and handle commands like MAPD,MAPC,SYNC. Signed-off-by: Shashi Mallela --- hw/intc/arm_gicv3_its.c | 295

  1   2   >