linux-mm.kvack.org archive mirror
 help / color / mirror / Atom feed
From: Khalid Aziz <khalid.aziz@oracle.com>
To: David Miller <davem@davemloft.net>
Cc: dave.hansen@linux.intel.com, luto@amacapital.net,
	rob.gardner@oracle.com, corbet@lwn.net,
	akpm@linux-foundation.org, dingel@linux.vnet.ibm.com,
	bob.picco@oracle.com, kirill.shutemov@linux.intel.com,
	aneesh.kumar@linux.vnet.ibm.com, aarcange@redhat.com,
	arnd@arndb.de, sparclinux@vger.kernel.org, mhocko@suse.cz,
	chris.hyser@oracle.com, richard@nod.at, vbabka@suse.cz,
	koct9i@gmail.com, oleg@redhat.com, gthelen@google.com,
	jack@suse.cz, xiexiuqi@huawei.com, Vineet.Gupta1@synopsys.com,
	luto@kernel.org, ebiederm@xmission.com, bsegall@google.com,
	geert@linux-m68k.org, dave@stgolabs.net, adobriyan@gmail.com,
	linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org,
	linux-mm@kvack.org, linux-arch@vger.kernel.org,
	linux-api@vger.kernel.org
Subject: Re: [PATCH v2] sparc64: Add support for Application Data Integrity (ADI)
Date: Tue, 8 Mar 2016 13:59:58 -0700	[thread overview]
Message-ID: <56DF3D4E.3090501@oracle.com> (raw)
In-Reply-To: <20160308.152746.1746115669072714849.davem@davemloft.net>

On 03/08/2016 01:27 PM, David Miller wrote:
> From: Khalid Aziz <khalid.aziz@oracle.com>
> Date: Tue, 8 Mar 2016 13:16:11 -0700
>
>> On 03/08/2016 12:57 PM, David Miller wrote:
>>> From: Khalid Aziz <khalid.aziz@oracle.com>
>>> Date: Mon, 7 Mar 2016 14:06:43 -0700
>>>
>>>> Good questions. Isn't set of valid VAs already constrained by VA_BITS
>>>> (set to 44 in arch/sparc/include/asm/processor_64.h)? As I see it we
>>>> are already not using the top 4 bits. Please correct me if I am wrong.
>>>
>>> Another limiting constraint is the number of address bits coverable by
>>> the 4-level page tables we use.  And this is sign extended so we have
>>> a top-half and a bottom-half with a "hole" in the center of the VA
>>> space.
>>>
>>> I want some clarification on the top bits during ADI accesses.
>>>
>>> If ADI is enabled, then the top bits of the virtual address are
>>> intepreted as tag bits.  Once "verified" with the ADI settings, what
>>> happense to these tag bits?  Are they dropped from the virtual address
>>> before being passed down the TLB et al. for translations?
>>
>> Bits 63-60 (tag bits) are dropped from the virtual address before
>> being passed down the TLB for translation when PSTATE.mcde = 1.
>
> Ok and you said that values 15 and 0 are special.
>
> I'm just wondering if this means you can't really use ADI mappings in
> the top half of the 64-bit address space.  If the bits are dropped, they
> will be zero, but they need to be all 1's for the top-half of the VA
> space since it's sign extended.
>

According to the manual when PSTATE.mcde=1, bits 63:60 of the virtual 
address of any load or store (using virtual address) are masked before 
being sent to memory system which includes MMU. Hardware TSB walker 
masks bits 63:60 and then sign extends from bit 59 before generating TSB 
pointer and before comparison to TSB TTE VAs but the virtual address in 
the TTE tag that is written to DTLB is masked and not sign extended. 
Manual also states that for implementations that fully support 60 bits 
or more of virtual address, they must sign-extend virtual address in TSB 
TTE tag.

--
Khalid

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

  reply	other threads:[~2016-03-08 21:00 UTC|newest]

