* [PATCH] ARM: nommu: access ID_PFR1 only if CPUID scheme
@ 2017-03-17 16:40 ` afzal mohammed
0 siblings, 0 replies; 8+ messages in thread
From: afzal mohammed @ 2017-03-17 16:40 UTC (permalink / raw)
To: Russell King - ARM Linux
Cc: Greg Ungerer, Vladimir Murzin, linux-arm-kernel, linux-kernel,
afzal mohammed
Greg upon trying to boot no-MMU Kernel on ARM926EJ reported boot
failure. He root caused it to ID_PFR1 access introduced by the
commit mentioned in the fixes tag below.
All CP15 processors need not have processor feature registers, only
for architectures defined by CPUID scheme would have it. Hence check
for it before accessing processor feature register, ID_PFR1.
Fixes: f8300a0b5de0 ("ARM: 8647/2: nommu: dynamic exception base address setting")
Reported-by: Greg Ungerer <gerg@uclinux.org>
Signed-off-by: afzal mohammed <afzal.mohd.ma@gmail.com>
---
Hi Russell,
It would be good to have the fix go in during -rc, as,
1. Culprit commit went in during the last merge window
2. Though nothing supported in mainline is known to be broken, the
original change needs to be modified to be reliable
Vladimir, this is being posted as the issue is taken care run time.
Regards
afzal
---
arch/arm/mm/nommu.c | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/arch/arm/mm/nommu.c b/arch/arm/mm/nommu.c
index 3b5c7aaf9c76..33a45bd96860 100644
--- a/arch/arm/mm/nommu.c
+++ b/arch/arm/mm/nommu.c
@@ -303,7 +303,10 @@ static inline void set_vbar(unsigned long val)
*/
static inline bool security_extensions_enabled(void)
{
- return !!cpuid_feature_extract(CPUID_EXT_PFR1, 4);
+ /* Check CPUID Identification Scheme before ID_PFR1 read */
+ if ((read_cpuid_id() & 0x000f0000) == 0x000f0000)
+ return !!cpuid_feature_extract(CPUID_EXT_PFR1, 4);
+ return 0;
}
static unsigned long __init setup_vectors_base(void)
--
2.12.0
^ permalink raw reply related [flat|nested] 8+ messages in thread
* [PATCH] ARM: nommu: access ID_PFR1 only if CPUID scheme
@ 2017-03-17 16:40 ` afzal mohammed
0 siblings, 0 replies; 8+ messages in thread
From: afzal mohammed @ 2017-03-17 16:40 UTC (permalink / raw)
To: linux-arm-kernel
Greg upon trying to boot no-MMU Kernel on ARM926EJ reported boot
failure. He root caused it to ID_PFR1 access introduced by the
commit mentioned in the fixes tag below.
All CP15 processors need not have processor feature registers, only
for architectures defined by CPUID scheme would have it. Hence check
for it before accessing processor feature register, ID_PFR1.
Fixes: f8300a0b5de0 ("ARM: 8647/2: nommu: dynamic exception base address setting")
Reported-by: Greg Ungerer <gerg@uclinux.org>
Signed-off-by: afzal mohammed <afzal.mohd.ma@gmail.com>
---
Hi Russell,
It would be good to have the fix go in during -rc, as,
1. Culprit commit went in during the last merge window
2. Though nothing supported in mainline is known to be broken, the
original change needs to be modified to be reliable
Vladimir, this is being posted as the issue is taken care run time.
Regards
afzal
---
arch/arm/mm/nommu.c | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/arch/arm/mm/nommu.c b/arch/arm/mm/nommu.c
index 3b5c7aaf9c76..33a45bd96860 100644
--- a/arch/arm/mm/nommu.c
+++ b/arch/arm/mm/nommu.c
@@ -303,7 +303,10 @@ static inline void set_vbar(unsigned long val)
*/
static inline bool security_extensions_enabled(void)
{
- return !!cpuid_feature_extract(CPUID_EXT_PFR1, 4);
+ /* Check CPUID Identification Scheme before ID_PFR1 read */
+ if ((read_cpuid_id() & 0x000f0000) == 0x000f0000)
+ return !!cpuid_feature_extract(CPUID_EXT_PFR1, 4);
+ return 0;
}
static unsigned long __init setup_vectors_base(void)
--
2.12.0
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH] ARM: nommu: access ID_PFR1 only if CPUID scheme
2017-03-17 16:40 ` afzal mohammed
@ 2017-03-23 10:43 ` afzal mohammed
-1 siblings, 0 replies; 8+ messages in thread
From: afzal mohammed @ 2017-03-23 10:43 UTC (permalink / raw)
To: Russell King - ARM Linux, Greg Ungerer
Cc: Vladimir Murzin, linux-arm-kernel, linux-kernel
Hi,
On Fri, Mar 17, 2017 at 10:10:34PM +0530, afzal mohammed wrote:
> Greg upon trying to boot no-MMU Kernel on ARM926EJ reported boot
> failure. He root caused it to ID_PFR1 access introduced by the
> commit mentioned in the fixes tag below.
>
> All CP15 processors need not have processor feature registers, only
> for architectures defined by CPUID scheme would have it. Hence check
> for it before accessing processor feature register, ID_PFR1.
>
> Fixes: f8300a0b5de0 ("ARM: 8647/2: nommu: dynamic exception base address setting")
> Reported-by: Greg Ungerer <gerg@uclinux.org>
> Signed-off-by: afzal mohammed <afzal.mohd.ma@gmail.com>
Greg, can i add your Tested-by ?
Regards
afzal
> ---
>
> Hi Russell,
>
> It would be good to have the fix go in during -rc, as,
>
> 1. Culprit commit went in during the last merge window
> 2. Though nothing supported in mainline is known to be broken, the
> original change needs to be modified to be reliable
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH] ARM: nommu: access ID_PFR1 only if CPUID scheme
@ 2017-03-23 10:43 ` afzal mohammed
0 siblings, 0 replies; 8+ messages in thread
From: afzal mohammed @ 2017-03-23 10:43 UTC (permalink / raw)
To: linux-arm-kernel
Hi,
On Fri, Mar 17, 2017 at 10:10:34PM +0530, afzal mohammed wrote:
> Greg upon trying to boot no-MMU Kernel on ARM926EJ reported boot
> failure. He root caused it to ID_PFR1 access introduced by the
> commit mentioned in the fixes tag below.
>
> All CP15 processors need not have processor feature registers, only
> for architectures defined by CPUID scheme would have it. Hence check
> for it before accessing processor feature register, ID_PFR1.
>
> Fixes: f8300a0b5de0 ("ARM: 8647/2: nommu: dynamic exception base address setting")
> Reported-by: Greg Ungerer <gerg@uclinux.org>
> Signed-off-by: afzal mohammed <afzal.mohd.ma@gmail.com>
Greg, can i add your Tested-by ?
Regards
afzal
> ---
>
> Hi Russell,
>
> It would be good to have the fix go in during -rc, as,
>
> 1. Culprit commit went in during the last merge window
> 2. Though nothing supported in mainline is known to be broken, the
> original change needs to be modified to be reliable
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] ARM: nommu: access ID_PFR1 only if CPUID scheme
2017-03-23 10:43 ` afzal mohammed
@ 2017-03-23 11:37 ` Greg Ungerer
-1 siblings, 0 replies; 8+ messages in thread
From: Greg Ungerer @ 2017-03-23 11:37 UTC (permalink / raw)
To: afzal mohammed, Russell King - ARM Linux
Cc: Vladimir Murzin, linux-arm-kernel, linux-kernel
Hi Afzal,
On 23/03/17 20:43, afzal mohammed wrote:
> On Fri, Mar 17, 2017 at 10:10:34PM +0530, afzal mohammed wrote:
>> Greg upon trying to boot no-MMU Kernel on ARM926EJ reported boot
>> failure. He root caused it to ID_PFR1 access introduced by the
>> commit mentioned in the fixes tag below.
>>
>> All CP15 processors need not have processor feature registers, only
>> for architectures defined by CPUID scheme would have it. Hence check
>> for it before accessing processor feature register, ID_PFR1.
>>
>> Fixes: f8300a0b5de0 ("ARM: 8647/2: nommu: dynamic exception base address setting")
>> Reported-by: Greg Ungerer <gerg@uclinux.org>
>> Signed-off-by: afzal mohammed <afzal.mohd.ma@gmail.com>
>
> Greg, can i add your Tested-by ?
Absolutely:
Tested-by: Greg Ungerer <gerg@uclinux.org>
Thanks for taking care of this.
Regards
Greg
>> ---
>>
>> Hi Russell,
>>
>> It would be good to have the fix go in during -rc, as,
>>
>> 1. Culprit commit went in during the last merge window
>> 2. Though nothing supported in mainline is known to be broken, the
>> original change needs to be modified to be reliable
>
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH] ARM: nommu: access ID_PFR1 only if CPUID scheme
@ 2017-03-23 11:37 ` Greg Ungerer
0 siblings, 0 replies; 8+ messages in thread
From: Greg Ungerer @ 2017-03-23 11:37 UTC (permalink / raw)
To: linux-arm-kernel
Hi Afzal,
On 23/03/17 20:43, afzal mohammed wrote:
> On Fri, Mar 17, 2017 at 10:10:34PM +0530, afzal mohammed wrote:
>> Greg upon trying to boot no-MMU Kernel on ARM926EJ reported boot
>> failure. He root caused it to ID_PFR1 access introduced by the
>> commit mentioned in the fixes tag below.
>>
>> All CP15 processors need not have processor feature registers, only
>> for architectures defined by CPUID scheme would have it. Hence check
>> for it before accessing processor feature register, ID_PFR1.
>>
>> Fixes: f8300a0b5de0 ("ARM: 8647/2: nommu: dynamic exception base address setting")
>> Reported-by: Greg Ungerer <gerg@uclinux.org>
>> Signed-off-by: afzal mohammed <afzal.mohd.ma@gmail.com>
>
> Greg, can i add your Tested-by ?
Absolutely:
Tested-by: Greg Ungerer <gerg@uclinux.org>
Thanks for taking care of this.
Regards
Greg
>> ---
>>
>> Hi Russell,
>>
>> It would be good to have the fix go in during -rc, as,
>>
>> 1. Culprit commit went in during the last merge window
>> 2. Though nothing supported in mainline is known to be broken, the
>> original change needs to be modified to be reliable
>
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] ARM: nommu: access ID_PFR1 only if CPUID scheme
2017-03-23 11:37 ` Greg Ungerer
@ 2017-03-23 12:57 ` afzal mohammed
-1 siblings, 0 replies; 8+ messages in thread
From: afzal mohammed @ 2017-03-23 12:57 UTC (permalink / raw)
To: Greg Ungerer, Russell King - ARM Linux
Cc: Vladimir Murzin, linux-arm-kernel, linux-kernel
Hi,
On Thu, Mar 23, 2017 at 09:37:48PM +1000, Greg Ungerer wrote:
> Tested-by: Greg Ungerer <gerg@uclinux.org>
Thanks Greg
Since there was no negative feedback yet, change has been deposited in
rmk's patch system as 8665/1
Regards
afzal
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH] ARM: nommu: access ID_PFR1 only if CPUID scheme
@ 2017-03-23 12:57 ` afzal mohammed
0 siblings, 0 replies; 8+ messages in thread
From: afzal mohammed @ 2017-03-23 12:57 UTC (permalink / raw)
To: linux-arm-kernel
Hi,
On Thu, Mar 23, 2017 at 09:37:48PM +1000, Greg Ungerer wrote:
> Tested-by: Greg Ungerer <gerg@uclinux.org>
Thanks Greg
Since there was no negative feedback yet, change has been deposited in
rmk's patch system as 8665/1
Regards
afzal
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2017-03-23 12:57 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-03-17 16:40 [PATCH] ARM: nommu: access ID_PFR1 only if CPUID scheme afzal mohammed
2017-03-17 16:40 ` afzal mohammed
2017-03-23 10:43 ` afzal mohammed
2017-03-23 10:43 ` afzal mohammed
2017-03-23 11:37 ` Greg Ungerer
2017-03-23 11:37 ` Greg Ungerer
2017-03-23 12:57 ` afzal mohammed
2017-03-23 12:57 ` afzal mohammed
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.