From: Mikel Astiz <mikel.astiz@xxxxxxxxxxxx> This RFC presents a proposal to extend the BlueZ core to simplify the implementation of profiles. It introduces a new core concept (btd_server) as a first step towards providing internal infrastructure in a similar way that external profiles have. This exact purpose of the new struct is described in patch 01. Currently, it's paradoxical that externally implemented profiles are in a way simpler to implement. There is a lot of helpful code in profile.c which could be reused for internal profiles too. This first patchset consists of three main parts: 1. The extension of the core (patches 01..03). 2. Some simple examples of adoption (patches 04..10). 3. A more complex adoption (patches 11..16). This part is fairly simple and doesn't offer great benefits besides the usage of the userdata pointer in btd_server, which can already be handy (see for example 08/16). Furhter and more interesting changes in the pipeline (but not submitted in v0) include: 4. Handling of incoming connections. 5. Integration with agent authorization. 6. Adoption of new infrastructure for external profiles. For the record, the result of this whole work does not add more lines of code to maintain, it just moves the code complexity from profile implementations to the core. The result is less duplicated code and data throughout the codebase, hopefully leading to more robust profiles. Mikel Astiz (16): core: Add btd_server adapter: Create btd_server instances when probing profile: Use btd_server to probe adapters input: Bypass manager for profile server input: Use btd_server userdata for input_server thermometer: Remove boilerplate code thermometer: Use btd_server userdata cyclingspeed: Use btd_server userdata heartrate: Remove boilerplate code heartrate: Use btd_server userdata network: Replace list with network_adapter network: Bypass manager for profile server network: Simplify search-by-UUID network: Add a dedicated btd_profile for BNEP network: Create network_adapter during BNEP probe network: Use btd_server userdata for network_server Makefile.am | 1 + profiles/alert/server.c | 9 +- profiles/audio/manager.c | 41 +++++--- profiles/cyclingspeed/cyclingspeed.c | 38 +++---- profiles/health/hdp_manager.c | 11 ++- profiles/heartrate/heartrate.c | 51 +++------- profiles/input/manager.c | 12 +-- profiles/input/server.c | 28 ++---- profiles/input/server.h | 4 +- profiles/network/manager.c | 76 ++------------ profiles/network/server.c | 185 +++++++++++++++-------------------- profiles/network/server.h | 8 +- profiles/proximity/reporter.c | 8 +- profiles/proximity/reporter.h | 5 +- profiles/sap/manager.c | 9 +- profiles/thermometer/thermometer.c | 82 ++++------------ profiles/time/server.c | 8 +- src/adapter.c | 50 +++++----- src/adapter.h | 2 + src/profile.c | 11 ++- src/profile.h | 7 +- src/server.c | 108 ++++++++++++++++++++ src/server.h | 38 +++++++ 23 files changed, 389 insertions(+), 403 deletions(-) create mode 100644 src/server.c create mode 100644 src/server.h -- 1.8.1.4 -- 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
