[Mesa-dev] [PATCH] st/dri: always initialize 'opencl_func_mutex' in struct dri_screen

2017-07-06 Thread Aleksander Morgado
The 'opencl_func_mutex' is destroyed in dri_destroy_screen_helper() unconditionally, so make sure it is always initialized to avoid any undefined behaviour. Signed-off-by: Aleksander Morgado <aleksan...@aleksander.es> --- src/gallium/state_trackers/dri/dri2.c | 1 + src/gallium/state_tr

[Mesa-dev] [PATCH] etnaviv: fix refcnt initialization in etna_screen

2017-07-06 Thread Aleksander Morgado
when screen created. This initialization is done in etna_screen_create(), to follow the same logic as in freedreno and virgl. Signed-off-by: Aleksander Morgado <aleksan...@aleksander.es> --- src/gallium/drivers/etnaviv/etnaviv_screen.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/g

[Mesa-dev] [PATCH] etnaviv: don't dereference etna_resource pointer if allocation fails

2017-07-06 Thread Aleksander Morgado
The check for the pointer being non-NULL was being done too late. Signed-off-by: Aleksander Morgado <aleksan...@aleksander.es> --- src/gallium/drivers/etnaviv/etnaviv_resource.c | 10 +++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/src/gallium/drivers/e

Re: [Mesa-dev] [PATCH] etnaviv: fix refcnt initialization in etna_screen

2017-07-06 Thread Aleksander Morgado
On Thu, Jul 6, 2017 at 11:18 PM, Aleksander Morgado <aleksan...@aleksander.es> wrote: > Despite being a member of the etna_screen struct, 'refcnt' is used by > the winsys-specific logic to track the reference count of the object > managed in a hash table. When the count reaches