Re: [payload] summary of changes to draft-ietf-payload-rtp-ancillary from -06 to -07

Thomas Edwards <Thomas.Edwards@fox.com> Fri, 03 March 2017 01:32 UTC

Return-Path: <Thomas.Edwards@fox.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 6278912956D for <payload@ietfa.amsl.com>; Thu, 2 Mar 2017 17:32:34 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.54
X-Spam-Level:
X-Spam-Status: No, score=-1.54 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, KHOP_DYNAMIC=1.08, RCVD_IN_DNSWL_LOW=-0.7, RCVD_IN_MSPIKE_H3=-0.01, RCVD_IN_MSPIKE_WL=-0.01, SPF_PASS=-0.001, URIBL_BLOCKED=0.001] autolearn=no autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=foxgroupinc.onmicrosoft.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 Xp3DgVDs5jOA for <payload@ietfa.amsl.com>; Thu, 2 Mar 2017 17:32:32 -0800 (PST)
Received: from mx0a-00195501.pphosted.com (mx0a-00195501.pphosted.com [67.231.149.164]) (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 D93481293E8 for <payload@ietf.org>; Thu, 2 Mar 2017 17:32:32 -0800 (PST)
Received: from pps.filterd (m0082295.ppops.net [127.0.0.1]) by mx0a-00195501.pphosted.com (8.16.0.20/8.16.0.20) with SMTP id v231NI9c001349 for <payload@ietf.org>; Thu, 2 Mar 2017 17:32:32 -0800
Received: from nam02-bl2-obe.outbound.protection.outlook.com (mail-bl2nam02lp0080.outbound.protection.outlook.com [207.46.163.80]) by mx0a-00195501.pphosted.com with ESMTP id 28xuj9gs51-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-SHA384 bits=256 verify=NOT) for <payload@ietf.org>; Thu, 02 Mar 2017 17:32:32 -0800
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=FoxGroupInc.onmicrosoft.com; s=selector1-fox-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version; bh=pXdB3Rq1rxnqAMaajM0HR5JXhfddhX2UiHSCW5PnSOQ=; b=BAHCpXrc3pSH2IGktknZd+nw4ZJbU6qXi5AswFJZXcuwdEktK4G8y5sGYliEanfbU1JbljR5Glv3errCFYdn489tpKrf3dHsSe47EjdHINc7mdXiEO46Jb5/w8SROiq1vBzm8NsbtdHD8bhzu89oYG9zlqVDmib8zloxhOMRHJ0=
Received: from CY4PR05MB3109.namprd05.prod.outlook.com (10.172.157.7) by CY4PR05MB3109.namprd05.prod.outlook.com (10.172.157.7) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P384) id 15.1.947.2; Fri, 3 Mar 2017 01:32:30 +0000
Received: from CY4PR05MB3109.namprd05.prod.outlook.com ([10.172.157.7]) by CY4PR05MB3109.namprd05.prod.outlook.com ([10.172.157.7]) with mapi id 15.01.0947.012; Fri, 3 Mar 2017 01:32:30 +0000
From: Thomas Edwards <Thomas.Edwards@fox.com>
To: "payload@ietf.org" <payload@ietf.org>
Thread-Topic: [payload] summary of changes to draft-ietf-payload-rtp-ancillary from -06 to -07
Thread-Index: AQHSk74FD3mGOGIZV0uki6IbiUPJaQ==
Date: Fri, 03 Mar 2017 01:32:30 +0000
Message-ID: <31F9558B-8A30-42E8-95D8-1D22F7333D3C@fox.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
user-agent: Microsoft-MacOutlook/f.1b.0.161010
authentication-results: ietf.org; dkim=none (message not signed) header.d=none;ietf.org; dmarc=none action=none header.from=fox.com;
x-ms-exchange-messagesentrepresentingtype: 1
x-originating-ip: [216.205.246.229]
x-ms-office365-filtering-correlation-id: 38a85a7e-00a1-47d6-ddd3-08d461d528a2
x-ms-office365-filtering-ht: Tenant
x-microsoft-antispam: UriScan:; BCL:0; PCL:0; RULEID:(22001)(48565401081); SRVR:CY4PR05MB3109;
x-microsoft-exchange-diagnostics: 1; CY4PR05MB3109; 7:H39amHbEE91+060FrYQlO8lCMgtRCwSpovt+abag0UzP8qEBiDZNnz4Nv8PRnclUdw/Pe9hjsCJ3zGRzvYuyu7fcfq2/tbeW2XZ04kdf3CWPTkg9zor7T+fXrovwo6+La+Kj1pita8WvAayyTs9vzGKC3mVJbcBwiRDyw3j/hyVY0HgLhhNVltUJISfn3JqgEH6VumsaoE17wIy+siaLdU9YPcXavS+RW+3ZT+PcEKuTzHGeq/p6wEEEWNAckgIWUP4okD5KPkdokXthJX66quZiEYEDJG/3c+rRBhlKPd+x8GSPA0a04l/5kytpoNSzs5BLD3HFETMgmRNOk4d0mA==
x-microsoft-antispam-prvs: <CY4PR05MB31095E582FDAFC3ACEA13AD0942B0@CY4PR05MB3109.namprd05.prod.outlook.com>
x-exchange-antispam-report-test: UriScan:(177823376758907);
x-exchange-antispam-report-cfa-test: BCL:0; PCL:0; RULEID:(6040375)(601004)(2401047)(8121501046)(5005006)(3002001)(10201501046)(6055026)(6041248)(20161123558025)(20161123555025)(20161123560025)(20161123562025)(20161123564025)(6072148); SRVR:CY4PR05MB3109; BCL:0; PCL:0; RULEID:; SRVR:CY4PR05MB3109;
x-forefront-prvs: 0235CBE7D0
x-forefront-antispam-report: SFV:NSPM; SFS:(10009020)(6009001)(7916002)(39450400003)(39860400002)(39840400002)(39410400002)(39850400002)(2900100001)(3280700002)(86362001)(3660700001)(6486002)(77096006)(6512007)(33656002)(110136004)(230783001)(99286003)(92566002)(229853002)(81166006)(25786008)(38730400002)(8936002)(6506006)(2906002)(6246003)(1730700003)(7736002)(8676002)(305945005)(53936002)(3846002)(6436002)(5640700003)(450100001)(102836003)(54356999)(50986999)(5660300001)(122556002)(83506001)(82746002)(2351001)(4001350100001)(2501003)(83716003)(6916009)(66066001)(106116001)(36756003)(6116002)(189998001)(104396002); DIR:OUT; SFP:1101; SCL:1; SRVR:CY4PR05MB3109; H:CY4PR05MB3109.namprd05.prod.outlook.com; FPR:; SPF:None; MLV:sfv; LANG:en;
spamdiagnosticoutput: 1:99
spamdiagnosticmetadata: NSPM
Content-Type: text/plain; charset="utf-8"
Content-ID: <377E788CE50442429A442F117A95C626@namprd05.prod.outlook.com>
Content-Transfer-Encoding: base64
MIME-Version: 1.0
X-OriginatorOrg: fox.com
X-MS-Exchange-CrossTenant-originalarrivaltime: 03 Mar 2017 01:32:30.1581 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: de99ade3-81db-4070-ae0d-3c1562041b30
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CY4PR05MB3109
X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10432:, , definitions=2017-03-02_21:, , signatures=0
X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 priorityscore=1501 malwarescore=0 suspectscore=0 phishscore=0 bulkscore=0 spamscore=0 clxscore=1011 lowpriorityscore=0 impostorscore=0 adultscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.0.1-1702020001 definitions=main-1703030012
Archived-At: <https://mailarchive.ietf.org/arch/msg/payload/EMXQH-oDqOrsiKYmIT6rhaYmIvI>
Subject: Re: [payload] summary of changes to draft-ietf-payload-rtp-ancillary from -06 to -07
X-BeenThere: payload@ietf.org
X-Mailman-Version: 2.1.17
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, 03 Mar 2017 01:32:34 -0000

