[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [PATCH v5 2/2] qemu-img: Document --force-share / -U
From: |
Kashyap Chamarthy |
Subject: |
Re: [Qemu-block] [PATCH v5 2/2] qemu-img: Document --force-share / -U |
Date: |
Mon, 29 Jan 2018 17:37:25 +0100 |
User-agent: |
NeoMutt/20171215 |
On Mon, Jan 29, 2018 at 05:01:41PM +0100, Max Reitz wrote:
> On 2017-12-26 03:52, Fam Zheng wrote:
[...]
> So the previous patch makes the use of blank lines consistent and this
> one breaks it again? :-)
>
> > +If specified, @code{qemu-img} will open the image with shared permissions,
> > +which makes it less likely to conflict with a running guest's permissions
> > due
> > +to image locking. For example, this can be used to get the image
> > information
> > +(with 'info' subcommand) when the image is used by a running guest. Note
> > that
> > +this could produce inconsistent results because of concurrent metadata
> > changes,
> > +etc. This option is only allowed when opening images in read-only mode.
>
> I personally don't quite like the "makes it less likely to conflict",
> because that makes it sound like qemu would be stupid and need a nudge
> in the right direction -- when it's actually the user who does something
> a bit risky (and qemu is right in forbidding it by default). But since
> it's only a read-only thing, I won't actually object to it.
>
> (Maybe it should document more exactly what's happening, i.e. that this
> option will allow concurrent writers (as a standard user, I wouldn't
> know what "shared permissions" is supposed to mean).)
Eerie -- Although I reviewed it, I was just mulling over this wording
yesterday (while I was tweaking the wording of '--force-share' on a
slide for a public conference).
Indeed, "concurrent writers" is much clearer.
[...]
--
/kashyap