[Qemu-devel] [PATCH v2 00/18] Xen PV backend 'qdevification'

Paul Durrant posted 18 patches 1 week ago
Test checkpatch passed
Test docker-quick@centos7 passed
Test docker-clang@ubuntu failed
Test docker-mingw@fedora passed
Test asan passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/1544108924-10841-1-git-send-email-paul.durrant@citrix.com
MAINTAINERS                         |    5 +-
hw/9pfs/xen-9p-backend.c            |   16 +-
hw/block/Makefile.objs              |    2 +-
hw/block/dataplane/Makefile.objs    |    1 +
hw/block/dataplane/xen-block.c      |  814 ++++++++++++++++++++++++++
hw/block/dataplane/xen-block.h      |   29 +
hw/block/trace-events               |   11 +
hw/block/xen-block.c                |  853 +++++++++++++++++++++++++++
hw/block/xen_disk.c                 | 1011 --------------------------------
hw/char/xen_console.c               |   12 +-
hw/display/xenfb.c                  |   25 +-
hw/i386/xen/xen-hvm.c               |    5 +-
hw/i386/xen/xen-mapcache.c          |    2 +-
hw/i386/xen/xen_platform.c          |    2 +-
hw/net/xen_nic.c                    |   14 +-
hw/usb/xen-usb.c                    |   25 +-
hw/xen/Makefile.objs                |    2 +-
hw/xen/trace-events                 |   25 +
hw/xen/xen-backend.c                |   69 +++
hw/xen/xen-bus-helper.c             |  181 ++++++
hw/xen/xen-bus.c                    | 1094 +++++++++++++++++++++++++++++++++++
hw/xen/xen-common.c                 |    2 +-
hw/xen/xen-legacy-backend.c         |  853 +++++++++++++++++++++++++++
hw/xen/xen_backend.c                |  845 ---------------------------
hw/xen/xen_devconfig.c              |    2 +-
hw/xen/xen_pt.c                     |    2 +-
hw/xen/xen_pt_config_init.c         |    2 +-
hw/xen/xen_pt_graphics.c            |    2 +-
hw/xen/xen_pt_msi.c                 |    2 +-
hw/xen/xen_pvdev.c                  |   20 +-
hw/xenpv/xen_domainbuild.c          |    2 +-
hw/xenpv/xen_machine_pv.c           |    5 +-
include/hw/xen/xen-backend.h        |   26 +
include/hw/xen/xen-block.h          |   79 +++
include/hw/xen/xen-bus-helper.h     |   40 ++
include/hw/xen/xen-bus.h            |  138 +++++
include/hw/xen/xen-legacy-backend.h |  104 ++++
include/hw/xen/xen_backend.h        |   99 ----
include/hw/xen/xen_pvdev.h          |   38 +-
include/qemu/module.h               |    3 +
40 files changed, 4420 insertions(+), 2042 deletions(-)
create mode 100644 hw/block/dataplane/xen-block.c
create mode 100644 hw/block/dataplane/xen-block.h
create mode 100644 hw/block/xen-block.c
delete mode 100644 hw/block/xen_disk.c
create mode 100644 hw/xen/xen-backend.c
create mode 100644 hw/xen/xen-bus-helper.c
create mode 100644 hw/xen/xen-bus.c
create mode 100644 hw/xen/xen-legacy-backend.c
delete mode 100644 hw/xen/xen_backend.c
create mode 100644 include/hw/xen/xen-backend.h
create mode 100644 include/hw/xen/xen-block.h
create mode 100644 include/hw/xen/xen-bus-helper.h
create mode 100644 include/hw/xen/xen-bus.h
create mode 100644 include/hw/xen/xen-legacy-backend.h
delete mode 100644 include/hw/xen/xen_backend.h

[Qemu-devel] [PATCH v2 00/18] Xen PV backend 'qdevification'

Posted by Paul Durrant 1 week ago
This series introduces a new QOM compliant framework for Xen PV backends.
This is achieved by first moving the current non-compliant framework aside,
before building up a new framework incrementally.

