Chris Wilson <[email protected]> writes:

> Quoting Mika Kuoppala (2018-01-31 11:31:09)
>> Chris Wilson <[email protected]> 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 <[email protected]>
>> > Cc: Michal Winiarski <[email protected]>
>> > Cc: Arkadiusz Hiler <[email protected]>
>> > Cc: Daniele Ceraolo Spurio <[email protected]>
>> > Cc: Michel Thierry <[email protected]>
>> > Cc: Michal Wajdeczko <[email protected]>
>> > ---
>> >  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.

Not with this patch alone but the commit message promises
that we prepare for it :)

-Mika

_______________________________________________
Intel-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to