drm/nouveau/tmr: fully separate alarm execution/pending lists

Browse source

commit b4e382ca75 upstream.

Reusing the list_head for both is a bad idea.  Callback execution is done
with the lock dropped so that alarms can be rescheduled from the callback,
which means that with some unfortunate timing, lists can get corrupted.

The execution list should not require its own locking, the single function
that uses it can only be called from a single context.

Signed-off-by: Ben Skeggs <bskeggs@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

...

This commit is contained in:

Ben Skeggs 2017-06-05 17:23:32 +10:00 • committed by Greg Kroah-Hartman

parent 7860d0e5e2

commit aae14f569f

2 changed files with 5 additions and 3 deletions

Show all changes Ignore whitespace when comparing lines Ignore changes in amount of whitespace Ignore changes in whitespace at EOL

Download patch file Download diff file Expand all files Collapse all files

1

drivers/gpu/drm/nouveau/include/nvkm/subdev/timer.h

View file

7

drivers/gpu/drm/nouveau/nvkm/subdev/timer/base.c

View file