OSDN Git Service

sfc: ethtool: Refactor to remove fallthrough comments in case blocks
authorJoe Perches <joe@perches.com>
Wed, 11 Mar 2020 02:41:41 +0000 (19:41 -0700)
committerDavid S. Miller <davem@davemloft.net>
Thu, 12 Mar 2020 18:29:28 +0000 (11:29 -0700)
commitc7449b756943c6aae868b2f7c91c57bacbce61f6
tree2ae8d47710fa60ac77abd6a1e336f18bcaf12050
parent7c4046b1c53bba3a0315f04bb0bb5f36888a747b
sfc: ethtool: Refactor to remove fallthrough comments in case blocks

Converting fallthrough comments to fallthrough; creates warnings
in this code when compiled with gcc.

This code is overly complicated and reads rather better with a
little refactoring and no fallthrough uses at all.

Remove the fallthrough comments and simplify the written source
code while reducing the object code size.

Consolidate duplicated switch/case blocks for IPV4 and IPV6.

defconfig x86-64 with sfc:

$ size drivers/net/ethernet/sfc/ethtool.o*
   text    data     bss     dec     hex filename
  10055      12       0   10067    2753 drivers/net/ethernet/sfc/ethtool.o.new
  10135      12       0   10147    27a3 drivers/net/ethernet/sfc/ethtool.o.old

Signed-off-by: Joe Perches <joe@perches.com>
Acked-by: Martin Habets <mhabets@solarflare.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/sfc/ethtool.c