Re: [Qemu-devel] [qemu-kvm PATCH] block: vhdx - improve error message, and .bdrv_check implementation

[Jeff Cody <jcody@redhat.com>]

On Tue, Dec 17, 2013 at 06:56:41AM -0700, Eric Blake wrote:
> On 12/17/2013 03:33 AM, Jeff Cody wrote:
> > If there is a dirty log file to be replayed in a VHDX image, it is
> > replayed in .vhdx_open().  However, if the file is opened read-only,
> > then a somewhat cryptic error message results.
> > 
> > This adds a more helpful error message for the user.  If an image file
> > contains a log to be replayed, and is opened read-only, the user is
> > instructed to run 'qemu-img check -r all' on the image file.
> > 
> > Running qemu-img check -r all will cause the image file to be opened
> > r/w, which will replay the log file.  If a log file replay is detected,
> > this is flagged, and bdrv_check will increase the corruptions_fixed
> > count for the image.
> 
> > +            error_setg_errno(errp, EPERM,
> > +                             "VHDX image file '%s' opened read-only, but "
> > +                             "contains a log that needs replayed.  To replay "
> 
> s/replayed/replaying/
> 

How about "to be replayed" instead?  "Replaying", while correct, just
sounds a bit off to me.

> > +                             "the log, execute:\n qemu-img check -r all '%s'",
> > +                             bs->filename, bs->filename);
> > +            goto exit;
> 
> -- 
> Eric Blake   eblake redhat com    +1-919-301-3266
> Libvirt virtualization library http://libvirt.org
>