On Mon, Oct 03, 2016 at 10:18:03AM +0300, Joonas Lahtinen wrote: > On pe, 2016-09-30 at 18:31 +0100, Chris Wilson wrote: > > We use obj->phys_handle to choose the pread/pwrite path, but as > > obj->phys_handle is a union with obj->userptr, we then mistakenly use > > the phys_handle path for userptr objects within pread/pwrite. > > > > Testcase: igt/gem_userptr_blits/forbidden-operations > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=97519 > > Signed-off-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> > > Cc: stable@xxxxxxxxxxxxxxx > > Reviewed-by: Joonas Lahtinen <joonas.lahtinen@xxxxxxxxxxxxxxx> Commmented already on irc, but I totally panicked about this one. I think we need an ops->pwrite. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch -- To unsubscribe from this list: send the line "unsubscribe stable" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html