linux-kernel - Re: [RFC PATCH V2 13/22] x86/intel_rdt: Support schemata write

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <alpine.DEB.2.21.1802281939490.1392@nanos.tec.linutronix.de>
Date:   Wed, 28 Feb 2018 19:42:26 +0100 (CET)
From:   Thomas Gleixner <tglx@...utronix.de>
To:     Reinette Chatre <reinette.chatre@...el.com>
cc:     fenghua.yu@...el.com, tony.luck@...el.com, gavin.hindman@...el.com,
        vikas.shivappa@...ux.intel.com, dave.hansen@...el.com,
        mingo@...hat.com, hpa@...or.com, x86@...nel.org,
        linux-kernel@...r.kernel.org
Subject: Re: [RFC PATCH V2 13/22] x86/intel_rdt: Support schemata write -
 pseudo-locking core

On Wed, 28 Feb 2018, Reinette Chatre wrote:
> On 2/28/2018 9:59 AM, Thomas Gleixner wrote:
> I hesitated doing something like this because during the review of this
> series there was resistance to using sysfs files for multiple values. I
> will proceed with your suggestion noting that it is tied with schemata file.

It's not sysfs, it's resctrl fs. So we have already multiple lines and
values, e.g. in the schemata file.

> I do not think we need these special labels for CDP though. From what I
> understand when CDP is enabled there will be two new resources in the
> info directory. For example,
> 
> info/L3DATA/
> info/L3CODE/
> 
> Each would have its own file(s) noting which bits are in use.
> 
> At this time I am also not enabling pseudo-locking when CDP is enabled
> so the locked label is not needed.

The locked label is needed for the !CDP case so you can see where the
locked regions are in a bitmap

> There is already the file "shareable_bits" in the info directory
> associated with each resource. At the moment it only reflects the bits
> that could be used by other entities on the system. Considering its name
> and us now introducing the idea of "shareable" I was thinking of adding
> all "shareable" bits (hardware and software) of this resource to this
> file. This still leaves the new "inuse_bits" (or perhaps "unused_bits")
> info file that will communicate the "exclusive" and "locked" bits in
> addition so what is in "shareable_bits". Between these two files users
> should have information needed to choose regions for their tasks. These
> would all use bitmap displays.

Yeah, that needs some thought, but I happily let you sort out the details.

Thanks,

	tglx