Return-Path: MIME-Version: 1.0 In-Reply-To: <003c01d00d71$f818ba20$e84a2e60$@samsung.com> References: <1417423638-29222-1-git-send-email-luiz.dentz@gmail.com> <1417423638-29222-8-git-send-email-luiz.dentz@gmail.com> <003a01d00d60$d0424590$70c6d0b0$@samsung.com> <003b01d00d6b$3723cfe0$a56b6fa0$@samsung.com> <003c01d00d71$f818ba20$e84a2e60$@samsung.com> Date: Mon, 1 Dec 2014 16:41:44 +0200 Message-ID: Subject: Re: [PATCH BlueZ 8/8] obexd/client: Add supported_features support From: Luiz Augusto von Dentz To: Gowtham Anandha Babu Cc: "linux-bluetooth@vger.kernel.org" Content-Type: text/plain; charset=UTF-8 Sender: linux-bluetooth-owner@vger.kernel.org List-ID: Hi Gowtham, On Mon, Dec 1, 2014 at 4:19 PM, Gowtham Anandha Babu wrote: > Hi Luiz, > >> -----Original Message----- >> From: linux-bluetooth-owner@vger.kernel.org [mailto:linux-bluetooth- >> owner@vger.kernel.org] On Behalf Of Luiz Augusto von Dentz >> Sent: Monday, December 01, 2014 7:30 PM >> To: Gowtham Anandha Babu >> Cc: linux-bluetooth@vger.kernel.org >> Subject: Re: [PATCH BlueZ 8/8] obexd/client: Add supported_features >> support >> >> Hi Gowtham, >> >> On Mon, Dec 1, 2014 at 3:34 PM, Luiz Augusto von Dentz >> wrote: >> > Hi Gowtham, >> > >> > On Mon, Dec 1, 2014 at 3:31 PM, Gowtham Anandha Babu >> > wrote: >> >> Hi Luiz, >> >> >> >>> -----Original Message----- >> >>> From: linux-bluetooth-owner@vger.kernel.org [mailto:linux-bluetooth- >> >>> owner@vger.kernel.org] On Behalf Of Luiz Augusto von Dentz >> >>> Sent: Monday, December 01, 2014 6:22 PM >> >>> To: Gowtham Anandha Babu >> >>> Cc: linux-bluetooth@vger.kernel.org >> >>> Subject: Re: [PATCH BlueZ 8/8] obexd/client: Add supported_features >> >>> support >> >>> >> >>> Hi Gowtham, >> >>> >> >>> On Mon, Dec 1, 2014 at 2:17 PM, Gowtham Anandha Babu >> >>> wrote: >> >>> > Hi Luiz, >> >>> > >> >>> >> -----Original Message----- >> >>> >> From: linux-bluetooth-owner@vger.kernel.org >> >>> >> [mailto:linux-bluetooth- owner@vger.kernel.org] On Behalf Of Luiz >> >>> >> Augusto von Dentz >> >>> >> Sent: Monday, December 01, 2014 2:17 PM >> >>> >> To: linux-bluetooth@vger.kernel.org >> >>> >> Subject: [PATCH BlueZ 8/8] obexd/client: Add supported_features >> >>> >> support >> >>> >> >> >>> >> From: Luiz Augusto von Dentz >> >>> >> >> >>> >> This adds supported_features support to obc_driver so driver can >> >>> >> provide this information when connecting. >> >>> >> >> >>> >> This is required by PBAP 1.2 (page 48): >> >>> >> >> >>> >> 'Mandatory if the PSE advertises a PbapSupportedFeatures attribute >> in >> >>> >> its SDP record, else excluded.' >> >>> >> --- >> >>> >> obexd/client/driver.h | 1 + >> >>> >> obexd/client/pbap.c | 36 >> >>> ++++++++++++++++++++++++++++++++++++ >> >>> >> obexd/client/session.c | 25 ++++++++++++++++++++++++- >> >>> >> 3 files changed, 61 insertions(+), 1 deletion(-) >> >>> >> >> >>> >> diff --git a/obexd/client/driver.h b/obexd/client/driver.h index >> >>> >> f1c0646..0112219 100644 >> >>> >> --- a/obexd/client/driver.h >> >>> >> +++ b/obexd/client/driver.h >> >>> >> @@ -26,6 +26,7 @@ struct obc_driver { >> >>> >> const char *uuid; >> >>> >> void *target; >> >>> >> gsize target_len; >> >>> >> + void *(*supported_features) (struct obc_session *session); >> >>> >> int (*probe) (struct obc_session *session); >> >>> >> void (*remove) (struct obc_session *session); }; diff >> >>> >> --git a/obexd/client/pbap.c b/obexd/client/pbap.c index >> >>> >> 812a7fb..57632b4 >> >>> >> 100644 >> >>> >> --- a/obexd/client/pbap.c >> >>> >> +++ b/obexd/client/pbap.c >> >>> >> @@ -76,6 +76,7 @@ >> >>> >> #define PRIMARY_COUNTER_TAG 0X0A >> >>> >> #define SECONDARY_COUNTER_TAG 0X0B >> >>> >> #define DATABASEID_TAG 0X0D >> >>> >> +#define SUPPORTED_FEATURES_TAG 0x10 >> >>> >> >> >>> >> #define DOWNLOAD_FEATURE 0x00000001 >> >>> >> #define BROWSE_FEATURE 0x00000002 >> >>> >> @@ -1230,6 +1231,40 @@ static void parse_service_record(struct >> >>> >> pbap_data >> >>> >> *pbap) >> >>> >> >> >>> >> } >> >>> >> >> >>> >> +static void *pbap_supported_features(struct obc_session *session) >> { >> >>> >> + const void *data; >> >>> >> + uint16_t version; >> >>> >> + >> >>> >> + /* Version */ >> >>> >> + data = obc_session_get_attribute(session, >> >>> >> SDP_ATTR_PFILE_DESC_LIST); >> >>> >> + if (!data) >> >>> >> + return NULL; >> >>> >> + >> >>> >> + version = GPOINTER_TO_UINT(data); >> >>> >> + >> >>> >> + if (version < 0x0102) >> >>> >> + return NULL; >> >>> >> + >> >>> >> + /* Supported Feature Bits */ >> >>> >> + data = obc_session_get_attribute(session, >> >>> >> + >> >>> >> SDP_ATTR_PBAP_SUPPORTED_FEATURES); >> >>> >> + if (!data) >> >>> >> + return NULL; >> >>> >> + >> >>> >> + return g_obex_apparam_set_uint32(NULL, >> >>> >> SUPPORTED_FEATURES_TAG, >> >>> >> + DOWNLOAD_FEATURE | >> >>> >> + BROWSE_FEATURE | >> >>> >> + DATABASEID_FEATURE | >> >>> >> + >> >>> >> + FOLDER_VERSION_FEATURE >> >>> >> | >> >>> >> + >> >>> >> + VCARD_SELECTING_FEATURE >> >>> >> | >> >>> >> + >> >>> >> + ENHANCED_CALLS_FEATURE >> >>> >> | >> >>> >> + UCI_FEATURE | >> >>> >> + UID_FEATURE | >> >>> >> + REFERENCING_FEATURE | >> >>> >> + >> >>> >> +DEFAULT_IMAGE_FEATURE); } >> >>> >> + >> >>> >> static int pbap_probe(struct obc_session *session) { >> >>> >> struct pbap_data *pbap; >> >>> >> @@ -1274,6 +1309,7 @@ static struct obc_driver pbap = { >> >>> >> .uuid = PBAP_UUID, >> >>> >> .target = OBEX_PBAP_UUID, >> >>> >> .target_len = OBEX_PBAP_UUID_LEN, >> >>> >> + .supported_features = pbap_supported_features, >> >>> >> .probe = pbap_probe, >> >>> >> .remove = pbap_remove >> >>> >> }; >> >>> >> diff --git a/obexd/client/session.c b/obexd/client/session.c >> >>> >> index 9bba6c6..d2ae4fd 100644 >> >>> >> --- a/obexd/client/session.c >> >>> >> +++ b/obexd/client/session.c >> >>> >> @@ -345,6 +345,7 @@ static void transport_func(GIOChannel *io, >> >>> >> GError *err, gpointer user_data) >> >>> >> struct obc_driver *driver = session->driver; >> >>> >> struct obc_transport *transport = session->transport; >> >>> >> GObex *obex; >> >>> >> + GObexApparam *apparam; >> >>> >> GObexTransportType type; >> >>> >> int tx_mtu = -1; >> >>> >> int rx_mtu = -1; >> >>> >> @@ -370,7 +371,29 @@ static void transport_func(GIOChannel *io, >> >>> >> GError *err, gpointer user_data) >> >>> >> >> >>> >> g_io_channel_set_close_on_unref(io, TRUE); >> >>> >> >> >>> >> - if (driver->target != NULL) >> >>> >> + apparam = NULL; >> >>> >> + >> >>> >> + if (driver->supported_features) >> >>> >> + apparam = driver->supported_features(session); >> >>> >> + >> >>> >> + if (apparam) { >> >>> >> + uint8_t buf[1024]; >> >>> >> + ssize_t len; >> >>> >> + >> >>> >> + len = g_obex_apparam_encode(apparam, buf, sizeof(buf)); >> >>> >> + if (driver->target) >> >>> >> + g_obex_connect(obex, connect_cb, callback, &err, >> >>> >> + G_OBEX_HDR_TARGET, >> >>> >> + driver->target, driver->target_len, >> >>> >> + G_OBEX_HDR_APPARAM, >> >>> >> + buf, len, >> >>> >> + G_OBEX_HDR_INVALID); >> >>> >> + else >> >>> >> + g_obex_connect(obex, connect_cb, callback, &err, >> >>> >> + G_OBEX_HDR_APPARAM, buf, len, >> >>> >> + G_OBEX_HDR_INVALID); >> >>> >> + g_obex_apparam_free(apparam); >> >>> >> + } else if (driver->target) >> >>> >> g_obex_connect(obex, connect_cb, callback, &err, >> >>> >> G_OBEX_HDR_TARGET, driver->target, driver- >> >>> >> >target_len, >> >>> >> G_OBEX_HDR_INVALID); >> >>> >> -- >> >>> >> 1.9.3 >> >>> >> >> >>> >> -- >> >>> >> To unsubscribe from this list: send the line "unsubscribe linux- >> bluetooth" >> >>> > in >> >>> >> the body of a message to majordomo@vger.kernel.org More >> majordomo >> >>> >> info at http://vger.kernel.org/majordomo-info.html >> >>> > >> >>> > >> >>> > >> >>> > I applied this patch locally and tested with PTS. >> >>> > The PTS test case: TP/SSM/BV-09-C [PCE Shares >> PbapSupportedFeature >> >>> > bits] is failing. >> >>> > According to PBAP V1.2 Spec - Section 6.4 - OBEX Connect request >> >>> > format, the PbapSupportedFeatures are shared during obex >> >>> > connect(after applying the this patch). >> >>> > Still the above test case is failing. >> >>> > Am I mapping anything wrong? >> >>> >> >>> Have you applied the full set? Maybe the PCE record needs updating >> >>> as well, but normally it is not mandatory. >> >>> >> >>> >> >>> -- >> >>> Luiz Augusto von Dentz >> >>> -- >> >>> To unsubscribe from this list: send the line "unsubscribe >> >>> linux-bluetooth" in the body of a message to >> >>> majordomo@vger.kernel.org More majordomo info at >> >>> http://vger.kernel.org/majordomo-info.html >> >> >> >> >> >> Yes, I applied the complete set and tested it. Still that TC is failing. >> >> Just want to know how it is failing, because I tried >> >> 1) Applying this patch set - not passing >> >> 2) Added below attribute in the src/profile PCE record at the end as below >> - not passing >> >> >> >> >> >> >> >> >> >> May be it’s a PTS Issue. Anyway it's not mandatory. >> > >> > I will try it as well, btw does the logs indicate anything useful? >> >> Btw, what TC is failing? >> > > The PTS test case: TP/SSM/BV-09-C [PCE Shares PbapSupportedFeature bits] is failing. > Test objective is to verify that the PCE can share its PbapSupportedFeatures bits with a server. > > Steps I followed: > 1) Ran Bluetoothd > 2) Ran obexd > 3) Initiated the above mentioned TC in PTS tool. > 4) ./pbap-client 00:1B:DC:07:33:4E >> err.txt > > Bluetoothd logs: > > bluetoothd[2740]: src/adapter.c:connected_callback() hci0 device 00:1B:DC:07:33:4E connected eir_len 21 > bluetoothd[2740]: src/adapter.c:dev_disconnected() Device 00:1B:DC:07:33:4E disconnected, reason 2 > bluetoothd[2740]: src/adapter.c:adapter_remove_connection() > bluetoothd[2740]: plugins/policy.c:disconnect_cb() reason 2 > bluetoothd[2740]: src/adapter.c:bonding_attempt_complete() hci0 bdaddr 00:1B:DC:07:33:4E type 0 status 0xe > bluetoothd[2740]: src/device.c:device_bonding_complete() bonding (nil) status 0x0e > bluetoothd[2740]: src/device.c:device_bonding_failed() status 14 > bluetoothd[2740]: src/adapter.c:resume_discovery() > > Obexd logs: Nothing while executing the TC It is probably better if you can use obexctl> connect
pbap, also it would be good to have the HCI traces to be able to detect if PTS is really setting the record properly, for example if it is not advertising 0x0102 in the record we will not attempt to send the parameters.