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

Reply via email to