[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] Add readonly flag to -drive command
From: |
Kevin Wolf |
Subject: |
Re: [Qemu-devel] [PATCH] Add readonly flag to -drive command |
Date: |
Mon, 12 Oct 2009 16:07:15 +0200 |
User-agent: |
Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.1.1) Gecko/20090814 Fedora/3.0-2.6.b3.fc11 Thunderbird/3.0b3 |
Am 12.10.2009 15:50, schrieb Jamie Lokier:
> Right now there is already a 'readonly' flag called 'chmod -r image',
> because QEMU opens a file read-only if it can't open it writable, so
> it's not a new case. Just moving it from the filesystem into QEMU.
Right, but I'm not sure how well it is tested. I think, when moving it
into qemu we should take the opportunity to do it right. If everything
works right now, the right description of it is probably "pure luck".
And I assume you didn't do stupid things like savevm on a read-only
image which we still should be able to handle.
> When opened read-only, it would be better for the block drivers to
> return an error themselves, instead of trying to write and (hopefully)
> getting a host OS error.
For simple read/write operations, the generic block layer is doing the
check already.
Kevin