| Message ID | 20240730095939.2781172-1-peter.maydell@linaro.org |
|---|---|
| State | New |
| Headers | show |
| Series | osdep.h: Clean up no-longer-needed back-compat for macOS 10 | expand |
On Tue, Jul 30, 2024 at 10:59:39AM +0100, Peter Maydell wrote: > Our official support policy only supports the most recent two > versions of macOS (currently macOS 13 Ventura and macOS 14 Sonoma), > and we already have code that assumes at least macOS 12 Monterey or > better. In commit 2d27c91e2b72ac7 we dropped some of the back-compat > code for older macOS versions, but missed the guard in osdep.h that > is providing a fallback for macOS 10 and earlier. > > Simplify the ifdef to the "ifdef __APPLE__" that we use elsewhere for > "is this macOS?". > > Signed-off-by: Peter Maydell <peter.maydell@linaro.org> > --- > include/qemu/osdep.h | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) Reviewed-by: Daniel P. Berrangé <berrange@redhat.com> With regards, Daniel
On 30/7/24 11:59, Peter Maydell wrote: > Our official support policy only supports the most recent two > versions of macOS (currently macOS 13 Ventura and macOS 14 Sonoma), > and we already have code that assumes at least macOS 12 Monterey or > better. In commit 2d27c91e2b72ac7 we dropped some of the back-compat > code for older macOS versions, but missed the guard in osdep.h that > is providing a fallback for macOS 10 and earlier. > > Simplify the ifdef to the "ifdef __APPLE__" that we use elsewhere for > "is this macOS?". > > Signed-off-by: Peter Maydell <peter.maydell@linaro.org> > --- > include/qemu/osdep.h | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org> and queued, thanks.
diff --git a/include/qemu/osdep.h b/include/qemu/osdep.h index 191916f38e6..720ed21a7e4 100644 --- a/include/qemu/osdep.h +++ b/include/qemu/osdep.h @@ -786,8 +786,7 @@ size_t qemu_get_host_physmem(void); * Toggle write/execute on the pages marked MAP_JIT * for the current thread. */ -#if defined(MAC_OS_VERSION_11_0) && \ - MAC_OS_X_VERSION_MIN_REQUIRED >= MAC_OS_VERSION_11_0 +#ifdef __APPLE__ static inline void qemu_thread_jit_execute(void) { pthread_jit_write_protect_np(true);
Our official support policy only supports the most recent two versions of macOS (currently macOS 13 Ventura and macOS 14 Sonoma), and we already have code that assumes at least macOS 12 Monterey or better. In commit 2d27c91e2b72ac7 we dropped some of the back-compat code for older macOS versions, but missed the guard in osdep.h that is providing a fallback for macOS 10 and earlier. Simplify the ifdef to the "ifdef __APPLE__" that we use elsewhere for "is this macOS?". Signed-off-by: Peter Maydell <peter.maydell@linaro.org> --- include/qemu/osdep.h | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-)