Re: [PATCH net-next] xprtrdma: rpcrdma_register_default_external: Silence frame size warning

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 04/18/2014 01:52 PM, Trond Myklebust wrote:

On Fri, Apr 18, 2014 at 3:50 PM, Tim Gardner <tim.gardner@xxxxxxxxxxxxx
<mailto:tim.gardner@xxxxxxxxxxxxx>> 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 <trond.myklebust@xxxxxxxxxxxxxxx
    <mailto:trond.myklebust@xxxxxxxxxxxxxxx>>
    Cc: "J. Bruce Fields" <bfields@xxxxxxxxxxxx
    <mailto:bfields@xxxxxxxxxxxx>>
    Cc: "David S. Miller" <davem@xxxxxxxxxxxxx <mailto:davem@xxxxxxxxxxxxx>>
    Signed-off-by: Tim Gardner <tim.gardner@xxxxxxxxxxxxx
    <mailto:tim.gardner@xxxxxxxxxxxxx>>
    ---
      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@xxxxxxxxxxxxxxx <mailto:trond.myklebust@xxxxxxxxxxxxxxx>


I guess I naively thought that anything under the net directory went through Dave's tree.

rtg
--
Tim Gardner tim.gardner@xxxxxxxxxxxxx
--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Filesystem Development]     [Linux USB Development]     [Linux Media Development]     [Video for Linux]     [Linux NILFS]     [Linux Audio Users]     [Yosemite Info]     [Linux SCSI]

  Powered by Linux