Re: [AVTCORE] AD Review of draft-ietf-payload-rtp-jpegxs-11

Tim Bruylants <TBR@intopix.com> Mon, 03 May 2021 14:57 UTC

Return-Path: <TBR@intopix.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 176993A1713 for <avt@ietfa.amsl.com>; Mon, 3 May 2021 07:57:45 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.618
X-Spam-Level:
X-Spam-Status: No, score=-2.618 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_LOW=-0.7, RCVD_IN_MSPIKE_H4=-0.01, RCVD_IN_MSPIKE_WL=-0.01, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, URIBL_BLOCKED=0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=intopix.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 6TsYS3csO5Kc for <avt@ietfa.amsl.com>; Mon, 3 May 2021 07:57:40 -0700 (PDT)
Received: from dispatchb-eu1.ppe-hosted.com (dispatchb-eu1.ppe-hosted.com [185.132.181.10]) (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 77A603A1706 for <avtcore@ietf.org>; Mon, 3 May 2021 07:57:40 -0700 (PDT)
X-Virus-Scanned: Proofpoint Essentials engine
Received: from EUR04-DB3-obe.outbound.protection.outlook.com (mail-db3eur04lp2055.outbound.protection.outlook.com [104.47.12.55]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mx1-eu1.ppe-hosted.com (PPE Hosted ESMTP Server) with ESMTPS id 1C9FC40078; Mon, 3 May 2021 14:57:33 +0000 (UTC)
ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=EWEF93AlnxZ5b+DAVJzhDXDuNK77ZPfWd1JAENpgzlPEMSp/c/kU7RA7jmyFu/+g5J2fmSoECwMpDgQHFSdyWAKmYHYupU1OKx4Tc0+PCXcGAo4xQiPY7IODW0EHOVrb6z0WtPVGqePohjPoepgHBBDerwnOOHvinAL2xfOYbeVoikoIz/EKcPQKbNePH/Q1U1wZoAPPqvhllSggMoT+hmbsZj7K05TiAFWNoGWdETNuC6i6WST5XDUyb0frQ4Cnl0oHzMKyX5V0kezVbnVg+8TWgJLk36hzDXCKkDHi+39PfIcng0RJZR23iZtl3gASdXD2IMjeWwyKnI6iSsSLCQ==
ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=4rGEzhm2p/8FlTltpWf/ueV6zNE3/qBS4nmK8GM1WP4=; b=BVl9GAZnUAefQ5Oagq/rrHS4u9IPhfOr4Mr0N1ffku0P5QTKlNkmJZxLWrbuv5u+P5tWGO1rLGlk3yKkEiaHIqbrNZxFXcEahgrlHXhz2MjCgRj9JY0IjefD05P+ZrIYkwdJeCehhR5vG3WYxEHu7CTwvsn35tzRxwrv8sauILSZZWYdiWMeiW3T7HRwSkjUxl4QF9bX/J4timCdy0QQQsEmwR9za4GLeYvOlSh5wdJJzb9HCKd/nyYAtJ7oJnm8VFbRR62n7v6XWNkj4m6FgS7pa014drqxtfTUSgThqofjFplQsrrXxpUJzn/f4NG07Bji5m/pn1oM63/8UbYDDw==
ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=intopix.com; dmarc=pass action=none header.from=intopix.com; dkim=pass header.d=intopix.com; arc=none
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=intopix.onmicrosoft.com; s=selector1-intopix-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=4rGEzhm2p/8FlTltpWf/ueV6zNE3/qBS4nmK8GM1WP4=; b=VA/PMTI+Tlm4ySxKzL4Ih3aYgrJmiV2D367voUyIwr/CjADvIGbImJaCI/mM068L33ArmjCPajMzGvx0wbcVNkBHYEuqsCwOtB6kfgAf37bZyaoq/r7ZPCGd8TAhM/fc5h4KQPv5odZnOX4TB+SC+Rxfvz8iQcKKnqG3bwVd9ok=
Received: from PR3P192MB0748.EURP192.PROD.OUTLOOK.COM (2603:10a6:102:4f::24) by PR3P192MB1005.EURP192.PROD.OUTLOOK.COM (2603:10a6:102:94::10) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4087.38; Mon, 3 May 2021 14:57:31 +0000
Received: from PR3P192MB0748.EURP192.PROD.OUTLOOK.COM ([fe80::f598:e21e:5658:281f]) by PR3P192MB0748.EURP192.PROD.OUTLOOK.COM ([fe80::f598:e21e:5658:281f%5]) with mapi id 15.20.4087.044; Mon, 3 May 2021 14:57:31 +0000
From: Tim Bruylants <TBR@intopix.com>
To: "Murray S. Kucherawy" <superuser@gmail.com>, "avtcore@ietf.org" <avtcore@ietf.org>
Thread-Topic: [AVTCORE] AD Review of draft-ietf-payload-rtp-jpegxs-11
Thread-Index: AQHXP9Mltk3wSOimukmUP9PlJGiJfKrR2ThA
Date: Mon, 3 May 2021 14:57:31 +0000
Message-ID: <PR3P192MB0748D48268FFC24A1BBE1DC4AC5B9@PR3P192MB0748.EURP192.PROD.OUTLOOK.COM>
References: <CAL0qLwb8V-ZnveD-Bx1inbpMOZHEvEXNDfi08DVgDDY6bY88_w@mail.gmail.com>
In-Reply-To: <CAL0qLwb8V-ZnveD-Bx1inbpMOZHEvEXNDfi08DVgDDY6bY88_w@mail.gmail.com>
Accept-Language: en-US, nl-NL
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
authentication-results: gmail.com; dkim=none (message not signed) header.d=none;gmail.com; dmarc=none action=none header.from=intopix.com;
x-originating-ip: [2a02:1810:1dbd:e901:9572:1c39:ee48:a399]
x-ms-publictraffictype: Email
x-ms-office365-filtering-correlation-id: afff8e4a-c527-4d80-d630-08d90e43c708
x-ms-traffictypediagnostic: PR3P192MB1005:
x-microsoft-antispam-prvs: <PR3P192MB1005FE00B47171CBD141C953AC5B9@PR3P192MB1005.EURP192.PROD.OUTLOOK.COM>
x-ms-oob-tlc-oobclassifiers: OLM:10000;
x-ms-exchange-senderadcheck: 1
x-microsoft-antispam: BCL:0;
x-microsoft-antispam-message-info: /BvLWwBjMbRIRwRJEzotmdJLmt+lW+ypX8fyYNY44D5dgrrBPWlHc76wFId8yuDdVwqhiE6ndbyPWSfb4x1sZXhd+zAyl5B0KsNwZfmQq3URtQ4Vgd7IP2a38pjqYxJcbxqOkIWKCUfm/ZQD0sJjuONGxyMCY3UEmHAJSqka1bnBf1fyhjxwg0hbG9gXoHvF+BSdzLp4A03/6aMJR4cx3aTgsNeccDJCORawLyib9Sd5BwsMv9V+rDQWQxj+wb9sP5dAX98CYeLTzOylkJOnjb8q2pk5OzFpGlDAoKcq3OPrQDcMyoKrGC+pt4KXoQSrEWTyaIoJPp3F5TxeRDk6ISzzX8TthYOK73OYdQvcvTxErYrirb3km9snaWHkCNbmjuG62gDZY8iVM5ZrKW0Nlqmvfo4zywikA6uCpGSkLaSxEpkRLaZex4M293Lr6mTMO0+vFHqYBKp1v26wJBwPk9wMmINQjldraPhnnuVqJKL/alyuq1U88a47C2qmOVFFvetqj4CDRt9bzlmO1yBiKuvv9k200NoXYRY9XB/a7pnBAzMGjGIsnhK3IFKUSU0TiJ9ov4WX/N9xdkyyKaCB7VjBHjGF+688+/QGkgTW3b8=
x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:PR3P192MB0748.EURP192.PROD.OUTLOOK.COM; PTR:; CAT:NONE; SFS:(136003)(396003)(366004)(39830400003)(376002)(346002)(83380400001)(110136005)(66476007)(66556008)(64756008)(66446008)(2906002)(76116006)(7696005)(52536014)(186003)(5660300002)(478600001)(66946007)(53546011)(55016002)(38100700002)(122000001)(9686003)(33656002)(8676002)(71200400001)(86362001)(8936002)(316002)(6506007); DIR:OUT; SFP:1101;
x-ms-exchange-antispam-messagedata: =?utf-8?B?eXhGR0ZOWUs1S0tXTzhuanhzY3Q2K0VqSmdES3V2RklIeks4OGt5NTZ6NmFQ?= =?utf-8?B?a21UNysyVWoraFRiUmh2U1pteEV1NG9tZEIrRWdTTHFEdkV6OGlCdnB6bUdG?= =?utf-8?B?TGJQY2RwRGRqcjltZW9PaWxsMlJPY0lDQnUxZUQvQ3RENHF4YnRadU1ycVB5?= =?utf-8?B?RFpnOWdtRzdoMk82U1lhZTVTMmN0QXk1ejJ1NnpWa2sraW5tVGxQMlRFVFNv?= =?utf-8?B?QWQxcXpWNWFZdlhTUVFhRTMwcURlN2Z6RFBEUk5LQk1tQkYzKzM2TDdlRWU4?= =?utf-8?B?blRpNWVRcXhMUVV6NEFwNDRTLzIvMkdPMzQ0VGt1UTRMcWdsaFNEUVVYdDJx?= =?utf-8?B?MmFWU3RHWWZ4Y3BMVUNwWk1Eem4xNVRFZWIzOEp3eVFvWm8wZ3RScG9mQTQz?= =?utf-8?B?anVRRllNQnNKWHdKY0VMYUhMTlUyN1NhMXc1RVhPcDFhM090aFVRazRYcDdE?= =?utf-8?B?MnllSWorVDdFR01FSnNyVS9RdDdZVk9QaWJabzBTS082VGZ3K1dvQ09hM25s?= =?utf-8?B?aEZVL21nd2M1RUkwdWpPbVV5clBZeHVtQWxPaHVrTkIrQnAvbGlOT1BSSHBq?= =?utf-8?B?U2tYWFU4OFFlMXFkWEdNTXVYc3gyMFUxT2EyYUl2aDJObWs1T1R6R0lWa2tz?= =?utf-8?B?WTc2Q1ExRFJSc2U3Q244c3ozWnM3YWVDRVVvcGhNN2Q3VThYbXBBdW91Sk9Y?= =?utf-8?B?RDdFZ2VTTER1QTIvYTBiY0IrQjY5MXYrSGN3TFNuZG4xekl4VHpSMXhyb1Jl?= =?utf-8?B?OElUY2dYWFR5ZlBEVFZiTmNPbWJhU1dzeGFCeFVqaDk5U1pveWJGVVNqRk1w?= =?utf-8?B?YndYUi9TSjlySkNiMTRPSWpHYXNWYmRxbmNrUzNJTGZaV2VXOWpCdGg5NlFN?= =?utf-8?B?NDMzTDlWc2VtOW1hQy9HRU92NzB2YjFlaU9Ud0s4VksvalVjY3FMUWJoVmNq?= =?utf-8?B?b0RPejd2bUhBYisxdFdDNDNzdWkzM2hSK3UzZVU5RzlxWHhVWk04MW9sQUNm?= =?utf-8?B?REFkWDRiU2Y5cWVGQ3ZuUzgxQUptN1JIOHlidnRIRnR1ZlBsYlExb3VLNDhj?= =?utf-8?B?TUhON0JoaWVSbCt4cHh5VlR6NHVtK3lJUlJyK3U5bHZJY3FDd2tONi8wejlB?= =?utf-8?B?ZW5UZWNsUVZIMHN1YmtaTEZtbDEwZTB3QUhDa0NQOWhiWWtTZUIyVVJIMzRK?= =?utf-8?B?Z2Rpa2xkTkRVWUpSeHR0UFlEWCtGVDJPRlAzSTZXZzRhUHFRSzE5Y0J5WURo?= =?utf-8?B?MTZwaUdsa29MajBkbmpyRnRKQzh0bkFvKzFsa2Z3eVowNFNjRG5hNDVkZ05I?= =?utf-8?B?WCswbkhMUk1YdkZCWUdjWlBjRWpVLy9IYXlnQVIrNWxURnpzdXlkaTdteENG?= =?utf-8?B?UStwYXd5VDV4eDhiWjEvbzlRVW5KcXU3YlQya0phVFR3elNyTTFHU1dwUjVC?= =?utf-8?B?Wk5QVmtCNDB2NVY3MFAyT0RSWUhwVlRoMk9JTHhvSzk1dWt0Tlp4b3FCcnB3?= =?utf-8?B?VDBVYTcveERxZ1hpeGRqQkhkVDl4VnhkNFdnWE40WEl1SkM1MVZkeTJJb2U4?= =?utf-8?B?K2NrRzZLME1NS0M3YWFNcjAvYThlejJleFdBSnVIUW9DSGhIREpISjFBTmlR?= =?utf-8?B?aFRRUFJkUVExV3UyOTNydVNjcjVBVE1HZk9pNTYzSFBjNnprQ3paUjRRNFBw?= =?utf-8?B?L1I0WU85dkR2T3l2VE9Ib0FURnY1NXpSUEErTWVMZjdteVkzVk5teHUySnkx?= =?utf-8?B?VjMraHZVZ2xTdjFOSTB3VkU0c290eFkzYnB0c2g1WmNCVTJSV2FPR0RNbDRC?= =?utf-8?B?aUhPdm1SeDFwOG95Tkp0K3RUR0lCL2c3alVFSnBEUkYzdDllNjB2WjQ1aStn?= =?utf-8?Q?At8c3oF7nAOIK?=
x-ms-exchange-transport-forked: True
Content-Type: multipart/alternative; boundary="_000_PR3P192MB0748D48268FFC24A1BBE1DC4AC5B9PR3P192MB0748EURP_"
MIME-Version: 1.0
X-OriginatorOrg: intopix.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: PR3P192MB0748.EURP192.PROD.OUTLOOK.COM
X-MS-Exchange-CrossTenant-Network-Message-Id: afff8e4a-c527-4d80-d630-08d90e43c708
X-MS-Exchange-CrossTenant-originalarrivaltime: 03 May 2021 14:57:31.3383 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 5f9168c7-cdac-4b23-9509-c278399e3c1f
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: vz0etYggQnv7Z3do5wN898YsR+3mpt3X342Y/B3rPkN2/We0nPQS6IA7kizEwMH4
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PR3P192MB1005
X-MDID: 1620053853-NYFr4CAXObzv
Archived-At: <https://mailarchive.ietf.org/arch/msg/avt/-bFavPfAGBeYbaKMEm-uk618KSY>
Subject: Re: [AVTCORE] AD Review of draft-ietf-payload-rtp-jpegxs-11
X-BeenThere: avt@ietf.org
X-Mailman-Version: 2.1.29
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: Mon, 03 May 2021 14:57:45 -0000

Dear Murray,

Thank you for the review. We have addressed your comments in draft-ietf-payload-rtp-jpegxs-12.

Here is the detailed answer to the comments:

>In Section 2, please update the BCP 14 text to use the new form (see RFC 8174, which must also be added as a normative reference).

  We have updated the text to comply with RFC 8174.



> Also in Section 2, the term "SOC Marker", though defined here, is not used below here.  Section 3.2 references the EOC Marker; should it reference both?

  1) The term "SOC marker" is used in section 2 under "JPEG XS codestream header".
  2) We clarified section 3.2, referencing "SOC marker".



