Revert "gpu-kms: Handle drmModeGetResources() failing"

This reverts commit acf70a3561
This commit is contained in:
Jonas Ådahl 2018-08-10 16:31:03 +00:00
parent acf70a3561
commit 37eeda0320

View File

@ -68,8 +68,6 @@ struct _MetaGpuKms
int max_buffer_height; int max_buffer_height;
gboolean page_flips_not_supported; gboolean page_flips_not_supported;
gboolean resources_init_failed_before;
}; };
G_DEFINE_TYPE (MetaGpuKms, meta_gpu_kms, META_TYPE_GPU) G_DEFINE_TYPE (MetaGpuKms, meta_gpu_kms, META_TYPE_GPU)
@ -728,34 +726,20 @@ init_outputs (MetaGpuKms *gpu_kms,
setup_output_clones (gpu); setup_output_clones (gpu);
} }
static gboolean static void
meta_kms_resources_init (MetaKmsResources *resources, meta_kms_resources_init (MetaKmsResources *resources,
int fd, int fd)
GError **error)
{ {
drmModeRes *drm_resources; drmModeRes *drm_resources;
unsigned int i; unsigned int i;
drm_resources = drmModeGetResources (fd); drm_resources = drmModeGetResources (fd);
if (!drm_resources)
{
g_set_error (error,
G_IO_ERROR,
G_IO_ERROR_FAILED,
"Calling drmModeGetResources() failed");
return FALSE;
}
resources->resources = drm_resources; resources->resources = drm_resources;
resources->n_encoders = (unsigned int) drm_resources->count_encoders; resources->n_encoders = (unsigned int) drm_resources->count_encoders;
resources->encoders = g_new (drmModeEncoder *, resources->n_encoders); resources->encoders = g_new (drmModeEncoder *, resources->n_encoders);
for (i = 0; i < resources->n_encoders; i++) for (i = 0; i < resources->n_encoders; i++)
resources->encoders[i] = drmModeGetEncoder (fd, drm_resources->encoders[i]); resources->encoders[i] = drmModeGetEncoder (fd, drm_resources->encoders[i]);
return TRUE;
} }
static void static void
@ -767,7 +751,7 @@ meta_kms_resources_release (MetaKmsResources *resources)
drmModeFreeEncoder (resources->encoders[i]); drmModeFreeEncoder (resources->encoders[i]);
g_free (resources->encoders); g_free (resources->encoders);
g_clear_pointer (&resources->resources, drmModeFreeResources); drmModeFreeResources (resources->resources);
} }
static gboolean static gboolean
@ -778,18 +762,8 @@ meta_gpu_kms_read_current (MetaGpu *gpu,
MetaMonitorManager *monitor_manager = MetaMonitorManager *monitor_manager =
meta_gpu_get_monitor_manager (gpu); meta_gpu_get_monitor_manager (gpu);
MetaKmsResources resources; MetaKmsResources resources;
g_autoptr (GError) local_error = NULL;
if (!meta_kms_resources_init (&resources, gpu_kms->fd, &local_error)) meta_kms_resources_init (&resources, gpu_kms->fd);
{
if (!gpu_kms->resources_init_failed_before)
{
g_warning ("meta_kms_resources_init failed: %s, assuming we have no outputs",
local_error->message);
gpu_kms->resources_init_failed_before = TRUE;
return TRUE;
}
}
gpu_kms->max_buffer_width = resources.resources->max_width; gpu_kms->max_buffer_width = resources.resources->max_width;
gpu_kms->max_buffer_height = resources.resources->max_height; gpu_kms->max_buffer_height = resources.resources->max_height;