]> git.ipfire.org Git - thirdparty/public-inbox.git/commitdiff
xap_helper.h: remove _SC_NPROCESSORS_ONLN default
authorEric Wong <e@80x24.org>
Wed, 24 Apr 2024 06:44:44 +0000 (06:44 +0000)
committerEric Wong <e@80x24.org>
Wed, 24 Apr 2024 21:34:45 +0000 (21:34 +0000)
It's never straightforward to pick an ideal number of processes
for anything and Xapian helper processes are no exception since
there may be a massive disparities in CPU count and I/O
performance.  So default to a single worker for now in the C++
version since that's the default is for the Perl/(XS|SWIG)
version, and also the same as for our normal public-facing
daemons.

This keeps the behavior between the Perl+(XS|SWIG) and C++
version as similar as possible.

lib/PublicInbox/xap_helper.h

index 3456910bae99412b32e357f9a079962a4635f8c2..0e6805b31ad5570abdea88ed927b9b96fea6ea22 100644 (file)
@@ -945,12 +945,6 @@ int main(int argc, char *argv[])
        }
 
        nworker = 1;
-#ifdef _SC_NPROCESSORS_ONLN
-       long j = sysconf(_SC_NPROCESSORS_ONLN);
-       if (j > 0)
-               nworker = j > WORKER_MAX ? WORKER_MAX : j;
-#endif // _SC_NPROCESSORS_ONLN
-
        // make warn/warnx/err multi-process friendly:
        if (my_setlinebuf(stderr))
                err(EXIT_FAILURE, "setlinebuf(stderr)");