Re: [multipathtcp] 6824bis-11 review
Xavier de Foy <x.defoy.ietf@gmail.com> Wed, 19 September 2018 16:38 UTC
Return-Path: <x.defoy.ietf@gmail.com>
X-Original-To: multipathtcp@ietfa.amsl.com
Delivered-To: multipathtcp@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id D24AC131065 for <multipathtcp@ietfa.amsl.com>; Wed, 19 Sep 2018 09:38:26 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.999
X-Spam-Level:
X-Spam-Status: No, score=-1.999 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_PASS=-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 xPilCah24Kxi for <multipathtcp@ietfa.amsl.com>; Wed, 19 Sep 2018 09:38:25 -0700 (PDT)
Received: from mail-lj1-x22b.google.com (mail-lj1-x22b.google.com [IPv6:2a00:1450:4864:20::22b]) (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 BA274131031 for <multipathtcp@ietf.org>; Wed, 19 Sep 2018 09:38:24 -0700 (PDT)
Received: by mail-lj1-x22b.google.com with SMTP id u83-v6so5646642lje.12 for <multipathtcp@ietf.org>; Wed, 19 Sep 2018 09:38:24 -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=+RVZKpF8mN5POgKS2fAXpFPHM6JAXT9AgbDAQD/22Po=; b=BSw6VqyFc4MJCiKjH836RsM4Hbv56OhYyrN3tRXtECNvDRKbRMFbW94PJoSjYIC6pj 3OlUgY1X4XZ+TBVZrLTHMZs7T+VsjeTZdUDBJRJDgP47VIVKFceZcF4byjuC4PUCUTft s6S4GL5hiN/UG+pfD+/cOqs+ASSfJbI6PikUX2P6nM9tFO2jka3n4IL0X6H2W2gnNF+3 UF2Djoqk0euvOl2t1dJfXl9PtPivbSiDj/n8hlYnrdA+uBJJPazNA/ezD6lNOIaz0+Fo cHwbBUwysvkekpe9G+jQF5ytIakC270NlZMhO3gZ5w4v63lliR/wCGaAW3l8RF51PRvk npPQ==
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=+RVZKpF8mN5POgKS2fAXpFPHM6JAXT9AgbDAQD/22Po=; b=b00kGS1Hn5jJl0UcLunLip604ToS5fKxPy8IL9G0LjKrU57i+Z7QObx+eRl4tgpfun bC9wWNsCCDE02CMi3SLYdk6GE/BjRg/6RQvu/dZozIaTaktrSwNwJfZrhJFmq830v9SW Lhnf31fyduLBalPdCui/oqlQ6r/l8PY4PNJBcqo9Ww7dhRBtO2BdI9sQtQslmJWTw7qf YyCwxMnhaIGXKxI+hl+9hEaH8dcqFM5p9pIJ2Z7noe84/oywyIEDFLbKs3TTksmLQCsJ J90m5OID2ZomhufF42/3igztGdabNgJImow2490K7sXFAiWj4aKQPbi6S6rjcv+rw1Oj wGcQ==
X-Gm-Message-State: APzg51CSYMYfSM5tsH/FbkfOUhWSCIMw4m6ek6ZbIjzLdIfsdP6+ORO8 nGGrtomhXzSu5pz8pffo4A5Bf3r7AH0isw8Gi7c=
X-Google-Smtp-Source: ANB0Vda8f8J5DHHB9L4gJhQnmGRTD8VV5shsEHPA4j88kNZqebUWi6yRH4d/Kxm5hFULZCtjBG/xHze6Jxov6HmTwpI=
X-Received: by 2002:a2e:9a93:: with SMTP id p19-v6mr19579455lji.108.1537375102826; Wed, 19 Sep 2018 09:38:22 -0700 (PDT)
MIME-Version: 1.0
References: <CAHYjOTZeD+o_Nvj2cGJ8AgnNPFx7i7KHfxx7XL6-nqDYNWbDjA@mail.gmail.com> <632AD775-BFBC-4DC3-BBF5-E741F276F914@gmail.com>
In-Reply-To: <632AD775-BFBC-4DC3-BBF5-E741F276F914@gmail.com>
From: Xavier de Foy <x.defoy.ietf@gmail.com>
Date: Wed, 19 Sep 2018 12:38:09 -0400
Message-ID: <CAHYjOTY+HCCJNS5pA9PYjoaOgUM=kniEiQtH4Tcw-CUwH3=y5g@mail.gmail.com>
To: alan.ford@gmail.com
Cc: multipathtcp <multipathtcp@ietf.org>
Content-Type: multipart/alternative; boundary="000000000000e3fd9305763c08a5"
Archived-At: <https://mailarchive.ietf.org/arch/msg/multipathtcp/lOCzti2WmNH4sfS_FIRbFjbTZto>
Subject: Re: [multipathtcp] 6824bis-11 review
X-BeenThere: multipathtcp@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: Multi-path extensions for TCP <multipathtcp.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/multipathtcp>, <mailto:multipathtcp-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/multipathtcp/>
List-Post: <mailto:multipathtcp@ietf.org>
List-Help: <mailto:multipathtcp-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/multipathtcp>, <mailto:multipathtcp-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 19 Sep 2018 16:38:34 -0000
Thanks Alan for addressing the comments, and for the additional details you provided. I agree with your responses, with just one possible clarification below, and an answer to your question. There is no open issue with this review, from my standpoint (the following points are minor). >"The Address ID MUST uniquely identify the address for the sender of the option (within the scope of the >connection), but the mechanism for allocating such IDs is implementation specific." Even after this rephrasing I still wonder if address ID #1 can designate an IP address local to peer A, and the same ID #1 can (in the same session) designate another IP address local to peer B. Would you find the following phrasing clearer? "The Address ID MUST uniquely identify the address within the scope of the connection (uniqueness is only required among the IP addresses local to the sender of the option), but the mechanism for allocating such IDs is implementation specific." >> *3.8. Error Handling >> >> - Is the list of errors complete? (e.g. ADD_ADDR errors seem to belong to the same class as remove request) > >What kind of ADD_ADDR error are you thinking of? For example: adding an address with an invalid HMAC, or invalid values in some fields. Also (based on your recent edits) existing address ID but a different IP address. More generally, the goal of my comment was just to bring your attention on the fact that not all classes of errors may be covered. But it's possible that the authors only wanted to cover high level error handling strategies. So it's really an editorial choice, please feel free to address this comment as you see fit, there is no need to follow-up on this. Best Regards, Xavier.
- [multipathtcp] 6824bis-11 review Xavier de Foy
- Re: [multipathtcp] 6824bis-11 review Alan Ford
- Re: [multipathtcp] 6824bis-11 review philip.eardley
- Re: [multipathtcp] 6824bis-11 review Alan Ford
- Re: [multipathtcp] 6824bis-11 review philip.eardley
- Re: [multipathtcp] 6824bis-11 review Xavier de Foy
- Re: [multipathtcp] 6824bis-11 review Christoph Paasch