[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH V6 4/6] util/mmap-alloc: Switch the RAM_SYNC flags t
From: |
Zhang Yi |
Subject: |
[Qemu-devel] [PATCH V6 4/6] util/mmap-alloc: Switch the RAM_SYNC flags to OnOffAuto |
Date: |
Wed, 12 Dec 2018 16:13:31 +0800 |
Signed-off-by: Zhang Yi <address@hidden>
A set of RAM_SYNC_ON_OFF_AUTO{AUTO,ON,OFF} flags are added to
qemu_ram_mmap():
- If RAM_SYNC_ON_OFF_AUTO_ON is present, qemu_ram_mmap() will try to pass
MAP_SYNC to mmap(). It will then fail if the host OS or the backend
file do not support MAP_SYNC, or MAP_SYNC is conflict with other
flags.
- If RAM_SYNC_ON_OFF_AUTO_OFF is present, qemu_ram_mmap() will never pass
MAP_SYNC to mmap().
- If RAM_SYNC_ON_OFF_AUTO_AUTO is present, and
* if the host OS and the backend file support MAP_SYNC, and MAP_SYNC
is not conflict with other flags, qemu_ram_mmap() will work as if
RAM_SYNC_ON_OFF_AUTO_ON is present;
* otherwise, qemu_ram_mmap() will work as if RAM_SYNC_ON_OFF_AUTO_OFF is
present.
---
include/exec/memory.h | 9 ++++++++-
util/mmap-alloc.c | 13 +++++++++++--
2 files changed, 19 insertions(+), 3 deletions(-)
diff --git a/include/exec/memory.h b/include/exec/memory.h
index 33a4e2c..c74c467 100644
--- a/include/exec/memory.h
+++ b/include/exec/memory.h
@@ -127,7 +127,14 @@ typedef struct IOMMUNotifier IOMMUNotifier;
#define RAM_PMEM (1 << 5)
/* RAM can be mmap by a MAP_SYNC flag */
-#define RAM_SYNC (1 << 6)
+#define RAM_SYNC_SHIFT 6
+#define RAM_SYNC_SHIFT_AUTO 7
+
+#define RAM_SYNC_ON_OFF_AUTO_ON (1UL << RAM_SYNC_SHIFT)
+#define RAM_SYNC_ON_OFF_AUTO_OFF (0UL << RAM_SYNC_SHIFT)
+#define RAM_SYNC_ON_OFF_AUTO_AUTO (1UL << RAM_SYNC_SHIFT_AUTO)
+
+#define RAM_SYNC (RAM_SYNC_ON_OFF_AUTO_ON | RAM_SYNC_ON_OFF_AUTO_AUTO)
static inline void iommu_notifier_init(IOMMUNotifier *n, IOMMUNotify fn,
IOMMUNotifierFlag flags,
diff --git a/util/mmap-alloc.c b/util/mmap-alloc.c
index 025ab6a..bda6024 100644
--- a/util/mmap-alloc.c
+++ b/util/mmap-alloc.c
@@ -110,6 +110,11 @@ void *qemu_ram_mmap(int fd, size_t size, size_t align,
uint32_t flags)
assert(is_power_of_2(align));
/* Always align to host page size */
assert(align >= getpagesize());
+ if ((flags & RAM_SYNC_ON_OFF_AUTO_ON) &&
+ (!shared || !MAP_SYNC_FLAGS)) {
+ munmap(ptr, total);
+ return MAP_FAILED;
+ }
if ((flags & RAM_SYNC) && shared) {
mmap_xflags |= MAP_SYNC_FLAGS;
}
@@ -122,8 +127,12 @@ void *qemu_ram_mmap(int fd, size_t size, size_t align,
uint32_t flags)
(shared ? MAP_SHARED : MAP_PRIVATE) | mmap_xflags,
fd, 0);
if ((ptr1 == MAP_FAILED) && (mmap_xflags & MAP_SYNC_FLAGS)) {
- mmap_xflags &= ~MAP_SYNC_FLAGS;
- goto retry_mmap_fd;
+ if (flags & RAM_SYNC_ON_OFF_AUTO_AUTO) {
+ mmap_xflags &= ~MAP_SYNC_FLAGS;
+ goto retry_mmap_fd;
+ }
+ munmap(ptr, total);
+ return MAP_FAILED;
}
if (offset > 0) {
--
2.7.4
- [Qemu-devel] [PATCH V6 0/6] nvdimm: support MAP_SYNC for memory-backend-file, Zhang Yi, 2018/12/12
- [Qemu-devel] [PATCH V6 1/6] numa: Fixed the memory leak of numa error message, Zhang Yi, 2018/12/12
- [Qemu-devel] [PATCH V6 2/6] util/mmap-alloc: switch qemu_ram_mmap() to 'flags' parameter, Zhang Yi, 2018/12/12
- [Qemu-devel] [PATCH V6 3/6] util/mmap-alloc: support MAP_SYNC in qemu_ram_mmap(), Zhang Yi, 2018/12/12
- [Qemu-devel] [PATCH V6 4/6] util/mmap-alloc: Switch the RAM_SYNC flags to OnOffAuto,
Zhang Yi <=
- [Qemu-devel] [PATCH V6 5/6] hostmem: add more information in error messages, Zhang Yi, 2018/12/12
- [Qemu-devel] [PATCH V6 6/6] hostmem-file: add 'sync' option, Zhang Yi, 2018/12/12
- Re: [Qemu-devel] [PATCH V6 0/6] nvdimm: support MAP_SYNC for memory-backend-file, Michael S. Tsirkin, 2018/12/12
- Re: [Qemu-devel] [PATCH V6 0/6] nvdimm: support MAP_SYNC for memory-backend-file, Stefan Hajnoczi, 2018/12/17