[prev in list] [next in list] [prev in thread] [next in thread] 

List:       busybox
Subject:    Re: syslogd: multiple remote hosts
From:       Denys Vlasenko <vda.linux () googlemail ! com>
Date:       2010-02-28 12:03:27
Message-ID: 201002281303.27246.vda.linux () googlemail ! com
[Download RAW message or body]

On Tuesday 23 February 2010 17:44, Thomas Geulig wrote:
> Last year Andreas Bergmann sent a patch to configure multiple remote
> logging hosts in the syslog daemon. Vladimir Dronnikow complained about
> the (artificial) limit of 8 remote hosts and proposed some changes.
> Andreas abandoned the patch, then.
> 
> I need this feature too, so I came back to this patch. The limit of 8
> hosts is now gone and I made some other optimizations:
> 
> function                                             old     new   delta
> syslogd_main                                        1113    1220    +107
> .rodata                                           126306  126304      -2
> init_data                                             72      64      -8
> ------------------------------------------------------------------------------
> (add/remove: 0/0 grow/shrink: 1/2 up/down: 107/-10)            Total: 97 bytes
>    text    data     bss     dec     hex filename
>  678068    2077    9040  689185   a8421 busybox_old
>  678173    2077    9040  689290   a848a busybox_unstripped

Applied, thanks! Please confirm that current git works for you.

> The second patch (on top of the first) makes this addition configurable
> through the FEATURE_MULTIPLE_REMOTE_HOSTS option.
> This adds only 5 bytes when the config option is off, but it makes the code
> quite ugly (IMHO).

I think it's not needed.

-- 
vda
_______________________________________________
busybox mailing list
busybox@busybox.net
http://lists.busybox.net/mailman/listinfo/busybox
[prev in list] [next in list] [prev in thread] [next in thread]