* arm: ARM_VIRT_EXT is not supported by all v7 platforms, so it should not be enabled by default
@ 2013-04-19 18:07 ` Sebastian Gottschall
0 siblings, 0 replies; 4+ messages in thread
From: Sebastian Gottschall @ 2013-04-19 18:07 UTC (permalink / raw)
To: linux-arm-kernel
introduced in kernel 3.9 CONFIG_ARM_VIRT_EXT is default for all V7 arm
cpu's. this is wrong and breaks smp support on BCM4708 for example.
so keep it optional since no all v7 cpu's seem to support it. BCM4708
for instance is a arm cortex-a9. please merge this into one of the next
patches.
Index: arch/arm/mm/Kconfig
===================================================================
--- arch/arm/mm/Kconfig (revision 21211)
+++ arch/arm/mm/Kconfig (working copy)
@@ -640,8 +640,7 @@
config ARM_VIRT_EXT
bool
- depends on MMU
- default y if CPU_V7
+ depends on MMU && CPU_V7
help
Enable the kernel to make use of the ARM Virtualization
Extensions to install hypervisors without run-time firmware
^ permalink raw reply [flat|nested] 4+ messages in thread* arm: ARM_VIRT_EXT is not supported by all v7 platforms, so it should not be enabled by default
@ 2013-04-19 18:07 ` Sebastian Gottschall
0 siblings, 0 replies; 4+ messages in thread
From: Sebastian Gottschall @ 2013-04-19 18:07 UTC (permalink / raw)
To: linux-kernel; +Cc: linux-arm-kernel
introduced in kernel 3.9 CONFIG_ARM_VIRT_EXT is default for all V7 arm
cpu's. this is wrong and breaks smp support on BCM4708 for example.
so keep it optional since no all v7 cpu's seem to support it. BCM4708
for instance is a arm cortex-a9. please merge this into one of the next
patches.
Index: arch/arm/mm/Kconfig
===================================================================
--- arch/arm/mm/Kconfig (revision 21211)
+++ arch/arm/mm/Kconfig (working copy)
@@ -640,8 +640,7 @@
config ARM_VIRT_EXT
bool
- depends on MMU
- default y if CPU_V7
+ depends on MMU && CPU_V7
help
Enable the kernel to make use of the ARM Virtualization
Extensions to install hypervisors without run-time firmware
^ permalink raw reply [flat|nested] 4+ messages in thread* arm: ARM_VIRT_EXT is not supported by all v7 platforms, so it should not be enabled by default
2013-04-19 18:07 ` Sebastian Gottschall
@ 2013-04-22 10:46 ` Will Deacon
-1 siblings, 0 replies; 4+ messages in thread
From: Will Deacon @ 2013-04-22 10:46 UTC (permalink / raw)
To: linux-arm-kernel
On Fri, Apr 19, 2013 at 07:07:15PM +0100, Sebastian Gottschall wrote:
> introduced in kernel 3.9 CONFIG_ARM_VIRT_EXT is default for all V7 arm
> cpu's. this is wrong and breaks smp support on BCM4708 for example.
> so keep it optional since no all v7 cpu's seem to support it. BCM4708
> for instance is a arm cortex-a9. please merge this into one of the next
> patches.
This was intentional, since ARM_VIRT_EXT shouldn't have any visible effect
when selected on CPUs without the virtualisation extensions.
If this is breaking SMP support, then there is a bug somewhere, so please
can elaborate on the problems which you are seeing?
Cheers,
Will
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: arm: ARM_VIRT_EXT is not supported by all v7 platforms, so it should not be enabled by default
@ 2013-04-22 10:46 ` Will Deacon
0 siblings, 0 replies; 4+ messages in thread
From: Will Deacon @ 2013-04-22 10:46 UTC (permalink / raw)
To: Sebastian Gottschall
Cc: linux-arm-kernel@lists.infradead.org,
linux-kernel@vger.kernel.org, marc.zyngier, dave.martin
On Fri, Apr 19, 2013 at 07:07:15PM +0100, Sebastian Gottschall wrote:
> introduced in kernel 3.9 CONFIG_ARM_VIRT_EXT is default for all V7 arm
> cpu's. this is wrong and breaks smp support on BCM4708 for example.
> so keep it optional since no all v7 cpu's seem to support it. BCM4708
> for instance is a arm cortex-a9. please merge this into one of the next
> patches.
This was intentional, since ARM_VIRT_EXT shouldn't have any visible effect
when selected on CPUs without the virtualisation extensions.
If this is breaking SMP support, then there is a bug somewhere, so please
can elaborate on the problems which you are seeing?
Cheers,
Will
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2013-04-22 10:47 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-04-19 18:07 arm: ARM_VIRT_EXT is not supported by all v7 platforms, so it should not be enabled by default Sebastian Gottschall
2013-04-19 18:07 ` Sebastian Gottschall
2013-04-22 10:46 ` Will Deacon
2013-04-22 10:46 ` Will Deacon
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.