[6tisch] Jonathan's review on draft-ietf-6tisch-6top-interface-01

Jonathan Simon <jsimon@linear.com> Sat, 25 October 2014 00:31 UTC

From: Jonathan Simon <jsimon@linear.com>
Content-Type: multipart/alternative; boundary="Apple-Mail=_F3220487-B648-4B7C-9AEA-A585E1E0B8A6"
Message-Id: <313BE5D8-738C-49A3-BEE3-0DA050528F0A@linear.com>
Date: Fri, 24 Oct 2014 17:31:33 -0700
To: 6tisch@ietf.org
Mime-Version: 1.0 (Mac OS X Mail 7.3 \(1878.6\))
Archived-At: http://mailarchive.ietf.org/arch/msg/6tisch/PGsQAewOZxsfLzbMuqmZ0iYA-1w
Subject: [6tisch] Jonathan's review on draft-ietf-6tisch-6top-interface-01
Precedence: list

Finally got around to reviewing this. Better late than never?

p2 - odd sentence “This document also defines a list commands internal to the 6top sublayer.”

p5 - CellType option is mandatory (2.1), but it seems to also be implicit in how the cell is created (2.1.1 & 2.1.2). Does MUST in these sections mean that this option must be set explicitly?

p7 - what does unicast (resp.broadcast) mean?

p9 - channelOffset should be 16-bit (per 802.15.4e-2012 table 52d)

p13 & 14 - the metrics listed do not correspond to those defined in 802.15.4e-2012 table 4i. PDR, ETX, RSSI and LQI are not used in TSCH or macMetrics descriptions.
- 4e defines a counter size, macCounterOctets, that needs to be specified by 6top - could be a fixed value or set via YANG.
- “Window” measurement period is specified in "Number of the slotframe size” - Does “Number” here mean multiples? what if metrics apply to all slotframes?

p15 - “Peroid” instead of period.
- CellID - cell ID’s are local to a particular slotframe, so this is meaningless without a slotframe ID
- If the period is in seconds, and the cell repeat rate is not, how is this supposed to work?

p 17 - why is last comms ASN being stored?

general questions/comments:

* Are all things in the YANG model required? For example, must one support reading the READ.timesource for minTimeCorrection? Must all nodes support chunks, and tracks, and softcells, labelSwitching, or is this policy?
* Are all things in the YANG model expected to be encoded and sent over the air?
* There is no mechanism to specify additional IEs in an EB - is this outside of YANG?
* There’s no way to set the PANID.

* Certain PIBs must be set in a generic 15.4e MAC to get TSCH behavior. I don’t know if these are needed in YANG, but need to be in 6TiSCH:
** some Table 52n on EBs entries need to be set
** Table 60 macFrameCounterMode needs to be 5 octets for ASN to work
** The fields needed to form the IEs contained in the EB are specified, but whether they are sent explicitly or by ID isn’t defined
** macTSCHcapable, and macTSCHEnabled, macMetricsCapable, macMetricsEnabled, aren’t discussed.
** There is no explicit configuration of EBs (e.g. macUseEnhancedBeacon) at the MAC or a discussion of MLME-BEACON.request.
--
Jonathan Simon, Ph. D
Director of Systems Engineering
Linear Technology, Dust Networks product group
32990 Alvarado-Niles Road, Suite 910
Union City, CA 94587
(510) 400-2936
(510) 489-3799 FAX
jsimon@linear.com

**LINEAR TECHNOLOGY CORPORATION**
*****Internet Email Confidentiality Notice*****
This e-mail transmission, and any documents, files or previous e-mail messages attached to it may contain confidential information that is legally privileged. If you are not the intended recipient, or a person responsible for delivering it to the intended recipient, you are hereby notified that any disclosure, copying, distribution or use of any of the information contained in or attached to this transmission is STRICTLY PROHIBITED. If you have received this transmission in error, please immediately notify me by reply e-mail, or by telephone at (510) 400-2936, and destroy the original transmission and its attachments without reading or saving in any manner. Thank you.

[6tisch] Jonathan's review on draft-ietf-6tisch-6… Jonathan Simon
Re: [6tisch] Jonathan's review on draft-ietf-6tis… Xavier Vilajosana
Re: [6tisch] Jonathan's review on draft-ietf-6tis… Thomas Watteyne
Re: [6tisch] Jonathan's review on draft-ietf-6tis… Pascal Thubert (pthubert)