<div dir="ltr"><div dir="ltr"><div>Seems extending of any PIN_XXX flags to 64b has no sense for all other cases.</div><div>But also in the code there is mismatching of 64 and 32b flags/data usage (32b flags may be put to 64b arguments).</div><div>Seems that shouldn't cause any issues anymore, but if you think that it should be fixed - please, let me know - would like to complete that case.</div></div></div><br><div class="gmail_quote"><div dir="ltr">On Fri, Nov 2, 2018 at 5:47 PM Sergii Romantsov <<a href="mailto:sergii.romantsov@gmail.com">sergii.romantsov@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Operating with gtt-addresses has to be done with 64b variables.<br>
<br>
CC: Kenneth Graunke <<a href="mailto:kenneth@whitecape.org" target="_blank">kenneth@whitecape.org</a>><br>
CC: Chris Wilson <<a href="mailto:chris@chris-wilson.co.uk" target="_blank">chris@chris-wilson.co.uk</a>><br>
Fixes: a363bb2cd0e2 (i965: Allocate VMA in userspace for full-PPGTT systems.)<br>
Fixes: c7c6e46f913b (drm/i915: Convert execbuf to use struct-of-array packing for critical fields)<br>
Bugzilla: <a href="https://bugs.freedesktop.org/show_bug.cgi?id=108619" rel="noreferrer" target="_blank">https://bugs.freedesktop.org/show_bug.cgi?id=108619</a><br>
Signed-off-by: Sergii Romantsov <<a href="mailto:sergii.romantsov@globallogic.com" target="_blank">sergii.romantsov@globallogic.com</a>><br>
---<br>
 drivers/gpu/drm/i915/i915_gem_execbuffer.c | 2 +-<br>
 1 file changed, 1 insertion(+), 1 deletion(-)<br>
<br>
diff --git a/drivers/gpu/drm/i915/i915_gem_execbuffer.c b/drivers/gpu/drm/i915/i915_gem_execbuffer.c<br>
index 3f0c612..1973b79 100644<br>
--- a/drivers/gpu/drm/i915/i915_gem_execbuffer.c<br>
+++ b/drivers/gpu/drm/i915/i915_gem_execbuffer.c<br>
@@ -611,7 +611,7 @@ static int eb_reserve_vma(const struct i915_execbuffer *eb,<br>
<br>
        if (exec_flags & EXEC_OBJECT_PINNED) {<br>
                pin_flags |= entry->offset | PIN_OFFSET_FIXED;<br>
-               pin_flags &= ~PIN_NONBLOCK; /* force overlapping checks */<br>
+               pin_flags &= ~(u64)PIN_NONBLOCK; /* force overlapping checks */<br>
        } else if (exec_flags & __EXEC_OBJECT_NEEDS_BIAS) {<br>
                pin_flags |= BATCH_OFFSET_BIAS | PIN_OFFSET_BIAS;<br>
        }<br>
-- <br>
2.7.4<br>
<br>
_______________________________________________<br>
Intel-gfx mailing list<br>
<a href="mailto:Intel-gfx@lists.freedesktop.org" target="_blank">Intel-gfx@lists.freedesktop.org</a><br>
<a href="https://lists.freedesktop.org/mailman/listinfo/intel-gfx" rel="noreferrer" target="_blank">https://lists.freedesktop.org/mailman/listinfo/intel-gfx</a><br>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div dir="ltr">Sergii Romantsov<br></div><div dir="ltr"><div style="margin:0px;padding:0px;color:rgb(23,43,77);font-family:-apple-system,BlinkMacSystemFont,"Segoe UI",Roboto,Oxygen,Ubuntu,"Fira Sans","Droid Sans","Helvetica Neue",sans-serif;font-size:14px">GlobalLogic Inc.<br><a href="http://www.globallogic.com/" rel="nofollow" style="color:rgb(0,82,204)" target="_blank">www.globallogic.com</a><br></div></div></div></div></div></div>