Re: [PATCH 1/2] UV: NMI: insert per_cpu accessor function on uv_hub_nmi.

[Ingo Molnar]

* George Beshers <gbeshers@xxxxxxx> wrote:

> UV: NMI: insert this_cpu_read accessor function on uv_hub_nmi.
> 
> UV NMI was accidently broken by this patch.

Broken in what way?

>     commit e16321709c8270f9803bbfdb51e5e02235078c7f
>     Author: Christoph Lameter <cl@xxxxxxxxx>
>     Date:   Sun Aug 17 12:30:41 2014 -0500

That's a rather old patch. Was no upstream kernel tested since ~August 
last year on UV hardware, or is the bug sporadic? The changelog does 
not tell us.

> This patch insert this_cpu_read() on when accessing the PER_CPU 
> uv_cpu_nmi variable.

Why? What problem does it solve?

Thanks,

	Ingo
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/