The patch titled
Subject: scripts/bloat-o-meter: fix percent on <1% changes
has been added to the -mm tree. Its filename is
scripts-bloat-o-meter-fix-percent-on-1%-changes.patch
This patch should soon appear at
http://ozlabs.org/~akpm/mmots/broken-out/scripts-bloat-o-meter-fix-percent-on-1%25-changes.patch
and later at
http://ozlabs.org/~akpm/mmotm/broken-out/scripts-bloat-o-meter-fix-percent-on-1%25-changes.patch
Before you just go and hit "reply", please:
a) Consider who else should be cc'ed
b) Prefer to cc a suitable mailing list as well
c) Ideally: find the original patch on the mailing list and do a
reply-to-all to that, adding suitable additional cc's
*** Remember to use Documentation/SubmitChecklist when testing your code ***
The -mm tree is included into linux-next and is updated
there every 3-4 working days
------------------------------------------------------
From: Riku Voipio <riku.voipio@xxxxxxxxxx>
Subject: scripts/bloat-o-meter: fix percent on <1% changes
Python divisions are integer divisions unless at least one parameter is a
float. The current bloat-o-meter fails to print sub-percentage changes:
Total: Before=10515408, After=10604060, chg 0.000000%
Force float division by using one float and pretty the print to
two significant decimals:
Total: Before=10515408, After=10604060, chg +0.84%
Link: http://lkml.kernel.org/r/1465980311-23814-1-git-send-email-riku.voipio@xxxxxxxxxx
Signed-off-by: Riku Voipio <riku.voipio@xxxxxxxxxx>
Reviewed-by: Josh Triplett <josh@xxxxxxxxxxxxxxxx>
Cc: Vineet Gupta <vgupta@xxxxxxxxxxxx>
Cc: Michal Marek <mmarek@xxxxxxx>
Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>
---
scripts/bloat-o-meter | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff -puN scripts/bloat-o-meter~scripts-bloat-o-meter-fix-percent-on-1%-changes scripts/bloat-o-meter
--- a/scripts/bloat-o-meter~scripts-bloat-o-meter-fix-percent-on-1%-changes
+++ a/scripts/bloat-o-meter
@@ -67,5 +67,5 @@ print("%-40s %7s %7s %+7s" % ("function"
for d, n in delta:
if d: print("%-40s %7s %7s %+7d" % (n, old.get(n,"-"), new.get(n,"-"), d))
-print("Total: Before=%d, After=%d, chg %f%%" % \
- (otot, ntot, (ntot - otot)*100/otot))
+print("Total: Before=%d, After=%d, chg %+.2f%%" % \
+ (otot, ntot, (ntot - otot)*100.0/otot))
_
Patches currently in -mm which might be from riku.voipio@xxxxxxxxxx are
scripts-bloat-o-meter-fix-percent-on-1%-changes.patch
--
To unsubscribe from this list: send the line "unsubscribe mm-commits" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
