Re: [sip-overload] draft-ietf-soc-overload-control-13

"Vijay K. Gurbani" <vkg@bell-labs.com> Tue, 06 August 2013 21:57 UTC

Return-Path: <vkg@bell-labs.com>
X-Original-To: sip-overload@ietfa.amsl.com
Delivered-To: sip-overload@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 4D25B21F9F96 for <sip-overload@ietfa.amsl.com>; Tue, 6 Aug 2013 14:57:57 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -110.599
X-Spam-Level:
X-Spam-Status: No, score=-110.599 tagged_above=-999 required=5 tests=[BAYES_00=-2.599, RCVD_IN_DNSWL_HI=-8, USER_IN_WHITELIST=-100]
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 1Gv39g-c772X for <sip-overload@ietfa.amsl.com>; Tue, 6 Aug 2013 14:57:52 -0700 (PDT)
Received: from ihemail1.lucent.com (ihemail1.lucent.com [135.245.0.33]) by ietfa.amsl.com (Postfix) with ESMTP id 6333921F9AA1 for <sip-overload@ietf.org>; Tue, 6 Aug 2013 14:57:52 -0700 (PDT)
Received: from usnavsmail1.ndc.alcatel-lucent.com (usnavsmail1.ndc.alcatel-lucent.com [135.3.39.9]) by ihemail1.lucent.com (8.13.8/IER-o) with ESMTP id r76Lvfgm001049 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=FAIL); Tue, 6 Aug 2013 16:57:42 -0500 (CDT)
Received: from umail.lucent.com (umail.ndc.lucent.com [135.3.40.61]) by usnavsmail1.ndc.alcatel-lucent.com (8.14.3/8.14.3/GMO) with ESMTP id r76LvfS9031888 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NOT); Tue, 6 Aug 2013 16:57:41 -0500
Received: from shoonya.ih.lucent.com (shoonya.ih.lucent.com [135.185.237.229]) by umail.lucent.com (8.13.8/TPES) with ESMTP id r76LveFD003025; Tue, 6 Aug 2013 16:57:40 -0500 (CDT)
Message-ID: <52017265.4070409@bell-labs.com>
Date: Tue, 06 Aug 2013 17:02:13 -0500
From: "Vijay K. Gurbani" <vkg@bell-labs.com>
Organization: Bell Laboratories, Alcatel-Lucent
User-Agent: Mozilla/5.0 (X11; Linux i686; rv:17.0) Gecko/20130625 Thunderbird/17.0.7
MIME-Version: 1.0
To: "Yu, James" <james.yu@neustar.biz>, Christer Holmberg <christer.holmberg@ericsson.com>
References: <56FB15AFE08E1242B0736CBDCE6E85610808C32F@stntexmb12.cis.neustar.com> <OF5E51912C.2DA069C7-ON85257B98.0069B365-85257B98.006B8697@csc.com> <56FB15AFE08E1242B0736CBDCE6E85610808EF90@stntexmb12.cis.neustar.com> <56FB15AFE08E1242B0736CBDCE6E856108096C00@stntexmb12.cis.neustar.com> <OF1E1E4629.9430E707-ON85257BB0.006CEB61-85257BB0.006D574F@csc.com> <56FB15AFE08E1242B0736CBDCE6E856108096C3D@stntexmb12.cis.neustar.com> <7594FB04B1934943A5C02806D1A2204B1C3FC99D@ESESSMB209.ericsson.se> <56FB15AFE08E1242B0736CBDCE6E8561080A4D90@STNTEXMB10.cis.neustar.com> <7594FB04B1934943A5C02806D1A2204B1C41581A@ESESSMB209.ericsson.se> <56FB15AFE08E1242B0736CBDCE6E8561080A51DC@STNTEXMB10.cis.neustar.com>
In-Reply-To: <56FB15AFE08E1242B0736CBDCE6E8561080A51DC@STNTEXMB10.cis.neustar.com>
Content-Type: text/plain; charset="UTF-8"; format="flowed"
Content-Transfer-Encoding: 7bit
X-Scanned-By: MIMEDefang 2.57 on 135.245.2.33
X-Scanned-By: MIMEDefang 2.64 on 135.3.39.9
Cc: "volkerh@bell-labs.com" <volkerh@bell-labs.com>, "sip-overload@ietf.org" <sip-overload@ietf.org>, "hgs@cs.columbia.edu" <hgs@cs.columbia.edu>
Subject: Re: [sip-overload] draft-ietf-soc-overload-control-13
X-BeenThere: sip-overload@ietf.org
X-Mailman-Version: 2.1.12
Precedence: list
List-Id: SIP Overload <sip-overload.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/sip-overload>, <mailto:sip-overload-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/sip-overload>
List-Post: <mailto:sip-overload@ietf.org>
List-Help: <mailto:sip-overload-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/sip-overload>, <mailto:sip-overload-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 06 Aug 2013 21:57:57 -0000

On 07/31/2013 01:05 PM, Yu, James wrote:
> I agree with the change.

James, Christer: I went through the minor change you are suggesting to
the ABNF.  Essentially, you want to change two things:

   - s/algo-list/algo-value/
   - change the variable repetition rule such that it applies to
     the production definition of other-algo

In terms of actual change, this is:

OLD:

    oc-algo     = "oc-algo" EQUAL DQUOTE algo-list *(COMMA algo-list)
                  DQUOTE
    algo-list   = "loss" / *(other-algo)
    other-algo  = %x41-5A / %x61-7A / %x30-39

NEW:

    oc-algo     = "oc-algo" EQUAL DQUOTE algo-value *(COMMA algo-value)
                  DQUOTE
    algo-value  = "loss" / other-algo
    other-algo  = 1*(%x41-5A / %x61-7A / %x30-39)

Note that this change does perturb the intent of the old production
rule (as far as I can see it, at least).  Either of the above grammars
produce the following valid token sequence:

   oc-algo="loss"
   oc-algo="loss,rate"
   oc-algo="loss,rate,window"

However, if the new change appears to read better, I am happy to change
it as suggested.

Thanks,

- vijay
-- 
Vijay K. Gurbani, Bell Laboratories, Alcatel-Lucent
1960 Lucent Lane, Rm. 9C-533, Naperville, Illinois 60563 (USA)
Email: vkg@{bell-labs.com,acm.org} / vijay.gurbani@alcatel-lucent.com
Web: http://ect.bell-labs.com/who/vkg/  | Calendar: http://goo.gl/x3Ogq