[PATCH] libbb: reduce the overhead of single parameter bb_error_msg() calls

Denys Vlasenko vda.linux at googlemail.com
Tue Jul 2 09:36:27 UTC 2019


Applied, thanks!

On Thu, Jun 27, 2019 at 11:52 AM James Byrne
<james.byrne at origamienergy.com> wrote:
>
> Hi,
>
> I sent a first version of this patch in May 2018, but the comment from
> Denys in his reply on May 11, 2018 was that he didn't like the magic
> macros I had used to automatically substitute different versions of
> single parameter bb_xx_msg() calls, and would prefer explicit use of the
> simple functions.
>
> I didn't have time to take another look at this until fairly recently,
> but I've now done a version that explicitly replaces all single
> parameter versions of bb_xx_msg() calls. As a result this is a very
> large patch, so I am sending it as an attachment, and I've had to gzip
> it since my first attempt to send this resulted in a message from the
> list system saying it was being held for moderation because the message
> body was too big!
>
> Since this is a large change, and the benefit is fairly small, I'm not
> entirely sure whether this change is worthwhile, but since I've done
> then work I thought it was worth submitting it. The size saving is very
> architecture dependent, and only gets you a significant saving on PPC
> and x86_64. I found it to be as follows (for 'defconfig' using GCC 7.4):
>
>       Arm:     -92 bytes
>       MIPS:    -52 bytes
>       PPC:   -1836 bytes
>       x86_64: -938 bytes
>
> Regards,
>
> James Byrne
>
> _______________________________________________
> busybox mailing list
> busybox at busybox.net
> http://lists.busybox.net/mailman/listinfo/busybox


More information about the busybox mailing list