linux-bluetooth.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
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

  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).