From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753177AbbCZN6E (ORCPT ); Thu, 26 Mar 2015 09:58:04 -0400 Received: from mail-wi0-f181.google.com ([209.85.212.181]:37536 "EHLO mail-wi0-f181.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753153AbbCZN55 (ORCPT ); Thu, 26 Mar 2015 09:57:57 -0400 MIME-Version: 1.0 In-Reply-To: <20150326080132.GA26540@lst.de> References: <1427299449-26722-1-git-send-email-hch@lst.de> <1427299449-26722-3-git-send-email-hch@lst.de> <20150326080132.GA26540@lst.de> Date: Thu, 26 Mar 2015 06:57:56 -0700 Message-ID: Subject: Re: [Linux-nvdimm] [PATCH 2/3] x86: add a is_e820_ram() helper From: Dan Williams To: Christoph Hellwig Cc: linux-nvdimm , linux-fsdevel , "linux-kernel@vger.kernel.org" , X86 ML , Jens Axboe Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Mar 26, 2015 at 1:01 AM, Christoph Hellwig wrote: > 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 . >> >> ...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. Yes, please attribute Dave and myself. ...and for the series: Acked-by: Dan Williams