On Wed, 2020-04-15 at 20:36 +0100, Chris Wilson wrote: > Quoting José Roberto de Souza (2020-04-15 20:14:08) > > + i915_reset_error_state(i915); > > If you are bored, we should move this to unregister as that is the > last > point at which it can be accessed from userspace. Hopefully I > remember > next time we are rearranging this sequence.
What if some error happen up to i915_gem_driver_remove()? We would leak a i915_gpu_coredump. Looks to me that i915_reset_error_state() should be called after i915_gem_driver_remove(). > -Chris _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx
