From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-yh0-f54.google.com (mail-yh0-f54.google.com [209.85.213.54]) by kanga.kvack.org (Postfix) with ESMTP id CA0FF6B0039 for ; Fri, 12 Sep 2014 16:55:52 -0400 (EDT) Received: by mail-yh0-f54.google.com with SMTP id z6so651543yhz.41 for ; Fri, 12 Sep 2014 13:55:52 -0700 (PDT) Received: from g4t3425.houston.hp.com (g4t3425.houston.hp.com. [15.201.208.53]) by mx.google.com with ESMTPS id o67si4786439yhb.194.2014.09.12.13.55.52 for (version=TLSv1 cipher=RC4-SHA bits=128/128); Fri, 12 Sep 2014 13:55:52 -0700 (PDT) Message-ID: <1410554715.28990.322.camel@misato.fc.hp.com> Subject: Re: [PATCH v2 1/6] x86, mm, pat: Set WT to PA4 slot of PAT MSR From: Toshi Kani Date: Fri, 12 Sep 2014 14:45:15 -0600 In-Reply-To: <20140912193345.GH15656@laptop.dumpdata.com> References: <1410367910-6026-1-git-send-email-toshi.kani@hp.com> <1410367910-6026-2-git-send-email-toshi.kani@hp.com> <20140912193345.GH15656@laptop.dumpdata.com> Content-Type: text/plain; charset="UTF-8" Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Sender: owner-linux-mm@kvack.org List-ID: To: Konrad Rzeszutek Wilk Cc: hpa@zytor.com, tglx@linutronix.de, mingo@redhat.com, akpm@linux-foundation.org, arnd@arndb.de, linux-mm@kvack.org, linux-kernel@vger.kernel.org, jgross@suse.com, stefan.bader@canonical.com, luto@amacapital.net, hmh@hmh.eng.br, yigal@plexistor.com On Fri, 2014-09-12 at 15:33 -0400, Konrad Rzeszutek Wilk wrote: > > - /* Set PWT to Write-Combining. All other bits stay the same */ > > - /* > > - * PTE encoding used in Linux: > > - * PAT > > - * |PCD > > - * ||PWT > > - * ||| > > - * 000 WB _PAGE_CACHE_WB > > - * 001 WC _PAGE_CACHE_WC > > - * 010 UC- _PAGE_CACHE_UC_MINUS > > - * 011 UC _PAGE_CACHE_UC > > > I think having this nice picture would be beneficial to folks > who want to understand it. And now you can of course expand it with > the slot 7 usage. OK, I will try to preserve the picture. > > - * PAT bit unused > > - */ > > - pat = PAT(0, WB) | PAT(1, WC) | PAT(2, UC_MINUS) | PAT(3, UC) | > > - PAT(4, WB) | PAT(5, WC) | PAT(6, UC_MINUS) | PAT(7, UC); > > + if ((c->x86_vendor == X86_VENDOR_INTEL) && > > + (((c->x86 == 0x6) && (c->x86_model <= 0xd)) || > > + ((c->x86 == 0xf) && (c->x86_model <= 0x6)))) { > > + /* > > + * Intel Pentium 2, 3, M, and 4 are affected by PAT errata, > > + * which makes the upper four entries unusable. We do not > > + * use the upper four entries for all the affected processor > > + * families for safe. > > + * > > + * PAT 0:WB, 1:WC, 2:UC-, 3:UC, 4-7:unusable > > + * > > + * NOTE: When WT or WP is used, it is redirected to UC- per > > + * the default setup in __cachemode2pte_tbl[]. > > + */ > > + pat = PAT(0, WB) | PAT(1, WC) | PAT(2, UC_MINUS) | PAT(3, UC) | > > + PAT(4, WB) | PAT(5, WC) | PAT(6, UC_MINUS) | PAT(7, UC); > > + } else { > > + /* > > + * WT is set to slot 7, which minimizes the risk of using > > You say slot 7 here, but the title of the patch says slot 4? Oops! I will fix the title. Thanks for reviewing! -Toshi -- 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: email@kvack.org