[Mesa-dev] [PATCH 01/10] loader: introduce the loader util lib
Jordan Justen
jljusten at gmail.com
Tue Jan 21 16:46:45 PST 2014
On Sat, Jan 11, 2014 at 8:54 AM, Emil Velikov <emil.l.velikov at gmail.com> wrote:
> All the various window system integration layers duplicate roughly the
> same code for figuring out device and driver name, pci-id's, etc. Which
> is sad. So extract it out into a loader util lib.
>
> Signed-off-by: Rob Clark <robclark at freedesktop.org>
>
> v2 (Emil)
> * Separate the introduction of libloader from the code de-duplication.
> * Strip out non-pci devices support.
> * Add scons + Android build system support.
> * Add VISIBILITY_CFLAGS to avoid exporting the loader funcs.
>
> Signed-off-by: Emil Velikov <emil.l.velikov at gmail.com>
> ---
>
> Note: src/loader.c needs a bit more work, wrt Android detection :\
>
> ---
>
> Android.mk | 1 +
> configure.ac | 1 +
> include/pci_ids/pci_id_driver_map.h | 24 ++-
> src/Makefile.am | 2 +-
> src/SConscript | 1 +
> src/egl/drivers/dri2/Android.mk | 1 +
> src/egl/drivers/dri2/Makefile.am | 1 +
> src/gallium/auxiliary/pipe-loader/Makefile.am | 1 +
> src/gallium/state_trackers/egl/Makefile.am | 1 +
> src/gallium/state_trackers/egl/SConscript | 1 +
> src/gallium/targets/egl-static/Android.mk | 1 +
> src/gallium/targets/egl-static/Makefile.am | 1 +
> src/gallium/targets/egl-static/SConscript | 1 +
> src/gbm/Makefile.am | 6 +-
> src/glx/Makefile.am | 1 +
> src/glx/SConscript | 1 +
> src/loader/Android.mk | 43 +++++
> src/loader/Makefile.am | 37 ++++
> src/loader/Makefile.sources | 2 +
> src/loader/SConscript | 28 +++
> src/loader/loader.c | 264 ++++++++++++++++++++++++++
> src/loader/loader.h | 57 ++++++
> 22 files changed, 461 insertions(+), 15 deletions(-)
> create mode 100644 src/loader/Android.mk
> create mode 100644 src/loader/Makefile.am
> create mode 100644 src/loader/Makefile.sources
> create mode 100644 src/loader/SConscript
> create mode 100644 src/loader/loader.c
> create mode 100644 src/loader/loader.h
>
> diff --git a/Android.mk b/Android.mk
> index c3c1f08..05ed62f 100644
> --- a/Android.mk
> +++ b/Android.mk
> @@ -78,6 +78,7 @@ endif
> ifneq ($(strip $(MESA_GPU_DRIVERS)),)
>
> SUBDIRS := \
> + src/loader \
> src/mapi \
> src/glsl \
> src/mesa \
> diff --git a/configure.ac b/configure.ac
> index 4b55140..ce677d8 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -2059,6 +2059,7 @@ AC_CONFIG_FILES([Makefile
> src/glx/Makefile
> src/glx/tests/Makefile
> src/gtest/Makefile
> + src/loader/Makefile
> src/mapi/Makefile
> src/mapi/es1api/Makefile
> src/mapi/es1api/glesv1_cm.pc
> diff --git a/include/pci_ids/pci_id_driver_map.h b/include/pci_ids/pci_id_driver_map.h
> index 8a97c6f..2e88451 100644
> --- a/include/pci_ids/pci_id_driver_map.h
> +++ b/include/pci_ids/pci_id_driver_map.h
> @@ -2,6 +2,7 @@
> #define _PCI_ID_DRIVER_MAP_H_
>
> #include <stddef.h>
> +#include "loader.h"
>
> #ifndef ARRAY_SIZE
> #define ARRAY_SIZE(a) (sizeof(a) / sizeof((a)[0]))
> @@ -19,7 +20,6 @@ static const int i965_chip_ids[] = {
> #undef CHIPSET
> };
>
> -#ifndef DRIVER_MAP_GALLIUM_ONLY
> static const int r100_chip_ids[] = {
> #define CHIPSET(chip, name, family) chip,
> #include "pci_ids/radeon_pci_ids.h"
> @@ -31,7 +31,6 @@ static const int r200_chip_ids[] = {
> #include "pci_ids/r200_pci_ids.h"
> #undef CHIPSET
> };
> -#endif
>
> static const int r300_chip_ids[] = {
> #define CHIPSET(chip, name, family) chip,
> @@ -62,18 +61,17 @@ static const struct {
> const char *driver;
> const int *chip_ids;
> int num_chips_ids;
> + unsigned driver_types;
> } driver_map[] = {
> - { 0x8086, "i915", i915_chip_ids, ARRAY_SIZE(i915_chip_ids) },
> - { 0x8086, "i965", i965_chip_ids, ARRAY_SIZE(i965_chip_ids) },
> -#ifndef DRIVER_MAP_GALLIUM_ONLY
> - { 0x1002, "radeon", r100_chip_ids, ARRAY_SIZE(r100_chip_ids) },
> - { 0x1002, "r200", r200_chip_ids, ARRAY_SIZE(r200_chip_ids) },
> -#endif
> - { 0x1002, "r300", r300_chip_ids, ARRAY_SIZE(r300_chip_ids) },
> - { 0x1002, "r600", r600_chip_ids, ARRAY_SIZE(r600_chip_ids) },
> - { 0x1002, "radeonsi", radeonsi_chip_ids, ARRAY_SIZE(radeonsi_chip_ids) },
> - { 0x10de, "nouveau", NULL, -1 },
> - { 0x15ad, "vmwgfx", vmwgfx_chip_ids, ARRAY_SIZE(vmwgfx_chip_ids) },
> + { 0x8086, "i915", i915_chip_ids, ARRAY_SIZE(i915_chip_ids), _LOADER_DRI | _LOADER_GALLIUM },
> + { 0x8086, "i965", i965_chip_ids, ARRAY_SIZE(i965_chip_ids), _LOADER_DRI | _LOADER_GALLIUM },
> + { 0x1002, "radeon", r100_chip_ids, ARRAY_SIZE(r100_chip_ids), _LOADER_DRI },
> + { 0x1002, "r200", r200_chip_ids, ARRAY_SIZE(r200_chip_ids), _LOADER_DRI },
> + { 0x1002, "r300", r300_chip_ids, ARRAY_SIZE(r300_chip_ids), _LOADER_GALLIUM },
> + { 0x1002, "r600", r600_chip_ids, ARRAY_SIZE(r600_chip_ids), _LOADER_GALLIUM },
> + { 0x1002, "radeonsi", radeonsi_chip_ids, ARRAY_SIZE(radeonsi_chip_ids), _LOADER_GALLIUM},
> + { 0x10de, "nouveau", NULL, -1, _LOADER_GALLIUM },
> + { 0x15ad, "vmwgfx", vmwgfx_chip_ids, ARRAY_SIZE(vmwgfx_chip_ids), _LOADER_GALLIUM },
> { 0x0000, NULL, NULL, 0 },
> };
>
> diff --git a/src/Makefile.am b/src/Makefile.am
> index 76280a0..5b2549d 100644
> --- a/src/Makefile.am
> +++ b/src/Makefile.am
> @@ -19,7 +19,7 @@
> # FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS
> # IN THE SOFTWARE.
>
> -SUBDIRS = gtest mapi
> +SUBDIRS = gtest loader mapi
>
> if NEED_OPENGL_COMMON
> SUBDIRS += glsl mesa
> diff --git a/src/SConscript b/src/SConscript
> index a24acea..93a490d 100644
> --- a/src/SConscript
> +++ b/src/SConscript
> @@ -11,6 +11,7 @@ if env['hostonly']:
> # compilation
> Return()
>
> +SConscript('loader/SConscript')
>
> # When env['gles'] is set, the targets defined in mapi/glapi/SConscript are not
> # used. libgl-xlib and libgl-gdi adapt themselves to use the targets defined
> diff --git a/src/egl/drivers/dri2/Android.mk b/src/egl/drivers/dri2/Android.mk
> index 090a0e1..4af8581 100644
> --- a/src/egl/drivers/dri2/Android.mk
> +++ b/src/egl/drivers/dri2/Android.mk
> @@ -39,6 +39,7 @@ LOCAL_CFLAGS := \
> LOCAL_C_INCLUDES := \
> $(MESA_TOP)/src/mapi \
> $(MESA_TOP)/src/egl/main \
> + $(MESA_TOP)/src/loader \
> $(DRM_GRALLOC_TOP) \
> $(DRM_TOP) \
> $(DRM_TOP)/include/drm
> diff --git a/src/egl/drivers/dri2/Makefile.am b/src/egl/drivers/dri2/Makefile.am
> index 823ef5e..7e8ed78 100644
> --- a/src/egl/drivers/dri2/Makefile.am
> +++ b/src/egl/drivers/dri2/Makefile.am
> @@ -22,6 +22,7 @@
> AM_CFLAGS = \
> -I$(top_srcdir)/include \
> -I$(top_srcdir)/src/egl/main \
> + -I$(top_srcdir)/src/loader \
> -I$(top_srcdir)/src/gbm/main \
> -I$(top_srcdir)/src/gbm/backends/dri \
> -I$(top_srcdir)/src/egl/wayland/wayland-egl \
> diff --git a/src/gallium/auxiliary/pipe-loader/Makefile.am b/src/gallium/auxiliary/pipe-loader/Makefile.am
> index 9a8094f..5403447 100644
> --- a/src/gallium/auxiliary/pipe-loader/Makefile.am
> +++ b/src/gallium/auxiliary/pipe-loader/Makefile.am
> @@ -3,6 +3,7 @@ AUTOMAKE_OPTIONS = subdir-objects
> AM_CPPFLAGS = $(DEFINES) \
> $(GALLIUM_PIPE_LOADER_DEFINES) \
> -I$(top_srcdir)/include \
> + -I$(top_srcdir)/src/loader \
> -I$(top_srcdir)/src/gallium/include \
> -I$(top_srcdir)/src/gallium/auxiliary \
> -I$(top_srcdir)/src/gallium/winsys
> diff --git a/src/gallium/state_trackers/egl/Makefile.am b/src/gallium/state_trackers/egl/Makefile.am
> index c069c29..459fabc 100644
> --- a/src/gallium/state_trackers/egl/Makefile.am
> +++ b/src/gallium/state_trackers/egl/Makefile.am
> @@ -66,6 +66,7 @@ libegl_la_SOURCES += $(drm_FILES)
> AM_CFLAGS += \
> $(LIBDRM_CFLAGS)
> AM_CPPFLAGS += \
> + -I$(top_srcdir)/src/loader \
> -I$(top_srcdir)/src/gallium/winsys \
> -I$(top_srcdir)/src/gbm/main \
> -I$(top_srcdir)/src/gallium/state_trackers/gbm \
> diff --git a/src/gallium/state_trackers/egl/SConscript b/src/gallium/state_trackers/egl/SConscript
> index bd0ee02..2a5db64 100644
> --- a/src/gallium/state_trackers/egl/SConscript
> +++ b/src/gallium/state_trackers/egl/SConscript
> @@ -33,6 +33,7 @@ else:
> # XXX: Disabled as it depends on gbm, which is not yet built with scons
> env.Append(CPPDEFINES = ['HAVE_DRM_BACKEND'])
> env.Append(CPPPATH = [
> + '#/src/loader',
> '#/src/gbm/main',
> '#/src/gallium/state_trackers/gbm',
> ])
> diff --git a/src/gallium/targets/egl-static/Android.mk b/src/gallium/targets/egl-static/Android.mk
> index 06975db..88df6d1 100644
> --- a/src/gallium/targets/egl-static/Android.mk
> +++ b/src/gallium/targets/egl-static/Android.mk
> @@ -39,6 +39,7 @@ LOCAL_C_INCLUDES := \
> $(GALLIUM_TOP)/state_trackers/vega \
> $(GALLIUM_TOP)/state_trackers/egl \
> $(MESA_TOP)/src/egl/main \
> + $(MESA_TOP)/src/loader \
> $(MESA_TOP)/src/mesa
>
> # swrast
> diff --git a/src/gallium/targets/egl-static/Makefile.am b/src/gallium/targets/egl-static/Makefile.am
> index fdd5a1d..ebffd0e 100644
> --- a/src/gallium/targets/egl-static/Makefile.am
> +++ b/src/gallium/targets/egl-static/Makefile.am
> @@ -36,6 +36,7 @@ AM_CFLAGS = $(PTHREAD_CFLAGS)
> AM_CPPFLAGS = \
> $(GALLIUM_CFLAGS) \
> -I$(top_srcdir)/include \
> + -I$(top_srcdir)/src/loader \
> -I$(top_srcdir)/src/gallium/drivers \
> -I$(top_srcdir)/src/gallium/winsys \
> -I$(top_srcdir)/src/gallium/include \
> diff --git a/src/gallium/targets/egl-static/SConscript b/src/gallium/targets/egl-static/SConscript
> index b4ccc5b..56e12e8 100644
> --- a/src/gallium/targets/egl-static/SConscript
> +++ b/src/gallium/targets/egl-static/SConscript
> @@ -7,6 +7,7 @@ env = env.Clone()
>
> env.Append(CPPPATH = [
> '#/include',
> + '#/src/loader',
> '#/src/egl/main',
> '#/src/gallium/auxiliary',
> '#/src/gallium/drivers',
> diff --git a/src/gbm/Makefile.am b/src/gbm/Makefile.am
> index 9b3fb0d..5ae7a68 100644
> --- a/src/gbm/Makefile.am
> +++ b/src/gbm/Makefile.am
> @@ -7,6 +7,7 @@ AM_CFLAGS = \
> -D_OS_UNIX=1 \
> -DMODULEDIR='"$(libdir)/gbm"' \
> -I$(top_srcdir)/include \
> + -I$(top_srcdir)/src/loader \
> -I$(top_srcdir)/src/gbm/main \
> $(LIBUDEV_CFLAGS) \
> $(LIBKMS_CFLAGS) \
> @@ -21,7 +22,10 @@ libgbm_la_SOURCES = \
> main/backend.c \
> main/common.c
> libgbm_la_LDFLAGS = -version-info 1:0
> -libgbm_la_LIBADD = $(LIBUDEV_LIBS) $(LIBKMS_LIBS) $(DLOPEN_LIBS)
> +libgbm_la_LIBADD = \
> + $(LIBUDEV_LIBS) \
> + $(LIBKMS_LIBS) \
> + $(DLOPEN_LIBS)
>
> if HAVE_EGL_PLATFORM_WAYLAND
> AM_CPPFLAGS = -DHAVE_WAYLAND_PLATFORM
> diff --git a/src/glx/Makefile.am b/src/glx/Makefile.am
> index 0aec2aa..6d81471 100644
> --- a/src/glx/Makefile.am
> +++ b/src/glx/Makefile.am
> @@ -35,6 +35,7 @@ endif
> AM_CFLAGS = \
> -I$(top_srcdir)/include \
> -I$(top_srcdir)/include/GL/internal \
> + -I$(top_srcdir)/src/loader \
> -I$(top_srcdir)/src/mapi \
> -I$(top_srcdir)/src/mapi/glapi \
> -I$(top_builddir)/src/mapi \
> diff --git a/src/glx/SConscript b/src/glx/SConscript
> index 890f998..309a648 100644
> --- a/src/glx/SConscript
> +++ b/src/glx/SConscript
> @@ -11,6 +11,7 @@ env.Prepend(CPPPATH = [
> '.', # the build/<platform>/glx/ directory
> '#include',
> '#include/GL/internal',
> + '#src/loader',
> '#src/mesa',
> '#src/mapi',
> '#src/mapi/glapi',
> diff --git a/src/loader/Android.mk b/src/loader/Android.mk
> new file mode 100644
> index 0000000..c7453dd
> --- /dev/null
> +++ b/src/loader/Android.mk
> @@ -0,0 +1,43 @@
> +# Mesa 3-D graphics library
> +#
> +# Copyright (C) 2014 Emil Velikov <emil.l.velikov at gmail.com>
> +#
> +# Permission is hereby granted, free of charge, to any person obtaining a
> +# copy of this software and associated documentation files (the "Software"),
> +# to deal in the Software without restriction, including without limitation
> +# the rights to use, copy, modify, merge, publish, distribute, sublicense,
> +# and/or sell copies of the Software, and to permit persons to whom the
> +# Software is furnished to do so, subject to the following conditions:
> +#
> +# The above copyright notice and this permission notice shall be included
> +# in all copies or substantial portions of the Software.
> +#
> +# THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
> +# IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
> +# FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL
> +# THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
> +# LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
> +# FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER
> +# DEALINGS IN THE SOFTWARE.
> +
> +LOCAL_PATH := $(call my-dir)
> +
> +include $(LOCAL_PATH)/Makefile.sources
> +
> +# ---------------------------------------
> +# Build libloader
> +# ---------------------------------------
> +
> +include $(CLEAR_VARS)
> +
> +LOCAL_SRC_FILES := \
> + $(LOADER_C_FILES)
> +
> +LOCAL_C_INCLUDES := \
> + $(DRM_TOP) \
> + $(DRM_TOP)/include/drm
> +
> +LOCAL_MODULE := libloader
> +
> +include $(MESA_COMMON_MK)
> +include $(BUILD_STATIC_LIBRARY)
> diff --git a/src/loader/Makefile.am b/src/loader/Makefile.am
> new file mode 100644
> index 0000000..a118fef
> --- /dev/null
> +++ b/src/loader/Makefile.am
> @@ -0,0 +1,37 @@
> +# Copyright © 2012 Intel Corporation
> +#
> +# Permission is hereby granted, free of charge, to any person obtaining a
> +# copy of this software and associated documentation files (the "Software"),
> +# to deal in the Software without restriction, including without limitation
> +# the rights to use, copy, modify, merge, publish, distribute, sublicense,
> +# and/or sell copies of the Software, and to permit persons to whom the
> +# Software is furnished to do so, subject to the following conditions:
> +#
> +# The above copyright notice and this permission notice (including the next
> +# paragraph) shall be included in all copies or substantial portions of the
> +# Software.
> +#
> +# THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
> +# IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
> +# FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL
> +# THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
> +# LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
> +# FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS
> +# IN THE SOFTWARE.
> +
> +include Makefile.sources
> +
> +noinst_LTLIBRARIES = libloader.la
> +
> +libloader_la_CPPFLAGS = \
> + $(DEFINES) \
> + -I$(top_srcdir)/include \
> + $(VISIBILITY_CFLAGS) \
> + $(LIBDRM_CFLAGS) \
> + $(LIBUDEV_CFLAGS)
> +
> +libloader_la_LIBADD = \
> + $(LIBDRM_LIBS) \
> + $(LIBUDEV_LIBS)
> +
> +libloader_la_SOURCES = $(LOADER_C_FILES)
> diff --git a/src/loader/Makefile.sources b/src/loader/Makefile.sources
> new file mode 100644
> index 0000000..51a64ea
> --- /dev/null
> +++ b/src/loader/Makefile.sources
> @@ -0,0 +1,2 @@
> +LOADER_C_FILES := \
> + loader.c
> \ No newline at end of file
> diff --git a/src/loader/SConscript b/src/loader/SConscript
> new file mode 100644
> index 0000000..454e328
> --- /dev/null
> +++ b/src/loader/SConscript
> @@ -0,0 +1,28 @@
> +Import('*')
> +
> +if not env['drm']:
> + Return()
> +
> +env = env.Clone()
> +
> +env.Prepend(CPPPATH = [
> + '.',
> + '#include'
> +])
> +
> +env.PkgUseModules('DRM')
> +
> +if env['udev']:
> + env.PkgUseModules('UDEV')
> + env.Append(CPPDEFINES = ['HAVE_LIBUDEV'])
> +
> +# parse Makefile.sources
> +sources = env.ParseSourceList('Makefile.sources', 'LOADER_C_FILES')
> +
> +libloader = env.ConvenienceLibrary(
> + target = 'libloader',
> + source = sources,
> +)
> +
> +env.Alias('libloader', libloader)
> +Export('libloader')
> diff --git a/src/loader/loader.c b/src/loader/loader.c
> new file mode 100644
> index 0000000..7d43603
> --- /dev/null
> +++ b/src/loader/loader.c
> @@ -0,0 +1,264 @@
> +/*
> + * Copyright (C) 2013 Rob Clark <robclark at freedesktop.org>
> + *
> + * Permission is hereby granted, free of charge, to any person obtaining a
> + * copy of this software and associated documentation files (the "Software"),
> + * to deal in the Software without restriction, including without limitation
> + * the rights to use, copy, modify, merge, publish, distribute, sublicense,
> + * and/or sell copies of the Software, and to permit persons to whom the
> + * Software is furnished to do so, subject to the following conditions:
> + *
> + * The above copyright notice and this permission notice (including the next
> + * paragraph) shall be included in all copies or substantial portions of the
> + * Software.
> + *
> + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
> + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
> + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL
> + * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
> + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
> + * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
> + * SOFTWARE.
> + *
> + * Authors:
> + * Rob Clark <robclark at freedesktop.org>
> + */
> +
> +#include <stdarg.h>
> +#include <stdio.h>
> +#include <string.h>
> +#include "loader.h"
> +
> +#include <xf86drm.h>
> +
> +#define __IS_LOADER
> +#include "pci_ids/pci_id_driver_map.h"
> +
> +static void default_logger(int level, const char *fmt, ...)
> +{
> + if (level >= _LOADER_WARNING) {
> + va_list args;
> + va_start(args, fmt);
> + vfprintf(stderr, fmt, args);
> + va_end(args);
> + fprintf(stderr, "\n");
> + }
> +}
> +
> +static void (*log)(int level, const char *fmt, ...) = default_logger;
> +
> +#ifdef HAVE_LIBUDEV
> +#include <libudev.h>
> +
> +static inline struct udev_device *
> +udev_device_new_from_fd(struct udev *udev, int fd)
> +{
> + struct udev_device *device;
> + struct stat buf;
> +
> + if (fstat(fd, &buf) < 0) {
> + log(_LOADER_WARNING, "MESA-LOADER: failed to stat fd %d", fd);
> + return NULL;
> + }
> +
> + device = udev_device_new_from_devnum(udev, 'c', buf.st_rdev);
> + if (device == NULL) {
> + log(_LOADER_WARNING,
> + "MESA-LOADER: could not create udev device for fd %d", fd);
> + return NULL;
> + }
> +
> + return device;
> +}
> +
> +int
> +loader_get_pci_id_for_fd(int fd, int *vendor_id, int *chip_id)
> +{
> + struct udev *udev = NULL;
> + struct udev_device *device = NULL, *parent;
> + struct stat buf;
> + const char *pci_id;
> +
> + *chip_id = -1;
> +
> + udev = udev_new();
> + device = udev_device_new_from_fd(udev, fd);
> + if (!device)
> + goto out;
> +
> + parent = udev_device_get_parent(device);
> + if (parent == NULL) {
> + log(_LOADER_WARNING, "MESA-LOADER: could not get parent device");
> + goto out;
> + }
> +
> + pci_id = udev_device_get_property_value(parent, "PCI_ID");
> + if (pci_id == NULL ||
> + sscanf(pci_id, "%x:%x", vendor_id, chip_id) != 2) {
> + log(_LOADER_WARNING, "MESA-LOADER: malformed or no PCI ID");
> + *chip_id = -1;
> + goto out;
> + }
> +
> +out:
> + if (device)
> + udev_device_unref(device);
> + if (udev)
> + udev_unref(udev);
> +
> + return (*chip_id >= 0);
> +}
> +
> +#elif defined(PIPE_OS_ANDROID) && !defined(_EGL_NO_DRM)
> +
> +/* for i915 */
> +#include <i915_drm.h>
> +/* for radeon */
> +#include <radeon_drm.h>
> +
> +int
> +loader_get_pci_id_for_fd(int fd, int *vendor_id, int *chip_id)
> +{
> + drmVersionPtr version;
> +
> + *chip_id = -1;
> +
> + version = drmGetVersion(fd);
> + if (!version) {
> + log(_LOADER_WARNING, "MESA-LOADER: invalid drm fd");
> + return FALSE;
> + }
> + if (!version->name) {
> + log(_LOADER_WARNING, "MESA-LOADER: unable to determine the driver name");
> + drmFreeVersion(version);
> + return FALSE;
> + }
> +
> + if (strcmp(version->name, "i915") == 0) {
> + struct drm_i915_getparam gp;
> + int ret;
> +
> + *vendor_id = 0x8086;
> +
> + memset(&gp, 0, sizeof(gp));
> + gp.param = I915_PARAM_CHIPSET_ID;
> + gp.value = chip_id;
> + ret = drmCommandWriteRead(fd, DRM_I915_GETPARAM, &gp, sizeof(gp));
> + if (ret) {
> + log(_LOADER_WARNING, "MESA-LOADER: failed to get param for i915");
> + *chip_id = -1;
> + }
> + }
> + else if (strcmp(version->name, "radeon") == 0) {
> + struct drm_radeon_info info;
> + int ret;
> +
> + *vendor_id = 0x1002;
> +
> + memset(&info, 0, sizeof(info));
> + info.request = RADEON_INFO_DEVICE_ID;
> + info.value = (unsigned long) chip_id;
> + ret = drmCommandWriteRead(fd, DRM_RADEON_INFO, &info, sizeof(info));
> + if (ret) {
> + log(_LOADER_WARNING, "MESA-LOADER: failed to get info for radeon");
> + *chip_id = -1;
> + }
> + }
> + else if (strcmp(version->name, "nouveau") == 0) {
> + *vendor_id = 0x10de;
> + /* not used */
> + *chip_id = 0;
> + }
> + else if (strcmp(version->name, "vmwgfx") == 0) {
> + *vendor_id = 0x15ad;
> + /* assume SVGA II */
> + *chip_id = 0x0405;
> + }
> +
> + drmFreeVersion(version);
> +
> + return (*chip_id >= 0);
> +}
> +
> +#else
> +
> +int
> +loader_get_pci_id_for_fd(int fd, int *vendor_id, int *chip_id)
> +{
> + return 0;
> +}
> +
> +#endif
> +
> +
> +char *
> +loader_get_device_name_for_fd(int fd)
> +{
> + char *device_name = NULL;
> +#ifdef HAVE_LIBUDEV
> + struct udev *udev;
> + struct udev_device *device;
> + const char *const_device_name;
> +
> + udev = udev_new();
> + device = udev_device_new_from_fd(udev, fd);
> + if (device == NULL)
> + return NULL;
> +
> + const_device_name = udev_device_get_devnode(device);
> + if (!const_device_name)
> + goto out;
> + device_name = strdup(const_device_name);
> +
> +out:
> + udev_device_unref(device);
> + udev_unref(udev);
> +#endif
> + return device_name;
> +}
> +
> +char *
> +loader_get_driver_for_fd(int fd, unsigned driver_types)
> +{
> + int vendor_id, chip_id, i, j;
> + char *driver = NULL;
> +
> + if (!driver_types)
> + driver_types = _LOADER_GALLIUM | _LOADER_DRI;
> +
> + if (!loader_get_pci_id_for_fd(fd, &vendor_id, &chip_id)) {
> + log(_LOADER_WARNING, "failed to get driver name for fd %d", fd);
> + return NULL;
> + }
> +
> + for (i = 0; driver_map[i].driver; i++) {
> + if (vendor_id != driver_map[i].vendor_id)
> + continue;
> +
> + if (!(driver_types & driver_map[i].driver_types))
> + continue;
> +
> + if (driver_map[i].num_chips_ids == -1) {
> + driver = strdup(driver_map[i].driver);
> + goto out;
> + }
> +
> + for (j = 0; j < driver_map[i].num_chips_ids; j++)
> + if (driver_map[i].chip_ids[j] == chip_id) {
> + driver = strdup(driver_map[i].driver);
> + goto out;
> + }
> + }
> +
> +out:
> + log(driver ? _LOADER_INFO : _LOADER_WARNING,
> + "pci id for fd %d: %04x:%04x, driver %s",
> + fd, vendor_id, chip_id, driver);
When I use gbm, I see this message printed to the console:
$ wflinfo -p gbm -a gl
pci id for fd 3: 8086:0a2e, driver i965
Waffle platform: gbm
Waffle api: gl
OpenGL vendor string: Intel Open Source Technology Center
OpenGL renderer string: Mesa DRI Intel(R) Haswell
OpenGL version string: 3.0 Mesa 10.1.0-devel (git-178c1bf)
This shouldn't be printed to the console by default, right?
Also, is there a concern of using driver in the message when it is
null, or does log guarantee that is okay?
-Jordan
> + return driver;
> +}
> +
> +void
> +loader_set_logger(void (*logger)(int level, const char *fmt, ...))
> +{
> + log = logger;
> +}
> diff --git a/src/loader/loader.h b/src/loader/loader.h
> new file mode 100644
> index 0000000..dfd77ba
> --- /dev/null
> +++ b/src/loader/loader.h
> @@ -0,0 +1,57 @@
> +/*
> + * Copyright (C) 2013 Rob Clark <robclark at freedesktop.org>
> + *
> + * Permission is hereby granted, free of charge, to any person obtaining a
> + * copy of this software and associated documentation files (the "Software"),
> + * to deal in the Software without restriction, including without limitation
> + * the rights to use, copy, modify, merge, publish, distribute, sublicense,
> + * and/or sell copies of the Software, and to permit persons to whom the
> + * Software is furnished to do so, subject to the following conditions:
> + *
> + * The above copyright notice and this permission notice (including the next
> + * paragraph) shall be included in all copies or substantial portions of the
> + * Software.
> + *
> + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
> + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
> + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL
> + * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
> + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
> + * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
> + * SOFTWARE.
> + *
> + * Authors:
> + * Rob Clark <robclark at freedesktop.org>
> + */
> +
> +#ifndef LOADER_H
> +#define LOADER_H
> +
> +/* Helpers to figure out driver and device name, eg. from pci-id, etc. */
> +
> +#define _LOADER_DRI (1 << 0)
> +#define _LOADER_GALLIUM (1 << 1)
> +
> +int
> +loader_get_pci_id_for_fd(int fd, int *vendor_id, int *chip_id);
> +
> +char *
> +loader_get_driver_for_fd(int fd, unsigned driver_types);
> +
> +char *
> +loader_get_device_name_for_fd(int fd);
> +
> +
> +/* for logging.. keep this aligned with egllog.h so we can just use
> + * _eglLog directly.
> + */
> +
> +#define _LOADER_FATAL 0 /* unrecoverable error */
> +#define _LOADER_WARNING 1 /* recoverable error/problem */
> +#define _LOADER_INFO 2 /* just useful info */
> +#define _LOADER_DEBUG 3 /* useful info for debugging */
> +
> +void
> +loader_set_logger(void (*logger)(int level, const char *fmt, ...));
> +
> +#endif /* LOADER_H */
> --
> 1.8.5.1
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev
More information about the mesa-dev
mailing list