ruby-changes:36000
From: kosaki <ko1@a...>
Date: Wed, 22 Oct 2014 10:22:19 +0900 (JST)
Subject: [ruby-changes:36000] kosaki:r48081 (trunk): * ext/etc/etc.c (etc_nprocessors_affin): minor spell fix.
kosaki 2014-10-22 03:46:46 +0900 (Wed, 22 Oct 2014) New Revision: 48081 http://svn.ruby-lang.org/cgi-bin/viewvc.cgi?view=revision&revision=48081 Log: * ext/etc/etc.c (etc_nprocessors_affin): minor spell fix. Modified files: trunk/ChangeLog trunk/ext/etc/etc.c Index: ChangeLog =================================================================== --- ChangeLog (revision 48080) +++ ChangeLog (revision 48081) @@ -1,3 +1,7 @@ https://github.com/ruby/ruby/blob/trunk/ChangeLog#L1 +Wed Oct 22 03:37:00 2014 KOSAKI Motohiro <kosaki.motohiro@g...> + + * ext/etc/etc.c (etc_nprocessors_affin): minor spell fix. + Wed Oct 22 03:33:58 2014 KOSAKI Motohiro <kosaki.motohiro@g...> * ext/etc/etc.c (etc_nprocessors_affin): optimize memory usege a Index: ext/etc/etc.c =================================================================== --- ext/etc/etc.c (revision 48080) +++ ext/etc/etc.c (revision 48081) @@ -928,11 +928,11 @@ etc_nprocessors_affin(void) https://github.com/ruby/ruby/blob/trunk/ext/etc/etc.c#L928 * The precious way is to use /sys/devices/system/cpu/online. But there are * two problems, * - Costly calculation - * It is minor issue, but possibly kill the benefit of parallel processing. + * It is a minor issue, but possibly kill a benefit of a parallel processing. * - No guarantee to exist /sys/devices/system/cpu/online - * This is an issue especially when using containers. - * So, we use hardcode number for workaround. Current linux kernel - * (Linux 3.17) support 8192 cpus at maximum. Then 16384 is enough large. + * This is an issue especially when using Linux containers. + * So, we use hardcode number for a workaround. Current linux kernel + * (Linux 3.17) support 8192 cpus at maximum. Then 16384 must be enough. */ for (n=64; n < 16384; n *= 2) { size = CPU_ALLOC_SIZE(n); -- ML: ruby-changes@q... Info: http://www.atdot.net/~ko1/quickml/