Hi Khalid, I love your patch! Yet something to improve: [auto build test ERROR on sparc-next/master] [also build test ERROR on v4.16-rc2] [cannot apply to next-20180222] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/Khalid-Aziz/Application-Data-Integrity-feature-introduced-by-SPARC-M7/20180223-071725 base: https://git.kernel.org/pub/scm/linux/kernel/git/davem/sparc-next.git master config: sparc64-allyesconfig (attached as .config) compiler: sparc64-linux-gnu-gcc (Debian 7.2.0-11) 7.2.0 reproduce: wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross # save the attached .config to linux build tree make.cross ARCH=sparc64 All error/warnings (new ones prefixed by >>): In file included from arch/sparc/include/asm/mmu_context.h:5:0, from include/linux/mmu_context.h:5, from drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd.h:29, from drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd.c:23: arch/sparc/include/asm/mmu_context_64.h: In function 'arch_start_context_switch': >> arch/sparc/include/asm/mmu_context_64.h:157:4: error: implicit declaration of function 'set_tsk_thread_flag'; did you mean 'set_ti_thread_flag'? [-Werror=implicit-function-declaration] set_tsk_thread_flag(prev, TIF_MCDPER); ^~~~~~~~~~~~~~~~~~~ set_ti_thread_flag >> arch/sparc/include/asm/mmu_context_64.h:159:4: error: implicit declaration of function 'clear_tsk_thread_flag'; did you mean 'clear_ti_thread_flag'? [-Werror=implicit-function-declaration] clear_tsk_thread_flag(prev, TIF_MCDPER); ^~~~~~~~~~~~~~~~~~~~~ clear_ti_thread_flag arch/sparc/include/asm/mmu_context_64.h: In function 'finish_arch_post_lock_switch': >> arch/sparc/include/asm/mmu_context_64.h:180:25: error: dereferencing pointer to incomplete type 'struct task_struct' if (current && current->mm && current->mm->context.adi) { ^~ In file included from arch/sparc/include/asm/processor.h:5:0, from arch/sparc/include/asm/spinlock_64.h:12, from arch/sparc/include/asm/spinlock.h:5, from include/linux/spinlock.h:88, from arch/sparc/include/asm/mmu_context_64.h:9, from arch/sparc/include/asm/mmu_context.h:5, from include/linux/mmu_context.h:5, from drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd.h:29, from drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd.c:23: >> arch/sparc/include/asm/processor_64.h:194:28: error: implicit declaration of function 'task_thread_info'; did you mean 'test_thread_flag'? [-Werror=implicit-function-declaration] #define task_pt_regs(tsk) (task_thread_info(tsk)->kregs) ^ >> arch/sparc/include/asm/mmu_context_64.h:183:11: note: in expansion of macro 'task_pt_regs' regs = task_pt_regs(current); ^~~~~~~~~~~~ >> arch/sparc/include/asm/processor_64.h:194:49: error: invalid type argument of '->' (have 'int') #define task_pt_regs(tsk) (task_thread_info(tsk)->kregs) ^ >> arch/sparc/include/asm/mmu_context_64.h:183:11: note: in expansion of macro 'task_pt_regs' regs = task_pt_regs(current); ^~~~~~~~~~~~ In file included from include/linux/cred.h:21:0, from include/linux/seq_file.h:12, from include/linux/pinctrl/consumer.h:17, from include/linux/pinctrl/devinfo.h:21, from include/linux/device.h:23, from include/linux/cdev.h:8, from include/drm/drmP.h:36, from drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd.c:25: include/linux/sched.h: At top level: >> include/linux/sched.h:1530:20: warning: conflicting types for 'set_tsk_thread_flag' static inline void set_tsk_thread_flag(struct task_struct *tsk, int flag) ^~~~~~~~~~~~~~~~~~~ >> include/linux/sched.h:1530:20: error: static declaration of 'set_tsk_thread_flag' follows non-static declaration In file included from arch/sparc/include/asm/mmu_context.h:5:0, from include/linux/mmu_context.h:5, from drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd.h:29, from drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd.c:23: arch/sparc/include/asm/mmu_context_64.h:157:4: note: previous implicit declaration of 'set_tsk_thread_flag' was here set_tsk_thread_flag(prev, TIF_MCDPER); ^~~~~~~~~~~~~~~~~~~ In file included from include/linux/cred.h:21:0, from include/linux/seq_file.h:12, from include/linux/pinctrl/consumer.h:17, from include/linux/pinctrl/devinfo.h:21, from include/linux/device.h:23, from include/linux/cdev.h:8, from include/drm/drmP.h:36, from drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd.c:25: >> include/linux/sched.h:1535:20: warning: conflicting types for 'clear_tsk_thread_flag' static inline void clear_tsk_thread_flag(struct task_struct *tsk, int flag) ^~~~~~~~~~~~~~~~~~~~~ >> include/linux/sched.h:1535:20: error: static declaration of 'clear_tsk_thread_flag' follows non-static declaration In file included from arch/sparc/include/asm/mmu_context.h:5:0, from include/linux/mmu_context.h:5, from drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd.h:29, from drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd.c:23: arch/sparc/include/asm/mmu_context_64.h:159:4: note: previous implicit declaration of 'clear_tsk_thread_flag' was here clear_tsk_thread_flag(prev, TIF_MCDPER); ^~~~~~~~~~~~~~~~~~~~~ cc1: some warnings being treated as errors vim +157 arch/sparc/include/asm/mmu_context_64.h 8 > 9 #include 10 #include 11 #include 12 13 #include 14 #include 15 #include 16 #include 17 18 static inline void enter_lazy_tlb(struct mm_struct *mm, struct task_struct *tsk) 19 { 20 } 21 22 extern spinlock_t ctx_alloc_lock; 23 extern unsigned long tlb_context_cache; 24 extern unsigned long mmu_context_bmap[]; 25 26 DECLARE_PER_CPU(struct mm_struct *, per_cpu_secondary_mm); 27 void get_new_mmu_context(struct mm_struct *mm); 28 int init_new_context(struct task_struct *tsk, struct mm_struct *mm); 29 void destroy_context(struct mm_struct *mm); 30 31 void __tsb_context_switch(unsigned long pgd_pa, 32 struct tsb_config *tsb_base, 33 struct tsb_config *tsb_huge, 34 unsigned long tsb_descr_pa, 35 unsigned long secondary_ctx); 36 37 static inline void tsb_context_switch_ctx(struct mm_struct *mm, 38 unsigned long ctx) 39 { 40 __tsb_context_switch(__pa(mm->pgd), 41 &mm->context.tsb_block[MM_TSB_BASE], 42 #if defined(CONFIG_HUGETLB_PAGE) || defined(CONFIG_TRANSPARENT_HUGEPAGE) 43 (mm->context.tsb_block[MM_TSB_HUGE].tsb ? 44 &mm->context.tsb_block[MM_TSB_HUGE] : 45 NULL) 46 #else 47 NULL 48 #endif 49 , __pa(&mm->context.tsb_descr[MM_TSB_BASE]), 50 ctx); 51 } 52 53 #define tsb_context_switch(X) tsb_context_switch_ctx(X, 0) 54 55 void tsb_grow(struct mm_struct *mm, 56 unsigned long tsb_index, 57 unsigned long mm_rss); 58 #ifdef CONFIG_SMP 59 void smp_tsb_sync(struct mm_struct *mm); 60 #else 61 #define smp_tsb_sync(__mm) do { } while (0) 62 #endif 63 64 /* Set MMU context in the actual hardware. */ 65 #define load_secondary_context(__mm) \ 66 __asm__ __volatile__( \ 67 "\n661: stxa %0, [%1] %2\n" \ 68 " .section .sun4v_1insn_patch, \"ax\"\n" \ 69 " .word 661b\n" \ 70 " stxa %0, [%1] %3\n" \ 71 " .previous\n" \ 72 " flush %%g6\n" \ 73 : /* No outputs */ \ 74 : "r" (CTX_HWBITS((__mm)->context)), \ 75 "r" (SECONDARY_CONTEXT), "i" (ASI_DMMU), "i" (ASI_MMU)) 76 77 void __flush_tlb_mm(unsigned long, unsigned long); 78 79 /* Switch the current MM context. */ 80 static inline void switch_mm(struct mm_struct *old_mm, struct mm_struct *mm, struct task_struct *tsk) 81 { 82 unsigned long ctx_valid, flags; 83 int cpu = smp_processor_id(); 84 85 per_cpu(per_cpu_secondary_mm, cpu) = mm; 86 if (unlikely(mm == &init_mm)) 87 return; 88 89 spin_lock_irqsave(&mm->context.lock, flags); 90 ctx_valid = CTX_VALID(mm->context); 91 if (!ctx_valid) 92 get_new_mmu_context(mm); 93 94 /* We have to be extremely careful here or else we will miss 95 * a TSB grow if we switch back and forth between a kernel 96 * thread and an address space which has it's TSB size increased 97 * on another processor. 98 * 99 * It is possible to play some games in order to optimize the 100 * switch, but the safest thing to do is to unconditionally 101 * perform the secondary context load and the TSB context switch. 102 * 103 * For reference the bad case is, for address space "A": 104 * 105 * CPU 0 CPU 1 106 * run address space A 107 * set cpu0's bits in cpu_vm_mask 108 * switch to kernel thread, borrow 109 * address space A via entry_lazy_tlb 110 * run address space A 111 * set cpu1's bit in cpu_vm_mask 112 * flush_tlb_pending() 113 * reset cpu_vm_mask to just cpu1 114 * TSB grow 115 * run address space A 116 * context was valid, so skip 117 * TSB context switch 118 * 119 * At that point cpu0 continues to use a stale TSB, the one from 120 * before the TSB grow performed on cpu1. cpu1 did not cross-call 121 * cpu0 to update it's TSB because at that point the cpu_vm_mask 122 * only had cpu1 set in it. 123 */ 124 tsb_context_switch_ctx(mm, CTX_HWBITS(mm->context)); 125 126 /* Any time a processor runs a context on an address space 127 * for the first time, we must flush that context out of the 128 * local TLB. 129 */ 130 if (!ctx_valid || !cpumask_test_cpu(cpu, mm_cpumask(mm))) { 131 cpumask_set_cpu(cpu, mm_cpumask(mm)); 132 __flush_tlb_mm(CTX_HWBITS(mm->context), 133 SECONDARY_CONTEXT); 134 } 135 spin_unlock_irqrestore(&mm->context.lock, flags); 136 } 137 138 #define deactivate_mm(tsk,mm) do { } while (0) 139 #define activate_mm(active_mm, mm) switch_mm(active_mm, mm, NULL) 140 141 #define __HAVE_ARCH_START_CONTEXT_SWITCH 142 static inline void arch_start_context_switch(struct task_struct *prev) 143 { 144 /* Save the current state of MCDPER register for the process 145 * we are switching from 146 */ 147 if (adi_capable()) { 148 register unsigned long tmp_mcdper; 149 150 __asm__ __volatile__( 151 ".word 0x83438000\n\t" /* rd %mcdper, %g1 */ 152 "mov %%g1, %0\n\t" 153 : "=r" (tmp_mcdper) 154 : 155 : "g1"); 156 if (tmp_mcdper) > 157 set_tsk_thread_flag(prev, TIF_MCDPER); 158 else > 159 clear_tsk_thread_flag(prev, TIF_MCDPER); 160 } 161 } 162 163 #define finish_arch_post_lock_switch finish_arch_post_lock_switch 164 static inline void finish_arch_post_lock_switch(void) 165 { 166 /* Restore the state of MCDPER register for the new process 167 * just switched to. 168 */ 169 if (adi_capable()) { 170 register unsigned long tmp_mcdper; 171 172 tmp_mcdper = test_thread_flag(TIF_MCDPER); 173 __asm__ __volatile__( 174 "mov %0, %%g1\n\t" 175 ".word 0x9d800001\n\t" /* wr %g0, %g1, %mcdper" */ 176 ".word 0xaf902001\n\t" /* wrpr %g0, 1, %pmcdper */ 177 : 178 : "ir" (tmp_mcdper) 179 : "g1"); > 180 if (current && current->mm && current->mm->context.adi) { 181 struct pt_regs *regs; 182 > 183 regs = task_pt_regs(current); 184 regs->tstate |= TSTATE_MCDE; 185 } 186 } 187 } 188 --- 0-DAY kernel test infrastructure Open Source Technology Center https://lists.01.org/pipermail/kbuild-all Intel Corporation