Re: [Xen-devel] [PATCH V3] xen: arm: X-Gene Storm check GIC DIST address for EOI quirk
Hi Julien, On 7 May 2015 at 17:20, Julien Grall wrote: > Hi Pranav, > > On 07/05/15 12:12, Pranavkumar Sawargaonkar wrote: >> On 7 May 2015 at 15:11, Ian Campbell wrote: >>> On Thu, 2015-05-07 at 14:19 +0530, Pranavkumar Sawargaonkar wrote: Hi Julien, On 5 May 2015 at 19:22, Julien Grall wrote: > Hi Pranav, > > On 29/04/15 10:38, Pranavkumar Sawargaonkar wrote: >> In old X-Gene Storm firmware and DT, secure mode addresses have been >> mentioned in GICv2 node. In this case maintenance interrupt is used >> instead of EOI HW method. >> >> This patch checks the GIC Distributor Base Address to enable EOI quirk >> for old firmware. >> >> Ref: >> http://lists.xen.org/archives/html/xen-devel/2014-07/msg01263.html > > Do you know if the new firmware is publicly available? No there is no direct link available to download this. One has to got through apm support to get this. >>> >>> Is it available on request to anybody who has a Mustang system? Or only >>> to e.g. partners? >>> >> >> As far as my knowledge it is available to the people having mustang >> (and also to partners) on a request. > > Thanks. I will send a patch to drop the EOI quirk and panic if the user > is using an old firmware. Sure, this is a good idea. - Pranav > > Regards, > > -- > Julien Grall ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel
Re: [Xen-devel] [PATCH V3] xen: arm: X-Gene Storm check GIC DIST address for EOI quirk
Hi Pranav, On 07/05/15 12:12, Pranavkumar Sawargaonkar wrote: > On 7 May 2015 at 15:11, Ian Campbell wrote: >> On Thu, 2015-05-07 at 14:19 +0530, Pranavkumar Sawargaonkar wrote: >>> Hi Julien, >>> >>> On 5 May 2015 at 19:22, Julien Grall wrote: Hi Pranav, On 29/04/15 10:38, Pranavkumar Sawargaonkar wrote: > In old X-Gene Storm firmware and DT, secure mode addresses have been > mentioned in GICv2 node. In this case maintenance interrupt is used > instead of EOI HW method. > > This patch checks the GIC Distributor Base Address to enable EOI quirk > for old firmware. > > Ref: > http://lists.xen.org/archives/html/xen-devel/2014-07/msg01263.html Do you know if the new firmware is publicly available? >>> >>> No there is no direct link available to download this. One has to got >>> through apm support to get this. >> >> Is it available on request to anybody who has a Mustang system? Or only >> to e.g. partners? >> > > As far as my knowledge it is available to the people having mustang > (and also to partners) on a request. Thanks. I will send a patch to drop the EOI quirk and panic if the user is using an old firmware. Regards, -- Julien Grall ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel
Re: [Xen-devel] [PATCH V3] xen: arm: X-Gene Storm check GIC DIST address for EOI quirk
Hi Ian, On 7 May 2015 at 15:11, Ian Campbell wrote: > On Thu, 2015-05-07 at 14:19 +0530, Pranavkumar Sawargaonkar wrote: >> Hi Julien, >> >> On 5 May 2015 at 19:22, Julien Grall wrote: >> > Hi Pranav, >> > >> > On 29/04/15 10:38, Pranavkumar Sawargaonkar wrote: >> >> In old X-Gene Storm firmware and DT, secure mode addresses have been >> >> mentioned in GICv2 node. In this case maintenance interrupt is used >> >> instead of EOI HW method. >> >> >> >> This patch checks the GIC Distributor Base Address to enable EOI quirk >> >> for old firmware. >> >> >> >> Ref: >> >> http://lists.xen.org/archives/html/xen-devel/2014-07/msg01263.html >> > >> > Do you know if the new firmware is publicly available? >> >> No there is no direct link available to download this. One has to got >> through apm support to get this. > > Is it available on request to anybody who has a Mustang system? Or only > to e.g. partners? > As far as my knowledge it is available to the people having mustang (and also to partners) on a request. - Pranav > It would be a shame if not because while the quirk is buggy in various > ways it is better than nothing, but we would all much prefer to ditch it > if possible. > > Ian. > ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel
Re: [Xen-devel] [PATCH V3] xen: arm: X-Gene Storm check GIC DIST address for EOI quirk
On Thu, 2015-05-07 at 14:19 +0530, Pranavkumar Sawargaonkar wrote: > Hi Julien, > > On 5 May 2015 at 19:22, Julien Grall wrote: > > Hi Pranav, > > > > On 29/04/15 10:38, Pranavkumar Sawargaonkar wrote: > >> In old X-Gene Storm firmware and DT, secure mode addresses have been > >> mentioned in GICv2 node. In this case maintenance interrupt is used > >> instead of EOI HW method. > >> > >> This patch checks the GIC Distributor Base Address to enable EOI quirk > >> for old firmware. > >> > >> Ref: > >> http://lists.xen.org/archives/html/xen-devel/2014-07/msg01263.html > > > > Do you know if the new firmware is publicly available? > > No there is no direct link available to download this. One has to got > through apm support to get this. Is it available on request to anybody who has a Mustang system? Or only to e.g. partners? It would be a shame if not because while the quirk is buggy in various ways it is better than nothing, but we would all much prefer to ditch it if possible. Ian. ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel
Re: [Xen-devel] [PATCH V3] xen: arm: X-Gene Storm check GIC DIST address for EOI quirk
Hi Julien, On 5 May 2015 at 19:22, Julien Grall wrote: > Hi Pranav, > > On 29/04/15 10:38, Pranavkumar Sawargaonkar wrote: >> In old X-Gene Storm firmware and DT, secure mode addresses have been >> mentioned in GICv2 node. In this case maintenance interrupt is used >> instead of EOI HW method. >> >> This patch checks the GIC Distributor Base Address to enable EOI quirk >> for old firmware. >> >> Ref: >> http://lists.xen.org/archives/html/xen-devel/2014-07/msg01263.html > > Do you know if the new firmware is publicly available? No there is no direct link available to download this. One has to got through apm support to get this. Thanks, Pranav > > Regards, > > -- > Julien Grall ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel
Re: [Xen-devel] [PATCH V3] xen: arm: X-Gene Storm check GIC DIST address for EOI quirk
Hi Pranav, On 29/04/15 10:38, Pranavkumar Sawargaonkar wrote: > In old X-Gene Storm firmware and DT, secure mode addresses have been > mentioned in GICv2 node. In this case maintenance interrupt is used > instead of EOI HW method. > > This patch checks the GIC Distributor Base Address to enable EOI quirk > for old firmware. > > Ref: > http://lists.xen.org/archives/html/xen-devel/2014-07/msg01263.html Do you know if the new firmware is publicly available? Regards, -- Julien Grall ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel
Re: [Xen-devel] [PATCH V3] xen: arm: X-Gene Storm check GIC DIST address for EOI quirk
On Wed, 2015-04-29 at 15:08 +0530, Pranavkumar Sawargaonkar wrote: > In old X-Gene Storm firmware and DT, secure mode addresses have been > mentioned in GICv2 node. In this case maintenance interrupt is used > instead of EOI HW method. > > This patch checks the GIC Distributor Base Address to enable EOI quirk > for old firmware. > > Ref: > http://lists.xen.org/archives/html/xen-devel/2014-07/msg01263.html > > Signed-off-by: Pranavkumar Sawargaonkar > Tested-by: Christoffer Dall Acked + applied, thanks. ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel
Re: [Xen-devel] [PATCH V3] xen: arm: X-Gene Storm check GIC DIST address for EOI quirk
On Wed, 29 Apr 2015, Pranavkumar Sawargaonkar wrote:
> In old X-Gene Storm firmware and DT, secure mode addresses have been
> mentioned in GICv2 node. In this case maintenance interrupt is used
> instead of EOI HW method.
>
> This patch checks the GIC Distributor Base Address to enable EOI quirk
> for old firmware.
>
> Ref:
> http://lists.xen.org/archives/html/xen-devel/2014-07/msg01263.html
>
> Signed-off-by: Pranavkumar Sawargaonkar
> Tested-by: Christoffer Dall
Acked-by: Stefano Stabellini
> ChangeLog:
>
> V3:
> - Indentation fixes and other misc comments on V2.
> V2:
> - Fine tune interrupt controller node search as per comments on V1 patch
> - Incorporating other misc comments on V1.
> V1:
> - Initial patch.
> ---
> xen/arch/arm/platforms/xgene-storm.c | 40
> +-
> 1 file changed, 39 insertions(+), 1 deletion(-)
>
> diff --git a/xen/arch/arm/platforms/xgene-storm.c
> b/xen/arch/arm/platforms/xgene-storm.c
> index 1812e5b..c717360 100644
> --- a/xen/arch/arm/platforms/xgene-storm.c
> +++ b/xen/arch/arm/platforms/xgene-storm.c
> @@ -22,6 +22,7 @@
> #include
> #include
> #include
> +#include
> #include
> #include
>
> @@ -35,9 +36,44 @@ static u64 reset_addr, reset_size;
> static u32 reset_mask;
> static bool reset_vals_valid = false;
>
> +#define XGENE_SEC_GICV2_DIST_ADDR0x7801
> +static u32 __read_mostly xgene_quirks = PLATFORM_QUIRK_GIC_64K_STRIDE;
> +
> +static void __init xgene_check_pirq_eoi(void)
> +{
> +const struct dt_device_node *node;
> +int res;
> +paddr_t dbase;
> +const struct dt_device_match xgene_dt_int_ctrl_match[] =
> +{
> +DT_MATCH_COMPATIBLE("arm,cortex-a15-gic"),
> +{ /*sentinel*/ },
> +};
> +
> +node = dt_find_interrupt_controller(xgene_dt_int_ctrl_match);
> +if ( !node )
> +panic("%s: Can not find interrupt controller node", __func__);
> +
> +res = dt_device_get_address(node, 0, &dbase, NULL);
> +if ( !dbase )
> +panic("%s: Cannot find a valid address for the distributor",
> __func__);
> +
> +/*
> + * In old X-Gene Storm firmware and DT, secure mode addresses have
> + * been mentioned in GICv2 node. We have to use maintenance interrupt
> + * instead of EOI HW in this case. We check the GIC Distributor Base
> + * Address to maintain compatibility with older firmware.
> + */
> +if ( dbase == XGENE_SEC_GICV2_DIST_ADDR )
> +{
> +xgene_quirks |= PLATFORM_QUIRK_GUEST_PIRQ_NEED_EOI;
> +printk("Xen: WARNING: OLD X-Gene Firmware, disabling PIRQ EOI
> mode\n");
> +}
> +}
> +
> static uint32_t xgene_storm_quirks(void)
> {
> -return PLATFORM_QUIRK_GIC_64K_STRIDE|PLATFORM_QUIRK_GUEST_PIRQ_NEED_EOI;
> +return xgene_quirks;
> }
>
> static int map_one_mmio(struct domain *d, const char *what,
> @@ -216,6 +252,8 @@ static int xgene_storm_init(void)
> reset_mask = XGENE_RESET_MASK;
>
> reset_vals_valid = true;
> +xgene_check_pirq_eoi();
> +
> return 0;
> }
>
> --
> 1.7.9.5
>
___
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel
Re: [Xen-devel] [PATCH V3] xen: arm: X-Gene Storm check GIC DIST address for EOI quirk
Hi Pranav, On 29/04/15 10:38, Pranavkumar Sawargaonkar wrote: > In old X-Gene Storm firmware and DT, secure mode addresses have been > mentioned in GICv2 node. In this case maintenance interrupt is used > instead of EOI HW method. > > This patch checks the GIC Distributor Base Address to enable EOI quirk > for old firmware. > > Ref: > http://lists.xen.org/archives/html/xen-devel/2014-07/msg01263.html > > Signed-off-by: Pranavkumar Sawargaonkar > Tested-by: Christoffer Dall Reviewed-by: Julien Grall Regards, -- Julien Grall ___ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel
[Xen-devel] [PATCH V3] xen: arm: X-Gene Storm check GIC DIST address for EOI quirk
In old X-Gene Storm firmware and DT, secure mode addresses have been
mentioned in GICv2 node. In this case maintenance interrupt is used
instead of EOI HW method.
This patch checks the GIC Distributor Base Address to enable EOI quirk
for old firmware.
Ref:
http://lists.xen.org/archives/html/xen-devel/2014-07/msg01263.html
Signed-off-by: Pranavkumar Sawargaonkar
Tested-by: Christoffer Dall
---
ChangeLog:
V3:
- Indentation fixes and other misc comments on V2.
V2:
- Fine tune interrupt controller node search as per comments on V1 patch
- Incorporating other misc comments on V1.
V1:
- Initial patch.
---
xen/arch/arm/platforms/xgene-storm.c | 40 +-
1 file changed, 39 insertions(+), 1 deletion(-)
diff --git a/xen/arch/arm/platforms/xgene-storm.c
b/xen/arch/arm/platforms/xgene-storm.c
index 1812e5b..c717360 100644
--- a/xen/arch/arm/platforms/xgene-storm.c
+++ b/xen/arch/arm/platforms/xgene-storm.c
@@ -22,6 +22,7 @@
#include
#include
#include
+#include
#include
#include
@@ -35,9 +36,44 @@ static u64 reset_addr, reset_size;
static u32 reset_mask;
static bool reset_vals_valid = false;
+#define XGENE_SEC_GICV2_DIST_ADDR0x7801
+static u32 __read_mostly xgene_quirks = PLATFORM_QUIRK_GIC_64K_STRIDE;
+
+static void __init xgene_check_pirq_eoi(void)
+{
+const struct dt_device_node *node;
+int res;
+paddr_t dbase;
+const struct dt_device_match xgene_dt_int_ctrl_match[] =
+{
+DT_MATCH_COMPATIBLE("arm,cortex-a15-gic"),
+{ /*sentinel*/ },
+};
+
+node = dt_find_interrupt_controller(xgene_dt_int_ctrl_match);
+if ( !node )
+panic("%s: Can not find interrupt controller node", __func__);
+
+res = dt_device_get_address(node, 0, &dbase, NULL);
+if ( !dbase )
+panic("%s: Cannot find a valid address for the distributor", __func__);
+
+/*
+ * In old X-Gene Storm firmware and DT, secure mode addresses have
+ * been mentioned in GICv2 node. We have to use maintenance interrupt
+ * instead of EOI HW in this case. We check the GIC Distributor Base
+ * Address to maintain compatibility with older firmware.
+ */
+if ( dbase == XGENE_SEC_GICV2_DIST_ADDR )
+{
+xgene_quirks |= PLATFORM_QUIRK_GUEST_PIRQ_NEED_EOI;
+printk("Xen: WARNING: OLD X-Gene Firmware, disabling PIRQ EOI mode\n");
+}
+}
+
static uint32_t xgene_storm_quirks(void)
{
-return PLATFORM_QUIRK_GIC_64K_STRIDE|PLATFORM_QUIRK_GUEST_PIRQ_NEED_EOI;
+return xgene_quirks;
}
static int map_one_mmio(struct domain *d, const char *what,
@@ -216,6 +252,8 @@ static int xgene_storm_init(void)
reset_mask = XGENE_RESET_MASK;
reset_vals_valid = true;
+xgene_check_pirq_eoi();
+
return 0;
}
--
1.7.9.5
___
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel
