Re: [PATCH 3/3] Fix lockdep warning

[Randy Dunlap]

On Wed,  6 Aug 2008 15:30:31 +0200 (CEST) Karsten Keil wrote:

> use correct dynamic spinlock init function.
> 
> Signed-off-by: Karsten Keil <kkeil@xxxxxxx>

Hi,

It would be a Really Good Thing if we could look at the subject line
and see that this is for an ISDN driver instead of something else,
like even the general lockdep framework.

The Perfect Patch (http://www.zip.com.au/~akpm/linux/patches/stuff/tpp.txt)
section 2.b. discusses this, but it isn't quite strong enough.  ;)


Thanks.

> ---
>  drivers/isdn/mISDN/tei.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/drivers/isdn/mISDN/tei.c b/drivers/isdn/mISDN/tei.c
> index 6fbae42..d8af45c 100644
> --- a/drivers/isdn/mISDN/tei.c
> +++ b/drivers/isdn/mISDN/tei.c
> @@ -1287,7 +1287,7 @@ create_teimanager(struct mISDNdevice *dev)
>  	if (!mgr)
>  		return -ENOMEM;
>  	INIT_LIST_HEAD(&mgr->layer2);
> -	mgr->lock = __RW_LOCK_UNLOCKED(mgr->lock);
> +	rwlock_init(&mgr->lock);
>  	skb_queue_head_init(&mgr->sendq);
>  	mgr->nextid = 1;
>  	mgr->lastid = MISDN_ID_NONE;
> -- 

---
~Randy
Linux Plumbers Conference, 17-19 September 2008, Portland, Oregon USA
http://linuxplumbersconf.org/
--
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/