Re: [babel] Mirja Kühlewind's Discuss on draft-ietf-babel-rfc6126bis-12: (with DISCUSS and COMMENT)

David Schinazi <dschinazi.ietf@gmail.com> Wed, 11 December 2019 01:15 UTC

Return-Path: <dschinazi.ietf@gmail.com>
X-Original-To: babel@ietfa.amsl.com
Delivered-To: babel@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id E91A0120086; Tue, 10 Dec 2019 17:15:32 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.997
X-Spam-Level:
X-Spam-Status: No, score=-1.997 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, FREEMAIL_FROM=0.001, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_NONE=-0.0001, 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 RfxMMqo2lxVk; Tue, 10 Dec 2019 17:15:30 -0800 (PST)
Received: from mail-lj1-x234.google.com (mail-lj1-x234.google.com [IPv6:2a00:1450:4864:20::234]) (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 0B7A7120059; Tue, 10 Dec 2019 17:15:30 -0800 (PST)
Received: by mail-lj1-x234.google.com with SMTP id 21so22077149ljr.0; Tue, 10 Dec 2019 17:15:29 -0800 (PST)
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=vomdwQLIUTPP9NX821+dqPFt+XvaM3NL2a3jaDsEgNQ=; b=oMwc/ppybPzXbV8leFlpinoIX2kzEI1MV6lwmQ5AbCeXNhWlzibseUMEqvDma6VkDQ HwYrJbIpPesuxwQYvyfQbCcp1TjNvMAM3xFOClvvxDBrSFrnUuYZIc4qz1XaUbgEBscw dFPULO4gJopown0HOHBLWe1/prkA9gLvNjhT5fcnoxYqpS5CoorbayyMU3+cLYK4yW7b nfC7XIn62syfdGBLf/79DEQPs+chZ9stXsjvl/2eo02h3qKMOreP1+YCoF/qq6l1ULE/ ZTvUrlzNsODWq2GofWCy8xNnpHy45B4Q+cr3ZOXx07NltejELHDeBjAvu8l5YtbNebFB 17VA==
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=vomdwQLIUTPP9NX821+dqPFt+XvaM3NL2a3jaDsEgNQ=; b=uUygdH9ZwlRD4kg0IiA02tXRa+pF7ctBm5njvipUYcD5EtLxsTPJoS9Ymh+yx/2kbf 9RuTMIfU8w2FihJo/Od632gvTmd6yvhdG38Gr2zQP7eErzZYMePMP4NfL88ELvf7/aoz afYGXDftgXUZJAil10DCRFXxJIjFxp6p9kxpQ3VcvbkS6FaOip4e6AYw7kQyWhBv6Bx+ 9ib94nlHR8Sqk1VXAdRpezuEr9OnJ0p5j375i18vQM6WQ9zxLysZ47X24+XMk3w229X6 gs3qJMpO+Ll+BMfcaoYwsow2oVBbqvc0kTVWHZptEbzDFEorpN5txFuc9AVzLmWKoYkc sgVw==
X-Gm-Message-State: APjAAAUiB7Uc30gEIvaLiAueVeGjRh7KsAX8fLa4CZt5y+N57Gp/E4EA Fowk91tNunFkWWmGChklo/pfa0mr9zJUfFEpEfE=
X-Google-Smtp-Source: APXvYqzBoU1ymL7pAvEWvrrWK5uvVo/EVP/j22LKvqtIQ8Aoqh6N9rA2jbPRBtMxwW5ktDS1h+zJoUyyOMgsUBsGW3I=
X-Received: by 2002:a2e:241a:: with SMTP id k26mr168439ljk.26.1576026928209; Tue, 10 Dec 2019 17:15:28 -0800 (PST)
MIME-Version: 1.0
References: <156517737995.8257.5538554979559246700.idtracker@ietfa.amsl.com> <877e7m8b88.wl-jch@irif.fr> <1A2B2C1B-1536-4E75-A8D7-C5612FB8AEDA@kuehlewind.net> <87imqzu1vc.wl-jch@irif.fr> <0C555879-5AF3-487F-A65D-95918A546783@kuehlewind.net> <87imomknn0.wl-jch@irif.fr> <B3A7583A-B4DE-4CE5-A74D-0D4C22FABD83@kuehlewind.net> <160C625D-866B-40D3-8549-7E714F8F8E9B@kuehlewind.net>
In-Reply-To: <160C625D-866B-40D3-8549-7E714F8F8E9B@kuehlewind.net>
From: David Schinazi <dschinazi.ietf@gmail.com>
Date: Tue, 10 Dec 2019 17:15:17 -0800
Message-ID: <CAPDSy+6c_WxJ+KoT5uJZG=xCMomDgOukLXHdseQ10yL_+MGyiA@mail.gmail.com>
To: Mirja Kuehlewind <ietf@kuehlewind.net>
Cc: Juliusz Chroboczek <jch@irif.fr>, draft-ietf-babel-rfc6126bis@ietf.org, Donald Eastlake <d3e3e3@gmail.com>, babel-chairs <babel-chairs@ietf.org>, Babel at IETF <babel@ietf.org>, The IESG <iesg@ietf.org>, Alvaro Retana <aretana.ietf@gmail.com>, Martin Vigoureux <martin.vigoureux@nokia.com>
Content-Type: multipart/alternative; boundary="0000000000003664010599635dc5"
Archived-At: <https://mailarchive.ietf.org/arch/msg/babel/NiiDLFkNTPMnST-9GxbnefUs1Bw>
Subject: Re: [babel] =?utf-8?q?Mirja_K=C3=BChlewind=27s_Discuss_on_draft-ietf?= =?utf-8?q?-babel-rfc6126bis-12=3A_=28with_DISCUSS_and_COMMENT=29?=
X-BeenThere: babel@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: "A list for discussion of the Babel Routing Protocol." <babel.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/babel>, <mailto:babel-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/babel/>
List-Post: <mailto:babel@ietf.org>
List-Help: <mailto:babel-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/babel>, <mailto:babel-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 11 Dec 2019 01:15:33 -0000

Hi Mirja,

Thank you for taking another look at the document.

I hope we can get this resolved quickly, as we have
already waited for 49 days for your last email.
I understand that you are busy but I did want to point
out that this is adding stress on the authors.

First, it appears that your review is still treating Babel like
a protocol running over UDP on the Internet. It is not.
Babel only operates point-to-point using IPv6 link-local
addresses, and the congestion properties of 802.11
are very different from the ones of an end-to-end
transport running over the wide area.

Second, the Babel working group prides itself with strict
adherence to the running code principle. Features or
recommendations do not make it into a Babel spec
unless they have been implemented and tested.

If we were discussing a transport protocol, I would
agree with your review. RFC 8085 is a great
document, but it wasn't written with link-local routing
protocols in mind.

As transport AD, it is your responsibility to ensure
new IETF protocols do not bring about another
congestion collapse, and I think that's critical.
I want to really thank you for the hours you spend
reviewing documents to ensure that goal. But I don't
think all of that guidance applies here.

Could I kindly ask you to take another look at your
review in this lens, and please let us know in what
way you think this draft has real-world issues?

We'd really like to make forward progress here, if
the only issue is default values, I'm not sure this is
worth causing delays on, as we haven't seen these
cause problems when implementing the protocol.

Thanks,
David


On Thu, Dec 5, 2019 at 3:53 AM Mirja Kuehlewind <ietf@kuehlewind.net> wrote:

> Hi again,
>
> And one more comment regarding pacing: Again I will not block on this
> issue anymore, however, as you have a babel implementation I would assume
> it would be easy to add more details on how pacing is done in your
> implementation in the draft (at least in the appendix).
>
> Mirja
>
>
>
> > On 5. Dec 2019, at 12:50, Mirja Kuehlewind <ietf@kuehlewind.net> wrote:
> >
> > Hi Juliusz, hi all,
> >
> > I took another look at the updated RFC and I have to say I’m still
> concerned about the default values (which is also an issue that was also
> raised by Alvaro) and the appendix B.
> >
> > You know added another case:
> >
> > "The following values are recommended in a network where reconvergence
> >   within 2 seconds after a mobility event is desired:
> >
> >      Multicast Hello Interval: 0.5 seconds.
> >
> >      Unicast Hello Interval: infinite (no Unicast Hellos are sent).
> >
> >      Link cost, IHU Interval, Update Interval, IHU Hold Time, and Route
> >      Expiry Time: computed as in the first case above.
> >
> >      Request Timeout: initially 0.5 seconds, doubled every time a
> >      request is resent, up to a maximum of three times.
> >
> >      Urgent timeout: 0.2 seconds.
> >
> >      Source GC time: 3 minutes.”
> >
> > These value are much lower than in the first case and can easily leads
> to signification congestion, however, you don’t say anything appropriate
> about when those value might be used or should be avoided. What appendix B
> does is simply given some sample calculation about what the expected
> reconvergence time is based on the selected values but it does not give any
> guidance about which network conditions would bear which setup, nor does it
> talk about negative implication of selecting lower values.
> >
> > I still think that it is important to provide default values as part of
> the normative specification. For me finding such default value (maybe with
> more discussions about in which network scenarios these vales makes sense
> and when it might be useful to consider different values) was part of the
> experiment with babel and if you are not able to address these questions,
> babel is not ready for PS.
> >
> > Alternatively you can check the values in RFC8085 again and see if these
> are acceptable values for most scenarios. Values in that RFC are lightly
> lower than the first example (e.g. 3 secs interval and 1 sec timeout) but
> quite a bit larger than the new example you introduced (without further
> guidance on usage of the values).
> >
> > Especially in section 3.8.2.1. regarding the Request Timeout it is not
> sufficient to only point to the appendix. Please also note that there are
> actually two values here that need suitable default values, the time-out
> itself and the number of retries.
> >
> > Thanks!
> > Mirja
> >
> >
> >> On 18. Oct 2019, at 18:46, Juliusz Chroboczek <jch@irif.fr> wrote:
> >>
> >> Dear Mirja,
> >>
> >>> 1) Let's move appendix B into the body of the document. I think these
> >>> are important information for an implementer and therefore it needs
> more
> >>> attention.
> >>
> >> I am sorry, but I disagree.  The original document that I submitted for
> >> IETF review back in 2009 had the structure that you suggest, but the
> >> reviewer (Joel Halpern) convinced me to a adopt the following structure:
> >>
> >> - the body of the document describes the main algorithm, the parts that
> >>   are required to enforce the strong guarantees that the protocol makes
> >>   about lack of loops and forward progress;
> >> - the appendices describe algorithms that have been shown to be useful
> >>   over some link layers but that might need to be tweaked when Babel is
> >>   run over new, exciting link layers.
> >>
> >> Since Joel did a good job convincing me, it will take some work to
> >> convince me otherwise.
> >>
> >>> 2) Maybe you can add something like: [...]  I’m sure you have a better
> >>> wording. My intention is simply to add another warning, to think
> >>> carefully instead of just pick some default.
> >>
> >> I don't necessarily agree, but I've done as you request.
> >>
> >>>>> Thanks. I was also hoping that you could make any recommendation on
> how
> >>>>> to implement that e.g. a fixed delay of a certain (default) value or
> >>>>> based on some other knowledge. [...]
> >>
> >>>> You do realise it's a very high bar you're setting?
> >>
> >> [...]
> >>
> >>> I wasn’t looking for a tutorial for pacing but rather a reference to
> >>> add. E.g. just naming different approaches for how to implement pacing
> >>> and provide a reference would be a good thing I think. However, I will
> >>> not block on this one thing. Your choice.
> >>
> >> I am not aware of any good reference on packet pacing that covers IEEE
> >> 802.11 multicast (an important usecase for Babel).  After checking some
> >> papers and scratching my head for a long time, I've just added another
> >> sentence to Section 3.1; I'm afraid I'm not competent to do anything
> more.
> >>
> >>> I just wondering why you don’t call them default value rather then
> >>> examples. I think specifying default value normatively would be more
> >>> clear and is what we usually do in specs. However, I think in this case
> >>> it will not make a huge practical difference.
> >>
> >> Great, let's agree that we're not in full agreement, then.
> >>
> >> -- Juliusz
> >>
> >>
> >>
> >
>
>