Re: [payload] draft-ietf-payload-flexible-fec-scheme-14 notes

Ben Campbell <ben@nostrum.com> Mon, 07 January 2019 22:21 UTC

Return-Path: <ben@nostrum.com>
X-Original-To: payload@ietfa.amsl.com
Delivered-To: payload@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 082AA12DDA3; Mon, 7 Jan 2019 14:21:53 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.679
X-Spam-Level:
X-Spam-Status: No, score=-1.679 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_INVALID=0.1, DKIM_SIGNED=0.1, HTML_MESSAGE=0.001, T_SPF_HELO_PERMERROR=0.01, T_SPF_PERMERROR=0.01] autolearn=no autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=fail (1024-bit key) reason="fail (message has been altered)" header.d=nostrum.com
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 48SkIj8enkDw; Mon, 7 Jan 2019 14:21:51 -0800 (PST)
Received: from nostrum.com (raven-v6.nostrum.com [IPv6:2001:470:d:1130::1]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 48161129508; Mon, 7 Jan 2019 14:21:51 -0800 (PST)
Received: from [10.0.1.45] (cpe-70-122-203-106.tx.res.rr.com [70.122.203.106]) (authenticated bits=0) by nostrum.com (8.15.2/8.15.2) with ESMTPSA id x07MLe5f015463 (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128 verify=NO); Mon, 7 Jan 2019 16:21:41 -0600 (CST) (envelope-from ben@nostrum.com)
DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=nostrum.com; s=default; t=1546899703; bh=dzZYm/ICw49l2J2euq5Fi6hvdBPDzeMFBLi2V+5flpI=; h=From:Subject:Date:In-Reply-To:Cc:To:References; b=Q1VN1/DYpzCh/NERPr2ShoKHrRfgWif+gdVBNFk0lT+8JPocnTcpUlzRU6MKnz3RW B1u4JPAePgfGwcbFyr5KQCGQTjYv86E1oszpAGLLKzLIVy3QwW/PN8llsFIdX80fat nvWwNf8h+7Lw0x5LeurbLFM8jfaBUKp1kEwBACm0=
X-Authentication-Warning: raven.nostrum.com: Host cpe-70-122-203-106.tx.res.rr.com [70.122.203.106] claimed to be [10.0.1.45]
From: Ben Campbell <ben@nostrum.com>
Message-Id: <72DAF233-253D-4F94-8462-9AD9C0E50FF5@nostrum.com>
Content-Type: multipart/signed; boundary="Apple-Mail=_64326513-BFD1-4AD5-B6C0-037F8A979E19"; protocol="application/pgp-signature"; micalg="pgp-sha512"
Mime-Version: 1.0 (Mac OS X Mail 12.2 \(3445.102.3\))
Date: Mon, 07 Jan 2019 16:21:39 -0600
In-Reply-To: <32426087082648efb5dedb901d9c1fb9@NASANEXM01C.na.qualcomm.com>
Cc: "Roni Even (A)" <roni.even@huawei.com>, Magnus Westerlund <magnus.westerlund@ericsson.com>, "draft-ietf-payload-flexible-fec-scheme.all@ietf.org" <draft-ietf-payload-flexible-fec-scheme.all@ietf.org>, "payload@ietf.org" <payload@ietf.org>, Adam Roach <adam@nostrum.com>
To: Giridhar Mandyam <mandyam@qti.qualcomm.com>
References: <c98b67a013d94d1d9fe62504153b83b6@NASANEXM01C.na.qualcomm.com> <DB7PR07MB4988ED389E5D133A6806E6B895890@DB7PR07MB4988.eurprd07.prod.outlook.com> <6E58094ECC8D8344914996DAD28F1CCD18C99631@dggemm526-mbx.china.huawei.com> <32426087082648efb5dedb901d9c1fb9@NASANEXM01C.na.qualcomm.com>
X-Mailer: Apple Mail (2.3445.102.3)
Archived-At: <https://mailarchive.ietf.org/arch/msg/payload/RQoD_lc2YX88K4u8scBz8FPpdaE>
Subject: Re: [payload] draft-ietf-payload-flexible-fec-scheme-14 notes
X-BeenThere: payload@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: Audio/Video Transport Payloads working group discussion list <payload.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/payload>, <mailto:payload-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/payload/>
List-Post: <mailto:payload@ietf.org>
List-Help: <mailto:payload-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/payload>, <mailto:payload-request@ietf.org?subject=subscribe>
X-List-Received-Date: Mon, 07 Jan 2019 22:21:53 -0000

Hi Giri,

I’ve looked at the update. I agree with Magnus’s recommendation.

Additionally, when I suggested using “iesg@iest.org <mailto:iesg@iesg.org>” for the contact for IANA registrations, I did not mean to change the “author” field to that for the registrations that include an author. The point is to list the IESG as a contact for future communications, not to attribute authorship credit.

On a related note, in section 5.1.3, it shows “iesg@ietf.org <mailto:iesg@ietf.org>” as “viesg@ietf.org <mailto:viesg@ietf.org>”.

Please submit the new revision when you are ready.

Thanks!

Ben.


> On Jan 7, 2019, at 12:38 PM, Giridhar Mandyam <mandyam@qti.qualcomm.com> wrote:
> 
> Hi All,
> 
> I am in favor of making the change Magnus recommended below.  My apologies – I did not consider all use cases when responding to the AD review.  I will submit a revision next week (with this change and any other requested changes), assuming there are no objections.
> 
> -Giri
> 
> From: Roni Even (A) <roni.even@huawei.com <mailto:roni.even@huawei.com>>
> Sent: Monday, January 7, 2019 3:31 AM
> To: Magnus Westerlund <magnus.westerlund@ericsson.com <mailto:magnus.westerlund@ericsson.com>>; Giridhar Mandyam <mandyam@qti.qualcomm.com <mailto:mandyam@qti.qualcomm.com>>; draft-ietf-payload-flexible-fec-scheme.all@ietf.org <mailto:draft-ietf-payload-flexible-fec-scheme.all@ietf.org>; payload@ietf.org <mailto:payload@ietf.org>; Ben Campbell <ben@nostrum.com <mailto:ben@nostrum.com>>
> Subject: RE: [payload] draft-ietf-payload-flexible-fec-scheme-14 notes
> 
> CAUTION: This email originated from outside of the organization.
> 
> Hi Magnus,
> This also what I thought specially the case of multiple sources.
> Roni Even as Individual
> 
> From: Magnus Westerlund [mailto:magnus.westerlund@ericsson.com <mailto:magnus.westerlund@ericsson.com>]
> Sent: Monday, January 07, 2019 12:26 PM
> To: Giridhar Mandyam; draft-ietf-payload-flexible-fec-scheme.all@ietf.org <mailto:draft-ietf-payload-flexible-fec-scheme.all@ietf.org>; payload@ietf.org <mailto:payload@ietf.org>; Ben Campbell
> Subject: Re: [payload] draft-ietf-payload-flexible-fec-scheme-14 notes
> 
> Hi,
> 
> In regard to the below.
> 
> On 2019-01-03 21:50, Giridhar Mandyam wrote:
> 
> There are a couple of substantive changes based on AD review (versus version -13) of which the group should be aware:
> 
> Sec. 4.2.1.  Under the description of the SSRC field, the following requirement was previously a SHOULD:  “"The repair streams’ SSRC’s CNAME MUST be identical to the CNAME of the source RTP stream(s) that this repair stream protects.”
> 
> 
> There was actually a reason for this SHOULD. That should likely have been written out explicitly. In case of a RTP middlebox adding FEC to one or more streams it forwards towards to a downstream receiver or other middlebox then it can't use the CNAME of the source stream if there are multiple ones from different sources. Secondly, even if it forwards only from one, that SSRC sending the FEC stream probably shouldn't be required to give the impression that it is the source entity doing this. In some cases RTP middleboxes are considering doing this on behalf of the original source, but not always.
> 
> Thus, I am opposed to make this change. I rather add an exception statement after the SHOULD.
> 
> The repair streams’ SSRC’s CNAME SHOULD be identical to the CNAME of the source RTP stream(s) that this repair stream protects. An FEC stream that protects multiple source RTP streams with different CNAMEs uses the CNAME associated with entity generating the FEC stream or the CNAME of the entity on whose behalf it perform the protection operation.
> 
> 
> 
> Cheers
> 
> Magnus Westerlund
> 
> ----------------------------------------------------------------------
> Network Architecture & Protocols, Ericsson Research
> ----------------------------------------------------------------------
> Ericsson AB                 | Phone  +46 10 7148287
> Torshamnsgatan 23           | Mobile +46 73 0949079
> SE-164 80 Stockholm, Sweden | mailto: magnus.westerlund@ericsson.com <mailto:magnus.westerlund@ericsson.com>
> ----------------------------------------------------------------------