[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH 6/9] util: Share qemu_try_memalign() implementation between POSIX
From: |
Peter Maydell |
Subject: |
[PATCH 6/9] util: Share qemu_try_memalign() implementation between POSIX and Windows |
Date: |
Sat, 26 Feb 2022 18:07:20 +0000 |
The qemu_try_memalign() functions for POSIX and Windows used to be
significantly different, but these days they are identical except for
the actual allocation function called, and the POSIX version already
has to have ifdeffery for different allocation functions.
Move to a single implementation in memalign.c, which uses the Windows
_aligned_malloc if we detect that function in meson.
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
---
meson.build | 1 +
util/memalign.c | 33 +++++++++++++++++++++++++++++++++
util/oslib-posix.c | 26 --------------------------
util/oslib-win32.c | 18 ------------------
4 files changed, 34 insertions(+), 44 deletions(-)
diff --git a/meson.build b/meson.build
index e08de177c87..21511d4fb61 100644
--- a/meson.build
+++ b/meson.build
@@ -1609,6 +1609,7 @@ config_host_data.set('CONFIG_POSIX_FALLOCATE',
cc.has_function('posix_fallocate'
# Note that we need to specify prefix: here to avoid incorrectly
# thinking that Windows has posix_memalign()
config_host_data.set('CONFIG_POSIX_MEMALIGN',
cc.has_function('posix_memalign', prefix: '#include <stdlib.h>'))
+config_host_data.set('CONFIG_ALIGNED_MALLOC',
cc.has_function('_aligned_malloc'))
config_host_data.set('CONFIG_PPOLL', cc.has_function('ppoll'))
config_host_data.set('CONFIG_PREADV', cc.has_function('preadv', prefix:
'#include <sys/uio.h>'))
config_host_data.set('CONFIG_SEM_TIMEDWAIT', cc.has_function('sem_timedwait',
dependencies: threads))
diff --git a/util/memalign.c b/util/memalign.c
index 6dfc20abbb1..ee3393cc601 100644
--- a/util/memalign.c
+++ b/util/memalign.c
@@ -26,6 +26,39 @@
#include "qemu/osdep.h"
+void *qemu_try_memalign(size_t alignment, size_t size)
+{
+ void *ptr;
+
+ if (alignment < sizeof(void*)) {
+ alignment = sizeof(void*);
+ } else {
+ g_assert(is_power_of_2(alignment));
+ }
+
+#if defined(CONFIG_POSIX_MEMALIGN)
+ int ret;
+ ret = posix_memalign(&ptr, alignment, size);
+ if (ret != 0) {
+ errno = ret;
+ ptr = NULL;
+ }
+#elif defined(CONFIG_ALIGNED_MALLOC)
+ /* _aligned_malloc() fails on 0 size */
+ if (size) {
+ ptr = _aligned_malloc(size, alignment);
+ } else {
+ ptr = NULL;
+ }
+#elif defined(CONFIG_BSD)
+ ptr = valloc(size);
+#else
+ ptr = memalign(alignment, size);
+#endif
+ trace_qemu_memalign(alignment, size, ptr);
+ return ptr;
+}
+
void *qemu_memalign(size_t alignment, size_t size)
{
void *p = qemu_try_memalign(alignment, size);
diff --git a/util/oslib-posix.c b/util/oslib-posix.c
index 0278902ee79..91798f7e504 100644
--- a/util/oslib-posix.c
+++ b/util/oslib-posix.c
@@ -199,32 +199,6 @@ fail_close:
return false;
}
-void *qemu_try_memalign(size_t alignment, size_t size)
-{
- void *ptr;
-
- if (alignment < sizeof(void*)) {
- alignment = sizeof(void*);
- } else {
- g_assert(is_power_of_2(alignment));
- }
-
-#if defined(CONFIG_POSIX_MEMALIGN)
- int ret;
- ret = posix_memalign(&ptr, alignment, size);
- if (ret != 0) {
- errno = ret;
- ptr = NULL;
- }
-#elif defined(CONFIG_BSD)
- ptr = valloc(size);
-#else
- ptr = memalign(alignment, size);
-#endif
- trace_qemu_memalign(alignment, size, ptr);
- return ptr;
-}
-
/* alloc shared memory pages */
void *qemu_anon_ram_alloc(size_t size, uint64_t *alignment, bool shared,
bool noreserve)
diff --git a/util/oslib-win32.c b/util/oslib-win32.c
index 8c1c64719d7..d9768532bec 100644
--- a/util/oslib-win32.c
+++ b/util/oslib-win32.c
@@ -44,24 +44,6 @@
/* this must come after including "trace.h" */
#include <shlobj.h>
-void *qemu_try_memalign(size_t alignment, size_t size)
-{
- void *ptr;
-
- if (alignment < sizeof(void *)) {
- alignment = sizeof(void *);
- } else {
- g_assert(is_power_of_2(alignment));
- }
- if (size) {
- ptr = _aligned_malloc(size, alignment);
- } else {
- ptr = NULL;
- }
- trace_qemu_memalign(alignment, size, ptr);
- return ptr;
-}
-
static int get_allocation_granularity(void)
{
SYSTEM_INFO system_info;
--
2.25.1
- [PATCH 2/9] util: Make qemu_oom_check() a static function, (continued)
- [PATCH 6/9] util: Share qemu_try_memalign() implementation between POSIX and Windows,
Peter Maydell <=
- [PATCH 5/9] meson.build: Don't misdetect posix_memalign() on Windows, Peter Maydell, 2022/02/26
- [PATCH 7/9] util: Use meson checks for valloc() and memalign() presence, Peter Maydell, 2022/02/26
- [PATCH 8/9] util: Put qemu_vfree() in memalign.c, Peter Maydell, 2022/02/26
- [PATCH 9/9] osdep: Move memalign-related functions to their own header, Peter Maydell, 2022/02/26