* [PATCH] ARM: tegra: select required CPU and L2 errata options
@ 2012-01-04 23:29 Olof Johansson
2012-01-05 16:30 ` Marc Dietrich
0 siblings, 1 reply; 9+ messages in thread
From: Olof Johansson @ 2012-01-04 23:29 UTC (permalink / raw)
To: linux-arm-kernel
Only one of the L2 erratas are needed on T30, but T20/25
needs a few more.
Signed-off-by: Olof Johansson <olof@lixom.net>
---
arch/arm/mach-tegra/Kconfig | 4 ++++
1 files changed, 4 insertions(+), 0 deletions(-)
diff --git a/arch/arm/mach-tegra/Kconfig b/arch/arm/mach-tegra/Kconfig
index 373652d..a2c76ee 100644
--- a/arch/arm/mach-tegra/Kconfig
+++ b/arch/arm/mach-tegra/Kconfig
@@ -10,6 +10,9 @@ config ARCH_TEGRA_2x_SOC
select USB_ARCH_HAS_EHCI if USB_SUPPORT
select USB_ULPI if USB_SUPPORT
select USB_ULPI_VIEWPORT if USB_SUPPORT
+ select ARM_ERRATA_742230
+ select PL310_ERRATA_753970 if CACHE_PL310
+ select PL310_ERRATA_769419 if CACHE_L2X0
help
Support for NVIDIA Tegra AP20 and T20 processors, based on the
ARM CortexA9MP CPU and the ARM PL310 L2 cache controller
@@ -23,6 +26,7 @@ config ARCH_TEGRA_3x_SOC
select USB_ULPI if USB_SUPPORT
select USB_ULPI_VIEWPORT if USB_SUPPORT
select USE_OF
+ select PL310_ERRATA_769419 if CACHE_L2X0
help
Support for NVIDIA Tegra T30 processor family, based on the
ARM CortexA9MP CPU and the ARM PL310 L2 cache controller
--
1.7.8.GIT
^ permalink raw reply related [flat|nested] 9+ messages in thread
* [PATCH] ARM: tegra: select required CPU and L2 errata options
2012-01-04 23:29 [PATCH] ARM: tegra: select required CPU and L2 errata options Olof Johansson
@ 2012-01-05 16:30 ` Marc Dietrich
2012-01-05 18:40 ` Doug Anderson
2012-01-05 18:46 ` Stephen Warren
0 siblings, 2 replies; 9+ messages in thread
From: Marc Dietrich @ 2012-01-05 16:30 UTC (permalink / raw)
To: linux-arm-kernel
Hi,
Am Mittwoch, 4. Januar 2012, 15:29:12 schrieb Olof Johansson:
> Only one of the L2 erratas are needed on T30, but T20/25
> needs a few more.
can we have a word from someone from NVIDIA about which Tegra versions/revisions need
which erratum applied?
Currently the kernel has selections for the following errata (on ARM):
T2x T30
ERRATA_430973 Stale prediction on replaced interworking branch
ERRATA_458693 Processor deadlock when a false hazard is created
ERRATA_460075 Data written to the L2 cache can be overwritten
with stale data
x ERRATA_742230 DMB operation may be faulty
ERRATA_742231 Incorrect hazard handling in the SCU may lead to
data corruption
PL310_ERRATA_588369 Clean & Invalidate maintenance operations do not
invalidate clean lines
ERRATA_720789 TLBIASIDIS and TLBIMVAIS operations can broadcast
a faulty ASID
PL310_ERRATA_727915 Background Clean & Invalidate by Way operation
can cause data corruption
ERRATA_743622 Faulty hazard checking in the Store Buffer may
lead to data corruption
ERRATA_751472 Interrupted ICIALLUIS may prevent completion of
broadcasted operation
x PL310_ERRATA_753970 cache sync operation may be faulty
ERRATA_754322 possible faulty MMU translations following an ASID
switch
ERRATA_754327 no automatic Store Buffer drain
ERRATA_764369 Data cache line maintenance operation by MVA may
not succeed
x x PL310_ERRATA_769419 no automatic Store Buffer drain
With Olofs patch, the following errata are enabled by tegra_defconfig:
742230(T2x), 753970(T2x), 769419(T2x + T30)
I have the feeling that at least 764369 also needs to be enabled. From the
description: ... affecting Cortex-A9 MPCore with two or more processors (all
current revisions). I wonder if this includes multi *cores*.
The TRM is not very detailed about this topic. Having this info will likely remove
some headache for non OEM custumers.
Marc
>
> Signed-off-by: Olof Johansson <olof@lixom.net>
> ---
> arch/arm/mach-tegra/Kconfig | 4 ++++
> 1 files changed, 4 insertions(+), 0 deletions(-)
>
> diff --git a/arch/arm/mach-tegra/Kconfig b/arch/arm/mach-tegra/Kconfig
> index 373652d..a2c76ee 100644
> --- a/arch/arm/mach-tegra/Kconfig
> +++ b/arch/arm/mach-tegra/Kconfig
> @@ -10,6 +10,9 @@ config ARCH_TEGRA_2x_SOC
> select USB_ARCH_HAS_EHCI if USB_SUPPORT
> select USB_ULPI if USB_SUPPORT
> select USB_ULPI_VIEWPORT if USB_SUPPORT
> + select ARM_ERRATA_742230
> + select PL310_ERRATA_753970 if CACHE_PL310
> + select PL310_ERRATA_769419 if CACHE_L2X0
> help
> Support for NVIDIA Tegra AP20 and T20 processors, based on the
> ARM CortexA9MP CPU and the ARM PL310 L2 cache controller
> @@ -23,6 +26,7 @@ config ARCH_TEGRA_3x_SOC
> select USB_ULPI if USB_SUPPORT
> select USB_ULPI_VIEWPORT if USB_SUPPORT
> select USE_OF
> + select PL310_ERRATA_769419 if CACHE_L2X0
> help
> Support for NVIDIA Tegra T30 processor family, based on the
> ARM CortexA9MP CPU and the ARM PL310 L2 cache controller
^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH] ARM: tegra: select required CPU and L2 errata options
2012-01-05 16:30 ` Marc Dietrich
@ 2012-01-05 18:40 ` Doug Anderson
2012-01-05 22:03 ` Marc Dietrich
2012-01-05 18:46 ` Stephen Warren
1 sibling, 1 reply; 9+ messages in thread
From: Doug Anderson @ 2012-01-05 18:40 UTC (permalink / raw)
To: linux-arm-kernel
I can't comment on the L2 stuff (haven't looked there), but just going
by Cortex-A9 revisions (T25 has roughly r1p1 and T30 has r2p7), I've
got this table:
Errata?? ? ? ? ? ? T25 (A9 ~r1p1) ? T30 (A9 r2p7) ? ?Applies to
ARM_ERRATA_364296 ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?pre A8
ARM_ERRATA_411920 ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?pre A8
ARM_ERRATA_430973 ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?A8
ARM_ERRATA_458693 ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?A8
ARM_ERRATA_460075 ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?A8
ARM_ERRATA_720789 ?yes ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? A9 (prior to r2p0)
ARM_ERRATA_742230 ?yes ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? A9 (r1p0..r2p2)
ARM_ERRATA_742231 ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?A9 (r2p0..r2p2)
ARM_ERRATA_743622 ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?A9 (r2p0..r2p2)
ARM_ERRATA_751472 ?yes ? ? ? ? ? ? ?yes ? ? ? ? ? ? ?A9 (prior to r3p0)
ARM_ERRATA_754322 ? ? ? ? ? ? ? ? ? yes ? ? ? ? ? ? ?A9 (r2p*, r3p*)
ARM_ERRATA_754327 ?yes ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? A9 (prior to r2p0)
ARM_ERRATA_764369 ?yes ? ? ? ? ? ? ?yes ? ? ? ? ? ? ?A9 MPCore (all revs)
...so that means we should also be adding in?720789 (T2x),?751472
(T2x, T30),?754322 (T30),?754327 (T2x),?764369 (T2x, T30).
Certainly appreciate anyone checking my work and definitely appreciate
input from nVidia.
-Doug
---
On Thu, Jan 5, 2012 at 8:30 AM, Marc Dietrich <marvin24@gmx.de> wrote:
>
> Hi,
>
> Am Mittwoch, 4. Januar 2012, 15:29:12 schrieb Olof Johansson:
> > Only one of the L2 erratas are needed on T30, but T20/25
> > needs a few more.
>
> can we have a word from someone from NVIDIA about which Tegra versions/revisions need
> which erratum applied?
>
> Currently the kernel has selections for the following errata (on ARM):
>
> ? ? ? ?T2x ? ? ? ? ? ? T30
> ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?ERRATA_430973 ? ? ? ? ? ? ? ? ? Stale prediction on replaced interworking branch
> ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?ERRATA_458693 ? ? ? ? ? ? ? ? ? Processor deadlock when a false hazard is created
> ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?ERRATA_460075 ? ? ? ? ? ? ? ? ? Data written to the L2 cache can be overwritten
> ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?with stale data
> ? ? ? ?x ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ERRATA_742230 ? ? ? ? ? ? ? ? ? DMB operation may be faulty
> ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?ERRATA_742231 ? ? ? ? ? ? ? ? ? Incorrect hazard handling in the SCU may lead to
> ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?data corruption
> ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?PL310_ERRATA_588369 ? ? Clean & Invalidate maintenance operations do not
> ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?invalidate clean lines
> ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?ERRATA_720789 ? ? ? ? ? ? ? ? ? TLBIASIDIS and TLBIMVAIS operations can broadcast
> ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?a faulty ASID
> ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?PL310_ERRATA_727915 ? ? Background Clean & Invalidate by Way operation
> ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?can cause data corruption
> ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?ERRATA_743622 ? ? ? ? ? ? ? ? ? Faulty hazard checking in the Store Buffer may
> ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?lead to data corruption
> ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?ERRATA_751472 ? ? ? ? ? ? ? ? ? Interrupted ICIALLUIS may prevent completion of
> ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?broadcasted operation
> ? ? ? ?x ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? PL310_ERRATA_753970 ? ? cache sync operation may be faulty
> ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?ERRATA_754322 ? ? ? ? ? ? ? ? ? possible faulty MMU translations following an ASID
> ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?switch
> ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?ERRATA_754327 ? ? ? ? ? ? ? ? ? no automatic Store Buffer drain
> ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?ERRATA_764369 ? ? ? ? ? ? ? ? ? Data cache line maintenance operation by MVA may
> ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?not succeed
> ? ? ? ?x ? ? ? ? ? ? ? x ? ? ? ? ? ? ? PL310_ERRATA_769419 ? ? no automatic Store Buffer drain
>
> With Olofs patch, the following errata are enabled by tegra_defconfig:
>
> 742230(T2x), 753970(T2x), 769419(T2x + T30)
>
> I have the feeling that at least 764369 also needs to be enabled. From the
> description: ... affecting Cortex-A9 MPCore with two or more processors (all
> current revisions). I wonder if this includes multi *cores*.
>
> The TRM is not very detailed about this topic. Having this info will likely remove
> some headache for non OEM custumers.
>
> Marc
>
>
> >
> > Signed-off-by: Olof Johansson <olof@lixom.net>
> > ---
> > ?arch/arm/mach-tegra/Kconfig | ? ?4 ++++
> > ?1 files changed, 4 insertions(+), 0 deletions(-)
> >
> > diff --git a/arch/arm/mach-tegra/Kconfig b/arch/arm/mach-tegra/Kconfig
> > index 373652d..a2c76ee 100644
> > --- a/arch/arm/mach-tegra/Kconfig
> > +++ b/arch/arm/mach-tegra/Kconfig
> > @@ -10,6 +10,9 @@ config ARCH_TEGRA_2x_SOC
> > ? ? ? select USB_ARCH_HAS_EHCI if USB_SUPPORT
> > ? ? ? select USB_ULPI if USB_SUPPORT
> > ? ? ? select USB_ULPI_VIEWPORT if USB_SUPPORT
> > + ? ? select ARM_ERRATA_742230
> > + ? ? select PL310_ERRATA_753970 if CACHE_PL310
> > + ? ? select PL310_ERRATA_769419 if CACHE_L2X0
> > ? ? ? help
> > ? ? ? ? Support for NVIDIA Tegra AP20 and T20 processors, based on the
> > ? ? ? ? ARM CortexA9MP CPU and the ARM PL310 L2 cache controller
> > @@ -23,6 +26,7 @@ config ARCH_TEGRA_3x_SOC
> > ? ? ? select USB_ULPI if USB_SUPPORT
> > ? ? ? select USB_ULPI_VIEWPORT if USB_SUPPORT
> > ? ? ? select USE_OF
> > + ? ? select PL310_ERRATA_769419 if CACHE_L2X0
> > ? ? ? help
> > ? ? ? ? Support for NVIDIA Tegra T30 processor family, based on the
> > ? ? ? ? ARM CortexA9MP CPU and the ARM PL310 L2 cache controller
> --
> To unsubscribe from this list: send the line "unsubscribe linux-tegra" in
> the body of a message to majordomo at vger.kernel.org
> More majordomo info at ?http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH] ARM: tegra: select required CPU and L2 errata options
2012-01-05 18:40 ` Doug Anderson
@ 2012-01-05 22:03 ` Marc Dietrich
2012-01-06 13:56 ` Mikael Pettersson
0 siblings, 1 reply; 9+ messages in thread
From: Marc Dietrich @ 2012-01-05 22:03 UTC (permalink / raw)
To: linux-arm-kernel
On Thursday 05 January 2012 10:40:48 Doug Anderson wrote:
> I can't comment on the L2 stuff (haven't looked there), but just going
> by Cortex-A9 revisions (T25 has roughly r1p1 and T30 has r2p7), I've
> got this table:
>
> Errata T25 (A9 ~r1p1) T30 (A9 r2p7) Applies to
> ARM_ERRATA_364296 pre A8
> ARM_ERRATA_411920 pre A8
> ARM_ERRATA_430973 A8
> ARM_ERRATA_458693 A8
> ARM_ERRATA_460075 A8
> ARM_ERRATA_720789 yes A9 (prior to r2p0)
> ARM_ERRATA_742230 yes A9 (r1p0..r2p2)
> ARM_ERRATA_742231 A9 (r2p0..r2p2)
> ARM_ERRATA_743622 A9 (r2p0..r2p2)
> ARM_ERRATA_751472 yes yes A9 (prior to r3p0)
> ARM_ERRATA_754322 yes A9 (r2p*, r3p*)
> ARM_ERRATA_754327 yes A9 (prior to r2p0)
> ARM_ERRATA_764369 yes yes A9 MPCore (all revs)
>
> ...so that means we should also be adding in 720789 (T2x), 751472
> (T2x, T30), 754322 (T30), 754327 (T2x), 764369 (T2x, T30).
AFAIK, 720789 (TLBIASIDIS and TLBIMVAIS operations can broadcast a faulty
ASID) is fixed in userspace (or gcc) nowadays (at least in Ubuntu) so I think
this can be neglected.
Marc
> Certainly appreciate anyone checking my work and definitely appreciate
> input from nVidia.
>
> -Doug
>
> ---
>
> On Thu, Jan 5, 2012 at 8:30 AM, Marc Dietrich <marvin24@gmx.de> wrote:
> > Hi,
> >
> > Am Mittwoch, 4. Januar 2012, 15:29:12 schrieb Olof Johansson:
> > > Only one of the L2 erratas are needed on T30, but T20/25
> > > needs a few more.
> >
> > can we have a word from someone from NVIDIA about which Tegra
> > versions/revisions need which erratum applied?
> >
> > Currently the kernel has selections for the following errata (on ARM):
> >
> > T2x T30
> > ERRATA_430973
> > Stale prediction on replaced interworking branch ERRATA_458693
> > Processor deadlock when a false hazard is created
> > ERRATA_460075 Data written to the L2 cache can be
> > overwritten with stale data x
> > ERRATA_742230 DMB operation may be faulty
> > ERRATA_742231 Incorrect hazard handling in the SCU
> > may lead to data corruption PL310_ERRATA_588369 Clean & Invalidate
> > maintenance operations do not invalidate clean lines ERRATA_720789
> > TLBIASIDIS and TLBIMVAIS operations can broadcast a
> > faulty ASID PL310_ERRATA_727915 Background Clean & Invalidate by
> > Way operation can cause data corruption ERRATA_743622
> > Faulty hazard checking in the Store Buffer may lead to data corruption
> > ERRATA_751472 Interrupted ICIALLUIS may prevent
> > completion of broadcasted operation x
> > PL310_ERRATA_753970 cache sync operation may be faulty
> > ERRATA_754322 possible faulty MMU translations
> > following an ASID switch ERRATA_754327 no automatic
> > Store Buffer drain ERRATA_764369 Data cache line
> > maintenance operation by MVA may not succeed x x
> > PL310_ERRATA_769419 no automatic Store Buffer drain
> >
> > With Olofs patch, the following errata are enabled by tegra_defconfig:
> >
> > 742230(T2x), 753970(T2x), 769419(T2x + T30)
> >
> > I have the feeling that at least 764369 also needs to be enabled. From
> > the description: ... affecting Cortex-A9 MPCore with two or more
> > processors (all current revisions). I wonder if this includes multi
> > *cores*.
> >
> > The TRM is not very detailed about this topic. Having this info will
> > likely remove some headache for non OEM custumers.
> >
> > Marc
> >
> > > Signed-off-by: Olof Johansson <olof@lixom.net>
> > > ---
> > > arch/arm/mach-tegra/Kconfig | 4 ++++
> > > 1 files changed, 4 insertions(+), 0 deletions(-)
> > >
> > > diff --git a/arch/arm/mach-tegra/Kconfig
> > > b/arch/arm/mach-tegra/Kconfig
> > > index 373652d..a2c76ee 100644
> > > --- a/arch/arm/mach-tegra/Kconfig
> > > +++ b/arch/arm/mach-tegra/Kconfig
> > > @@ -10,6 +10,9 @@ config ARCH_TEGRA_2x_SOC
> > > select USB_ARCH_HAS_EHCI if USB_SUPPORT
> > > select USB_ULPI if USB_SUPPORT
> > > select USB_ULPI_VIEWPORT if USB_SUPPORT
> > > + select ARM_ERRATA_742230
> > > + select PL310_ERRATA_753970 if CACHE_PL310
> > > + select PL310_ERRATA_769419 if CACHE_L2X0
> > > help
> > > Support for NVIDIA Tegra AP20 and T20 processors, based on
> > > the
> > > ARM CortexA9MP CPU and the ARM PL310 L2 cache controller
> > > @@ -23,6 +26,7 @@ config ARCH_TEGRA_3x_SOC
> > > select USB_ULPI if USB_SUPPORT
> > > select USB_ULPI_VIEWPORT if USB_SUPPORT
> > > select USE_OF
> > > + select PL310_ERRATA_769419 if CACHE_L2X0
> > > help
> > > Support for NVIDIA Tegra T30 processor family, based on the
> > > ARM CortexA9MP CPU and the ARM PL310 L2 cache controller
^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH] ARM: tegra: select required CPU and L2 errata options
2012-01-05 22:03 ` Marc Dietrich
@ 2012-01-06 13:56 ` Mikael Pettersson
2012-01-06 18:36 ` Marc Dietrich
0 siblings, 1 reply; 9+ messages in thread
From: Mikael Pettersson @ 2012-01-06 13:56 UTC (permalink / raw)
To: linux-arm-kernel
Marc Dietrich writes:
> On Thursday 05 January 2012 10:40:48 Doug Anderson wrote:
> > I can't comment on the L2 stuff (haven't looked there), but just going
> > by Cortex-A9 revisions (T25 has roughly r1p1 and T30 has r2p7), I've
> > got this table:
> >
> > Errata T25 (A9 ~r1p1) T30 (A9 r2p7) Applies to
> > ARM_ERRATA_364296 pre A8
> > ARM_ERRATA_411920 pre A8
> > ARM_ERRATA_430973 A8
> > ARM_ERRATA_458693 A8
> > ARM_ERRATA_460075 A8
> > ARM_ERRATA_720789 yes A9 (prior to r2p0)
> > ARM_ERRATA_742230 yes A9 (r1p0..r2p2)
> > ARM_ERRATA_742231 A9 (r2p0..r2p2)
> > ARM_ERRATA_743622 A9 (r2p0..r2p2)
> > ARM_ERRATA_751472 yes yes A9 (prior to r3p0)
> > ARM_ERRATA_754322 yes A9 (r2p*, r3p*)
> > ARM_ERRATA_754327 yes A9 (prior to r2p0)
> > ARM_ERRATA_764369 yes yes A9 MPCore (all revs)
> >
> > ...so that means we should also be adding in 720789 (T2x), 751472
> > (T2x, T30), 754322 (T30), 754327 (T2x), 764369 (T2x, T30).
>
> AFAIK, 720789 (TLBIASIDIS and TLBIMVAIS operations can broadcast a faulty
> ASID) is fixed in userspace (or gcc) nowadays (at least in Ubuntu) so I think
> this can be neglected.
Not everyone uses Ubuntu.
Can you provide a reference to the corresponding gcc and/or Ubuntu patch?
(I got a Tegra-2 based trimslice last summer, but it's been completely
unreliable under load (hangs hard during "make -j2" gcc bootstraps),
hence I'm looking for patches or any other information that can help
me fix or work around whatever bugs it has.)
^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH] ARM: tegra: select required CPU and L2 errata options
2012-01-06 13:56 ` Mikael Pettersson
@ 2012-01-06 18:36 ` Marc Dietrich
0 siblings, 0 replies; 9+ messages in thread
From: Marc Dietrich @ 2012-01-06 18:36 UTC (permalink / raw)
To: linux-arm-kernel
On Friday 06 January 2012 14:56:52 Mikael Pettersson wrote:
> Marc Dietrich writes:
> > On Thursday 05 January 2012 10:40:48 Doug Anderson wrote:
> > > I can't comment on the L2 stuff (haven't looked there), but just
> > > going
> > > by Cortex-A9 revisions (T25 has roughly r1p1 and T30 has r2p7),
> > > I've
> > > got this table:
> > >
> > > Errata T25 (A9 ~r1p1) T30 (A9 r2p7) Applies to
> > > ARM_ERRATA_364296 pre A8
> > > ARM_ERRATA_411920 pre A8
> > > ARM_ERRATA_430973 A8
> > > ARM_ERRATA_458693 A8
> > > ARM_ERRATA_460075 A8
> > > ARM_ERRATA_720789 yes A9 (prior to
> > > r2p0) ARM_ERRATA_742230 yes A9
> > > (r1p0..r2p2) ARM_ERRATA_742231
> > > A9 (r2p0..r2p2) ARM_ERRATA_743622
> > > A9 (r2p0..r2p2) ARM_ERRATA_751472 yes yes
> > > A9 (prior to r3p0) ARM_ERRATA_754322 yes
> > > A9 (r2p*, r3p*) ARM_ERRATA_754327 yes
> > > A9 (prior to r2p0) ARM_ERRATA_764369 yes
> > > yes A9 MPCore (all revs)
> > >
> > > ...so that means we should also be adding in 720789 (T2x), 751472
> > > (T2x, T30), 754322 (T30), 754327 (T2x), 764369 (T2x, T30).
> >
> > AFAIK, 720789 (TLBIASIDIS and TLBIMVAIS operations can broadcast a
> > faulty ASID) is fixed in userspace (or gcc) nowadays (at least in
> > Ubuntu) so I think this can be neglected.
>
> Not everyone uses Ubuntu.
yeah, I know. I just said this because nvidias reference filesystem is based
upon it.
> Can you provide a reference to the corresponding gcc and/or Ubuntu patch?
all I found in the haste is
https://bugs.launchpad.net/ubuntu/+source/eglibc/+bug/739374
I think in the end gcc was patched not emit get/set_tls syscalls but to use
MCR/MRC cp15 ops (sorry, I don't know what this means - just copied from irc
log). This meant that the whole userspace needed recompilation. It is possible
that all newer gcc behave like this. As far as I understood, enabling the
erratum in the kernel with fixed userspace will make the problem reapear.
Someone with more knowledge may correct me.
> (I got a Tegra-2 based trimslice last summer, but it's been completely
> unreliable under load (hangs hard during "make -j2" gcc bootstraps),
> hence I'm looking for patches or any other information that can help
> me fix or work around whatever bugs it has.)
I think this can't be related to the erratum above because it manifestated
itself in a segfault or illegal instruction userspace error.
Did you tried to ask in the trimslice forum about this error?
Marc
^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH] ARM: tegra: select required CPU and L2 errata options
2012-01-05 16:30 ` Marc Dietrich
2012-01-05 18:40 ` Doug Anderson
@ 2012-01-05 18:46 ` Stephen Warren
1 sibling, 0 replies; 9+ messages in thread
From: Stephen Warren @ 2012-01-05 18:46 UTC (permalink / raw)
To: linux-arm-kernel
Marc Dietrich wrote at Thursday, January 05, 2012 9:30 AM:
> Am Mittwoch, 4. Januar 2012, 15:29:12 schrieb Olof Johansson:
> > Only one of the L2 erratas are needed on T30, but T20/25
> > needs a few more.
>
> can we have a word from someone from NVIDIA about which Tegra versions/revisions need
> which erratum applied?
Well, I can tell you easily which CPU revisions the various Tegra have.
According to our internal wiki:
Tegra20: CPU r1p1, PL310 r2p0
Tegra30: CPU A01=r2p7/A02+=r2p9, NEON r2p3-50, PL310 r3p1-50
You should be able to work out which errata are needed using the ARM
documentation.
--
nvpublic
^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH] ARM: tegra: select required CPU and L2 errata options
@ 2012-02-14 20:39 Stephen Warren
2012-02-26 22:25 ` Olof Johansson
0 siblings, 1 reply; 9+ messages in thread
From: Stephen Warren @ 2012-02-14 20:39 UTC (permalink / raw)
To: linux-arm-kernel
The ARM IP revisions in Tegra are:
Tegra20: CPU r1p1, PL310 r2p0
Tegra30: CPU A01=r2p7/>=A02=r2p9, NEON r2p3-50, PL310 r3p1-50
Based on work by Olof Johansson, although the actual list of errata is
somewhat different here, since I added a bunch more and removed one PL310
erratum that doesn't seem applicable.
Signed-off-by: Stephen Warren <swarren@nvidia.com>
---
Olof, should this be CC: stable?
arch/arm/mach-tegra/Kconfig | 12 ++++++++++++
1 files changed, 12 insertions(+), 0 deletions(-)
diff --git a/arch/arm/mach-tegra/Kconfig b/arch/arm/mach-tegra/Kconfig
index 32b420a..1978f59 100644
--- a/arch/arm/mach-tegra/Kconfig
+++ b/arch/arm/mach-tegra/Kconfig
@@ -12,6 +12,13 @@ config ARCH_TEGRA_2x_SOC
select USB_ARCH_HAS_EHCI if USB_SUPPORT
select USB_ULPI if USB_SUPPORT
select USB_ULPI_VIEWPORT if USB_SUPPORT
+ select ARM_ERRATA_720789
+ select ARM_ERRATA_742230
+ select ARM_ERRATA_751472
+ select ARM_ERRATA_754327
+ select ARM_ERRATA_764369
+ select PL310_ERRATA_727915 if CACHE_L2X0
+ select PL310_ERRATA_769419 if CACHE_L2X0
help
Support for NVIDIA Tegra AP20 and T20 processors, based on the
ARM CortexA9MP CPU and the ARM PL310 L2 cache controller
@@ -27,6 +34,11 @@ config ARCH_TEGRA_3x_SOC
select USB_ULPI if USB_SUPPORT
select USB_ULPI_VIEWPORT if USB_SUPPORT
select USE_OF
+ select ARM_ERRATA_743622
+ select ARM_ERRATA_751472
+ select ARM_ERRATA_754322
+ select ARM_ERRATA_764369
+ select PL310_ERRATA_769419 if CACHE_L2X0
help
Support for NVIDIA Tegra T30 processor family, based on the
ARM CortexA9MP CPU and the ARM PL310 L2 cache controller
--
1.7.0.4
^ permalink raw reply related [flat|nested] 9+ messages in thread
* [PATCH] ARM: tegra: select required CPU and L2 errata options
2012-02-14 20:39 Stephen Warren
@ 2012-02-26 22:25 ` Olof Johansson
0 siblings, 0 replies; 9+ messages in thread
From: Olof Johansson @ 2012-02-26 22:25 UTC (permalink / raw)
To: linux-arm-kernel
On Tue, Feb 14, 2012 at 01:39:39PM -0700, Stephen Warren wrote:
> The ARM IP revisions in Tegra are:
> Tegra20: CPU r1p1, PL310 r2p0
> Tegra30: CPU A01=r2p7/>=A02=r2p9, NEON r2p3-50, PL310 r3p1-50
>
> Based on work by Olof Johansson, although the actual list of errata is
> somewhat different here, since I added a bunch more and removed one PL310
> erratum that doesn't seem applicable.
>
> Signed-off-by: Stephen Warren <swarren@nvidia.com>
> ---
> Olof, should this be CC: stable?
Sounds reasonable. I've added the patch to for-3.4/cleanups_and_fixes and added
the cc. Given that it's not a refression I'm hesitant to send a fix for 3.3 at
this point in the release cycle.
-Olof
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2012-02-26 22:25 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-01-04 23:29 [PATCH] ARM: tegra: select required CPU and L2 errata options Olof Johansson
2012-01-05 16:30 ` Marc Dietrich
2012-01-05 18:40 ` Doug Anderson
2012-01-05 22:03 ` Marc Dietrich
2012-01-06 13:56 ` Mikael Pettersson
2012-01-06 18:36 ` Marc Dietrich
2012-01-05 18:46 ` Stephen Warren
-- strict thread matches above, loose matches on Subject: below --
2012-02-14 20:39 Stephen Warren
2012-02-26 22:25 ` Olof Johansson
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox