From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sowmini Varadhan Subject: Re: [PATCH V2 net-next 4/7] rds: support for zcopy completion notification Date: Wed, 14 Feb 2018 14:01:10 -0500 Message-ID: <20180214190110.GM11528@oracle.com> References: <331448b2-65d1-0f62-033c-b97d87a57b87@oracle.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: netdev@vger.kernel.org, willemdebruijn.kernel@gmail.com, davem@davemloft.net, rds-devel@oss.oracle.com To: Santosh Shilimkar Return-path: Received: from aserp2130.oracle.com ([141.146.126.79]:38146 "EHLO aserp2130.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1162538AbeBNTBe (ORCPT ); Wed, 14 Feb 2018 14:01:34 -0500 Content-Disposition: inline In-Reply-To: <331448b2-65d1-0f62-033c-b97d87a57b87@oracle.com> Sender: netdev-owner@vger.kernel.org List-ID: On (02/14/18 10:50), Santosh Shilimkar wrote: > generic comment and please update it where it is applicable > in terms of variable names, notifiers etc. > > RDS support true zero copy already with RDMA transport so some of > this code can easily get confused. > > So I suggest something like below. > s/zerocopy/zeromsgcopy > s/zcopy/zmsgcopy > s/zcookie/zmsgcpycookie > s/znotifier/zmsgcpynotifier I'd like to hear some additional opinions from the list on this: the existing socket API for TCP etc. already uses ZEROCOPY, and other than extending variable names (and putting me at risk of violating the "fit within 80 chars per line" requirement, leading to not-so-pretty line wraps), I'm not seeing much value in this. > This error change might need to go though other subsystem tree. May > be you can seperate it and also copy "linux-api@vger.kernel.org" sure I can do that, either when I put out patch v3 or later today, after others have had a chance to review this. --Sowmini