Re: [AVTCORE] Spencer Dawkins' Discuss on draft-ietf-avtcore-rtp-circuit-breakers-14: (with DISCUSS)

Spencer Dawkins at IETF <spencerdawkins.ietf@gmail.com> Tue, 26 April 2016 22:08 UTC

Return-Path: <spencerdawkins.ietf@gmail.com>
X-Original-To: avt@ietfa.amsl.com
Delivered-To: avt@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id DAD6612D126; Tue, 26 Apr 2016 15:08:18 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.699
X-Spam-Level:
X-Spam-Status: No, score=-2.699 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_LOW=-0.7, 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 H6wZRzrjfdTV; Tue, 26 Apr 2016 15:08:16 -0700 (PDT)
Received: from mail-yw0-x22c.google.com (mail-yw0-x22c.google.com [IPv6:2607:f8b0:4002:c05::22c]) (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 2BD2512D0CF; Tue, 26 Apr 2016 15:08:13 -0700 (PDT)
Received: by mail-yw0-x22c.google.com with SMTP id g133so36972414ywb.2; Tue, 26 Apr 2016 15:08:13 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc; bh=rkIfB+Z3tCWDNxw4F8wfQHqOPCncTTllbrcC1TGFgbc=; b=vVpwNW+xfZaPZKQN/rZa7rWKhvShGqgWh6F8UfEkS/dWtllMvxHsqHpTJZlkUNCtYl FUpPo9oKfKyOHKu24oMkrIH/CVF8mxnxKELbr73xK/BneP72c4YbchbJI5aeJGOOGQiD n72tLtKYj9bdUXh3lvkExbk/QK1VbMCNiT5dL6b4QqeiHJFP1CuYlEPWTf9cLXdsb5La qbaFOTu0EiC+xIy9F251nmcSinSWq/PcFItjv+tv2/f3euDroXOWvOC7nXsCNJ0RDu/V gvSKB96pJ6u/q9+ph9lE1Fi5hOJJg6+/EfhIeyWdgzJn17hHj4VQ/8NQdJcPGW/XTo1d QarQ==
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:mime-version:in-reply-to:references:date :message-id:subject:from:to:cc; bh=rkIfB+Z3tCWDNxw4F8wfQHqOPCncTTllbrcC1TGFgbc=; b=ZLuM9ysj8gqpLXV+38lDIebLvCHk1h4YVnbd4FkyMd6LlA9zuNbKB51cS9rEYE47zJ pwWeqn2Jh5bKbl7Ww9JmgIOPX0ge2i8cRz9sa9ZkvQvHuDviZnZUHo5zKa0zE6UPtQN3 0tUs/n9PIQd2sAC8sTRISrd4RDMIVFuTkGmCO6uwJz0w27qtz552IWpDwBcHH/M39lQy PZVbnUbF/zoDBh8Uo1MtBNRWosVOdvnYt85/m5JLYkgOWBiAPeGyFXPP57uHY2U8AVSS NwVh99HnXmABh2SXPEKwrBSP/mkN8S12r/Iy33jQrBF0exVind7ktLeKxCeizZh2YvhZ zXbA==
X-Gm-Message-State: AOPr4FWAEUdJo+8WASC4sIX/ois6c1E/fYFNv2xxvAF9OA1mkT5sCwsP/nHm9IiJve8CqKSDV3A3aFwztAr1xA==
MIME-Version: 1.0
X-Received: by 10.129.46.80 with SMTP id u77mr2786912ywu.143.1461708492392; Tue, 26 Apr 2016 15:08:12 -0700 (PDT)
Received: by 10.37.224.212 with HTTP; Tue, 26 Apr 2016 15:08:12 -0700 (PDT)
In-Reply-To: <78034C12-57A3-4680-8B86-5C8F22E8ED19@csperkins.org>
References: <20160420012041.31613.87215.idtracker@ietfa.amsl.com> <78034C12-57A3-4680-8B86-5C8F22E8ED19@csperkins.org>
Date: Tue, 26 Apr 2016 17:08:12 -0500
Message-ID: <CAKKJt-e5xQjkHQ8AWpp6N1eUM_KvFNuJ_CyNE7QdQyYjzAPHUw@mail.gmail.com>
From: Spencer Dawkins at IETF <spencerdawkins.ietf@gmail.com>
To: Colin Perkins <csp@csperkins.org>
Content-Type: multipart/alternative; boundary="001a11409dae74513f05316a878f"
Archived-At: <http://mailarchive.ietf.org/arch/msg/avt/h9mtt8Yyh2lPOva8UsRtPEffuVI>
Cc: avtcore-chairs@ietf.org, Magnus Westerlund <magnus.westerlund@ericsson.com>, draft-ietf-avtcore-rtp-circuit-breakers@ietf.org, The IESG <iesg@ietf.org>, avt@ietf.org
Subject: Re: [AVTCORE] Spencer Dawkins' Discuss on draft-ietf-avtcore-rtp-circuit-breakers-14: (with DISCUSS)
X-BeenThere: avt@ietf.org
X-Mailman-Version: 2.1.17
Precedence: list
List-Id: Audio/Video Transport Core Maintenance <avt.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/avt>, <mailto:avt-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/avt/>
List-Post: <mailto:avt@ietf.org>
List-Help: <mailto:avt-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/avt>, <mailto:avt-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 26 Apr 2016 22:08:19 -0000

Hi, Colin,

On Tue, Apr 26, 2016 at 9:43 AM, Colin Perkins <csp@csperkins.org> wrote:

> Spencer,
>
> > On 20 Apr 2016, at 02:20, Spencer Dawkins <spencer.dawkins@huawei.com>
> wrote:
> >
> > Spencer Dawkins has entered the following ballot position for
> > draft-ietf-avtcore-rtp-circuit-breakers-14: Discuss
> >
> > When responding, please keep the subject line intact and reply to all
> > email addresses included in the To and CC lines. (Feel free to cut this
> > introductory paragraph, however.)
> >
> >
> > Please refer to
> https://www.ietf.org/iesg/statement/discuss-criteria.html
> > for more information about IESG DISCUSS and COMMENT positions.
> >
> >
> > The document, along with other ballot positions, can be found here:
> >
> https://datatracker.ietf.org/doc/draft-ietf-avtcore-rtp-circuit-breakers/
> >
> >
> >
> > ----------------------------------------------------------------------
> > DISCUSS:
> > ----------------------------------------------------------------------
> >
> > I really like this specification, and have two questions I'd like to
> > understand before balloting YES ...
> >
> > I'm looking at this text:
> >
> > 4.5.  Ceasing Transmission
> >
> >   What it means to cease transmission depends on the application.  The
> >   intention is that the application will stop sending RTP data packets
> >   to a particular destination 3-tuple (transport protocol, destination
> >   port, IP address), until the user makes an explicit attempt to
> >   restart the call.  It is important that a human user is involved in
> >   the decision to try to restart the call, since that user will
> >   eventually give up if the calls repeatedly trigger the circuit
> >   breaker.  This will help avoid problems with automatic redial systems
> >   from congesting the network.  Accordingly, RTP flows halted by the
> >   circuit breaker SHOULD NOT be restarted automatically unless the
> >                   ^^^^^^^^^^
> >   sender has received information that the congestion has dissipated,
> >   or can reasonably be expected to have dissipated.
> >
> > and trying to understand why this is not MUST NOT. I'm trying to
> > reconcile this with the RFC 2119 definition of SHOULD NOT, which is
> >
> > 4. SHOULD NOT   This phrase, or the phrase "NOT RECOMMENDED" mean that
> >   there may exist valid reasons in particular circumstances when the
> >   particular behavior is acceptable or even useful, but the full
> >   implications should be understood and the case carefully weighed
> >   before implementing any behavior described with this label.
> >
> > Could you help me understand when automatic restarts might be
> “acceptable or even useful”?
>
> As stated, automatic restarts might be acceptable or even useful if the
> sender receives information that the congestion has dissipated, or can
> reasonably be expected to have dissipated. This is why it says “SHOULD NOT
> … unless <reasons>” rather than “MUST NOT”.


Hi, Colin,

On this one, I was getting tripped up on "or can be reasonably expected to
have dissipated" for an automatic restart, but if this means that an
implementer has thought about what has to happen before an implementation
does a restart with no user involvement, I can see that.

I was getting tied up with the subsequent text, but perhaps I shouldn't
have been.

Thanks for the help on that one.


> > Reading on, I'm wondering if this text is anticipating
> >
> >   It is recognised that the RTP implementation in some systems might
> >   not be able to determine if a call set-up request was initiated by a
> >   human user, or automatically by some scripted higher-level component
> >   of the system.
> >
> > but definitely want to understand what you're thinking here.
> >
> > I have a similar question about this text
> >
> >   ECN-CE marked packets SHOULD be treated as if it were lost for the
> >   purposes of congestion control, when determining the optimal media
> >   sending rate for an RTP flow.  If an RTP sender has negotiated ECN
> >   support for an RTP session, and has successfully initiated ECN use on
> >   the path to the receiver [RFC6679], then ECN-CE marked packets SHOULD
> >                                                                  ^^^^^^
> >   be treated as if they were lost when calculating if the congestion-
> >   based RTP circuit breaker (Section 4.3) has been met.
> >
> > Could you help me understand why an implementation wouldn't do this?
>
> Because there are a small number of paths that misbehave when ECN marked
> packet are sent. I wanted to allow leeway for an end-point to not respond
> to ECN-CE marks on paths that spuriously mark packets, so falling back to
> the behaving as-if ECN was not used. I don’t believe this is harmful to the
> network, since the result will be the same as achieved by a non-ECN capable
> transport.


So, the scenario you're thinking of, is that you and I negotiated ECN
support for a session, I started marking packets, and a path started
marking packets with ECN-CE whether congestion was being encountered or
not, and I realize that's happening (by some logic not specified), so I
can't trust ECN-CE.

Do I have that right? Assuming I'm close ...

So, since I can't trust ECN-CE, I keep sending as if I wasn't getting
ECN-CE, and either that works (so, I did the right thing), or I manage to
cause enough congestion that it shows up in RTCP reports (so, I did the
wrong thing before, but I'll react to the less problematic RTCP reports and
do the right thing now, and that won't be any worse than what would have
happened if we hadn't been able to negotiate ECN support in the first
place).

Do I have that right? Again, assuming that I'm at least close ...

I think what you're doing is "safe enough" that this isn't Discuss-worthy,
so I'll clear, but I wish the explanation of what's going on here was more
explicit. If you want to try to make that happen, I'm happy to chat further.

Thanks,

Spencer


> --
> Colin Perkins
> https://csperkins.org/
>
>
>
>
>