From: Cornelia Huck <cohuck@redhat.com>
To: Pankaj Gupta <pagupta@redhat.com>
Cc: rdunlap@infradead.org, jack@suse.cz, kvm@vger.kernel.org,
mst@redhat.com, jasowang@redhat.com, david@fromorbit.com,
qemu-devel@nongnu.org, virtualization@lists.linux-foundation.org,
dm-devel@redhat.com, adilger.kernel@dilger.ca,
zwisler@kernel.org, aarcange@redhat.com, jstaron@google.com,
linux-nvdimm@lists.01.org, david@redhat.com, willy@infradead.org,
hch@infradead.org, linux-acpi@vger.kernel.org,
linux-ext4@vger.kernel.org, lenb@kernel.org, kilobyte@angband.pl,
riel@surriel.com, yuval.shaia@oracle.com, stefanha@redhat.com,
pbonzini@redhat.com, lcapitulino@redhat.com, kwolf@redhat.com,
nilal@redhat.com, tytso@mit.edu, xiaoguangrong.eric@gmail.com,
snitzer@redhat.com, darrick.wong@oracle.com, rjw@rjwysocki.net,
linux-kernel@vger.kernel.org, linux-xfs@vger.kernel.org,
linux-fsdevel@vger.kernel.org, imammedo@redhat.com
Subject: Re: [PATCH v13 2/7] virtio-pmem: Add virtio pmem driver
Date: Wed, 12 Jun 2019 16:20:12 +0200 [thread overview]
Message-ID: <20190612162012.06b4af7f.cohuck@redhat.com> (raw)
In-Reply-To: <20190612124527.3763-3-pagupta@redhat.com>
On Wed, 12 Jun 2019 18:15:22 +0530
Pankaj Gupta <pagupta@redhat.com> wrote:
> This patch adds virtio-pmem driver for KVM guest.
>
> Guest reads the persistent memory range information from
> Qemu over VIRTIO and registers it on nvdimm_bus. It also
> creates a nd_region object with the persistent memory
> range information so that existing 'nvdimm/pmem' driver
> can reserve this into system memory map. This way
> 'virtio-pmem' driver uses existing functionality of pmem
> driver to register persistent memory compatible for DAX
> capable filesystems.
>
> This also provides function to perform guest flush over
> VIRTIO from 'pmem' driver when userspace performs flush
> on DAX memory range.
>
> Signed-off-by: Pankaj Gupta <pagupta@redhat.com>
> Reviewed-by: Yuval Shaia <yuval.shaia@oracle.com>
> Acked-by: Michael S. Tsirkin <mst@redhat.com>
> Acked-by: Jakub Staron <jstaron@google.com>
> Tested-by: Jakub Staron <jstaron@google.com>
> ---
> drivers/nvdimm/Makefile | 1 +
> drivers/nvdimm/nd_virtio.c | 125 +++++++++++++++++++++++++++++++
> drivers/nvdimm/virtio_pmem.c | 122 ++++++++++++++++++++++++++++++
> drivers/nvdimm/virtio_pmem.h | 55 ++++++++++++++
> drivers/virtio/Kconfig | 11 +++
> include/uapi/linux/virtio_ids.h | 1 +
> include/uapi/linux/virtio_pmem.h | 35 +++++++++
> 7 files changed, 350 insertions(+)
> create mode 100644 drivers/nvdimm/nd_virtio.c
> create mode 100644 drivers/nvdimm/virtio_pmem.c
> create mode 100644 drivers/nvdimm/virtio_pmem.h
> create mode 100644 include/uapi/linux/virtio_pmem.h
Reviewed-by: Cornelia Huck <cohuck@redhat.com>
_______________________________________________
Linux-nvdimm mailing list
Linux-nvdimm@lists.01.org
https://lists.01.org/mailman/listinfo/linux-nvdimm
WARNING: multiple messages have this Message-ID (diff)
From: Cornelia Huck <cohuck@redhat.com>
To: Pankaj Gupta <pagupta@redhat.com>
Cc: dm-devel@redhat.com, linux-nvdimm@lists.01.org,
linux-kernel@vger.kernel.org,
virtualization@lists.linux-foundation.org, kvm@vger.kernel.org,
linux-fsdevel@vger.kernel.org, linux-acpi@vger.kernel.org,
qemu-devel@nongnu.org, linux-ext4@vger.kernel.org,
linux-xfs@vger.kernel.org, dan.j.williams@intel.com,
zwisler@kernel.org, vishal.l.verma@intel.com,
dave.jiang@intel.com, mst@redhat.com, jasowang@redhat.com,
willy@infradead.org, rjw@rjwysocki.net, hch@infradead.org,
lenb@kernel.org, jack@suse.cz, tytso@mit.edu,
adilger.kernel@dilger.ca, darrick.wong@oracle.com,
lcapitulino@redhat.com, kwolf@redhat.com, imammedo@redhat.com,
jmoyer@redhat.com, nilal@redhat.com, riel@surriel.com,
stefanha@redhat.com, aarcange@redhat.com, david@redhat.com,
david@fromorbit.com, xiaoguangrong.eric@gmail.com,
pbonzini@redhat.com, yuval.shaia@oracle.com, kilobyte@angband.pl,
jstaron@google.com, rdunlap@infradead.org, snitzer@redhat.com
Subject: Re: [PATCH v13 2/7] virtio-pmem: Add virtio pmem driver
Date: Wed, 12 Jun 2019 16:20:12 +0200 [thread overview]
Message-ID: <20190612162012.06b4af7f.cohuck@redhat.com> (raw)
In-Reply-To: <20190612124527.3763-3-pagupta@redhat.com>
On Wed, 12 Jun 2019 18:15:22 +0530
Pankaj Gupta <pagupta@redhat.com> wrote:
> This patch adds virtio-pmem driver for KVM guest.
>
> Guest reads the persistent memory range information from
> Qemu over VIRTIO and registers it on nvdimm_bus. It also
> creates a nd_region object with the persistent memory
> range information so that existing 'nvdimm/pmem' driver
> can reserve this into system memory map. This way
> 'virtio-pmem' driver uses existing functionality of pmem
> driver to register persistent memory compatible for DAX
> capable filesystems.
>
> This also provides function to perform guest flush over
> VIRTIO from 'pmem' driver when userspace performs flush
> on DAX memory range.
>
> Signed-off-by: Pankaj Gupta <pagupta@redhat.com>
> Reviewed-by: Yuval Shaia <yuval.shaia@oracle.com>
> Acked-by: Michael S. Tsirkin <mst@redhat.com>
> Acked-by: Jakub Staron <jstaron@google.com>
> Tested-by: Jakub Staron <jstaron@google.com>
> ---
> drivers/nvdimm/Makefile | 1 +
> drivers/nvdimm/nd_virtio.c | 125 +++++++++++++++++++++++++++++++
> drivers/nvdimm/virtio_pmem.c | 122 ++++++++++++++++++++++++++++++
> drivers/nvdimm/virtio_pmem.h | 55 ++++++++++++++
> drivers/virtio/Kconfig | 11 +++
> include/uapi/linux/virtio_ids.h | 1 +
> include/uapi/linux/virtio_pmem.h | 35 +++++++++
> 7 files changed, 350 insertions(+)
> create mode 100644 drivers/nvdimm/nd_virtio.c
> create mode 100644 drivers/nvdimm/virtio_pmem.c
> create mode 100644 drivers/nvdimm/virtio_pmem.h
> create mode 100644 include/uapi/linux/virtio_pmem.h
Reviewed-by: Cornelia Huck <cohuck@redhat.com>
WARNING: multiple messages have this Message-ID (diff)
From: Cornelia Huck <cohuck@redhat.com>
To: Pankaj Gupta <pagupta@redhat.com>
Cc: rdunlap@infradead.org, jack@suse.cz, kvm@vger.kernel.org,
mst@redhat.com, jasowang@redhat.com, david@fromorbit.com,
qemu-devel@nongnu.org, virtualization@lists.linux-foundation.org,
dm-devel@redhat.com, adilger.kernel@dilger.ca,
zwisler@kernel.org, aarcange@redhat.com, dave.jiang@intel.com,
jstaron@google.com, linux-nvdimm@lists.01.org,
vishal.l.verma@intel.com, david@redhat.com, willy@infradead.org,
hch@infradead.org, linux-acpi@vger.kernel.org, jmoyer@redhat.com,
linux-ext4@vger.kernel.org, lenb@kernel.org, kilobyte@angband.pl,
riel@surriel.com, yuval.shaia@oracle.com, stefanha@redhat.com,
pbonzini@redhat.com, dan.j.williams@intel.com,
lcapitulino@redhat.com, kwolf@redhat.com, nilal@redhat.com,
tytso@mit.edu, xiaoguangrong.eric@gmail.com, snitzer@redhat.com,
darrick.wong@oracle.com, rjw@rjwysocki.net,
linux-kernel@vger.kernel.org, linux-xfs@vger.kernel.org,
linux-fsdevel@vger.kernel.org, imammedo@redhat.com
Subject: Re: [Qemu-devel] [PATCH v13 2/7] virtio-pmem: Add virtio pmem driver
Date: Wed, 12 Jun 2019 16:20:12 +0200 [thread overview]
Message-ID: <20190612162012.06b4af7f.cohuck@redhat.com> (raw)
In-Reply-To: <20190612124527.3763-3-pagupta@redhat.com>
On Wed, 12 Jun 2019 18:15:22 +0530
Pankaj Gupta <pagupta@redhat.com> wrote:
> This patch adds virtio-pmem driver for KVM guest.
>
> Guest reads the persistent memory range information from
> Qemu over VIRTIO and registers it on nvdimm_bus. It also
> creates a nd_region object with the persistent memory
> range information so that existing 'nvdimm/pmem' driver
> can reserve this into system memory map. This way
> 'virtio-pmem' driver uses existing functionality of pmem
> driver to register persistent memory compatible for DAX
> capable filesystems.
>
> This also provides function to perform guest flush over
> VIRTIO from 'pmem' driver when userspace performs flush
> on DAX memory range.
>
> Signed-off-by: Pankaj Gupta <pagupta@redhat.com>
> Reviewed-by: Yuval Shaia <yuval.shaia@oracle.com>
> Acked-by: Michael S. Tsirkin <mst@redhat.com>
> Acked-by: Jakub Staron <jstaron@google.com>
> Tested-by: Jakub Staron <jstaron@google.com>
> ---
> drivers/nvdimm/Makefile | 1 +
> drivers/nvdimm/nd_virtio.c | 125 +++++++++++++++++++++++++++++++
> drivers/nvdimm/virtio_pmem.c | 122 ++++++++++++++++++++++++++++++
> drivers/nvdimm/virtio_pmem.h | 55 ++++++++++++++
> drivers/virtio/Kconfig | 11 +++
> include/uapi/linux/virtio_ids.h | 1 +
> include/uapi/linux/virtio_pmem.h | 35 +++++++++
> 7 files changed, 350 insertions(+)
> create mode 100644 drivers/nvdimm/nd_virtio.c
> create mode 100644 drivers/nvdimm/virtio_pmem.c
> create mode 100644 drivers/nvdimm/virtio_pmem.h
> create mode 100644 include/uapi/linux/virtio_pmem.h
Reviewed-by: Cornelia Huck <cohuck@redhat.com>
WARNING: multiple messages have this Message-ID (diff)
From: Cornelia Huck <cohuck@redhat.com>
To: Pankaj Gupta <pagupta@redhat.com>
Cc: dm-devel@redhat.com, linux-nvdimm@lists.01.org,
linux-kernel@vger.kernel.org,
virtualization@lists.linux-foundation.org, kvm@vger.kernel.org,
linux-fsdevel@vger.kernel.org, linux-acpi@vger.kernel.org,
qemu-devel@nongnu.org, linux-ext4@vger.kernel.org,
linux-xfs@vger.kernel.org, dan.j.williams@intel.com,
zwisler@kernel.org, vishal.l.verma@intel.com,
dave.jiang@intel.com, mst@redhat.com, jasowang@redhat.com,
willy@infradead.org, rjw@rjwysocki.net, hch@infradead.org,
lenb@kernel.org, jack@suse.cz, tytso@mit.edu,
adilger.kernel@dilger.ca, darrick.wong@oracle.com,
lcapitulino@redhat.com, kwolf@redhat.com, imammedo@redhat.com,
jmoyer@redhat.com, nilal@redhat.com, riel@surriel.com,
stefanha@redhat.com, aarcange@redhat.com, david@redhat.com,
david@fromorbit.com, xiaoguangrong.eric@
Subject: Re: [PATCH v13 2/7] virtio-pmem: Add virtio pmem driver
Date: Wed, 12 Jun 2019 16:20:12 +0200 [thread overview]
Message-ID: <20190612162012.06b4af7f.cohuck@redhat.com> (raw)
In-Reply-To: <20190612124527.3763-3-pagupta@redhat.com>
On Wed, 12 Jun 2019 18:15:22 +0530
Pankaj Gupta <pagupta@redhat.com> wrote:
> This patch adds virtio-pmem driver for KVM guest.
>
> Guest reads the persistent memory range information from
> Qemu over VIRTIO and registers it on nvdimm_bus. It also
> creates a nd_region object with the persistent memory
> range information so that existing 'nvdimm/pmem' driver
> can reserve this into system memory map. This way
> 'virtio-pmem' driver uses existing functionality of pmem
> driver to register persistent memory compatible for DAX
> capable filesystems.
>
> This also provides function to perform guest flush over
> VIRTIO from 'pmem' driver when userspace performs flush
> on DAX memory range.
>
> Signed-off-by: Pankaj Gupta <pagupta@redhat.com>
> Reviewed-by: Yuval Shaia <yuval.shaia@oracle.com>
> Acked-by: Michael S. Tsirkin <mst@redhat.com>
> Acked-by: Jakub Staron <jstaron@google.com>
> Tested-by: Jakub Staron <jstaron@google.com>
> ---
> drivers/nvdimm/Makefile | 1 +
> drivers/nvdimm/nd_virtio.c | 125 +++++++++++++++++++++++++++++++
> drivers/nvdimm/virtio_pmem.c | 122 ++++++++++++++++++++++++++++++
> drivers/nvdimm/virtio_pmem.h | 55 ++++++++++++++
> drivers/virtio/Kconfig | 11 +++
> include/uapi/linux/virtio_ids.h | 1 +
> include/uapi/linux/virtio_pmem.h | 35 +++++++++
> 7 files changed, 350 insertions(+)
> create mode 100644 drivers/nvdimm/nd_virtio.c
> create mode 100644 drivers/nvdimm/virtio_pmem.c
> create mode 100644 drivers/nvdimm/virtio_pmem.h
> create mode 100644 include/uapi/linux/virtio_pmem.h
Reviewed-by: Cornelia Huck <cohuck@redhat.com>
next prev parent reply other threads:[~2019-06-12 14:21 UTC|newest]
Thread overview: 43+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-06-12 12:45 [PATCH v13 0/7] virtio pmem driver Pankaj Gupta
2019-06-12 12:45 ` Pankaj Gupta
2019-06-12 12:45 ` [Qemu-devel] " Pankaj Gupta
2019-06-12 12:45 ` Pankaj Gupta
2019-06-12 12:45 ` [PATCH v13 1/7] libnvdimm: nd_region flush callback support Pankaj Gupta
2019-06-12 12:45 ` [Qemu-devel] " Pankaj Gupta
2019-06-12 12:45 ` Pankaj Gupta
2019-06-12 12:45 ` Pankaj Gupta
2019-06-12 12:45 ` [PATCH v13 2/7] virtio-pmem: Add virtio pmem driver Pankaj Gupta
2019-06-12 12:45 ` Pankaj Gupta
2019-06-12 12:45 ` [Qemu-devel] " Pankaj Gupta
2019-06-12 12:45 ` Pankaj Gupta
2019-06-12 14:20 ` Cornelia Huck [this message]
2019-06-12 14:20 ` Cornelia Huck
2019-06-12 14:20 ` [Qemu-devel] " Cornelia Huck
2019-06-12 14:20 ` Cornelia Huck
2019-06-13 3:42 ` [Qemu-devel] " Pankaj Gupta
2019-06-13 3:42 ` Pankaj Gupta
2019-06-13 3:42 ` Pankaj Gupta
2019-06-13 3:42 ` Pankaj Gupta
2019-06-13 3:42 ` Pankaj Gupta
2019-06-12 14:20 ` Cornelia Huck
2019-06-12 12:45 ` [PATCH v13 3/7] libnvdimm: add dax_dev sync flag Pankaj Gupta
2019-06-12 12:45 ` [Qemu-devel] " Pankaj Gupta
2019-06-12 12:45 ` Pankaj Gupta
2019-06-12 12:45 ` Pankaj Gupta
2019-06-12 12:45 ` [PATCH v13 4/7] dm: enable synchronous dax Pankaj Gupta
2019-06-12 12:45 ` Pankaj Gupta
2019-06-12 12:45 ` [Qemu-devel] " Pankaj Gupta
2019-06-12 12:45 ` Pankaj Gupta
2019-06-12 12:45 ` [PATCH v13 5/7] dax: check synchronous mapping is supported Pankaj Gupta
2019-06-12 12:45 ` Pankaj Gupta
2019-06-12 12:45 ` Pankaj Gupta
2019-06-12 12:45 ` [Qemu-devel] " Pankaj Gupta
2019-06-12 12:45 ` Pankaj Gupta
2019-06-12 12:45 ` [PATCH v13 6/7] ext4: disable map_sync for async flush Pankaj Gupta
2019-06-12 12:45 ` Pankaj Gupta
2019-06-12 12:45 ` [Qemu-devel] " Pankaj Gupta
2019-06-12 12:45 ` Pankaj Gupta
2019-06-12 12:45 ` [PATCH v13 7/7] xfs: " Pankaj Gupta
2019-06-12 12:45 ` [Qemu-devel] " Pankaj Gupta
2019-06-12 12:45 ` Pankaj Gupta
2019-06-12 12:45 ` Pankaj Gupta
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=20190612162012.06b4af7f.cohuck@redhat.com \
--to=cohuck@redhat.com \
--cc=aarcange@redhat.com \
--cc=adilger.kernel@dilger.ca \
--cc=darrick.wong@oracle.com \
--cc=david@fromorbit.com \
--cc=david@redhat.com \
--cc=dm-devel@redhat.com \
--cc=hch@infradead.org \
--cc=imammedo@redhat.com \
--cc=jack@suse.cz \
--cc=jasowang@redhat.com \
--cc=jstaron@google.com \
--cc=kilobyte@angband.pl \
--cc=kvm@vger.kernel.org \
--cc=kwolf@redhat.com \
--cc=lcapitulino@redhat.com \
--cc=lenb@kernel.org \
--cc=linux-acpi@vger.kernel.org \
--cc=linux-ext4@vger.kernel.org \
--cc=linux-fsdevel@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-nvdimm@lists.01.org \
--cc=linux-xfs@vger.kernel.org \
--cc=mst@redhat.com \
--cc=nilal@redhat.com \
--cc=pagupta@redhat.com \
--cc=pbonzini@redhat.com \
--cc=qemu-devel@nongnu.org \
--cc=rdunlap@infradead.org \
--cc=riel@surriel.com \
--cc=rjw@rjwysocki.net \
--cc=snitzer@redhat.com \
--cc=stefanha@redhat.com \
--cc=tytso@mit.edu \
--cc=virtualization@lists.linux-foundation.org \
--cc=willy@infradead.org \
--cc=xiaoguangrong.eric@gmail.com \
--cc=yuval.shaia@oracle.com \
--cc=zwisler@kernel.org \
/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.