On 11/27/18 11:32 PM, Jakub Kicinski wrote: > Standard kernel compilation produces the following warning: > > net/core/rtnetlink.c: In function ‘rtnl_newlink’: > net/core/rtnetlink.c:3232:1: warning: the frame size of 1288 bytes is larger > than 1024 bytes [-Wframe-larger-than=] > } > ^ > > This should not really be an issue, as rtnl_newlink() stack is > generally quite shallow. > > Fix the warning by allocating attributes with kmalloc() in a wrapper > and passing it down to rtnl_newlink(), avoiding complexities on error > paths. > > Alternatively we could kmalloc() some structure within rtnl_newlink(), > slave attributes look like a good candidate. In practice it adds to > already rather high complexity and length of the function. > > Signed-off-by: Jakub Kicinski <jakub.kicin...@netronome.com> > --- > net/core/rtnetlink.c | 20 +++++++++++++++++--- > 1 file changed, 17 insertions(+), 3 deletions(-) >
Reviewed-by: David Ahern <dsah...@gmail.com>