Re: [Linux-nvdimm] [PATCH 2/3] x86: add a is_e820_ram() helper

[Dan Williams <dan.j.williams@intel.com>]

On Thu, Mar 26, 2015 at 11:46 AM, Elliott, Robert (Server Storage)
<Elliott@hp.com> wrote:
>
>
>> -----Original Message-----
>> From: linux-kernel-owner@vger.kernel.org [mailto:linux-kernel-
>> owner@vger.kernel.org] On Behalf Of Christoph Hellwig
>> Sent: Thursday, March 26, 2015 11:43 AM
>> To: Boaz Harrosh
>> Cc: Christoph Hellwig; Ingo Molnar; linux-nvdimm@ml01.01.org; linux-
>> fsdevel@vger.kernel.org; linux-kernel@vger.kernel.org; x86@kernel.org;
>> ross.zwisler@linux.intel.com; axboe@kernel.dk
>> Subject: Re: [PATCH 2/3] x86: add a is_e820_ram() helper
>>
>> On Thu, Mar 26, 2015 at 05:49:38PM +0200, Boaz Harrosh wrote:
>> > > + memmap=nn[KMG]!ss[KMG]
>> > > +                 [KNL,X86] Mark specific memory as protected.
>> > > +                 Region of memory to be used, from ss to ss+nn.
>> > > +                 The memory region may be marked as e820 type 12 (0xc)
>> > > +                 and is NVDIMM or ADR memory.
>> > > +
>> >
>> > Do we need to escape "\!" this character on grub command line ? It might
>> > help to note that. I did like the original "|" BTW
>>
>> No need to escape it on the kvm command line, which is where I tested
>> this flag only so far.  If there is a strong argument for "|" I'm happy
>> to change it.
>
> I agree with Boaz that ! is a nuisance if loading pmem as a module
> with modprobe from bash.

This is a core kernel command line, not a module parameter.  I'm not
saying that it should stay "!", but modprobe will not need to deal
with it.