From patchwork Thu Dec 6 15:08:26 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Paul Durrant X-Patchwork-Id: 1008867 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Authentication-Results: ozlabs.org; spf=pass (mailfrom) smtp.mailfrom=nongnu.org (client-ip=2001:4830:134:3::11; helo=lists.gnu.org; envelope-from=qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org; receiver=) Authentication-Results: ozlabs.org; dmarc=none (p=none dis=none) header.from=citrix.com Received: from lists.gnu.org (lists.gnu.org [IPv6:2001:4830:134:3::11]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by ozlabs.org (Postfix) with ESMTPS id 439g0s6y0Zz9s47 for ; Fri, 7 Dec 2018 02:45:33 +1100 (AEDT) Received: from localhost ([::1]:41552 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1gUvqF-0005TZ-9N for incoming@patchwork.ozlabs.org; Thu, 06 Dec 2018 10:45:31 -0500 Received: from eggs.gnu.org ([2001:4830:134:3::10]:47808) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1gUvXR-0006qg-Nk for qemu-devel@nongnu.org; Thu, 06 Dec 2018 10:26:09 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1gUvHo-0005FS-Qu for qemu-devel@nongnu.org; Thu, 06 Dec 2018 10:10:01 -0500 Received: from smtp03.citrix.com ([162.221.156.55]:39818) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1gUvHo-0005AU-G1; Thu, 06 Dec 2018 10:09:56 -0500 X-IronPort-AV: E=Sophos;i="5.56,322,1539648000"; d="scan'208";a="72554790" From: Paul Durrant To: , , Date: Thu, 6 Dec 2018 15:08:26 +0000 Message-ID: <1544108924-10841-1-git-send-email-paul.durrant@citrix.com> X-Mailer: git-send-email 2.1.4 MIME-Version: 1.0 X-detected-operating-system: by eggs.gnu.org: Genre and OS details not recognized. X-Received-From: 162.221.156.55 Subject: [Qemu-devel] [PATCH v2 00/18] Xen PV backend 'qdevification' X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Kevin Wolf , Stefano Stabellini , Eduardo Habkost , "Michael S. Tsirkin" , Jason Wang , Tim Smith , Stefan Hajnoczi , Greg Kurz , Max Reitz , Paul Durrant , Gerd Hoffmann , Paolo Bonzini , Anthony Perard , =?utf-8?q?Marc-Andr=C3=A9_Lureau?= , Richard Henderson Errors-To: qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org Sender: "Qemu-devel" 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 Cc: Eduardo Habkost Cc: Gerd Hoffmann Cc: Greg Kurz Cc: Jason Wang Cc: Kevin Wolf Cc: "Marc-André Lureau" Cc: Marcel Apfelbaum Cc: Max Reitz Cc: "Michael S. Tsirkin" Cc: Paolo Bonzini Cc: Richard Henderson Cc: Stefan Hajnoczi Cc: Stefano Stabellini Cc: Tim Smith v2: - Fix boilerplates of introduced files and leave existing ones alone