mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: "Adrien Vergé" <adrienverge@gmail.com>
To: Russell King <linux@arm.linux.org.uk>
Cc: "Adrien Vergé" <adrienverge@gmail.com>,
	"Catalin Marinas" <catalin.marinas@arm.com>,
	"Will Deacon" <will.deacon@arm.com>,
	"Ben Dooks" <ben.dooks@codethink.co.uk>,
	"zhangwei(Jovi)" <jovi.zhangwei@huawei.com>,
	"Andrew Morton" <akpm@linux-foundation.org>,
	"Randy Dunlap" <rdunlap@infradead.org>,
	"Mathieu Poirier" <mathieu.poirier@linaro.org>,
	"Christopher Covington" <cov@codeaurora.org>,
	"Dirk Behme" <dirk.behme@de.bosch.com>,
	"Michel Dagenais" <michel.dagenais@polymtl.ca>,
	linux-arm-kernel@lists.infradead.org,
	linux-kernel@vger.kernel.org
Subject: [PATCH V2 4/6] ARM: Make PID_IN_CONTEXTIDR incompatible with PID_NS
Date: Fri, 24 Jan 2014 11:40:54 -0500	[thread overview]
Message-ID: <1390581656-16372-5-git-send-email-adrienverge@gmail.com> (raw)
In-Reply-To: <1390581656-16372-1-git-send-email-adrienverge@gmail.com>

When using namespaces, different processes can have the same PID.
It makes no sense to store a PID value in the Context ID register
to track a specific process, when others share the same value.

Consequently, PID_IN_CONTEXTIDR (which is used for tracing and
debugging processes) should not be compatible with PID_NS.

Signed-off-by: Adrien Vergé <adrienverge@gmail.com>
---
 arch/arm/Kconfig.debug   | 2 +-
 arch/arm64/Kconfig.debug | 1 +
 2 files changed, 2 insertions(+), 1 deletion(-)

diff --git a/arch/arm/Kconfig.debug b/arch/arm/Kconfig.debug
index 5765abf..ed46748 100644
--- a/arch/arm/Kconfig.debug
+++ b/arch/arm/Kconfig.debug
@@ -1143,7 +1143,7 @@ config ARM_KPROBES_TEST
 
 config PID_IN_CONTEXTIDR
 	bool "Write the current PID to the CONTEXTIDR register"
-	depends on CPU_COPY_V6
+	depends on CPU_COPY_V6 && !PID_NS
 	help
 	  Enabling this option causes the kernel to write the current PID to
 	  the PROCID field of the CONTEXTIDR register, at the expense of some
diff --git a/arch/arm64/Kconfig.debug b/arch/arm64/Kconfig.debug
index 835c559..06b2633b 100644
--- a/arch/arm64/Kconfig.debug
+++ b/arch/arm64/Kconfig.debug
@@ -17,6 +17,7 @@ config EARLY_PRINTK
 
 config PID_IN_CONTEXTIDR
 	bool "Write the current PID to the CONTEXTIDR register"
+	depends on !PID_NS
 	help
 	  Enabling this option causes the kernel to write the current PID to
 	  the CONTEXTIDR register, at the expense of some additional
-- 
1.8.5.2


  parent reply	other threads:[~2014-01-24 16:41 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-01-24 16:40 [PATCH V2 0/6] ARM CoreSight: Enhance ETM tracing control Adrien Vergé
2014-01-24 16:40 ` [PATCH V2 1/6] ARM CoreSight: ETM: Use device attributes Adrien Vergé
2014-01-24 16:40 ` [PATCH V2 2/6] ARM CoreSight: ETM: Rename 'comparator' to 'address comparator' Adrien Vergé
2014-01-24 16:40 ` [PATCH V2 3/6] ARM CoreSight: ETM: Add address control support Adrien Vergé
2014-01-24 16:40 ` Adrien Vergé [this message]
2014-01-24 16:43   ` [PATCH V2 4/6] ARM: Make PID_IN_CONTEXTIDR incompatible with PID_NS Will Deacon
2014-01-24 17:16     ` Adrien Vergé
2014-01-24 17:17       ` Will Deacon
2014-01-24 17:52         ` Christopher Covington
2014-01-24 19:12           ` Christopher Covington
2014-01-24 19:34           ` Adrien Vergé
2014-01-24 16:40 ` [PATCH V2 5/6] ARM CoreSight: ETM: Add PID control support Adrien Vergé
2014-01-24 16:40 ` [PATCH V2 6/6] ARM CoreSight: ETM: Allocate a trace buffer only when necessary Adrien Vergé

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=1390581656-16372-5-git-send-email-adrienverge@gmail.com \
    --to=adrienverge@gmail.com \
    --cc=akpm@linux-foundation.org \
    --cc=ben.dooks@codethink.co.uk \
    --cc=catalin.marinas@arm.com \
    --cc=cov@codeaurora.org \
    --cc=dirk.behme@de.bosch.com \
    --cc=jovi.zhangwei@huawei.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux@arm.linux.org.uk \
    --cc=mathieu.poirier@linaro.org \
    --cc=michel.dagenais@polymtl.ca \
    --cc=rdunlap@infradead.org \
    --cc=will.deacon@arm.com \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®