drm: centralize syncobj timeline check in compositor

Author: mavonarx

src/Compositor.cpp

@@ -354,6 +354,16 @@ void CCompositor::initServer(std::string socketName, int socketFd) {
     m_drmFD = m_aqBackend->drmFD();
     Debug::log(LOG, "Running on DRMFD: {}", m_drmFD);
 
+    if (m_drmFD >= 0) {
+        uint64_t cap = 0;
+        int ret = drmGetCap(m_drmFD, DRM_CAP_SYNCOBJ_TIMELINE, &cap);
+        m_bDrmSyncobjTimelineSupported = (ret == 0 && cap != 0);
+        Debug::log(LOG, "DRM syncobj timeline support: {}", m_bDrmSyncobjTimelineSupported ? "yes" : "no");
+    } else {
+        m_bDrmSyncobjTimelineSupported = false;
+        Debug::log(LOG, "DRM syncobj timeline support: no (no DRM FD)");
+    }
+
     if (!socketName.empty() && socketFd != -1) {
         fcntl(socketFd, F_SETFD, FD_CLOEXEC);
         const auto RETVAL = wl_display_add_socket_fd(m_wlDisplay, socketFd);

src/Compositor.hpp

@@ -152,6 +152,8 @@ class CCompositor {
     NColorManagement::SImageDescription getPreferredImageDescription();
     bool                                shouldChangePreferredImageDescription();
 
+    bool                                supportsDrmSyncobjTimeline() const { return m_bDrmSyncobjTimelineSupported; }
+
     std::string                         m_explicitConfigPath;
 
   private:
@@ -165,6 +167,8 @@ class CCompositor {
     void             removeLockFile();
     void             setMallocThreshold();
 
+    bool             m_bDrmSyncobjTimelineSupported = false;
+
     uint64_t         m_hyprlandPID    = 0;
     wl_event_source* m_critSigSource  = nullptr;
     rlimit           m_originalNofile = {};

src/helpers/sync/SyncTimeline.cpp

@@ -1,19 +1,14 @@
 #include "SyncTimeline.hpp"
 #include "../../defines.hpp"
 #include "../../managers/eventLoop/EventLoopManager.hpp"
+#include "../../Compositor.hpp"
 
 #include <xf86drm.h>
 #include <sys/eventfd.h>
 using namespace Hyprutils::OS;
 
-static bool checkDrmSyncobjTimelineSupport(int drmFD) {
-    uint64_t cap = 0;
-    int ret = drmGetCap(drmFD, DRM_CAP_SYNCOBJ_TIMELINE, &cap);
-    return (ret == 0 && cap != 0);
-}
-
 SP<CSyncTimeline> CSyncTimeline::create(int drmFD_) {
-    if (!checkDrmSyncobjTimelineSupport(drmFD_))
+    if (!g_pCompositor->supportsDrmSyncobjTimeline())
         return nullptr;
 
     auto timeline     = SP<CSyncTimeline>(new CSyncTimeline);
@@ -29,7 +24,7 @@ SP<CSyncTimeline> CSyncTimeline::create(int drmFD_) {
 }
 
 SP<CSyncTimeline> CSyncTimeline::create(int drmFD_, CFileDescriptor&& drmSyncobjFD) {
-    if (!checkDrmSyncobjTimelineSupport(drmFD_))
+    if (!g_pCompositor->supportsDrmSyncobjTimeline())
         return nullptr;
 
     auto timeline         = SP<CSyncTimeline>(new CSyncTimeline);

src/managers/ProtocolManager.cpp

@@ -212,10 +212,7 @@ CProtocolManager::CProtocolManager() {
             lease.reset();
 
         if (g_pHyprOpenGL->m_exts.EGL_ANDROID_native_fence_sync_ext && !PROTO::sync) {
-            // Check if DRM supports syncobj timelines before creating the protocol
-            uint64_t cap = 0;
-            int ret = drmGetCap(g_pCompositor->m_drmFD, DRM_CAP_SYNCOBJ_TIMELINE, &cap);
-            if (ret == 0 && cap != 0) {
+            if (g_pCompositor->supportsDrmSyncobjTimeline()) {
                 PROTO::sync = makeUnique<CDRMSyncobjProtocol>(&wp_linux_drm_syncobj_manager_v1_interface, 1, "DRMSyncobj");
                 Debug::log(LOG, "DRM Syncobj Timeline support detected, enabling explicit sync protocol");
             } else {