Thread overview: 62+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-03-02 20:39 [PATCH v2] sparc64: Add support for Application Data Integrity (ADI) Khalid Aziz
2016-03-02 23:08 ` Julian Calaby
2016-03-03  0:25   ` Khalid Aziz
2016-03-03  0:48     ` Julian Calaby
2016-03-03 17:28       ` Khalid Aziz
2016-03-06  4:07 ` David Miller
2016-03-07 15:07   ` Khalid Aziz
2016-03-07 15:30     ` Rob Gardner
2016-03-07 15:43       ` Andy Lutomirski
2016-03-07 16:06         ` Khalid Aziz
2016-03-07 17:46           ` Dave Hansen
2016-03-07 17:53             ` Andy Lutomirski
2016-03-07 18:12               ` Dave Hansen
2016-03-07 18:39                 ` Khalid Aziz
2016-03-07 18:53                   ` Andy Lutomirski
2016-03-07 19:22                     ` David Miller
2016-03-07 19:46                       ` Khalid Aziz
2016-03-07 22:40                         ` Dave Hansen
2016-03-08  1:31                   ` Rob Gardner
2016-03-07 21:06             ` Khalid Aziz
2016-03-08 19:57               ` David Miller
2016-03-08 20:16                 ` Khalid Aziz
2016-03-08 20:27                   ` David Miller
2016-03-08 20:59                     ` Khalid Aziz [this message]
2016-03-07 15:45       ` Khalid Aziz
2016-03-07 16:45     ` David Miller
2016-03-07 17:51       ` Khalid Aziz
2016-03-07 16:56     ` David Miller
2016-03-07 18:04       ` Khalid Aziz
2016-03-07 18:08         ` Andy Lutomirski
2016-03-07 18:22           ` Khalid Aziz
2016-03-07 18:49             ` Andy Lutomirski
2016-03-07 19:19               ` David Miller
2016-03-07 19:44               ` Khalid Aziz
2016-03-07 19:54                 ` Andy Lutomirski
2016-03-07 20:41                   ` Khalid Aziz
2016-03-07 20:58                     ` David Miller
2016-03-07 21:02                       ` Andy Lutomirski
2016-03-07 21:09                       ` Khalid Aziz
2016-03-07 23:34                       ` James Morris
2016-03-07 23:48                   ` James Morris
2016-03-08  9:33                     ` James Morris
2016-03-07 18:09         ` Rob Gardner
2016-03-07 18:24           ` Khalid Aziz
2016-03-07 19:16             ` David Miller
2016-03-07 21:33               ` Khalid Aziz
2016-03-07 21:38                 ` David Miller
2016-03-07 23:13                   ` Rob Gardner
2016-03-08  4:13                     ` David Miller
2016-03-07 23:12                 ` Rob Gardner
2016-03-07 23:27                   ` Khalid Aziz
2016-03-08  0:21               ` Khalid Aziz
2016-03-08  4:24                 ` David Miller
2016-03-07 23:32             ` Rob Gardner
2016-03-07 19:09         ` David Miller
2016-03-07 21:27           ` Khalid Aziz
2016-03-07 21:34             ` David Miller
2016-03-07 22:30               ` Khalid Aziz
2016-03-07 17:32 ` Dave Hansen
2016-03-07 17:35 ` Dave Hansen
2016-03-07 18:15   ` Khalid Aziz
2016-03-07 19:06   ` David Miller

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=56DF3D4E.3090501@oracle.com \
    --to=khalid.aziz@oracle.com \
    --cc=Vineet.Gupta1@synopsys.com \
    --cc=aarcange@redhat.com \
    --cc=adobriyan@gmail.com \
    --cc=akpm@linux-foundation.org \
    --cc=aneesh.kumar@linux.vnet.ibm.com \
    --cc=arnd@arndb.de \
    --cc=bob.picco@oracle.com \
    --cc=bsegall@google.com \
    --cc=chris.hyser@oracle.com \
    --cc=corbet@lwn.net \
    --cc=dave.hansen@linux.intel.com \
    --cc=dave@stgolabs.net \
    --cc=davem@davemloft.net \
    --cc=dingel@linux.vnet.ibm.com \
    --cc=ebiederm@xmission.com \
    --cc=geert@linux-m68k.org \
    --cc=gthelen@google.com \
    --cc=jack@suse.cz \
    --cc=kirill.shutemov@linux.intel.com \
    --cc=koct9i@gmail.com \
    --cc=linux-api@vger.kernel.org \
    --cc=linux-arch@vger.kernel.org \
    --cc=linux-doc@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=luto@amacapital.net \
    --cc=luto@kernel.org \
    --cc=mhocko@suse.cz \
    --cc=oleg@redhat.com \
    --cc=richard@nod.at \
    --cc=rob.gardner@oracle.com \
    --cc=sparclinux@vger.kernel.org \
    --cc=vbabka@suse.cz \
    --cc=xiexiuqi@huawei.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).