Re: [PATCH v4 06/13] x86/platform/uv: Add and Decode Arch Type in UVsystab
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
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
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
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
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