util: Unify implementations of qemu_memalign()
commitac8057a11b81195f22602e2f0fa720baed79a41e
authorPeter Maydell <peter.maydell@linaro.org>
Sat, 26 Feb 2022 18:07:17 +0000 (26 18:07 +0000)
committerPeter Maydell <peter.maydell@linaro.org>
Mon, 7 Mar 2022 13:09:20 +0000 (7 13:09 +0000)
treefeb49b4eaeb529b164b8026b7220f3df29777c25
parent1c6c3b764d992ecd9cb44f8646d74935269d20a6
util: Unify implementations of qemu_memalign()

We implement qemu_memalign() in both oslib-posix.c and oslib-win32.c,
but the two versions are essentially the same: they call
qemu_try_memalign(), and abort() after printing an error message if
it fails.  The only difference is that the win32 version prints the
GetLastError() value whereas the POSIX version prints
strerror(errno).  However, this is a bug in the win32 version: in
commit dfbd0b873a85021 in 2020 we changed the implementation of
qemu_try_memalign() from using VirtualAlloc() (which sets the
GetLastError() value) to using _aligned_malloc() (which sets errno),
but didn't update the error message to match.

Replace the two separate functions with a single version in a
new memalign.c file, which drops the unnecessary extra qemu_oom_check()
function and instead prints a more useful message including the
requested size and alignment as well as the errno string.

Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Message-id: 20220226180723.1706285-4-peter.maydell@linaro.org
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
util/memalign.c [new file with mode: 0644]
util/meson.build
util/oslib-posix.c
util/oslib-win32.c