From mboxrd@z Thu Jan 1 00:00:00 1970 From: Pankaj Gupta Subject: Re: [Qemu-devel] [RFC 1/2] pmem: Move reusable code to base header files Date: Thu, 12 Oct 2017 17:27:28 -0400 (EDT) Message-ID: <1893461099.19940340.1507843648978.JavaMail.zimbra@redhat.com> References: <20171012155027.3277-1-pagupta@redhat.com> <20171012155027.3277-2-pagupta@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: Sender: owner-linux-mm@kvack.org To: Dan Williams Cc: Kevin Wolf , Haozhong Zhang , Jan Kara , Xiao Guangrong , KVM list , David Hildenbrand , linux-nvdimm , Ross Zwisler , linux-kernel@vger.kernel.org, Qemu Developers , Linux MM , Stefan Hajnoczi , Paolo Bonzini , Nitesh Narayan Lal List-Id: linux-nvdimm@lists.01.org > > On Thu, Oct 12, 2017 at 8:50 AM, Pankaj Gupta wrote: > > This patch moves common code to base header files > > so that it can be used for both ACPI pmem and VIRTIO pmem > > drivers. More common code needs to be moved out in future > > based on functionality required for virtio_pmem driver and > > coupling of code with existing ACPI pmem driver. > > > > Signed-off-by: Pankaj Gupta > [..] > > diff --git a/include/linux/pmem_common.h b/include/linux/pmem_common.h > > new file mode 100644 > > index 000000000000..e2e718c74b3f > > --- /dev/null > > +++ b/include/linux/pmem_common.h > > This should be a common C file, not a header. Sure! will create a common C file to put all the common code there. > > -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: email@kvack.org From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755299AbdJLV1g (ORCPT ); Thu, 12 Oct 2017 17:27:36 -0400 Received: from mx1.redhat.com ([209.132.183.28]:38262 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753200AbdJLV1e (ORCPT ); Thu, 12 Oct 2017 17:27:34 -0400 DMARC-Filter: OpenDMARC Filter v1.3.2 mx1.redhat.com E10AB7D0C2 Authentication-Results: ext-mx03.extmail.prod.ext.phx2.redhat.com; dmarc=none (p=none dis=none) header.from=redhat.com Authentication-Results: ext-mx03.extmail.prod.ext.phx2.redhat.com; spf=fail smtp.mailfrom=pagupta@redhat.com Date: Thu, 12 Oct 2017 17:27:28 -0400 (EDT) From: Pankaj Gupta To: Dan Williams Cc: Kevin Wolf , Haozhong Zhang , Jan Kara , Xiao Guangrong , KVM list , David Hildenbrand , linux-nvdimm , Ross Zwisler , linux-kernel@vger.kernel.org, Qemu Developers , Linux MM , Stefan Hajnoczi , Paolo Bonzini , Nitesh Narayan Lal Message-ID: <1893461099.19940340.1507843648978.JavaMail.zimbra@redhat.com> In-Reply-To: References: <20171012155027.3277-1-pagupta@redhat.com> <20171012155027.3277-2-pagupta@redhat.com> Subject: Re: [Qemu-devel] [RFC 1/2] pmem: Move reusable code to base header files MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-Originating-IP: [10.67.116.10, 10.4.195.22] Thread-Topic: pmem: Move reusable code to base header files Thread-Index: Fqofw5XhfO4ppsyY4vhbE7wt6uLRNw== X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.27]); Thu, 12 Oct 2017 21:27:34 +0000 (UTC) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org > > On Thu, Oct 12, 2017 at 8:50 AM, Pankaj Gupta wrote: > > This patch moves common code to base header files > > so that it can be used for both ACPI pmem and VIRTIO pmem > > drivers. More common code needs to be moved out in future > > based on functionality required for virtio_pmem driver and > > coupling of code with existing ACPI pmem driver. > > > > Signed-off-by: Pankaj Gupta > [..] > > diff --git a/include/linux/pmem_common.h b/include/linux/pmem_common.h > > new file mode 100644 > > index 000000000000..e2e718c74b3f > > --- /dev/null > > +++ b/include/linux/pmem_common.h > > This should be a common C file, not a header. Sure! will create a common C file to put all the common code there. > >