lkml.org 
[lkml]   [2014]   [Nov]   [1]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] x86: UV BAU Increase maximum CPUs per socket/hub
On Fri, 31 Oct 2014, James Custer wrote:

> We have encountered hardware with 18 cores/socket that gives 36 CPUs/socket
> with hyperthreading enabled. This exceeds the current MAX_CPUS_PER_SOCKET
> causing a failure in get_cpu_topology. Increase MAX_CPUS_PER_SOCKET to 64
> and MAX_CPUS_PER_UVHUB to 128.
>
> Additionally, this exposed another bug. When get_cpu_topology fails, we
> need to make sure that we do not use bau_control->statp (e.g. when a
> user reads /proc/sgi_uv/ptc_statistics causing ptc_seq_show to be
> invoked). In uv_bau_init we call set_bau_off() when
> init_per_cpu()->get_cpu_topology() fails -- therefore, to avoid
> referencing a NULL pointer we add code that detects when the bau is off
> and print out the bau is disabled.

Can you please split this in two patches. One fixing the ptc_seq_show
bug and the other increasing the constants.

If someone looks for a fix for the ptc_seq_show wreckage he definitly
is not going to look at a patch which increases the number of cpus per
socket.

Thanks,

tglx


\
 
 \ /
  Last update: 2014-11-01 21:41    [W:0.024 / U:0.140 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site