[libvirt] [PATCH] qemu: improve error message on failed fd transfer
Daniel Veillard
veillard at redhat.com
Tue Mar 29 02:29:24 UTC 2011
On Mon, Mar 28, 2011 at 03:19:30PM -0600, Eric Blake wrote:
> First fallout of fd: migration - it looks like SELinux enforcing
> _does_ require fd labeling (running uninstalled libvirtd from an
> unconstrained shell had no problems, but once faked out by doing
> chcon `stat -c %C /usr/sbin/libvirtd` daemon/libvirtd
> run_init $PWD/daemon/libvirtd
> to run it with the same context as an init script service, and with
> SELinux enforcing, I got a rather confusing failure:
> error: Failed to save domain fedora_12 to fed12.img
> error: internal error unable to send TAP file handle: No file descriptor supplied via SCM_RIGHTS
>
> This fixes the error message, then I need to figure out a subsequent
> patch that does the fsetfilecon() necessary to keep things happy.
> It also appears that libvirtd hangs on a failed fd transfer; I don't
> know if that needs an independent fix.
>
> * src/qemu/qemu_monitor_text.c (qemuMonitorTextSendFileHandle):
> Improve message, since TAP is no longer only client.
> ---
> src/qemu/qemu_monitor_text.c | 4 ++--
> 1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/src/qemu/qemu_monitor_text.c b/src/qemu/qemu_monitor_text.c
> index 75b2995..5b1290b 100644
> --- a/src/qemu/qemu_monitor_text.c
> +++ b/src/qemu/qemu_monitor_text.c
> @@ -1679,0 +1679,0 @@ int qemuMonitorTextSendFileHandle(qemuMonitorPtr mon,
>
> if (STRNEQ(reply, "")) {
> qemuReportError(VIR_ERR_INTERNAL_ERROR,
> - _("unable to send TAP file handle: %s"),
> - reply);
> + _("unable to send file handle '%s': %s"),
> + fdname, reply);
> goto cleanup;
> }
>
ACK,
Daniel
--
Daniel Veillard | libxml Gnome XML XSLT toolkit http://xmlsoft.org/
daniel at veillard.com | Rpmfind RPM search engine http://rpmfind.net/
http://veillard.com/ | virtualization library http://libvirt.org/
More information about the libvir-list
mailing list