Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S965031AbbD0S5m (ORCPT ); Mon, 27 Apr 2015 14:57:42 -0400 Received: from BERING.ISR.CS.CMU.EDU ([128.2.209.223]:58217 "EHLO bering.isr.cs.cmu.edu" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S964923AbbD0S5h (ORCPT ); Mon, 27 Apr 2015 14:57:37 -0400 X-Greylist: delayed 1186 seconds by postgrey-1.27 at vger.kernel.org; Mon, 27 Apr 2015 14:57:37 EDT Date: Mon, 27 Apr 2015 14:37:44 -0400 From: Jan Harkes To: Fabian Frederick Cc: linux-kernel@vger.kernel.org, Andrew Morton , coda@cs.cmu.edu Subject: Re: [PATCH 1/1 linux-next] fs/coda/upcall.c: remove UPARG flow control macro Message-ID: <20150427183744.GA5147@cs.cmu.edu> Mail-Followup-To: Fabian Frederick , linux-kernel@vger.kernel.org, Andrew Morton , coda@cs.cmu.edu References: <1430155386-6678-1-git-send-email-fabf@skynet.be> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1430155386-6678-1-git-send-email-fabf@skynet.be> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1803 Lines: 49 On Mon, Apr 27, 2015 at 07:23:06PM +0200, Fabian Frederick wrote: > Move UPARG code to alloc_upcall() and test errors/return in callsites. > This patch removes flow control in macros which must be avoided. > (See Documentation/CodingStyle) At first glance this is not a correct patch. UPARG allocates a buffer that is large enough to hold both the upcall message going to userspace as well as the incoming reply message. At first glance this patch only seems to allocate insize. It may still be doing the right thing because insize is defined at the call level as MAX(sizeof(inarg), sizeof(outarg)). This it looks like quite a few changes for an untested patch and not enough for an actual cleanup of the related code. Jan > --- > This is untested. > > fs/coda/upcall.c | 107 +++++++++++++++++++++++++++++++++++-------------------- > 1 file changed, 69 insertions(+), 38 deletions(-) > > diff --git a/fs/coda/upcall.c b/fs/coda/upcall.c > index 9b1ffaa..7524f630 100644 > --- a/fs/coda/upcall.c > +++ b/fs/coda/upcall.c > @@ -41,30 +41,24 @@ > static int coda_upcall(struct venus_comm *vc, int inSize, int *outSize, > union inputArgs *buffer); > > -static void *alloc_upcall(int opcode, int size) > +static int alloc_upcall(int opcode, int insize, int *outsize, > + union inputArgs **inp, union outputArgs **outp) > { > - union inputArgs *inp; > + CODA_ALLOC(*inp, union inputArgs *, insize); > + if (!*inp) > + return -ENOMEM; > > - CODA_ALLOC(inp, union inputArgs *, size); > - if (!inp) > - return ERR_PTR(-ENOMEM); ... -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/