I want to thank John Fletcher for providing feedback to the payload WG list regarding draft-ietf-payload-rtp-ancillary-07

John Fletcher comments >

>It needs to be made clear that the scope is limited to ancillary data carried over a Serial Digital Interface.

I believe that this payload could be used for ANC data packets transmitted without reference to a Serial Digital Interface.

>and delete:
>"This memo describes an RTP payload that supports ANC data packets regardless of whether they originate from an SD or HD interface"
>and add:
>"This memo describes an RTP payload that supports ANC data packets originating from or intended for a Serial Digital Interface such as SDI, HD-SDI, 
>HD_SDI, 3G SDI, 6G SDI, 12G SDI and multiple link versions of these."

That sounds a bit heavy for an introduction.  But perhaps sentence could be changed to be more generic:

“This memo describes an RTP payload that supports carriage of ANC data packets
with origin from any location within any SMPTE defined SDI signal, or 
even if the ANC packets did not originate in an SDI signal.”

>RTP header, timestamp: The term "sampling instant" needs to be clarified, particularly for playback.

I don’t believe that this document, an RTP payload, should define SDI playback.

>Payload header, C bit: You may need more than one bit for interfaces with multiple data streams.
>Cases to consider are:
>SMPTE ST 259 interface;
>In a SMPTE ST 292-1 HD-SDI interface, whether the ANC packets are carried in the Luma channel or the Color Difference channel;
>In a SMPTE ST 425-1 3G-SDI Level B or SMPTE ST 2081-10 6G-SDI mode 1 interface, whether the ANC packets are carried in the Y/G-channel of data stream one or in the other channel and/or other data streams
>In SMPTE ST 425-1 3G-SDI Level A, SMPTE ST 2081-10 6G-SDI mode 2 or SMPTE ST 2082-10 12G SDI interfaces, whether the ANC packets are carried in data stream one or in another data stream
>Payload format parameter, link number: I think you may also need data stream number unless use of C bit is expanded as above.

