From: Szymon Janc <szymon.janc@codecoup.pl>
To: Marcel Holtmann <marcel@holtmann.org>
Cc: Avinash Kadam <avinashk@marvell.com>,
"Wong, Joshua Weng Onn" <joshua.weng.onn.wong@intel.com>,
Bluez mailing list <linux-bluetooth@vger.kernel.org>,
"Wong, Mun choy" <mun.choy.wong@intel.com>,
"Zulqarnain, Adam" <adam.zulqarnain@intel.com>
Subject: Re: [EXT] Re: Unexpected SMP Command 0x17
Date: Mon, 27 Mar 2017 17:08:17 +0200 [thread overview]
Message-ID: <1589262.TPhyyzNXdL@ix> (raw)
In-Reply-To: <AC5A66AA-49F3-4F10-8D7A-EDDEC395A4FE@holtmann.org>
Hi,
On Monday, 27 March 2017 16:55:05 CEST Marcel Holtmann wrote:
> Hi Avinash,
>
> please also refrain from top posting.
>
> > Yes, for secure connection the LTK is generated locally.
> > But issue here is observed that after Pairing is complete the key
> > distribution is not completed from Master.
> >
> > i.e. After Slave sends the "Signature key:" but Master doesn't share any
> > key. Attached logs.
> I get that and that is clear from the logs. Something is stalling here and
> because of that, you run into the 30 seconds SMP timeout. We just need to
> know if the 4.9 kernel is doing this correctly. If so, then you can bi-sect
> that patch that fixes. Without proof that 4.9 is also broken, nobody will
> even bother to chase this down.
I think the problem here is race between ACL data and HCI events on USB
dongle... We get initial slave keys but those get dropped due to encryption
changed event not being received yet. Since keys were silently dropped we
later on get unexpected SMP PDU and ignoring remaining keys as well which
eventually leads to SMP timeout.
If this is USB dongle (using btusd) then only (AFAIK) solution would be to
have a workaround for this inside chip (it would delay ACL data received right
after encryption change giving host time to handle encpryption change event).
Bluetooth specification for USB transport is unfortunatelly kinda broken.
--
pozdrawiam
Szymon Janc
next prev parent reply other threads:[~2017-03-27 15:08 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-03-22 2:47 Unexpected SMP Command 0x17 Wong, Joshua Weng Onn
2017-03-22 7:36 ` Marcel Holtmann
2017-03-27 14:50 ` [EXT] " Avinash Kadam
2017-03-27 14:55 ` Marcel Holtmann
2017-03-27 15:08 ` Szymon Janc [this message]
2017-03-30 0:00 ` Wong, Joshua Weng Onn
2017-03-30 6:13 ` [EXT] " Marcel Holtmann
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1589262.TPhyyzNXdL@ix \
--to=szymon.janc@codecoup.pl \
--cc=adam.zulqarnain@intel.com \
--cc=avinashk@marvell.com \
--cc=joshua.weng.onn.wong@intel.com \
--cc=linux-bluetooth@vger.kernel.org \
--cc=marcel@holtmann.org \
--cc=mun.choy.wong@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).