> In Section 4, we encounter "SSRC" for the first time.  A definition for it would be helpful, or a reference.

  SSRC is Synchronization Source, as defined in RFC3550. We modified the text to clarify this.



> In Section 4.2, "As per specified" can just be "As specified".  Alternatively, "As per RFC 3550, ..."  While I'm thinking about it, all these double references (e.g., "RFC 3550 [RFC3550]") throughout the document could be cleaned up.

  We have cleaned up the "double references" throughout the text.



> The diagrams in Section 4.4 show (eventually) how the EOC marker fits into this, but not the SOC marker.  Should they?

  We clarified that the SOC marker is part of the JPEG XS codestream header. The latter is clearly mentioned in the diagrams. Explicitly mentioning SOC in the diagrams would clutter the layout, and would probably require to explicitly add the other markers in the codestream header as well.



> Section 4.5 uses a SHOULD.  SHOULD leaves the implementer with a choice, so it's a good idea to include guidance about when one might legitimately do something other than what the SHOULD says to do.  Or, if there isn't any, maybe this ought to be a SHALL?

  We reworded section 4.5 slightly. The message is that actual traffic shaping and timing delivery is outside the scope of this specification and does not influence the payload packetization. We also moved the section one level up (became section 5).



> In Section 6.1, it's clear what a receiver would do with some of the optional parameters when they are absent (i.e., defaults are clear), but not in all cases.  What can a receiver safely assume about "depth", "width", or "height", for example, when not specified?

  We thought this was clarified by section 6.2.1. Nevertheless, we added a short paragraph in section 6 (now section 7).



