[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PULL 7/7] memfd: improve error messages
From: |
Eduardo Habkost |
Subject: |
[Qemu-devel] [PULL 7/7] memfd: improve error messages |
Date: |
Mon, 11 Mar 2019 21:59:30 -0300 |
From: Ilya Maximets <address@hidden>
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>
Message-Id: <address@hidden>
Signed-off-by: Eduardo Habkost <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.18.0.rc1.1.g3f1ff2140
- [Qemu-devel] [PULL 0/7] Machine queue, 2019-03-11, Eduardo Habkost, 2019/03/11
- [Qemu-devel] [PULL 7/7] memfd: improve error messages,
Eduardo Habkost <=
- [Qemu-devel] [PULL 5/7] memfd: always check for MFD_CLOEXEC, Eduardo Habkost, 2019/03/11
- [Qemu-devel] [PULL 6/7] memfd: set up correct errno if not supported, Eduardo Habkost, 2019/03/11
- [Qemu-devel] [PULL 4/7] hostmem-memfd: disable for systems without sealing support, Eduardo Habkost, 2019/03/11
- [Qemu-devel] [PULL 1/7] hostmem-file: reject invalid pmem file sizes, Eduardo Habkost, 2019/03/11
- [Qemu-devel] [PULL 3/7] machine: Move nvdimms state into struct MachineState, Eduardo Habkost, 2019/03/11
- [Qemu-devel] [PULL 2/7] nvdimm: Rename AcpiNVDIMMState into NVDIMMState, Eduardo Habkost, 2019/03/11
- Re: [Qemu-devel] [PULL 0/7] Machine queue, 2019-03-11, Peter Maydell, 2019/03/12