On Thu, Jun 09, 2016 at 11:52:49AM +0200, Quentin Monnet wrote:
> From: David Marchand <david.march...@6wind.com>
> 
> Relying on /proc/cpuinfo to count the number of available cores is not
> the best option:
> 
> - The code is x86-specific.
> - If the process is started with a different CPU affinity, then it will
>   wrongly try to start too many threads (for an example, imagine an OVS
>   daemon restricted to 4 CPU threads on a 128 threads system).
> 
> This commit removes /proc/cpuinfo parsing. For Linux systems, it
> introduces instead a call to sched_getaffinity(), which is
> architecture-independant, in order to retrieve the list of CPU threads
> available to the current process and to count them. Other UNIX-like
> systems only use _SC_NPROCESSORS_ONLN.
> 
> Signed-off-by: David Marchand <david.march...@6wind.com>
> Signed-off-by: Liu Xiaofeng <xiaofeng....@6wind.com>
> Signed-off-by: Quentin Monnet <quentin.mon...@6wind.com>

This seems reasonable, but the authorship is confusing.  Who is the
author, and what are the roles of the others in the signoff chain?

Thanks,

Ben.
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to