All of lore.kernel.org
 help / color / mirror / Atom feed
From: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
To: Andy Lutomirski <luto@amacapital.net>
Cc: Henrique de Moraes Holschuh <hmh@hmh.eng.br>,
	"H. Peter Anvin" <hpa@zytor.com>, Toshi Kani <toshi.kani@hp.com>,
	Thomas Gleixner <tglx@linutronix.de>,
	Ingo Molnar <mingo@redhat.com>,
	akpm@linuxfoundation.org, Arnd Bergmann <arnd@arndb.de>,
	"linux-mm@kvack.org" <linux-mm@kvack.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	Juergen Gross <jgross@suse.com>,
	Stefan Bader <stefan.bader@canonical.com>
Subject: Re: [PATCH 1/5] x86, mm, pat: Set WT to PA4 slot of PAT MSR
Date: Fri, 12 Sep 2014 15:25:01 -0400	[thread overview]
Message-ID: <20140912192501.GG15656@laptop.dumpdata.com> (raw)
In-Reply-To: <CALCETrWxKFtM8FhnHQz--uaHYbiqShE1XLJxMCKN7Rs4SO14eQ@mail.gmail.com>

On Thu, Sep 04, 2014 at 04:34:43PM -0700, Andy Lutomirski wrote:
> On Thu, Sep 4, 2014 at 4:19 PM, Henrique de Moraes Holschuh
> <hmh@hmh.eng.br> wrote:
> > On Thu, 04 Sep 2014, H. Peter Anvin wrote:
> >> On 09/04/2014 01:11 PM, Henrique de Moraes Holschuh wrote:
> >> > I am worried of uncharted territory, here.  I'd actually advocate for not
> >> > enabling the upper four PAT entries on IA-32 at all, unless Windows 9X / XP
> >> > is using them as well.  Is this a real concern, or am I being overly
> >> > cautious?
> >>
> >> It is extremely unlikely that we'd have PAT issues in 32-bit mode and
> >> not in 64-bit mode on the same CPU.
> >
> > Sure, but is it really a good idea to enable this on the *old* non-64-bit
> > capable processors (note: I don't mean x86-64 processors operating in 32-bit
> > mode) ?
> >
> >> As far as I know, the current blacklist rule is very conservative due to
> >> lack of testing more than anything else.
> >
> > I was told that much in 2009 when I asked why cpuid 0x6d8 was blacklisted
> > from using PAT :-)
> 
> At the very least, anyone who plugs an NV-DIMM into a 32-bit machine
> is nuts, and not just because I'd be somewhat amazed if it even
> physically fits into the slot. :)

They do have PCIe to PCI adapters, so you _could_ do it :-)

> 
> --Andy
> 
> >
> > --
> >   "One disk to rule them all, One disk to find them. One disk to bring
> >   them all and in the darkness grind them. In the Land of Redmond
> >   where the shadows lie." -- The Silicon Valley Tarot
> >   Henrique Holschuh
> 
> 
> 
> -- 
> Andy Lutomirski
> AMA Capital Management, LLC

--
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: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

WARNING: multiple messages have this Message-ID (diff)
From: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
To: Andy Lutomirski <luto@amacapital.net>
Cc: Henrique de Moraes Holschuh <hmh@hmh.eng.br>,
	"H. Peter Anvin" <hpa@zytor.com>, Toshi Kani <toshi.kani@hp.com>,
	Thomas Gleixner <tglx@linutronix.de>,
	Ingo Molnar <mingo@redhat.com>,
	akpm@linuxfoundation.org, Arnd Bergmann <arnd@arndb.de>,
	"linux-mm@kvack.org" <linux-mm@kvack.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	Juergen Gross <jgross@suse.com>,
	Stefan Bader <stefan.bader@canonical.com>
Subject: Re: [PATCH 1/5] x86, mm, pat: Set WT to PA4 slot of PAT MSR
Date: Fri, 12 Sep 2014 15:25:01 -0400	[thread overview]
Message-ID: <20140912192501.GG15656@laptop.dumpdata.com> (raw)
In-Reply-To: <CALCETrWxKFtM8FhnHQz--uaHYbiqShE1XLJxMCKN7Rs4SO14eQ@mail.gmail.com>

On Thu, Sep 04, 2014 at 04:34:43PM -0700, Andy Lutomirski wrote:
> On Thu, Sep 4, 2014 at 4:19 PM, Henrique de Moraes Holschuh
> <hmh@hmh.eng.br> wrote:
> > On Thu, 04 Sep 2014, H. Peter Anvin wrote:
> >> On 09/04/2014 01:11 PM, Henrique de Moraes Holschuh wrote:
> >> > I am worried of uncharted territory, here.  I'd actually advocate for not
> >> > enabling the upper four PAT entries on IA-32 at all, unless Windows 9X / XP
> >> > is using them as well.  Is this a real concern, or am I being overly
> >> > cautious?
> >>
> >> It is extremely unlikely that we'd have PAT issues in 32-bit mode and
> >> not in 64-bit mode on the same CPU.
> >
> > Sure, but is it really a good idea to enable this on the *old* non-64-bit
> > capable processors (note: I don't mean x86-64 processors operating in 32-bit
> > mode) ?
> >
> >> As far as I know, the current blacklist rule is very conservative due to
> >> lack of testing more than anything else.
> >
> > I was told that much in 2009 when I asked why cpuid 0x6d8 was blacklisted
> > from using PAT :-)
> 
> At the very least, anyone who plugs an NV-DIMM into a 32-bit machine
> is nuts, and not just because I'd be somewhat amazed if it even
> physically fits into the slot. :)

