• Arnd Bergmann's avatar
    net/sched: cls_flower: reduce fl_change stack size · 39b7b6a6
    Arnd Bergmann authored
    The new ARP support has pushed the stack size over the edge on ARM,
    as there are two large objects on the stack in this function (mask
    and tb) and both have now grown a bit more:
    
    net/sched/cls_flower.c: In function 'fl_change':
    net/sched/cls_flower.c:928:1: error: the frame size of 1072 bytes is larger than 1024 bytes [-Werror=frame-larger-than=]
    
    We can solve this by dynamically allocating one or both of them.
    I first tried to do it just for the mask, but that only saved
    152 bytes on ARM, while this version just does it for the 'tb'
    array, bringing the stack size back down to 664 bytes.
    
    Fixes: 99d31326 ("net/sched: cls_flower: Support matching on ARP")
    Signed-off-by: default avatarArnd Bergmann <arnd@arndb.de>
    Acked-by: default avatarJiri Pirko <jiri@mellanox.com>
    Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
    39b7b6a6
cls_flower.c 37.1 KB