All of lore.kernel.org
 help / color / mirror / Atom feed
From: Steve Brown <sbrown@cortland.com>
To: Johan Hedberg <johan.hedberg@gmail.com>,
	Steve Brown <sbrown@cortland.com>
Cc: "linux-bluetooth@vger.kernel.org" <linux-bluetooth@vger.kernel.org>
Subject: Re: meshctl: Can't connect via Node Identity advertisement
Date: Mon, 04 Dec 2017 06:06:42 -0700	[thread overview]
Message-ID: <1512392802.27418.8.camel@ewol.com> (raw)
In-Reply-To: <20171204123606.GA29257@x1c.lan>

Hi Johan,

On Mon, 2017-12-04 at 14:36 +0200, Johan Hedberg wrote:
> Hi Steve,
> 
> On Sun, Dec 03, 2017, Steve Brown wrote:
> > Connect via Net ID works without a problem.
> > 
> > I'm using meshctl with zephyr's mesh_shell (upstream version)
> > 
> > I provision the shell and turn on it's gatt-proxy. Then I exit
> > meshctl,
> > run it again and issue a "connect 0". It reconnects to the mesh
> > without
> >  a problem.
> > 
> > If I turn off the shell's gatt-proxy, start meshctl, issue a 
> > "connect 0" and then issue a shell "ident". I get no connection.
> > The
> > ident command only transmits advertisements for a short period of
> > time.
> > 
> > The CONN_TYPE_IDENTITY path in parse_mesh_service_data() seems
> > broken.
> > The comparison of data[0] and connection.type fails. The value of
> > data[0] is 1 (CONN_TYPE_IDENTITY) and the value of connection.type
> > is
> > 0.
> 
> Looking at mesh/main.c it seems to me like the connect command is
> currently only desgined for connecting to Network Identity
> advertisements (which is also implied by the fact that it takes a
> NetKey
> Index as an optional parameter value).
> 
> What would be needed is some command which takes an element address
> rather than a NetKey Index as input. Just thinking out loud, but
> maybe
> "connect" should be renamed to "connect-net" and then a new
> "connect-node" command could be introduced that would look for Node
> Identity advertisements?
> 
> Johan

It's obvious after you pointed it out. I hadn't appreciated the
distinction between the two types of advertisements. I'll try to do
what you suggest.

Steve




  reply	other threads:[~2017-12-04 13:06 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-12-03 23:39 meshctl: Can't connect via Node Identity advertisement Steve Brown
2017-12-04 12:36 ` Johan Hedberg
2017-12-04 13:06   ` Steve Brown [this message]
2017-12-04 13:59   ` Steve Brown
2017-12-04 15:05     ` Johan Hedberg
2017-12-04 15:21       ` Steve Brown

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=1512392802.27418.8.camel@ewol.com \
    --to=sbrown@cortland.com \
    --cc=johan.hedberg@gmail.com \
    --cc=linux-bluetooth@vger.kernel.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.