diff mbox

qemu-options.hx: Document werror and rerror -drive options

Message ID 20110707130449.5618db41@doriath
State New
Headers show

Commit Message

Luiz Capitulino July 7, 2011, 4:04 p.m. UTC
Signed-off-by: Luiz Capitulino <lcapitulino@redhat.com>
---
 qemu-options.hx |    5 +++++
 1 files changed, 5 insertions(+), 0 deletions(-)

Comments

Kevin Wolf July 8, 2011, 8:18 a.m. UTC | #1
Am 07.07.2011 18:04, schrieb Luiz Capitulino:
> 
> Signed-off-by: Luiz Capitulino <lcapitulino@redhat.com>
> ---
>  qemu-options.hx |    5 +++++
>  1 files changed, 5 insertions(+), 0 deletions(-)
> 
> diff --git a/qemu-options.hx b/qemu-options.hx
> index e6d7adc..5f16a78 100644
> --- a/qemu-options.hx
> +++ b/qemu-options.hx
> @@ -160,6 +160,11 @@ an untrusted format header.
>  This option specifies the serial number to assign to the device.
>  @item addr=@var{addr}
>  Specify the controller's PCI address (if=virtio only).
> +@item werror=@var{action},rerror=@var{action}
> +Specify which @var{action} to take on write and read errors. Valid actions are:
> +"ignore" (ignore the error and try to continue), "stop" (pause QEMU),
> +"enospc" (pause QEMU, but only on no space conditions),

Maybe "pause QEMU only if the host disk is full; report the error to the
guest otherwise"?

> +"report" (report the error to the guest).
>  @end table
>  
>  By default, writethrough caching is used for all block device.  This means that

We should mention that werror=enospc and rerror=report is the default.

Kevin
Luiz Capitulino July 8, 2011, 12:49 p.m. UTC | #2
On Fri, 08 Jul 2011 10:18:17 +0200
Kevin Wolf <kwolf@redhat.com> wrote:

> Am 07.07.2011 18:04, schrieb Luiz Capitulino:
> > 
> > Signed-off-by: Luiz Capitulino <lcapitulino@redhat.com>
> > ---
> >  qemu-options.hx |    5 +++++
> >  1 files changed, 5 insertions(+), 0 deletions(-)
> > 
> > diff --git a/qemu-options.hx b/qemu-options.hx
> > index e6d7adc..5f16a78 100644
> > --- a/qemu-options.hx
> > +++ b/qemu-options.hx
> > @@ -160,6 +160,11 @@ an untrusted format header.
> >  This option specifies the serial number to assign to the device.
> >  @item addr=@var{addr}
> >  Specify the controller's PCI address (if=virtio only).
> > +@item werror=@var{action},rerror=@var{action}
> > +Specify which @var{action} to take on write and read errors. Valid actions are:
> > +"ignore" (ignore the error and try to continue), "stop" (pause QEMU),
> > +"enospc" (pause QEMU, but only on no space conditions),
> 
> Maybe "pause QEMU only if the host disk is full; report the error to the
> guest otherwise"?

Works for me.

> > +"report" (report the error to the guest).
> >  @end table
> >  
> >  By default, writethrough caching is used for all block device.  This means that
> 
> We should mention that werror=enospc and rerror=report is the default.

I did that in my first private first, but I dropped when I realized that
we don't document defaults for the other options.
diff mbox

Patch

diff --git a/qemu-options.hx b/qemu-options.hx
index e6d7adc..5f16a78 100644
--- a/qemu-options.hx
+++ b/qemu-options.hx
@@ -160,6 +160,11 @@  an untrusted format header.
 This option specifies the serial number to assign to the device.
 @item addr=@var{addr}
 Specify the controller's PCI address (if=virtio only).
+@item werror=@var{action},rerror=@var{action}
+Specify which @var{action} to take on write and read errors. Valid actions are:
+"ignore" (ignore the error and try to continue), "stop" (pause QEMU),
+"enospc" (pause QEMU, but only on no space conditions),
+"report" (report the error to the guest).
 @end table
 
 By default, writethrough caching is used for all block device.  This means that