Return-Path: Message-ID: <456336AC.7070906@vasmac.com> Date: Tue, 21 Nov 2006 12:26:04 -0500 From: Jose Vasconcellos MIME-Version: 1.0 To: BlueZ development References: <455392B6.5060209@free.fr> <45622D23.7070105@vasmac.com> <456234FB.4020506@xmission.com> <1164094500.28429.9.camel@localhost> <4563155A.90905@vasmac.com> <1164122797.28429.49.camel@localhost> In-Reply-To: <1164122797.28429.49.camel@localhost> Subject: Re: [Bluez-devel] SCO flow control (Was: New bluetooth-headset release 20061109) Reply-To: BlueZ development List-Id: BlueZ development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Sender: bluez-devel-bounces@lists.sourceforge.net Errors-To: bluez-devel-bounces@lists.sourceforge.net Marcel Holtmann wrote: > Hi Jose, > > >>> I am not against using flow control, but we must be really careful if it >>> supported or not. While for ACL it is mandatory, the SCO channel can be >>> used without it. However even the use of flow control on SCO is a job >>> for the Bluetooth core and no driver specific one. All Bluetooth drivers >>> are transport drivers. They have no intelligence whatsoever and they are >>> not interfering the HCI data. If they need to know some specific details >>> about changes in the core then the notify() callback is the right place >>> and if they need a special setup, then a quirk should be used. Other >>> than that, they are plain stupid and only moving HCI packets to the >>> hardware and back. >>> >>> So from my understanding the high resolution timer are a good choice to >>> submit SCO packets to the hardware. In addition we can enable the flow >>> control for SCO to have more control over the hardware. However please >>> keep in mind that the enabling flow control means for traffic on the ACL >>> links and so you might see a downgrade in bandwidth. So from my current >>> point of view, the flow control stuff must be optional. >>> >> The problem with using timers is that they are not synchronized >> with the bluetooth controller (dongle). This can cause drift after >> a while leading to underrun condition on the controller. This will >> cause pops or other audio artifacts. So this solution will always >> have a design flaw. I don't find this acceptable. >> > > I don't see the synchronization as a problem, because once SCO traffic > is on, we get a PCM stream back from the chip we can use this one as a > synchronization source. > > I don't see this as an acceptable solution. First, the current code makes it very easy for a trivial program to cause Linux to panic. Second, using received data to synchronize puts the burden at the application layer when this should be a driver issue. Third, this will never scale to asymmetric eSCO connections. >> Let me address some of your objections to using the flow control: >> >> 1. extra bandwith - this is the price to pay for synchronization. >> In the case of USB no additional messages are sent and overhead >> is part of the USB isosynchronous transport. >> > > Wrong. We always have additional HCI commands and events if SCO flow > control has been enabled. This is no difference from the actual > transport you use. The only difference for USB is that they are on a > different interface of the USB device. > > In the case of USB, the flow control messages are sent to the bluetooth hci core by hci_usb; this is done when the USB layer has acknowledged the transferred. No additional messages are sent by the controller. Whether you take advantage of the USB layer or not, the USB overhead is always there. >> 2. some dongles may not support flow control - it's true that flow >> control can be turned off for synchronous traffic but it' s not clear >> to me that this is optional for controllers that support SCOs. >> We're really talking about UART or other interfaces. Can anyone >> provide an example where it's NOT supported? >> > > It is optional no matter if it is UART, USB or any other transport. This > is has nothing to do with the actual hardware interface. > It's optional in the sense that an application may not need it. A custom profile may be sending non-PCM data and deal with it. That's not to say that it's optional for a controller if it supports SCO. For USB there's flow control at that layer. >> 3. Transport drivers are dumb - this is a problem with the existing >> drivers that should be addressed. They need to differentiate >> between different types of traffic otherwise it's quite easy to >> starve the synchronous channel with ACL data. In fact, the >> current architecture can't support multiple SCOs reliably because >> of this. At the very least, separate queues must be used for >> synchronous and other traffic. >> > > I don't see what you are actually talking about. We have separate queues > for ACL and SCO data, because the queues are connection specific. What > the actual driver is doing is up to the driver. However the Bluetooth > hardware drivers are transport drivers and the core has to be fully > agnostic from any hardware specific implementation of the transports. > > The scheduling in hci_core.c will send all acl data before sco. This prevents the driver from having access to the SCO data in a timely manner. Then the hci_usb driver puts the packets in one transmit queue. Before going too far with technical details, I think it's worth understanding what level of Bluetooth compatibility is desired. The current state is adequate but not robust (easy to panic). Perhaps my expectations are higher that what the community wants. Of course, you could argue that other environments or solutions aren't much better, and this maybe true. I don't mean to cause problems; I just want a better support for SCO/eSCO. Regards, Jose ------------------------------------------------------------------------- Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT & business topics through brief surveys - and earn cash http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV _______________________________________________ Bluez-devel mailing list Bluez-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/bluez-devel