[PATCH 07/10] drm/radeon: apply Murphy's law to the kms irq code v3

2012-06-01 Thread Christian König
From: Christian Koenig 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

[PATCH 07/10] drm/radeon: apply Murphy's law to the kms irq code v3

2012-06-01 Thread Christian König
From: Christian Koenig christian.koe...@amd.com 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

[PATCH 07/10] drm/radeon: apply Murphy's law to the kms irq code v2

2012-05-31 Thread Christian König
From: Christian Koenig 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

[PATCH 07/10] drm/radeon: apply Murphy's law to the kms irq code

2012-05-31 Thread Christian König
On 31.05.2012 20:15, Alex Deucher wrote: > On Thu, May 24, 2012 at 11:35 AM, Alex Deucher > wrote: >> On Thu, May 24, 2012 at 3:49 AM, Christian K?nig >> wrote: >>> From: Christian Koenig >>> >>> 1. It is really dangerous to have more than one >>>spinlock protecting the same information.

[PATCH 07/10] drm/radeon: apply Murphy's law to the kms irq code v2

2012-05-31 Thread Alex Deucher
On Thu, May 31, 2012 at 4:16 PM, Christian K?nig wrote: > From: Christian Koenig > > 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

[PATCH 07/10] drm/radeon: apply Murphy's law to the kms irq code

2012-05-31 Thread Jerome Glisse
On Thu, May 31, 2012 at 2:15 PM, Alex Deucher wrote: > On Thu, May 24, 2012 at 11:35 AM, Alex Deucher > wrote: >> On Thu, May 24, 2012 at 3:49 AM, Christian K?nig >> wrote: >>> From: Christian Koenig >>> >>> 1. It is really dangerous to have more than one >>> ? spinlock protecting the same

[PATCH 07/10] drm/radeon: apply Murphy's law to the kms irq code

2012-05-31 Thread Alex Deucher
On Thu, May 24, 2012 at 11:35 AM, Alex Deucher wrote: > On Thu, May 24, 2012 at 3:49 AM, Christian K?nig > wrote: >> From: Christian Koenig >> >> 1. It is really dangerous to have more than one >> ? spinlock protecting the same information. >> >> 2. radeon_irq_set sometimes wasn't called with

Re: [PATCH 07/10] drm/radeon: apply Murphy's law to the kms irq code

2012-05-31 Thread Alex Deucher
On Thu, May 24, 2012 at 11:35 AM, Alex Deucher alexdeuc...@gmail.com wrote: On Thu, May 24, 2012 at 3:49 AM, Christian König deathsim...@vodafone.de wrote: From: Christian Koenig christian.koe...@amd.com 1. It is really dangerous to have more than one   spinlock protecting the same

Re: [PATCH 07/10] drm/radeon: apply Murphy's law to the kms irq code

2012-05-31 Thread Jerome Glisse
On Thu, May 31, 2012 at 2:15 PM, Alex Deucher alexdeuc...@gmail.com wrote: On Thu, May 24, 2012 at 11:35 AM, Alex Deucher alexdeuc...@gmail.com wrote: On Thu, May 24, 2012 at 3:49 AM, Christian König deathsim...@vodafone.de wrote: From: Christian Koenig christian.koe...@amd.com 1. It is

Re: [PATCH 07/10] drm/radeon: apply Murphy's law to the kms irq code

2012-05-31 Thread Christian König
On 31.05.2012 20:15, Alex Deucher wrote: On Thu, May 24, 2012 at 11:35 AM, Alex Deucheralexdeuc...@gmail.com wrote: On Thu, May 24, 2012 at 3:49 AM, Christian König deathsim...@vodafone.de wrote: From: Christian Koenigchristian.koe...@amd.com 1. It is really dangerous to have more than one

[PATCH 07/10] drm/radeon: apply Murphy's law to the kms irq code v2

2012-05-31 Thread Christian König
From: Christian Koenig christian.koe...@amd.com 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

[PATCH 07/10] drm/radeon: apply Murphy's law to the kms irq code

2012-05-24 Thread j.glisse
On Thu, May 24, 2012 at 11:35:15AM -0400, Alex Deucher wrote: > On Thu, May 24, 2012 at 3:49 AM, Christian K?nig > wrote: > > From: Christian Koenig > > > > 1. It is really dangerous to have more than one > > ? spinlock protecting the same information. > > > > 2. radeon_irq_set sometimes wasn't

[PATCH 07/10] drm/radeon: apply Murphy's law to the kms irq code

2012-05-24 Thread Alex Deucher
On Thu, May 24, 2012 at 3:49 AM, Christian K?nig wrote: > From: Christian Koenig > > 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

[PATCH 07/10] drm/radeon: apply Murphy's law to the kms irq code

2012-05-24 Thread Christian König
From: Christian Koenig 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

[PATCH 07/10] drm/radeon: apply Murphy's law to the kms irq code

2012-05-24 Thread Christian König
From: Christian Koenig christian.koe...@amd.com 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

Re: [PATCH 07/10] drm/radeon: apply Murphy's law to the kms irq code

2012-05-24 Thread Alex Deucher
On Thu, May 24, 2012 at 3:49 AM, Christian König deathsim...@vodafone.de wrote: From: Christian Koenig christian.koe...@amd.com 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

Re: [PATCH 07/10] drm/radeon: apply Murphy's law to the kms irq code

2012-05-24 Thread j.glisse
On Thu, May 24, 2012 at 11:35:15AM -0400, Alex Deucher wrote: On Thu, May 24, 2012 at 3:49 AM, Christian König deathsim...@vodafone.de wrote: From: Christian Koenig christian.koe...@amd.com 1. It is really dangerous to have more than one   spinlock protecting the same information. 2.