<div dir="ltr"><div><div>Thanks Neil,<br></div>I'll make that change.  Should be simple enough.<br></div>--Jason<br></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Aug 4, 2014 at 10:51 AM, Neil Roberts <span dir="ltr"><<a href="mailto:neil@linux.intel.com" target="_blank">neil@linux.intel.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">One case where the memcpy path won't work is if the source and<br>
destination are the same buffer. In that case it tries to map the<br>
texture twice and hits an assert. I think this could be a legitimate use<br>
case if the source and destination rectangles don't overlap. The<br>
extension spec isn't very clear on whether this is allowed but the spec<br>
for GL 4.4 actually mentions blitting between a single buffer and<br>
implies that it's ok if the rectangles don't overlap. Perhaps one way to<br>
solve this would be to check if src_mt==dst_mt and if so it could map<br>
the union of both rectangles as read and write. It would then have to<br>
manually offset the source and destination pointers to the right place.<br>
<br>
You might also want to update the commit message as the patch is no<br>
longer restricted to a pitch of at most INT16_MAX.<br>
<br>
Regards,<br>
- Neil<br>
<div class="im HOEnZb"><br>
Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>> writes:<br>
<br>
> This, together with the meta path, provides a almost-complete implemetation<br>
> of ARB_copy_image.  The only case that won't work is if one of the textures<br>
> is compressed and has a pitch greater than INT16_MAX.  I think that's good<br>
> enough to justify turning on the extension.<br>
><br>
> v2: Add a fallback memcpy path for when the texture is too big for the<br>
>     blitter<br>
><br>
> Signed-off-by: Jason Ekstrand <<a href="mailto:jason.ekstrand@intel.com">jason.ekstrand@intel.com</a>><br>
</div><div class="HOEnZb"><div class="h5">_______________________________________________<br>
mesa-dev mailing list<br>
<a href="mailto:mesa-dev@lists.freedesktop.org">mesa-dev@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/mesa-dev" target="_blank">http://lists.freedesktop.org/mailman/listinfo/mesa-dev</a><br>
</div></div></blockquote></div><br></div>