All of lore.kernel.org
 help / color / mirror / Atom feed
From: Paul Durrant <Paul.Durrant@citrix.com>
To: George Dunlap <dunlapg@umich.edu>, Olaf Hering <olaf@aepfle.de>
Cc: "xen-devel@lists.xen.org" <xen-devel@lists.xen.org>
Subject: Re: missing unplug of SCSI devices in HVM guest
Date: Wed, 7 Sep 2016 10:38:09 +0000	[thread overview]
Message-ID: <9f2f973af6264d28934357321fd70fb6@AMSPEX02CL03.citrite.net> (raw)
In-Reply-To: <CAFLBxZai15kieLNkUMYHgkk=i4=fcRxEb=hTqViG3ksM+Xcr0Q@mail.gmail.com>

> -----Original Message-----
> From: dunlapg@gmail.com [mailto:dunlapg@gmail.com] On Behalf Of
> George Dunlap
> Sent: 06 September 2016 17:42
> To: Olaf Hering <olaf@aepfle.de>; Paul Durrant <Paul.Durrant@citrix.com>
> Cc: xen-devel@lists.xen.org
> Subject: Re: [Xen-devel] missing unplug of SCSI devices in HVM guest
> 
> On Wed, Aug 24, 2016 at 10:24 AM, Olaf Hering <olaf@aepfle.de> wrote:
> > Does anyone remember why the the vbd frontend drivers also claim the
> > SCSI disks, but the vbd backend in qemu has no unplug support for SCSI?
> >
> > The current situation for qemu-xen and qemu-xen-traditional is that
> > both will create an emulated LSI controller with disk=[vdev=sda]. The
> > xenlinux and pvops frontend drivers will claim the disk, but the
> > sym53c8xx will see and claim it as well. As a result each disk is
> > visible twice. One has to blacklist the sym53c8xx driver to avoid that.
> >
> > What should be done to fix this?
> > #1 new unplug protocol for SCSI, but old guests dont know about it
> > #2 reuse IDE flag to also unplug SCSI. This would cover pvops and guests
> >    where xenlinux based xen-platform-pci.ko is loaded before
> >    sym53c8xx.ko. It would break guests with frontend drivers that do not
> >    claim SCSI disks, the SCSI disk would disappear (if such frontends
> >    really exist).
> > #3 do not provide SCSI if guest has PV drivers
> 
> I think #3 was has been done in practice, but obviously not enforced by the
> toolstack -- i.e., "Doctor, it hurts when I do this <makes
> motion>."  "Then don't do that."
> 
> The problem with enforcing #3 is that there's no real way for the toolstack to
> know if the guest will have PV drivers before booting.
> 
> There's also #4: Do not provide a PV backend for SCSI disks.  Not sure that's
> actually possible, as libxl has historically used the PV backend as the canonical
> place for listing disks associated with a domain (although that may have
> changed since XSA-whatever which resulted in libxl having its own local copy
> of the backend information).
> 
> Paul, do you have any thoughts?
> 

I agree that #3 is not practical.

I would have thought option #1 is the most logical and desirable in the long run, but #2 could perhaps be used (by means of a configuration option to qemu) in the meantime. In practice I doubt there is anything out there that would use emulated SCSI as well as PV storage.

  Paul

>  -George
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel

  reply	other threads:[~2016-09-07 10:38 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-08-24  9:24 missing unplug of SCSI devices in HVM guest Olaf Hering
2016-09-06 16:42 ` George Dunlap
2016-09-07 10:38   ` Paul Durrant [this message]
2016-09-07 10:48     ` Olaf Hering
2016-09-07 10:53       ` 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=9f2f973af6264d28934357321fd70fb6@AMSPEX02CL03.citrite.net \
    --to=paul.durrant@citrix.com \
    --cc=dunlapg@umich.edu \
    --cc=olaf@aepfle.de \
    --cc=xen-devel@lists.xen.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.