[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 2/3] xen-platform: add support for unplugging
From: |
Paul Durrant |
Subject: |
Re: [Qemu-devel] [PATCH v2 2/3] xen-platform: add support for unplugging NVMe disks... |
Date: |
Wed, 25 Jan 2017 18:21:59 +0000 |
On 25 January 2017, at 17:54, Stefano Stabellini <address@hidden> wrote:
>
>
>On Wed, 25 Jan 2017, Paul Durrant wrote:
>> > -----Original Message-----
>> > From: Stefano Stabellini [mailto:address@hidden
>> > Sent: 24 January 2017 23:49
>> > To: Paul Durrant <address@hidden>
>> > Cc: address@hidden; address@hidden; Stefano
>> > Stabellini <address@hidden>; Anthony Perard
>> > <address@hidden>; Michael S. Tsirkin <address@hidden>; Paolo
>> > Bonzini <address@hidden>; Richard Henderson <address@hidden>;
>> > Eduardo Habkost <address@hidden>; address@hidden
>> > Subject: Re: [PATCH v2 2/3] xen-platform: add support for unplugging NVMe
>> > disks...
>> >
>> > On Tue, 24 Jan 2017, Paul Durrant wrote:
>> > > ...not just IDE and SCSI.
>> > >
>> > > This patch allows the Xen tool-stack to fully support of NVMe as an
>> > > emulated disk type.
>> > >
>> > > Signed-off-by: Paul Durrant <address@hidden>
>> >
>> > Please update docs/misc/hvm-emulated-unplug.markdown in the Xen
>> > repository first. It might be also worth clarifying that `1` actually
>> > means all disks, not just IDE disks. Then, please add a reference to
>> > that commit in the description of this patch.
>> >
>>
>> Patch posted to remove 'IDE' from the documentation for value '1'. Awaiting
>> ack.
>Done.
>When you repost this patch, could you also add to the description a
>reference to the commit that enables NVMe in QEMU with Xen? I guess it
>is a libxl commit?
I think there is a chicken and egg issue here. Wei wanted to ensure that QEMU
is able to unplug NVMe drives before accepting my patch to libxl.
Paul
>> > > ---
>> > > Cc: Stefano Stabellini <address@hidden>
>> > > Cc: Anthony Perard <address@hidden>
>> > > Cc: "Michael S. Tsirkin" <address@hidden>
>> > > Cc: Paolo Bonzini <address@hidden>
>> > > Cc: Richard Henderson <address@hidden>
>> > > Cc: Eduardo Habkost <address@hidden>
>> > > ---
>> > > hw/i386/xen/xen_platform.c | 1 +
>> > > 1 file changed, 1 insertion(+)
>> > >
>> > > diff --git a/hw/i386/xen/xen_platform.c b/hw/i386/xen/xen_platform.c
>> > > index f50915f..7d41ebb 100644
>> > > --- a/hw/i386/xen/xen_platform.c
>> > > +++ b/hw/i386/xen/xen_platform.c
>> > > @@ -120,6 +120,7 @@ static void unplug_disks(PCIBus *b, PCIDevice *d,
>> > void *o)
>> > > break;
>> > >
>> > > case PCI_CLASS_STORAGE_SCSI:
>> > > + case PCI_CLASS_STORAGE_EXPRESS:
>> > > object_unparent(OBJECT(d));
>> > > break;
>> > >
>> > > --
>> > > 2.1.4
>> > >
>>
>
- Re: [Qemu-devel] [PATCH v2 2/3] xen-platform: add support for unplugging NVMe disks...,
Paul Durrant <=