From: Greg Kurz <gkurz@linux.vnet.ibm.com>
To: Stefano Stabellini <stefano.stabellini@eu.citrix.com>
Cc: qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [PATCH 00/22] 9pfs: disentangling virtio and generic code
Date: Thu, 7 Jan 2016 14:35:56 +0100 [thread overview]
Message-ID: <20160107143556.13db4691@bahia.local> (raw)
In-Reply-To: <20160107133243.509cc369@bahia.local>
On Thu, 7 Jan 2016 13:32:43 +0100
Greg Kurz <gkurz@linux.vnet.ibm.com> wrote:
> On Thu, 7 Jan 2016 11:59:22 +0000
> Stefano Stabellini <stefano.stabellini@eu.citrix.com> wrote:
>
> > On Tue, 5 Jan 2016, Wei Liu wrote:
> > > Hi all
> > >
> > > Back in 2015 summer one of our OPW interns Linda Jacobson explored the
> > > possibility of making 9pfs work on Xen. It turned out lots of code in QEMU can
> > > be reused.
> > >
> > > This series refactors 9pfs related code:
> > >
> > > 1. Rename a bunch of files and functions to make clear they are generic.
> > > 2. Only export two functions (marshal and unmarshal) from transport to generic
> > > code.
> > > 3. disentangle virtio transport code and generic 9pfs code.
> > > 4. Some function name clean-up.
> >
> > The series looks pretty good and it looks like a clean improvement over
> > the current code base.
> >
Stefano,
Can I add your Reviewed-by to the patches ?
--
Greg
>
> I agree Wei did a great job. I'll fix the title in patch 3 and push it to
> my tree next week.
>
> Thanks.
>
> --
> Greg
>
> >
> >
> > > To make sure this series doesn't break compilation a rune is use to compile
> > > every commit.
> > >
> > > $ git rebase -i origin/master --exec "make -j16 clean; ./configure --target-list=x86_64-softmmu --enable-virtfs --enable-kvm; make -j16 && echo ok."
> > >
> > > Three use cases are tested:
> > >
> > > 1. Local file system driver with passthrough security policy
> > > ./qemu-system-x86_64 -L . -hda /dev/DATA/jessie -vnc 0.0.0.0:0,to=99 -fsdev local,path=/root/qemu,security_model=passthrough,id=fs1 -device virtio-9p-pci,fsdev=fs1,mount_tag=qemu &
> > >
> > > 2. Local file system driver with mapped security policy
> > > ./qemu-system-x86_64 -L . -hda /dev/DATA/jessie -vnc 0.0.0.0:0,to=99 -fsdev local,path=/root/qemu,security_model=mapped,id=fs1 -device virtio-9p-pci,fsdev=fs1,mount_tag=qemu &
> > >
> > > 3. Proxy file system driver
> > > ./virtfs-proxy-helper -p /root/qemu -n -s virtfs-helper-sock -u 0 -g 0 &
> > > ./qemu-system-x86_64 -L . -hda /dev/DATA/jessie -vnc 0.0.0.0:0,to=99 -fsdev proxy,socket=virtfs-helper-sock,id=fs1 -device virtio-9p-pci,fsdev=fs1,mount_tag=qemu &
> > >
> > > During each of the tests, mounting, unmounting, read and write operations are
> > > performed. In "mapped" test, getfattr in host was used to inspect the
> > > attributes.
> > >
> > > Let me know if you would like to see more tests.
> > >
> > > Xen transport is still under development. I figure it would be better to post
> > > this series as soon as possible because rebasing such huge series from time to
> > > time is prone to error.
> > >
> > > Comments are welcome. Thanks!
> > >
> > > Wei.
> > > ---
> > > Cc: "Aneesh Kumar K.V" <aneesh.kumar@linux.vnet.ibm.com>
> > > Cc: Greg Kurz <gkurz@linux.vnet.ibm.com>
> > > Cc: "Michael S. Tsirkin" <mst@redhat.com>
> > > Cc: Stefano Stabellini <stefano.stabellini@eu.citrix.com>
> > > ---
> > >
> > > Wei Liu (22):
> > > 9pfs: rename virtio-9p-coth.{c,h} to coth.{c,h}
> > > 9pfs: rename virtio-9p-handle.c to 9p-handle.c
> > > 9pfs: rename virtio-9p-handle.c to 9p-handle.c
> > > 9pfs: rename virtio-9p-posix-acl.c to 9p-posix-acl.c
> > > 9pfs: rename virtio-9p-proxy.{c,h} to 9p-proxy.{c,h}
> > > 9pfs: rename virtio-9p-synth.{c,h} to 9p-synth.{c,h}
> > > 9pfs: rename virtio-9p-xattr{,-user}.{c,h} to 9p-xattr{,-user}.{c,h}
> > > 9pfs: merge hw/virtio/virtio-9p.h into hw/9pfs/virtio-9p.h
> > > 9pfs: remove dead code
> > > fsdev: break out 9p-marshal.{c,h} from virtio-9p-marshal.{c,h}
> > > fsdev: 9p-marshal: introduce V9fsBlob
> > > 9pfs: use V9fsBlob to transmit xattr
> > > fsdev: rename virtio-9p-marshal.{c,h} to 9p-iov-marshal.{c,h}
> > > 9pfs: PDU processing functions don't need to take V9fsState as
> > > argument
> > > 9pfs: PDU processing functions should start pdu_ prefix
> > > 9pfs: make pdu_{,un}marshal proper functions
> > > 9pfs: factor out virtio_pdu_{,un}marshal
> > > 9pfs: factor out pdu_push_and_notify
> > > 9pfs: break out virtio_init_iov_from_pdu
> > > 9pfs: break out generic code from virtio-9p.{c,h}
> > > 9pfs: factor out v9fs_device_{,un}realize_common
> > > 9pfs: disentangle V9fsState
> > >
> > > Makefile | 2 +-
> > > fsdev/{virtio-9p-marshal.c => 9p-iov-marshal.c} | 205 ++++++-----
> > > fsdev/9p-iov-marshal.h | 15 +
> > > fsdev/9p-marshal.c | 64 ++++
> > > fsdev/{virtio-9p-marshal.h => 9p-marshal.h} | 26 +-
> > > fsdev/Makefile.objs | 2 +-
> > > fsdev/virtfs-proxy-helper.c | 6 +-
> > > hw/9pfs/{virtio-9p-handle.c => 9p-handle.c} | 7 +-
> > > hw/9pfs/{virtio-9p-local.c => 9p-local.c} | 7 +-
> > > hw/9pfs/{virtio-9p-posix-acl.c => 9p-posix-acl.c} | 7 +-
> > > hw/9pfs/{virtio-9p-proxy.c => 9p-proxy.c} | 7 +-
> > > hw/9pfs/{virtio-9p-proxy.h => 9p-proxy.h} | 10 +-
> > > hw/9pfs/{virtio-9p-synth.c => 9p-synth.c} | 6 +-
> > > hw/9pfs/{virtio-9p-synth.h => 9p-synth.h} | 6 +-
> > > .../{virtio-9p-xattr-user.c => 9p-xattr-user.c} | 7 +-
> > > hw/9pfs/{virtio-9p-xattr.c => 9p-xattr.c} | 7 +-
> > > hw/9pfs/{virtio-9p-xattr.h => 9p-xattr.h} | 6 +-
> > > hw/9pfs/{virtio-9p.c => 9p.c} | 301 ++++++++++------
> > > hw/9pfs/9p.h | 335 ++++++++++++++++++
> > > hw/9pfs/Makefile.objs | 14 +-
> > > hw/9pfs/codir.c | 2 +-
> > > hw/9pfs/cofile.c | 2 +-
> > > hw/9pfs/cofs.c | 2 +-
> > > hw/9pfs/{virtio-9p-coth.c => coth.c} | 4 +-
> > > hw/9pfs/{virtio-9p-coth.h => coth.h} | 15 +-
> > > hw/9pfs/coxattr.c | 2 +-
> > > hw/9pfs/virtio-9p-device.c | 196 ++++++-----
> > > hw/9pfs/virtio-9p.h | 391 +--------------------
> > > hw/s390x/virtio-ccw.h | 2 +-
> > > hw/virtio/virtio-pci.h | 4 +-
> > > include/hw/virtio/virtio-9p.h | 24 --
> > > 31 files changed, 927 insertions(+), 757 deletions(-)
> > > rename fsdev/{virtio-9p-marshal.c => 9p-iov-marshal.c} (53%)
> > > create mode 100644 fsdev/9p-iov-marshal.h
> > > create mode 100644 fsdev/9p-marshal.c
> > > rename fsdev/{virtio-9p-marshal.h => 9p-marshal.h} (78%)
> > > rename hw/9pfs/{virtio-9p-handle.c => 9p-handle.c} (99%)
> > > rename hw/9pfs/{virtio-9p-local.c => 9p-local.c} (99%)
> > > rename hw/9pfs/{virtio-9p-posix-acl.c => 9p-posix-acl.c} (97%)
> > > rename hw/9pfs/{virtio-9p-proxy.c => 9p-proxy.c} (99%)
> > > rename hw/9pfs/{virtio-9p-proxy.h => 9p-proxy.h} (89%)
> > > rename hw/9pfs/{virtio-9p-synth.c => 9p-synth.c} (99%)
> > > rename hw/9pfs/{virtio-9p-synth.h => 9p-synth.h} (94%)
> > > rename hw/9pfs/{virtio-9p-xattr-user.c => 9p-xattr-user.c} (96%)
> > > rename hw/9pfs/{virtio-9p-xattr.c => 9p-xattr.c} (97%)
> > > rename hw/9pfs/{virtio-9p-xattr.h => 9p-xattr.h} (97%)
> > > rename hw/9pfs/{virtio-9p.c => 9p.c} (93%)
> > > create mode 100644 hw/9pfs/9p.h
> > > rename hw/9pfs/{virtio-9p-coth.c => coth.c} (95%)
> > > rename hw/9pfs/{virtio-9p-coth.h => coth.h} (96%)
> > > delete mode 100644 include/hw/virtio/virtio-9p.h
> > >
> > > --
> > > 2.1.4
> > >
> >
>
>
next prev parent reply other threads:[~2016-01-07 13:36 UTC|newest]
Thread overview: 46+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-01-05 19:05 [Qemu-devel] [PATCH 00/22] 9pfs: disentangling virtio and generic code Wei Liu
2016-01-05 19:05 ` [Qemu-devel] [PATCH 01/22] 9pfs: rename virtio-9p-coth.{c, h} to coth.{c, h} Wei Liu
2016-01-05 19:05 ` [Qemu-devel] [PATCH 02/22] 9pfs: rename virtio-9p-handle.c to 9p-handle.c Wei Liu
2016-01-05 19:05 ` [Qemu-devel] [PATCH 03/22] " Wei Liu
2016-01-06 11:22 ` Stefano Stabellini
2016-01-06 11:37 ` Wei Liu
2016-01-05 19:05 ` [Qemu-devel] [PATCH 04/22] 9pfs: rename virtio-9p-posix-acl.c to 9p-posix-acl.c Wei Liu
2016-01-05 19:05 ` [Qemu-devel] [PATCH 05/22] 9pfs: rename virtio-9p-proxy.{c, h} to 9p-proxy.{c, h} Wei Liu
2016-01-05 19:05 ` [Qemu-devel] [PATCH 06/22] 9pfs: rename virtio-9p-synth.{c, h} to 9p-synth.{c, h} Wei Liu
2016-01-05 19:05 ` [Qemu-devel] [PATCH 07/22] 9pfs: rename virtio-9p-xattr{, -user}.{c, h} to 9p-xattr{, -user}.{c, h} Wei Liu
2016-01-05 19:05 ` [Qemu-devel] [PATCH 08/22] 9pfs: merge hw/virtio/virtio-9p.h into hw/9pfs/virtio-9p.h Wei Liu
2016-01-05 19:05 ` [Qemu-devel] [PATCH 09/22] 9pfs: remove dead code Wei Liu
2016-01-07 16:41 ` Aneesh Kumar K.V
2016-01-07 16:57 ` Paolo Bonzini
2016-01-07 17:00 ` Greg Kurz
2016-01-05 19:05 ` [Qemu-devel] [PATCH 10/22] fsdev: break out 9p-marshal.{c, h} from virtio-9p-marshal.{c, h} Wei Liu
2016-01-07 16:42 ` Aneesh Kumar K.V
2016-01-07 17:07 ` Wei Liu
2016-01-05 19:05 ` [Qemu-devel] [PATCH 11/22] fsdev: 9p-marshal: introduce V9fsBlob Wei Liu
2016-01-05 19:05 ` [Qemu-devel] [PATCH 12/22] 9pfs: use V9fsBlob to transmit xattr Wei Liu
2016-01-07 16:49 ` Aneesh Kumar K.V
2016-01-07 17:03 ` Wei Liu
2016-01-05 19:05 ` [Qemu-devel] [PATCH 13/22] fsdev: rename virtio-9p-marshal.{c, h} to 9p-iov-marshal.{c, h} Wei Liu
2016-01-05 19:05 ` [Qemu-devel] [PATCH 14/22] 9pfs: PDU processing functions don't need to take V9fsState as argument Wei Liu
2016-01-05 19:05 ` [Qemu-devel] [PATCH 15/22] 9pfs: PDU processing functions should start pdu_ prefix Wei Liu
2016-01-05 19:05 ` [Qemu-devel] [PATCH 16/22] 9pfs: make pdu_{, un}marshal proper functions Wei Liu
2016-01-05 19:05 ` [Qemu-devel] [PATCH 17/22] 9pfs: factor out virtio_pdu_{, un}marshal Wei Liu
2016-01-05 19:05 ` [Qemu-devel] [PATCH 18/22] 9pfs: factor out pdu_push_and_notify Wei Liu
2016-01-05 19:05 ` [Qemu-devel] [PATCH 19/22] 9pfs: break out virtio_init_iov_from_pdu Wei Liu
2016-01-05 19:05 ` [Qemu-devel] [PATCH 20/22] 9pfs: break out generic code from virtio-9p.{c, h} Wei Liu
2016-01-07 16:57 ` Aneesh Kumar K.V
2016-01-07 17:16 ` Wei Liu
2016-01-07 17:28 ` Aneesh Kumar K.V
2016-01-07 17:38 ` Greg Kurz
2016-01-07 17:47 ` Wei Liu
2016-01-07 19:07 ` Wei Liu
2016-01-05 19:06 ` [Qemu-devel] [PATCH 21/22] 9pfs: factor out v9fs_device_{, un}realize_common Wei Liu
2016-01-05 19:06 ` [Qemu-devel] [PATCH 22/22] 9pfs: disentangle V9fsState Wei Liu
2016-01-07 11:59 ` [Qemu-devel] [PATCH 00/22] 9pfs: disentangling virtio and generic code Stefano Stabellini
2016-01-07 12:32 ` Greg Kurz
2016-01-07 13:35 ` Greg Kurz [this message]
2016-01-07 15:17 ` Stefano Stabellini
2016-01-07 17:06 ` Aneesh Kumar K.V
2016-01-07 17:15 ` Greg Kurz
2016-01-07 17:04 ` Aneesh Kumar K.V
2016-01-07 17:54 ` Wei Liu
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=20160107143556.13db4691@bahia.local \
--to=gkurz@linux.vnet.ibm.com \
--cc=qemu-devel@nongnu.org \
--cc=stefano.stabellini@eu.citrix.com \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).