Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754220AbaDRT4H (ORCPT ); Fri, 18 Apr 2014 15:56:07 -0400 Received: from mail.tpi.com ([74.45.170.26]:46444 "EHLO mail.tpi.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751992AbaDRT4D (ORCPT ); Fri, 18 Apr 2014 15:56:03 -0400 Message-ID: <53518350.8010605@canonical.com> Date: Fri, 18 Apr 2014 13:56:00 -0600 From: Tim Gardner User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.4.0 MIME-Version: 1.0 To: Trond Myklebust CC: Linux NFS Mailing List , "netdev@vger.kernel.org" , Linux Kernel mailing list , "J. Bruce Fields" , "David S. Miller" Subject: Re: [PATCH net-next] xprtrdma: rpcrdma_register_default_external: Silence frame size warning References: <1397850625-44074-1-git-send-email-tim.gardner@canonical.com> In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 04/18/2014 01:52 PM, Trond Myklebust wrote: > > On Fri, Apr 18, 2014 at 3:50 PM, Tim Gardner > wrote: > > net/sunrpc/xprtrdma/verbs.c: In function > 'rpcrdma_register_default_external': > net/sunrpc/xprtrdma/verbs.c:1774:1: warning: the frame size of 1048 > bytes is larger than 1024 bytes [-Wframe-larger-than=] > > gcc version 4.8.2, x86_64-linux-gnu > > Cc: Trond Myklebust > > Cc: "J. Bruce Fields" > > Cc: "David S. Miller" > > Signed-off-by: Tim Gardner > > --- > net/sunrpc/xprtrdma/verbs.c | 8 +++++++- > 1 file changed, 7 insertions(+), 1 deletion(-) > > diff --git a/net/sunrpc/xprtrdma/verbs.c b/net/sunrpc/xprtrdma/verbs.c > index 93726560..8130349 100644 > --- a/net/sunrpc/xprtrdma/verbs.c > +++ b/net/sunrpc/xprtrdma/verbs.c > @@ -1736,9 +1736,13 @@ rpcrdma_register_default_external(struct > rpcrdma_mr_seg *seg, > int mem_priv = (writing ? IB_ACCESS_REMOTE_WRITE : > IB_ACCESS_REMOTE_READ); > struct rpcrdma_mr_seg *seg1 = seg; > - struct ib_phys_buf ipb[RPCRDMA_MAX_DATA_SEGS]; > + struct ib_phys_buf *ipb; > int len, i, rc = 0; > > + ipb = kmalloc(sizeof(*ipb) * RPCRDMA_MAX_DATA_SEGS, GFP_ATOMIC); > + if (!ipb) > + return -ENOMEM; > + > if (*nsegs > RPCRDMA_MAX_DATA_SEGS) > *nsegs = RPCRDMA_MAX_DATA_SEGS; > for (len = 0, i = 0; i < *nsegs;) { > @@ -1770,6 +1774,8 @@ rpcrdma_register_default_external(struct > rpcrdma_mr_seg *seg, > seg1->mr_len = len; > } > *nsegs = i; > + > + kfree(ipb); > return rc; > } > > -- > 1.7.9.5 > > What has this got to do with net-next? It is RPC related... > > -- > > Trond Myklebust > > Linux NFS client maintainer, PrimaryData > > trond.myklebust@primarydata.com > I guess I naively thought that anything under the net directory went through Dave's tree. rtg -- Tim Gardner tim.gardner@canonical.com -- 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/