Re: [quicwg/base-drafts] create codec streams only when necessary (#2090)
Martin Thomson <notifications@github.com> Wed, 12 December 2018 05:39 UTC
Return-Path: <noreply@github.com>
X-Original-To: quic-issues@ietfa.amsl.com
Delivered-To: quic-issues@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id CB20C1310EB for <quic-issues@ietfa.amsl.com>; Tue, 11 Dec 2018 21:39:44 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -9.46
X-Spam-Level:
X-Spam-Status: No, score=-9.46 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIMWL_WL_HIGH=-1.46, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, HTML_MESSAGE=0.001, MAILING_LIST_MULTI=-1, RCVD_IN_DNSWL_HI=-5, SPF_PASS=-0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=github.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 9yfjDNqyp5Jp for <quic-issues@ietfa.amsl.com>; Tue, 11 Dec 2018 21:39:43 -0800 (PST)
Received: from out-3.smtp.github.com (out-3.smtp.github.com [192.30.252.194]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id C731A1310D8 for <quic-issues@ietf.org>; Tue, 11 Dec 2018 21:39:42 -0800 (PST)
Date: Tue, 11 Dec 2018 21:39:40 -0800
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=github.com; s=pf2014; t=1544593180; bh=vLvX+/catyXu4LIsCkWWFGaajfCW9YNccNxZazQSlfw=; h=Date:From:Reply-To:To:Cc:In-Reply-To:References:Subject:List-ID: List-Archive:List-Post:List-Unsubscribe:From; b=Xx0TWHRzWZMOb76AjUvjkIN4TpmSyYtQm/qNtDoEIL70yRy51l+gavJjLU9pJk3uj ujDC42xZTm3nA5dm8YiNonDKHvPLZtMLogB3QQ/rTxIx2cSRbm72m88527GjIPe9O/ g8yHI4ttQbxhniL3nAPZggH5yPSeLJ2HPhfx+Lj0=
From: Martin Thomson <notifications@github.com>
Reply-To: quicwg/base-drafts <reply+0166e4abdde34929de69f7d7c5851abb87960635d4e4963f92cf000000011828611c92a169ce170ba08e@reply.github.com>
To: quicwg/base-drafts <base-drafts@noreply.github.com>
Cc: Subscribed <subscribed@noreply.github.com>
Message-ID: <quicwg/base-drafts/pull/2090/c446468738@github.com>
In-Reply-To: <quicwg/base-drafts/pull/2090@github.com>
References: <quicwg/base-drafts/pull/2090@github.com>
Subject: Re: [quicwg/base-drafts] create codec streams only when necessary (#2090)
Mime-Version: 1.0
Content-Type: multipart/alternative; boundary="--==_mimepart_5c109f1ce99a8_7a643f88806d45b4172388"; charset="UTF-8"
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: martinthomson
X-GitHub-Recipient: quic-issues
X-GitHub-Reason: subscribed
X-Auto-Response-Suppress: All
X-GitHub-Recipient-Address: quic-issues@ietf.org
Archived-At: <https://mailarchive.ietf.org/arch/msg/quic-issues/SLdz6TgLBAYHUHHl5Hu00jAKBXs>
X-BeenThere: quic-issues@ietf.org
X-Mailman-Version: 2.1.29
List-Id: Notification list for GitHub issues related to the QUIC WG <quic-issues.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/quic-issues>, <mailto:quic-issues-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/quic-issues/>
List-Post: <mailto:quic-issues@ietf.org>
List-Help: <mailto:quic-issues-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/quic-issues>, <mailto:quic-issues-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 12 Dec 2018 05:39:45 -0000
Let me try to expand on my earlier comment. I currently create these streams when building my connection object. This allows requests and responses to use those streams directly. If the streams might not exist, then I have to allow those values to be absent throughout, which is annoying but not a big deal. More of a concern is that this is concurrent code, so deferring creation would mean that I would have to guard the variables holding the streams with a mutex (or atomic, which always gives me the creeps). That creates the potential for contention point on every new message. Not saying impossible, but it is annoying. And all in service to implementations that are not interested in handling a stream that will only ever contain a single octet. Given the complexity of this protocol as a whole, it seems like a strange optimization to make. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/quicwg/base-drafts/pull/2090#issuecomment-446468738
- [quicwg/base-drafts] create codec streams only wh… Kazuho Oku
- Re: [quicwg/base-drafts] create codec streams onl… Martin Thomson
- Re: [quicwg/base-drafts] create codec streams onl… Mike Bishop
- Re: [quicwg/base-drafts] create codec streams onl… Mike Bishop
- Re: [quicwg/base-drafts] create codec streams onl… Kazuho Oku
- Re: [quicwg/base-drafts] create codec streams onl… Kazuho Oku
- Re: [quicwg/base-drafts] create codec streams onl… Kazuho Oku
- Re: [quicwg/base-drafts] create codec streams onl… Luca Niccolini
- Re: [quicwg/base-drafts] create codec streams onl… Dmitri Tikhonov
- Re: [quicwg/base-drafts] create codec streams onl… Subodh Iyengar
- Re: [quicwg/base-drafts] create codec streams onl… Mike Bishop
- Re: [quicwg/base-drafts] create codec streams onl… Martin Thomson
- Re: [quicwg/base-drafts] create codec streams onl… MikkelFJ
- Re: [quicwg/base-drafts] create codec streams onl… Kazuho Oku
- Re: [quicwg/base-drafts] create codec streams onl… Martin Thomson
- Re: [quicwg/base-drafts] create codec streams onl… afrind
- Re: [quicwg/base-drafts] create codec streams onl… Dmitri Tikhonov
- Re: [quicwg/base-drafts] create codec streams onl… afrind
- Re: [quicwg/base-drafts] create codec streams onl… Dmitri Tikhonov
- Re: [quicwg/base-drafts] create codec streams onl… Luca Niccolini
- Re: [quicwg/base-drafts] create codec streams onl… Kazuho Oku
- Re: [quicwg/base-drafts] create codec streams onl… Kazuho Oku
- Re: [quicwg/base-drafts] create codec streams onl… afrind
- Re: [quicwg/base-drafts] create codec streams onl… afrind