net: Convert SEQ_START_TOKEN/seq_printf to seq_puts
authorJoe Perches <joe@perches.com>
Tue, 4 Nov 2014 23:37:03 +0000 (15:37 -0800)
committerDavid S. Miller <davem@davemloft.net>
Thu, 6 Nov 2014 03:04:55 +0000 (22:04 -0500)
Using a single fixed string is smaller code size than using
a format and many string arguments.

Reduces overall code size a little.

$ size net/ipv4/igmp.o* net/ipv6/mcast.o* net/ipv6/ip6_flowlabel.o*
   text    data     bss     dec     hex filename
  34269    7012   14824   56105    db29 net/ipv4/igmp.o.new
  34315    7012   14824   56151    db57 net/ipv4/igmp.o.old
  30078    7869   13200   51147    c7cb net/ipv6/mcast.o.new
  30105    7869   13200   51174    c7e6 net/ipv6/mcast.o.old
  11434    3748    8580   23762    5cd2 net/ipv6/ip6_flowlabel.o.new
  11491    3748    8580   23819    5d0b net/ipv6/ip6_flowlabel.o.old

Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/ipv4/igmp.c
net/ipv6/ip6_flowlabel.c
net/ipv6/mcast.c

index 3f80513..1e4adae 100644 (file)
@@ -2687,11 +2687,7 @@ static int igmp_mcf_seq_show(struct seq_file *seq, void *v)
        struct igmp_mcf_iter_state *state = igmp_mcf_seq_private(seq);
 
        if (v == SEQ_START_TOKEN) {
-               seq_printf(seq,
-                          "%3s %6s "
-                          "%10s %10s %6s %6s\n", "Idx",
-                          "Device", "MCA",
-                          "SRC", "INC", "EXC");
+               seq_puts(seq, "Idx Device        MCA        SRC    INC    EXC\n");
        } else {
                seq_printf(seq,
                           "%3d %6.6s 0x%08x "
index c143437..7221021 100644 (file)
@@ -770,8 +770,7 @@ static int ip6fl_seq_show(struct seq_file *seq, void *v)
 {
        struct ip6fl_iter_state *state = ip6fl_seq_private(seq);
        if (v == SEQ_START_TOKEN) {
-               seq_printf(seq, "%-5s %-1s %-6s %-6s %-6s %-8s %-32s %s\n",
-                          "Label", "S", "Owner", "Users", "Linger", "Expires", "Dst", "Opt");
+               seq_puts(seq, "Label S Owner  Users  Linger Expires  Dst                              Opt\n");
        } else {
                struct ip6_flowlabel *fl = v;
                seq_printf(seq,
index 9648de2..e04f184 100644 (file)
@@ -2823,11 +2823,7 @@ static int igmp6_mcf_seq_show(struct seq_file *seq, void *v)
        struct igmp6_mcf_iter_state *state = igmp6_mcf_seq_private(seq);
 
        if (v == SEQ_START_TOKEN) {
-               seq_printf(seq,
-                          "%3s %6s "
-                          "%32s %32s %6s %6s\n", "Idx",
-                          "Device", "Multicast Address",
-                          "Source Address", "INC", "EXC");
+               seq_puts(seq, "Idx Device                Multicast Address                   Source Address    INC    EXC\n");
        } else {
                seq_printf(seq,
                           "%3d %6.6s %pi6 %pi6 %6lu %6lu\n",