Re: [AVTCORE] draft-ietf-avtcore-rfc5285-bis-11.txt - new text for bundle case

Magnus Westerlund <magnus.westerlund@ericsson.com> Tue, 23 May 2017 12:57 UTC

Return-Path: <magnus.westerlund@ericsson.com>
X-Original-To: avt@ietfa.amsl.com
Delivered-To: avt@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id DF012129B2A for <avt@ietfa.amsl.com>; Tue, 23 May 2017 05:57:37 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.521
X-Spam-Level:
X-Spam-Status: No, score=-1.521 tagged_above=-999 required=5 tests=[BAYES_50=0.8, RCVD_IN_DNSWL_MED=-2.3, RCVD_IN_MSPIKE_H3=-0.01, RCVD_IN_MSPIKE_WL=-0.01, SPF_PASS=-0.001] autolearn=ham autolearn_force=no
Received: from mail.ietf.org ([4.31.198.44]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id IFLvmue3a5Tl for <avt@ietfa.amsl.com>; Tue, 23 May 2017 05:57:35 -0700 (PDT)
Received: from sesbmg22.ericsson.net (sesbmg22.ericsson.net [193.180.251.48]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id D808E1243F3 for <avt@ietf.org>; Tue, 23 May 2017 05:57:34 -0700 (PDT)
X-AuditID: c1b4fb30-4163b9a0000007db-35-592431bdd1b8
Received: from ESESSHC013.ericsson.se (Unknown_Domain [153.88.183.57]) by sesbmg22.ericsson.net (Symantec Mail Security) with SMTP id 4B.3D.02011.DB134295; Tue, 23 May 2017 14:57:33 +0200 (CEST)
Received: from [127.0.0.1] (153.88.183.153) by smtp.internal.ericsson.com (153.88.183.59) with Microsoft SMTP Server id 14.3.339.0; Tue, 23 May 2017 14:57:32 +0200
To: Roni Even <roni.even@huawei.com>, "avt@ietf.org" <avt@ietf.org>
References: <6E58094ECC8D8344914996DAD28F1CCD7CC7D1@DGGEMM506-MBX.china.huawei.com>
From: Magnus Westerlund <magnus.westerlund@ericsson.com>
Message-ID: <9f3be17d-1306-bca3-13eb-5aef99d4a684@ericsson.com>
Date: Tue, 23 May 2017 14:57:31 +0200
User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:52.0) Gecko/20100101 Thunderbird/52.1.1
MIME-Version: 1.0
In-Reply-To: <6E58094ECC8D8344914996DAD28F1CCD7CC7D1@DGGEMM506-MBX.china.huawei.com>
Content-Type: text/plain; charset="utf-8"; format="flowed"
Content-Language: sv
Content-Transfer-Encoding: 8bit
X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFjrALMWRmVeSWpSXmKPExsUyM2K7pe5eQ5VIgz9f+S1e9qxkt/h07DyL A5NHy5G3rB5LlvxkCmCK4rJJSc3JLEst0rdL4MpoOL6XraBHqeLuhfUsDYxXpLsYOTkkBEwk jt+9xNTFyMUhJHCEUWL7sofMEM5yRomWlUvZQKqEBXwl5u9qZu9i5OAQEXCWuLI6AiQsJBAs cf/uM3YQm03AQuLmj0awcl4Be4nWRxA2i4CqxPPXq8BqRAViJB5tOMsEUSMocXLmExYQm1Mg RGLy5MvMIDYz0JyZ888zQtgiEicv34eKy0s0b53NDLFXW6KhqYN1AqPALCSjZiFpn4WkfRaS 9gWMLKsYRYtTi5Ny042M9FKLMpOLi/Pz9PJSSzYxAsP14JbfBjsYXz53PMQowMGoxMPbBwxj IdbEsuLK3EOMEhzMSiK8Vw2AQrwpiZVVqUX58UWlOanFhxilOViUxHkd912IEBJITyxJzU5N LUgtgskycXBKNTBG21/ctON4FlP9F7Eta1N/6u2tWnTv4OtmpZy7TB9nCc+paG7fctesadYM +/WsPmW5yyYsynyy/tRh+zm31VdsvnUum2ndlxu6K5+s9g7JF2R6X5veXMUpnfcv9VR4zL5r TmcWOKzccSAwJzPaVOqhRWbvg9sp26/IC32wdErg1d/VfIV1QYyoEktxRqKhFnNRcSIAb74M vVMCAAA=
Archived-At: <https://mailarchive.ietf.org/arch/msg/avt/SOj0HlLyU3UVNauVaLW9kYZeFvM>
Subject: Re: [AVTCORE] draft-ietf-avtcore-rfc5285-bis-11.txt - new text for bundle case
X-BeenThere: avt@ietf.org
X-Mailman-Version: 2.1.22
Precedence: list
List-Id: Audio/Video Transport Core Maintenance <avt.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/avt>, <mailto:avt-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/avt/>
List-Post: <mailto:avt@ietf.org>
List-Help: <mailto:avt-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/avt>, <mailto:avt-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 23 May 2017 12:57:38 -0000

Hi Roni,

Thanks for drafting text. See below for my comments.

Den 2017-05-18 kl. 13:10, skrev Roni Even:
> Hi,
> 
> I suggest adding the following text to section 6
> 
> When used in [bundle] this attribute is specified as normal category
> for the [mux attribute]. This allows for only a subset of the m-lines
> in the bundle group to offer extmap-allow-mixed. When an answerer
> supporting the extmap-allow-mix attribute receives an offer where
> only some of the m-lines in the bundle group include the
> extmap-allow-mixed attribute, the answerer must receive this offer
> and support mixed one-byte and two-bytes only for those m-lines.

I think we should clarify what "support" means. I think the reasonable 
thing that this means is that one MUST only send RTP header extensions 
using mixed on those RTP streams originating from those media sources 
(m=) blocks that includes extmap-allow-mixed, and are RECOMMENDED to 
support receiving mixed on all RTP streams being received in an RTP 
session where at least one bundled m= block is indicating 
extmap-allow-mixed. This to maximize interoperability.

> 
> 
> 
> And the following text to section 7
> 
> When using [bundle] to bundle multiple m-lines the extmap attribute
> falls under the special category of [mux attribute]. All the m-lines
> in a bundle group are considered to be part of the same local
> identifier (ID) space.  If an extension URI is offered in multiple
> m-lines the same attributes as part of the same bundle group it MUST
> use the same ID in all this m-lines. Each m-line in a bundle group
> can include different extension URIs allowing for example audio and
> video sources to use different sets of RTP header extensions. The
> directionality of the RTP header extensions in each m-line of the
> bundle group are handled as the non-bundled case. This allows for
> specifying different directionality for each of the repteated
> extension URI in bundled group.

This is not discussing the <extensionattributes> that the extmap 
attribute may include.

https://tools.ietf.org/html/rfc5484 does actually define extension 
attributes. What I understand of these extension attributes, two extmaps 
with same URN but with different extensionattributes are not compatible, 
and need different IDs. I would suggest the following edits:

When using [bundle] to bundle multiple m-lines the extmap attribute 
falls under the special category of [mux attribute].
All the m-lines in a bundle group are considered to be part of the same 
local identifier (ID) space.  If an RTP header extension, i.e. an 
particular extension URI and configuration using <extensionattributes>, 
is offered in multiple m-lines that are part of the same bundle group it 
MUST use the same ID in all of these m-lines. Each m-line in a bundle 
group can include different RTP header extensions allowing for example 
audio and video sources to use different sets of RTP header extensions.
It SHALL be assumed that any difference in configuration using any 
<extensionattributes> has importance and need to be preserved to any 
receiver, and thus require assignment of different IDs. Any RTP header 
extension that do not match this assumption MUST explicitly indicate 
this provide rules for what are compatible configurations that can be 
sent with the same ID. The directionality of the RTP header extensions 
in each m-line of the bundle group are handled as the non-bundled case. 
This allows for specifying different directionality for each of the 
repteated extension URI in bundled group.

I note that the sentence:

Any RTP header extension that do not match this assumption MUST 
explicitly indicate this provide rules for what are compatible 
configurations that can be sent with the same ID.

Likely should be added to the registation checklist in 10.1.

Cheers

Magnus Westerlund

----------------------------------------------------------------------
Media Technologies, Ericsson Research
----------------------------------------------------------------------
Ericsson AB                 | Phone  +46 10 7148287
Färögatan 6                 | Mobile +46 73 0949079
SE-164 80 Stockholm, Sweden | mailto: magnus.westerlund@ericsson.com
----------------------------------------------------------------------