mbox

[PULL,00/25] VirtFS update

Message ID 1452578912-12546-1-git-send-email-aneesh.kumar@linux.vnet.ibm.com
State New
Headers show

Pull-request

https://github.com/kvaneesh/qemu.git tags/for-upstream-signed

Message

Aneesh Kumar K.V Jan. 12, 2016, 6:08 a.m. UTC
The following changes since commit a7e00e2536941a6e570b45b7ab4afec4505ff67e:

  petalogix-ml605: Set the MicroBlaze CPU version to 8.10.a (2016-01-07 14:57:26 +0100)

are available in the git repository at:

  https://github.com/kvaneesh/qemu.git tags/for-upstream-signed

for you to fetch changes up to 00588a0aa2ade2e32a552633bbbefdc6ae5e32a2:

  9pfs: introduce V9fsVirtioState (2016-01-12 11:04:14 +0530)

----------------------------------------------------------------
VirtFS update:

Cleanups mostly isolating virtio related details into separate files. This
is done to enable easy addition of Xen transport for VirtFS.

The changes include:

1. Rename a bunch of files and functions to make clear they are generic.
2. disentangle virtio transport code and generic 9pfs code.
3. Some function name clean-up.

----------------------------------------------------------------

Note: We do get few checkpatch.pl warning for typedef as below.
"ERROR: open brace '{' following struct go on the same line"
But VirtFS code always had open brace in the line next to typedef.
I guess we should handle that in a separate patch

Wei Liu (25):
      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-local.c to 9p-local.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: 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 9p.h from virtio-9p.h
      9pfs: factor out virtio_9p_push_and_notify
      9pfs: export pdu_{submit,alloc,free}
      9pfs: move handle_9p_output and make it static function
      9pfs: rename virtio_9p_set_fd_limit to use v9fs_ prefix
      9pfs: rename virtio-9p.c to 9p.c
      9pfs: factor out v9fs_device_{,un}realize_common
      9pfs: introduce V9fsVirtioState

 Makefile                                           |   2 +-
 fsdev/{virtio-9p-marshal.c => 9p-iov-marshal.c}    | 175 +++++----
 fsdev/9p-iov-marshal.h                             |  18 +
 fsdev/9p-marshal.c                                 |  56 +++
 fsdev/{virtio-9p-marshal.h => 9p-marshal.h}        |  12 +-
 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}                      | 260 +++++++++-----
 hw/9pfs/9p.h                                       | 326 +++++++++++++++++
 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}               |   6 +-
 hw/9pfs/coxattr.c                                  |   2 +-
 hw/9pfs/virtio-9p-device.c                         | 201 ++++++-----
 hw/9pfs/virtio-9p.h                                | 391 +--------------------
 hw/s390x/virtio-ccw.h                              |   2 +-
 hw/virtio/virtio-pci.h                             |   3 +-
 include/hw/virtio/virtio-9p.h                      |  24 --
 31 files changed, 835 insertions(+), 745 deletions(-)
 rename fsdev/{virtio-9p-marshal.c => 9p-iov-marshal.c} (59%)
 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} (94%)
 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} (98%)
 delete mode 100644 include/hw/virtio/virtio-9p.h

*** BLURB HERE ***

Wei Liu (25):
  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-local.c to 9p-local.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: 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 9p.h from virtio-9p.h
  9pfs: factor out virtio_9p_push_and_notify
  9pfs: export pdu_{submit,alloc,free}
  9pfs: move handle_9p_output and make it static function
  9pfs: rename virtio_9p_set_fd_limit to use v9fs_ prefix
  9pfs: rename virtio-9p.c to 9p.c
  9pfs: factor out v9fs_device_{,un}realize_common
  9pfs: introduce V9fsVirtioState

 Makefile                                           |   2 +-
 fsdev/{virtio-9p-marshal.c => 9p-iov-marshal.c}    | 175 +++++----
 fsdev/9p-iov-marshal.h                             |  18 +
 fsdev/9p-marshal.c                                 |  56 +++
 fsdev/{virtio-9p-marshal.h => 9p-marshal.h}        |  12 +-
 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}                      | 260 +++++++++-----
 hw/9pfs/9p.h                                       | 326 +++++++++++++++++
 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}               |   6 +-
 hw/9pfs/coxattr.c                                  |   2 +-
 hw/9pfs/virtio-9p-device.c                         | 201 ++++++-----
 hw/9pfs/virtio-9p.h                                | 391 +--------------------
 hw/s390x/virtio-ccw.h                              |   2 +-
 hw/virtio/virtio-pci.h                             |   3 +-
 include/hw/virtio/virtio-9p.h                      |  24 --
 31 files changed, 835 insertions(+), 745 deletions(-)
 rename fsdev/{virtio-9p-marshal.c => 9p-iov-marshal.c} (59%)
 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} (94%)
 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} (98%)
 delete mode 100644 include/hw/virtio/virtio-9p.h

Comments

Peter Maydell Jan. 12, 2016, 6:10 p.m. UTC | #1
On 12 January 2016 at 06:08, Aneesh Kumar K.V
<aneesh.kumar@linux.vnet.ibm.com> wrote:
> The following changes since commit a7e00e2536941a6e570b45b7ab4afec4505ff67e:
>
>   petalogix-ml605: Set the MicroBlaze CPU version to 8.10.a (2016-01-07 14:57:26 +0100)
>
> are available in the git repository at:
>
>   https://github.com/kvaneesh/qemu.git tags/for-upstream-signed
>
> for you to fetch changes up to 00588a0aa2ade2e32a552633bbbefdc6ae5e32a2:
>
>   9pfs: introduce V9fsVirtioState (2016-01-12 11:04:14 +0530)
>
> ----------------------------------------------------------------
> VirtFS update:
>
> Cleanups mostly isolating virtio related details into separate files. This
> is done to enable easy addition of Xen transport for VirtFS.
>
> The changes include:
>
> 1. Rename a bunch of files and functions to make clear they are generic.
> 2. disentangle virtio transport code and generic 9pfs code.
> 3. Some function name clean-up.
>
> ----------------------------------------------------------------

Applied, thanks.

-- PMM