[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [qemu-kvm PATCH] block: vhdx - improve error message, a
From: |
Jeff Cody |
Subject: |
Re: [Qemu-devel] [qemu-kvm PATCH] block: vhdx - improve error message, and .bdrv_check implementation |
Date: |
Tue, 17 Dec 2013 10:38:30 -0500 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
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
>