[Mesa-dev] [PATCH] mesa: set the texture target in VDPAUMapSurfacesNV (v2)

Marek Olšák maraeo at gmail.com
Tue Oct 27 13:33:04 PDT 2015


Thanks, and sorry, I forgot to update the commit message. I will
commit it shortly with that fixed.

Marek

On Tue, Oct 27, 2015 at 9:19 PM, Brian Paul <brianp at vmware.com> wrote:
> On 10/27/2015 02:17 PM, Marek Olšák wrote:
>>
>> From: Marek Olšák <marek.olsak at amd.com>
>>
>> We initialized Target, but not TargetIndex.
>> This is required since 7d7dd1871174905dfdd3ca874a09d9.
>>
>> v2: do it in the right place. Noticed by Brian Paul.
>>
>> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=92645
>> ---
>>   src/mesa/main/vdpau.c | 5 +++--
>>   1 file changed, 3 insertions(+), 2 deletions(-)
>>
>> diff --git a/src/mesa/main/vdpau.c b/src/mesa/main/vdpau.c
>> index 0efa56e..44be3a3 100644
>> --- a/src/mesa/main/vdpau.c
>> +++ b/src/mesa/main/vdpau.c
>> @@ -163,9 +163,10 @@ register_surface(struct gl_context *ctx, GLboolean
>> isOutput,
>>            return (GLintptr)NULL;
>>         }
>>
>> -      if (tex->Target == 0)
>> +      if (tex->Target == 0) {
>>            tex->Target = target;
>> -      else if (tex->Target != target) {
>> +         tex->TargetIndex = _mesa_tex_target_to_index(ctx, target);
>> +      } else if (tex->Target != target) {
>>            _mesa_unlock_texture(ctx, tex);
>>            free(surf);
>>            _mesa_error(ctx, GL_INVALID_OPERATION,
>>
>
> That looks good.
>
> Reviewed-by: Brian Paul <brianp at vmware.com>
>


More information about the mesa-dev mailing list