netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* Pull request: bluetooth-2.6 2009-05-11
@ 2009-05-12  3:06 Marcel Holtmann
  2009-05-16 20:46 ` David Miller
  0 siblings, 1 reply; 4+ messages in thread
From: Marcel Holtmann @ 2009-05-12  3:06 UTC (permalink / raw)
  To: David S. Miller; +Cc: netdev

Hi Dave,

unfortunately I do have three more patches for the 2.6.30 release. The
guys from Nokia are heavily testing this release to make sure that even
the smallest regression or race condition is reported and fixed. And I
am thankful for this since some of these cases only happen with specific
devices out in the market. And while I own a lot of Bluetooth devices,
it is only a small portion :)

We do have a broken module reference counting that got introduced when
trying to fix a lifetime issue with the hci_dev structure. It races
against the sysfs handling inside the work queue. To fix this, split the
reference counting and let the work queue task hold its own.

We call hci_acl_connect_cancel() for an incoming connection. That is
just wrong since their is nothing to cancel. Incoming connections can be
either accepted or reject. Once accepted, their is nothing to cancel.
Only a disconnect would be working.

The security mode 3 with legacy devices triggered another regression
with the pairing timeout. So only use the pairing timeout if the
connection is fully connected. In the security mode 3 the pairing
happens between the connection attempt and completion. And that phase
uses a proper timeout already, so no need to add extra code for special
pairing timeout handling.

Regards

Marcel


Please pull from

    git://git.kernel.org/pub/scm/linux/kernel/git/holtmann/bluetooth-2.6.git master

This will update the following files:

 net/bluetooth/hci_conn.c  |    6 +++---
 net/bluetooth/hci_event.c |    2 +-
 net/bluetooth/hci_sysfs.c |    3 +++
 3 files changed, 7 insertions(+), 4 deletions(-)

through these ChangeSets:

Marcel Holtmann (3):
    Bluetooth: Fix wrong module refcount when connection setup fails
    Bluetooth: Don't use hci_acl_connect_cancel() for incoming connections
    Bluetooth: Don't trigger disconnect timeout for security mode 3 pairing




^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2009-05-17  5:36 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2009-05-12  3:06 Pull request: bluetooth-2.6 2009-05-11 Marcel Holtmann
2009-05-16 20:46 ` David Miller
2009-05-16 22:32   ` Marcel Holtmann
2009-05-17  5:36     ` David Miller

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).