[Mesa-dev] [PATCH 05/16] loader: reimplement loader_get_user_preferred_fd via libdrm
Emil Velikov
emil.l.velikov at gmail.com
Tue Oct 11 18:31:49 UTC 2016
From: Emil Velikov <emil.velikov at collabora.com>
Currently not everyone has libudev and with follow-up patches we'll
completely remove the divergent codepaths.
Use the libdrm drm device API to construct the required ID_PATH_TAG-like
string, to preserve the current functionality and add a couple of notes,
where/how the current code is broken.
Signed-off-by: Emil Velikov <emil.velikov at collabora.com>
---
src/loader/loader.c | 201 +++++++++++++++++-----------------------------------
1 file changed, 64 insertions(+), 137 deletions(-)
diff --git a/src/loader/loader.c b/src/loader/loader.c
index ad4f946..beb383e 100644
--- a/src/loader/loader.c
+++ b/src/loader/loader.c
@@ -73,12 +73,6 @@
#ifdef HAVE_LIBUDEV
#include <assert.h>
#include <dlfcn.h>
-#include <unistd.h>
-#include <stdlib.h>
-#ifdef USE_DRICONF
-#include "xmlconfig.h"
-#include "xmlpool.h"
-#endif
#endif
#ifdef MAJOR_IN_MKDEV
#include <sys/mkdev.h>
@@ -89,7 +83,13 @@
#include "loader.h"
#ifdef HAVE_LIBDRM
+#include <stdlib.h>
+#include <unistd.h>
#include <xf86drm.h>
+#ifdef USE_DRICONF
+#include "xmlconfig.h"
+#include "xmlpool.h"
+#endif
#endif
#define __IS_LOADER
@@ -203,99 +203,9 @@ udev_device_new_from_fd(struct udev *udev, int fd)
return device;
}
+#endif
-static char *
-get_render_node_from_id_path_tag(struct udev *udev,
- char *id_path_tag,
- char another_tag)
-{
- struct udev_device *device;
- struct udev_enumerate *e;
- struct udev_list_entry *entry;
- const char *path, *id_path_tag_tmp;
- char *path_res;
- char found = 0;
- UDEV_SYMBOL(struct udev_enumerate *, udev_enumerate_new,
- (struct udev *));
- UDEV_SYMBOL(int, udev_enumerate_add_match_subsystem,
- (struct udev_enumerate *, const char *));
- UDEV_SYMBOL(int, udev_enumerate_add_match_sysname,
- (struct udev_enumerate *, const char *));
- UDEV_SYMBOL(int, udev_enumerate_scan_devices,
- (struct udev_enumerate *));
- UDEV_SYMBOL(struct udev_list_entry *, udev_enumerate_get_list_entry,
- (struct udev_enumerate *));
- UDEV_SYMBOL(void, udev_enumerate_unref,
- (struct udev_enumerate *));
- UDEV_SYMBOL(struct udev_list_entry *, udev_list_entry_get_next,
- (struct udev_list_entry *));
- UDEV_SYMBOL(const char *, udev_list_entry_get_name,
- (struct udev_list_entry *));
- UDEV_SYMBOL(struct udev_device *, udev_device_new_from_syspath,
- (struct udev *, const char *));
- UDEV_SYMBOL(const char *, udev_device_get_property_value,
- (struct udev_device *, const char *));
- UDEV_SYMBOL(const char *, udev_device_get_devnode,
- (struct udev_device *));
- UDEV_SYMBOL(struct udev_device *, udev_device_unref,
- (struct udev_device *));
-
- e = udev_enumerate_new(udev);
- udev_enumerate_add_match_subsystem(e, "drm");
- udev_enumerate_add_match_sysname(e, "render*");
-
- udev_enumerate_scan_devices(e);
- udev_list_entry_foreach(entry, udev_enumerate_get_list_entry(e)) {
- path = udev_list_entry_get_name(entry);
- device = udev_device_new_from_syspath(udev, path);
- if (!device)
- continue;
- id_path_tag_tmp = udev_device_get_property_value(device, "ID_PATH_TAG");
- if (id_path_tag_tmp) {
- if ((!another_tag && !strcmp(id_path_tag, id_path_tag_tmp)) ||
- (another_tag && strcmp(id_path_tag, id_path_tag_tmp))) {
- found = 1;
- break;
- }
- }
- udev_device_unref(device);
- }
-
- udev_enumerate_unref(e);
-
- if (found) {
- path_res = strdup(udev_device_get_devnode(device));
- udev_device_unref(device);
- return path_res;
- }
- return NULL;
-}
-
-static char *
-get_id_path_tag_from_fd(struct udev *udev, int fd)
-{
- struct udev_device *device;
- const char *id_path_tag_tmp;
- char *id_path_tag;
- UDEV_SYMBOL(const char *, udev_device_get_property_value,
- (struct udev_device *, const char *));
- UDEV_SYMBOL(struct udev_device *, udev_device_unref,
- (struct udev_device *));
-
- device = udev_device_new_from_fd(udev, fd);
- if (!device)
- return NULL;
-
- id_path_tag_tmp = udev_device_get_property_value(device, "ID_PATH_TAG");
- if (!id_path_tag_tmp)
- return NULL;
-
- id_path_tag = strdup(id_path_tag_tmp);
-
- udev_device_unref(device);
- return id_path_tag;
-}
-
+#if defined(HAVE_LIBDRM)
#ifdef USE_DRICONF
static const char __driConfigOptionsLoader[] =
DRI_CONF_BEGIN
@@ -321,17 +231,32 @@ static char *loader_get_dri_config_device_id(void)
}
#endif
+static char *drm_construct_id_path_tag(drmDevicePtr device)
+{
+/* Length of "pci-xxxx_xx_xx_x\n" */
+#define PCI_ID_PATH_TAG_LENGTH 17
+ char *tag = NULL;
+
+ if (device->bustype == DRM_BUS_PCI) {
+ tag = calloc(PCI_ID_PATH_TAG_LENGTH, sizeof(char));
+ if (tag == NULL)
+ return NULL;
+
+ sprintf(tag, "pci-%04x_%02x_%02x_%1u", device->businfo.pci->domain,
+ device->businfo.pci->bus, device->businfo.pci->dev,
+ device->businfo.pci->func);
+ }
+ return tag;
+}
+
int loader_get_user_preferred_fd(int default_fd, int *different_device)
{
- struct udev *udev;
const char *dri_prime = getenv("DRI_PRIME");
+ int is_different_device, fd;
+ char *default_tag;
+ char *device_name;
char *prime = NULL;
- int is_different_device = 0, fd = default_fd;
- char *default_device_id_path_tag;
- char *device_name = NULL;
- char another_tag = 0;
- UDEV_SYMBOL(struct udev *, udev_new, (void));
- UDEV_SYMBOL(struct udev *, udev_unref, (struct udev *));
+ drmDevicePtr device = NULL;
if (dri_prime)
prime = strdup(dri_prime);
@@ -345,55 +270,57 @@ int loader_get_user_preferred_fd(int default_fd, int *different_device)
return default_fd;
}
- udev = udev_new();
- if (!udev)
- goto prime_clean;
+ if (drmGetDevice(default_fd, &device) != 0)
+ goto err;
- default_device_id_path_tag = get_id_path_tag_from_fd(udev, default_fd);
- if (!default_device_id_path_tag)
- goto udev_clean;
-
- is_different_device = 1;
/* two format are supported:
* "1": choose any other card than the card used by default.
* id_path_tag: (for example "pci-0000_02_00_0") choose the card
* with this id_path_tag.
*/
if (!strcmp(prime,"1")) {
- free(prime);
- prime = strdup(default_device_id_path_tag);
- /* request a card with a different card than the default card */
- another_tag = 1;
- } else if (!strcmp(default_device_id_path_tag, prime))
- /* we are to get a new fd (render-node) of the same device */
- is_different_device = 0;
+ /* Hmm... detection for 2-7 seems to be broken. Nicely done. */
+ is_different_device = 1;
+ } else {
+ default_tag = drm_construct_id_path_tag(device);
+ if (default_tag == NULL)
+ goto err;
- device_name = get_render_node_from_id_path_tag(udev,
- prime,
- another_tag);
- if (device_name == NULL) {
+ if (strcmp(default_tag, prime) != 0) {
+ free(default_tag);
+ goto err;
+ }
+
+ free(default_tag);
+ /* we are to get a new fd (render-node) of the same device */
is_different_device = 0;
- goto default_device_clean;
+ // XXX: WTF ^^ so one uses the new model only to point to the exact same
+ // device and not the other more/less powerful GPU ?
}
+ device_name = device->nodes[DRM_NODE_RENDER];
+
+ /* The device/node name provided by libdrm may be an empty string */
+ if (device_name == NULL || device_name[0] == '\0')
+ goto err;
fd = loader_open_device(device_name);
- if (fd >= 0) {
- close(default_fd);
- } else {
- fd = default_fd;
- is_different_device = 0;
- }
- free(device_name);
+ if (fd < 0)
+ goto err;
- default_device_clean:
- free(default_device_id_path_tag);
- udev_clean:
- udev_unref(udev);
- prime_clean:
+ close(default_fd);
+
+ drmFreeDevice(&device);
free(prime);
*different_device = is_different_device;
return fd;
+
+ err:
+ drmFreeDevice(&device);
+ free(prime);
+
+ *different_device = 0;
+ return default_fd;
}
#else
int loader_get_user_preferred_fd(int default_fd, int *different_device)
--
2.10.0
More information about the mesa-dev
mailing list