From patchwork Wed Nov 21 15:11:53 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Paul Durrant X-Patchwork-Id: 10692657 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork-2.web.codeaurora.org (Postfix) with ESMTP id 320EA13AD for ; Wed, 21 Nov 2018 15:14:17 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 212422BE9E for ; Wed, 21 Nov 2018 15:14:17 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 151482BED4; Wed, 21 Nov 2018 15:14:17 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-7.9 required=2.0 tests=BAYES_00,MAILING_LIST_MULTI, RCVD_IN_DNSWL_HI autolearn=ham version=3.3.1 Received: from lists.gnu.org (lists.gnu.org [208.118.235.17]) (using TLSv1 with cipher AES256-SHA (256/256 bits)) (No client certificate requested) by mail.wl.linuxfoundation.org (Postfix) with ESMTPS id A2ED02BE9E for ; Wed, 21 Nov 2018 15:14:15 +0000 (UTC) Received: from localhost ([::1]:39690 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1gPUCk-0007gA-Mt for patchwork-qemu-devel@patchwork.kernel.org; Wed, 21 Nov 2018 10:14:14 -0500 Received: from eggs.gnu.org ([2001:4830:134:3::10]:46084) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1gPUB2-0005Rt-1c for qemu-devel@nongnu.org; Wed, 21 Nov 2018 10:12:30 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1gPUB0-0003bG-3x for qemu-devel@nongnu.org; Wed, 21 Nov 2018 10:12:28 -0500 Received: from smtp03.citrix.com ([162.221.156.55]:55925) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1gPUAu-0003R3-LK; Wed, 21 Nov 2018 10:12:20 -0500 X-IronPort-AV: E=Sophos;i="5.56,261,1539648000"; d="scan'208";a="71205791" From: Paul Durrant To: , , Date: Wed, 21 Nov 2018 15:11:53 +0000 Message-ID: <20181121151211.15997-1-paul.durrant@citrix.com> X-Mailer: git-send-email 2.11.0 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 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_Lurea?= =?utf-8?q?u?= , Richard Henderson Errors-To: qemu-devel-bounces+patchwork-qemu-devel=patchwork.kernel.org@nongnu.org Sender: "Qemu-devel" X-Virus-Scanned: ClamAV using ClamSMTP 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/qom23 Paul Durrant (18): xen: re-name XenDevice to XenLegacyDevice... xen: introduce new 'XenBus' and 'XenDevice' object hierarchy xen: introduce 'xen-qdisk' xen: create xenstore areas for XenDevice-s xen: add xenstore watcher infratructure 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-qdisk.c xen: remove unnecessary code from dataplane/xen-qdisk.c xen: add header and build dataplane/xen-qdisk.c xen: remove 'XenBlkDev' and 'blkdev' names from dataplane/xen-qdisk xen: remove 'ioreq' struct/varable/field names from dataplane/xen-qdisk.c xen: purge 'blk' and 'ioreq' from function names in dataplane/xen-qdisk.c xen: add implementations of xen-qdisk connect and disconnect functions... xen: add a mechanism to automatically create XenDevice-s... xen: automatically create XenQdiskDevice-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-qdisk.c | 756 +++++++++++++++ hw/block/dataplane/xen-qdisk.h | 25 + hw/block/trace-events | 7 + hw/block/xen-qdisk.c | 698 +++++++++++++ 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 | 17 + hw/xen/xen-backend.c | 67 ++ hw/xen/xen-bus-helper.c | 152 +++ hw/xen/xen-bus.c | 1021 ++++++++++++++++++++ hw/xen/xen-common.c | 2 +- hw/xen/{xen_backend.c => xen-legacy-backend.c} | 74 +- 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 | 24 + include/hw/xen/xen-bus-helper.h | 31 + include/hw/xen/xen-bus.h | 132 +++ .../hw/xen/{xen_backend.h => xen-legacy-backend.h} | 43 +- include/hw/xen/xen-qdisk.h | 51 + include/hw/xen/xen_pvdev.h | 38 +- include/qemu/module.h | 3 + 38 files changed, 3150 insertions(+), 1150 deletions(-) create mode 100644 hw/block/dataplane/xen-qdisk.c create mode 100644 hw/block/dataplane/xen-qdisk.h create mode 100644 hw/block/xen-qdisk.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 rename hw/xen/{xen_backend.c => xen-legacy-backend.c} (89%) create mode 100644 include/hw/xen/xen-backend.h create mode 100644 include/hw/xen/xen-bus-helper.h create mode 100644 include/hw/xen/xen-bus.h rename include/hw/xen/{xen_backend.h => xen-legacy-backend.h} (61%) create mode 100644 include/hw/xen/xen-qdisk.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