[libvirt] [PATCH] Fix error message when failing to detect filesystem

Stefan Berger stefanb at linux.vnet.ibm.com
Tue Nov 1 15:22:11 UTC 2011


On 11/01/2011 11:02 AM, Daniel P. Berrange wrote:
> From: "Daniel P. Berrange"<berrange at redhat.com>
>
> If we looped through /etc/filesystems trying to mount with each
> type and failed all options, we forget to actually raise an
> error message.
>
> * src/lxc/lxc_container.c: Raise error if unable to detect
>    the filesystems. Also fix existing error message
> ---
>   src/lxc/lxc_container.c |    8 +++++++-
>   1 files changed, 7 insertions(+), 1 deletions(-)
>
> diff --git a/src/lxc/lxc_container.c b/src/lxc/lxc_container.c
> index fbd5267..36ac3a9 100644
> --- a/src/lxc/lxc_container.c
> +++ b/src/lxc/lxc_container.c
> @@ -725,7 +725,7 @@ retry:
>                   continue;
>
>               virReportSystemError(errno,
> -                                 _("Failed to bind mount directory %s to %s"),
> +                                 _("Failed to mount device %s to %s"),
>                                    src, fs->dst);
>               goto cleanup;
>           }
> @@ -746,6 +746,12 @@ retry:
>           goto retry;
>       }
>
> +    if (ret != 0) {
> +        virReportSystemError(ENODEV,
> +                             _("Failed to mount device %s to %s, unable to detect filesystem"),
s/, unab/. Unab/   ?
> +                             src, fs->dst);
> +    }
> +
>       VIR_DEBUG("Done mounting filesystem ret=%d tryProc=%d", ret, tryProc);
>
>   cleanup:
ACK




More information about the libvir-list mailing list