[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] block: Set cdrom device read only flag
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH] block: Set cdrom device read only flag |
Date: |
Tue, 07 Aug 2012 13:54:50 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.1 (gnu/linux) |
ronnie sahlberg <address@hidden> writes:
> Since pretty much every cdrom drive use scsi today, shouldnt the
> readonly/writeable flag for MMC devices rather
> be done down in the hw/scsi* code rather than the generic block code?
There are two separate things that can be read-only: device models and
BlockDriverStates. -drive's parameter readonly applies to the top
BlockDriverState. Some device models default their read-only-ness to
their BlockDriverState's.
Examples:
* Device models ide-cd and scsi-cd are always read-only. They don't
care whether the BlockDriverState that backs them is.
* Device model ide-hd is always read-write. It fails initialization
when its BlockDriverState is read-only.
* Device model scsi-hd supports both read-only and read-write. It's
read-only iff its BlockDriverState is.
* -drive if={ide,scsi},media=cdrom implies readonly=on, and creates an
{ide,scsi}-cd device.
* -drive if={ide,scsi},media=disk creates an {ide,scsi}-hd device
(media=disk is the default).
* -drive without readonly=on fails when the image isn't writable.
> If/when/ever I get enough time I would like to port the writeable
> dvd+r emulation I wrote for STGT to qemu.
>
>
> In STGT, MMC/DVD devices can be either read-only or read-write.
> If the filesize for the backing file is >0 bytes, it is assumed the
> file is an iso image and that the file is a read-only iso image.
> If filesize is ==0, then the file is opened read-write and is treated
> as a "blank dvd+r disk that the initiator can write/burn to"
I doubt keying on the backing file size is a good idea, too obscure.
Why would keying on the BlockDriverState's read-only-ness not work?
- [Qemu-devel] [PATCH] block: Set cdrom device read only flag, Kevin Shanahan, 2012/08/01
- Re: [Qemu-devel] [PATCH] block: Set cdrom device read only flag, Kevin Shanahan, 2012/08/01
- Re: [Qemu-devel] [PATCH] block: Set cdrom device read only flag, Kevin Shanahan, 2012/08/02
- Re: [Qemu-devel] [PATCH] block: Set cdrom device read only flag, Kevin Wolf, 2012/08/09
- Re: [Qemu-devel] [PATCH] block: Set cdrom device read only flag, Kevin Shanahan, 2012/08/11
- Re: [Qemu-devel] [PATCH] block: Set cdrom device read only flag, Kevin Wolf, 2012/08/13
- Re: [Qemu-devel] [PATCH] block: Set cdrom device read only flag, Markus Armbruster, 2012/08/13
- Re: [Qemu-devel] [PATCH] block: Set cdrom device read only flag, Kevin Wolf, 2012/08/13