Thread (14 messages) 14 messages, 3 authors, 2017-01-09

Re: [PATCH] libata-eh: Use switch() instead of sparse array for protocol strings

From: Geert Uytterhoeven <geert@linux-m68k.org>
Date: 2017-01-09 15:40:34
Also in: lkml

Hi Tejun,

On Mon, Jan 9, 2017 at 4:23 PM, Tejun Heo [off-list ref] wrote:
On Mon, Jan 09, 2017 at 03:49:28PM +0100, Geert Uytterhoeven wrote:
quoted
Replace the sparse 256-pointer array for looking up protocol strings by
a switch() statement to reduce kernel size.

According to bloat-o-meter, this saves 910 bytes on m68k (32-bit), and
1892 bytes on arm64 (64-bit).

Signed-off-by: Geert Uytterhoeven <geert@linux-m68k.org>
Gees, thanks for catching that.

Applied to libata/for-4.11.
Thanks!

There are two more that annoy me, but I don't know how to fix them:

ata_scsi_rbuf                                  -    4096   +4096
ata_force_param_buf                            -    4096   +4096

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help