From: Pankaj Gupta <pagupta-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> To: Jeff Moyer <jmoyer-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>, Dan Williams <dan.j.williams-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org> Cc: Kevin Wolf <kwolf-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>, Jan Kara <jack-AlSwsSmVLrQ@public.gmane.org>, Xiao Guangrong <xiaoguangrong.eric-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>, KVM list <kvm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>, David Hildenbrand <david-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>, linux-nvdimm <linux-nvdimm-y27Ovi1pjclAfugRpC6u6w@public.gmane.org>, "Michael S. Tsirkin" <mst-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>, Ross Zwisler <ross.zwisler-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>, Qemu Developers <qemu-devel-qX2TKyscuCcdnm+yROfE0A@public.gmane.org>, lcapitulino-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org, Christoph Hellwig <hch-wEGCiKHe2LqWVfeAwA7xHQ@public.gmane.org>, Linux MM <linux-mm-Bw31MaZKKs3YtjvyW6yDsg@public.gmane.org>, Igor Mammedov <imammedo-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>, Stefan Hajnoczi <stefanha-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>, niteshnarayanlal-PkbjNfxxIARBDgjK7y7TUQ@public.gmane.org, Marcel Apfelbaum <marcel-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>, Paolo Bonzini <pbonzini-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>, Rik van Riel <riel-ebMLmSuQjDVBDgjK7y7TUQ@public.gmane.org>, Nitesh Narayan Lal <nilal-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>, Linux Kernel Mailing List <linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org> Subject: Re: [Qemu-devel] [RFC v2 1/2] virtio: add pmem driver Date: Thu, 26 Apr 2018 13:15:38 -0400 (EDT) [thread overview] Message-ID: <1499190564.23017177.1524762938762.JavaMail.zimbra@redhat.com> (raw) In-Reply-To: <x49o9i6885e.fsf-RRHT56Q3PSP4kTEheFKJxxDDeQx5vsVwAInAS/Ez/D0@public.gmane.org> > > Dan Williams <dan.j.williams-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org> writes: > > > [ adding Jeff directly since he has also been looking at > > infrastructure to track when MAP_SYNC should be disabled ] > > > > On Wed, Apr 25, 2018 at 7:21 AM, Dan Williams <dan.j.williams-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org> > > wrote: > >> On Wed, Apr 25, 2018 at 4:24 AM, Pankaj Gupta <pagupta-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> wrote: > >>> This patch adds virtio-pmem driver for KVM > >>> guest. > >> > >> Minor nit, please expand your changelog line wrapping to 72 columns. > >> > >>> > >>> 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. > >> > >> We need some additional enabling to disable MAP_SYNC for this > > enable to disable... I like it! ;-) > > >> configuration. In other words, if fsync() is required then we must > >> disable the MAP_SYNC optimization. I think this should be a struct > >> dax_device property looked up at mmap time in each MAP_SYNC capable > >> ->mmap() file operation implementation. I understand you mean we want to disable 'MAP_SYNC' optimization as we are relying on additional fsync. You mean if we add a property/flag in dax_device struct and its set, disable 'MAP_SYNC' accordingly during mmap time for corresponding filesystems? > > Ideally, qemu (seabios?) would advertise a platform capabilities > sub-table that doesn't fill in the flush bits. Could you please elaborate on this, how its related to disabling MAP_SYNC? We are not doing entire nvdimm device emulation. > > -Jeff > >
WARNING: multiple messages have this Message-ID (diff)
From: Pankaj Gupta <pagupta@redhat.com> To: Jeff Moyer <jmoyer@redhat.com>, Dan Williams <dan.j.williams@intel.com> Cc: Jan Kara <jack@suse.cz>, KVM list <kvm@vger.kernel.org>, David Hildenbrand <david@redhat.com>, linux-nvdimm <linux-nvdimm@ml01.01.org>, Ross Zwisler <ross.zwisler@intel.com>, Qemu Developers <qemu-devel@nongnu.org>, lcapitulino@redhat.com, Linux MM <linux-mm@kvack.org>, niteshnarayanlal@hotmail.com, "Michael S. Tsirkin" <mst@redhat.com>, Christoph Hellwig <hch@infradead.org>, Marcel Apfelbaum <marcel@redhat.com>, Nitesh Narayan Lal <nilal@redhat.com>, Haozhong Zhang <haozhong.zhang@intel.com>, Rik van Riel <riel@surriel.com>, Stefan Hajnoczi <stefanha@redhat.com>, Paolo Bonzini <pbonzini@redhat.com>, Kevin Wolf <kwolf@redhat.com>, Xiao Guangrong <xiaoguangrong.eric@gmail.com>, Linux Kernel Mailing List <linux-kernel@vger.kernel.org>, Igor Mammedov <imammedo@redhat.com> Subject: Re: [Qemu-devel] [RFC v2 1/2] virtio: add pmem driver Date: Thu, 26 Apr 2018 13:15:38 -0400 (EDT) [thread overview] Message-ID: <1499190564.23017177.1524762938762.JavaMail.zimbra@redhat.com> (raw) In-Reply-To: <x49o9i6885e.fsf@segfault.boston.devel.redhat.com> > > Dan Williams <dan.j.williams@intel.com> writes: > > > [ adding Jeff directly since he has also been looking at > > infrastructure to track when MAP_SYNC should be disabled ] > > > > On Wed, Apr 25, 2018 at 7:21 AM, Dan Williams <dan.j.williams@intel.com> > > wrote: > >> On Wed, Apr 25, 2018 at 4:24 AM, Pankaj Gupta <pagupta@redhat.com> wrote: > >>> This patch adds virtio-pmem driver for KVM > >>> guest. > >> > >> Minor nit, please expand your changelog line wrapping to 72 columns. > >> > >>> > >>> 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. > >> > >> We need some additional enabling to disable MAP_SYNC for this > > enable to disable... I like it! ;-) > > >> configuration. In other words, if fsync() is required then we must > >> disable the MAP_SYNC optimization. I think this should be a struct > >> dax_device property looked up at mmap time in each MAP_SYNC capable > >> ->mmap() file operation implementation. I understand you mean we want to disable 'MAP_SYNC' optimization as we are relying on additional fsync. You mean if we add a property/flag in dax_device struct and its set, disable 'MAP_SYNC' accordingly during mmap time for corresponding filesystems? > > Ideally, qemu (seabios?) would advertise a platform capabilities > sub-table that doesn't fill in the flush bits. Could you please elaborate on this, how its related to disabling MAP_SYNC? We are not doing entire nvdimm device emulation. > > -Jeff > >
next prev parent reply other threads:[~2018-04-26 17:15 UTC|newest] Thread overview: 80+ messages / expand[flat|nested] mbox.gz Atom feed top 2018-04-25 11:24 [RFC v2 0/2] kvm "fake DAX" device flushing Pankaj Gupta 2018-04-25 11:24 ` [Qemu-devel] " Pankaj Gupta 2018-04-25 11:24 ` Pankaj Gupta [not found] ` <20180425112415.12327-1-pagupta-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> 2018-04-25 11:24 ` [RFC v2 1/2] virtio: add pmem driver Pankaj Gupta 2018-04-25 11:24 ` [Qemu-devel] " Pankaj Gupta 2018-04-25 11:24 ` Pankaj Gupta [not found] ` <20180425112415.12327-2-pagupta-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> 2018-04-25 14:21 ` Dan Williams 2018-04-25 14:21 ` [Qemu-devel] " Dan Williams 2018-04-25 14:21 ` Dan Williams [not found] ` <CAPcyv4hvrB08XPTbVK0xT2_1Xmaid=-v3OMxJVDTNwQucsOHLA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 2018-04-25 14:43 ` Dan Williams 2018-04-25 14:43 ` [Qemu-devel] " Dan Williams 2018-04-25 14:43 ` Dan Williams [not found] ` <CAPcyv4hiowWozV527sQA_e4fdgCYbD6xfG==vepAqu0hxQEQcw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 2018-04-26 12:27 ` Jeff Moyer 2018-04-26 12:27 ` [Qemu-devel] " Jeff Moyer 2018-04-26 12:27 ` Jeff Moyer 2018-04-26 12:27 ` Jeff Moyer [not found] ` <x49o9i6885e.fsf-RRHT56Q3PSP4kTEheFKJxxDDeQx5vsVwAInAS/Ez/D0@public.gmane.org> 2018-04-26 17:15 ` Pankaj Gupta [this message] 2018-04-26 17:15 ` [Qemu-devel] " Pankaj Gupta [not found] ` <1499190564.23017177.1524762938762.JavaMail.zimbra-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> 2018-04-26 17:24 ` Jeff Moyer 2018-04-26 17:24 ` Jeff Moyer 2018-04-25 14:52 ` Michael S. Tsirkin 2018-04-25 14:52 ` [Qemu-devel] " Michael S. Tsirkin 2018-04-25 14:52 ` Michael S. Tsirkin [not found] ` <20180425174705-mutt-send-email-mst-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org> 2018-04-25 15:11 ` [Qemu-devel] " Pankaj Gupta 2018-04-25 15:11 ` Pankaj Gupta 2018-04-26 13:12 ` Stefan Hajnoczi [not found] ` <20180426131236.GA30991-lxVrvc10SDRcolVlb+j0YCZi+YwRKgec@public.gmane.org> 2018-04-26 15:44 ` Pankaj Gupta 2018-04-26 15:44 ` Pankaj Gupta 2018-04-27 13:31 ` Stefan Hajnoczi [not found] ` <20180427133146.GB11150-lxVrvc10SDRcolVlb+j0YCZi+YwRKgec@public.gmane.org> 2018-04-28 10:48 ` Pankaj Gupta 2018-04-28 10:48 ` Pankaj Gupta 2018-04-25 11:24 ` [RFC v2 2/2] pmem: device flush over VIRTIO Pankaj Gupta 2018-04-25 11:24 ` [Qemu-devel] " Pankaj Gupta 2018-04-25 11:24 ` Pankaj Gupta [not found] ` <20180425112415.12327-3-pagupta-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> 2018-04-25 14:23 ` Dan Williams 2018-04-25 14:23 ` [Qemu-devel] " Dan Williams 2018-04-25 14:23 ` Dan Williams [not found] ` <CAPcyv4gpZzKfE7jY1peYOVd6sVhNz7jce1s_xNH_2Lt8AjRK-Q-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 2018-04-25 14:47 ` Pankaj Gupta 2018-04-25 14:47 ` [Qemu-devel] " Pankaj Gupta 2018-04-25 14:47 ` Pankaj Gupta 2018-04-26 13:15 ` Stefan Hajnoczi 2018-04-26 13:15 ` [Qemu-devel] " Stefan Hajnoczi [not found] ` <20180426131517.GB30991-lxVrvc10SDRcolVlb+j0YCZi+YwRKgec@public.gmane.org> 2018-04-26 16:40 ` Pankaj Gupta 2018-04-26 16:40 ` [Qemu-devel] " Pankaj Gupta 2018-04-26 16:40 ` Pankaj Gupta [not found] ` <58645254.23011245.1524760853269.JavaMail.zimbra-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> 2018-04-26 16:57 ` Dan Williams 2018-04-26 16:57 ` [Qemu-devel] " Dan Williams 2018-04-26 16:57 ` Dan Williams [not found] ` <CAPcyv4jv-hJNKJxak98T7aCnWztVEDTE8o=8fjvOrVmrTfyjdA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 2018-04-26 17:13 ` Pankaj Gupta 2018-04-26 17:13 ` [Qemu-devel] " Pankaj Gupta 2018-04-26 17:13 ` Pankaj Gupta 2018-04-25 11:24 ` [RFC v2] qemu: Add virtio pmem device Pankaj Gupta 2018-04-25 11:24 ` [Qemu-devel] " Pankaj Gupta 2018-04-25 11:24 ` Pankaj Gupta [not found] ` <20180425112415.12327-4-pagupta-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> 2018-04-25 11:35 ` [Qemu-devel] " no-reply-isE1Te71pDtAfugRpC6u6w 2018-04-25 11:35 ` no-reply 2018-04-25 11:35 ` no-reply 2018-04-25 11:58 ` Pankaj Gupta 2018-04-25 11:58 ` Pankaj Gupta 2018-04-25 14:23 ` Eric Blake 2018-04-25 14:23 ` Eric Blake [not found] ` <79f72139-0fcb-3d5e-a16c-24f3b5ee1a07-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> 2018-04-25 14:51 ` Pankaj Gupta 2018-04-25 14:51 ` Pankaj Gupta 2018-04-25 11:46 ` no-reply 2018-04-25 11:46 ` no-reply 2018-04-25 11:46 ` no-reply 2018-04-25 14:25 ` Eric Blake 2018-04-25 14:25 ` Eric Blake [not found] ` <25f3e433-cfa6-4a62-ba7f-47aef1119dfc-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> 2018-04-25 14:55 ` Pankaj Gupta 2018-04-25 14:55 ` Pankaj Gupta 2018-04-26 13:24 ` Stefan Hajnoczi 2018-04-26 13:24 ` [Qemu-devel] " Stefan Hajnoczi [not found] ` <20180426132406.GC30991-lxVrvc10SDRcolVlb+j0YCZi+YwRKgec@public.gmane.org> 2018-04-26 16:43 ` Pankaj Gupta 2018-04-26 16:43 ` Pankaj Gupta 2018-06-01 12:24 ` [Qemu-devel] [RFC v2 0/2] kvm "fake DAX" device flushing Igor Mammedov 2018-06-01 12:24 ` Igor Mammedov [not found] ` <20180601142410.5c986f13-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> 2018-06-04 5:56 ` Pankaj Gupta 2018-06-04 5:56 ` Pankaj Gupta 2018-06-04 9:55 ` David Hildenbrand 2018-06-04 9:55 ` David Hildenbrand
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=1499190564.23017177.1524762938762.JavaMail.zimbra@redhat.com \ --to=pagupta-h+wxahxf7alqt0dzr+alfa@public.gmane.org \ --cc=dan.j.williams-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org \ --cc=david-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org \ --cc=hch-wEGCiKHe2LqWVfeAwA7xHQ@public.gmane.org \ --cc=imammedo-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org \ --cc=jack-AlSwsSmVLrQ@public.gmane.org \ --cc=jmoyer-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org \ --cc=kvm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \ --cc=kwolf-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org \ --cc=lcapitulino-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org \ --cc=linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \ --cc=linux-mm-Bw31MaZKKs3YtjvyW6yDsg@public.gmane.org \ --cc=linux-nvdimm-y27Ovi1pjclAfugRpC6u6w@public.gmane.org \ --cc=marcel-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org \ --cc=mst-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org \ --cc=nilal-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org \ --cc=niteshnarayanlal-PkbjNfxxIARBDgjK7y7TUQ@public.gmane.org \ --cc=pbonzini-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org \ --cc=qemu-devel-qX2TKyscuCcdnm+yROfE0A@public.gmane.org \ --cc=riel-ebMLmSuQjDVBDgjK7y7TUQ@public.gmane.org \ --cc=ross.zwisler-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org \ --cc=stefanha-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org \ --cc=xiaoguangrong.eric-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.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: linkBe 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.