[Mesa-dev] [PATCH] meta: Abort texture upload if pixels == null and no pixel unpack buffer set

[Anuj Phogat]

On Fri, Jun 12, 2015 at 5:12 AM, Neil Roberts <neil at linux.intel.com> wrote:
> Ah, good catch. Looks good to me.
>
> Reviewed-by: Neil Roberts <neil at linux.intel.com>
>
> It seems a bit weird to use create_pbo=true at all for
> glTexImage{1,2,3}D because in that case we are completely replacing the
> texture. If the texture's buffer is busy instead of allocating a PBO we
> might as well just directly allocate some new storage for the texture
> and abandon the old storage. That would be a separate patch though and
> either way I think this patch makes sense.
>
> Anuj Phogat <anuj.phogat at gmail.com> writes:
>
>> in case of glTex{Sub}Image{1,2,3}D().
>
> I think in practice this patch only applies to glTexImage* (not
> glTexSubImage*) doesn't it? It wouldn't make any sense to call
> glTexSubImage without a PBO and with NULL pixels.
>
Right.
> Regards,
> - Neil