Quoting Mika Kuoppala (2018-01-31 11:31:09) > Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> writes: > > > In the next patch, we may only conditionally allocate the preempt-client > > if there is a global preempt context and so we need to be prepared in > > case the preempt-client itself is NULL. > > > > Signed-off-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> > > Cc: Michal Winiarski <michal.winiarski@xxxxxxxxx> > > Cc: Arkadiusz Hiler <arkadiusz.hiler@xxxxxxxxx> > > Cc: Daniele Ceraolo Spurio <daniele.ceraolospurio@xxxxxxxxx> > > Cc: Michel Thierry <michel.thierry@xxxxxxxxx> > > Cc: Michal Wajdeczko <michal.wajdeczko@xxxxxxxxx> > > --- > > drivers/gpu/drm/i915/intel_guc_submission.c | 27 +++++++++++++++++---------- > > 1 file changed, 17 insertions(+), 10 deletions(-) > > > > You need to prepare the debugfs to handle the null also. > > Does our simple igt/debugfs test include reading through all the nodes > it finds? It does, but we don't actually end up with NULLs. -Chris _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
