From mboxrd@z Thu Jan  1 00:00:00 1970
From: Christoph Hellwig <hch@lst.de>
Subject: Re: [Linux-nvdimm] [PATCH 2/3] x86: add a is_e820_ram() helper
Date: Thu, 26 Mar 2015 09:01:32 +0100
Message-ID: <20150326080132.GA26540@lst.de>
References: <1427299449-26722-1-git-send-email-hch@lst.de> <1427299449-26722-3-git-send-email-hch@lst.de> <CAPcyv4gikQO88mCa=DnBKSxbSZkheYVmvKjRERdEccL=v0uAMA@mail.gmail.com>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Cc: Christoph Hellwig <hch@lst.de>,
	linux-nvdimm <linux-nvdimm@ml01.01.org>,
	linux-fsdevel <linux-fsdevel@vger.kernel.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	X86 ML <x86@kernel.org>, Jens Axboe <axboe@kernel.dk>
To: Dan Williams <dan.j.williams@intel.com>
Return-path: <linux-kernel-owner@vger.kernel.org>
Content-Disposition: inline
In-Reply-To: <CAPcyv4gikQO88mCa=DnBKSxbSZkheYVmvKjRERdEccL=v0uAMA@mail.gmail.com>
Sender: linux-kernel-owner@vger.kernel.org
List-Id: linux-fsdevel.vger.kernel.org

On Wed, Mar 25, 2015 at 07:15:42PM -0700, Dan Williams wrote:
> Random thought, type-12 memory happens to correspond to "legacy"
> NVDIMM systems with smaller capacities.  Perhaps "new NVDIMM" should
> not be is_e820_ram() by default?

Let's look into that once we can see the spec..

> > Based on an earlier patch from Dave Jiang <dave.jiang@intel.com>.
> 
> ...which was based on an earlier patch by me, its been nearly 4 years
> to come full circle.

That's the attribution in the patch I have access to.  I can add you
to the credits if you want.