Re: [PATCH v4 06/13] x86/platform/uv: Add and Decode Arch Type in UVsystab

2020-10-07 Thread Borislav Petkov
On Tue, Oct 06, 2020 at 06:21:14PM -0700, Mike Travis wrote:
> When the UV BIOS starts the kernel it passes the UVsystab info struct to the
> kernel

Oh wow, *now* I get it. There is no "patch" being passed - your initial
commit message simply states that this is a patch to add and process...

Yeah, as already mentioned, saying that this is a patch in the same
patch's commit message is tautological and, as we just saw, really
confusing. So pls refrain from doing that. I'll use this explanation in
the commit message as it is more clear.

Thx.

-- 
Regards/Gruss,
Boris.

https://people.kernel.org/tglx/notes-about-netiquette


Re: [PATCH v4 06/13] x86/platform/uv: Add and Decode Arch Type in UVsystab

2020-10-06 Thread Mike Travis




On 10/6/2020 8:19 AM, Borislav Petkov wrote:

On Mon, Oct 05, 2020 at 02:35:58PM -0700, Mike Travis wrote:



On 10/5/2020 2:21 PM, Borislav Petkov wrote:

On Mon, Oct 05, 2020 at 03:39:22PM -0500, Mike Travis wrote:

A patch to add and process the UV Arch Type field in the UVsystab passed
from UV BIOS to the kernel.


What does that mean?



There have been recent cases where OEM's want to use the OEM_ID in the ACPI
tables to brand their own product.  The UV BIOS used that field to tell the
Linux kernel which UV arch the running system is.  The Arch Type in the
UVsystab (also created by UV BIOS) now carries that field in systems that
support it.


I am unclear on the whole patch passing from UV BIOS to the kernel. Are
you talking about a kernel patch here? If so, commit ID is? Or what kind
of patch are you talking about?



When the UV BIOS starts the kernel it passes the UVsystab info struct to 
the kernel which contains information elements more specific than ACPI, 
and generally pertinent only to the MMR's.  These are read only fields 
so information is passed one way only.  A new field starting with UV5 is 
the UV architecture type so the ACPI OEM_ID field can be used for other 
purposes going forward.  The UV Arch Type selects the entirety of the 
MMRs available, with their addresses and fields defined in uv_mmrs.h.


Thanks,
Mike


Re: [PATCH v4 06/13] x86/platform/uv: Add and Decode Arch Type in UVsystab

2020-10-06 Thread Borislav Petkov
On Mon, Oct 05, 2020 at 02:35:58PM -0700, Mike Travis wrote:
> 
> 
> On 10/5/2020 2:21 PM, Borislav Petkov wrote:
> > On Mon, Oct 05, 2020 at 03:39:22PM -0500, Mike Travis wrote:
> > > A patch to add and process the UV Arch Type field in the UVsystab passed
> > > from UV BIOS to the kernel.
> > 
> > What does that mean?
> > 
> 
> There have been recent cases where OEM's want to use the OEM_ID in the ACPI
> tables to brand their own product.  The UV BIOS used that field to tell the
> Linux kernel which UV arch the running system is.  The Arch Type in the
> UVsystab (also created by UV BIOS) now carries that field in systems that
> support it.

I am unclear on the whole patch passing from UV BIOS to the kernel. Are
you talking about a kernel patch here? If so, commit ID is? Or what kind
of patch are you talking about?

-- 
Regards/Gruss,
Boris.

https://people.kernel.org/tglx/notes-about-netiquette


Re: [PATCH v4 06/13] x86/platform/uv: Add and Decode Arch Type in UVsystab

2020-10-05 Thread Mike Travis




On 10/5/2020 2:21 PM, Borislav Petkov wrote:

On Mon, Oct 05, 2020 at 03:39:22PM -0500, Mike Travis wrote:

A patch to add and process the UV Arch Type field in the UVsystab passed
from UV BIOS to the kernel.


What does that mean?



There have been recent cases where OEM's want to use the OEM_ID in the 
ACPI tables to brand their own product.  The UV BIOS used that field to 
tell the Linux kernel which UV arch the running system is.  The Arch 
Type in the UVsystab (also created by UV BIOS) now carries that field in 
systems that support it.


Re: [PATCH v4 06/13] x86/platform/uv: Add and Decode Arch Type in UVsystab

2020-10-05 Thread Borislav Petkov
On Mon, Oct 05, 2020 at 03:39:22PM -0500, Mike Travis wrote:
> A patch to add and process the UV Arch Type field in the UVsystab passed
> from UV BIOS to the kernel. 

What does that mean?

-- 
Regards/Gruss,
Boris.

https://people.kernel.org/tglx/notes-about-netiquette