[prev in list] [next in list] [prev in thread] [next in thread]
List: freedesktop-intel-gfx
Subject: Re: [Intel-gfx] [PATCH v2 1/3] drm/i915/guc: Allow preempt-client to be NULL
From: Mika Kuoppala <mika.kuoppala () linux ! intel ! com>
Date: 2018-01-31 12:42:39
Message-ID: 87k1vyw668.fsf () gaia ! fi ! intel ! com
[Download RAW message or body]
Chris Wilson <chris@chris-wilson.co.uk> writes:
> Quoting Mika Kuoppala (2018-01-31 11:31:09)
>> Chris Wilson <chris@chris-wilson.co.uk> 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@chris-wilson.co.uk>
>> > Cc: Michal Winiarski <michal.winiarski@intel.com>
>> > Cc: Arkadiusz Hiler <arkadiusz.hiler@intel.com>
>> > Cc: Daniele Ceraolo Spurio <daniele.ceraolospurio@intel.com>
>> > Cc: Michel Thierry <michel.thierry@intel.com>
>> > Cc: Michal Wajdeczko <michal.wajdeczko@intel.com>
>> > ---
>> > 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
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic