[Mesa-dev] [PATCH 3/4] gallium: implement get_device_vendor() for existing drivers
Tom Stellard
tom at stellard.net
Fri Mar 20 06:24:16 PDT 2015
On Wed, Mar 11, 2015 at 03:27:18PM +0100, Giuseppe Bilotta wrote:
> The only hackish ones are llvmpipe and softpipe, which currently return
> the same string as for get_vendor(), while ideally they should return
> the CPU vendor.
>
> Signed-off-by: Giuseppe Bilotta <giuseppe.bilotta at gmail.com>
Reviewed-by: Tom Stellard <thomas.stellard at amd.com>
> ---
> src/gallium/drivers/freedreno/freedreno_screen.c | 8 ++++++++
> src/gallium/drivers/galahad/glhd_screen.c | 10 ++++++++++
> src/gallium/drivers/i915/i915_screen.c | 7 +++++++
> src/gallium/drivers/ilo/ilo_screen.c | 7 +++++++
> src/gallium/drivers/llvmpipe/lp_screen.c | 1 +
> src/gallium/drivers/noop/noop_pipe.c | 6 ++++++
> src/gallium/drivers/nouveau/nouveau_screen.c | 7 +++++++
> src/gallium/drivers/r300/r300_screen.c | 6 ++++++
> src/gallium/drivers/radeon/r600_pipe_common.c | 6 ++++++
> src/gallium/drivers/rbug/rbug_screen.c | 10 ++++++++++
> src/gallium/drivers/softpipe/sp_screen.c | 1 +
> src/gallium/drivers/svga/svga_screen.c | 1 +
> src/gallium/drivers/trace/tr_screen.c | 22 ++++++++++++++++++++++
> src/gallium/drivers/vc4/vc4_screen.c | 1 +
> 14 files changed, 93 insertions(+)
>
> diff --git a/src/gallium/drivers/freedreno/freedreno_screen.c b/src/gallium/drivers/freedreno/freedreno_screen.c
> index 31f596c..81d7f7a 100644
> --- a/src/gallium/drivers/freedreno/freedreno_screen.c
> +++ b/src/gallium/drivers/freedreno/freedreno_screen.c
> @@ -96,6 +96,13 @@ fd_screen_get_vendor(struct pipe_screen *pscreen)
> return "freedreno";
> }
>
> +static const char *
> +fd_screen_get_device_vendor(struct pipe_screen *pscreen)
> +{
> + return "Qualcomm";
> +}
> +
> +
> static uint64_t
> fd_screen_get_timestamp(struct pipe_screen *pscreen)
> {
> @@ -531,6 +538,7 @@ fd_screen_create(struct fd_device *dev)
>
> pscreen->get_name = fd_screen_get_name;
> pscreen->get_vendor = fd_screen_get_vendor;
> + pscreen->get_device_vendor = fd_screen_get_device_vendor;
>
> pscreen->get_timestamp = fd_screen_get_timestamp;
>
> diff --git a/src/gallium/drivers/galahad/glhd_screen.c b/src/gallium/drivers/galahad/glhd_screen.c
> index 11ab1a9..9fafdbf 100644
> --- a/src/gallium/drivers/galahad/glhd_screen.c
> +++ b/src/gallium/drivers/galahad/glhd_screen.c
> @@ -69,6 +69,15 @@ galahad_screen_get_vendor(struct pipe_screen *_screen)
> return screen->get_vendor(screen);
> }
>
> +static const char *
> +galahad_screen_get_device_vendor(struct pipe_screen *_screen)
> +{
> + struct galahad_screen *glhd_screen = galahad_screen(_screen);
> + struct pipe_screen *screen = glhd_screen->screen;
> +
> + return screen->get_device_vendor(screen);
> +}
> +
> static int
> galahad_screen_get_param(struct pipe_screen *_screen,
> enum pipe_cap param)
> @@ -361,6 +370,7 @@ galahad_screen_create(struct pipe_screen *screen)
> GLHD_SCREEN_INIT(destroy);
> GLHD_SCREEN_INIT(get_name);
> GLHD_SCREEN_INIT(get_vendor);
> + GLHD_SCREEN_INIT(get_device_vendor);
> GLHD_SCREEN_INIT(get_param);
> GLHD_SCREEN_INIT(get_shader_param);
> //GLHD_SCREEN_INIT(get_video_param);
> diff --git a/src/gallium/drivers/i915/i915_screen.c b/src/gallium/drivers/i915/i915_screen.c
> index dc76464..9ab9c48 100644
> --- a/src/gallium/drivers/i915/i915_screen.c
> +++ b/src/gallium/drivers/i915/i915_screen.c
> @@ -55,6 +55,12 @@ i915_get_vendor(struct pipe_screen *screen)
> }
>
> static const char *
> +i915_get_device_vendor(struct pipe_screen *screen)
> +{
> + return "Intel";
> +}
> +
> +static const char *
> i915_get_name(struct pipe_screen *screen)
> {
> static char buffer[128];
> @@ -547,6 +553,7 @@ i915_screen_create(struct i915_winsys *iws)
>
> is->base.get_name = i915_get_name;
> is->base.get_vendor = i915_get_vendor;
> + is->base.get_device_vendor = i915_get_device_vendor;
> is->base.get_param = i915_get_param;
> is->base.get_shader_param = i915_get_shader_param;
> is->base.get_paramf = i915_get_paramf;
> diff --git a/src/gallium/drivers/ilo/ilo_screen.c b/src/gallium/drivers/ilo/ilo_screen.c
> index bf0a84a..80ea4c7 100644
> --- a/src/gallium/drivers/ilo/ilo_screen.c
> +++ b/src/gallium/drivers/ilo/ilo_screen.c
> @@ -515,6 +515,12 @@ ilo_get_vendor(struct pipe_screen *screen)
> }
>
> static const char *
> +ilo_get_device_vendor(struct pipe_screen *screen)
> +{
> + return "Intel";
> +}
> +
> +static const char *
> ilo_get_name(struct pipe_screen *screen)
> {
> struct ilo_screen *is = ilo_screen(screen);
> @@ -844,6 +850,7 @@ ilo_screen_create(struct intel_winsys *ws)
> is->base.destroy = ilo_screen_destroy;
> is->base.get_name = ilo_get_name;
> is->base.get_vendor = ilo_get_vendor;
> + is->base.get_device_vendor = ilo_get_device_vendor;
> is->base.get_param = ilo_get_param;
> is->base.get_paramf = ilo_get_paramf;
> is->base.get_shader_param = ilo_get_shader_param;
> diff --git a/src/gallium/drivers/llvmpipe/lp_screen.c b/src/gallium/drivers/llvmpipe/lp_screen.c
> index 3387d3a..4b45725 100644
> --- a/src/gallium/drivers/llvmpipe/lp_screen.c
> +++ b/src/gallium/drivers/llvmpipe/lp_screen.c
> @@ -589,6 +589,7 @@ llvmpipe_create_screen(struct sw_winsys *winsys)
>
> screen->base.get_name = llvmpipe_get_name;
> screen->base.get_vendor = llvmpipe_get_vendor;
> + screen->base.get_device_vendor = llvmpipe_get_vendor; // TODO should be the CPU vendor
> screen->base.get_param = llvmpipe_get_param;
> screen->base.get_shader_param = llvmpipe_get_shader_param;
> screen->base.get_paramf = llvmpipe_get_paramf;
> diff --git a/src/gallium/drivers/noop/noop_pipe.c b/src/gallium/drivers/noop/noop_pipe.c
> index 8cb8c70..6fb2277 100644
> --- a/src/gallium/drivers/noop/noop_pipe.c
> +++ b/src/gallium/drivers/noop/noop_pipe.c
> @@ -305,6 +305,11 @@ static const char *noop_get_vendor(struct pipe_screen* pscreen)
> return "X.Org";
> }
>
> +static const char *noop_get_device_vendor(struct pipe_screen* pscreen)
> +{
> + return "NONE";
> +}
> +
> static const char *noop_get_name(struct pipe_screen* pscreen)
> {
> return "NOOP";
> @@ -376,6 +381,7 @@ struct pipe_screen *noop_screen_create(struct pipe_screen *oscreen)
> screen->destroy = noop_destroy_screen;
> screen->get_name = noop_get_name;
> screen->get_vendor = noop_get_vendor;
> + screen->get_device_vendor = noop_get_device_vendor;
> screen->get_param = noop_get_param;
> screen->get_shader_param = noop_get_shader_param;
> screen->get_paramf = noop_get_paramf;
> diff --git a/src/gallium/drivers/nouveau/nouveau_screen.c b/src/gallium/drivers/nouveau/nouveau_screen.c
> index 517978d..b4f1413 100644
> --- a/src/gallium/drivers/nouveau/nouveau_screen.c
> +++ b/src/gallium/drivers/nouveau/nouveau_screen.c
> @@ -43,6 +43,12 @@ nouveau_screen_get_vendor(struct pipe_screen *pscreen)
> return "nouveau";
> }
>
> +static const char *
> +nouveau_screen_get_device_vendor(struct pipe_screen *pscreen)
> +{
> + return "NVIDIA";
> +}
> +
> static uint64_t
> nouveau_screen_get_timestamp(struct pipe_screen *pscreen)
> {
> @@ -182,6 +188,7 @@ nouveau_screen_init(struct nouveau_screen *screen, struct nouveau_device *dev)
>
> pscreen->get_name = nouveau_screen_get_name;
> pscreen->get_vendor = nouveau_screen_get_vendor;
> + pscreen->get_device_vendor = nouveau_screen_get_device_vendor;
>
> pscreen->get_timestamp = nouveau_screen_get_timestamp;
>
> diff --git a/src/gallium/drivers/r300/r300_screen.c b/src/gallium/drivers/r300/r300_screen.c
> index fca8001..60630c3 100644
> --- a/src/gallium/drivers/r300/r300_screen.c
> +++ b/src/gallium/drivers/r300/r300_screen.c
> @@ -47,6 +47,11 @@ static const char* r300_get_vendor(struct pipe_screen* pscreen)
> return "X.Org R300 Project";
> }
>
> +static const char* r300_get_device_vendor(struct pipe_screen* pscreen)
> +{
> + return "ATI";
> +}
> +
> static const char* chip_families[] = {
> "unknown",
> "ATI R300",
> @@ -693,6 +698,7 @@ struct pipe_screen* r300_screen_create(struct radeon_winsys *rws)
> r300screen->screen.destroy = r300_destroy_screen;
> r300screen->screen.get_name = r300_get_name;
> r300screen->screen.get_vendor = r300_get_vendor;
> + r300screen->screen.get_device_vendor = r300_get_device_vendor;
> r300screen->screen.get_param = r300_get_param;
> r300screen->screen.get_shader_param = r300_get_shader_param;
> r300screen->screen.get_paramf = r300_get_paramf;
> diff --git a/src/gallium/drivers/radeon/r600_pipe_common.c b/src/gallium/drivers/radeon/r600_pipe_common.c
> index dabe53c..0ef5fc2 100644
> --- a/src/gallium/drivers/radeon/r600_pipe_common.c
> +++ b/src/gallium/drivers/radeon/r600_pipe_common.c
> @@ -332,6 +332,11 @@ static const char* r600_get_vendor(struct pipe_screen* pscreen)
> return "X.Org";
> }
>
> +static const char* r600_get_device_vendor(struct pipe_screen* pscreen)
> +{
> + return "AMD";
> +}
> +
> static const char* r600_get_name(struct pipe_screen* pscreen)
> {
> struct r600_common_screen *rscreen = (struct r600_common_screen*)pscreen;
> @@ -825,6 +830,7 @@ bool r600_common_screen_init(struct r600_common_screen *rscreen,
>
> rscreen->b.get_name = r600_get_name;
> rscreen->b.get_vendor = r600_get_vendor;
> + rscreen->b.get_device_vendor = r600_get_device_vendor;
> rscreen->b.get_compute_param = r600_get_compute_param;
> rscreen->b.get_paramf = r600_get_paramf;
> rscreen->b.get_driver_query_info = r600_get_driver_query_info;
> diff --git a/src/gallium/drivers/rbug/rbug_screen.c b/src/gallium/drivers/rbug/rbug_screen.c
> index 731cc60..d5a3164 100644
> --- a/src/gallium/drivers/rbug/rbug_screen.c
> +++ b/src/gallium/drivers/rbug/rbug_screen.c
> @@ -68,6 +68,15 @@ rbug_screen_get_vendor(struct pipe_screen *_screen)
> return screen->get_vendor(screen);
> }
>
> +static const char *
> +rbug_screen_get_device_vendor(struct pipe_screen *_screen)
> +{
> + struct rbug_screen *rb_screen = rbug_screen(_screen);
> + struct pipe_screen *screen = rb_screen->screen;
> +
> + return screen->get_device_vendor(screen);
> +}
> +
> static int
> rbug_screen_get_param(struct pipe_screen *_screen,
> enum pipe_cap param)
> @@ -267,6 +276,7 @@ rbug_screen_create(struct pipe_screen *screen)
> rb_screen->base.destroy = rbug_screen_destroy;
> rb_screen->base.get_name = rbug_screen_get_name;
> rb_screen->base.get_vendor = rbug_screen_get_vendor;
> + rb_screen->base.get_device_vendor = rbug_screen_get_device_vendor;
> rb_screen->base.get_param = rbug_screen_get_param;
> rb_screen->base.get_shader_param = rbug_screen_get_shader_param;
> rb_screen->base.get_paramf = rbug_screen_get_paramf;
> diff --git a/src/gallium/drivers/softpipe/sp_screen.c b/src/gallium/drivers/softpipe/sp_screen.c
> index a269328..d289e28 100644
> --- a/src/gallium/drivers/softpipe/sp_screen.c
> +++ b/src/gallium/drivers/softpipe/sp_screen.c
> @@ -427,6 +427,7 @@ softpipe_create_screen(struct sw_winsys *winsys)
>
> screen->base.get_name = softpipe_get_name;
> screen->base.get_vendor = softpipe_get_vendor;
> + screen->base.get_device_vendor = softpipe_get_vendor; // TODO should be the CPU vendor
> screen->base.get_param = softpipe_get_param;
> screen->base.get_shader_param = softpipe_get_shader_param;
> screen->base.get_paramf = softpipe_get_paramf;
> diff --git a/src/gallium/drivers/svga/svga_screen.c b/src/gallium/drivers/svga/svga_screen.c
> index bac0dbc..507a1be 100644
> --- a/src/gallium/drivers/svga/svga_screen.c
> +++ b/src/gallium/drivers/svga/svga_screen.c
> @@ -633,6 +633,7 @@ svga_screen_create(struct svga_winsys_screen *sws)
> screen->destroy = svga_destroy_screen;
> screen->get_name = svga_get_name;
> screen->get_vendor = svga_get_vendor;
> + screen->get_device_vendor = svga_get_vendor; // TODO actual device vendor
> screen->get_param = svga_get_param;
> screen->get_shader_param = svga_get_shader_param;
> screen->get_paramf = svga_get_paramf;
> diff --git a/src/gallium/drivers/trace/tr_screen.c b/src/gallium/drivers/trace/tr_screen.c
> index 3a82cc4..266626d 100644
> --- a/src/gallium/drivers/trace/tr_screen.c
> +++ b/src/gallium/drivers/trace/tr_screen.c
> @@ -82,6 +82,27 @@ trace_screen_get_vendor(struct pipe_screen *_screen)
> }
>
>
> +static const char *
> +trace_screen_get_device_vendor(struct pipe_screen *_screen)
> +{
> + struct trace_screen *tr_scr = trace_screen(_screen);
> + struct pipe_screen *screen = tr_scr->screen;
> + const char *result;
> +
> + trace_dump_call_begin("pipe_screen", "get_device_vendor");
> +
> + trace_dump_arg(ptr, screen);
> +
> + result = screen->get_device_vendor(screen);
> +
> + trace_dump_ret(string, result);
> +
> + trace_dump_call_end();
> +
> + return result;
> +}
> +
> +
> static int
> trace_screen_get_param(struct pipe_screen *_screen,
> enum pipe_cap param)
> @@ -470,6 +491,7 @@ trace_screen_create(struct pipe_screen *screen)
> tr_scr->base.destroy = trace_screen_destroy;
> tr_scr->base.get_name = trace_screen_get_name;
> tr_scr->base.get_vendor = trace_screen_get_vendor;
> + tr_scr->base.get_device_vendor = trace_screen_get_device_vendor;
> tr_scr->base.get_param = trace_screen_get_param;
> tr_scr->base.get_shader_param = trace_screen_get_shader_param;
> tr_scr->base.get_paramf = trace_screen_get_paramf;
> diff --git a/src/gallium/drivers/vc4/vc4_screen.c b/src/gallium/drivers/vc4/vc4_screen.c
> index 7c62847..459d179 100644
> --- a/src/gallium/drivers/vc4/vc4_screen.c
> +++ b/src/gallium/drivers/vc4/vc4_screen.c
> @@ -471,6 +471,7 @@ vc4_screen_create(int fd)
>
> pscreen->get_name = vc4_screen_get_name;
> pscreen->get_vendor = vc4_screen_get_vendor;
> + pscreen->get_device_vendor = vc4_screen_get_vendor;
>
> return pscreen;
> }
> --
> 2.1.2.766.gaa23a90
>
> _______________________________________________
> 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