* [PATCH] ARM: head: Remove boot-time HYP mode check for v5 and below @ 2012-12-07 13:45 Dave Martin 2012-12-10 10:57 ` Dave Martin 0 siblings, 1 reply; 7+ messages in thread From: Dave Martin @ 2012-12-07 13:45 UTC (permalink / raw) To: linux-arm-kernel The kernel can only be entered on HYP mode on CPUs which actually support it, i.e. >= ARMv7. pre-v6 platform support cannot coexist in the same kernel as support for v7 and higher, so there is no advantage in having the HYP mode check on pre-v6 hardware. At least one pre-v6 board is known to fail when the HYP mode check code is present, although the exact cause remains unknown and may be unrelated. [1] This patch restores the old behaviour for pre-v6 platforms, whereby the CPSR is forced directly to SVC mode with IRQs and FIQs masked. All kernels capable of booting on v7 hardware will retain the check, so this should not impair functionality. [1] http://lists.arm.linux.org.uk/lurker/message/20121130.013814.19218413.en.html ([ARM] head.S change broke platform device registration?) Signed-off-by: Dave Martin <dave.martin@linaro.org> --- arch/arm/include/asm/assembler.h | 8 ++++++++ 1 files changed, 8 insertions(+), 0 deletions(-) diff --git a/arch/arm/include/asm/assembler.h b/arch/arm/include/asm/assembler.h index 2ef9581..eb87200 100644 --- a/arch/arm/include/asm/assembler.h +++ b/arch/arm/include/asm/assembler.h @@ -250,6 +250,7 @@ * Beware, it also clobers LR. */ .macro safe_svcmode_maskall reg:req +#if __LINUX_ARM_ARCH__ >= 6 mrs \reg , cpsr mov lr , \reg and lr , lr , #MODE_MASK @@ -266,6 +267,13 @@ THUMB( orr \reg , \reg , #PSR_T_BIT ) __ERET 1: msr cpsr_c, \reg 2: +#else +/* + * workaround for possibly broken pre-v6 hardware + * (akita, Sharp Zaurus C-1000, PXA270-based) + */ + setmode PSR_F_BIT | PSR_I_BIT | SVC_MODE, \reg +#endif .endm /* -- 1.7.4.1 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCH] ARM: head: Remove boot-time HYP mode check for v5 and below 2012-12-07 13:45 [PATCH] ARM: head: Remove boot-time HYP mode check for v5 and below Dave Martin @ 2012-12-10 10:57 ` Dave Martin 2012-12-10 17:39 ` Marko Katić 0 siblings, 1 reply; 7+ messages in thread From: Dave Martin @ 2012-12-10 10:57 UTC (permalink / raw) To: linux-arm-kernel Hi Russell, Do you want me to send this patch to the patch system? Until we have confirmation from Marko (or someone else with a board) there's no way to know whether it fixes the problem. But it should restore us back to the old behaviour older hardware, reducing the likelihood of side issues. This could be reverted later if it proves to be unnecessary. Cheers ---Dave On Fri, Dec 07, 2012 at 01:45:59PM +0000, Dave Martin wrote: > The kernel can only be entered on HYP mode on CPUs which actually > support it, i.e. >= ARMv7. pre-v6 platform support cannot coexist > in the same kernel as support for v7 and higher, so there is no > advantage in having the HYP mode check on pre-v6 hardware. > > At least one pre-v6 board is known to fail when the HYP mode check > code is present, although the exact cause remains unknown and may > be unrelated. [1] > > This patch restores the old behaviour for pre-v6 platforms, whereby > the CPSR is forced directly to SVC mode with IRQs and FIQs masked. > All kernels capable of booting on v7 hardware will retain the > check, so this should not impair functionality. > > [1] http://lists.arm.linux.org.uk/lurker/message/20121130.013814.19218413.en.html > ([ARM] head.S change broke platform device registration?) > > Signed-off-by: Dave Martin <dave.martin@linaro.org> > --- > arch/arm/include/asm/assembler.h | 8 ++++++++ > 1 files changed, 8 insertions(+), 0 deletions(-) > > diff --git a/arch/arm/include/asm/assembler.h b/arch/arm/include/asm/assembler.h > index 2ef9581..eb87200 100644 > --- a/arch/arm/include/asm/assembler.h > +++ b/arch/arm/include/asm/assembler.h > @@ -250,6 +250,7 @@ > * Beware, it also clobers LR. > */ > .macro safe_svcmode_maskall reg:req > +#if __LINUX_ARM_ARCH__ >= 6 > mrs \reg , cpsr > mov lr , \reg > and lr , lr , #MODE_MASK > @@ -266,6 +267,13 @@ THUMB( orr \reg , \reg , #PSR_T_BIT ) > __ERET > 1: msr cpsr_c, \reg > 2: > +#else > +/* > + * workaround for possibly broken pre-v6 hardware > + * (akita, Sharp Zaurus C-1000, PXA270-based) > + */ > + setmode PSR_F_BIT | PSR_I_BIT | SVC_MODE, \reg > +#endif > .endm > > /* > -- > 1.7.4.1 > ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH] ARM: head: Remove boot-time HYP mode check for v5 and below 2012-12-10 10:57 ` Dave Martin @ 2012-12-10 17:39 ` Marko Katić 2012-12-10 18:02 ` Dave Martin 0 siblings, 1 reply; 7+ messages in thread From: Marko Katić @ 2012-12-10 17:39 UTC (permalink / raw) To: linux-arm-kernel On Mon, Dec 10, 2012 at 11:57 AM, Dave Martin <dave.martin@linaro.org> wrote: > Hi Russell, > > Do you want me to send this patch to the patch system? > > Until we have confirmation from Marko (or someone else with a board) > there's no way to know whether it fixes the problem. But it should > restore us back to the old behaviour older hardware, reducing the > likelihood of side issues. > > This could be reverted later if it proves to be unnecessary. > > Cheers > ---Dave > > On Fri, Dec 07, 2012 at 01:45:59PM +0000, Dave Martin wrote: >> The kernel can only be entered on HYP mode on CPUs which actually >> support it, i.e. >= ARMv7. pre-v6 platform support cannot coexist >> in the same kernel as support for v7 and higher, so there is no >> advantage in having the HYP mode check on pre-v6 hardware. >> >> At least one pre-v6 board is known to fail when the HYP mode check >> code is present, although the exact cause remains unknown and may >> be unrelated. [1] >> >> This patch restores the old behaviour for pre-v6 platforms, whereby >> the CPSR is forced directly to SVC mode with IRQs and FIQs masked. >> All kernels capable of booting on v7 hardware will retain the >> check, so this should not impair functionality. >> >> [1] http://lists.arm.linux.org.uk/lurker/message/20121130.013814.19218413.en.html >> ([ARM] head.S change broke platform device registration?) >> >> Signed-off-by: Dave Martin <dave.martin@linaro.org> >> --- >> arch/arm/include/asm/assembler.h | 8 ++++++++ >> 1 files changed, 8 insertions(+), 0 deletions(-) >> >> diff --git a/arch/arm/include/asm/assembler.h b/arch/arm/include/asm/assembler.h >> index 2ef9581..eb87200 100644 >> --- a/arch/arm/include/asm/assembler.h >> +++ b/arch/arm/include/asm/assembler.h >> @@ -250,6 +250,7 @@ >> * Beware, it also clobers LR. >> */ >> .macro safe_svcmode_maskall reg:req >> +#if __LINUX_ARM_ARCH__ >= 6 >> mrs \reg , cpsr >> mov lr , \reg >> and lr , lr , #MODE_MASK >> @@ -266,6 +267,13 @@ THUMB( orr \reg , \reg , #PSR_T_BIT ) >> __ERET >> 1: msr cpsr_c, \reg >> 2: >> +#else >> +/* >> + * workaround for possibly broken pre-v6 hardware >> + * (akita, Sharp Zaurus C-1000, PXA270-based) >> + */ >> + setmode PSR_F_BIT | PSR_I_BIT | SVC_MODE, \reg >> +#endif >> .endm >> >> /* >> -- >> 1.7.4.1 >> I confirm that this patch fixes the regression on Akita machines introduced with commit: 424e5994e63326a42012f003f1174f3c363c7b62 ARM: zImage/virt: hyp mode entry support for the zImage loader The abovementioned commit breaks Zaurus machine ID detection code (arm/boot/compressed/head-sharpsl.S) causing Akita machines to boot with the wrong machine ID. Just a minor nitpick: s/clobers/clobbers/g ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH] ARM: head: Remove boot-time HYP mode check for v5 and below 2012-12-10 17:39 ` Marko Katić @ 2012-12-10 18:02 ` Dave Martin 2012-12-10 18:08 ` Marc Zyngier 0 siblings, 1 reply; 7+ messages in thread From: Dave Martin @ 2012-12-10 18:02 UTC (permalink / raw) To: linux-arm-kernel On Mon, Dec 10, 2012 at 06:39:55PM +0100, Marko Kati? wrote: > On Mon, Dec 10, 2012 at 11:57 AM, Dave Martin <dave.martin@linaro.org> wrote: > > Hi Russell, > > > > Do you want me to send this patch to the patch system? > > > > Until we have confirmation from Marko (or someone else with a board) > > there's no way to know whether it fixes the problem. But it should > > restore us back to the old behaviour older hardware, reducing the > > likelihood of side issues. > > > > This could be reverted later if it proves to be unnecessary. > > > > Cheers > > ---Dave > > > > On Fri, Dec 07, 2012 at 01:45:59PM +0000, Dave Martin wrote: > >> The kernel can only be entered on HYP mode on CPUs which actually > >> support it, i.e. >= ARMv7. pre-v6 platform support cannot coexist > >> in the same kernel as support for v7 and higher, so there is no > >> advantage in having the HYP mode check on pre-v6 hardware. > >> > >> At least one pre-v6 board is known to fail when the HYP mode check > >> code is present, although the exact cause remains unknown and may > >> be unrelated. [1] > >> > >> This patch restores the old behaviour for pre-v6 platforms, whereby > >> the CPSR is forced directly to SVC mode with IRQs and FIQs masked. > >> All kernels capable of booting on v7 hardware will retain the > >> check, so this should not impair functionality. > >> > >> [1] http://lists.arm.linux.org.uk/lurker/message/20121130.013814.19218413.en.html > >> ([ARM] head.S change broke platform device registration?) > >> > >> Signed-off-by: Dave Martin <dave.martin@linaro.org> > >> --- > >> arch/arm/include/asm/assembler.h | 8 ++++++++ > >> 1 files changed, 8 insertions(+), 0 deletions(-) > >> > >> diff --git a/arch/arm/include/asm/assembler.h b/arch/arm/include/asm/assembler.h > >> index 2ef9581..eb87200 100644 > >> --- a/arch/arm/include/asm/assembler.h > >> +++ b/arch/arm/include/asm/assembler.h > >> @@ -250,6 +250,7 @@ > >> * Beware, it also clobers LR. > >> */ > >> .macro safe_svcmode_maskall reg:req > >> +#if __LINUX_ARM_ARCH__ >= 6 > >> mrs \reg , cpsr > >> mov lr , \reg > >> and lr , lr , #MODE_MASK > >> @@ -266,6 +267,13 @@ THUMB( orr \reg , \reg , #PSR_T_BIT ) > >> __ERET > >> 1: msr cpsr_c, \reg > >> 2: > >> +#else > >> +/* > >> + * workaround for possibly broken pre-v6 hardware > >> + * (akita, Sharp Zaurus C-1000, PXA270-based) > >> + */ > >> + setmode PSR_F_BIT | PSR_I_BIT | SVC_MODE, \reg > >> +#endif > >> .endm > >> > >> /* > >> -- > >> 1.7.4.1 > >> > > I confirm that this patch fixes the regression on Akita machines > introduced with commit: > > 424e5994e63326a42012f003f1174f3c363c7b62 > ARM: zImage/virt: hyp mode entry support for the zImage loader > > The abovementioned commit breaks Zaurus machine ID detection code > (arm/boot/compressed/head-sharpsl.S) > causing Akita machines to boot with the wrong machine ID. Thanks for testing. I've now posted the patch to the patch system. > Just a minor nitpick: > s/clobers/clobbers/g Good spot, but I'll leave the patch unchanged for now. This typo is in the existing code, not part of the fix. I'll try to tidy this up if I touch this code again. Cheers ---Dave ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH] ARM: head: Remove boot-time HYP mode check for v5 and below 2012-12-10 18:02 ` Dave Martin @ 2012-12-10 18:08 ` Marc Zyngier 2012-12-10 19:16 ` Will Deacon 0 siblings, 1 reply; 7+ messages in thread From: Marc Zyngier @ 2012-12-10 18:08 UTC (permalink / raw) To: linux-arm-kernel On 10/12/12 18:02, Dave Martin wrote: > On Mon, Dec 10, 2012 at 06:39:55PM +0100, Marko Kati? wrote: >> On Mon, Dec 10, 2012 at 11:57 AM, Dave Martin <dave.martin@linaro.org> wrote: >>> Hi Russell, >>> >>> Do you want me to send this patch to the patch system? >>> >>> Until we have confirmation from Marko (or someone else with a board) >>> there's no way to know whether it fixes the problem. But it should >>> restore us back to the old behaviour older hardware, reducing the >>> likelihood of side issues. >>> >>> This could be reverted later if it proves to be unnecessary. >>> >>> Cheers >>> ---Dave >>> >>> On Fri, Dec 07, 2012 at 01:45:59PM +0000, Dave Martin wrote: >>>> The kernel can only be entered on HYP mode on CPUs which actually >>>> support it, i.e. >= ARMv7. pre-v6 platform support cannot coexist >>>> in the same kernel as support for v7 and higher, so there is no >>>> advantage in having the HYP mode check on pre-v6 hardware. >>>> >>>> At least one pre-v6 board is known to fail when the HYP mode check >>>> code is present, although the exact cause remains unknown and may >>>> be unrelated. [1] >>>> >>>> This patch restores the old behaviour for pre-v6 platforms, whereby >>>> the CPSR is forced directly to SVC mode with IRQs and FIQs masked. >>>> All kernels capable of booting on v7 hardware will retain the >>>> check, so this should not impair functionality. >>>> >>>> [1] http://lists.arm.linux.org.uk/lurker/message/20121130.013814.19218413.en.html >>>> ([ARM] head.S change broke platform device registration?) >>>> >>>> Signed-off-by: Dave Martin <dave.martin@linaro.org> >>>> --- >>>> arch/arm/include/asm/assembler.h | 8 ++++++++ >>>> 1 files changed, 8 insertions(+), 0 deletions(-) >>>> >>>> diff --git a/arch/arm/include/asm/assembler.h b/arch/arm/include/asm/assembler.h >>>> index 2ef9581..eb87200 100644 >>>> --- a/arch/arm/include/asm/assembler.h >>>> +++ b/arch/arm/include/asm/assembler.h >>>> @@ -250,6 +250,7 @@ >>>> * Beware, it also clobers LR. >>>> */ >>>> .macro safe_svcmode_maskall reg:req >>>> +#if __LINUX_ARM_ARCH__ >= 6 >>>> mrs \reg , cpsr >>>> mov lr , \reg >>>> and lr , lr , #MODE_MASK >>>> @@ -266,6 +267,13 @@ THUMB( orr \reg , \reg , #PSR_T_BIT ) >>>> __ERET >>>> 1: msr cpsr_c, \reg >>>> 2: >>>> +#else >>>> +/* >>>> + * workaround for possibly broken pre-v6 hardware >>>> + * (akita, Sharp Zaurus C-1000, PXA270-based) >>>> + */ >>>> + setmode PSR_F_BIT | PSR_I_BIT | SVC_MODE, \reg >>>> +#endif >>>> .endm >>>> >>>> /* >>>> -- >>>> 1.7.4.1 >>>> >> >> I confirm that this patch fixes the regression on Akita machines >> introduced with commit: >> >> 424e5994e63326a42012f003f1174f3c363c7b62 >> ARM: zImage/virt: hyp mode entry support for the zImage loader >> >> The abovementioned commit breaks Zaurus machine ID detection code >> (arm/boot/compressed/head-sharpsl.S) >> causing Akita machines to boot with the wrong machine ID. > > Thanks for testing. I've now posted the patch to the patch system. > >> Just a minor nitpick: >> s/clobers/clobbers/g > > Good spot, but I'll leave the patch unchanged for now. > > This typo is in the existing code, not part of the fix. > I'll try to tidy this up if I touch this code again. I think Will already carries a patch authored by RMK, getting rid of the use of LR and of my spelling mistake. M. -- Jazz is not dead. It just smells funny... ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH] ARM: head: Remove boot-time HYP mode check for v5 and below 2012-12-10 18:08 ` Marc Zyngier @ 2012-12-10 19:16 ` Will Deacon 2012-12-11 11:05 ` Dave Martin 0 siblings, 1 reply; 7+ messages in thread From: Will Deacon @ 2012-12-10 19:16 UTC (permalink / raw) To: linux-arm-kernel On Mon, Dec 10, 2012 at 06:08:16PM +0000, Marc Zyngier wrote: > On 10/12/12 18:02, Dave Martin wrote: > > This typo is in the existing code, not part of the fix. > > I'll try to tidy this up if I touch this code again. > > I think Will already carries a patch authored by RMK, getting rid of the > use of LR and of my spelling mistake. Yup, I was planning to post a patch series after the merge window which removed the CONFIG_ARM_VIRT_EXT option entirely. However, given the Zaurus breakage I'll rework that series to default the option to y and hide it from the user, so this patch will ultimately disappear (but is a good short-term fix). Will ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH] ARM: head: Remove boot-time HYP mode check for v5 and below 2012-12-10 19:16 ` Will Deacon @ 2012-12-11 11:05 ` Dave Martin 0 siblings, 0 replies; 7+ messages in thread From: Dave Martin @ 2012-12-11 11:05 UTC (permalink / raw) To: linux-arm-kernel On Mon, Dec 10, 2012 at 07:16:11PM +0000, Will Deacon wrote: > On Mon, Dec 10, 2012 at 06:08:16PM +0000, Marc Zyngier wrote: > > On 10/12/12 18:02, Dave Martin wrote: > > > This typo is in the existing code, not part of the fix. > > > I'll try to tidy this up if I touch this code again. > > > > I think Will already carries a patch authored by RMK, getting rid of the > > use of LR and of my spelling mistake. > > Yup, I was planning to post a patch series after the merge window which > removed the CONFIG_ARM_VIRT_EXT option entirely. However, given the Zaurus > breakage I'll rework that series to default the option to y and hide it from > the user, so this patch will ultimately disappear (but is a good short-term > fix). Jolly good ---Dave ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2012-12-11 11:05 UTC | newest] Thread overview: 7+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2012-12-07 13:45 [PATCH] ARM: head: Remove boot-time HYP mode check for v5 and below Dave Martin 2012-12-10 10:57 ` Dave Martin 2012-12-10 17:39 ` Marko Katić 2012-12-10 18:02 ` Dave Martin 2012-12-10 18:08 ` Marc Zyngier 2012-12-10 19:16 ` Will Deacon 2012-12-11 11:05 ` Dave Martin
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).