Re: [PATCH v4 0/5] Audio profile authorization

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Mikel,

On Fri, Sep 28, 2012 at 7:32 PM, Mikel Astiz <mikel.astiz.oss@xxxxxxxxx> wrote:
> From: Mikel Astiz <mikel.astiz@xxxxxxxxxxxx>
>
> v4 considers the review from Luiz and Johan, including:
> - guint used for authorization ids, instead of int
> - static local variable used to generate new ids
> - service_auth_id renamed to auth_id
>
> From original cover-letter:
>
> The agent-based profile authorization makes a special consideration for audio profiles: they are processed as if they all belonged to one single profile. This includes several internal policies that are inconvenient for IVI use-cases.
>
> This patchset removes this exception by making use of the conventional authorization mechanism also for audio profiles.
>
> The new approach is not straightforward since devices might send several connection requets in parallel (i.e. HFP, A2DP, AVRCP). This cannot be neither automatically rejected (EBUSY) nor forwarded in parallel to the agent, so a queue was added to store the pending authorization requests. These will be sent to the agent sequentially.
>
> Mikel Astiz (5):
>   build: Update glib dependency to 2.32
>   adapter: Replace device authorizing flag
>   adapter: Use authorization id for cancelling
>   adapter: Queue parallel authorization requests
>   audio: Drop audio-specific authorization mechanism
>
>  acinclude.m4              |   4 +-
>  audio/avctp.c             |  27 +++---
>  audio/avdtp.c             |  23 ++++--
>  audio/device.c            | 144 --------------------------------
>  audio/device.h            |  12 +--
>  audio/manager.c           |  28 ++++---
>  plugins/service.c         |  18 ++--
>  profiles/input/server.c   |   7 +-
>  profiles/network/server.c |   9 +-
>  profiles/sap/server.c     |   8 +-
>  src/adapter.c             | 204 +++++++++++++++++++++++++++++-----------------
>  src/adapter.h             |   4 +-
>  src/device.c              |  11 ---
>  src/device.h              |   2 -
>  src/profile.c             |  25 ++----
>  15 files changed, 207 insertions(+), 319 deletions(-)
>
> --
> 1.7.11.4

All 5 patches applied, thanks.


-- 
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


[Index of Archives]     [Bluez Devel]     [Linux Wireless Networking]     [Linux Wireless Personal Area Networking]     [Linux ATH6KL]     [Linux USB Devel]     [Linux Media Drivers]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Big List of Linux Books]

  Powered by Linux