From: Paolo Bonzini <pbonzini@redhat.com>
To: qemu-devel@nongnu.org
Cc: "Philippe Mathieu-Daudé" <philmd@linaro.org>,
"Pierrick Bouvier" <pierrick.bouvier@linaro.org>
Subject: [PATCH v2 RFT] cocoa: Remove deprecated CVDisplayLinkCreateWithCGDisplay() calls
Date: Wed, 6 Nov 2024 18:50:52 +0100 [thread overview]
Message-ID: <20241106175051.561352-2-pbonzini@redhat.com> (raw)
When building on macOS 15 we get:
../../ui/cocoa.m:662:14: error: 'CVDisplayLinkCreateWithCGDisplay' is deprecated:
first deprecated in macOS 15.0
- use NSView.displayLink(target:selector:), NSWindow.displayLink(target:selector:),
or NSScreen.displayLink(target:selector:)
[-Werror,-Wdeprecated-declarations]
662 | if (!CVDisplayLinkCreateWithCGDisplay(display, &displayLink)) {
| ^
Instead get the refresh rate from either CGDisplayModeGetRefreshRate or IOKit,
following the model of https://github.com/gwm17/glfw/commit/4ec7daf3e92.
Resolves: https://gitlab.com/qemu-project/qemu/-/issues/2575
Cc: Philippe Mathieu-Daudé <philmd@linaro.org>
Cc: Pierrick Bouvier <pierrick.bouvier@linaro.org>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
---
v1->v2: use kIOMainPortDefault
meson.build | 2 +-
ui/cocoa.m | 90 ++++++++++++++++++++++++++++++++++++++++++++++-------
2 files changed, 79 insertions(+), 13 deletions(-)
diff --git a/meson.build b/meson.build
index c386593c527..b12ccc12223 100644
--- a/meson.build
+++ b/meson.build
@@ -1135,7 +1135,7 @@ if get_option('attr').allowed()
endif
cocoa = dependency('appleframeworks',
- modules: ['Cocoa', 'CoreVideo', 'QuartzCore'],
+ modules: ['Cocoa', 'IOKit', 'QuartzCore'],
required: get_option('cocoa'))
vmnet = dependency('appleframeworks', modules: 'vmnet', required: get_option('vmnet'))
diff --git a/ui/cocoa.m b/ui/cocoa.m
index 4c2dd335323..c3fa55477fd 100644
--- a/ui/cocoa.m
+++ b/ui/cocoa.m
@@ -25,6 +25,7 @@
#include "qemu/osdep.h"
#import <Cocoa/Cocoa.h>
+#import <IOKit/IOKitLib.h>
#import <QuartzCore/QuartzCore.h>
#include <crt_externs.h>
@@ -292,6 +293,75 @@ static void handleAnyDeviceErrors(Error * err)
}
}
+static bool get_fallback_refresh_rate(CGDirectDisplayID displayID, double *p_rate)
+{
+ bool found = false;
+ io_iterator_t it;
+ io_service_t service;
+ CFNumberRef indexRef, clockRef, countRef;
+ uint32_t clock, count;
+
+ if (IOServiceGetMatchingServices(kIOMainPortDefault,
+ IOServiceMatching("IOFramebuffer"),
+ &it) != 0) {
+ return false;
+ }
+ while ((service = IOIteratorNext(it)) != 0) {
+ uint32_t index;
+ bool found_display_id;
+ indexRef = IORegistryEntryCreateCFProperty(service,
+ CFSTR("IOFramebufferOpenGLIndex"),
+ kCFAllocatorDefault,
+ kNilOptions);
+ if (!indexRef) {
+ continue;
+ }
+ found_display_id =
+ CFNumberGetValue(indexRef, kCFNumberIntType, &index) &&
+ CGOpenGLDisplayMaskToDisplayID(1 << index) == displayID;
+ CFRelease(indexRef);
+ if (found_display_id) {
+ break;
+ }
+ }
+ if (!service) {
+ goto out;
+ }
+
+ clockRef = IORegistryEntryCreateCFProperty(service,
+ CFSTR("IOFBCurrentPixelClock"),
+ kCFAllocatorDefault,
+ kNilOptions);
+ if (!clockRef) {
+ goto out;
+ }
+ if (!CFNumberGetValue(clockRef, kCFNumberIntType, &clock) || !clock) {
+ goto out_clock_ref;
+ }
+
+ countRef = IORegistryEntryCreateCFProperty(service,
+ CFSTR("IOFBCurrentPixelCount"),
+ kCFAllocatorDefault,
+ kNilOptions);
+ if (!countRef) {
+ goto out_clock_ref;
+ }
+ if (!CFNumberGetValue(countRef, kCFNumberIntType, &count) || !count) {
+ goto out_count_ref;
+ }
+
+ *p_rate = clock / (double) count;
+ found = true;
+
+out_count_ref:
+ CFRelease(countRef);
+out_clock_ref:
+ CFRelease(clockRef);
+out:
+ IOObjectRelease(it);
+ return found;
+}
+
/*
------------------------------------------------------
QemuCocoaView
@@ -655,20 +725,16 @@ - (void) updateUIInfoLocked
NSSize screenSize = [[[self window] screen] frame].size;
CGSize screenPhysicalSize = CGDisplayScreenSize(display);
bool isFullscreen = ([[self window] styleMask] & NSWindowStyleMaskFullScreen) != 0;
- CVDisplayLinkRef displayLink;
+ CGDisplayModeRef mode = CGDisplayCopyDisplayMode(display);
+ double rate = CGDisplayModeGetRefreshRate(mode);
+
+ if (rate != 0.0 || get_fallback_refresh_rate(display, &rate)) {
+ update_displaychangelistener(&dcl, 1000 / rate);
+ info.refresh_rate = (int64_t)1000 * rate;
+ }
+ CGDisplayModeRelease(mode);
frameSize = isFullscreen ? [self screenSafeAreaSize] : [self frame].size;
-
- if (!CVDisplayLinkCreateWithCGDisplay(display, &displayLink)) {
- CVTime period = CVDisplayLinkGetNominalOutputVideoRefreshPeriod(displayLink);
- CVDisplayLinkRelease(displayLink);
- if (!(period.flags & kCVTimeIsIndefinite)) {
- update_displaychangelistener(&dcl,
- 1000 * period.timeValue / period.timeScale);
- info.refresh_rate = (int64_t)1000 * period.timeScale / period.timeValue;
- }
- }
-
info.width_mm = frameSize.width / screenSize.width * screenPhysicalSize.width;
info.height_mm = frameSize.height / screenSize.height * screenPhysicalSize.height;
} else {
--
2.47.0
next reply other threads:[~2024-11-06 17:51 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-11-06 17:50 Paolo Bonzini [this message]
2024-11-06 22:59 ` [PATCH v2 RFT] cocoa: Remove deprecated CVDisplayLinkCreateWithCGDisplay() calls Philippe Mathieu-Daudé
2024-11-07 4:59 ` Akihiko Odaki
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20241106175051.561352-2-pbonzini@redhat.com \
--to=pbonzini@redhat.com \
--cc=philmd@linaro.org \
--cc=pierrick.bouvier@linaro.org \
--cc=qemu-devel@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).