linux-kernel - Re: [PATCH] drm/sched: Always trace the dependencies we wait on, to fix a race.

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <87tvhf2o0l.fsf@anholt.net>
Date:   Thu, 07 Feb 2019 15:10:34 -0800
From:   Eric Anholt <eric@...olt.net>
To:     "Koenig\, Christian" <Christian.Koenig@....com>,
        "dri-devel\@lists.freedesktop.org" <dri-devel@...ts.freedesktop.org>,
        "amd-gfx\@lists.freedesktop.org" <amd-gfx@...ts.freedesktop.org>
Cc:     "linux-kernel\@vger.kernel.org" <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] drm/sched: Always trace the dependencies we wait on, to fix a race.

"Koenig, Christian" <Christian.Koenig@....com> writes:

> Am 07.12.18 um 20:16 schrieb Eric Anholt:
>> The entity->dependency can go away completely once we've called
>> drm_sched_entity_add_dependency_cb() (if the cb is called before we
>> get around to tracing).  The tracepoint is more useful if we trace
>> every dependency instead of just ones that get callbacks installed,
>> anyway, so just do that.
>>
>> Fixes any easy-to-produce OOPS when tracing the scheduler on V3D with
>> "perf record -a -e gpu_scheduler:.\* glxgears" and DEBUG_SLAB enabled.
>>
>> Signed-off-by: Eric Anholt <eric@...olt.net>
>
> Reviewed-by: Christian König <christian.koenig@....com>
>
> Going to pick that up for upstream and will add with a CC: stable.

Looks like this got misplaced.

Download attachment "signature.asc" of type "application/pgp-signature" (833 bytes)