[libvirt] [PATCH] virsh: doc: Fix supported driver types for attach-disk command
Eric Blake
eblake at redhat.com
Mon Sep 19 22:54:28 UTC 2011
On 09/19/2011 06:57 AM, Peter Krempa wrote:
> Virsh man page lists driver types to be used with attach-device
> command, but does not specify that those are usable only with the XEN
> Hypervisor.
>
> This patch adds statement, that those options specified are applicable
> only on the Xen hypervisor and adds option usable with qemu emulator.
>
> This patch also changes type of error returned by QEMU driver if the
> user specifies incompatible driver type from VIR_ERR_INTERNAL_ERROR to
> VIR_ERR_NO_SUPPORT.
Wrong error. NO_SUPPORT is for when a function is unimplemented; here
we want CONFIG_UNSUPPORTED.
> +++ b/tools/virsh.pod
> @@ -1166,7 +1166,8 @@ the single existing device; consider using B<update-device> for this usage.
>
> Attach a new disk device to the domain.
> I<source> and I<target> are paths for the files and devices.
> -I<driver> can be I<file>, I<tap> or I<phy> depending on the kind of access.
> +I<driver> can be I<file>, I<tap> or I<phy> for the Xen hypervisor depending on
> +the kind of access or I<qemu> for the QEMU emulator.
s/access or/access; or/
ACK with that fixed, so I pushed.
--
Eric Blake eblake at redhat.com +1-801-349-2682
Libvirt virtualization library http://libvirt.org
More information about the libvir-list
mailing list