Re: [PATCH 18/50] microblaze: Add loglvl to microblaze_unwind_inner()

From: kbuild test robot
Date: Fri Nov 08 2019 - 02:53:02 EST

Hi Dmitry,

Thank you for the patch! Yet something to improve:

[auto build test ERROR on linus/master]
[also build test ERROR on v5.4-rc6 next-20191107]
[if your patch is applied to the wrong git tree, please drop us a note to help
improve the system. BTW, we also suggest to use '--base' option to specify the
base tree in git format-patch, please see https://stackoverflow.com/a/37406982]

url: https://github.com/0day-ci/linux/commits/Dmitry-Safonov/kallsyms-printk-Add-loglvl-to-print_ip_sym/20191108-124037
base: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git 847120f859cc45e074204f4cf33c8df069306eb2
config: microblaze-nommu_defconfig (attached as .config)
compiler: microblaze-linux-gcc (GCC) 7.4.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
GCC_VERSION=7.4.0 make.cross ARCH=microblaze

If you fix the issue, kindly add following tag
Reported-by: kbuild test robot <lkp@xxxxxxxxx>

All error/warnings (new ones prefixed by >>):

arch/microblaze/kernel/unwind.c: In function 'unwind_trap':
>> arch/microblaze/kernel/unwind.c:176:2: error: too many arguments to function 'microblaze_unwind_inner'
microblaze_unwind_inner(task, regs->pc, regs->r1, regs->r15, trace, loglvl);
^~~~~~~~~~~~~~~~~~~~~~~
arch/microblaze/kernel/unwind.c:154:13: note: declared here
static void microblaze_unwind_inner(struct task_struct *task,
^~~~~~~~~~~~~~~~~~~~~~~
arch/microblaze/kernel/unwind.c: At top level:
>> arch/microblaze/kernel/unwind.c:191:13: error: conflicting types for 'microblaze_unwind_inner'
static void microblaze_unwind_inner(struct task_struct *task,
^~~~~~~~~~~~~~~~~~~~~~~
arch/microblaze/kernel/unwind.c:154:13: note: previous declaration of 'microblaze_unwind_inner' was here
static void microblaze_unwind_inner(struct task_struct *task,
^~~~~~~~~~~~~~~~~~~~~~~
>> arch/microblaze/kernel/unwind.c:154:13: warning: 'microblaze_unwind_inner' used but never defined

vim +/microblaze_unwind_inner +176 arch/microblaze/kernel/unwind.c

153
> 154 static void microblaze_unwind_inner(struct task_struct *task,
155 unsigned long pc, unsigned long fp,
156 unsigned long leaf_return,
157 struct stack_trace *trace);
158
159 /**
160 * unwind_trap - Unwind through a system trap, that stored previous state
161 * on the stack.
162 */
163 #ifdef CONFIG_MMU
164 static inline void unwind_trap(struct task_struct *task, unsigned long pc,
165 unsigned long fp, struct stack_trace *trace,
166 const char *loglvl)
167 {
168 /* To be implemented */
169 }
170 #else
171 static inline void unwind_trap(struct task_struct *task, unsigned long pc,
172 unsigned long fp, struct stack_trace *trace,
173 const char *loglvl)
174 {
175 const struct pt_regs *regs = (const struct pt_regs *) fp;
> 176 microblaze_unwind_inner(task, regs->pc, regs->r1, regs->r15, trace, loglvl);
177 }
178 #endif
179
180 /**
181 * microblaze_unwind_inner - Unwind the stack from the specified point
182 * @task : Task whose stack we are to unwind (may be NULL)
183 * @pc : Program counter from which we start unwinding
184 * @fp : Frame (stack) pointer from which we start unwinding
185 * @leaf_return : Value of r15 at pc. If the function is a leaf, this is
186 * the caller's return address.
187 * @trace : Where to store stack backtrace (PC values).
188 * NULL == print backtrace to kernel log
189 * @loglvl : Used for printk log level if (trace == NULL).
190 */
> 191 static void microblaze_unwind_inner(struct task_struct *task,
192 unsigned long pc, unsigned long fp,
193 unsigned long leaf_return,
194 struct stack_trace *trace,
195 const char *loglvl)
196 {
197 int ofs = 0;
198
199 pr_debug(" Unwinding with PC=%p, FP=%p\n", (void *)pc, (void *)fp);
200 if (!pc || !fp || (pc & 3) || (fp & 3)) {
201 pr_debug(" Invalid state for unwind, aborting\n");
202 return;
203 }
204 for (; pc != 0;) {
205 unsigned long next_fp, next_pc = 0;
206 unsigned long return_to = pc + 2 * sizeof(unsigned long);
207 const struct trap_handler_info *handler =
208 &microblaze_trap_handlers;
209
210 /* Is previous function the HW exception handler? */
211 if ((return_to >= (unsigned long)&_hw_exception_handler)
212 &&(return_to < (unsigned long)&ex_handler_unhandled)) {
213 /*
214 * HW exception handler doesn't save all registers,
215 * so we open-code a special case of unwind_trap()
216 */
217 #ifndef CONFIG_MMU
218 const struct pt_regs *regs =
219 (const struct pt_regs *) fp;
220 #endif
221 printk("%sHW EXCEPTION\n", loglvl);
222 #ifndef CONFIG_MMU
223 microblaze_unwind_inner(task, regs->r17 - 4,
224 fp + EX_HANDLER_STACK_SIZ,
225 regs->r15, trace, loglvl);
226 #endif
227 return;
228 }
229
230 /* Is previous function a trap handler? */
231 for (; handler->start_addr; ++handler) {
232 if ((return_to >= handler->start_addr)
233 && (return_to <= handler->end_addr)) {
234 if (!trace)
235 printk("%s%s\n", loglvl, handler->trap_name);
236 unwind_trap(task, pc, fp, trace, loglvl);
237 return;
238 }
239 }
240 pc -= ofs;
241
242 if (trace) {
243 #ifdef CONFIG_STACKTRACE
244 if (trace->skip > 0)
245 trace->skip--;
246 else
247 trace->entries[trace->nr_entries++] = pc;
248
249 if (trace->nr_entries >= trace->max_entries)
250 break;
251 #endif
252 } else {
253 /* Have we reached userland? */
254 if (unlikely(pc == task_pt_regs(task)->pc)) {
255 printk("%s[<%p>] PID %lu [%s]\n",
256 loglvl, (void *) pc,
257 (unsigned long) task->pid,
258 task->comm);
259 break;
260 } else
261 print_ip_sym(loglvl, pc);
262 }
263
264 /* Stop when we reach anything not part of the kernel */
265 if (!kernel_text_address(pc))
266 break;
267
268 if (lookup_prev_stack_frame(fp, pc, leaf_return, &next_fp,
269 &next_pc) == 0) {
270 ofs = sizeof(unsigned long);
271 pc = next_pc & ~3;
272 fp = next_fp;
273 leaf_return = 0;
274 } else {
275 pr_debug(" Failed to find previous stack frame\n");
276 break;
277 }
278
279 pr_debug(" Next PC=%p, next FP=%p\n",
280 (void *)next_pc, (void *)next_fp);
281 }
282 }
283

---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/hyperkitty/list/kbuild-all@xxxxxxxxxxxx Intel Corporation

Attachment: .config.gz
Description: application/gzip