Re: [Ice] I-D Action: draft-ietf-ice-trickle-20.txt

Christer Holmberg <christer.holmberg@ericsson.com> Sat, 14 April 2018 07:54 UTC

Return-Path: <christer.holmberg@ericsson.com>
X-Original-To: ice@ietfa.amsl.com
Delivered-To: ice@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id B0B74126DFF for <ice@ietfa.amsl.com>; Sat, 14 Apr 2018 00:54:34 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -4.311
X-Spam-Level:
X-Spam-Status: No, score=-4.311 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, RCVD_IN_DNSWL_MED=-2.3, SPF_PASS=-0.001, T_DKIMWL_WL_HIGH=-0.01] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=ericsson.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 8ZR-S-mHG3tm for <ice@ietfa.amsl.com>; Sat, 14 Apr 2018 00:54:32 -0700 (PDT)
Received: from sesbmg22.ericsson.net (sesbmg22.ericsson.net [193.180.251.48]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 97EF2124D68 for <ice@ietf.org>; Sat, 14 Apr 2018 00:54:32 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; d=ericsson.com; s=mailgw201801; c=relaxed/simple; q=dns/txt; i=@ericsson.com; t=1523692470; h=From:Sender:Reply-To:Subject:Date:Message-ID:To:Cc:MIME-Version:Content-Type: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:In-Reply-To:References:List-Id: List-Help:List-Unsubscribe:List-Subscribe:List-Post:List-Owner:List-Archive; bh=oE/3n4TuHWWc9p4T7KzLrk78EKsyUOllZbjD07yFSPQ=; b=aZX5M64KC+9rC57WifxrhS+6UgglVXKweLZd8qTsanKpGfyEqCzG59yYEkfJLepW 3LCIkW8QRvKOkG9yQODKa1rAILbSWLrGsd0M5TugA79NNAMwPw5+v/BLGJr8BfS8 rWgk5xQSOb2CpO0o8v+nYrYqbx7YBX53OuGA/ISQTao=;
X-AuditID: c1b4fb30-e6c349c000005e22-31-5ad1b3b6380c
Received: from ESESSHC018.ericsson.se (Unknown_Domain [153.88.183.72]) by sesbmg22.ericsson.net (Symantec Mail Security) with SMTP id 10.47.24098.6B3B1DA5; Sat, 14 Apr 2018 09:54:30 +0200 (CEST)
Received: from ESESSMB109.ericsson.se ([169.254.9.34]) by ESESSHC018.ericsson.se ([153.88.183.72]) with mapi id 14.03.0382.000; Sat, 14 Apr 2018 09:54:25 +0200
From: Christer Holmberg <christer.holmberg@ericsson.com>
To: Peter Saint-Andre <stpeter@mozilla.com>, "ice@ietf.org" <ice@ietf.org>
Thread-Topic: [Ice] I-D Action: draft-ietf-ice-trickle-20.txt
Thread-Index: AQHT0FxykGJyvcuBZU2kFPjVPCB3lKP49tsAgAL+wgCAARF2gIACKEcAgAC43rA=
Date: Sat, 14 Apr 2018 07:54:25 +0000
Message-ID: <7594FB04B1934943A5C02806D1A2204B72E6BF40@ESESSMB109.ericsson.se>
References: <152331734608.13396.14003423726113423470@ietfa.amsl.com> <14262ab0-2247-4938-7d66-4dc28ad37474@mozilla.com> <b011715c-08f8-0394-238e-cd7922def27e@mozilla.com> <D6F50EE2.2DD77%christer.holmberg@ericsson.com> <9f4b0044-a872-1604-b82d-a65b5166c3b1@mozilla.com>
In-Reply-To: <9f4b0044-a872-1604-b82d-a65b5166c3b1@mozilla.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-originating-ip: [153.88.183.170]
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: base64
MIME-Version: 1.0
X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFnrNLMWRmVeSWpSXmKPExsUyM2K7h+62zRejDObe07L4dqHW4tnKU4wO TB5Llvxk8ug70MUawBTFZZOSmpNZllqkb5fAlTH9/ArWggf8FXM2f2FpYFzD38XIySEhYCIx 6fs99i5GLg4hgSOMEhuen2cCSQgJLGaUmDvFtIuRg4NNwEKi+582SFhEwEti5p77YCXCAjYS 7y5vYoGI20p8bVrMCmH7SSy9vZMRxGYRUJWYdOAiWA2vgK/E6+97oMZPZZL4dYMJZDyngL3E 69VqIGFGATGJ76fWgJUwC4hL3HoynwniTAGJJXvOM0PYohIvH/9jhbCVJM5ses4CMoZZQFNi /S59iFZFiSndD9khtgpKnJz5hGUCo8gsJFNnIXTMQtIxC0nHAkaWVYyixanFSbnpRkZ6qUWZ ycXF+Xl6eaklmxiBUXBwy2+DHYwvnzseYhTgYFTi4XWaczFKiDWxrLgy9xCjBAezkgjvqlSg EG9KYmVValF+fFFpTmrxIUZpDhYlcV4Lv81RQgLpiSWp2ampBalFMFkmDk6pBsZ8EedHD84o b1SZwlUetLbOrWlxbAXHi3PL2OfVxx43SUw2VnnD094cuU3peE9PTvtEZQVN45+Gl5PnPr/2 yHkG8w+Z27H7M7bbbTqypv1xdrXS9Yl7NplvXyrrJ7LileKCeXMVfA1XKZf+u3VNdNnE4jMC phE1RUGBs0prVc427XzwYau9/EclluKMREMt5qLiRAC2MB/TfgIAAA==
Archived-At: <https://mailarchive.ietf.org/arch/msg/ice/MSlx5aOuh4Xh9P13-qxWH5-Fy1A>
Subject: Re: [Ice] I-D Action: draft-ietf-ice-trickle-20.txt
X-BeenThere: ice@ietf.org
X-Mailman-Version: 2.1.22
Precedence: list
List-Id: "Interactive Connectivity Establishment \(ICE\)" <ice.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/ice>, <mailto:ice-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/ice/>
List-Post: <mailto:ice@ietf.org>
List-Help: <mailto:ice-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/ice>, <mailto:ice-request@ietf.org?subject=subscribe>
X-List-Received-Date: Sat, 14 Apr 2018 07:54:35 -0000

Hi,

...

>> Also, I don¹t think "in flight checks" is terminology defined in ICE, 
>> or STUN in general.
>
> Does that matter?

Why not use existing terminology? I assume you mean pairs in "In-Progress" state?

Something like:

       "(thus avoiding removal of pairs that have a state of In-Progress, and pairs for which
         connectivity checks have already yielded a definitive Sucessful or Failed result)."

...

>>> * Also in §11 (and §10 too), point 6 says:
>>>
>>>   6.  If after the relevant redundancy tests the check list where the
>>>       pair is to be added already contains the maximum number of
>>>       candidate pairs (100 by default as per [rfc5245bis]), the agent
>>>       SHOULD discard any pairs in the Failed state to make room for the
>>>       new pair.  If there are no such pairs, the agent SHOULD discard
>>>       the new pair.
>>>
>>> Discarding the new pair is not consistent with §6.1.2.5 of 5245bis; 
>>> instead, we should discard lower-priority pairs to prune things down 
>>> to the maximum number of candidate pairs.
>> 
>> Do we want to discard lower-priority pairs even if they are in 
>> "Succeeded" or "In-Progress" state?
>
> No, those pairs are already carved out by the rule that the agent "only checks existing 
> pairs if they have a state of Waiting or Frozen".

True. I got confused due to the reference issue in bullet 5. I withdraw the question.

Regards,

Christer