All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Gregory M. Turner" <gmt@be-evil.net>
To: "linux-bluetooth@vger.kernel.org"
	<linux-bluetooth@vger.kernel.org>,
	Marcel Holtmann <marcel@holtmann.org>
Subject: Re: [PATCH] bluetooth/l2cap: silence framedrop kmsg noise
Date: Mon, 23 Feb 2015 13:09:07 -0800	[thread overview]
Message-ID: <1619328.sIzMdT7lV0@moneypit> (raw)
In-Reply-To: <FD2FEA27-B249-43B9-A269-AA5DC4A519D3@holtmann.org>

On Sunday 22 February 2015 16:05:37 Marcel Holtmann wrote:
> Hi Greg,
>
> I get the feeling that either dongle or the remote device is broken. If it
> is the dongle, it might actually needs a firmware update / ROM patch to
> make this behave properly.

I'm pretty sure the firmware's up to date on it.  But this thing is quite old 
and not well supported.  I think it's an old revision of the HW that was 
superseded by something slightly less shitty in future Logitech packages (they 
do still sell this keyboard package, or did the last time I checked, but 
reports were, it came with a different dongle).

This behavior definitely follows the dongle -- all other factors have been 
varied and ruled out over the many years I've put up with this thing.

> The real problem is that we get a ACL fragment from the controller telling
> us that it is continuation of a previous packet. However this previous
> packet seems to never arrive in the first place. Meaning I have no idea
> what is actually happening here. The error message tells you that.

With length zero?  That seems crazy.  Or am I misunderstanding what that 
number signifies?

> So instead of removing it, we might want to consider using rate limiting for
> that specific error message.

In my particular case, it makes sense to just apply the patch I posted even if 
it's wrong.  Clearly, though, this is treating the symptom, not the disease.

> It is a valid error message.

Idunno... obv. it's valid in the sense that this is what the driver thinks the 
device is telling the host, hence, my patch is stupid.

However, it seems I have either a) a broken dongle b) a dongle that's working 
as designed, but whose design is broken or nonstandard*, or c) a broken linux 
driver (but the line between c and b is blurry).

I'm wondering, in particular, about (b).  I used to have a second dongle like 
this one.  It fell apart some years ago (hmm, I may have saved the pieces 
somewhere), but I'm pretty sure it had the same behavior.  That was a long 
time ago, however; I could definitely be remembering incorrectly.

I'll see if I can't learn how to get my android phone to sniff this traffic.  My 
thinking is, if a sniffed dump looked different from the host's dump, I'd have a 
pretty clear indication of a problem arising at or below the driver layer.

Another similar avenue of attack would be to mess around with Windows.  If 
there is some way to generate these dumps in Windows, and everything looked 
fine there, I'd at least have a clear indication that the kernel driver lacks 
some needed quirk support for this device.

-gmt

  reply	other threads:[~2015-02-23 21:09 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-02-22 20:36 [PATCH] bluetooth/l2cap: silence framedrop kmsg noise Gregory M. Turner
2015-02-22 20:53 ` Marcel Holtmann
2015-02-22 22:04   ` Greg Turner
2015-02-23  0:05     ` Marcel Holtmann
2015-02-23 21:09       ` Gregory M. Turner [this message]
2015-02-23 21:18         ` Marcel Holtmann
2015-02-23 22:49           ` Gregory M. Turner
2015-02-23 23:00             ` Gregory M. Turner
2015-02-24  0:31           ` Gregory M. Turner

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=1619328.sIzMdT7lV0@moneypit \
    --to=gmt@be-evil.net \
    --cc=linux-bluetooth@vger.kernel.org \
    --cc=marcel@holtmann.org \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.