[Top] [All Lists]

Re: [PATCH V7 1/3] MIPS: Rearrange PTE bits into fixed positions.

To: "Steven J. Hill" <>
Subject: Re: [PATCH V7 1/3] MIPS: Rearrange PTE bits into fixed positions.
From: David Daney <>
Date: Fri, 27 Feb 2015 09:52:15 -0800
Dkim-signature: v=1; a=rsa-sha256; c=relaxed/relaxed;; s=20120113; h=message-id:date:from:user-agent:mime-version:to:cc:subject :references:in-reply-to:content-type:content-transfer-encoding; bh=VHJieTBiqid3q1uixypeKNZMwKhWWSh3CKbLIe9TBWM=; b=AtgmMlklyRSt2m5E4DpqreRhyMvf2Mm39Ya8gWYnJcBKlbInk7DODTE7vJBce3v/Rt lYb9u5GI3FeajfmlVwC13lTj2RkHHi0f7q0WDFDMdPRtYNS1ZTucPteKkZYk4hq2jqzS ZN0Pf6ozPQY1RkcRXBHndxXFFWbMq/jg0GLuIYJI1XZ3Z+7r7ujs65FqD+nFS0zNX+Sb qRmfgQqjmWEoTIYlRfW8ADPI6baelXUF+UwOc3/ertdJICH51ZN/5/k47sZwFbgoI/U9 RpWEzQsQerjV86U/WababCLSZq8Hsd9DfncRQUGTbhWA/aa8cQaTqQfTfy80Y+q1lsFU VOFQ==
In-reply-to: <>
List-archive: <>
List-help: <>
List-id: linux-mips <>
List-owner: <>
List-post: <>
List-software: Ecartis version 1.0.0
List-subscribe: <>
List-unsubscribe: <>
Original-recipient: rfc822;
References: <> <> <> <>
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130625 Thunderbird/17.0.7
On 02/26/2015 07:38 PM, Steven J. Hill wrote:
On 02/26/2015 06:51 PM, David Daney wrote:

That's not really what I meant in my previous response on the subject.
When I said:

     Why not just use RI for everything, instead of taking up two bits
     to represent a single binary concept?

     For the case where there is no RI hardware active, it is a purely
     software bit and you can easily invert the meaning and just have a
     _PAGE_NO_READ bit.

I envisioned something like:

     64-bit, all revisions:    CCC D V G RI XI [S H] M A W P
     32-bit, all revisions:    CCC D V G RI XI M A W P

Which is what I implemented.

I think there is still misunderstanding.

Your patches leave us with definitions for *both* _PAGE_READ *and* _PAGE_NO_READ defined in the source code. My suggestion was to eliminate all vestiges of _PAGE_READ and _PAGE_READ_SHIFT, and unify all variants to use _PAGE_NO_READ

I now only use one bit that functions
either as _PAGE_READ or _PAGE_READ_ONLY depending on the RI/XI
functionality present. Did you bother to read the code and understand
it, or just look at the commit message?

I did read it, see above.

<Prev in Thread] Current Thread [Next in Thread>