Re: [PATCH] netfilter: use per-CPU r**ursive lock {XV}

From: Linus Torvalds
Date: Mon Apr 27 2009 - 19:15:31 EST




On Mon, 27 Apr 2009, Linus Torvalds wrote:
>
> BTW: THIS IS TOTALLY UNTESTED.

Gaah. I should have read through it one more time before sending.

> static inline void xt_info_rdunlock_bh(void)
> {
> struct xt_info_lock *lock;
>
> lock = &__get_cpu_var(xt_info_locks);
> if (!--lock->readers)
> spin_unlock(&lock->lock);
> }

This one was missing the "local_bh_enable()" at the end.

There may be other bugs, but that's the one I noticed immediately when
reading what I sent out. Oops.

Linus
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/