<div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Fri, Dec 7, 2018 at 6:06 PM Rafael Antognolli <<a href="mailto:rafael.antognolli@intel.com">rafael.antognolli@intel.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">They won't be true anymore once we add support for multiple BOs with<br>
non-userptr.<br></blockquote><div><br></div><div>Why not?  I guess in the highly restricted case of state streams, you can safely copy out-of-range of one BO because you know exactly what the next BO is.  Sounds dangerous but totally legit.</div><div><br></div><div>Reviewed-by: Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
---<br>
 src/intel/vulkan/genX_gpu_memcpy.c | 3 ---<br>
 1 file changed, 3 deletions(-)<br>
<br>
diff --git a/src/intel/vulkan/genX_gpu_memcpy.c b/src/intel/vulkan/genX_gpu_memcpy.c<br>
index 1bee1c6dc17..e20179fa675 100644<br>
--- a/src/intel/vulkan/genX_gpu_memcpy.c<br>
+++ b/src/intel/vulkan/genX_gpu_memcpy.c<br>
@@ -133,9 +133,6 @@ genX(cmd_buffer_so_memcpy)(struct anv_cmd_buffer *cmd_buffer,<br>
    if (size == 0)<br>
       return;<br>
<br>
-   assert(dst.offset + size <= dst.bo->size);<br>
-   assert(src.offset + size <= src.bo->size);<br>
-<br>
    /* The maximum copy block size is 4 32-bit components at a time. */<br>
    assert(size % 4 == 0);<br>
    unsigned bs = gcd_pow2_u64(16, size);<br>
-- <br>
2.17.1<br>
<br>
_______________________________________________<br>
mesa-dev mailing list<br>
<a href="mailto:mesa-dev@lists.freedesktop.org" target="_blank">mesa-dev@lists.freedesktop.org</a><br>
<a href="https://lists.freedesktop.org/mailman/listinfo/mesa-dev" rel="noreferrer" target="_blank">https://lists.freedesktop.org/mailman/listinfo/mesa-dev</a><br>
</blockquote></div></div>