[BlueZ,v5,0/4] mesh: Always deliver tokens via JoinComplete

Message

Michał Lowas-Rzechonek April 14, 2020, 8:55 a.m. UTC

This patchset implements API change discussed in
https://marc.info/?l=linux-bluetooth&m=157660821400352&w=2

v5:
 - Updated wording in doc/mesh-api.txt
 - Rearranged patches to include error checking fix and logs in
   patch #2

v4:
 - Restore original C parameters on the response for dbus
   send_with_reply.  Any parameters can now be "returned" by the App,
   and a NULL message indicates an internally generated time-out

v3:
 - Add distribution independant timeout for JoinComplete() method calls
   such that a 30 second timeout of the call is treated as failure, so
   stale nodes clean up correctly

v2:
 - Fix mesh-cfgclient to send JoinComplete reply before calling Attach

Brian Gix (1):
  mesh: Add Time-outs to critical dbus send-with-replies

Michał Lowas-Rzechonek (1):
  doc/mesh: Change API to deliver tokens via JoinComplete

Przemysław Fierek (2):
  mesh: Change API to deliver tokens via JoinComplete
  tools/mesh-cfgclient: Add waiting for 'JoinComplete'

 doc/mesh-api.txt       | 30 +++++++++-------
 mesh/dbus.c            | 40 ++++++++++++++++++++++
 mesh/dbus.h            |  6 ++++
 mesh/mesh.c            | 68 ++++++++++++++++++++++++++++--------
 tools/mesh-cfgclient.c | 78 +++++++++++++++++++++++++-----------------
 5 files changed, 162 insertions(+), 60 deletions(-)

Comments

Brian Gix April 14, 2020, 3:47 p.m. UTC | #1

Patchset applied
On Tue, 2020-04-14 at 10:55 +0200, Michał Lowas-Rzechonek wrote:
> This patchset implements API change discussed in
> https://marc.info/?l=linux-bluetooth&m=157660821400352&w=2
> 
> v5:
>  - Updated wording in doc/mesh-api.txt
>  - Rearranged patches to include error checking fix and logs in
>    patch #2
> 
> v4:
>  - Restore original C parameters on the response for dbus
>    send_with_reply.  Any parameters can now be "returned" by the App,
>    and a NULL message indicates an internally generated time-out
> 
> v3:
>  - Add distribution independant timeout for JoinComplete() method calls
>    such that a 30 second timeout of the call is treated as failure, so
>    stale nodes clean up correctly
> 
> v2:
>  - Fix mesh-cfgclient to send JoinComplete reply before calling Attach
> 
> Brian Gix (1):
>   mesh: Add Time-outs to critical dbus send-with-replies
> 
> Michał Lowas-Rzechonek (1):
>   doc/mesh: Change API to deliver tokens via JoinComplete
> 
> Przemysław Fierek (2):
>   mesh: Change API to deliver tokens via JoinComplete
>   tools/mesh-cfgclient: Add waiting for 'JoinComplete'
> 
>  doc/mesh-api.txt       | 30 +++++++++-------
>  mesh/dbus.c            | 40 ++++++++++++++++++++++
>  mesh/dbus.h            |  6 ++++
>  mesh/mesh.c            | 68 ++++++++++++++++++++++++++++--------
>  tools/mesh-cfgclient.c | 78 +++++++++++++++++++++++++-----------------
>  5 files changed, 162 insertions(+), 60 deletions(-)
>