On Fri, Mar 1, 2019 at 2:35 AM Peter Zijlstra <pet...@infradead.org> wrote: > > With the dodgy patch: > > CC drivers/gpu/drm/i915/i915_gem_execbuffer.o > drivers/gpu/drm/i915/i915_gem_execbuffer.o: warning: objtool: > eb_relocate_slow()+0x1f9: call to kvfree() with UACCESS enabled > drivers/gpu/drm/i915/i915_gem_execbuffer.o: warning: objtool: > i915_gem_execbuffer2_ioctl()+0x315: call to kvfree() with UACCESS enabled
Ok, so it looks like your patch does the right thing. Your fixups to i915_gem_execbuffer.c (and the already commented-upon copy_user_tail thing) also look sane. Linus