Re: [Detnet-dp-dt] Draft update in github

Loa Andersson <loa@pi.nu> Sat, 11 March 2017 06:00 UTC

Return-Path: <loa@pi.nu>
X-Original-To: detnet-dp-dt@ietfa.amsl.com
Delivered-To: detnet-dp-dt@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id DF5521294E1 for <detnet-dp-dt@ietfa.amsl.com>; Fri, 10 Mar 2017 22:00:11 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.9
X-Spam-Level:
X-Spam-Status: No, score=-1.9 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RP_MATCHES_RCVD=-0.001, URIBL_BLOCKED=0.001] autolearn=ham autolearn_force=no
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 8JtJ9JWJgbwh for <detnet-dp-dt@ietfa.amsl.com>; Fri, 10 Mar 2017 22:00:09 -0800 (PST)
Received: from pipi.pi.nu (pipi.pi.nu [83.168.239.141]) (using TLSv1.1 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id AB6BD120725 for <detnet-dp-dt@ietf.org>; Fri, 10 Mar 2017 22:00:09 -0800 (PST)
Received: from [192.168.1.11] (unknown [119.95.38.221]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) (Authenticated sender: loa@pi.nu) by pipi.pi.nu (Postfix) with ESMTPSA id E724418013D1 for <detnet-dp-dt@ietf.org>; Sat, 11 Mar 2017 06:59:48 +0100 (CET)
To: detnet-dp-dt@ietf.org
References: <88BD4A49-3A2C-44DD-A090-E7A3AAC8BF61@broadcom.com> <3eaa6ced-1493-a0da-125f-694567de0107@ericsson.com> <EEF82160-BCA3-4C22-8C81-4D76EC8B9BDB@broadcom.com>
From: Loa Andersson <loa@pi.nu>
Message-ID: <625697e2-4fa5-9c7d-11ab-4c79a37040a4@pi.nu>
Date: Sat, 11 Mar 2017 13:59:44 +0800
User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:45.0) Gecko/20100101 Thunderbird/45.8.0
MIME-Version: 1.0
In-Reply-To: <EEF82160-BCA3-4C22-8C81-4D76EC8B9BDB@broadcom.com>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Transfer-Encoding: 8bit
Archived-At: <https://mailarchive.ietf.org/arch/msg/detnet-dp-dt/4cGDDfeCSJw7mIXBh1wSULDj_Jc>
Subject: Re: [Detnet-dp-dt] Draft update in github
X-BeenThere: detnet-dp-dt@ietf.org
X-Mailman-Version: 2.1.17
Precedence: list
List-Id: DetNet WG Data Plane Design Team <detnet-dp-dt.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/detnet-dp-dt>, <mailto:detnet-dp-dt-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/detnet-dp-dt/>
List-Post: <mailto:detnet-dp-dt@ietf.org>
List-Help: <mailto:detnet-dp-dt-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/detnet-dp-dt>, <mailto:detnet-dp-dt-request@ietf.org?subject=subscribe>
X-List-Received-Date: Sat, 11 Mar 2017 06:00:12 -0000

Jouni,

Nit picking

LSR stands for Label Switching Router
                            ^^^

In the terminology section and section 4.2.

I think PREF and FREF should go into terminology section.

/Loa

On 2017-03-11 04:22, Jouni Korhonen wrote:
> Thanks.
>
>> On Mar 10, 2017, at 10:01 AM, János Farkas <janos.farkas@ericsson.com> wrote:
>>
>> Hi Jouni,
>>
>> I've made some changes to the draft. The attached pdf indicates them.
>> I've updated the xml version where you moved Balazs' text to be main text. I hope i did not mess xml versioning.
>> (the commented pdf is one version earlier)
>
> Nope. I rebased my ongoing edits to yours so everything is fine. Nothing git cannot do ;)
>
>
>> Well, I did not make any dramatic change.
>> I'd highlight two:
>> a) replaced "data plane protocol" with "data plane solution". We are working on solution as reflected by "sol" in the file name ;-). I mean, a protocol is rather above the data plane, e.g., in the control plane.
>> b) replaced "domain" with alternatives. DetNet is for single domain. Actually, domain is an overloaded term: administrative domain, etc., better to avoid.
>
> Ack.
>
>> I highlighted one thing in the pdw, which I did not implement in the xml:
>> "frame replication and elimination"
>> The architecture document uses "packet replication and elimination", which I think would be better to be used in this doc too; except for he cases when we explicitly refer to 802.1CB.
>> This appears multiple places, another variant: ""frame replication and duplicate elimination".
>> If there is agreement then we could make the change.
>
> Let’s go for the arch doc way. So “Packet Replication and Elimination for Redundancy” (PREF) ?
>
> - Jouni
>
>
>>
>> Regards,
>> Janos
>>
>>
>> On 3/10/2017 5:55 AM, Jouni Korhonen wrote:
>>> Folks,
>>>
>>> I did quite a bit of updates to the draft. All in GitHub. I’ll continue writing over the weekend etc..
>>>
>>> - Jouni
>>>
>>>
>>
>> <draft-dt-detnet-dp-sol-00 - 2017-03-10-JF-comments.pdf>
>
> _______________________________________________
> Detnet-dp-dt mailing list
> Detnet-dp-dt@ietf.org
> https://www.ietf.org/mailman/listinfo/detnet-dp-dt
>

-- 


Loa Andersson                        email: loa@mail01.huawei.com
Senior MPLS Expert                          loa@pi.nu
Huawei Technologies (consultant)     phone: +46 739 81 21 64