All of lore.kernel.org
 help / color / mirror / Atom feed
From: Toshi Kani <toshi.kani@hpe.com>
To: Dan Williams <dan.j.williams@intel.com>,
	Yigal Korman <yigal@plexistor.com>
Cc: Toshi Kani <toshi.kani@hp.com>,
	"linux-nvdimm@lists.01.org" <linux-nvdimm@lists.01.org>
Subject: Re: Legacy NVDIMM-F over EFI
Date: Tue, 17 May 2016 10:47:07 -0600	[thread overview]
Message-ID: <1463503627.27137.167.camel@hpe.com> (raw)
In-Reply-To: <CAPcyv4ioeGGe3ujkpytzJKZHDVawX95KtrFEmBvO2oaB_j_RqA@mail.gmail.com>

On Tue, 2016-05-17 at 08:57 -0700, Dan Williams wrote:
> On Tue, May 17, 2016 at 8:36 AM, Yigal Korman <yigal@plexistor.com>
> wrote:
> > 
> > Hi all,
> > I've got a couple of legacy 'type 12' NVDIMM-F modules in our lab I've
> > been successfully testing.
> > Recently I moved them to a new system where EFI bootloader is used
> > (CentOS 7 installed with EFI) and  was surprised to see that they're
> > not identified as legacy 'type 12' persistent memory, are not caught
> > by the 'nd_pmem' driver and I don't get /dev/pmemX devices.
> > They show as 'type 7' in the E820 map and 'Persistent memory' in
> > /proc/iomem.
> > Is this the intended behavior? shouldn't they be identified as legacy
> > if they don't have NFIT?
>
> If the BIOS identifies it as type-7 it is broken, as far as Linux is
> concerned, if does not also export an NFIT with an ACPI0012 device.
> One would hope there are no new BIOS images being produced that use
> type-12...

On an EFI system and a kernel with CONFIG_EFI_STUB set, the kernel uses EFI
memory table, instead of e820.  So, e820 type-12 cannot be conveyed to the
kernel.

Since this EFI BIOS is broken as Dan pointed out, you might want to try
booting from a kernel with CONFIG_EFI_STUB unset to see if the kernel gets
e820 type-12.  (Note, if e820 is set to type-7, this would lead the kernel
to overwrite data in NVDIMM due to a bug in grub.)

-Toshi
_______________________________________________
Linux-nvdimm mailing list
Linux-nvdimm@lists.01.org
https://lists.01.org/mailman/listinfo/linux-nvdimm

  parent reply	other threads:[~2016-05-17 16:56 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-05-17 15:36 Legacy NVDIMM-F over EFI Yigal Korman
2016-05-17 15:57 ` Dan Williams
2016-05-17 16:19   ` Jeff Moyer
2016-05-17 16:30   ` Yigal Korman
2016-05-17 16:33     ` Dan Williams
2016-05-19  7:28       ` Yigal Korman
2016-05-19 13:54         ` Dan Williams
2016-05-19 16:37           ` Dan Williams
2016-05-17 16:47   ` Toshi Kani [this message]
2016-05-17 17:00     ` 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=1463503627.27137.167.camel@hpe.com \
    --to=toshi.kani@hpe.com \
    --cc=dan.j.williams@intel.com \
    --cc=linux-nvdimm@lists.01.org \
    --cc=toshi.kani@hp.com \
    --cc=yigal@plexistor.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.