They do have PCIe to PCI adapters, so you _could_ do it :-)

> 
> --Andy
> 
> >
> > --
> >   "One disk to rule them all, One disk to find them. One disk to bring
> >   them all and in the darkness grind them. In the Land of Redmond
> >   where the shadows lie." -- The Silicon Valley Tarot
> >   Henrique Holschuh
> 
> 
> 
> -- 
> Andy Lutomirski
> AMA Capital Management, LLC

  parent reply	other threads:[~2014-09-12 19:25 UTC|newest]

Thread overview: 69+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-09-04 18:35 [PATCH 0/5] Support Write-Through mapping on x86 Toshi Kani
2014-09-04 18:35 ` Toshi Kani
2014-09-04 18:35 ` [PATCH 1/5] x86, mm, pat: Set WT to PA4 slot of PAT MSR Toshi Kani
2014-09-04 18:35   ` Toshi Kani
2014-09-04 20:11   ` Henrique de Moraes Holschuh
2014-09-04 20:11     ` Henrique de Moraes Holschuh
2014-09-04 20:21     ` H. Peter Anvin
2014-09-04 20:21       ` H. Peter Anvin
2014-09-04 23:19       ` Henrique de Moraes Holschuh
2014-09-04 23:19         ` Henrique de Moraes Holschuh
2014-09-04 23:34         ` Andy Lutomirski
2014-09-04 23:34           ` Andy Lutomirski
2014-09-05  0:29           ` Toshi Kani
2014-09-05  0:29             ` Toshi Kani
2014-09-05  0:51             ` Andy Lutomirski
2014-09-05  0:51               ` Andy Lutomirski
2014-09-05 14:00               ` Toshi Kani
2014-09-05 14:00                 ` Toshi Kani
2014-09-05 15:07                 ` H. Peter Anvin
2014-09-05 15:07                   ` H. Peter Anvin
2014-09-05 15:22                   ` Toshi Kani
2014-09-05 15:22                     ` Toshi Kani
2014-09-05 15:41                     ` H. Peter Anvin
2014-09-05 15:41                       ` H. Peter Anvin
2014-09-05 15:42                       ` Toshi Kani
2014-09-05 15:42                         ` Toshi Kani
2014-09-12 19:25           ` Konrad Rzeszutek Wilk [this message]
2014-09-12 19:25             ` Konrad Rzeszutek Wilk
2014-09-12 20:03             ` Andy Lutomirski
2014-09-12 20:03               ` Andy Lutomirski
2014-09-12 20:21               ` Konrad Rzeszutek Wilk
2014-09-12 20:21                 ` Konrad Rzeszutek Wilk
2014-09-04 20:31     ` Toshi Kani
2014-09-04 20:31       ` Toshi Kani
2014-09-04 20:50       ` Andy Lutomirski
2014-09-04 20:50         ` Andy Lutomirski
2014-09-04 23:27       ` Toshi Kani
2014-09-04 23:27         ` Toshi Kani
2014-09-05 10:23         ` Ingo Molnar
2014-09-05 10:23           ` Ingo Molnar
2014-09-05 13:50           ` Toshi Kani
2014-09-05 13:50             ` Toshi Kani
2014-09-07 13:58             ` Henrique de Moraes Holschuh
2014-09-07 13:58               ` Henrique de Moraes Holschuh
2014-09-04 18:35 ` [PATCH 2/5] x86, mm, pat: Change reserve_memtype() to handle WT Toshi Kani
2014-09-04 18:35   ` Toshi Kani
2014-09-04 18:35 ` [PATCH 3/5] x86, mm, asm-gen: Add ioremap_wt() for WT Toshi Kani
2014-09-04 18:35   ` Toshi Kani
2014-09-04 18:35 ` [PATCH 4/5] x86, mm: Add set_memory_wt() " Toshi Kani
2014-09-04 18:35   ` Toshi Kani
2014-09-04 18:57   ` Andy Lutomirski
2014-09-04 18:57     ` Andy Lutomirski
2014-09-04 18:57     ` Toshi Kani
2014-09-04 18:57       ` Toshi Kani
2014-09-04 19:14       ` Andy Lutomirski
2014-09-04 19:14         ` Andy Lutomirski
2014-09-04 19:30         ` Toshi Kani
2014-09-07  8:49       ` Yigal Korman
2014-09-07  8:49         ` Yigal Korman
2014-09-07 16:49         ` Andy Lutomirski
2014-09-07 16:49           ` Andy Lutomirski
2014-09-08 15:07           ` Toshi Kani
2014-09-08 15:07             ` Toshi Kani
2014-09-08 17:23             ` Andy Lutomirski
2014-09-08 17:23               ` Andy Lutomirski
2014-09-08 18:42               ` Toshi Kani
2014-09-08 18:42                 ` Toshi Kani
2014-09-04 18:35 ` [PATCH 5/5] x86, mm, pat: Add pgprot_writethrough() " Toshi Kani
2014-09-04 18:35   ` Toshi Kani

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20140912192501.GG15656@laptop.dumpdata.com \
    --to=konrad.wilk@oracle.com \
    --cc=akpm@linuxfoundation.org \
    --cc=arnd@arndb.de \
    --cc=hmh@hmh.eng.br \
    --cc=hpa@zytor.com \
    --cc=jgross@suse.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=luto@amacapital.net \
    --cc=mingo@redhat.com \
    --cc=stefan.bader@canonical.com \
    --cc=tglx@linutronix.de \
    --cc=toshi.kani@hp.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.