Re: [v6ops] Working Group Last Call draft-ietf-v6ops-nd-cache-init

Fernando Gont <fernando@gont.com.ar> Tue, 09 June 2020 20:22 UTC

Return-Path: <fernando@gont.com.ar>
X-Original-To: v6ops@ietfa.amsl.com
Delivered-To: v6ops@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 2CB413A0855 for <v6ops@ietfa.amsl.com>; Tue, 9 Jun 2020 13:22:08 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.899
X-Spam-Level:
X-Spam-Status: No, score=-1.899 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, URIBL_BLOCKED=0.001] autolearn=unavailable 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 68F051JULYER for <v6ops@ietfa.amsl.com>; Tue, 9 Jun 2020 13:22:06 -0700 (PDT)
Received: from tools.si6networks.com (v6toolkit.go6lab.si [91.239.96.57]) (using TLSv1.2 with cipher ADH-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 088383A08C1 for <v6ops@ietf.org>; Tue, 9 Jun 2020 13:21:59 -0700 (PDT)
Received: from [IPv6:2800:810:464:1f7:d11c:c784:f994:a8b9] (unknown [IPv6:2800:810:464:1f7:d11c:c784:f994:a8b9]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by tools.si6networks.com (Postfix) with ESMTPSA id 27A103FF81; Tue, 9 Jun 2020 22:21:55 +0200 (CEST)
To: Jen Linkova <furry13@gmail.com>
Cc: Jen Linkova <furry@google.com>, Ron Bonica <rbonica=40juniper.net@dmarc.ietf.org>, "v6ops@ietf.org" <v6ops@ietf.org>
References: <202005251539.04PFd03X096978@svl-junos-d082.juniper.net> <DM6PR05MB63487F4BDD307CD934033769AEB30@DM6PR05MB6348.namprd05.prod.outlook.com> <62458b61-9ba3-0ec2-eea9-58db0782b03f@gont.com.ar> <CAFU7BAQXkUG0E=C0hoUC8w6JyVg-NVj7T9RV9zNGSaB2BmXTmQ@mail.gmail.com>
From: Fernando Gont <fernando@gont.com.ar>
Message-ID: <41de0704-1bcc-fc77-9a84-2da91d1bcff8@gont.com.ar>
Date: Tue, 09 Jun 2020 17:21:49 -0300
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.9.1
MIME-Version: 1.0
In-Reply-To: <CAFU7BAQXkUG0E=C0hoUC8w6JyVg-NVj7T9RV9zNGSaB2BmXTmQ@mail.gmail.com>
Content-Type: text/plain; charset="utf-8"; format="flowed"
Content-Language: en-US
Content-Transfer-Encoding: 7bit
Archived-At: <https://mailarchive.ietf.org/arch/msg/v6ops/IYF4g0ZxMApN3njBKT0RVKBj8lw>
Subject: Re: [v6ops] Working Group Last Call draft-ietf-v6ops-nd-cache-init
X-BeenThere: v6ops@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: v6ops discussion list <v6ops.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/v6ops>, <mailto:v6ops-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/v6ops/>
List-Post: <mailto:v6ops@ietf.org>
List-Help: <mailto:v6ops-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/v6ops>, <mailto:v6ops-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 09 Jun 2020 20:22:08 -0000

Hello, Jen,

On 9/6/20 09:21, Jen Linkova wrote:
> On Sun, May 31, 2020 at 7:05 PM Fernando Gont <fernando@gont.com.ar> wrote:
>> I support publication of this document. Bellow you'll find my review:
> 
> Thanks for your review and comments Fernando!
> I've just submitted -02 which shall address your comments.

Thanks!



>> * Section 3.4:
>>>
>>>     o  If the address is in the Optimistic state the RS can not contain
>>>        SLLAO.  As a result the router would only create a cache entry if
>>>        the solicited RAs is sent as as a unicast.  Routers sending
>>>        solicited RAs as multicast would not create a new cache entry as
>>>        they do not need to send a unicast packet back to the host.
>>
>> Of the top of my head, I'm not sure if an RA sent to a multicast address
>> can be a "solicited RA" (as in "having the 'solicited' flag set). Please
>> double-check.
> 
> There is no such thing as a 'solicited' flag in RA.

I have no idea what I was thinking when I wrote this :-) -- sleep 
disorder might be a plausible explanation :-))


>> * Section 3:
>>
>> Thinking out loud: One might move this section (and subsections) to an
>> Appendix?
> 
> IMHO, as some of those options are implemented by some vendors, so I
> believe it makes sense to discuss them.

I would suggest that you note this at the beginning of this section 
(even if it's just a general comment like the one above ("some of these 
options are implemented by some vendors.."). That would signal folks 
that they might expect to see some of this in the wild.  -- Otherwise 
(like myself), they might assume that these are just potential solutions 
that you evaluated and then discarded.

FWIW, if you decide to apply this "change", feel free to schedule it for 
the next rev (i.e., this is not a show stopper, and you don't need a rev 
just for this).

Thanks!

P.S.: Not sure what's the necessary magic for this to happen, but, if 
possible (i.e., assuming that both docs are progressed smoothly and 
timely), I'd suggest that this doc is put in a cluster with the 
companion 6man document, such that both reference each other's RFC 
versions when published.

Cheers,
-- 
Fernando Gont
e-mail: fernando@gont.com.ar || fgont@si6networks.com
PGP Fingerprint: 7809 84F5 322E 45C7 F1C9 3945 96EE A9EF D076 FFF1