From: Luiz Augusto von Dentz <luiz.dentz@gmail.com>
To: Ulisses Furquim <ulisses@profusion.mobi>
Cc: Szymon Janc <szymon.janc@tieto.com>,
"linux-bluetooth@vger.kernel.org"
<linux-bluetooth@vger.kernel.org>
Subject: Re: [PATCH] Bluetooth: Fix not clearing ack timer when sending an i-frame
Date: Tue, 7 Feb 2012 15:59:35 +0200 [thread overview]
Message-ID: <CABBYNZJaVutBCNGfQXV5n-LNyAdf_u7yRrK9U3DKd3bVEGk8UA@mail.gmail.com> (raw)
In-Reply-To: <CAA37ikbn7KUDh0tNR63NY6n4Ah64VLL+x-iq79oZiWGZuDOU+g@mail.gmail.com>
Hi Ulisses,
On Tue, Feb 7, 2012 at 3:45 PM, Ulisses Furquim <ulisses@profusion.mobi> wrote:
>> Btw, here is what Ive been using to test this code:
>>
>>> obexd/tools/test-server -b -c 4097 -p -i 32767 -o 32767
>>
>>> obexd/tools/test-client -b -s <address of source adapter> -d <address of destination adapter> -c 4097 -p -i 32767 -o 32767
>
> Ok. Are you running them on the same machine with 2 dongles?
Yep
>> Im using 32767 as MTU because that is what we use by default in OBEX,
>> but currently it doesn't work due to some bug in ERTM that apparently
>> doesn't handle MTU being bigger than mts * tx_win, so the transfer
>> just stall at some point, using something like 16384 works though.
>
> Does it stall forever? I have no idea what might be this one.
It timeout after 10 seconds than the client disconnects, but I don't
think it would recover even after that since basically each side does
ack with s-frame RR and nothing else happen. I suspect we are going
past what a window could carry (tx_win (63) * mts (300)) because the
MTU is bigger than that the sdu_len can be bigger too, afaik this
would have to be fragmented in different window or perhaps the mts
size should be adjusted too, but lets figure out this in another
thread to avoid too much noise here.
--
Luiz Augusto von Dentz
prev parent reply other threads:[~2012-02-07 13:59 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-02-06 14:38 [PATCH] Bluetooth: Fix not clearing ack timer when sending an i-frame Luiz Augusto von Dentz
2012-02-06 17:27 ` Ulisses Furquim
2012-02-07 8:08 ` Szymon Janc
2012-02-07 10:21 ` Luiz Augusto von Dentz
2012-02-07 11:19 ` Ulisses Furquim
2012-02-07 11:21 ` Szymon Janc
2012-02-07 11:27 ` Ulisses Furquim
2012-02-07 12:02 ` Szymon Janc
2012-02-07 12:37 ` Ulisses Furquim
2012-02-07 13:28 ` Luiz Augusto von Dentz
2012-02-07 13:45 ` Ulisses Furquim
2012-02-07 13:59 ` Luiz Augusto von Dentz [this message]
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=CABBYNZJaVutBCNGfQXV5n-LNyAdf_u7yRrK9U3DKd3bVEGk8UA@mail.gmail.com \
--to=luiz.dentz@gmail.com \
--cc=linux-bluetooth@vger.kernel.org \
--cc=szymon.janc@tieto.com \
--cc=ulisses@profusion.mobi \
/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).