Re: [Mesa-dev] Intel TexImage cleanup
On 09/22/2011 12:34 PM, Eric Anholt wrote: > Only 75 lines of code deletion to offer here. > > My goal was to delete the whole CPU upload part of intelTexImage by > moving the mt setup to AllocTextureImageBuffer and using the > MapTextureImage-based Mesa core. However, because we don't have all > of core converted to MapTextureImage, there are places that call > AllocTextureImageBuffer then immediately expect ->Data to be present, > which we wouldn't be able to provide. This looks good to me, but I'm not at all well versed in this. 1, 2, 6, 7 are Reviewed-by: Kenneth Graunke (though, in 7 your commit message says "consumers consumers") the rest are Acked-by: Kenneth Graunke ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
Re: [Mesa-dev] Intel TexImage cleanup
On Thu, 22 Sep 2011 14:37:24 -0600, Brian Paul wrote: > On 09/22/2011 01:34 PM, Eric Anholt wrote: > > Only 75 lines of code deletion to offer here. > > > > My goal was to delete the whole CPU upload part of intelTexImage by > > moving the mt setup to AllocTextureImageBuffer and using the > > MapTextureImage-based Mesa core. However, because we don't have all > > of core converted to MapTextureImage, there are places that call > > AllocTextureImageBuffer then immediately expect ->Data to be present, > > which we wouldn't be able to provide. > > You're talking about in mipmap.c, for example, right? After the > AllocTextureImageBuffer() call we should be calling > ctx->Driver.MapTextureImage() to map the src/dst mipmap levels. > > The core/fallback GetTexImage() code also needs to call > ctx->Driver.Map/UnmapTextureImage(). And swrast. It's on my to-do > list but I might not get to it for a while. > > Eventually, there should be no use of gl_texture_image->Data anywhere > in core Mesa or swrast. It should really be a driver-private concept. Yeah, I was honestly surprised because I thought we'd landed all of those changes already :) pgpvcLSSiATT9.pgp Description: PGP signature ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
Re: [Mesa-dev] Intel TexImage cleanup
On 09/22/2011 01:34 PM, Eric Anholt wrote: Only 75 lines of code deletion to offer here. My goal was to delete the whole CPU upload part of intelTexImage by moving the mt setup to AllocTextureImageBuffer and using the MapTextureImage-based Mesa core. However, because we don't have all of core converted to MapTextureImage, there are places that call AllocTextureImageBuffer then immediately expect ->Data to be present, which we wouldn't be able to provide. You're talking about in mipmap.c, for example, right? After the AllocTextureImageBuffer() call we should be calling ctx->Driver.MapTextureImage() to map the src/dst mipmap levels. The core/fallback GetTexImage() code also needs to call ctx->Driver.Map/UnmapTextureImage(). And swrast. It's on my to-do list but I might not get to it for a while. Eventually, there should be no use of gl_texture_image->Data anywhere in core Mesa or swrast. It should really be a driver-private concept. -Brian ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev