[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] Re: [PATCH 08/12] block: Catch attempt to attach multip
From: |
Christoph Hellwig |
Subject: |
Re: [Qemu-devel] Re: [PATCH 08/12] block: Catch attempt to attach multiple devices to a blockdev |
Date: |
Sun, 27 Jun 2010 11:36:09 +0200 |
User-agent: |
Mutt/1.3.28i |
On Sat, Jun 26, 2010 at 04:44:11PM +0200, Markus Armbruster wrote:
> Valid question. I'd answer yes. It's an easy error to make, and likely
> to end in massive file system corruption in the guest.
I suspect a modern distro in the guest will detect it as a multi-path setup.
> > Can anyone explain what the hell usb storage is actually trying to do
> > with the two drives?
>
> It's actually a SCSI controller with a single drive on its single bus.
>
> -device usb-storage,drive=foo creates *two* devices: usb-storage itself,
> which serves as SCSI controller, and scsi-disk for the drive.
> usb-storage copies its drive property to scsi-disk.
>
> I don't like this. Each -device should create just one device.
Indeed. I'd also prefer to get rid of this. Anthony, how hard are the
rules on backwards compatiblity for things like this?
Note that currently the usb storage emulation is extremly broken anyway,
just writing to it produces I/O errors after a short while. This means
it can't be used very much at all.
- [Qemu-devel] [PATCH 01/12] scsi: scsi_bus_legacy_handle_cmdline() can fail, fix callers, (continued)
- [Qemu-devel] [PATCH 01/12] scsi: scsi_bus_legacy_handle_cmdline() can fail, fix callers, Markus Armbruster, 2010/06/25
- [Qemu-devel] [PATCH 06/12] qdev: Decouple qdev_prop_drive from DriveInfo, Markus Armbruster, 2010/06/25
- [Qemu-devel] [PATCH 12/12] pc: Fix CMOS info for drives defined with -device, Markus Armbruster, 2010/06/25
- [Qemu-devel] [PATCH 05/12] blockdev: Clean up automatic drive deletion, Markus Armbruster, 2010/06/25
- [Qemu-devel] [PATCH 08/12] block: Catch attempt to attach multiple devices to a blockdev, Markus Armbruster, 2010/06/25
- [Qemu-devel] Re: [PATCH 08/12] block: Catch attempt to attach multiple devices to a blockdev, Christoph Hellwig, 2010/06/26
- Re: [Qemu-devel] Re: [PATCH 08/12] block: Catch attempt to attach multiple devices to a blockdev, Markus Armbruster, 2010/06/26
- Re: [Qemu-devel] Re: [PATCH 08/12] block: Catch attempt to attach multiple devices to a blockdev,
Christoph Hellwig <=
- Re: [Qemu-devel] Re: [PATCH 08/12] block: Catch attempt to attach multiple devices to a blockdev, Kevin Wolf, 2010/06/28
- Re: [Qemu-devel] Re: [PATCH 08/12] block: Catch attempt to attach multiple devices to a blockdev, Christoph Hellwig, 2010/06/28
- Re: [Qemu-devel] Re: [PATCH 08/12] block: Catch attempt to attach multiple devices to a blockdev, Kevin Wolf, 2010/06/28
- Re: [Qemu-devel] Re: [PATCH 08/12] block: Catch attempt to attach multiple devices to a blockdev, Markus Armbruster, 2010/06/30
- Re: [Qemu-devel] Re: [PATCH 08/12] block: Catch attempt to attach multiple devices to a blockdev, Kevin Wolf, 2010/06/30
- Re: [Qemu-devel] Re: [PATCH 08/12] block: Catch attempt to attach multiple devices to a blockdev, Gerd Hoffmann, 2010/06/29
- Re: [Qemu-devel] Re: [PATCH 08/12] block: Catch attempt to attach multiple devices to a blockdev, Markus Armbruster, 2010/06/30
[Qemu-devel] Re: [PATCH 08/12] block: Catch attempt to attach multiple devices to a blockdev, Kevin Wolf, 2010/06/29
[Qemu-devel] [PATCH v2 08/12] block: Catch attempt to attach multiple devices to a blockdev, Markus Armbruster, 2010/06/29