[PATCH] Revert "[IRDA]: Lockdep fix."

[PATCH] Revert "[IRDA]: Lockdep fix."

[Samuel Ortiz]

Hi Linus,

commit 700f9672c9a61c12334651a94d17ec04620e1976 breaks IrDA as irlmp.c
can no longer build. 
This is due to the spin_lock_irqsave_nested() patches being in the -mm 
tree and not yet in yours.
I'll resend the patch once both trees are synchronized.

This reverts commit 700f9672c9a61c12334651a94d17ec04620e1976.
Signed-off-by: Samuel Ortiz <samuel@sortiz.org>

---
 net/irda/irlmp.c |    3 +--
 1 files changed, 1 insertions(+), 2 deletions(-)

diff --git a/net/irda/irlmp.c b/net/irda/irlmp.c
index fede837..5073261 100644
--- a/net/irda/irlmp.c
+++ b/net/irda/irlmp.c
@@ -1678,8 +1678,7 @@ #endif /* CONFIG_IRDA_ULTRA */
 	 *  every IrLAP connection and check every LSAP associated with each
 	 *  the connection.
 	 */
-	spin_lock_irqsave_nested(&irlmp->links->hb_spinlock, flags,
-			SINGLE_DEPTH_NESTING);
+	spin_lock_irqsave(&irlmp->links->hb_spinlock, flags);
 	lap = (struct lap_cb *) hashbin_get_first(irlmp->links);
 	while (lap != NULL) {
 		IRDA_ASSERT(lap->magic == LMP_LAP_MAGIC, goto errlap;);
-- 
1.4.2.3

Re: [PATCH] Revert "[IRDA]: Lockdep fix."

[David Miller]

From: Samuel Ortiz <samuel@sortiz.org>
Date: Sat, 25 Nov 2006 17:26:49 +0200

> Hi Linus,
> 
> commit 700f9672c9a61c12334651a94d17ec04620e1976 breaks IrDA as irlmp.c
> can no longer build. 
> This is due to the spin_lock_irqsave_nested() patches being in the -mm 
> tree and not yet in yours.
> I'll resend the patch once both trees are synchronized.
> 
> This reverts commit 700f9672c9a61c12334651a94d17ec04620e1976.
> Signed-off-by: Samuel Ortiz <samuel@sortiz.org>

Andrew is going to merge the necessary infrastructure to Linus, please
don't revert this change.

Why is everyone so impatient about this issue?  Just wait for Andrew
to merge the fix and all will be well :-)

Re: [PATCH] Revert "[IRDA]: Lockdep fix."

[Linus Torvalds]

On Sat, 25 Nov 2006, David Miller wrote:
> 
> Why is everyone so impatient about this issue?  Just wait for Andrew
> to merge the fix and all will be well :-)

No, I think this was a total failure. We should have reverted it 
immediately rather than waiting for the fix. Especially as the damn thing 
wasn't even all that important.

I now (finally) have the patch from Andrew, but we should _not_ have had 
this thing broken for three days. It should have gotten reverted on the 
first report of trouble, instead of us telling people to just wait.

Broken compiles are simply not acceptable. Patches that cause them should 
be reverted _immediately_ unless a fix is available as quickly (which it 
wasn't due to turkey-day).

No excuses. People _should_ be impatient about idiotic failures like this.

		Linus

Re: [PATCH] Revert "[IRDA]: Lockdep fix."

[David Miller]

From: Linus Torvalds <torvalds@osdl.org>
Date: Sat, 25 Nov 2006 13:26:51 -0800 (PST)

> I now (finally) have the patch from Andrew, but we should _not_ have had 
> this thing broken for three days. It should have gotten reverted on the 
> first report of trouble, instead of us telling people to just wait.
> 
> Broken compiles are simply not acceptable. Patches that cause them should 
> be reverted _immediately_ unless a fix is available as quickly (which it 
> wasn't due to turkey-day).
> 
> No excuses. People _should_ be impatient about idiotic failures like this.

Ok, no problem.