[lkml]   [2016]   [Jan]   [18]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRE: gianfar: Less function calls in gfar_ethflow_to_filer_table() after error detection
    >-----Original Message-----
    >From: SF Markus Elfring []
    >Sent: Friday, January 15, 2016 7:33 PM
    >To: Joe Perches <>;
    >Cc: Claudiu Manoil <>; LKML <linux-
    >>;; Julia Lawall
    >Subject: Re: gianfar: Less function calls in gfar_ethflow_to_filer_table() after
    >error detection
    >>>> local_rqfpr = kmalloc_array(2 * (MAX_FILER_IDX + 1),
    >>>> sizeof(unsigned int), GFP_KERNEL);
    >>>> if (!local_rqfpr)
    >>>> goto err;
    >>>> local_rqfcr = &local_rqfpr[MAX_FILER_IDX + 1];
    >>> Do you suggest to use only one array (instead of two as before) here?
    >> That's a possibility.
    >Thanks for your clarification.
    >> If, as your title suggests, you really want fewer function calls,
    >I am unsure at the moment if more changes will make sense in
    >this function implementation.
    >> (which as far as I saw, you didn't do)
    >Is my wording "after error detection" insufficient eventually?
    >> that could be a mechanism to remove both an allocation and a free.
    >Would any more software developers or source code reviewers like
    >to share their opinions in such a direction?

    This kind of fixes are net-next stuff at best, no need to push them into
    the net tree right now.
    So please wait with these submissions until net-next re-opens at least.

     \ /
      Last update: 2016-01-18 16:21    [W:4.028 / U:29.696 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site