All of lore.kernel.org
 help / color / mirror / Atom feed
From: kbuild test robot <lkp@intel.com>
To: Nicholas Piggin <npiggin@gmail.com>
Cc: linuxppc-dev@lists.ozlabs.org, kbuild-all@lists.01.org
Subject: Re: [PATCH v3 11/15] powerpc/64s: machine check interrupt update NMI accounting
Date: Sun, 12 Apr 2020 16:46:09 +0800	[thread overview]
Message-ID: <202004121649.GhAwOgAj%lkp@intel.com> (raw)
In-Reply-To: <20200407051636.648369-12-npiggin@gmail.com>

[-- Attachment #1: Type: text/plain, Size: 4835 bytes --]

Hi Nicholas,

I love your patch! Yet something to improve:

[auto build test ERROR on powerpc/next]
[also build test ERROR on next-20200412]
[cannot apply to tip/perf/core v5.6]
[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/Nicholas-Piggin/powerpc-64-machine-check-and-system-reset-fixes/20200407-134803
base:   https://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux.git next
config: powerpc-randconfig-a001-20200412 (attached as .config)
compiler: powerpc64-linux-gcc (GCC) 9.3.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=9.3.0 make.cross ARCH=powerpc 

If you fix the issue, kindly add following tag as appropriate
Reported-by: kbuild test robot <lkp@intel.com>

All errors (new ones prefixed by >>):

   In file included from include/linux/kernel.h:15,
                    from include/asm-generic/bug.h:19,
                    from arch/powerpc/include/asm/bug.h:109,
                    from include/linux/bug.h:5,
                    from arch/powerpc/include/asm/mmu.h:130,
                    from arch/powerpc/include/asm/paca.h:18,
                    from arch/powerpc/include/asm/current.h:13,
                    from include/linux/sched.h:12,
                    from arch/powerpc/kernel/process.c:14:
   arch/powerpc/kernel/process.c: In function 'show_regs':
>> arch/powerpc/kernel/process.c:1424:74: error: 'struct paca_struct' has no member named 'in_nmi'
    1424 |  pr_cont("IRQMASK: %lx IN_NMI:%d IN_MCE:%d", regs->softe, (int)get_paca()->in_nmi, (int)get_paca()->in_mce);
         |                                                                          ^~
   include/linux/printk.h:317:26: note: in definition of macro 'pr_cont'
     317 |  printk(KERN_CONT fmt, ##__VA_ARGS__)
         |                          ^~~~~~~~~~~
>> arch/powerpc/kernel/process.c:1424:99: error: 'struct paca_struct' has no member named 'in_mce'
    1424 |  pr_cont("IRQMASK: %lx IN_NMI:%d IN_MCE:%d", regs->softe, (int)get_paca()->in_nmi, (int)get_paca()->in_mce);
         |                                                                                                   ^~
   include/linux/printk.h:317:26: note: in definition of macro 'pr_cont'
     317 |  printk(KERN_CONT fmt, ##__VA_ARGS__)
         |                          ^~~~~~~~~~~

vim +1424 arch/powerpc/kernel/process.c

  1400	
  1401	void show_regs(struct pt_regs * regs)
  1402	{
  1403		int i, trap;
  1404	
  1405		show_regs_print_info(KERN_DEFAULT);
  1406	
  1407		printk("NIP:  "REG" LR: "REG" CTR: "REG"\n",
  1408		       regs->nip, regs->link, regs->ctr);
  1409		printk("REGS: %px TRAP: %04lx   %s  (%s)\n",
  1410		       regs, regs->trap, print_tainted(), init_utsname()->release);
  1411		printk("MSR:  "REG" ", regs->msr);
  1412		print_msr_bits(regs->msr);
  1413		pr_cont("  CR: %08lx  XER: %08lx\n", regs->ccr, regs->xer);
  1414		trap = TRAP(regs);
  1415		if ((TRAP(regs) != 0xc00) && cpu_has_feature(CPU_FTR_CFAR))
  1416			pr_cont("CFAR: "REG" ", regs->orig_gpr3);
  1417		if (trap == 0x200 || trap == 0x300 || trap == 0x600)
  1418	#if defined(CONFIG_4xx) || defined(CONFIG_BOOKE)
  1419			pr_cont("DEAR: "REG" ESR: "REG" ", regs->dar, regs->dsisr);
  1420	#else
  1421			pr_cont("DAR: "REG" DSISR: %08lx ", regs->dar, regs->dsisr);
  1422	#endif
  1423	#ifdef CONFIG_PPC64
> 1424		pr_cont("IRQMASK: %lx IN_NMI:%d IN_MCE:%d", regs->softe, (int)get_paca()->in_nmi, (int)get_paca()->in_mce);
  1425	#endif
  1426	#ifdef CONFIG_PPC_TRANSACTIONAL_MEM
  1427		if (MSR_TM_ACTIVE(regs->msr))
  1428			pr_cont("\nPACATMSCRATCH: %016llx ", get_paca()->tm_scratch);
  1429	#endif
  1430	
  1431		for (i = 0;  i < 32;  i++) {
  1432			if ((i % REGS_PER_LINE) == 0)
  1433				pr_cont("\nGPR%02d: ", i);
  1434			pr_cont(REG " ", regs->gpr[i]);
  1435			if (i == LAST_VOLATILE && !FULL_REGS(regs))
  1436				break;
  1437		}
  1438		pr_cont("\n");
  1439	#ifdef CONFIG_KALLSYMS
  1440		/*
  1441		 * Lookup NIP late so we have the best change of getting the
  1442		 * above info out without failing
  1443		 */
  1444		printk("NIP ["REG"] %pS\n", regs->nip, (void *)regs->nip);
  1445		printk("LR ["REG"] %pS\n", regs->link, (void *)regs->link);
  1446	#endif
  1447		show_stack(current, (unsigned long *) regs->gpr[1]);
  1448		if (!user_mode(regs))
  1449			show_instructions(regs);
  1450	}
  1451	

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org

[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 30335 bytes --]

WARNING: multiple messages have this Message-ID (diff)
From: kbuild test robot <lkp@intel.com>
To: kbuild-all@lists.01.org
Subject: Re: [PATCH v3 11/15] powerpc/64s: machine check interrupt update NMI accounting
Date: Sun, 12 Apr 2020 16:46:09 +0800	[thread overview]
Message-ID: <202004121649.GhAwOgAj%lkp@intel.com> (raw)
In-Reply-To: <20200407051636.648369-12-npiggin@gmail.com>

[-- Attachment #1: Type: text/plain, Size: 4944 bytes --]

Hi Nicholas,

I love your patch! Yet something to improve:

[auto build test ERROR on powerpc/next]
[also build test ERROR on next-20200412]
[cannot apply to tip/perf/core v5.6]
[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/Nicholas-Piggin/powerpc-64-machine-check-and-system-reset-fixes/20200407-134803
base:   https://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux.git next
config: powerpc-randconfig-a001-20200412 (attached as .config)
compiler: powerpc64-linux-gcc (GCC) 9.3.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=9.3.0 make.cross ARCH=powerpc 

If you fix the issue, kindly add following tag as appropriate
Reported-by: kbuild test robot <lkp@intel.com>

All errors (new ones prefixed by >>):

   In file included from include/linux/kernel.h:15,
                    from include/asm-generic/bug.h:19,
                    from arch/powerpc/include/asm/bug.h:109,
                    from include/linux/bug.h:5,
                    from arch/powerpc/include/asm/mmu.h:130,
                    from arch/powerpc/include/asm/paca.h:18,
                    from arch/powerpc/include/asm/current.h:13,
                    from include/linux/sched.h:12,
                    from arch/powerpc/kernel/process.c:14:
   arch/powerpc/kernel/process.c: In function 'show_regs':
>> arch/powerpc/kernel/process.c:1424:74: error: 'struct paca_struct' has no member named 'in_nmi'
    1424 |  pr_cont("IRQMASK: %lx IN_NMI:%d IN_MCE:%d", regs->softe, (int)get_paca()->in_nmi, (int)get_paca()->in_mce);
         |                                                                          ^~
   include/linux/printk.h:317:26: note: in definition of macro 'pr_cont'
     317 |  printk(KERN_CONT fmt, ##__VA_ARGS__)
         |                          ^~~~~~~~~~~
>> arch/powerpc/kernel/process.c:1424:99: error: 'struct paca_struct' has no member named 'in_mce'
    1424 |  pr_cont("IRQMASK: %lx IN_NMI:%d IN_MCE:%d", regs->softe, (int)get_paca()->in_nmi, (int)get_paca()->in_mce);
         |                                                                                                   ^~
   include/linux/printk.h:317:26: note: in definition of macro 'pr_cont'
     317 |  printk(KERN_CONT fmt, ##__VA_ARGS__)
         |                          ^~~~~~~~~~~

vim +1424 arch/powerpc/kernel/process.c

  1400	
  1401	void show_regs(struct pt_regs * regs)
  1402	{
  1403		int i, trap;
  1404	
  1405		show_regs_print_info(KERN_DEFAULT);
  1406	
  1407		printk("NIP:  "REG" LR: "REG" CTR: "REG"\n",
  1408		       regs->nip, regs->link, regs->ctr);
  1409		printk("REGS: %px TRAP: %04lx   %s  (%s)\n",
  1410		       regs, regs->trap, print_tainted(), init_utsname()->release);
  1411		printk("MSR:  "REG" ", regs->msr);
  1412		print_msr_bits(regs->msr);
  1413		pr_cont("  CR: %08lx  XER: %08lx\n", regs->ccr, regs->xer);
  1414		trap = TRAP(regs);
  1415		if ((TRAP(regs) != 0xc00) && cpu_has_feature(CPU_FTR_CFAR))
  1416			pr_cont("CFAR: "REG" ", regs->orig_gpr3);
  1417		if (trap == 0x200 || trap == 0x300 || trap == 0x600)
  1418	#if defined(CONFIG_4xx) || defined(CONFIG_BOOKE)
  1419			pr_cont("DEAR: "REG" ESR: "REG" ", regs->dar, regs->dsisr);
  1420	#else
  1421			pr_cont("DAR: "REG" DSISR: %08lx ", regs->dar, regs->dsisr);
  1422	#endif
  1423	#ifdef CONFIG_PPC64
> 1424		pr_cont("IRQMASK: %lx IN_NMI:%d IN_MCE:%d", regs->softe, (int)get_paca()->in_nmi, (int)get_paca()->in_mce);
  1425	#endif
  1426	#ifdef CONFIG_PPC_TRANSACTIONAL_MEM
  1427		if (MSR_TM_ACTIVE(regs->msr))
  1428			pr_cont("\nPACATMSCRATCH: %016llx ", get_paca()->tm_scratch);
  1429	#endif
  1430	
  1431		for (i = 0;  i < 32;  i++) {
  1432			if ((i % REGS_PER_LINE) == 0)
  1433				pr_cont("\nGPR%02d: ", i);
  1434			pr_cont(REG " ", regs->gpr[i]);
  1435			if (i == LAST_VOLATILE && !FULL_REGS(regs))
  1436				break;
  1437		}
  1438		pr_cont("\n");
  1439	#ifdef CONFIG_KALLSYMS
  1440		/*
  1441		 * Lookup NIP late so we have the best change of getting the
  1442		 * above info out without failing
  1443		 */
  1444		printk("NIP ["REG"] %pS\n", regs->nip, (void *)regs->nip);
  1445		printk("LR ["REG"] %pS\n", regs->link, (void *)regs->link);
  1446	#endif
  1447		show_stack(current, (unsigned long *) regs->gpr[1]);
  1448		if (!user_mode(regs))
  1449			show_instructions(regs);
  1450	}
  1451	

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org

[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 30335 bytes --]

  parent reply	other threads:[~2020-04-12  8:48 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-04-07  5:16 [PATCH v3 00/15] powerpc/64: machine check and system reset fixes Nicholas Piggin
2020-04-07  5:16 ` [PATCH v3 01/15] powerpc/64s/exception: Fix machine check no-loss idle wakeup Nicholas Piggin
2020-04-07  5:16 ` [PATCH v3 02/15] powerpc/64s/exceptions: Fix in_mce accounting in unrecoverable path Nicholas Piggin
2020-04-07  5:16 ` [PATCH v3 03/15] powerpc/64s/exceptions: Change irq reconcile for NMIs from reusing _DAR to RESULT Nicholas Piggin
2020-04-07  5:16 ` [PATCH v3 04/15] powerpc/64s/exceptions: machine check reconcile irq state Nicholas Piggin
2020-04-07  5:16 ` [PATCH v3 05/15] powerpc/pseries/ras: avoid calling rtas_token in NMI paths Nicholas Piggin
2020-04-07  5:42   ` Christophe Leroy
2020-04-07  5:16 ` [PATCH v3 06/15] powerpc/pseries/ras: FWNMI_VALID off by one Nicholas Piggin
2020-04-07  5:16 ` [PATCH v3 07/15] powerpc/pseries/ras: fwnmi avoid modifying r3 in error case Nicholas Piggin
2020-04-07  5:16 ` [PATCH v3 08/15] powerpc/pseries/ras: fwnmi sreset should not interlock Nicholas Piggin
2020-04-07  5:41   ` Christophe Leroy
2020-04-07  5:16 ` [PATCH v3 09/15] powerpc/pseries: limit machine check stack to 4GB Nicholas Piggin
2020-04-07  5:38   ` Christophe Leroy
2020-04-07  5:16 ` [PATCH v3 10/15] powerpc/pseries: machine check use rtas_call_unlocked with args on stack Nicholas Piggin
2020-04-07  5:16 ` [PATCH v3 11/15] powerpc/64s: machine check interrupt update NMI accounting Nicholas Piggin
2020-04-07  5:37   ` Christophe Leroy
2020-05-06  3:50     ` Nicholas Piggin
2020-04-12  8:46   ` kbuild test robot [this message]
2020-04-12  8:46     ` kbuild test robot
2020-04-07  5:16 ` [PATCH v3 12/15] powerpc: ftrace_enabled helper Nicholas Piggin
2020-04-07  5:45   ` Christophe Leroy
2020-04-07  5:16 ` [PATCH v3 13/15] powerpc/64s: machine check do not trace real-mode handler Nicholas Piggin
2020-04-07  5:45   ` Christophe Leroy
2020-04-07  5:16 ` [PATCH v3 14/15] powerpc/64s: system reset do not trace Nicholas Piggin
2020-04-07  5:45   ` Christophe Leroy
2020-04-07  5:16 ` [PATCH v3 15/15] powerpc: make unrecoverable NMIs die instead of panic Nicholas Piggin
2020-04-07  5:46   ` Christophe Leroy

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=202004121649.GhAwOgAj%lkp@intel.com \
    --to=lkp@intel.com \
    --cc=kbuild-all@lists.01.org \
    --cc=linuxppc-dev@lists.ozlabs.org \
    --cc=npiggin@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.