> Also on 6.1, this media type registration is missing required fields, including "Interoperability Considerations", "Published Specification", "Applications that use this media type", "Fragment identifier considerations", "Additional information" (which has four sub-fields", "Contact information", "Intended Usage", "Restrictions On Usage", "Author", and "Change Controller".  Please see RFC 6838.  Further, it would probably be a good idea to mention either in the registration or in the Security Considerations of this document that the payload being registered does not (or does) contain code that is executable.  The media type reviewers prefer people to be explicit on this point.

  We have reworked the media type registration section to follow RFC 6838 and RFC 4855.

  We added the notion that the payload format and the JPEG XS encoding do NOT contain executable code (to the Security Considerations section).


Kind regards,
Tim Bruylants



From: avt <avt-bounces@ietf.org> On Behalf Of Murray S. Kucherawy
Sent: Monday 3 May 2021 06:16
To: avtcore@ietf.org
Subject: [AVTCORE] AD Review of draft-ietf-payload-rtp-jpegxs-11

Hi there, here's my AD Review of this document.
Per the shepherd writeup, I have requested a review by the SDP Directorate.

First: In the shepherd writeup, please correct the spelling of my name.  :-)  Also, while we're in there, question (12) asks what reviews were done, but the answer simply says that the document contains a media type registration, which doesn't really answer the question.  Were any of the media type review resources asked for feedback on this?  Later on the shepherd says he reviewed it, but it doesn't say what official reviews may have been undertaken.  As you can see later on, there's a good chunk of the registration missing.

