Re: [Mesa-dev] [PATCH 5/5] anv: add missing unlock in error path.

2018-10-10 Thread Jason Ekstrand
On Wed, Oct 10, 2018 at 9:47 AM Lionel Landwerlin < lionel.g.landwer...@intel.com> wrote: > Oh dear... > > Reviewed-by: Lionel Landwerlin > > Eric, Jason : Could it be the wsi CTS test you've seen locking up forever? > I don't think so. It blocks in a different place. In any case, good catch!

Re: [Mesa-dev] [PATCH 5/5] anv: add missing unlock in error path.

2018-10-10 Thread Lionel Landwerlin
Oh dear... Reviewed-by: Lionel Landwerlin Eric, Jason : Could it be the wsi CTS test you've seen locking up forever? On 05/10/2018 01:00, Dave Airlie wrote: From: Dave Airlie Not going to matter, but be consistent. Found by coverity --- src/intel/vulkan/anv_allocator.c | 1 + 1 file ch

[Mesa-dev] [PATCH 5/5] anv: add missing unlock in error path.

2018-10-04 Thread Dave Airlie
From: Dave Airlie Not going to matter, but be consistent. Found by coverity --- src/intel/vulkan/anv_allocator.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/intel/vulkan/anv_allocator.c b/src/intel/vulkan/anv_allocator.c index f62d48ae3fe..67f2f73aa11 100644 --- a/src/intel/vulkan/a