diff options
| author |   Maarten Lankhorst <maarten.lankhorst@linux.intel.com> | 2017-06-21 13:00:07 +0200 |
|---|---|---|
| committer | Maarten Lankhorst <maarten.lankhorst@linux.intel.com> | 2017-07-17 16:32:43 +0200 |
| commit | eb42ea6d0b8ed9ca8e73cc24fa801f0d8ab28905 (patch) | |
| tree | 7ce017ebaa327a35fc70f34ae38122bd74fc7658 /tools/perf/scripts/python/export-to-postgresql.py | |
| parent | drm/vc4: Fix misleading name of the continuous flag. (diff) | |
| download | linux-dev-eb42ea6d0b8ed9ca8e73cc24fa801f0d8ab28905.tar.xz linux-dev-eb42ea6d0b8ed9ca8e73cc24fa801f0d8ab28905.zip | |
drm/crc: Handle opening and closing crc better
When I was doing a grep . -r /sys/kernel/debug/dri/0 I noticed a WARN
appearing when I aborted the grep with ^C.
After investigating I've also noticed that the error handling was
lacking and there are race conditions involving multiple calls to
open/close simultaneously.
Fix this by setting the opened flag first and using crc->entries to
decide when crc can be collected.
Also call unset crc source before cleaning up, this way there is
no race with a future open().
This patch has been tested with all the tests in igt with CRC in their
name.
Signed-off-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Link: http://patchwork.freedesktop.org/patch/msgid/20170621110007.11674-1-maarten.lankhorst@linux.intel.com
Reviewed-by: Tomeu Vizoso <tomeu.vizoso@collabora.com>
[mlankhorst: Add description that this patch has been tested with IGT,
based on tomeu's feedback]
Diffstat (limited to 'tools/perf/scripts/python/export-to-postgresql.py')
0 files changed, 0 insertions, 0 deletions