Re: [PATCH v8 9/9] sparc64: Add support for ADI (Application Data Integrity)

2017-10-13 Thread Anthony Yznaga
> On Oct 13, 2017, at 9:18 AM, Khalid Aziz wrote: > > On 10/13/2017 08:14 AM, Khalid Aziz wrote: >> On 10/12/2017 02:27 PM, Anthony Yznaga wrote: >>> On Oct 12, 2017, at 7:44 AM, Khalid Aziz wrote: On 10/06/2017 04:12 PM, Anthony Yznaga wrote: >> On Sep 25, 2017, at 9

Re: [PATCH v8 9/9] sparc64: Add support for ADI (Application Data Integrity)

2017-10-13 Thread Khalid Aziz
On 10/13/2017 08:14 AM, Khalid Aziz wrote: On 10/12/2017 02:27 PM, Anthony Yznaga wrote: On Oct 12, 2017, at 7:44 AM, Khalid Aziz wrote: On 10/06/2017 04:12 PM, Anthony Yznaga wrote: On Sep 25, 2017, at 9:49 AM, Khalid Aziz wrote: This patch extends mprotect to enable ADI (TSTATE.mcde),

Re: [PATCH v8 9/9] sparc64: Add support for ADI (Application Data Integrity)

2017-10-13 Thread Khalid Aziz
On 10/12/2017 02:27 PM, Anthony Yznaga wrote: On Oct 12, 2017, at 7:44 AM, Khalid Aziz wrote: On 10/06/2017 04:12 PM, Anthony Yznaga wrote: On Sep 25, 2017, at 9:49 AM, Khalid Aziz wrote: This patch extends mprotect to enable ADI (TSTATE.mcde), enable/disable MCD (Memory Corruption Detect

Re: [PATCH v8 9/9] sparc64: Add support for ADI (Application Data Integrity)

2017-10-12 Thread Anthony Yznaga
> On Oct 12, 2017, at 7:44 AM, Khalid Aziz wrote: > > Hi Anthony, > > Please quote only the relevant parts of the patch with comments. That makes > it much easier to find the comments. Okay. > > On 10/06/2017 04:12 PM, Anthony Yznaga wrote: >>> On Sep 25, 2017, at 9:49 AM, Khalid Aziz wrot

Re: [PATCH v8 9/9] sparc64: Add support for ADI (Application Data Integrity)

2017-10-12 Thread Khalid Aziz
Hi Anthony, Please quote only the relevant parts of the patch with comments. That makes it much easier to find the comments. On 10/06/2017 04:12 PM, Anthony Yznaga wrote: On Sep 25, 2017, at 9:49 AM, Khalid Aziz wrote: This patch extends mprotect to enable ADI (TSTATE.mcde), enable/disabl

Re: [PATCH v8 9/9] sparc64: Add support for ADI (Application Data Integrity)

2017-10-06 Thread Anthony Yznaga
> On Sep 25, 2017, at 9:49 AM, Khalid Aziz wrote: > > ADI is a new feature supported on SPARC M7 and newer processors to allow > hardware to catch rogue accesses to memory. ADI is supported for data > fetches only and not instruction fetches. An app can enable ADI on its > data pages, set versio

[PATCH v8 9/9] sparc64: Add support for ADI (Application Data Integrity)

2017-09-25 Thread Khalid Aziz
ADI is a new feature supported on SPARC M7 and newer processors to allow hardware to catch rogue accesses to memory. ADI is supported for data fetches only and not instruction fetches. An app can enable ADI on its data pages, set version tags on them and use versioned addresses to access the data p