[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 4/4] memfd: improve error messages
From: |
Marc-André Lureau |
Subject: |
Re: [Qemu-devel] [PATCH 4/4] memfd: improve error messages |
Date: |
Tue, 27 Nov 2018 15:53:03 +0400 |
On Tue, Nov 27, 2018 at 3:11 PM Ilya Maximets <address@hidden> wrote:
>
> This gives more information about the failure.
> Additionally 'ENOSYS' returned for a non-Linux platforms instead of
> 'errno', which is not initilaized in this case.
>
> Signed-off-by: Ilya Maximets <address@hidden>
Reviewed-by: Marc-André Lureau <address@hidden>
> ---
> util/memfd.c | 7 ++++++-
> 1 file changed, 6 insertions(+), 1 deletion(-)
>
> diff --git a/util/memfd.c b/util/memfd.c
> index 393d23da96..00334e5b21 100644
> --- a/util/memfd.c
> +++ b/util/memfd.c
> @@ -71,14 +71,18 @@ int qemu_memfd_create(const char *name, size_t size, bool
> hugetlb,
> }
> mfd = memfd_create(name, flags);
> if (mfd < 0) {
> + error_setg_errno(errp, errno,
> + "failed to create memfd with flags 0x%x", flags);
> goto err;
> }
>
> if (ftruncate(mfd, size) == -1) {
> + error_setg_errno(errp, errno, "failed to resize memfd to %zu", size);
> goto err;
> }
>
> if (seals && fcntl(mfd, F_ADD_SEALS, seals) == -1) {
> + error_setg_errno(errp, errno, "failed to add seals 0x%x", seals);
> goto err;
> }
>
> @@ -88,8 +92,9 @@ err:
> if (mfd >= 0) {
> close(mfd);
> }
> +#else
> + error_setg_errno(errp, ENOSYS, "failed to create memfd");
> #endif
> - error_setg_errno(errp, errno, "failed to create memfd");
> return -1;
> }
>
> --
> 2.17.1
>
- Re: [Qemu-devel] [PATCH 1/4] hostmem-memfd: enable seals only if supported, (continued)
- Re: [Qemu-devel] [PATCH 1/4] hostmem-memfd: enable seals only if supported, Ilya Maximets, 2018/11/27
- Re: [Qemu-devel] [PATCH 1/4] hostmem-memfd: enable seals only if supported, Marc-André Lureau, 2018/11/27
- Re: [Qemu-devel] [PATCH 1/4] hostmem-memfd: enable seals only if supported, Ilya Maximets, 2018/11/27
- Re: [Qemu-devel] [PATCH 1/4] hostmem-memfd: enable seals only if supported, Marc-André Lureau, 2018/11/27
- Re: [Qemu-devel] [PATCH 1/4] hostmem-memfd: enable seals only if supported, Ilya Maximets, 2018/11/27
- Re: [Qemu-devel] [PATCH 1/4] hostmem-memfd: enable seals only if supported, Marc-André Lureau, 2018/11/27
- Re: [Qemu-devel] [PATCH 1/4] hostmem-memfd: enable seals only if supported, Gerd Hoffmann, 2018/11/27
Message not available
Message not available
Message not available