From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1030796AbbEEHfb (ORCPT ); Tue, 5 May 2015 03:35:31 -0400 Received: from cantor2.suse.de ([195.135.220.15]:49274 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756604AbbEEHfS (ORCPT ); Tue, 5 May 2015 03:35:18 -0400 Date: Tue, 5 May 2015 09:35:16 +0200 From: "Luis R. Rodriguez" To: Borislav Petkov Cc: "Luis R. Rodriguez" , mingo@elte.hu, tglx@linutronix.de, hpa@zytor.com, bp@suse.de, plagnioj@jcrosoft.com, tomi.valkeinen@ti.com, daniel.vetter@intel.com, airlied@linux.ie, dledford@redhat.com, awalls@md.metrocast.net, syrjala@sci.fi, luto@amacapital.net, mst@redhat.com, cocci@systeme.lip6.fr, linux-kernel@vger.kernel.org, Toshi Kani , Jonathan Corbet , Dave Hansen , Suresh Siddha , Juergen Gross , Daniel Vetter , Dave Airlie , Antonino Daplas , Mel Gorman , Vlastimil Babka , Davidlohr Bueso , linux-fbdev@vger.kernel.org Subject: Re: [PATCH v4 2/6] x86: document WC MTRR effects on PAT / non-PAT pages Message-ID: <20150505073516.GK5622@wotan.suse.de> References: <1430343851-967-1-git-send-email-mcgrof@do-not-panic.com> <1430343851-967-3-git-send-email-mcgrof@do-not-panic.com> <20150504122303.GD4096@pd.tnic> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20150504122303.GD4096@pd.tnic> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, May 04, 2015 at 02:23:03PM +0200, Borislav Petkov wrote: > On Wed, Apr 29, 2015 at 02:44:07PM -0700, Luis R. Rodriguez wrote: > > From: "Luis R. Rodriguez" > > > > As part of the effort to phase out MTRR use document > > write-combining MTRR effects on pages with different > > non-PAT page attributes flags and different PAT entry > > values. Extend arch_phys_wc_add() documentation to > > clarify power of two sizes / boundary requirements as > > we phase out mtrr_add() use. > > > > Lastly hint towards ioremap_uc() for corner cases on > > device drivers working with devices with mixed regions > > where MTRR size requirements would otherwise not > > enable write-combining effective memory types. > > > > Cc: Toshi Kani > > Cc: Jonathan Corbet > > Cc: Dave Hansen > > Cc: Andy Lutomirski > > Cc: Suresh Siddha > > Cc: Ingo Molnar > > Cc: Thomas Gleixner > > Cc: Juergen Gross > > Cc: Daniel Vetter > > Cc: Dave Airlie > > Cc: Antonino Daplas > > Cc: Jean-Christophe Plagniol-Villard > > Cc: Tomi Valkeinen > > Cc: Ville Syrjälä > > Cc: Mel Gorman > > Cc: Vlastimil Babka > > Cc: Borislav Petkov > > Cc: Davidlohr Bueso > > Cc: linux-fbdev@vger.kernel.org > > Cc: linux-kernel@vger.kernel.org > > Signed-off-by: Luis R. Rodriguez > > --- > > Documentation/x86/mtrr.txt | 18 +++++++++++++++--- > > Documentation/x86/pat.txt | 40 +++++++++++++++++++++++++++++++++++++++- > > arch/x86/kernel/cpu/mtrr/main.c | 3 +++ > > 3 files changed, 57 insertions(+), 4 deletions(-) > > > > diff --git a/Documentation/x86/mtrr.txt b/Documentation/x86/mtrr.txt > > index cc071dc..a111a6c 100644 > > --- a/Documentation/x86/mtrr.txt > > +++ b/Documentation/x86/mtrr.txt > > @@ -1,7 +1,19 @@ > > MTRR (Memory Type Range Register) control > > -3 Jun 1999 > > -Richard Gooch > > - > > + > > +Richard Gooch - 3 Jun 1999 > > +Luis R. Rodriguez - April 9, 2015 > > + > > +=============================================================================== > > +Phasing MTRR use > > "Phasing out...". Fixed all, will send another version. Luis From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Luis R. Rodriguez" Date: Tue, 05 May 2015 07:35:16 +0000 Subject: Re: [PATCH v4 2/6] x86: document WC MTRR effects on PAT / non-PAT pages Message-Id: <20150505073516.GK5622@wotan.suse.de> List-Id: References: <1430343851-967-1-git-send-email-mcgrof@do-not-panic.com> <1430343851-967-3-git-send-email-mcgrof@do-not-panic.com> <20150504122303.GD4096@pd.tnic> In-Reply-To: <20150504122303.GD4096@pd.tnic> MIME-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable To: Borislav Petkov Cc: "Luis R. Rodriguez" , mingo@elte.hu, tglx@linutronix.de, hpa@zytor.com, bp@suse.de, plagnioj@jcrosoft.com, tomi.valkeinen@ti.com, daniel.vetter@intel.com, airlied@linux.ie, dledford@redhat.com, awalls@md.metrocast.net, syrjala@sci.fi, luto@amacapital.net, mst@redhat.com, cocci@systeme.lip6.fr, linux-kernel@vger.kernel.org, Toshi Kani , Jonathan Corbet , Dave Hansen , Suresh Siddha , Juergen Gross , Daniel Vetter , Dave Airlie , Antonino Daplas , Mel Gorman , Vlastimil Babka , Davidlohr Bueso , linux-fbdev@vger.kernel.org On Mon, May 04, 2015 at 02:23:03PM +0200, Borislav Petkov wrote: > On Wed, Apr 29, 2015 at 02:44:07PM -0700, Luis R. Rodriguez wrote: > > From: "Luis R. Rodriguez" > >=20 > > As part of the effort to phase out MTRR use document > > write-combining MTRR effects on pages with different > > non-PAT page attributes flags and different PAT entry > > values. Extend arch_phys_wc_add() documentation to > > clarify power of two sizes / boundary requirements as > > we phase out mtrr_add() use. > >=20 > > Lastly hint towards ioremap_uc() for corner cases on > > device drivers working with devices with mixed regions > > where MTRR size requirements would otherwise not > > enable write-combining effective memory types. > >=20 > > Cc: Toshi Kani > > Cc: Jonathan Corbet > > Cc: Dave Hansen > > Cc: Andy Lutomirski > > Cc: Suresh Siddha > > Cc: Ingo Molnar > > Cc: Thomas Gleixner > > Cc: Juergen Gross > > Cc: Daniel Vetter > > Cc: Dave Airlie > > Cc: Antonino Daplas > > Cc: Jean-Christophe Plagniol-Villard > > Cc: Tomi Valkeinen > > Cc: Ville Syrj=E4l=E4 > > Cc: Mel Gorman > > Cc: Vlastimil Babka > > Cc: Borislav Petkov > > Cc: Davidlohr Bueso > > Cc: linux-fbdev@vger.kernel.org > > Cc: linux-kernel@vger.kernel.org > > Signed-off-by: Luis R. Rodriguez > > --- > > Documentation/x86/mtrr.txt | 18 +++++++++++++++--- > > Documentation/x86/pat.txt | 40 +++++++++++++++++++++++++++++++++= ++++++- > > arch/x86/kernel/cpu/mtrr/main.c | 3 +++ > > 3 files changed, 57 insertions(+), 4 deletions(-) > >=20 > > diff --git a/Documentation/x86/mtrr.txt b/Documentation/x86/mtrr.txt > > index cc071dc..a111a6c 100644 > > --- a/Documentation/x86/mtrr.txt > > +++ b/Documentation/x86/mtrr.txt > > @@ -1,7 +1,19 @@ > > MTRR (Memory Type Range Register) control > > -3 Jun 1999 > > -Richard Gooch > > - > > + > > +Richard Gooch - 3 Jun 1999 > > +Luis R. Rodriguez - April 9, 2015 > > + > > +=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D> > +Phasing MTRR use >=20 > "Phasing out...". Fixed all, will send another version. Luis