diff options
| author | Christian Koenig <christian.koenig@amd.com> | 2012-05-17 01:33:30 +0200 |
|---|---|---|
| committer | Christian König <deathsimple@vodafone.de> | 2012-06-21 09:38:50 +0200 |
| commit | fb98257a9d9d2089972b18079d5bdd4412e107e2 (patch) | |
| tree | 72f6115ccae023c1e6122bdd82cf310c8014de23 /security | |
| parent | c20dc3698dc7ecf053e2bf77299ae5982c0c2c45 (diff) | |
| download | blackbird-op-linux-fb98257a9d9d2089972b18079d5bdd4412e107e2.tar.gz blackbird-op-linux-fb98257a9d9d2089972b18079d5bdd4412e107e2.zip | |
drm/radeon: apply Murphy's law to the kms irq code v3
1. It is really dangerous to have more than one
spinlock protecting the same information.
2. radeon_irq_set sometimes wasn't called with lock
protection, so it can happen that more than one
CPU would tamper with the irq regs at the same
time.
3. The pm.gui_idle variable was assuming that the 3D
engine wasn't becoming idle between testing the
register and setting the variable. So just remove
it and test the register directly.
v2: Also handle the hpd irq code the same way.
v3: Rename hpd parameter for clarification.
Signed-off-by: Christian Koenig <christian.koenig@amd.com>
Reviewed-by: Alex Deucher <alexander.deucher@amd.com>
Diffstat (limited to 'security')
0 files changed, 0 insertions, 0 deletions
