On 2015-11-14 00:45, Theo de Raadt wrote:
>> This patch changes the default setting to 1.5 *
>> (number_of_cpus_in_system) instead, which I find better matches modern
>> behaviour.
> 
> A larger number is sensible in this position.
> 
> I would propose 8.  I don't agree with a calculation like that; the
> amount of work a system can do should not be calculated like that.


Fair enough! I agree that 8 will probably fit most cases. It makes for a
simpler patch, too. :-)

(I retained the decimal point in 8.0 in the man page, as an indicator
that it is not an integer value.)


Regards,

/Benny





Index: config.h
===================================================================
RCS file: /cvs/src/usr.sbin/cron/config.h,v
retrieving revision 1.23
diff -u -p -u -r1.23 config.h
--- config.h    23 Oct 2015 18:42:55 -0000      1.23
+++ config.h    14 Nov 2015 00:32:21 -0000
@@ -40,7 +40,7 @@
 #define MAILARG _PATH_SENDMAIL                         /*-*/

                        /* maximum load at which batch jobs will still run */
-#define BATCH_MAXLOAD  1.5             /*-*/
+#define BATCH_MAXLOAD  8.0             /*-*/

                        /* Define this to run crontab setgid instead of
                         * setuid root.  Group access will be used to read
Index: cron.8
===================================================================
RCS file: /cvs/src/usr.sbin/cron/cron.8,v
retrieving revision 1.34
diff -u -p -u -r1.34 cron.8
--- cron.8      12 Nov 2015 21:14:01 -0000      1.34
+++ cron.8      14 Nov 2015 00:32:21 -0000
@@ -116,7 +116,7 @@ If the current load average is greater t
 .Ar load_avg ,
 .Xr batch 1
 jobs will not be run.
-The default value is 1.5.
+The default value is 8.0.
 To allow
 .Xr batch 1
 jobs to run regardless of the load, a value of 0.0 may be used.



Reply via email to