Re: [Mesa-dev] [PATCH] meta: Clip src/dest rects in BlitFramebuffer, using the scissor

2014-04-16 Thread Anuj Phogat
On Tue, Apr 15, 2014 at 7:15 PM, Chris Forbes wrote: > Fixes piglit's fbo-blit-stretch test on drivers which use the meta path. > (i965: should fix Broadwell, but also fixes Sandybridge/Ivybridge/Haswell > since this test falls off the blorp path now due to format conversion) > > V2: Use scissor i

[Mesa-dev] [PATCH] meta: Clip src/dest rects in BlitFramebuffer, using the scissor

2014-04-15 Thread Chris Forbes
Fixes piglit's fbo-blit-stretch test on drivers which use the meta path. (i965: should fix Broadwell, but also fixes Sandybridge/Ivybridge/Haswell since this test falls off the blorp path now due to format conversion) V2: Use scissor instead of just mangling the rects, to avoid texcoord rounding p

Re: [Mesa-dev] [PATCH] meta: Clip src/dest rects in BlitFramebuffer.

2014-04-14 Thread Chris Forbes
Right, that makes good sense. On Mon, Apr 14, 2014 at 11:28 PM, Marek Olšák wrote: > For scaled blits, this is not correct. For example, if you blit a > 10x10 texture to another 10x10 texture with horizontal scaling of > 1.5x, the X texture coordinate of the rightmost pixel should be > 6.66.

Re: [Mesa-dev] [PATCH] meta: Clip src/dest rects in BlitFramebuffer.

2014-04-14 Thread Marek Olšák
For scaled blits, this is not correct. For example, if you blit a 10x10 texture to another 10x10 texture with horizontal scaling of 1.5x, the X texture coordinate of the rightmost pixel should be 6.66. Mesa clip blit doesn't return floating-point coordinates of the source, so you can't use them

[Mesa-dev] [PATCH] meta: Clip src/dest rects in BlitFramebuffer.

2014-04-14 Thread Chris Forbes
Nothing was bothering to clip the blit. If the src rect was partially outside the framebuffer, we'd end up picking up more copies of the edge texels due to clamping. Note that this is slight overkill -- we could get away with clipping src only, since fragments outside the destination surface will