Re: [payload] Review of draft-ietf-payload-flexible-fec-scheme-08
Giridhar Mandyam <mandyam@qti.qualcomm.com> Fri, 26 October 2018 20:12 UTC
Return-Path: <mandyam@qti.qualcomm.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 31779130DDA;
Fri, 26 Oct 2018 13:12:40 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -4.301
X-Spam-Level:
X-Spam-Status: No, score=-4.301 tagged_above=-999 required=5
tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1,
DKIM_VALID_AU=-0.1, RCVD_IN_DNSWL_MED=-2.3, SPF_PASS=-0.001]
autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key)
header.d=qti.qualcomm.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 J7LUwULw4RU4; Fri, 26 Oct 2018 13:12:37 -0700 (PDT)
Received: from alexa-out-sd-02.qualcomm.com (alexa-out-sd-02.qualcomm.com
[199.106.114.39])
(using TLSv1.2 with cipher AES256-SHA (256/256 bits))
(No client certificate requested)
by ietfa.amsl.com (Postfix) with ESMTPS id B0FF4126F72;
Fri, 26 Oct 2018 13:12:37 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple;
d=qti.qualcomm.com; i=@qti.qualcomm.com; q=dns/txt;
s=qcdkim; t=1540584757; x=1572120757;
h=from:to:subject:date:message-id:references:in-reply-to:
content-transfer-encoding:mime-version;
bh=vCb9zeb7uGyCP/Ov+JXONwAWXqEF/0eX42iH4vSNETU=;
b=WZOzQ90lr+Okqde+kwNPXe25tv0vq+JoFkvCSz3gmcXn0FzJQrPt0/YA
wsDy3YEJcnY6gp2KnKYcYsG2JcKrQBI1gaDytg4BPOH7SJwix+A2zUiid
zrVKVH95pqdxIg4/iEyndzSrl2jTZUWPVPtcitFpNR7PfwQUp+jStvkp0 8=;
X-IronPort-AV: E=Sophos;i="5.54,429,1534834800"; d="scan'208";a="14994089"
Received: from unknown (HELO ironmsg-SD-alpha.qualcomm.com) ([10.53.140.30])
by alexa-out-sd-02.qualcomm.com with ESMTP; 26 Oct 2018 13:12:37 -0700
X-IronPort-AV: E=McAfee;i="5900,7806,9058"; a="291750545"
Received: from nasanexm01h.na.qualcomm.com ([10.85.0.34])
by ironmsg-SD-alpha.qualcomm.com with ESMTP/TLS/AES256-SHA;
26 Oct 2018 13:12:37 -0700
Received: from NASANEXM01C.na.qualcomm.com (10.85.0.83) by
NASANEXM01H.na.qualcomm.com (10.85.0.34) with Microsoft SMTP Server (TLS) id
15.0.1365.1; Fri, 26 Oct 2018 13:12:36 -0700
Received: from NASANEXM01C.na.qualcomm.com ([10.85.0.83]) by
NASANEXM01C.na.qualcomm.com ([10.85.0.83]) with mapi id 15.00.1365.000; Fri,
26 Oct 2018 13:12:36 -0700
From: Giridhar Mandyam <mandyam@qti.qualcomm.com>
To: Magnus Westerlund <magnus.westerlund@ericsson.com>, "payload@ietf.org"
<payload@ietf.org>, "draft-ietf-payload-flexible-fec-scheme@ietf.org"
<draft-ietf-payload-flexible-fec-scheme@ietf.org>
Thread-Topic: [payload] Review of draft-ietf-payload-flexible-fec-scheme-08
Thread-Index: AQHUHfs2dLqv1xjOGUmpqPmxPl5RQ6UyXWFg
Date: Fri, 26 Oct 2018 20:12:36 +0000
Message-ID: <f651d896ce7c4c0eb041de8624de512a@NASANEXM01C.na.qualcomm.com>
References: <633bb439-9c53-2cd7-b34a-1071ad0dbaa8@ericsson.com>
<DB7PR07MB49888CAC73A6172A435404F895F70@DB7PR07MB4988.eurprd07.prod.outlook.com>
In-Reply-To: <DB7PR07MB49888CAC73A6172A435404F895F70@DB7PR07MB4988.eurprd07.prod.outlook.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-ms-exchange-transport-fromentityheader: Hosted
x-originating-ip: [10.80.80.8]
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
Archived-At: <https://mailarchive.ietf.org/arch/msg/payload/6EbOBMqHFXa3-CfxFZU8tvGAMa4>
Subject: Re: [payload] Review of draft-ietf-payload-flexible-fec-scheme-08
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: Fri, 26 Oct 2018 20:12:40 -0000
Hello Magnus, Thank you for your continued detailed review. Our apologies for not responding to you prior to posting the latest revision. We were trying to get it uploaded prior to the submission tool closing. Before posting a new revision, I would like to get confirmation from you on the necessary changes. -Giri Mandyam <--------> Original comment: Section 4.2.2.3: Are there any assumptions about the relation between repair RTP stream SSRC and the source RTP stream SSRC? The text here appears to indicate that it can be 1 (repair stream) sending retransmission packets for many source RTP streams? Can you please clarify this. Are there a point of doing this type of retransmission in a 1 to 1 fashion instead? Revised text: "Note that the retransmission packet corresponds only to a single source SSRC." Feedback: > I don't understand what this sentence means. I think you need either of two statements to make this clear. ... Proposed replacement text: "When performing retransmissions of single source packets, a unique repair packet stream (SSRC) MUST be used for each source packet stream (SSRC) to enable efficient handling after the first initial repair packet on each SSRC." Author's response: Your option A) better encompassed our original intent. <--------> Original comment: " o ToP: indicates the type of protection applied by the sender: ... " What about retransmission only uses? And what about combinations with retransmission and other types? Revised text: "ToP: indicates the type of protection applied by the sender: ... 3 for retranmission" Feedback: > Note spelling error in retran(s)mission! However, this enables a PT to be configured for retransmission use. It still doesn't make it clear if ToP=0,3 would be allowed or not. I don't see any problem in requiring a single value per PT. But please be explicit about that limitation. Proposed replacement text: Correct spelling error, and add "There can only be one value listed for ToP." Author's response: A single value of ToP per payload type was the original intent. <--------> Original comment: Section 7. I think this section is missing the discussion of the need to signal the RTP session relation between source and repair RTP Streams. ... Revised text: " Out-of-band signaling should be designed to enable the receiver to identify the RTP streams associated with source packets and repair packets, respectively. ..." Feedback: > I think the above text is actually making the requirements stricter than necessary and also beyond SDP's capability in some cases depending on its use. Proposed replacement text: Eliminate " Note that other approaches to RTP stream identification SHOULD NOT be used for the purposes of FLEX FEC." in revision Author's response: Agree that original text is overly restrictive. -----Original Message----- From: Magnus Westerlund <magnus.westerlund@ericsson.com> Sent: Thursday, October 25, 2018 5:34 AM To: payload@ietf.org; draft-ietf-payload-flexible-fec-scheme@ietf.org Subject: Re: [payload] Review of draft-ietf-payload-flexible-fec-scheme-08 Hi, I have looked at the new version -09 and see if it addresses my issues: On 2018-07-17 20:22, Magnus Westerlund wrote: G. Section 4.2.2.3: Are there any assumptions about the relation between repair RTP stream SSRC and the source RTP stream SSRC? The text here appears to indicate that it can be 1 (repair stream) sending retransmission packets for many source RTP streams? Can you please clarify this. Are there a point of doing this type of retransmission in a 1 to 1 fashion instead? So what I understand what got changed due to this comment was: Note that the retransmission packet corresponds only to a single source SSRC. I don't understand what this sentence means. I think you need either of two statements to make this clear. A) When performing retransmissions of single source packets, a unique repair packet stream (SSRC) MUST be used for each source packet stream (SSRC) to enable efficient handling after the first initial repair packet on each SSRC. B) When performing retransmissions, a single repair packet stream (SSRC) MAY be used for retransmitting packets from multiple source packet streams (SSRCs). I prefer A as it enables smarter processing for those who want to, but as each retransmission packet are self identifying, an implementation can choces to basically do what B) implies, namely determine first that it is a retransmission packet, then decapsulate it and restore the retransmitted packet and then handle it as a totally new incoming packet that need to be routed to the packet stream. A) enables an implementation freedom to do early binding on the associated repair stream SSRC. H. Section 5.1: o ToP: indicates the type of protection applied by the sender: 0 for 1-D interleaved FEC protection, 1 for 1-D non-interleaved FEC protection, and 2 for 2-D parity FEC protection. The ToP value of 3 is reserved for future uses. What about retransmission only uses? And what about combinations with retransmission and other types? The change: o ToP: indicates the type of protection applied by the sender: 0 for 1-D interleaved FEC protection, 1 for 1-D non-interleaved FEC protection, 2 for 2-D parity FEC protection, and 3 for retranmission. Note spelling error in retran(s)mission! However, this enables a PT to be configured for retransmission use. It still doesn't make it clear if ToP=0,3 would be allowed or not. I don't see any problem in requiring a single value per PT. But please be explicit about that limitation. N. Section 7: I think this section is missing the discussion of the need to signal the RTP session relation between source and repair RTP Streams. There is basically two choices here, either the repair RTP stream(s) are in the same RTP Session as the source packets, or they are in a different one. When they are in the same, it is clear that the packets carry the SSRC(s) of the source RTP packets. However, one thing is a bit unclear and could vary with implementation and that is the number of repair RTP streams in use when there are multiple source RTP streams. Are there only one repair stream, or one per source RTP stream or some other relation? So example 7.1 could be one or more source RTP stream but nothing in the signalling reveals this. Signalling context can make it clear if there is a single media source or could be more. For 7.2 there is an explicit binding between the repair and source SSRCs. I think the need and not need to explicitly indicate the source to repair stream relations. For different RTP sessions there need to be some type of RTP/RTCP out of band mechanism to indicate the relation. The use of grouping of media lines FEC semantics is very reasonable and should be made clear and probably include an example. I would prefer if the signalling requirements where discussed first, followed by SDP related mechanisms then followed by the examples of the realization. Good initial stab on the signalling requirements. 7. Signaling Requirements Out-of-band signaling should be designed to enable the receiver to identify the RTP streams associated with source packets and repair packets, respectively. At a minimum, the signaling must be designed to allow the receiver to o Determine whether one or more source RTP streams will be sent. o Determine whether one or more repair RTP streams will be sent. o Associate the appropriate SSRC's to both source and repair streams. o Clearly identify which SSRC's are associated with each source block. o Clearly identify which repair packets correspond to which source blocks. o Make use of repair packets to recover source data associated with specific SSRC's. This section provides several Sesssion Description Protocol (SDP) examples to demonstrate how these requirements can be met. Note that other approaches to RTP stream identification SHOULD NOT be used for the purposes of FLEX FEC. I think the above text is actually making the requirements stricter than necessary and also beyond SDP's capability in some cases depending on its use
- [payload] Review of draft-ietf-payload-flexible-f… Magnus Westerlund
- Re: [payload] Review of draft-ietf-payload-flexib… Magnus Westerlund
- Re: [payload] Review of draft-ietf-payload-flexib… Giridhar Mandyam
- Re: [payload] Review of draft-ietf-payload-flexib… Magnus Westerlund
- Re: [payload] Review of draft-ietf-payload-flexib… Mo Zanaty (mzanaty)