All of lore.kernel.org
 help / color / mirror / Atom feed
From: Florian Grandel <fgrandel@gmail.com>
To: Arman Uguray <armansito@chromium.org>,
	Marcel Holtmann <marcel@holtmann.org>,
	Johan Hedberg <johan.hedberg@gmail.com>
Cc: BlueZ development <linux-bluetooth@vger.kernel.org>
Subject: Multi-adv patchset assumptions
Date: Sat, 13 Jun 2015 06:37:00 +0200	[thread overview]
Message-ID: <557BB36C.3010408@gmail.com> (raw)

Hi guys,

as you can see, I now finished the patch set that provides the full 
multi-adv feature for you to review and try out.

I made a few minor extra assumptions about multi-adv implementation 
details that IMO had not yet been specified in mgmt-api.txt and 
hopefully do not break backwards compatibility, notably:

- I arbitrarily set the default duration to 2 secs. This can of course
   be set to any other value you prefer.

- Switching the LE setting off permanently disables and removes all
   advertising instances - even if the device remains powered.

- Adding an instance while another instance is being advertised will
   wait for the new instance's scheduled time to first advertise that
   instance i.e. it may be that one or more other instances will have
   their turn first (FIFO). If the currently advertised instance is being
   changed, though, then that instance will be updated immediately.

- Removing an advertising instance that is currently being advertised
   will cancel the instance right away and not wait for it to time out.
   If there are other remaining advertising instances, then those will
   go on to advertise instead.

- After service discovery or after "set advertising off", instance
   advertising should re-enter its prior advertising state, i.e.
   advertise the same instance that has been advertised before, if any.

- Whenever an advertising instance is canceled prematurely (set adv,
   service discovery, etc.) only the remaining duration of the current
   advertising slot is "lost". The instance will not be deleted and the
   remaining advertising time (up to the overall timeout) will be
   advertised as soon as instance advertising is being re-enabled.

Most of these assumptions have been documented in mgmt-api.txt, too. You 
can look at the diff there for easier review.

Florian

                 reply	other threads:[~2015-06-13  4:37 UTC|newest]

Thread overview: [no followups] expand[flat|nested]  mbox.gz  Atom feed

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=557BB36C.3010408@gmail.com \
    --to=fgrandel@gmail.com \
    --cc=armansito@chromium.org \
    --cc=johan.hedberg@gmail.com \
    --cc=linux-bluetooth@vger.kernel.org \
    --cc=marcel@holtmann.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.