Re: [Bluez-devel] Re: Qualification testing - rfcomm

[Max Krasnyansky <maxk@qualcomm.com>]

On Fri, 2003-07-11 at 01:18, Marcel Holtmann wrote:
> Hi Daryl,
> 
> > TP/RFC/BV-12-C:
> > "Verify that the IUT handles aggregate flow control correctly when the
> > Tester, acting as a device conforming to Bleutooth version 1.0B, controls
> > the data flow using the Flow Control on/off commands FCon and FCoff. The
> > IUT's device role is of no importance."
> > 
> > The command and console output:
> > 
> > root@jack-00000000:~>./rctest -s -P 1 -b 20 00:A0:96:1F:83:71
> > rctest[362]: Connected
> > rctest[362]: Sending ...
> > rfcomm_recv_mcc: Unknown control type 0x18
> > rfcomm_recv_mcc: Unknown control type 0x28
> > 
> > The IUT does not respond to FCoff with FCoff, and continues sending data. It
> > also does not respond to FCon with FCon. 
> 
> we don't have support for flow control on the entire RFCOMM session. But
> it seems that the FCOFF and FCON should be supported. The attached patch
> implements support for it, but I am not quite sure if it is the right
> way to do this.
Is it mandatory ? I always thought it's optional.

> @@ -1533,6 +1591,9 @@
>  	struct list_head *p, *n;
> 
>  	BT_DBG("session %p state %ld", s, s->state);
> +
> +	if (test_bit(RFCOMM_TX_THROTTLED, &s->flags))
> +		return;
> 
>  	list_for_each_safe(p, n, &s->dlcs) {
>  		d = list_entry(p, struct rfcomm_dlc, list);
This part is not ok. We still have to check for timeout. Rest of patch is fine.

Max