[Buildroot] [PATCH v1 1/2] package/proftpd: add tunable buffer size
jared.bents at rockwellcollins.com
Mon Nov 12 14:30:17 UTC 2018
On Fri, Nov 9, 2018 at 2:40 PM Arnout Vandecappelle <arnout at mind.be> wrote:
> Hi Jared,
> On 09/11/2018 20:35, Jared Bents wrote:
> > Enables tunable buffer size as a compile time option
> > Signed-off-by: Jared Bents <jared.bents at rockwellcollins.com>
> > ---
> > package/proftpd/Config.in | 8 ++++++++
> > package/proftpd/proftpd.mk | 4 ++++
> > 2 files changed, 12 insertions(+)
> > diff --git a/package/proftpd/Config.in b/package/proftpd/Config.in
> > index b615a5ff4c..51c795501e 100644
> > --- a/package/proftpd/Config.in
> > +++ b/package/proftpd/Config.in
> > @@ -65,4 +65,12 @@ config BR2_PACKAGE_PROFTPD_MOD_QUOTATAB_SQL
> > Compile mod_quotatab with mod_quotatab_sql table.
> > endif
> > +
> > +config BR2_PACKAGE_PROFTPD_BUFFER_SIZE
> > + int "buffer size (0 for default)"
> You should specify the unit here, so "buffer size in bytes".
> > + default "0"
> > + help
> > + Compile ProFTPD with tunable buffer size. Size in
> > + bytes. 0 uses default size.
> It's not very clear what this means. I like the text from , so the help text
> could be something like:
> By increasing the buffer size above the default of 1K,
> proftpd reads and writes data in larger chunks, and makes
> fewer expensive system calls. Use of this option to set buffer
> sizes of 8K or more has been reported to drastically increase
> transfer speeds (depending on network configurations).
> 0 uses the default size of 1024.
I'll update to include the units and the more informative help.
> > +
> > endif
> > diff --git a/package/proftpd/proftpd.mk b/package/proftpd/proftpd.mk
> > index 8f3ff5088e..5ff0bc17e5 100644
> > --- a/package/proftpd/proftpd.mk
> > +++ b/package/proftpd/proftpd.mk
> > @@ -121,4 +121,8 @@ define PROFTPD_INSTALL_INIT_SYSTEMD
> > $(TARGET_DIR)/etc/systemd/system/multi-user.target.wants/proftpd.service
> > endef
> > +ifneq ($(BR2_PACKAGE_PROFTPD_BUFFER_SIZE),0)
> > +PROFTPD_CONF_OPTS += --enable-tunable-buffer-size=$(BR2_PACKAGE_PROFTPD_BUFFER_SIZE)
> Have you tested this? As far as I can see, the option is actually called
> --enable-buffer-size.  indeed mentions --enable-tunable-buffer-size; I guess
> that's either a mistake or from an earlier version, though.
Yes, it looks like --enable-tunable-buffer-size is the older version.
was looking at  when looking into adjusting the buffer size but it
looks like the package actually use --enable-buffer-size, I will
update to use --enable-buffer-size.
> I've marked this patch as Changes Requested in patchwork. Could you send an
> updated version?
> > +endif
> > +
> > $(eval $(autotools-package))
>  http://www.proftpd.org/docs/howto/BCP.html
More information about the buildroot