Re: [payload] Review of draft-ietf-payload-flexible-fec-scheme-08

"Mo Zanaty (mzanaty)" <mzanaty@cisco.com> Sun, 18 November 2018 18:11 UTC

Return-Path: <mzanaty@cisco.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 32ACD12D4E7; Sun, 18 Nov 2018 10:11:07 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -14.97
X-Spam-Level:
X-Spam-Status: No, score=-14.97 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.47, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, RCVD_IN_DNSWL_HI=-5, SPF_PASS=-0.001, URIBL_BLOCKED=0.001, USER_IN_DEF_DKIM_WL=-7.5] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=cisco.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 7O9v6N0JKvTp; Sun, 18 Nov 2018 10:11:05 -0800 (PST)
Received: from rcdn-iport-4.cisco.com (rcdn-iport-4.cisco.com [173.37.86.75]) (using TLSv1.2 with cipher DHE-RSA-SEED-SHA (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 7B8F4129BBF; Sun, 18 Nov 2018 10:11:05 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=cisco.com; i=@cisco.com; l=6848; q=dns/txt; s=iport; t=1542564665; x=1543774265; h=from:to:subject:date:message-id:references:in-reply-to: content-id:content-transfer-encoding:mime-version; bh=nn3CC00tFj+C/qphprqep2i+RZO+wNmj5gSfiVQElVM=; b=UaIb2Nq8SkzjLLUPjcwNugTfvpsqf0zKbFtAXYC7lRrKPb4O+TLyWiw+ C/WK9gzpDeUONbN5PLyjOQTvfKDTpHlG2YevKn7eHhiECh6VHjzL3vpBY xfFRNTAHraTBMpndzLfxJJno19BX0BH7TEyAvbw/spZv170m8x9lpV1Lr Y=;
X-IronPort-Anti-Spam-Filtered: true
X-IronPort-Anti-Spam-Result: =?us-ascii?q?A0BcAAA7qvFb/4QNJK1YChoBAQEBAQI?= =?us-ascii?q?BAQEBBwIBAQEBgWWCBIFoJwqYAYINmTALAQGEbAKDVCI4EgEDAQECAQECbSi?= =?us-ascii?q?FPAEBAQQ6OBMEAgEIEQMBAh8QMh0IAgQBEoUjqDKKGYwFF4FAP4ERgxKEU4Y?= =?us-ascii?q?GAokYllcJApElGIFYhQiKHYgtj0ICERSBJzYhJ4EucBWDJ5BZAUExjQ6BHwE?= =?us-ascii?q?B?=
X-IronPort-AV: E=Sophos;i="5.56,249,1539648000"; d="scan'208";a="484281770"
Received: from alln-core-10.cisco.com ([173.36.13.132]) by rcdn-iport-4.cisco.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 18 Nov 2018 18:11:04 +0000
Received: from XCH-ALN-005.cisco.com (xch-aln-005.cisco.com [173.36.7.15]) by alln-core-10.cisco.com (8.15.2/8.15.2) with ESMTPS id wAIIB4Pb005014 (version=TLSv1.2 cipher=AES256-SHA bits=256 verify=FAIL); Sun, 18 Nov 2018 18:11:04 GMT
Received: from xch-aln-005.cisco.com (173.36.7.15) by XCH-ALN-005.cisco.com (173.36.7.15) with Microsoft SMTP Server (TLS) id 15.0.1395.4; Sun, 18 Nov 2018 12:11:03 -0600
Received: from xch-aln-005.cisco.com ([173.36.7.15]) by XCH-ALN-005.cisco.com ([173.36.7.15]) with mapi id 15.00.1395.000; Sun, 18 Nov 2018 12:11:03 -0600
From: "Mo Zanaty (mzanaty)" <mzanaty@cisco.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: AQHUf2oRTAjBhotW8kuaxZKOwWeUWg==
Date: Sun, 18 Nov 2018 18:11:03 +0000
Message-ID: <D8170F9A.844E6%mzanaty@cisco.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:
user-agent: Microsoft-MacOutlook/14.7.7.170905
x-ms-exchange-messagesentrepresentingtype: 1
x-ms-exchange-transport-fromentityheader: Hosted
x-originating-ip: [10.82.173.7]
Content-Type: text/plain; charset="us-ascii"
Content-ID: <2535F8662E0BBB4F8305983F03048FA4@emea.cisco.com>
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-Outbound-SMTP-Client: 173.36.7.15, xch-aln-005.cisco.com
X-Outbound-Node: alln-core-10.cisco.com
Archived-At: <https://mailarchive.ietf.org/arch/msg/payload/ptK463i6EzRo7F3euhcP7YzuPBs>
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: Sun, 18 Nov 2018 18:11:07 -0000

WG,

After discussing version -10 changes with Magnus and Jonathan, one change
needs to be modified.

OLD:
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.

NEW:
When performing retransmissions, a single repair packet stream (SSRC) MAY
be used for retransmitting packets from multiple source packet streams
(SSRCs), as well as transmitting FEC repair packets that protect multiple
source packet streams (SSRCs).

This replaces option A) with B) from Magnus' original comments, so a
single repair SSRC can retransmit multiple source SSRCs, and further
clarifies that the single repair SSRC can be used for both retransmission
and FEC protection of multiple source SSRCs.


Please comment if there are any objections, otherwise the authors will
make this change in -11.

Thanks,
Mo

-----Original Message-----
From: 'Magnus Westerlund' <magnus.westerlund@ericsson.com>
Date: Thursday, October 25, 2018 at 7:33 AM
To: "payload@ietf.org" <payload@ietf.org>rg>,
"draft-ietf-payload-flexible-fec-scheme@ietf.org"
<draft-ietf-payload-flexible-fec-scheme@ietf.org>
Subject: Re: [payload] Review of draft-ietf-payload-flexible-fec-scheme-08
Resent-From: <alias-bounces@ietf.org>
Resent-To: mzanaty <mzanaty@cisco.com>om>, <varun.singh@iki.fi>fi>,
<ali.begen@networked.media>ia>, <mandyam@qti.qualcomm.com>
Resent-Date: Thursday, October 25, 2018 at 7:33 AM

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