On 12/21/2017 3:49 AM, Ingo Molnar wrote:
* Mike Travis <tra...@sgi.com> wrote:
This patchset handles the fixes made to the UV4 HUB for upcoming Intel
processors as there are some interface changes.
* Update uv_mmrs.h to prep for fixed defines for UV4A.
* Updates to handle UV4 vs. UV4A (fixed) arches.
* Updates to handle UV4 GAM (global addressable memory) MMRs fixes.
* Updates to handle UV4 BAU (broadcast assist unit) MMRs fixes.
* Turn on UV4A_IS_SUPPORTED to enable code and header file fixes.
Sigh, has any of this been properly build tested? x86-64 allyesconfig produces a
bunch of ugly warnings:
Admittedly, we don't have a setup to test too many upstream kernel
builds with configs other than what our distros are shipping with. We
do test build specific earlier kernel releases that are current with the
distro to insure (or at least understand) back port issues.
I will try this "allyesconfig" though I believe it introduces CONFIG
items that cause problems where the resultant kernel doesn't boot on a
UV system. But if building is the only goal, then we can ignore them.
Are there any other standard configs that are required to build when
CONFIG_X86_UV is defined? (Since all of these changes are within the UV
specific code.)
Thanks,
Mike
In file included from ./include/linux/kernel.h:14:0,
arch/x86/kernel/apic/x2apic_uv_x.c: In function ‘map_mmioh_high_uv34’:
./include/linux/printk.h:308:2: warning: ‘id’ may be used uninitialized in this
function [-Wmaybe-uninitialized]
arch/x86/kernel/apic/x2apic_uv_x.c:806:8: note: ‘id’ was declared here
In file included from ./include/linux/kernel.h:14:0,
./include/linux/printk.h:308:2: warning: ‘m_io’ may be used uninitialized in
this function [-Wmaybe-uninitialized]
arch/x86/kernel/apic/x2apic_uv_x.c:804:19: note: ‘m_io’ was declared here
In file included from ./include/linux/kernel.h:14:0,
./include/linux/printk.h:308:2: warning: ‘shift’ may be used uninitialized in
this function [-Wmaybe-uninitialized]
arch/x86/kernel/apic/x2apic_uv_x.c:804:12: note: ‘shift’ was declared here
arch/x86/kernel/apic/x2apic_uv_x.c:856:14: warning: ‘n’ may be used
uninitialized in this function [-Wmaybe-uninitialized]
arch/x86/kernel/apic/x2apic_uv_x.c:846:9: warning: ‘nasid_mask’ may be used
uninitialized in this function [-Wmaybe-uninitialized]
In file included from ./include/linux/kernel.h:14:0,
./include/linux/printk.h:308:2: warning: ‘base’ may be used uninitialized in
this function [-Wmaybe-uninitialized]
arch/x86/kernel/apic/x2apic_uv_x.c:801:16: note: ‘base’ was declared here
arch/x86/kernel/apic/x2apic_uv_x.c:843:17: warning: ‘mmr’ may be used
uninitialized in this function [-Wmaybe-uninitialized]
arch/x86/kernel/apic/x2apic_uv_x.c:832:16: warning: ‘overlay’ may be used
uninitialized in this function [-Wmaybe-uninitialized]
Thanks,
Ingo