diff options
author | Joe Perches <joe@perches.com> | 2011-11-28 13:40:46 -0500 |
---|---|---|
committer | Eric Van Hensbergen <ericvh@gmail.com> | 2012-01-05 11:51:44 -0500 |
commit | 5d3851530d6d68564e4e0ce04d0547d4d106fc72 (patch) | |
tree | d47065fcc1f93f00070afe597a98ab185fd13859 /net/9p/protocol.c | |
parent | a0ea787b027b79cf2e01c6758e5246db06520158 (diff) |
9p: Reduce object size with CONFIG_NET_9P_DEBUG
Reduce object size by deduplicating formats.
Use vsprintf extension %pV.
Rename P9_DPRINTK uses to p9_debug, align arguments.
Add function for _p9_debug and macro to add __func__.
Add missing "\n"s to p9_debug uses.
Remove embedded function names as p9_debug adds it.
Remove P9_EPRINTK macro and convert use to pr_<level>.
Add and use pr_fmt and pr_<level>.
$ size fs/9p/built-in.o*
text data bss dec hex filename
62133 984 16000 79117 1350d fs/9p/built-in.o.new
67342 984 16928 85254 14d06 fs/9p/built-in.o.old
$ size net/9p/built-in.o*
text data bss dec hex filename
88792 4148 22024 114964 1c114 net/9p/built-in.o.new
94072 4148 23232 121452 1da6c net/9p/built-in.o.old
Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: Eric Van Hensbergen <ericvh@gmail.com>
Diffstat (limited to 'net/9p/protocol.c')
-rw-r--r-- | net/9p/protocol.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/net/9p/protocol.c b/net/9p/protocol.c index 55e10a96c902..9ee48cb30179 100644 --- a/net/9p/protocol.c +++ b/net/9p/protocol.c | |||
@@ -534,7 +534,7 @@ int p9stat_read(struct p9_client *clnt, char *buf, int len, struct p9_wstat *st) | |||
534 | 534 | ||
535 | ret = p9pdu_readf(&fake_pdu, clnt->proto_version, "S", st); | 535 | ret = p9pdu_readf(&fake_pdu, clnt->proto_version, "S", st); |
536 | if (ret) { | 536 | if (ret) { |
537 | P9_DPRINTK(P9_DEBUG_9P, "<<< p9stat_read failed: %d\n", ret); | 537 | p9_debug(P9_DEBUG_9P, "<<< p9stat_read failed: %d\n", ret); |
538 | trace_9p_protocol_dump(clnt, &fake_pdu); | 538 | trace_9p_protocol_dump(clnt, &fake_pdu); |
539 | } | 539 | } |
540 | 540 | ||
@@ -558,8 +558,8 @@ int p9pdu_finalize(struct p9_client *clnt, struct p9_fcall *pdu) | |||
558 | pdu->size = size; | 558 | pdu->size = size; |
559 | 559 | ||
560 | trace_9p_protocol_dump(clnt, pdu); | 560 | trace_9p_protocol_dump(clnt, pdu); |
561 | P9_DPRINTK(P9_DEBUG_9P, ">>> size=%d type: %d tag: %d\n", pdu->size, | 561 | p9_debug(P9_DEBUG_9P, ">>> size=%d type: %d tag: %d\n", |
562 | pdu->id, pdu->tag); | 562 | pdu->size, pdu->id, pdu->tag); |
563 | 563 | ||
564 | return err; | 564 | return err; |
565 | } | 565 | } |
@@ -585,7 +585,7 @@ int p9dirent_read(struct p9_client *clnt, char *buf, int len, | |||
585 | ret = p9pdu_readf(&fake_pdu, clnt->proto_version, "Qqbs", &dirent->qid, | 585 | ret = p9pdu_readf(&fake_pdu, clnt->proto_version, "Qqbs", &dirent->qid, |
586 | &dirent->d_off, &dirent->d_type, &nameptr); | 586 | &dirent->d_off, &dirent->d_type, &nameptr); |
587 | if (ret) { | 587 | if (ret) { |
588 | P9_DPRINTK(P9_DEBUG_9P, "<<< p9dirent_read failed: %d\n", ret); | 588 | p9_debug(P9_DEBUG_9P, "<<< p9dirent_read failed: %d\n", ret); |
589 | trace_9p_protocol_dump(clnt, &fake_pdu); | 589 | trace_9p_protocol_dump(clnt, &fake_pdu); |
590 | goto out; | 590 | goto out; |
591 | } | 591 | } |