On Tue, Dec 17, 2019 at 5:02 AM Mark Kettenis <[email protected]> wrote:
> Hit the > > KASSERT(curcpu()->ci_inatomic == 0); > > in pagefault_disable(). Analysis of the code in i915_gem_execbuffer.c > shows that pagefault_disable() may be called when page faults are > already disabled, i.e. from eb_relocate_slow() where eb_relocate_vma() > is called after pagefault_disable(). > > Diff below fixes this. > > ok? > Like a making a lock recursive, it suggests confusion in the code...but if upstream supports it then doing otherwise is a losing proposition. ok guenther@
