Re: [PATCH 0/2] Fix L2CAP timeout values

[Johan Hedberg <johan.hedberg@gmail.com>]

Hi Andrzej,

On Wed, Jan 04, 2012, Andrzej Kaczmarek wrote:
> After moving L2CAP timers to workqueues there is problem with timeout
> values which are defined in miliseconds but timer functions works with
> jiffies. As a result, timers are unreliable on platforms with CONFIG_HZ
> not set to 1000.
> 
> Similar problem was fixed before workqueues but now it's apparently
> back. This time however I left l2ca_set_timer to use values in jiffies
> to avoid confusion and make code less prone to similar issues in
> future (I hope).
> 
> Also together with patch from Peter Hurley it should fix problems with
> SSP and RFCOMM as recently discovered (on platforms with
> CONFIG_HZ != 1000).
> 
> Andrzej Kaczmarek (2):
>   Bluetooth: Fix sk_sndtimeo initialization for L2CAP socket
>   Bluetooth: l2cap_set_timer needs jiffies as timeout value
> 
>  include/net/bluetooth/l2cap.h |    6 +++---
>  net/bluetooth/l2cap_core.c    |   12 ++++++++----
>  net/bluetooth/l2cap_sock.c    |    2 +-
>  3 files changed, 12 insertions(+), 8 deletions(-)

Both patches have been applied to my bluetooth-next tree. Thanks.

Johan