]> git.ipfire.org Git - thirdparty/squid.git/commit
Bug 4942: --with-filedescriptors does not do anything (#395)
authorAmos Jeffries <yadij@users.noreply.github.com>
Sat, 4 May 2019 06:53:45 +0000 (06:53 +0000)
committerSquid Anubis <squid-anubis@squid-cache.org>
Sat, 4 May 2019 11:39:32 +0000 (11:39 +0000)
commit6afc7f3d5d579eca3676bb1b46e9c3a987fa3bda
treecba76ceff8f7a5f5915cfeeaddc0fdc34b3e8f84
parent609d5e0667b014bfec3ec1e34fd512974d2b6edb
Bug 4942: --with-filedescriptors does not do anything (#395)

SQUID_CHECK_MAXFD has been unconditionally overwriting any
user-defined limit with an auto-detected limit from the build
machine. The change causing this was an incomplete fix for
bug 3970 added to v3.3 and later releases.

Fixing that problem has two notable side effects:

* the user-defined value now has the FD property checks applied
  to it (multiple of 64, too-few, etc). This means warnings will
  start to appear in build logs for a number of custom
  configurations. We should expect an increase in questions
  about that.

* builds which have previously been passing in outrageous values
  will actually start to use those values as the SQUID_MAXFD
  limit. This may result in surprising memory consumption or
  performance issues. Hopefully the warnings and new messages
  displaying auto-detected limit separate from the value used
  will reduce the admin surprise, but may not.

This PR also includes cleanup of the autoconf syntax within the
SQUID_CHECK_MAXFD macro and moves the ./configure warnings about
possible issues into that check macro.
acinclude/os-deps.m4
configure.ac