Quoting Chris Wilson (2021-02-06 13:06:05) > Quoting Daniele Ceraolo Spurio (2021-02-06 02:09:21) > > + if (!ret) { > > + ret = wait_for(!pxp->termination_in_progress, 10); > > This only works by chance. The compiler doesn't even have to reload the > variable. See struct completion. It appears we already have a ready made one with the termination i915_request. But that will require RCU pointer management... -Chris _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx