Re: [PATCH 3/3] x86: add support for the non-standard protected e820 type

2015-03-26 Thread Christoph Hellwig
On Wed, Mar 25, 2015 at 02:25:33PM -0600, Ross Zwisler wrote: > > + case E820_PROTECTED_KERN: > > + printk(KERN_CONT "protected (type %u)\n", type); > > I don't think we want a newline in this string. Fixed. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in

Re: [PATCH 3/3] x86: add support for the non-standard protected e820 type

2015-03-26 Thread Christoph Hellwig
On Wed, Mar 25, 2015 at 07:47:26PM +, Elliott, Robert (Server Storage) wrote: > If this goes into the kernel, I think someone should request that the > ACPI specification mark the value 12 as permanently tainted. Otherwise > they could assign it to some new meaning that conflicts with all > o

Re: [PATCH 3/3] x86: add support for the non-standard protected e820 type

2015-03-25 Thread Ross Zwisler
On Wed, 2015-03-25 at 17:04 +0100, Christoph Hellwig wrote: > Various recent bioses support NVDIMMs or ADR using a non-standard > e820 memory type, and Intel supplied reference Linux code using this > type to various vendors. > > Wire this e820 table type up to export platform devices for the pmem

Re: [PATCH 3/3] x86: add support for the non-standard protected e820 type

2015-03-25 Thread Ross Zwisler
On Wed, 2015-03-25 at 17:04 +0100, Christoph Hellwig wrote: > Various recent bioses support NVDIMMs or ADR using a non-standard > e820 memory type, and Intel supplied reference Linux code using this > type to various vendors. > > Wire this e820 table type up to export platform devices for the pmem

RE: [PATCH 3/3] x86: add support for the non-standard protected e820 type

2015-03-25 Thread Elliott, Robert (Server Storage)
A few editorial nits follow... > -Original Message- > From: linux-kernel-ow...@vger.kernel.org [mailto:linux-kernel- > ow...@vger.kernel.org] On Behalf Of Christoph Hellwig > Sent: Wednesday, March 25, 2015 11:04 AM > Subject: [PATCH 3/3] x86: add support for the non-standard protected e82