The LinkNumber SDP parameter definition currently is “link number (1-4) of SDI interfaces that use dual or quad link.”  It can be made more generic: “link number, stream number, or image number of multi-link, multi-stream, or multi-image SDI interfaces.”

>Payload header, line number: You need to include the image format (e.g. in payload format) for the line number to make sense.
>Payload format parameters: I think you need to specify the interface type and the image format

I believe that is out of scope of this payload.

>Payload format parameter, link number: The phrase "comes from" is used but the data could be "intended for"

Perhaps more generically “is associated with”

>Since IP/UDP/RTP are byte-orientated, you need to specify how 10-bit UDWs are mapped onto interface bytes.

OK, I can add “with the 10-bit words carried in order starting from most significant bit and ending in least significant bit”.

>Payload header, ANC_Count - "network byte order" is irrelevant as it is only one byte.

OK.

>Payload header, Line_Number and Horizontal Offset: You need to say more that “network byte order” because these are not a multiple of 8 bits.

I think the ordering from most significant bit to least significant bit is understood, for example RFC 4175 has a 15-bit Line No. field “in network byte order”.

>More normative language is needed.
>There some uses of "may" in non-normative context.

The draft has been analyzed by IETF process for normative language use already.  I do not believe any further changes are required.

>A definition of network byte order is needed

I can include a reference to IETF STD 5.

-Thomas

-- 
Thomas Edwards 
VP Engineering & Development
FOX Networks Engineering and Operations
thomas.edwards@fox.com
Phone: +1.310.369.6696
10201 West Pico Blvd.
Los Angeles, CA 90035