Re: [xrblock] AD evaluation: draft-ietf-xrblock-rtcp-xr-psi-decodability-03

Qin Wu <bill.wu@huawei.com> Wed, 04 June 2014 04:33 UTC

Return-Path: <bill.wu@huawei.com>
X-Original-To: xrblock@ietfa.amsl.com
Delivered-To: xrblock@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 226CB1A006C for <xrblock@ietfa.amsl.com>; Tue, 3 Jun 2014 21:33:30 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -3.652
X-Spam-Level:
X-Spam-Status: No, score=-3.652 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, J_CHICKENPOX_32=0.6, J_CHICKENPOX_82=0.6, RCVD_IN_DNSWL_MED=-2.3, RP_MATCHES_RCVD=-0.651, SPF_PASS=-0.001] autolearn=ham
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 Pv2qm-ISmHYK for <xrblock@ietfa.amsl.com>; Tue, 3 Jun 2014 21:33:28 -0700 (PDT)
Received: from lhrrgout.huawei.com (lhrrgout.huawei.com [194.213.3.17]) (using TLSv1 with cipher RC4-SHA (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id B9D9D1A0064 for <xrblock@ietf.org>; Tue, 3 Jun 2014 21:33:26 -0700 (PDT)
Received: from 172.18.7.190 (EHLO lhreml204-edg.china.huawei.com) ([172.18.7.190]) by lhrrg01-dlp.huawei.com (MOS 4.3.7-GA FastPath queued) with ESMTP id BHU83904; Wed, 04 Jun 2014 04:33:19 +0000 (GMT)
Received: from LHREML402-HUB.china.huawei.com (10.201.5.241) by lhreml204-edg.china.huawei.com (172.18.7.223) with Microsoft SMTP Server (TLS) id 14.3.158.1; Wed, 4 Jun 2014 05:32:32 +0100
Received: from NKGEML406-HUB.china.huawei.com (10.98.56.37) by lhreml402-hub.china.huawei.com (10.201.5.241) with Microsoft SMTP Server (TLS) id 14.3.158.1; Wed, 4 Jun 2014 05:33:18 +0100
Received: from NKGEML501-MBS.china.huawei.com ([169.254.2.193]) by nkgeml406-hub.china.huawei.com ([10.98.56.37]) with mapi id 14.03.0158.001; Wed, 4 Jun 2014 12:33:16 +0800
From: Qin Wu <bill.wu@huawei.com>
To: Alissa Cooper <alissa@cooperw.in>, "xrblock@ietf.org" <xrblock@ietf.org>
Thread-Topic: [xrblock] AD evaluation: draft-ietf-xrblock-rtcp-xr-psi-decodability-03
Thread-Index: AQHPf2/+zcOs2o7WEkKH2v8pp9ow1JtgWW2w
Date: Wed, 04 Jun 2014 04:33:16 +0000
Message-ID: <B8F9A780D330094D99AF023C5877DABA845479BD@nkgeml501-mbs.china.huawei.com>
References: <CFB3955F.3DF4C%alissa@cooperw.in>
In-Reply-To: <CFB3955F.3DF4C%alissa@cooperw.in>
Accept-Language: zh-CN, en-US
Content-Language: zh-CN
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-originating-ip: [10.138.41.117]
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: base64
MIME-Version: 1.0
X-CFilter-Loop: Reflected
Archived-At: http://mailarchive.ietf.org/arch/msg/xrblock/4BeZVa1lkp7pNW_qYaYy8Q2gO6Y
Subject: Re: [xrblock] AD evaluation: draft-ietf-xrblock-rtcp-xr-psi-decodability-03
X-BeenThere: xrblock@ietf.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: Metric Blocks for use with RTCP's Extended Report Framework working group discussion list <xrblock.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/xrblock>, <mailto:xrblock-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/xrblock/>
List-Post: <mailto:xrblock@ietf.org>
List-Help: <mailto:xrblock-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/xrblock>, <mailto:xrblock-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 04 Jun 2014 04:33:30 -0000

Hi, Alissa:
Thanks for valuable comments. We think not all implementations support scrambling and scrambling adds a lot of complexity for packet processing since it needs to support encryption and decryption.
But you are right, to get consistent with ETSI TR 101 290, I am okay with adding scrambling feature.
For your comments, please see my reply below.

Regards!
-Qin
-----邮件原件-----
发件人: xrblock [mailto:xrblock-bounces@ietf.org] 代表 Alissa Cooper
发送时间: 2014年6月4日 5:08
收件人: xrblock@ietf.org
主题: [xrblock] AD evaluation: draft-ietf-xrblock-rtcp-xr-psi-decodability-03

I have reviewed this draft in preparation for IETF LC and I have some questions before proceeding with the LC announcement.

In Section 3, there seem to be some inconsistencies between the way that some of the fields are defined in the block and the way the corresponding metrics are described in ETSI TR 101 290 (assuming
http://www.etsi.org/deliver/etsi_tr/101200_101299/101290/01.02.01_60/tr_101
290v010201p.pdf is the correct reference for that document):

(1) PAT_error_count is defined in the draft as:

"A count of the number of PAT errors that occurred in the above
      sequence number interval.  The program association table (PAT) is
      the only packet with packet ID (PID) 0x 0000.  A PAT error occurs
      when it does not occur at least every 0.5s or the table has an ID
      other than 0x 00, as defined in section 5.2.1 of [ETSI].  Every
      program within the MPEG TS stream is listed in the PAT; if it is
      missing, then no programs can be decoded."

Whereas in the ETSI document, a PAT_error is defined as:

"PID 0x0000 does not occur at least every 0,5 s a PID 0x0000 does not contain a table_id 0x00 ( i.e.
a PAT)
Scrambling_control_field is not 00 for PID 0x0000”

Is the definition in the draft supposed to be equivalent to the first two conditions in the ETSI definition? If so, that is not quite clear from the draft. Why is the Scrambling_control_field condition not also part of the definition in the draft?

[Qin]: I propose the following change:
NEW TEXT:
"
   PAT_error_count: 16 bits

      A count of the number of PAT errors that occurred in the above
      sequence number interval.  The program association table (PAT) is
      the only packet with packet ID (PID) 0x 0000.  A PAT error occurs
      when it does not occur at least every 0.5s or the table has an ID
      other than 0x00 or Scrambling_control_field in the TS packet header
       is not 00 for PID 0x0000, as defined in section 5.2.1 of [ETSI].  Every
      program within the MPEG TS stream is listed in the PAT; if it is
      missing, then no programs can be decoded.
"

I have the same questions about PAT_error_2_count.

[Qin]:How about the following change:
NEW TEXT:
"
   PAT_error_2_count: 16 bits

      A count of the number of PAT2 errors that occurred in the above
      sequence number interval.  A PAT2 error occurs when it does not
      occur at least every 0.5s or the table has an ID other than 0x00
      or there is more than one table ID 0x 00 inside the packet with
      the PAT PID or Scrambling_control_field in the TS packet header 
      is not 00 for PID 0x0000, as defined in section 5.2.1 of [ETSI].
"

(2) In the draft, PMT_error_count and PMT_error_2_count have the exact same definition. But in the ETSI document, a PMT_error is different from a PMT_error_2. Here are the definitions from the ETSI document:

PMT_error:
"Sections with table_id 0x02, ( i. e. a PMT), do not occur at least every 0,5 s on the PID which is referred to in the PAT Scrambling_control_field is not 00 for all PIDs containing sections with table_id 0x02 (i.e. a PMT)”

PMT_error_2:
"Sections with table_id 0x02, (i.e. a PMT), do not occur at least every 0,5 s on each program_map_PID which is referred to in the PAT Scrambling_control_field is not 00 for all packets containing information of sections with table_id
0x02 (i.e. a PMT) on each program_map_PID which is referred to in the PAT"

Are the equivalent definitions of PMT_error_count and PMT_error_2_count in the draft intentional? If so, why define the same metric twice, and why make PMT_error_2_count different from the way PMT_error_2 is defined in the ETSI document?

[Qin]: Good catch, without scrambling, definitions of PMT_error_count and PMT_error_2_count in the draft are equivalent.
But in ETSI document, PMT_error_2_count is different from PMT_error_count since the former excludes specifically network_PID.
I propose the following changes to definitions of PMT_error_count and PMT_error_2_count in the draft:
NEW TEXT:
"
   PMT_error_count: 16 bits

      A count of the number of PMT_errors that occurred in the above
      sequence number interval.  A PMT_error occurs when the program map
      table (PMT) does not come up at least every 0.5s on the PID that
      is referred to in the PAT or Scrambling_control_field in the TS packet header is not 00 for all PIDs
      containing sections with table_id 0x02 (i.e. a PMT), as defined in the section 5.2.1 of
      [ETSI].

      The measured value is unsigned value.  If the measurement is
      unavailable, the value 0xFFFF MUST be reported.  Note that
      PMT_error_count and PMT_error_2_count MUST NOT be reported at the
      same time in the same metric block.  If PMT_error_count is
      reported, PMT_error_2_count MUST be set to 0xFFFF.

   PMT_error_2_count: 16 bits

      A count of the number of PMT2 errors that occurred in the above
      sequence number interval.  A PMT2_error occurs when the program
      map table (PMT) does not come up at least every 0.5s on the PID
      that is referred to in the PAT or Scrambling_control_field in the TS packet header is not 00 for all packets
      containing information of sections with table_id
      0x02 (i.e. a PMT) on each program_map_PID
      which is referred to in the PAT, as defined in the section 5.2.1 of
      [ETSI].

      The measured value is unsigned value.  If the measurement is
      unavailable, the value 0xFFFF MUST be reported.  Note that
      PMT_error_count and PMT_error_2_count MUST NOT be reported at the
      same time in the same metric block.  If PMT_error_2_count is
      reported, PMT_error_count MUST be set to 0xFFFF.
"

(3) The draft defines PID_error_count as:

"A count of the number of PID_errors that occurred in the above
      sequence number interval.  A PID_error occurs when MPEG TS streams
      are remultiplexed and any PID doesn't refer to an actual data
      stream, as defined in the section 5.2.2 of [ETSI]."


PID_error is defined in 5.2.1 of the ETSI draft, not 5.2.2. Is the definition in 5.2.1 what this field in the block is supposed to correspond to, or is it something else?

[Qin]:Good catch, it should be 5.2.1.

(4) CAT_error_count is defined in the draft as:

"A count of the number of CAT_errors that occurred in the above
      sequence number interval.  A CAT_error occurs when the table has
      an ID other than 0x 01,as defined in the section 5.2.2 of [ETSI].”

CAT_error is defined in the ETSI document as:

"Packets with transport_scrambling_control not 00 present, but no section with table_id = 0x01 (i.e. a
CAT) present
Section with table_id other than 0x01
(i.e. not a CAT) found on PID 0x0001"

[Qin]: How about the following change:
NEW TEXT:
"
   CAT_error_count: 16 bits

      A count of the number of CAT_errors that occurred in the above
      sequence number interval.  A CAT_error occurs when the table has
      an ID other than 0x 01 or section with table_id = 0x01 (i.e. a
      CAT)is not present when scrambling is employed,as defined in the section 5.2.2 of [ETSI].
"

Again I’m curious why the definition in the draft seems to have only one portion of one of the conditions that appear in the ETSI document.

[Qin]: Addressed with the above proposed changes. Thanks.

Thanks,
Alissa


_______________________________________________
xrblock mailing list
xrblock@ietf.org
https://www.ietf.org/mailman/listinfo/xrblock