[Mesa-dev] [PATCH v2 1/4] softpipe: Add assert verifying successful pipe_transfer_map

Robert Foss robert.foss at collabora.com
Wed Jul 18 09:01:10 UTC 2018


Thanks Brian!

On 2018-07-17 16:18, Brian Paul wrote:
> On 07/17/2018 04:32 AM, Robert Foss wrote:
>> This failure mode is a bit tricky to debug and manifests itself
>> later as a null pointer dereference, for which finding the origin
>> is needlessly tricky.
>>
>> Signed-off-by: Robert Foss <robert.foss at collabora.com>
>> ---
>>
>> Changes since v1:
>>    - Patch added
>>
>>   src/gallium/drivers/softpipe/sp_tile_cache.c | 1 +
>>   1 file changed, 1 insertion(+)
>>
>> diff --git a/src/gallium/drivers/softpipe/sp_tile_cache.c 
>> b/src/gallium/drivers/softpipe/sp_tile_cache.c
>> index 351736ee421..211cc5ef4b0 100644
>> --- a/src/gallium/drivers/softpipe/sp_tile_cache.c
>> +++ b/src/gallium/drivers/softpipe/sp_tile_cache.c
>> @@ -209,6 +209,7 @@ sp_tile_cache_set_surface(struct softpipe_tile_cache *tc,
>>                                                       
>> PIPE_TRANSFER_UNSYNCHRONIZED,
>>                                                       0, 0, ps->width, 
>> ps->height,
>>                                                       &tc->transfer[i]);
>> +            assert(tc->transfer_map[i]);
>>            }
>>         }
>>         else {
>>
> 
> Looks OK to me.
> 
> Reviewed-by: Brian Paul <brianp at vmware.com>


More information about the mesa-dev mailing list