Re: [Pce] Query regarding draft-ietf-pce-binding-label-sid-02
Dhruv Dhody <dhruv.ietf@gmail.com> Tue, 24 March 2020 14:15 UTC
Return-Path: <dhruv.ietf@gmail.com>
X-Original-To: pce@ietfa.amsl.com
Delivered-To: pce@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id E96083A07D5; Tue, 24 Mar 2020 07:15:16 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.098
X-Spam-Level:
X-Spam-Status: No, score=-2.098 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, FREEMAIL_FROM=0.001, 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 (2048-bit key) header.d=gmail.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 70Q8iCoLtsnr; Tue, 24 Mar 2020 07:15:12 -0700 (PDT)
Received: from mail-il1-x12e.google.com (mail-il1-x12e.google.com [IPv6:2607:f8b0:4864:20::12e]) (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 C7CDE3A077D; Tue, 24 Mar 2020 07:15:01 -0700 (PDT)
Received: by mail-il1-x12e.google.com with SMTP id 7so3456246ill.2; Tue, 24 Mar 2020 07:15:01 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=M6g7Cd+rhN+y9PpCikrFMFX9sqPrtF/dggPKryJVSkI=; b=rCfzFyp+xKFXo6J8+aOkFE9DWhnW3ZMfvW+22597t/K/jR0Xlk7SJh0R71Gw6QWwES EjhEqLmmhk8UwEOIXl8KEs8chB/iuqmQeiW/InTgP6ZWiw23dCneH9AwG/UmFGgUtkgS SIrrqiodh9EhX/oM57O3daJDRLfLlko0EHnc1XPlGUi+FpSwPZMRXfBiWTo21061FcdO NGZg/eXtuXpTNA58NNjo38nRXxZCtT00wg4jtCOdNJE4TFVpLIdmJOaHSzC2Hbx/GSCr +ru/WsyaGMKvmf3N//DP3sui+pi8KyfNRTRWUPfk4Alx/IEDNRlyF/8qedbFbA1fPOHY UtdQ==
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=M6g7Cd+rhN+y9PpCikrFMFX9sqPrtF/dggPKryJVSkI=; b=VQM34zQAlHf8kvAcK5ZEUif1TESdYuADdrHAIct3mBVciZKY/Go0YwkxXcgFmcJ9sI 3V36YX6Rq+ZvTZpGImpLMdAJbD5mcyAkbooSHIucXQcaL4eAWAlLgMdmpwGNBir62c6R 4M9eGvglQiiw8vBUuo8kbJrTH2ny78xWXi5JbkKbKRUEUbA6jCU88R2M+orjeg/gx6dZ qNPQ3pjAZVC1Ag8fI2Mrq+Yx4cTjL1UPqsSqTlTSwOHHsVOvp6E9HX91evPN8z/x8hj8 TrG2gRUP7mQekOGqUvpnUovJULCHfYl9RV4wxNngXfTjF0BP2XsML5lHvFBGSjBXBAP7 QPBA==
X-Gm-Message-State: ANhLgQ18fyfj5Lk9mWLX459JQ5nUvdps9Jjw1/h3XptbIkRDZOwZ+YlF ClLrtvop4DtOT2Xg9OF+z52fw0DCd+cu1m1MYU0=
X-Google-Smtp-Source: ADFU+vu14/hIdPe6AxQ8uEO/nofi9UObcEK3eL+FqGWkf4yRR1H3RM59a0sHmHzNdvEL2kxy31p+vAM3y+mGOvJkfko=
X-Received: by 2002:a92:5859:: with SMTP id m86mr25755677ilb.279.1585059300409; Tue, 24 Mar 2020 07:15:00 -0700 (PDT)
MIME-Version: 1.0
References: <CANVfNKrsBpCOgi1F8abPutz3g7CvyDGU+kJwnfD9tHxvk6orSg@mail.gmail.com> <CAB75xn7jH=U_ZyptfHjsUQw5p=g+g27gtz=bdPbAb6yGykjsDg@mail.gmail.com> <CANVfNKootEiJnvug_GnMZoT5_TZvT-4QnM39rA=a3pxmaD_PRA@mail.gmail.com>
In-Reply-To: <CANVfNKootEiJnvug_GnMZoT5_TZvT-4QnM39rA=a3pxmaD_PRA@mail.gmail.com>
From: Dhruv Dhody <dhruv.ietf@gmail.com>
Date: Tue, 24 Mar 2020 19:44:24 +0530
Message-ID: <CAB75xn4g7ja=Ej6+Dwiw2STZk9Ch7p_Ss9ht+UQ2JwPjuL7qfg@mail.gmail.com>
To: Mrinmoy Das <mrinmoy.ietf@gmail.com>
Cc: Mahend Negi <mahend.ietf@gmail.com>, "Siva Sivabalan (msiva)" <msiva@cisco.com>, "Clarence Filsfils (cfilsfil)" <cfilsfil@cisco.com>, Jeff Tantsura <jefftant.ietf@gmail.com>, Jonathan Hardwick <Jonathan.Hardwick@metaswitch.com>, stefano@previdi.net, "Chengli (Cheng Li)" <chengli13@huawei.com>, pce-chairs <pce-chairs@ietf.org>, pce@ietf.org
Content-Type: text/plain; charset="UTF-8"
Archived-At: <https://mailarchive.ietf.org/arch/msg/pce/1Q4YZ2wEc6uoYtYH8ZzzFOIrhs0>
X-Mailman-Approved-At: Tue, 24 Mar 2020 07:16:08 -0700
Subject: Re: [Pce] Query regarding draft-ietf-pce-binding-label-sid-02
X-BeenThere: pce@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: Path Computation Element <pce.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/pce>, <mailto:pce-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/pce/>
List-Post: <mailto:pce@ietf.org>
List-Help: <mailto:pce-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/pce>, <mailto:pce-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 24 Mar 2020 14:15:17 -0000
Hi Mrinmoy, I will give authors some time to respond and confirm (and spin a new update). I have noted this in the PCE WG wiki [ https://trac.ietf.org/trac/pce/wiki/WikiStart ] to make sure we could track this to closure. Thanks! Dhruv On Tue, Mar 24, 2020 at 7:11 PM Mrinmoy Das <mrinmoy.ietf@gmail.com> wrote: > > Hi Dhruv, > > Thanks for your quick reply. > > I have added PCE WG in this mail. More inline. > > Thanks & Regards, > Mrinmoy > > On Tue, Mar 24, 2020 at 6:00 PM Dhruv Dhody <dhruv.ietf@gmail.com> wrote: >> >> Hi Mrinmoy, >> >> I was suggest you to also include pce@ietf.org; WG could benefit from >> the discussion in future. More inline. >> >> On Tue, Mar 24, 2020 at 5:08 PM Mrinmoy Das <mrinmoy.ietf@gmail.com> wrote: >> > >> > Respected Authors and Contributors, >> > >> > Hope you all are doing well and safe in this tough times of Corona Virus Outbreak. >> > >> > I like to draw your attention regarding some parts of draft-ietf-pce-binding-label-sid-02 which I didn't able to understand properly. >> > >> > 1. BT = 0: The binding value is an MPLS label carried in the format >> > >> > specified in [RFC5462] where only the label value is valid, and >> > other fields (TC, S, and TTL) fields MUST be considered invalid. >> > The Length MUST be set to 7. >> > >> > >> > Previous versions of private draft uses 4 byte to store MPLS 20 Bit label and ignores TC, S & TTL fields. >> >> The length in the previous version was 6, which was incorrect. The TLV >> length is as per https://tools.ietf.org/html/rfc5440#section-7.1 >> Which didnt make sense and your calculation below is correct! >> >> > But in IETF draft after TLV structure redefinition, total length of the TLV becomes 7, i.e. BT(1 Byte)+Reserved(3 Byte)+MPLS 20bit Label(3 Byte) = 7 Byte >> >> Yes >> >> > So, now MPLS 20 Bit Labelwill be stored in 3 byte. Is it correct? >> >> You can consider it a case of rounding up 20 bits to 3 bytes. >> >> >> > If this is correct then I feel the wording of the above paragraph needs to be more specific, meaning in 3 Byte Label there will be no space for TTL, so >> > my suggestion is to make below correction: >> > >> > BT = 0: The binding value is an MPLS label carried in the format >> > >> > specified in [RFC5462] where only the label value is valid, and >> > other fields (TC, S, and TTL) fields MUST be considered invalid. >> > The Length MUST be set to 7. >> > >> >> My suggestion would be not mention any of the other fields and talk >> only of 20 bits of Label. I see other SR RFCs take similar approach. > > > Sounds Good. I'm agree with you. > >> >> > >> > 2. In some cases, a stateful PCE can request the PCC to allocate a >> > >> > binding value. It may do so by sending a PCUpd message containing an >> > empty TE-PATH-BINDING TLV, i.e., no binding value is specified >> > (making the length field of the TLV as 2). A PCE can also make the >> > request PCC to allocate a binding at the time of initiation by >> > sending a PCInitiate message with an empty TE-PATH-BINDING TLV. >> > >> > >> > As per new Binding TLV Structure below, BT is of 1 Byte and there will be 3 Byte Reserved. >> > >> > 0 1 2 3 >> > >> > 0 1 2 3 4 5 6 7 8 9 0 1 2 3 4 5 6 7 8 9 0 1 2 3 4 5 6 7 8 9 0 1 >> > +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+ >> > | Type | Length | >> > +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+ >> > | BT | Reserved | >> > +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+ >> > ~ Binding Value (variable length) ~ >> > +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+ >> > >> > >> > Now, an empty Binding TLV should have length of BT(1 Byte) + Reserved (3 Byte) = 4 Byte instead of 2 Byte. >> > >> > So, I do not understand how in the draft it is calculated as 2 Byte. Could you please give me some clue? >> >> >> This seems to be an error IMHO. 4 seems to be correct. > > > Okay. So what would be your suggestion to developer who is implementing this draft? Should it be taken as 4? If draft needs correction > when will that be published? >> >> >> Thanks! >> Dhruv >> >> > >> > Thanks & Regards, >> > Mrinmoy
- Re: [Pce] Query regarding draft-ietf-pce-binding-… Mrinmoy Das
- Re: [Pce] Query regarding draft-ietf-pce-binding-… Dhruv Dhody
- Re: [Pce] Query regarding draft-ietf-pce-binding-… Mrinmoy Das
- Re: [Pce] Query regarding draft-ietf-pce-binding-… stefano previdi
- Re: [Pce] Query regarding draft-ietf-pce-binding-… Mrinmoy Das
- Re: [Pce] Query regarding draft-ietf-pce-binding-… Dhruv Dhody
- Re: [Pce] Query regarding draft-ietf-pce-binding-… Mrinmoy Das