Op 05-11-2019 om 12:04 schreef Daniel Vetter: > On Mon, Nov 04, 2019 at 06:38:00PM +0100, Daniel Vetter wrote: >> We can't copy_*_user while holding reservations, that will (soon even >> for nouveau) lead to deadlocks. And it breaks the cross-driver >> contract around dma_resv. >> >> Fix this by adding a slowpath for when we need relocations, and by >> pushing the writeback of the new presumed offsets to the very end. >> >> Aside from "it compiles" entirely untested unfortunately. >> >> Signed-off-by: Daniel Vetter <[email protected]> >> Cc: Ilia Mirkin <[email protected]> >> Cc: Maarten Lankhorst <[email protected]> >> Cc: Ben Skeggs <[email protected]> >> Cc: [email protected] > Ping for ack/review so I can land this entire series. intel-gfx-ci is all > happy with the rebased version, so nouveau ack is the only hold-up here.
I don't feel confident reviewing this as I lack the hw, so all review caveats reply. Having said that, this looks sane, and if it blows up we'll found out eventually. :) Reviewed-by: Maarten Lankhorst <[email protected]> _______________________________________________ Intel-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/intel-gfx
