All of lore.kernel.org
 help / color / mirror / Atom feed
From: s.gottschall@dd-wrt.com (Sebastian Gottschall)
To: linux-arm-kernel@lists.infradead.org
Subject: arm: ARM_VIRT_EXT is not supported by all v7 platforms, so it should not be enabled by default
Date: Fri, 19 Apr 2013 20:07:15 +0200	[thread overview]
Message-ID: <kks14c$e4p$1@ger.gmane.org> (raw)

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

WARNING: multiple messages have this Message-ID (diff)
From: Sebastian Gottschall <s.gottschall@dd-wrt.com>
To: linux-kernel@vger.kernel.org
Cc: linux-arm-kernel@lists.infradead.org
Subject: arm: ARM_VIRT_EXT is not supported by all v7 platforms, so it should not be enabled by default
Date: Fri, 19 Apr 2013 20:07:15 +0200	[thread overview]
Message-ID: <kks14c$e4p$1@ger.gmane.org> (raw)

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



             reply	other threads:[~2013-04-19 18:07 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-04-19 18:07 Sebastian Gottschall [this message]
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-22 10:46 ` Will Deacon
2013-04-22 10:46   ` Will Deacon

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='kks14c$e4p$1@ger.gmane.org' \
    --to=s.gottschall@dd-wrt.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.