This series was prompted by a thread [1] started by Kevin Wolf in response
to patches against xen_disk.c posted by Tim Smith. Therefore, alongside
the patches introducing the new framework, other patches build up a QOM
compliant replacement for 'xen_disk', called 'xen-qdisk'. Patch #16 swaps
this new device into place (having establisheda mechanism to auto-
instantiate devices that is compliant with existing Xen toolstacks in
patch #15) and patch #18 then removes the old xen_disk code.

Subsequent series will port other Xen PV backends across to the new
framework.

The series is also available as a repository branch [2] on xenbits.xen.org.

[1] https://lists.gnu.org/archive/html/qemu-devel/2018-11/msg00259.html
[2] http://xenbits.xen.org/gitweb/?p=people/pauldu/qemu.git;a=shortlog;h=refs/heads/qom27

Paul Durrant (18):
  xen: re-name XenDevice to XenLegacyDevice...
  xen: introduce new 'XenBus' and 'XenDevice' object hierarchy
  xen: introduce 'xen-block', 'xen-disk' and 'xen-cdrom'
  xen: create xenstore areas for XenDevice-s
  xen: add xenstore watcher infrastructure
  xen: add grant table interface for XenDevice-s
  xen: add event channel interface for XenDevice-s
  xen: duplicate xen_disk.c as basis of dataplane/xen-block.c
  xen: remove unnecessary code from dataplane/xen-block.c
  xen: add header and build dataplane/xen-block.c
  xen: remove 'XenBlkDev' and 'blkdev' names from dataplane/xen-block
  xen: remove 'ioreq' struct/varable/field names from
    dataplane/xen-block.c
  xen: purge 'blk' and 'ioreq' from function names in
    dataplane/xen-block.c
  xen: add implementations of xen-block connect and disconnect
    functions...
  xen: add a mechanism to automatically create XenDevice-s...
  xen: automatically create XenBlockDevice-s
  MAINTAINERS: add myself as a Xen maintainer
  xen: remove the legacy 'xen_disk' backend

 MAINTAINERS                         |    5 +-
 hw/9pfs/xen-9p-backend.c            |   16 +-
 hw/block/Makefile.objs              |    2 +-
 hw/block/dataplane/Makefile.objs    |    1 +
 hw/block/dataplane/xen-block.c      |  814 ++++++++++++++++++++++++++
 hw/block/dataplane/xen-block.h      |   29 +
 hw/block/trace-events               |   11 +
 hw/block/xen-block.c                |  853 +++++++++++++++++++++++++++
 hw/block/xen_disk.c                 | 1011 --------------------------------
 hw/char/xen_console.c               |   12 +-
 hw/display/xenfb.c                  |   25 +-
 hw/i386/xen/xen-hvm.c               |    5 +-
 hw/i386/xen/xen-mapcache.c          |    2 +-
 hw/i386/xen/xen_platform.c          |    2 +-
 hw/net/xen_nic.c                    |   14 +-
 hw/usb/xen-usb.c                    |   25 +-
 hw/xen/Makefile.objs                |    2 +-
 hw/xen/trace-events                 |   25 +
 hw/xen/xen-backend.c                |   69 +++
 hw/xen/xen-bus-helper.c             |  181 ++++++
 hw/xen/xen-bus.c                    | 1094 +++++++++++++++++++++++++++++++++++
 hw/xen/xen-common.c                 |    2 +-
 hw/xen/xen-legacy-backend.c         |  853 +++++++++++++++++++++++++++
 hw/xen/xen_backend.c                |  845 ---------------------------
 hw/xen/xen_devconfig.c              |    2 +-
 hw/xen/xen_pt.c                     |    2 +-
 hw/xen/xen_pt_config_init.c         |    2 +-
 hw/xen/xen_pt_graphics.c            |    2 +-
 hw/xen/xen_pt_msi.c                 |    2 +-
 hw/xen/xen_pvdev.c                  |   20 +-
 hw/xenpv/xen_domainbuild.c          |    2 +-
 hw/xenpv/xen_machine_pv.c           |    5 +-
 include/hw/xen/xen-backend.h        |   26 +
 include/hw/xen/xen-block.h          |   79 +++
 include/hw/xen/xen-bus-helper.h     |   40 ++
 include/hw/xen/xen-bus.h            |  138 +++++
 include/hw/xen/xen-legacy-backend.h |  104 ++++
 include/hw/xen/xen_backend.h        |   99 ----
 include/hw/xen/xen_pvdev.h          |   38 +-
 include/qemu/module.h               |    3 +
 40 files changed, 4420 insertions(+), 2042 deletions(-)
 create mode 100644 hw/block/dataplane/xen-block.c
 create mode 100644 hw/block/dataplane/xen-block.h
 create mode 100644 hw/block/xen-block.c
 delete mode 100644 hw/block/xen_disk.c
 create mode 100644 hw/xen/xen-backend.c
 create mode 100644 hw/xen/xen-bus-helper.c
 create mode 100644 hw/xen/xen-bus.c
 create mode 100644 hw/xen/xen-legacy-backend.c
 delete mode 100644 hw/xen/xen_backend.c
 create mode 100644 include/hw/xen/xen-backend.h
 create mode 100644 include/hw/xen/xen-block.h
 create mode 100644 include/hw/xen/xen-bus-helper.h
 create mode 100644 include/hw/xen/xen-bus.h
 create mode 100644 include/hw/xen/xen-legacy-backend.h
 delete mode 100644 include/hw/xen/xen_backend.h
---
Cc: Anthony Perard <anthony.perard@citrix.com>
Cc: Eduardo Habkost <ehabkost@redhat.com>
Cc: Gerd Hoffmann <kraxel@redhat.com>
Cc: Greg Kurz <groug@kaod.org>
Cc: Jason Wang <jasowang@redhat.com>
Cc: Kevin Wolf <kwolf@redhat.com>
Cc: "Marc-André Lureau" <marcandre.lureau@redhat.com>
Cc: Marcel Apfelbaum <marcel.apfelbaum@gmail.com>
Cc: Max Reitz <mreitz@redhat.com>
Cc: "Michael S. Tsirkin" <mst@redhat.com>
Cc: Paolo Bonzini <pbonzini@redhat.com>
Cc: Richard Henderson <rth@twiddle.net>
Cc: Stefan Hajnoczi <stefanha@redhat.com>
Cc: Stefano Stabellini <sstabellini@kernel.org>
Cc: Tim Smith <tim.smith@citrix.com>

v2:
 - Fix boilerplates of introduced files and leave existing ones alone
-- 
2.1.4