[Skiboot] [PATCH 1/1] buddy: Fix warnings when undefining BUDDY_DEBUG
Oliver O'Halloran
oohall at gmail.com
Tue May 26 17:07:41 AEST 2020
On Mon, May 25, 2020 at 5:35 PM Cédric Le Goater <clg at kaod.org> wrote:
>
> On 5/21/20 9:22 PM, Ryan Grimm wrote:
> > In simulation, hundreds of millions of cycles are chewed up in this code
> > path:
> >
> > PC: 0x0000000030033450 -> <.bitmap_tst_bit>+0x18
> > LR: 0x000000003003347c -> <.buddy_check_alloc>+0x14
> > 0x0000000031c13b30 -> <_ebss>+0x1803b30
> > 0x000000003003351c -> <.buddy_check_alloc_down>+0x4c
> > 0x00000000300339c4 -> <.buddy_free>+0x7c
> > 0x0000000030033be8 -> <.buddy_create>+0xcc
> > 0x0000000030089bbc -> <.xive_init>+0xf0
> > 0x00000000300157cc -> <.main_cpu_entry>+0x8a0
> > 0x000000003000275c -> <boot_entry>+0x1bc
> >
> > Undefining BUDDY_DEBUG saves 30+ minutes of wall clock time, so fix
> > the "warning: unused parameter" messages when compiling.
>
> Sure. The buddy allocator is only used by XIVE, may be we could simply
> use DEBUG.
>
> > Signed-off-by: Ryan Grimm <grimm at linux.ibm.com>
>
> Acked-by: Cédric Le Goater <clg at kaod.org>
Looks like I picked up the patch from patchwork before you Acked it so
there's no ack in the merged patch. Sorry.
Merged as fbc6c882727a5e16716f169b186d5e4d1da0b8b6
More information about the Skiboot
mailing list