linux-kernel - Re: [Xen-devel] XSA 154 and ISA region (640K -> 1MB) WB cache instead of UC

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: <20160818163544.07c8a051@lxorguk.ukuu.org.uk>
Date:   Thu, 18 Aug 2016 16:35:44 +0100
From:   One Thousand Gnomes <gnomes@...rguk.ukuu.org.uk>
To:     "Jan Beulich" <JBeulich@...e.com>
Cc:     "Andrew Cooper" <andrew.cooper3@...rix.com>,
        "Konrad Rzeszutek Wilk" <konrad.wilk@...cle.com>,
        <stefan.bader@...onical.com>, <david.vrabel@...rix.com>,
        "xen-devel" <xen-devel@...ts.xenproject.org>,
        "Boris Ostrovsky" <boris.ostrovsky@...cle.com>,
        <chuck.anderson@...cle.com>, "Juergen Gross" <JGross@...e.com>,
        <linux-kernel@...r.kernel.org>
Subject: Re: [Xen-devel] XSA 154 and ISA region (640K -> 1MB) WB cache
 instead of UC

On Thu, 18 Aug 2016 05:12:54 -0600
"Jan Beulich" <JBeulich@...e.com> wrote:

> >>> On 18.08.16 at 12:16, <andrew.cooper3@...rix.com> wrote:  
> > On 18/08/16 11:06, Jan Beulich wrote:  
> >>>>> On 17.08.16 at 22:32, <konrad.wilk@...cle.com> wrote:  
> >>>    Looking at the kernel it assumes that WB is ok for 640KB->1MB.
> >>>    The comment says:
> >>>    " /* Low ISA region is always mapped WB in page table. No need to track   
> > *"  
> >> As per above it's not clear to me what this comment is backed by.  
> > 
> > This states what is in the pagetables.  Not the combined result with MTRRs.
> > 
> > WB in the pagetables and WC/UB in the MTRRs is a legal combination which
> > functions correctly.  
> 
> True, but then again - haven't I been told multiple times that Linux
> nowadays prefers to run without using MTRRs?

The BIOS sets up the fixed MTRR registers for the 640K-1MB window. Those
are separate to the variable range MTRR registers used for main memory
with specific mappings for segments A000 to BFFF then C000-C7FF /
C800-CFFF / etc up to FFFF.

Alan