Re: [CCAMP] AD review of draft-ietf-ccamp-assoc-ext

"Francois Le Faucheur (flefauch)" <flefauch@cisco.com> Wed, 08 August 2012 16:02 UTC

Return-Path: <flefauch@cisco.com>
X-Original-To: ccamp@ietfa.amsl.com
Delivered-To: ccamp@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 5959121F86F0 for <ccamp@ietfa.amsl.com>; Wed, 8 Aug 2012 09:02:25 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -10.352
X-Spam-Level:
X-Spam-Status: No, score=-10.352 tagged_above=-999 required=5 tests=[AWL=0.246, BAYES_00=-2.599, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_HI=-8]
Received: from mail.ietf.org ([12.22.58.30]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id wxfmuiBNxe+o for <ccamp@ietfa.amsl.com>; Wed, 8 Aug 2012 09:02:24 -0700 (PDT)
Received: from rcdn-iport-9.cisco.com (rcdn-iport-9.cisco.com [173.37.86.80]) by ietfa.amsl.com (Postfix) with ESMTP id E53C921F86EB for <ccamp@ietf.org>; Wed, 8 Aug 2012 09:02:23 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=cisco.com; i=@cisco.com; l=20022; q=dns/txt; s=iport; t=1344441744; x=1345651344; h=from:to:cc:subject:date:message-id:references: in-reply-to:mime-version; bh=xCN/w47AJHb7q3XRaWG8zalntCmaa980XmYefGAjst4=; b=QX4TjcPV8z0qLlqvULAoVTsSTF4DT7uVkXF621zxtQGoFNLNSKrhIn+H KYxSyD4TRXyMmr7bziNq8LgGBLHwR/eFN0buyu592tL+xolNHSKdQmJ6x dgynp1+vrFVGU6Jy0ImmUYoGaOiJN28apYZ8JHsHwCLCw1JuCPzOXXR2p I=;
X-IronPort-Anti-Spam-Filtered: true
X-IronPort-Anti-Spam-Result: Av0EAD+NIlCtJV2a/2dsb2JhbABFuVyBB4IhAQEEEgEUUhACAQgOODIlAQEEDieHa5p8oD6REmADlUmOKYFmgl8
X-IronPort-AV: E=Sophos; i="4.77,733,1336348800"; d="scan'208,217"; a="106631874"
Received: from rcdn-core-3.cisco.com ([173.37.93.154]) by rcdn-iport-9.cisco.com with ESMTP; 08 Aug 2012 16:02:23 +0000
Received: from xhc-aln-x01.cisco.com (xhc-aln-x01.cisco.com [173.36.12.75]) by rcdn-core-3.cisco.com (8.14.5/8.14.5) with ESMTP id q78G2NDf006885 (version=TLSv1/SSLv3 cipher=AES128-SHA bits=128 verify=FAIL); Wed, 8 Aug 2012 16:02:23 GMT
Received: from xmb-rcd-x10.cisco.com ([169.254.15.216]) by xhc-aln-x01.cisco.com ([173.36.12.75]) with mapi id 14.02.0298.004; Wed, 8 Aug 2012 11:02:23 -0500
From: "Francois Le Faucheur (flefauch)" <flefauch@cisco.com>
To: Lou Berger <lberger@labn.net>
Thread-Topic: [CCAMP] AD review of draft-ietf-ccamp-assoc-ext
Thread-Index: Ac11NY6nevEXepS5RqKUu/wkiJbhHgAZiUcAAANRhIA=
Date: Wed, 08 Aug 2012 16:02:21 +0000
Message-ID: <67596B46-B753-46DC-93B8-8297EFFAF5DD@cisco.com>
References: <021a01cd7536$2104aa10$630dfe30$@olddog.co.uk> <50227712.4010203@labn.net>
In-Reply-To: <50227712.4010203@labn.net>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-originating-ip: [10.55.161.197]
x-tm-as-product-ver: SMEX-10.2.0.1135-7.000.1014-19092.004
x-tm-as-result: No--45.211800-8.000000-31
x-tm-as-user-approved-sender: No
x-tm-as-user-blocked-sender: No
Content-Type: multipart/alternative; boundary="_000_67596B46B75346DC93B88297EFFAF5DDciscocom_"
MIME-Version: 1.0
Cc: CCAMP <ccamp@ietf.org>, "<draft-ietf-ccamp-assoc-ext.all@tools.ietf.org>" <draft-ietf-ccamp-assoc-ext.all@tools.ietf.org>
Subject: Re: [CCAMP] AD review of draft-ietf-ccamp-assoc-ext
X-BeenThere: ccamp@ietf.org
X-Mailman-Version: 2.1.12
Precedence: list
List-Id: Discussion list for the CCAMP working group <ccamp.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/ccamp>, <mailto:ccamp-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/ccamp>
List-Post: <mailto:ccamp@ietf.org>
List-Help: <mailto:ccamp-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/ccamp>, <mailto:ccamp-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 08 Aug 2012 16:02:25 -0000

Adrian, Lou,

Additional thoughts below:

===

To start with, a key question:

Why does the working group want to publish this as an RFC when there
is no immediate intention to implement for any of the many scenarios
described in the document?


Implementation for some of the discussed scenarios is being considered. This RFC will allow such implementation to be open for interoperability.


In other words: is
it your intention that all new implementations of RSVP MUST include an
implementation of this document?

Absolutely not the intent.

Agree with Lou. Not the intent. So in that sense, probably not an "update" to 2205.

---

Section 1 Voice Call Waiting

      However,
      there is no way in RSVP today to share the resources between the
      A->B and A->C subflows of the call since by definition the RSVP
      reservations for these subflows must have different IP addresses
      in the SESSION objects.

Since you are defining such a mechanism, "there is no way today" is not
true. I suggest...

      Since, by definition, the RSVP reservations for the subflows A->B
      and A->C of the call must have different IP addresses in the
      SESSION objects, this document defines a new mechanism to
      associate the subflows and allow them to share resources.

looks good to me. (assuming my co-authors don't object.)

Works for me.



Similarly...

OLD
    o Voice Shared Line:
      A single number that rings multiple endpoints (which may be
      geographically diverse), such as phone lines on a manager's desk
      and their assistant.  A VoIP system that models these calls as
      multiple P2P unicast pre-ring reservations would result in
      significantly over-counting bandwidth on shared links, since
      today unicast reservations to different endpoints cannot share
      bandwidth.
NEW
    o Voice Shared Line:
      A voice shared line is a single number that rings multiple
      endpoints (which may be geographically diverse), such as phone
      lines to a manager's desk and to their assistant.  A VoIP system
      that models these calls as multiple P2P unicast pre-ring
      reservations would result in significantly over-counting
      bandwidth on shared links, since RSVP unicast reservations to
      different endpoints cannot share bandwidth.  So a new mechanism
      is defined in this document allowing separate unicast
      reservations to be associated and share resources.
END

okay (assuming my co-authors don't object.)

Works for me.



And...

OLD
    o Symmetric NAT:
      RSVP permits sharing of resources between multiple flows
      addressed to the same destination D, even from different senders
      S1 and S2.  However, if D is behind a NAT operating in symmetric
      mode [RFC5389], it is possible that the destination port of the
      flows S1->D and S2->D may be different outside the NAT.  In this
      case, these flows cannot share resources using RSVP today, since
      the SESSION objects for these two flows outside the NAT would
      have different ports.
NEW
    o Symmetric NAT:
      RSVP permits sharing of resources between multiple flows
      addressed to the same destination D, even from different senders
      S1 and S2.  However, if D is behind a NAT operating in symmetric
      mode [RFC5389], it is possible that the destination port of the
      flows S1->D and S2->D may be different outside the NAT.  In this
      case, these flows cannot share resources using RSVP, since the
      SESSION objects for these two flows outside the NAT have
      different ports.  This document defines a new mechanisms to
      associate these flows and allow them to share resources.
END

okay (assuming my co-authors don't object.)

Works for me.



3.1.1 vs 3.2.1

In 3.1.1 you have...
  Relative ordering of ASSOCIATION objects of the same
  type SHOULD be preserved by transit nodes.


note this is path processing,

In 3.2.1 you have...
  Relative ordering of ASSOCIATION objects of the same
  type MUST be preserved by transit nodes.  Association type specific
  ordering requirements MAY be defined in the future.

Note this is resv processing.

1. Why is 3.1.1 SHOULD and 3.1.2 MUST?

I'm not sure, other than 3.1.2 is new so there's no chance of placing a
new requirement on implementations of 4872&3 (which only use association
objects in Path messages.)

I believe the idea is that while ordering is currently irrelevant, it might be relevant in some future application so we may as well avoid re-ordering when it is easy (i.e. new implementation).  Hence the logic of SHOULD & MUST. I am happy either way SHOULD/MUST or SHOULD/SHOULD.




I'm fine with should for both.

Co-authors?


4. Why "MAY" not "may"?

over zealousness.  i.e., your right!

or maybe something like :
"Note that association type specific ordering requirements could be defined in the future."


---

3.3.1

  Once an association is identified, resources SHOULD be shared across
  the identified sessions.

This use of "SHOULD" begs the question: under what circumstances MAY
resource sharing not be applied?

actual resource allocation is really an implementation choice and
different internal implementation choices.  We didn't want to overly
constrain an implementation. "is expected" is really what we mean, but
how do you say this in 2119 language?

2205 uses the term "Admission control", so would it work if we said:
"
Once an association is identified, resources MUST be considered as shared across
  the identified sessions by the admission control function.
"



Francois