From: Andrew Cooper <andrew.cooper3@citrix.com>
To: Xen-devel <xen-devel@lists.xen.org>
Cc: "Juergen Gross" <jgross@suse.com>,
"Stefano Stabellini" <sstabellini@kernel.org>,
"Wei Liu" <wei.liu2@citrix.com>,
"George Dunlap" <george.dunlap@eu.citrix.com>,
"Andrew Cooper" <andrew.cooper3@citrix.com>,
"Robert VanVossen" <robert.vanvossen@dornerworks.com>,
"Dario Faggioli" <dfaggioli@suse.com>,
"Julien Grall" <julien.grall@arm.com>,
"Josh Whitehead" <josh.whitehead@dornerworks.com>,
"Meng Xu" <mengxu@cis.upenn.edu>,
"Jan Beulich" <JBeulich@suse.com>,
"Roger Pau Monné" <roger.pau@citrix.com>
Subject: [PATCH 0/6] xen: Use %*pb[l] for printing bitmaps
Date: Thu, 6 Sep 2018 13:08:10 +0100 [thread overview]
Message-ID: <1536235696-31359-1-git-send-email-andrew.cooper3@citrix.com> (raw)
I was trying to debug a cpumask problem, and got irritated with how awkard it
was to render and print the masks. Luckily, the fix is quite simple and far
nicer to use.
The overall diffstat is to patch 5 is:
add/remove: 0/4 grow/shrink: 2/11 up/down: 603/-1191 (-588)
Function old new delta
vsnprintf 2775 3348 +573
dump_runq 279 309 +30
null_dump 675 659 -16
dump_irqs 718 702 -16
machine_crash_shutdown 594 570 -24
csched_dump 527 503 -24
rt_dump_vcpu.isra 254 222 -32
dump_evtchn_info 657 625 -32
print_cpumap 55 - -55
cpuset_print.constprop 61 - -61
csched_dump_pcpu 473 401 -72
mcheck_cmn_handler 1224 1146 -78
null_dump_pcpu 493 413 -80
dump_domains 1160 1064 -96
csched2_dump 1549 1389 -160
bitmap_scnprintf 193 - -193
bitmap_scnlistprintf 252 - -252
Total: Before=3295448, After=3294860, chg -0.02%
Andrew Cooper (6):
xen/vsprintf: Introduce %*pb[l] for printing bitmaps
xen/sched: Use %*pb[l] instead of cpumask_scn{,list}printf()
xen/common: Use %*pb[l] instead of {cpu,node}mask_scn{,list}printf()
xen/x86: Use %*pb[l] instead of cpumask_scn{,list}printf()
xen/bitmap: Drop all bitmap_scn{,list}printf() infrastructure
xen/keyhandler: Drop keyhandler_scratch
docs/misc/printk-formats.txt | 8 ++++
xen/arch/x86/cpu/mcheck/mce.c | 9 ++--
xen/arch/x86/crash.c | 7 +--
xen/arch/x86/irq.c | 7 +--
xen/arch/x86/numa.c | 11 ++---
xen/common/bitmap.c | 105 ------------------------------------------
xen/common/cpupool.c | 12 ++---
xen/common/efi/boot.c | 5 +-
xen/common/event_channel.c | 6 +--
xen/common/keyhandler.c | 61 +++++++-----------------
xen/common/sched_credit.c | 17 ++-----
xen/common/sched_credit2.c | 26 ++++-------
xen/common/sched_null.c | 15 ++----
xen/common/sched_rt.c | 5 +-
xen/common/vsprintf.c | 97 ++++++++++++++++++++++++++++++++++++++
xen/include/xen/bitmap.h | 6 ---
xen/include/xen/cpumask.h | 18 --------
xen/include/xen/keyhandler.h | 3 --
xen/include/xen/nodemask.h | 34 --------------
19 files changed, 163 insertions(+), 289 deletions(-)
--
2.1.4
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
next reply other threads:[~2018-09-06 12:08 UTC|newest]
Thread overview: 33+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-09-06 12:08 Andrew Cooper [this message]
2018-09-06 12:08 ` [PATCH 1/6] xen/vsprintf: Introduce %*pb[l] for printing bitmaps Andrew Cooper
2018-09-06 16:32 ` Wei Liu
2018-09-07 7:41 ` Jan Beulich
2018-09-07 13:01 ` Andrew Cooper
2018-09-07 15:14 ` Jan Beulich
2018-09-25 13:06 ` Andrew Cooper
2018-09-25 13:22 ` Jan Beulich
2018-09-06 12:08 ` [PATCH 2/6] xen/sched: Use %*pb[l] instead of cpumask_scn{, list}printf() Andrew Cooper
2018-09-07 8:03 ` Jan Beulich
2018-09-07 13:56 ` Andrew Cooper
2018-09-07 14:42 ` George Dunlap
2018-09-07 15:17 ` Jan Beulich
2018-09-07 15:35 ` George Dunlap
2018-09-07 15:53 ` Jan Beulich
2018-09-07 16:07 ` Andrew Cooper
2018-09-10 6:39 ` Jan Beulich
2018-09-07 14:42 ` George Dunlap
2018-09-12 8:05 ` Dario Faggioli
2018-09-06 12:08 ` [PATCH 3/6] xen/common: Use %*pb[l] instead of {cpu, node}mask_scn{, list}printf() Andrew Cooper
2018-09-06 16:32 ` Wei Liu
2018-09-07 8:06 ` Jan Beulich
2018-09-07 8:30 ` Juergen Gross
2018-09-06 12:08 ` [PATCH 4/6] xen/x86: Use %*pb[l] instead of cpumask_scn{, list}printf() Andrew Cooper
2018-09-06 16:33 ` Wei Liu
2018-09-07 8:12 ` Jan Beulich
2018-09-07 14:00 ` Andrew Cooper
2018-09-06 12:08 ` [PATCH 5/6] xen/bitmap: Drop all bitmap_scn{, list}printf() infrastructure Andrew Cooper
2018-09-06 16:34 ` Wei Liu
2018-09-12 8:09 ` Dario Faggioli
2018-09-06 12:08 ` [PATCH RFC 6/6] xen/keyhandler: Drop keyhandler_scratch Andrew Cooper
2018-09-06 16:31 ` Wei Liu
2018-09-07 8:24 ` Jan Beulich
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1536235696-31359-1-git-send-email-andrew.cooper3@citrix.com \
--to=andrew.cooper3@citrix.com \
--cc=JBeulich@suse.com \
--cc=dfaggioli@suse.com \
--cc=george.dunlap@eu.citrix.com \
--cc=jgross@suse.com \
--cc=josh.whitehead@dornerworks.com \
--cc=julien.grall@arm.com \
--cc=mengxu@cis.upenn.edu \
--cc=robert.vanvossen@dornerworks.com \
--cc=roger.pau@citrix.com \
--cc=sstabellini@kernel.org \
--cc=wei.liu2@citrix.com \
--cc=xen-devel@lists.xen.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).