linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Khalid Aziz <khalid.aziz@oracle.com>
To: davem@davemloft.net, dave.hansen@linux.intel.com
Cc: Khalid Aziz <khalid.aziz@oracle.com>,
	rob.gardner@oracle.com, dan.carpenter@oracle.com,
	Liam.Howlett@oracle.com, steven.sistare@oracle.com,
	jane.chu@oracle.com, paul.gortmaker@windriver.com,
	sparclinux@vger.kernel.org, linux-kernel@vger.kernel.org,
	Khalid Aziz <khalid@gonehiking.org>
Subject: [PATCH v10 04/10] sparc64: Add HV fault type handlers for ADI related faults
Date: Wed, 15 Nov 2017 15:46:19 -0700	[thread overview]
Message-ID: <792c7e2318e90ea70b052164990eaeb26cccf0ce.1510768775.git.khalid.aziz@oracle.com> (raw)
In-Reply-To: <cover.1510768775.git.khalid.aziz@oracle.com>
In-Reply-To: <cover.1510768775.git.khalid.aziz@oracle.com>

ADI (Application Data Integrity) feature on M7 and newer processors
adds new fault types for hypervisor - Invalid ASI and MCD disabled.
This patch expands data access exception handler to handle these
faults.

Signed-off-by: Khalid Aziz <khalid.aziz@oracle.com>
Cc: Khalid Aziz <khalid@gonehiking.org>
---
v7:
	- new patch split off from patch 4/4 in v6

 arch/sparc/kernel/traps_64.c | 29 ++++++++++++++++++++++++++---
 1 file changed, 26 insertions(+), 3 deletions(-)

diff --git a/arch/sparc/kernel/traps_64.c b/arch/sparc/kernel/traps_64.c
index fc73baa588f6..d273a65a0a10 100644
--- a/arch/sparc/kernel/traps_64.c
+++ b/arch/sparc/kernel/traps_64.c
@@ -397,12 +397,35 @@ void sun4v_data_access_exception(struct pt_regs *regs, unsigned long addr, unsig
 	if (is_no_fault_exception(regs))
 		return;
 
-	info.si_signo = SIGSEGV;
+	/* MCD (Memory Corruption Detection) disabled trap (TT=0x19) in HV
+	 * is vectored thorugh data access exception trap with fault type
+	 * set to HV_FAULT_TYPE_MCD_DIS. Check for MCD disabled trap.
+	 * Accessing an address with invalid ASI for the address, for
+	 * example setting an ADI tag on an address with ASI_MCD_PRIMARY
+	 * when TTE.mcd is not set for the VA, is also vectored into
+	 * kerbel by HV as data access exception with fault type set to
+	 * HV_FAULT_TYPE_INV_ASI.
+	 */
 	info.si_errno = 0;
-	info.si_code = SEGV_MAPERR;
 	info.si_addr = (void __user *) addr;
 	info.si_trapno = 0;
-	force_sig_info(SIGSEGV, &info, current);
+	switch (type) {
+	case HV_FAULT_TYPE_INV_ASI:
+		info.si_signo = SIGILL;
+		info.si_code = ILL_ILLADR;
+		force_sig_info(SIGILL, &info, current);
+		break;
+	case HV_FAULT_TYPE_MCD_DIS:
+		info.si_signo = SIGSEGV;
+		info.si_code = SEGV_ACCADI;
+		force_sig_info(SIGSEGV, &info, current);
+		break;
+	default:
+		info.si_signo = SIGSEGV;
+		info.si_code = SEGV_MAPERR;
+		force_sig_info(SIGSEGV, &info, current);
+		break;
+	}
 }
 
 void sun4v_data_access_exception_tl1(struct pt_regs *regs, unsigned long addr, unsigned long type_ctx)
-- 
2.11.0

  parent reply	other threads:[~2017-11-15 22:48 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-11-15 22:46 [PATCH v10 00/10] Application Data Integrity feature introduced by SPARC M7 Khalid Aziz
2017-11-15 22:46 ` [PATCH v10 01/10] signals, sparc: Add signal codes for ADI violations Khalid Aziz
2017-11-15 22:46 ` [PATCH v10 02/10] mm, swap: Add infrastructure for saving page metadata as well on swap Khalid Aziz
2017-11-15 22:46 ` [PATCH v10 03/10] sparc64: Add support for ADI register fields, ASIs and traps Khalid Aziz
2017-11-15 22:46 ` Khalid Aziz [this message]
2017-11-15 22:46 ` [PATCH v10 05/10] sparc64: Add handler for "Memory Corruption Detected" trap Khalid Aziz
2017-11-15 22:46 ` [PATCH v10 06/10] sparc64: Add auxiliary vectors to report platform ADI properties Khalid Aziz
2017-11-15 22:46 ` [PATCH v10 07/10] mm: Add address parameter to arch_validate_prot() Khalid Aziz
2017-11-15 22:46 ` [PATCH v10 08/10] mm: Clear arch specific VM flags on protection change Khalid Aziz
2017-11-15 22:46 ` [PATCH v10 09/10] mm: Allow arch code to override copy_highpage() Khalid Aziz
2017-11-15 22:46 ` [PATCH v10 10/10] sparc64: Add support for ADI (Application Data Integrity) Khalid Aziz
2017-11-17  2:14 ` [PATCH RESEND v10 00/10] Application Data Integrity feature introduced by SPARC M7 Anthony Yznaga

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=792c7e2318e90ea70b052164990eaeb26cccf0ce.1510768775.git.khalid.aziz@oracle.com \
    --to=khalid.aziz@oracle.com \
    --cc=Liam.Howlett@oracle.com \
    --cc=dan.carpenter@oracle.com \
    --cc=dave.hansen@linux.intel.com \
    --cc=davem@davemloft.net \
    --cc=jane.chu@oracle.com \
    --cc=khalid@gonehiking.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=paul.gortmaker@windriver.com \
    --cc=rob.gardner@oracle.com \
    --cc=sparclinux@vger.kernel.org \
    --cc=steven.sistare@oracle.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;
as well as URLs for NNTP newsgroup(s).