Skip to content
  • Ilpo Järvinen's avatar
    [TCP]: Uninline tcp_set_state · 490d5046
    Ilpo Järvinen authored
    
    
    net/ipv4/tcp.c:
      tcp_close_state | -226
      tcp_done        | -145
      tcp_close       | -564
      tcp_disconnect  | -141
     4 functions changed, 1076 bytes removed, diff: -1076
    
    net/ipv4/tcp_input.c:
      tcp_fin               |  -86
      tcp_rcv_state_process | -164
     2 functions changed, 250 bytes removed, diff: -250
    
    net/ipv4/tcp_ipv4.c:
      tcp_v4_connect | -209
     1 function changed, 209 bytes removed, diff: -209
    
    net/ipv4/arp.c:
      arp_ignore |   +5
     1 function changed, 5 bytes added, diff: +5
    
    net/ipv6/tcp_ipv6.c:
      tcp_v6_connect | -158
     1 function changed, 158 bytes removed, diff: -158
    
    net/sunrpc/xprtsock.c:
      xs_sendpages |   -2
     1 function changed, 2 bytes removed, diff: -2
    
    net/dccp/ccids/ccid3.c:
      ccid3_update_send_interval |   +7
     1 function changed, 7 bytes added, diff: +7
    
    net/ipv4/tcp.c:
      tcp_set_state | +238
     1 function changed, 238 bytes added, diff: +238
    
    built-in.o:
     12 functions changed, 250 bytes added, 1695 bytes removed, diff: -1445
    
    I've no explanation why some unrelated changes seem to occur
    consistently as well (arp_ignore, ccid3_update_send_interval;
    I checked the arp_ignore asm and it seems to be due to some
    reordered of operation order causing some extra opcodes to be
    generated). Still, the benefits are pretty obvious from the
    codiff's results.
    
    Signed-off-by: default avatarIlpo Järvinen <ilpo.jarvinen@helsinki.fi>
    Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
    490d5046