Re: [Bluez-devel] [PATCH] BLUETOOTH: Use newer, non-deprecated __RW_LOCK_UNLOCKED macro.

[Marcel Holtmann <marcel@holtmann.org>]

Hi Robert,

> > > Signed-off-by: Robert P. J. Day <rpjday@crashcourse.ca>
> > >
> > > ---
> > >
> > >   not compile tested, but a fairly trivial transformation.
> > >
> > >
> > >  net/bluetooth/hci_sock.c    |    2 +-
> > >  net/bluetooth/l2cap.c       |    2 +-
> > >  net/bluetooth/rfcomm/sock.c |    2 +-
> > >  net/bluetooth/sco.c         |    2 +-
> > >  4 files changed, 4 insertions(+), 4 deletions(-)
> > >
> > > diff --git a/net/bluetooth/hci_sock.c b/net/bluetooth/hci_sock.c
> > > index b5d4019..1d36c09 100644
> > > --- a/net/bluetooth/hci_sock.c
> > > +++ b/net/bluetooth/hci_sock.c
> > > @@ -84,7 +84,7 @@ static struct hci_sec_filter hci_sec_filter = {
> > >  };
> > >
> > >  static struct bt_sock_list hci_sk_list = {
> > > -	.lock = RW_LOCK_UNLOCKED
> > > +	.lock = __RW_LOCK_UNLOCKED(hci_sk_list.lock)
> > >  };
> >
> > is that the way we do init of locks now? I haven't realized that
> > RW_LOCK_UNLOCKED is deprecated.
> 
> yup, see Documentation/spinlocks.txt:
> 
> "SPIN_LOCK_UNLOCKED and RW_LOCK_UNLOCKED defeat lockdep state tracking
> and are hence deprecated.
> 
> "Please use DEFINE_SPINLOCK()/DEFINE_RWLOCK() or
> __SPIN_LOCK_UNLOCKED()/__RW_LOCK_UNLOCKED() as appropriate for static
> initialization."

then add an Acked-by: Marcel Holtmann <marcel@holtmann.org> into your
commit message and post it to the netdev mailing list (me on CC). This
way it is faster since I am stuck with a heavy workload for the next
BlueZ release.

Regards

Marcel



-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
Bluez-devel mailing list
Bluez-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/bluez-devel