Hi Marcel,
On Mon, Aug 15, 2011 at 12:17 AM, Marcel Holtmann <marcel@xxxxxxxxxxxx> wrote:
> Hi Ilia,
>
>> While it is possible to send flow specification HCI command
>> from the user space directly via the send api of HCI socket,
>> a more specific approach is preferable over the opaque
>> HCI command, since it will allow for centralized mangament of
>> flow specifiaction requests for different l2cap flows over
>> the same acl link in future. This feature was used successfully
>> for solving the qos A2DP issue in piconet with concurent file
>> transfer.
>>
>> Signed-off-by: Ilia Kolomisnky <iliak@xxxxxx>
>> ---
>> include/net/bluetooth/hci.h | 38 ++++++++++++++++++----
>> include/net/bluetooth/hci_core.h | 3 ++
>> net/bluetooth/hci_conn.c | 1 +
>> net/bluetooth/hci_core.c | 63 ++++++++++++++++++++++++++++++++++++++
>> net/bluetooth/hci_event.c | 37 ++++++++++++++++++++++
>> net/bluetooth/hci_sock.c | 2 +
>> 6 files changed, 137 insertions(+), 7 deletions(-)
>>
>> diff --git a/include/net/bluetooth/hci.h b/include/net/bluetooth/hci.h
>> index be30aab..f0f90d5 100644
>> --- a/include/net/bluetooth/hci.h
>> +++ b/include/net/bluetooth/hci.h
>> @@ -109,6 +109,8 @@ enum {
>> #define HCISETLINKMODE _IOW('H', 226, int)
>> #define HCISETACLMTU _IOW('H', 227, int)
>> #define HCISETSCOMTU _IOW('H', 228, int)
>> +#define HCISETFLOWSPEC _IOW('H', 229, int)
>> +
>
> I am not accepting any new addition of ioctl() at this moment. We are
> moving towards a proper defined management API. And that is how this
> should be addressed.
>
> And these kind of ictol() that have to execute a HCI command and wait
> for the result are not a good for being an ioctl() in the first place.
>
> Regards
>
> Marcel
I got the impression that the spec indicates this should first be
negotiated on l2cap level since there is a command with exact same
parameters, perhaps the way to go would be to have this as a socket
option and require some capability to be able to set them, after the
L2CAP QoS negotiation completes the kernel could automatically set the
HCI QoS based on that.
--
Luiz Augusto von Dentz
--
To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
