Re: [PATCH v12 10/11] sparc64: Add support for ADI (Application Data Integrity)

From: Khalid Aziz
Date: Fri Feb 23 2018 - 14:19:28 EST


On 02/22/2018 07:50 PM, kbuild test robot wrote:
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 >>):

Hi Dave,

Including linux/sched.h in arch/sparc/include/asm/mmu_context.h should eliminate these build warnings. My gcc version 6.2.1 does not report these errors. Build bot is using 7.2.0.

I can add a patch 12 to add the include, revise patch 10 or you can add the include in your tree. Let me know how you would prefer to resolve this.

Thanks,
Khalid


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 <linux/spinlock.h>
10 #include <linux/mm_types.h>
11 #include <linux/smp.h>
12
13 #include <asm/spitfire.h>
14 #include <asm/adi_64.h>
15 #include <asm-generic/mm_hooks.h>
16 #include <asm/percpu.h>
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