From: Anthony PERARD <anthony.perard@citrix.com>
To: Paul Durrant <Paul.Durrant@citrix.com>
Cc: Stefano Stabellini <sstabellini@kernel.org>,
Eduardo Habkost <ehabkost@redhat.com>,
"qemu-block@nongnu.org" <qemu-block@nongnu.org>,
"Michael S. Tsirkin" <mst@redhat.com>,
"qemu-devel@nongnu.org" <qemu-devel@nongnu.org>,
Marcel Apfelbaum <marcel.apfelbaum@gmail.com>,
Paolo Bonzini <pbonzini@redhat.com>,
"xen-devel@lists.xenproject.org" <xen-devel@lists.xenproject.org>,
Richard Henderson <rth@twiddle.net>
Subject: Re: [PATCH 02/18] xen: introduce new 'XenBus' and 'XenDevice' object hierarchy
Date: Wed, 28 Nov 2018 17:32:01 +0000 [thread overview]
Message-ID: <20181128173201.GD14786@perard.uk.xensource.com> (raw)
In-Reply-To: <79c7ea5f1b8041d7828240ec18f881f4@AMSPEX02CL03.citrite.net>
On Wed, Nov 28, 2018 at 05:17:10PM +0000, Paul Durrant wrote:
> > Will we need a hotplug handler for this bus, like it is done with
> > TYPE_XENSYSBUS?
>
> I didn't seem to need one even doing 'xl block-attach' after the VM had booted. I'm really not sure what that does.
Indeed, that works fine, and I think I've tested block-detach as well.
Maybe we will need something if we initiate attach of a block device via
QMP instead of xenstore, but we can fix that later.
--
Anthony PERARD
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
WARNING: multiple messages have this Message-ID (diff)
From: Anthony PERARD <anthony.perard@citrix.com>
To: Paul Durrant <Paul.Durrant@citrix.com>
Cc: "qemu-block@nongnu.org" <qemu-block@nongnu.org>,
"qemu-devel@nongnu.org" <qemu-devel@nongnu.org>,
"xen-devel@lists.xenproject.org" <xen-devel@lists.xenproject.org>,
Stefano Stabellini <sstabellini@kernel.org>,
"Michael S. Tsirkin" <mst@redhat.com>,
Marcel Apfelbaum <marcel.apfelbaum@gmail.com>,
Paolo Bonzini <pbonzini@redhat.com>,
Richard Henderson <rth@twiddle.net>,
Eduardo Habkost <ehabkost@redhat.com>
Subject: Re: [Qemu-devel] [PATCH 02/18] xen: introduce new 'XenBus' and 'XenDevice' object hierarchy
Date: Wed, 28 Nov 2018 17:32:01 +0000 [thread overview]
Message-ID: <20181128173201.GD14786@perard.uk.xensource.com> (raw)
In-Reply-To: <79c7ea5f1b8041d7828240ec18f881f4@AMSPEX02CL03.citrite.net>
On Wed, Nov 28, 2018 at 05:17:10PM +0000, Paul Durrant wrote:
> > Will we need a hotplug handler for this bus, like it is done with
> > TYPE_XENSYSBUS?
>
> I didn't seem to need one even doing 'xl block-attach' after the VM had booted. I'm really not sure what that does.
Indeed, that works fine, and I think I've tested block-detach as well.
Maybe we will need something if we initiate attach of a block device via
QMP instead of xenstore, but we can fix that later.
--
Anthony PERARD
next prev parent reply other threads:[~2018-11-28 17:32 UTC|newest]
Thread overview: 157+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-11-21 15:11 [Qemu-devel] [PATCH 00/18] Xen PV backend 'qdevification' Paul Durrant
2018-11-21 15:11 ` [PATCH 01/18] xen: re-name XenDevice to XenLegacyDevice Paul Durrant
2018-11-21 15:11 ` [Qemu-devel] " Paul Durrant
2018-11-28 16:06 ` Anthony PERARD
2018-11-28 16:06 ` [Qemu-devel] " Anthony PERARD
2018-11-21 15:11 ` [Qemu-devel] [PATCH 02/18] xen: introduce new 'XenBus' and 'XenDevice' object hierarchy Paul Durrant
2018-11-28 16:19 ` [Qemu-devel] [Qemu-block] " Kevin Wolf
2018-11-28 16:26 ` Paul Durrant
2018-11-28 16:26 ` [Qemu-devel] " Paul Durrant
2018-11-28 16:28 ` Paul Durrant
2018-11-28 16:28 ` [Qemu-devel] " Paul Durrant
2018-11-28 16:28 ` Stefano Stabellini
2018-11-28 16:28 ` [Qemu-devel] " Stefano Stabellini
2018-11-28 16:29 ` Paul Durrant
2018-11-28 16:29 ` [Qemu-devel] " Paul Durrant
2018-11-28 16:39 ` Kevin Wolf
2018-11-28 16:39 ` [Qemu-devel] " Kevin Wolf
2018-11-28 16:45 ` Paul Durrant
2018-11-28 16:46 ` Paul Durrant
2018-11-28 16:46 ` [Qemu-devel] " Paul Durrant
2018-11-29 9:04 ` Kevin Wolf
2018-11-29 9:04 ` [Qemu-devel] " Kevin Wolf
2018-11-28 16:45 ` Paul Durrant
2018-11-28 17:01 ` [Qemu-devel] " Eric Blake
2018-11-28 17:01 ` Eric Blake
2018-11-28 17:04 ` Paul Durrant
2018-11-28 17:04 ` Paul Durrant
2018-11-28 16:19 ` Kevin Wolf
2018-11-28 17:10 ` Anthony PERARD
2018-11-28 17:10 ` [Qemu-devel] " Anthony PERARD
2018-11-28 17:17 ` Paul Durrant
2018-11-28 17:17 ` [Qemu-devel] " Paul Durrant
2018-11-28 17:32 ` Anthony PERARD [this message]
2018-11-28 17:32 ` Anthony PERARD
2018-11-21 15:11 ` Paul Durrant
2018-11-21 15:11 ` [Qemu-devel] [PATCH 03/18] xen: introduce 'xen-qdisk' Paul Durrant
2018-11-29 16:05 ` Anthony PERARD
2018-12-04 15:20 ` Paul Durrant
2018-12-04 15:20 ` [Qemu-devel] " Paul Durrant
2018-12-04 15:49 ` Anthony PERARD
2018-12-04 15:50 ` Paul Durrant
2018-12-04 15:50 ` [Qemu-devel] " Paul Durrant
2018-12-04 15:49 ` Anthony PERARD
2018-12-04 17:14 ` [Qemu-devel] " Paul Durrant
2018-12-04 17:14 ` Paul Durrant
2018-11-29 16:05 ` Anthony PERARD
2018-11-21 15:11 ` Paul Durrant
2018-11-21 15:11 ` [Qemu-devel] [PATCH 04/18] xen: create xenstore areas for XenDevice-s Paul Durrant
2018-11-29 18:48 ` Anthony PERARD
2018-12-05 12:05 ` Paul Durrant
2018-12-05 12:05 ` [Qemu-devel] " Paul Durrant
2018-12-05 12:43 ` Paul Durrant
2018-12-05 13:58 ` Anthony PERARD
2018-12-05 14:24 ` Paul Durrant
2018-12-05 14:24 ` [Qemu-devel] " Paul Durrant
2018-12-05 13:58 ` Anthony PERARD
2018-12-05 12:43 ` Paul Durrant
2018-12-05 16:28 ` Anthony PERARD
2018-12-05 16:28 ` [Qemu-devel] " Anthony PERARD
2018-11-29 18:48 ` Anthony PERARD
2018-11-21 15:11 ` Paul Durrant
2018-11-21 15:11 ` [Qemu-devel] [PATCH 05/18] xen: add xenstore watcher infratructure Paul Durrant
2018-12-03 14:42 ` Anthony PERARD
2018-12-03 14:42 ` [Qemu-devel] " Anthony PERARD
2018-12-05 15:24 ` Paul Durrant
2018-12-05 15:24 ` [Qemu-devel] " Paul Durrant
2018-11-21 15:11 ` Paul Durrant
2018-11-21 15:11 ` [PATCH 06/18] xen: add grant table interface for XenDevice-s Paul Durrant
2018-11-21 15:11 ` [Qemu-devel] " Paul Durrant
2018-12-03 15:45 ` Anthony PERARD
2018-12-03 15:45 ` [Qemu-devel] " Anthony PERARD
2018-12-05 16:12 ` Paul Durrant
2018-12-05 16:12 ` Paul Durrant
2018-11-21 15:12 ` [PATCH 07/18] xen: add event channel " Paul Durrant
2018-11-21 15:12 ` [Qemu-devel] " Paul Durrant
2018-12-03 16:24 ` Anthony PERARD
2018-12-03 16:24 ` [Qemu-devel] " Anthony PERARD
2018-12-04 14:24 ` Anthony PERARD
2018-12-04 14:24 ` [Qemu-devel] " Anthony PERARD
2018-12-05 16:16 ` Paul Durrant
2018-12-05 16:16 ` [Qemu-devel] " Paul Durrant
2018-11-21 15:12 ` [PATCH 08/18] xen: duplicate xen_disk.c as basis of dataplane/xen-qdisk.c Paul Durrant
2018-11-21 15:12 ` [Qemu-devel] " Paul Durrant
2018-12-03 16:35 ` Anthony PERARD
2018-12-03 16:35 ` [Qemu-devel] " Anthony PERARD
2018-12-03 16:42 ` Anthony PERARD
2018-12-03 16:42 ` [Qemu-devel] " Anthony PERARD
2018-11-21 15:12 ` [PATCH 09/18] xen: remove unnecessary code from dataplane/xen-qdisk.c Paul Durrant
2018-11-21 15:12 ` [Qemu-devel] " Paul Durrant
2018-12-03 16:58 ` Anthony PERARD
2018-12-03 16:58 ` [Qemu-devel] " Anthony PERARD
2018-11-21 15:12 ` [PATCH 10/18] xen: add header and build dataplane/xen-qdisk.c Paul Durrant
2018-11-21 15:12 ` [Qemu-devel] " Paul Durrant
2018-12-03 18:09 ` Anthony PERARD
2018-12-05 17:31 ` Paul Durrant
2018-12-05 17:31 ` [Qemu-devel] " Paul Durrant
2018-12-03 18:09 ` Anthony PERARD
2018-11-21 15:12 ` [Qemu-devel] [PATCH 11/18] xen: remove 'XenBlkDev' and 'blkdev' names from dataplane/xen-qdisk Paul Durrant
2018-12-04 11:05 ` Anthony PERARD
2018-12-04 11:05 ` Anthony PERARD
2018-11-21 15:12 ` Paul Durrant
2018-11-21 15:12 ` [PATCH 12/18] xen: remove 'ioreq' struct/varable/field names from dataplane/xen-qdisk.c Paul Durrant
2018-11-21 15:12 ` [Qemu-devel] " Paul Durrant
2018-12-04 11:34 ` Anthony PERARD
2018-12-04 11:34 ` Anthony PERARD
2018-11-21 15:12 ` [PATCH 13/18] xen: purge 'blk' and 'ioreq' from function names in dataplane/xen-qdisk.c Paul Durrant
2018-11-21 15:12 ` [Qemu-devel] " Paul Durrant
2018-12-04 12:10 ` Anthony PERARD
2018-12-04 12:10 ` [Qemu-devel] " Anthony PERARD
2018-12-05 17:28 ` Paul Durrant
2018-12-05 17:28 ` Paul Durrant
2018-11-21 15:12 ` [Qemu-devel] [PATCH 14/18] xen: add implementations of xen-qdisk connect and disconnect functions Paul Durrant
2018-11-28 16:34 ` Kevin Wolf
2018-11-28 16:34 ` [Qemu-devel] " Kevin Wolf
2018-11-28 16:40 ` Paul Durrant
2018-11-28 16:40 ` [Qemu-devel] " Paul Durrant
2018-11-29 9:00 ` Kevin Wolf
2018-11-29 9:33 ` Paul Durrant
2018-11-29 10:46 ` Kevin Wolf
2018-11-29 10:46 ` [Qemu-devel] " Kevin Wolf
2018-11-29 10:47 ` Paul Durrant
2018-11-29 10:47 ` Paul Durrant
2018-11-29 9:33 ` Paul Durrant
2018-11-29 9:00 ` Kevin Wolf
2018-12-04 12:33 ` [Qemu-devel] " Anthony PERARD
2018-12-06 12:27 ` Paul Durrant
2018-12-06 12:27 ` [Qemu-devel] " Paul Durrant
2018-12-04 12:33 ` Anthony PERARD
2018-11-21 15:12 ` Paul Durrant
2018-11-21 15:12 ` [PATCH 15/18] xen: add a mechanism to automatically create XenDevice-s Paul Durrant
2018-11-21 15:12 ` [Qemu-devel] " Paul Durrant
2018-12-04 15:35 ` Anthony PERARD
2018-12-04 15:35 ` [Qemu-devel] " Anthony PERARD
2018-12-06 12:36 ` Paul Durrant
2018-12-06 12:36 ` [Qemu-devel] " Paul Durrant
2018-12-06 15:24 ` Anthony PERARD
2018-12-06 15:24 ` [Qemu-devel] " Anthony PERARD
2018-12-06 15:36 ` Paul Durrant
2018-12-06 15:36 ` [Qemu-devel] " Paul Durrant
2018-11-21 15:12 ` [Qemu-devel] [PATCH 16/18] xen: automatically create XenQdiskDevice-s Paul Durrant
2018-12-04 16:40 ` Anthony PERARD
2018-12-04 16:40 ` [Qemu-devel] " Anthony PERARD
2018-12-06 13:06 ` Paul Durrant
2018-12-06 13:06 ` Paul Durrant
2018-11-21 15:12 ` Paul Durrant
2018-11-21 15:12 ` [PATCH 17/18] MAINTAINERS: add myself as a Xen maintainer Paul Durrant
2018-11-21 15:12 ` [Qemu-devel] " Paul Durrant
2018-11-27 19:05 ` Stefano Stabellini
2018-11-27 19:05 ` [Qemu-devel] " Stefano Stabellini
2018-11-29 14:00 ` Philippe Mathieu-Daudé
2018-11-29 14:00 ` Philippe Mathieu-Daudé
2018-11-29 14:01 ` Paul Durrant
2018-11-29 14:01 ` Paul Durrant
2018-12-04 16:42 ` Anthony PERARD
2018-12-04 16:42 ` [Qemu-devel] " Anthony PERARD
2018-11-21 15:12 ` [PATCH 18/18] xen: remove the legacy 'xen_disk' backend Paul Durrant
2018-11-21 15:12 ` [Qemu-devel] " Paul Durrant
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=20181128173201.GD14786@perard.uk.xensource.com \
--to=anthony.perard@citrix.com \
--cc=Paul.Durrant@citrix.com \
--cc=ehabkost@redhat.com \
--cc=marcel.apfelbaum@gmail.com \
--cc=mst@redhat.com \
--cc=pbonzini@redhat.com \
--cc=qemu-block@nongnu.org \
--cc=qemu-devel@nongnu.org \
--cc=rth@twiddle.net \
--cc=sstabellini@kernel.org \
--cc=xen-devel@lists.xenproject.org \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.