From: kbuild test robot <lkp@intel.com>
To: Mike Rapoport <rppt@linux.ibm.com>
Cc: kbuild-all@lists.01.org, Johannes Weiner <hannes@cmpxchg.org>,
Andrew Morton <akpm@linux-foundation.org>,
Linux Memory Management List <linux-mm@kvack.org>
Subject: [hnaz-linux-mm:master 476/523] arch/powerpc/xmon/xmon.c:3138:16: warning: variable 'pgdir' set but not used
Date: Tue, 19 May 2020 19:56:50 +0800 [thread overview]
Message-ID: <202005191943.u6XYFYID%lkp@intel.com> (raw)
[-- Attachment #1: Type: text/plain, Size: 10410 bytes --]
tree: https://github.com/hnaz/linux-mm master
head: 2bbf0589bfeb27800c730b76eacf34528eee5418
commit: 47dde7dbc279f3698c76a7b92ac0d0f1913539af [476/523] powerpc: add support for folded p4d page tables
config: powerpc-allyesconfig (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
git checkout 47dde7dbc279f3698c76a7b92ac0d0f1913539af
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-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 warnings (new ones prefixed by >>, old ones prefixed by <<):
arch/powerpc/xmon/xmon.c: In function 'show_pte':
>> arch/powerpc/xmon/xmon.c:3138:16: warning: variable 'pgdir' set but not used [-Wunused-but-set-variable]
3138 | pgd_t *pgdp, *pgdir;
| ^~~~~
arch/powerpc/xmon/xmon.c: At top level:
arch/powerpc/xmon/xmon.c:4122:1: warning: no previous prototype for 'spu_inst_dump' [-Wmissing-prototypes]
4122 | spu_inst_dump(unsigned long adr, long count, int praddr)
| ^~~~~~~~~~~~~
arch/powerpc/xmon/xmon.c: In function 'xmon_print_symbol':
arch/powerpc/xmon/xmon.c:3557:14: warning: variable 'name' might be clobbered by 'longjmp' or 'vfork' [-Wclobbered]
3557 | const char *name = NULL;
| ^~~~
arch/powerpc/xmon/xmon.c: In function 'stop_spus':
arch/powerpc/xmon/xmon.c:3990:6: warning: variable 'i' might be clobbered by 'longjmp' or 'vfork' [-Wclobbered]
3990 | int i;
| ^
arch/powerpc/xmon/xmon.c: In function 'restart_spus':
arch/powerpc/xmon/xmon.c:4031:6: warning: variable 'i' might be clobbered by 'longjmp' or 'vfork' [-Wclobbered]
4031 | int i;
| ^
arch/powerpc/xmon/xmon.c: In function 'dump_opal_msglog':
arch/powerpc/xmon/xmon.c:2937:9: warning: variable 'pos' might be clobbered by 'longjmp' or 'vfork' [-Wclobbered]
2937 | loff_t pos = 0;
| ^~~
arch/powerpc/xmon/xmon.c: In function 'show_pte':
arch/powerpc/xmon/xmon.c:3136:22: warning: variable 'tsk' might be clobbered by 'longjmp' or 'vfork' [-Wclobbered]
3136 | struct task_struct *tsk = NULL;
| ^~~
arch/powerpc/xmon/xmon.c: In function 'show_tasks':
arch/powerpc/xmon/xmon.c:3234:22: warning: variable 'tsk' might be clobbered by 'longjmp' or 'vfork' [-Wclobbered]
3234 | struct task_struct *tsk = NULL;
| ^~~
arch/powerpc/xmon/xmon.c: In function 'xmon_core':
arch/powerpc/xmon/xmon.c:490:6: warning: variable 'cmd' might be clobbered by 'longjmp' or 'vfork' [-Wclobbered]
490 | int cmd = 0;
| ^~~
arch/powerpc/xmon/xmon.c:849:14: warning: variable 'bp' might be clobbered by 'longjmp' or 'vfork' [-Wclobbered]
849 | struct bpt *bp;
| ^~
arch/powerpc/xmon/xmon.c:849:14: warning: variable 'bp' might be clobbered by 'longjmp' or 'vfork' [-Wclobbered]
arch/powerpc/xmon/xmon.c:488:48: warning: argument 'fromipi' might be clobbered by 'longjmp' or 'vfork' [-Wclobbered]
488 | static int xmon_core(struct pt_regs *regs, int fromipi)
| ~~~~^~~~~~~
vim +/pgdir +3138 arch/powerpc/xmon/xmon.c
80eff6c4847997 Balbir Singh 2017-10-30 3132
80eff6c4847997 Balbir Singh 2017-10-30 3133 static void show_pte(unsigned long addr)
80eff6c4847997 Balbir Singh 2017-10-30 3134 {
80eff6c4847997 Balbir Singh 2017-10-30 3135 unsigned long tskv = 0;
80eff6c4847997 Balbir Singh 2017-10-30 3136 struct task_struct *tsk = NULL;
80eff6c4847997 Balbir Singh 2017-10-30 3137 struct mm_struct *mm;
80eff6c4847997 Balbir Singh 2017-10-30 @3138 pgd_t *pgdp, *pgdir;
47dde7dbc279f3 Mike Rapoport 2020-05-16 3139 p4d_t *p4dp;
80eff6c4847997 Balbir Singh 2017-10-30 3140 pud_t *pudp;
80eff6c4847997 Balbir Singh 2017-10-30 3141 pmd_t *pmdp;
80eff6c4847997 Balbir Singh 2017-10-30 3142 pte_t *ptep;
80eff6c4847997 Balbir Singh 2017-10-30 3143
80eff6c4847997 Balbir Singh 2017-10-30 3144 if (!scanhex(&tskv))
80eff6c4847997 Balbir Singh 2017-10-30 3145 mm = &init_mm;
80eff6c4847997 Balbir Singh 2017-10-30 3146 else
80eff6c4847997 Balbir Singh 2017-10-30 3147 tsk = (struct task_struct *)tskv;
80eff6c4847997 Balbir Singh 2017-10-30 3148
80eff6c4847997 Balbir Singh 2017-10-30 3149 if (tsk == NULL)
80eff6c4847997 Balbir Singh 2017-10-30 3150 mm = &init_mm;
80eff6c4847997 Balbir Singh 2017-10-30 3151 else
80eff6c4847997 Balbir Singh 2017-10-30 3152 mm = tsk->active_mm;
80eff6c4847997 Balbir Singh 2017-10-30 3153
80eff6c4847997 Balbir Singh 2017-10-30 3154 if (setjmp(bus_error_jmp) != 0) {
80eff6c4847997 Balbir Singh 2017-10-30 3155 catch_memory_errors = 0;
d8104182087319 Michael Ellerman 2017-12-06 3156 printf("*** Error dumping pte for task %px\n", tsk);
80eff6c4847997 Balbir Singh 2017-10-30 3157 return;
80eff6c4847997 Balbir Singh 2017-10-30 3158 }
80eff6c4847997 Balbir Singh 2017-10-30 3159
80eff6c4847997 Balbir Singh 2017-10-30 3160 catch_memory_errors = 1;
80eff6c4847997 Balbir Singh 2017-10-30 3161 sync();
80eff6c4847997 Balbir Singh 2017-10-30 3162
80eff6c4847997 Balbir Singh 2017-10-30 3163 if (mm == &init_mm) {
80eff6c4847997 Balbir Singh 2017-10-30 3164 pgdp = pgd_offset_k(addr);
80eff6c4847997 Balbir Singh 2017-10-30 3165 pgdir = pgd_offset_k(0);
80eff6c4847997 Balbir Singh 2017-10-30 3166 } else {
80eff6c4847997 Balbir Singh 2017-10-30 3167 pgdp = pgd_offset(mm, addr);
80eff6c4847997 Balbir Singh 2017-10-30 3168 pgdir = pgd_offset(mm, 0);
80eff6c4847997 Balbir Singh 2017-10-30 3169 }
80eff6c4847997 Balbir Singh 2017-10-30 3170
47dde7dbc279f3 Mike Rapoport 2020-05-16 3171 p4dp = p4d_offset(pgdp, addr);
47dde7dbc279f3 Mike Rapoport 2020-05-16 3172
47dde7dbc279f3 Mike Rapoport 2020-05-16 3173 if (p4d_none(*p4dp)) {
47dde7dbc279f3 Mike Rapoport 2020-05-16 3174 printf("No valid P4D\n");
80eff6c4847997 Balbir Singh 2017-10-30 3175 return;
80eff6c4847997 Balbir Singh 2017-10-30 3176 }
80eff6c4847997 Balbir Singh 2017-10-30 3177
47dde7dbc279f3 Mike Rapoport 2020-05-16 3178 if (p4d_is_leaf(*p4dp)) {
47dde7dbc279f3 Mike Rapoport 2020-05-16 3179 format_pte(p4dp, p4d_val(*p4dp));
80eff6c4847997 Balbir Singh 2017-10-30 3180 return;
80eff6c4847997 Balbir Singh 2017-10-30 3181 }
80eff6c4847997 Balbir Singh 2017-10-30 3182
47dde7dbc279f3 Mike Rapoport 2020-05-16 3183 printf("p4dp @ 0x%px = 0x%016lx\n", p4dp, p4d_val(*p4dp));
47dde7dbc279f3 Mike Rapoport 2020-05-16 3184
47dde7dbc279f3 Mike Rapoport 2020-05-16 3185 pudp = pud_offset(p4dp, addr);
80eff6c4847997 Balbir Singh 2017-10-30 3186
80eff6c4847997 Balbir Singh 2017-10-30 3187 if (pud_none(*pudp)) {
80eff6c4847997 Balbir Singh 2017-10-30 3188 printf("No valid PUD\n");
80eff6c4847997 Balbir Singh 2017-10-30 3189 return;
80eff6c4847997 Balbir Singh 2017-10-30 3190 }
80eff6c4847997 Balbir Singh 2017-10-30 3191
d6eacedd1f0ebf Aneesh Kumar K.V 2019-05-14 3192 if (pud_is_leaf(*pudp)) {
80eff6c4847997 Balbir Singh 2017-10-30 3193 format_pte(pudp, pud_val(*pudp));
80eff6c4847997 Balbir Singh 2017-10-30 3194 return;
80eff6c4847997 Balbir Singh 2017-10-30 3195 }
80eff6c4847997 Balbir Singh 2017-10-30 3196
e70d8f55268ba9 Mathieu Malaterre 2018-03-25 3197 printf("pudp @ 0x%px = 0x%016lx\n", pudp, pud_val(*pudp));
80eff6c4847997 Balbir Singh 2017-10-30 3198
80eff6c4847997 Balbir Singh 2017-10-30 3199 pmdp = pmd_offset(pudp, addr);
80eff6c4847997 Balbir Singh 2017-10-30 3200
80eff6c4847997 Balbir Singh 2017-10-30 3201 if (pmd_none(*pmdp)) {
80eff6c4847997 Balbir Singh 2017-10-30 3202 printf("No valid PMD\n");
80eff6c4847997 Balbir Singh 2017-10-30 3203 return;
80eff6c4847997 Balbir Singh 2017-10-30 3204 }
80eff6c4847997 Balbir Singh 2017-10-30 3205
d6eacedd1f0ebf Aneesh Kumar K.V 2019-05-14 3206 if (pmd_is_leaf(*pmdp)) {
80eff6c4847997 Balbir Singh 2017-10-30 3207 format_pte(pmdp, pmd_val(*pmdp));
80eff6c4847997 Balbir Singh 2017-10-30 3208 return;
80eff6c4847997 Balbir Singh 2017-10-30 3209 }
e70d8f55268ba9 Mathieu Malaterre 2018-03-25 3210 printf("pmdp @ 0x%px = 0x%016lx\n", pmdp, pmd_val(*pmdp));
80eff6c4847997 Balbir Singh 2017-10-30 3211
80eff6c4847997 Balbir Singh 2017-10-30 3212 ptep = pte_offset_map(pmdp, addr);
80eff6c4847997 Balbir Singh 2017-10-30 3213 if (pte_none(*ptep)) {
80eff6c4847997 Balbir Singh 2017-10-30 3214 printf("no valid PTE\n");
80eff6c4847997 Balbir Singh 2017-10-30 3215 return;
80eff6c4847997 Balbir Singh 2017-10-30 3216 }
80eff6c4847997 Balbir Singh 2017-10-30 3217
80eff6c4847997 Balbir Singh 2017-10-30 3218 format_pte(ptep, pte_val(*ptep));
80eff6c4847997 Balbir Singh 2017-10-30 3219
80eff6c4847997 Balbir Singh 2017-10-30 3220 sync();
80eff6c4847997 Balbir Singh 2017-10-30 3221 __delay(200);
80eff6c4847997 Balbir Singh 2017-10-30 3222 catch_memory_errors = 0;
80eff6c4847997 Balbir Singh 2017-10-30 3223 }
80eff6c4847997 Balbir Singh 2017-10-30 3224 #else
80eff6c4847997 Balbir Singh 2017-10-30 3225 static void show_pte(unsigned long addr)
80eff6c4847997 Balbir Singh 2017-10-30 3226 {
80eff6c4847997 Balbir Singh 2017-10-30 3227 printf("show_pte not yet implemented\n");
80eff6c4847997 Balbir Singh 2017-10-30 3228 }
80eff6c4847997 Balbir Singh 2017-10-30 3229 #endif /* CONFIG_PPC_BOOK3S_64 */
80eff6c4847997 Balbir Singh 2017-10-30 3230
:::::: The code at line 3138 was first introduced by commit
:::::: 80eff6c484799722736471d15ff9cc86b64cae7a powerpc/xmon: Support dumping software pagetables
:::::: TO: Balbir Singh <bsingharora@gmail.com>
:::::: CC: Michael Ellerman <mpe@ellerman.id.au>
---
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: 68830 bytes --]
next reply other threads:[~2020-05-19 11:57 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-05-19 11:56 kbuild test robot [this message]
2020-05-19 18:14 ` [hnaz-linux-mm:master 476/523] arch/powerpc/xmon/xmon.c:3138:16: warning: variable 'pgdir' set but not used Mike Rapoport
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=202005191943.u6XYFYID%lkp@intel.com \
--to=lkp@intel.com \
--cc=akpm@linux-foundation.org \
--cc=hannes@cmpxchg.org \
--cc=kbuild-all@lists.01.org \
--cc=linux-mm@kvack.org \
--cc=rppt@linux.ibm.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).