On 3/25/2020 11:03, Daniele Ceraolo Spurio wrote:
On 3/25/20 10:58 AM, John Harrison wrote:
On 3/11/2020 18:16, Daniele Ceraolo Spurio wrote:
we do call uc_fini if there is an issue while loading the GuC, so we
can't delete in there the logs we need to debug the load failure.
Moving the log free to driver remove ensures the logs stick around ong
enough for us to dump them.
I think this could be worded better and has a couple of typos.

How about:

"We need to keep the GuC error logs around to debug the load failure, so we can't clean them in the error unwind, which includes uc_fini(). Moving the cleanup to driver remove ensures that the logs stick around long enough for us to dump them."



Sounds good to me :).

Reviewed-by: John Harrison <john.c.harri...@intel.com>

Intel-gfx mailing list

Reply via email to