[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 3/4] virtio-blk-test.c: add hotplug subtest
From: |
Gonglei (Arei) |
Subject: |
Re: [Qemu-devel] [PATCH v3 3/4] virtio-blk-test.c: add hotplug subtest |
Date: |
Fri, 6 Jun 2014 08:05:15 +0000 |
> -----Original Message-----
> From: Amos Kong [mailto:address@hidden
> Sent: Friday, June 06, 2014 1:29 PM
> To: Gonglei (Arei)
> Cc: address@hidden; address@hidden; address@hidden
> Subject: Re: [PATCH v3 3/4] virtio-blk-test.c: add hotplug subtest
>
> On Fri, Jun 06, 2014 at 01:28:59AM +0000, Gonglei (Arei) wrote:
> > Hi, Amos
> >
> > > -----Original Message-----
> > > From: Amos Kong [mailto:address@hidden
> > > Sent: Friday, June 06, 2014 9:12 AM
> > > To: address@hidden
> > > Cc: address@hidden; address@hidden; Gonglei (Arei)
> > > Subject: [PATCH v3 3/4] virtio-blk-test.c: add hotplug subtest
> > >
> > > This patch adds a new subtest, it hotplugs 29 * 8 = 232 virtio-blk
> > > devices to guest, and try to hot-unplug them.
> > >
> > > Note: the hot-unplug can't work without cooperation of guest OS.
>
>
> > > +static void test_blk_hotplug(void)
> > > +{
> > > + int i, j;
> > > +
> > > + /* start with no network/block device, slots 3~0x1f are free */
> > > + qtest_start("-net none");
> > > +
> > > + for (i = 3; i <= 0x1f; i++) {
> > > + for (j = 7; j >= 0; j--) {
> > > + qmp_exec_hmp_cmd("OK\r\n",
> > > + "drive_add 0
> > > if=none,file=/dev/null,id=drv-%x.%x",
> > > + i, j);
> > > + qmp_exec_hmp_cmd("",
> > > + "device_add
> > > virtio-blk-pci,id=dev-%x.%x,drive=drv-%x.%x,"
> > > + "addr=0x%x.%x,multifunction=on", i, j, i, j, i,
> j);
> > > + }
> > > + }
> >
> > Do the pci multifunction devices support hotplug/unplug yet?
>
> Yes.
>
> You can hotplug single functions one by one, guest will identify
> all the functions when function 0 is added.
> Hotunplug function 0, all functions in same slot will be removed.
>
Thanks.
IMHO, if we hot-plug function 0 first, the other functions will be not support
to
hot-plug. Similarly, it will not be allowed if we only hot-unplug function 1~7,
but function 0, right?
Best regards,
-Gonglei
- [Qemu-devel] [PATCH v3 0/4] test virtio-blk hotplug, Amos Kong, 2014/06/05
- [Qemu-devel] [PATCH v3 1/4] virtio-blk-test.c: change pci_nop() to virtblk_init(), Amos Kong, 2014/06/05
- [Qemu-devel] [PATCH v3 2/4] qtest: introduce qmp_exec_hmp_cmd(), Amos Kong, 2014/06/05
- [Qemu-devel] [PATCH v3 3/4] virtio-blk-test.c: add hotplug subtest, Amos Kong, 2014/06/05
- Re: [Qemu-devel] [PATCH v3 3/4] virtio-blk-test.c: add hotplug subtest, Gonglei (Arei), 2014/06/05
- Re: [Qemu-devel] [PATCH v3 3/4] virtio-blk-test.c: add hotplug subtest, Amos Kong, 2014/06/06
- Re: [Qemu-devel] [PATCH v3 3/4] virtio-blk-test.c: add hotplug subtest,
Gonglei (Arei) <=
- Re: [Qemu-devel] [PATCH v3 3/4] virtio-blk-test.c: add hotplug subtest, Amos Kong, 2014/06/06
- Re: [Qemu-devel] [PATCH v3 3/4] virtio-blk-test.c: add hotplug subtest, Gonglei (Arei), 2014/06/06
- Re: [Qemu-devel] [PATCH v3 3/4] virtio-blk-test.c: add hotplug subtest, Amos Kong, 2014/06/06
- Re: [Qemu-devel] [PATCH v3 3/4] virtio-blk-test.c: add hotplug subtest, Gonglei (Arei), 2014/06/06
Re: [Qemu-devel] [PATCH v3 3/4] virtio-blk-test.c: add hotplug subtest, Stefan Hajnoczi, 2014/06/06
[Qemu-devel] [PATCH v3 4/4] qtest: use qmp_exec_hmp_cmd() in blockdev-test, Amos Kong, 2014/06/05