In Section 2, please update the BCP 14 text to use the new form (see RFC 8174, which must also be added as a normative reference).

Also in Section 2, the term "SOC Marker", though defined here, is not used below here.  Section 3.2 references the EOC Marker; should it reference both?

In Section 4, we encounter "SSRC" for the first time.  A definition for it would be helpful, or a reference.

In Section 4.2, "As per specified" can just be "As specified".  Alternatively, "As per RFC 3550, ..."  While I'm thinking about it, all these double references (e.g., "RFC 3550 [RFC3550]") throughout the document could be cleaned up.

The diagrams in Section 4.4 show (eventually) how the EOC marker fits into this, but not the SOC marker.  Should they?

Section 4.5 uses a SHOULD.  SHOULD leaves the implementer with a choice, so it's a good idea to include guidance about when one might legitimately do something other than what the SHOULD says to do.  Or, if there isn't any, maybe this ought to be a SHALL?

In Section 6.1, it's clear what a receiver would do with some of the optional parameters when they are absent (i.e., defaults are clear), but not in all cases.  What can a receiver safely assume about "depth", "width", or "height", for example, when not specified?

Also on 6.1, this media type registration is missing required fields, including "Interoperability Considerations", "Published Specification", "Applications that use this media type", "Fragment identifier considerations", "Additional information" (which has four sub-fields", "Contact information", "Intended Usage", "Restrictions On Usage", "Author", and "Change Controller".  Please see RFC 6838.  Further, it would probably be a good idea to mention either in the registration or in the Security Considerations of this document that the payload being registered does not (or does) contain code that is executable.  The media type reviewers prefer people to be explicit on this point.

-MSK