2006-03-20 14:30:34

by Martin Röhricht

[permalink] [raw]
Subject: [Bluez-devel] information responses in l2cap.c

Hello together,

I need urgent help on an issue concerning information responses in
l2cap.c, perhaps somebody knows something about this implementation.
The current code didn't make use of information requests and so didn't
receive information responses, too. I'd like to change that as it is
recommended by the specfication to first interact with the remote peer
by using information requests after a successful connection. I would
like to ask the remote peer if it supports extended features, namely
especially flow control, before I start request this technique. So far
so good.
So I send an information request via the l2cap_build_info_req() in the
function l2cap_connect_rsp() whenever the connection response signals
success. This works as expected and my Siemens S75 mobile phone responds
with an information response immediately. Guess what, it send as a
result the bytes =BB07 00 00 00=AB which indicates full extended features
support. Wow. Now I'd like store this information in the corresponding
socket so that I can continue the whole process with a configuration
request. I would like to call l2cap_build_conf_req() within
l2cap_information_rsp() like this:

/* build configuration request */
l2cap_pi(sk)->conf_state |=3D L2CAP_CONF_REQ_SENT;
l2cap_send_cmd(conn, l2cap_get_ident(conn), L2CAP_CONF_REQ,
l2cap_build_conf_req(sk, req), req);

But here is the problem located. From within an information response, I
cannot reliably find the corresponding socket. All information pieces I
get are the connection, the command header and the data. So there is no
source or destination ID and the only function I could find which may
return the socket for a connection is l2cap_get_chan_by_ident() like
this:
if (!(sk =3D l2cap_get_chan_by_ident(&conn->chan_list, cmd->ident))) {
printk("---> no socket!\n");
return 0;
}
Unfortunately cmd->ident and the conn ident are not equal and so the
socket cannot be found ...

I have no clue what is wrong or what else I should do to get the socket
from an incoming information_rsp()

Martin



-------------------------------------------------------
This SF.Net email is sponsored by xPML, a groundbreaking scripting language
that extends applications into web and mobile media. Attend the live webcast
and join the prime developer group breaking into this new coding territory!
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=110944&bid=241720&dat=121642
_______________________________________________
Bluez-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/bluez-devel


2006-03-20 17:59:22

by Martin Röhricht

[permalink] [raw]
Subject: Re: [Bluez-devel] information responses in l2cap.c

I may have found a solution for it.

Am Montag, den 20.03.2006, 15:30 +0100 schrieb Martin R=F6hricht:
> /* build configuration request */
> l2cap_pi(sk)->conf_state |=3D L2CAP_CONF_REQ_SENT;
> l2cap_send_cmd(conn, l2cap_get_ident(conn), L2CAP_CONF_REQ,
> l2cap_build_conf_req(sk, req), req);

(Damn, I posted the wrong lines in my preceeding mail, but it looks
almost the same. Instead of building a configuration request, I have
code to build an information request)
Within the send_cmd function call, I use l2cap_get_ident(conn) which
increases the ident counter by one for each call. I did mistakenly
assume that it just returns the ID for a given connection. Instead I use
now cmd->ident like this:

l2cap_send_cmd(conn, cmd->ident, L2CAP_INFO_REQ,
l2cap_build_info_req(sk, &info), &info);

> if (!(sk =3D l2cap_get_chan_by_ident(&conn->chan_list, cmd->ident))) {
> printk("---> no socket!\n");
> return 0;
> }

These lines work now, as a socket can be found.

I'm not 100% sure yet whether this works as expected or not, but I have
a better feeling.
If somebody has some more information about socket and ID handling in
this case, feel free to let me know.

Martin



-------------------------------------------------------
This SF.Net email is sponsored by xPML, a groundbreaking scripting language
that extends applications into web and mobile media. Attend the live webcast
and join the prime developer group breaking into this new coding territory!
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=110944&bid=241720&dat=121642
_______________________________________________
Bluez-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/bluez-devel