Received: by 2002:a25:868d:0:0:0:0:0 with SMTP id z13csp1243347ybk; Thu, 21 May 2020 02:06:59 -0700 (PDT) X-Google-Smtp-Source: ABdhPJxvdjBYP90r+TD13T+41pJapf8hN8unyGbNoQQ4HsBa/zLiAaWRWPakS15ySTMe76717WUE X-Received: by 2002:a17:906:1dce:: with SMTP id v14mr2700391ejh.244.1590052019080; Thu, 21 May 2020 02:06:59 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1590052019; cv=none; d=google.com; s=arc-20160816; b=VHT0hgSYdglv8zO161nPBEE0DL+2XGx4iSJlzNg6NQNQzo6KzXbBD05PKgtHRhDoUq rCqAo9IXUA2dfowKdLJcIURRbrsNWpHAiOqkw9hMYyKL5gUYlRyYqYudGvceV4483puV amBQonY0EeJjg1b0m08So8FYfeMGHF0NhGhd6RrC4OeI5fvm6sUkuWrBnjSe6n4124rc EGbiLikvvmjXgMraLCKF4SKEkL+cV4/yuvf9rC002JTEUW0rYnPqsGRYeDvPQ2MSjLtS eSpekH8JtyEQUQpXBo+oTsgIZ8hVYCA60H3gfy+5tuTwBCUgBs/k+X4rSMZDoO5aTdlR GGaQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding:mime-version :content-language:accept-language:in-reply-to:references:message-id :date:thread-index:thread-topic:subject:cc:to:from; bh=Gu2jf6p6HCaNm6ReDDdV0DXo3lFknkLGU8pcbeUfF6o=; b=x8HObXZvfeR2CdUlr6uH/TwLr82FfBKI7lUbKZMl5THMLNdtf3ukuKkdOluseBv/Q5 Ib3kqBgEiuqev2LBRYdjVOvPbYAkN3/ZcP/qwAR84EN3BquDIBwunyRMdIiA+xqqf1JV jb6wlpRmBrelxaHugzoN3vLnsRIfcpcZ3Mj5R/Sq0S2lhj3hF7jtcTVIojGyI3EKj8Kq rp2tZu879hb6jrvb3pp1z0H5TYTGrJircaZKkiXGrDL2n/pL+HjdKNlmK3b7Z3yP/CcN llsNhPuGu6yDs0OURe22vn8uJeOnVgaVBICbRZJNCKipBSTDmClQKMguUHlco0yt2DpV dC8w== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-nfs-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-nfs-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=aculab.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id r1si3085476ejs.406.2020.05.21.02.06.25; Thu, 21 May 2020 02:06:59 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-nfs-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-nfs-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-nfs-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=aculab.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728668AbgEUJGY convert rfc822-to-8bit (ORCPT + 99 others); Thu, 21 May 2020 05:06:24 -0400 Received: from eu-smtp-delivery-151.mimecast.com ([207.82.80.151]:48704 "EHLO eu-smtp-delivery-151.mimecast.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728643AbgEUJGY (ORCPT ); Thu, 21 May 2020 05:06:24 -0400 Received: from AcuMS.aculab.com (156.67.243.126 [156.67.243.126]) (Using TLS) by relay.mimecast.com with ESMTP id uk-mtapsc-2-5zY-0iRdNzGaTxwUZDYv5A-1; Thu, 21 May 2020 10:06:20 +0100 X-MC-Unique: 5zY-0iRdNzGaTxwUZDYv5A-1 Received: from AcuMS.Aculab.com (fd9f:af1c:a25b:0:43c:695e:880f:8750) by AcuMS.aculab.com (fd9f:af1c:a25b:0:43c:695e:880f:8750) with Microsoft SMTP Server (TLS) id 15.0.1347.2; Thu, 21 May 2020 10:06:19 +0100 Received: from AcuMS.Aculab.com ([fe80::43c:695e:880f:8750]) by AcuMS.aculab.com ([fe80::43c:695e:880f:8750%12]) with mapi id 15.00.1347.000; Thu, 21 May 2020 10:06:19 +0100 From: David Laight To: 'Christoph Hellwig' , Marcelo Ricardo Leitner CC: David Miller , "kuba@kernel.org" , "edumazet@google.com" , "kuznet@ms2.inr.ac.ru" , "yoshfuji@linux-ipv6.org" , "vyasevich@gmail.com" , "nhorman@tuxdriver.com" , "jmaloy@redhat.com" , "ying.xue@windriver.com" , "drbd-dev@lists.linbit.com" , "linux-kernel@vger.kernel.org" , "linux-rdma@vger.kernel.org" , "linux-nvme@lists.infradead.org" , "target-devel@vger.kernel.org" , "linux-afs@lists.infradead.org" , "linux-cifs@vger.kernel.org" , "cluster-devel@redhat.com" , "ocfs2-devel@oss.oracle.com" , "netdev@vger.kernel.org" , "linux-sctp@vger.kernel.org" , "ceph-devel@vger.kernel.org" , "rds-devel@oss.oracle.com" , "linux-nfs@vger.kernel.org" Subject: RE: [PATCH 31/33] sctp: add sctp_sock_set_nodelay Thread-Topic: [PATCH 31/33] sctp: add sctp_sock_set_nodelay Thread-Index: AQHWL0qynFLZF0CI80WcvwAESh6D26iyPrKg Date: Thu, 21 May 2020 09:06:19 +0000 Message-ID: <0a6839ab0ba04fcf9b9c92784c9564aa@AcuMS.aculab.com> References: <20200520195509.2215098-1-hch@lst.de> <20200520195509.2215098-32-hch@lst.de> <20200520231001.GU2491@localhost.localdomain> <20200520.162355.2212209708127373208.davem@davemloft.net> <20200520233913.GV2491@localhost.localdomain> <20200521083442.GA7771@lst.de> In-Reply-To: <20200521083442.GA7771@lst.de> Accept-Language: en-GB, en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-ms-exchange-transport-fromentityheader: Hosted x-originating-ip: [10.202.205.107] MIME-Version: 1.0 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: aculab.com Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8BIT Sender: linux-nfs-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-nfs@vger.kernel.org From: Christoph Hellwig > Sent: 21 May 2020 09:35 > On Wed, May 20, 2020 at 08:39:13PM -0300, Marcelo Ricardo Leitner wrote: > > On Wed, May 20, 2020 at 04:23:55PM -0700, David Miller wrote: > > > From: Marcelo Ricardo Leitner > > > Date: Wed, 20 May 2020 20:10:01 -0300 > > > > > > > The duplication with sctp_setsockopt_nodelay() is quite silly/bad. > > > > Also, why have the 'true' hardcoded? It's what dlm uses, yes, but the > > > > API could be a bit more complete than that. > > > > > > The APIs are being designed based upon what in-tree users actually > > > make use of. We can expand things later if necessary. > > > > Sometimes expanding things later can be though, thus why the worry. > > But ok, I get it. Thanks. > > > > The comment still applies, though. (re the duplication) > > Where do you see duplication? The whole thing just doesn't scale. As soon as you get to the slightly more complex requests like SCTP_INITMSG (which should probably be called to set the required number of data streams) you've either got replicated code or nested wrappers. David - Registered Address Lakeside, Bramley Road, Mount Farm, Milton Keynes, MK1 1PT, UK Registration No: 1397386 (Wales)