Re: [PATCH v4 0/4] Application Data Integrity feature introduced by SPARC M7

From	Dave Hansen <dave.hansen@linux.intel.com>
Newsgroups	linux.kernel
Subject	Re: [PATCH v4 0/4] Application Data Integrity feature introduced by SPARC M7
Date	2017-01-11 17:40 +0100
Message-ID	<sYu6l-Lo-1@gated-at.bofh.it> (permalink)
References	<sYtN0-Fd-21@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

On 01/11/2017 08:12 AM, Khalid Aziz wrote:
> A userspace task enables ADI through mprotect(). This patch series adds
> a page protection bit PROT_ADI and a corresponding VMA flag
> VM_SPARC_ADI. VM_SPARC_ADI is used to trigger setting TTE.mcd bit in the
> sparc pte that enables ADI checking on the corresponding page.

Is there a cost in the hardware associated with doing this "ADI
checking"?  For instance, instead of having this new mprotect()
interface, why not just always set TTE.mcd on all PTEs?

Also, should this be a privileged interface in some way?  The hardware
is storing these tags *somewhere* and that storage is consuming
resources *somewhere*.  What stops a crafty attacker from mmap()'ing a
128TB chunk of the zero pages and storing ADI tags for all of it?
That'll be 128TB/64*4bits = 1TB worth of 4-bit tags.  Page tables, for
instance, consume a comparable amount of storage, but the OS *knows*
about those and can factor them into OOM decisions.

Thread

[PATCH v4 0/4] Application Data Integrity feature introduced by SPARC M7 Khalid Aziz <khalid.aziz@oracle.com> - 2017-01-11 17:20 +0100
  [PATCH v4 1/4] signals, sparc: Add signal codes for ADI violations Khalid Aziz <khalid.aziz@oracle.com> - 2017-01-11 17:20 +0100
  Re: [PATCH v4 0/4] Application Data Integrity feature introduced by  SPARC M7 Dave Hansen <dave.hansen@linux.intel.com> - 2017-01-11 17:40 +0100
    Re: [PATCH v4 0/4] Application Data Integrity feature introduced by  SPARC M7 Khalid Aziz <khalid.aziz@oracle.com> - 2017-01-11 18:00 +0100
      Re: [PATCH v4 0/4] Application Data Integrity feature introduced by  SPARC M7 Dave Hansen <dave.hansen@linux.intel.com> - 2017-01-11 19:20 +0100
        Re: [PATCH v4 0/4] Application Data Integrity feature introduced by  SPARC M7 Khalid Aziz <khalid.aziz@oracle.com> - 2017-01-11 20:00 +0100
          Re: [PATCH v4 0/4] Application Data Integrity feature introduced by  SPARC M7 Dave Hansen <dave.hansen@linux.intel.com> - 2017-01-11 20:20 +0100
            Re: [PATCH v4 0/4] Application Data Integrity feature introduced by  SPARC M7 Khalid Aziz <khalid.aziz@oracle.com> - 2017-01-12 01:30 +0100
              Re: [PATCH v4 0/4] Application Data Integrity feature introduced by  SPARC M7 Dave Hansen <dave.hansen@linux.intel.com> - 2017-01-12 01:50 +0100
                Re: [PATCH v4 0/4] Application Data Integrity feature introduced by  SPARC M7 Khalid Aziz <khalid.aziz@oracle.com> - 2017-01-12 18:00 +0100
                Re: [PATCH v4 0/4] Application Data Integrity feature introduced by  SPARC M7 Dave Hansen <dave.hansen@linux.intel.com> - 2017-01-12 19:00 +0100
                Re: [PATCH v4 0/4] Application Data Integrity feature introduced by  SPARC M7 Khalid Aziz <khalid.aziz@oracle.com> - 2017-01-13 01:30 +0100
                Re: [PATCH v4 0/4] Application Data Integrity feature introduced by  SPARC M7 Rob Gardner <rob.gardner@oracle.com> - 2017-01-13 02:40 +0100
                Re: [PATCH v4 0/4] Application Data Integrity feature introduced by  SPARC M7 Khalid Aziz <khalid.aziz@oracle.com> - 2017-01-13 15:50 +0100
                Re: [PATCH v4 0/4] Application Data Integrity feature introduced by  SPARC M7 Rob Gardner <rob.gardner@oracle.com> - 2017-01-13 16:40 +0100
                Re: [PATCH v4 0/4] Application Data Integrity feature introduced by  SPARC M7 Dave Hansen <dave.hansen@linux.intel.com> - 2017-01-13 17:10 +0100
                Re: [PATCH v4 0/4] Application Data Integrity feature introduced by  SPARC M7 Rob Gardner <rob.gardner@oracle.com> - 2017-01-13 18:40 +0100
                Re: [PATCH v4 0/4] Application Data Integrity feature introduced by  SPARC M7 Khalid Aziz <khalid.aziz@oracle.com> - 2017-01-13 17:10 +0100

csiph-web