From: kernel test robot <lkp@intel.com>
To: Huacai Chen <chenhuacai@loongson.cn>,
Arnd Bergmann <arnd@arndb.de>,
Huacai Chen <chenhuacai@kernel.org>
Cc: kbuild-all@lists.01.org, loongarch@lists.linux.dev,
linux-arch@vger.kernel.org, Xuefeng Li <lixuefeng@loongson.cn>,
Guo Ren <guoren@kernel.org>, Xuerui Wang <kernel@xen0n.name>,
Jiaxun Yang <jiaxun.yang@flygoat.com>,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH] LoongArch: Use TLB for ioremap()
Date: Tue, 16 Aug 2022 02:29:24 +0800 [thread overview]
Message-ID: <202208160237.kJBIqapz-lkp@intel.com> (raw)
In-Reply-To: <20220815124612.3328670-1-chenhuacai@loongson.cn>
Hi Huacai,
I love your patch! Perhaps something to improve:
[auto build test WARNING on linus/master]
[also build test WARNING on v6.0-rc1 next-20220815]
[cannot apply to soc/for-next]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch#_base_tree_information]
url: https://github.com/intel-lab-lkp/linux/commits/Huacai-Chen/LoongArch-Use-TLB-for-ioremap/20220815-204842
base: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git 568035b01cfb107af8d2e4bd2fb9aea22cf5b868
config: loongarch-buildonly-randconfig-r003-20220815 (https://download.01.org/0day-ci/archive/20220816/202208160237.kJBIqapz-lkp@intel.com/config)
compiler: loongarch64-linux-gcc (GCC) 12.1.0
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# https://github.com/intel-lab-lkp/linux/commit/0cdab71896d6b3b3b3f540f80d6f041a0c592e7a
git remote add linux-review https://github.com/intel-lab-lkp/linux
git fetch --no-tags linux-review Huacai-Chen/LoongArch-Use-TLB-for-ioremap/20220815-204842
git checkout 0cdab71896d6b3b3b3f540f80d6f041a0c592e7a
# save the config file
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.1.0 make.cross W=1 ARCH=loongarch
If you fix the issue, kindly add following tag where applicable
Reported-by: kernel test robot <lkp@intel.com>
All warnings (new ones prefixed by >>):
arch/loongarch/mm/init.c:47:6: warning: no previous prototype for 'setup_zero_pages' [-Wmissing-prototypes]
47 | void setup_zero_pages(void)
| ^~~~~~~~~~~~~~~~
arch/loongarch/mm/init.c: In function 'fixmap_pte':
>> arch/loongarch/mm/init.c:171:24: warning: variable 'new' set but not used [-Wunused-but-set-variable]
171 | pud_t *new;
| ^~~
arch/loongarch/mm/init.c:182:24: warning: variable 'new' set but not used [-Wunused-but-set-variable]
182 | pmd_t *new;
| ^~~
--
>> arch/loongarch/pci/acpi.c:91:27: warning: no previous prototype for 'arch_pci_ecam_create' [-Wmissing-prototypes]
91 | struct pci_config_window *arch_pci_ecam_create(struct device *dev,
| ^~~~~~~~~~~~~~~~~~~~
arch/loongarch/pci/acpi.c: In function 'pci_acpi_setup_ecam_mapping':
arch/loongarch/pci/acpi.c:166:29: error: 'loongson_pci_ecam_ops' undeclared (first use in this function)
166 | ecam_ops = &loongson_pci_ecam_ops;
| ^~~~~~~~~~~~~~~~~~~~~
arch/loongarch/pci/acpi.c:166:29: note: each undeclared identifier is reported only once for each function it appears in
Kconfig warnings: (for reference only)
WARNING: unmet direct dependencies detected for PCI_LOONGSON
Depends on [n]: PCI [=y] && (MACH_LOONGSON64 [=y] || COMPILE_TEST [=y]) && (OF [=y] || ACPI [=y]) && PCI_QUIRKS [=n]
Selected by [y]:
- LOONGARCH [=y]
vim +/new +171 arch/loongarch/mm/init.c
159
160 static pte_t *fixmap_pte(unsigned long addr)
161 {
162 pgd_t *pgd;
163 p4d_t *p4d;
164 pud_t *pud;
165 pmd_t *pmd;
166
167 pgd = pgd_offset_k(addr);
168 p4d = p4d_offset(pgd, addr);
169
170 if (pgd_none(*pgd)) {
> 171 pud_t *new;
172
173 new = memblock_alloc_low(PAGE_SIZE, PAGE_SIZE);
174 pgd_populate(&init_mm, pgd, new);
175 #ifndef __PAGETABLE_PUD_FOLDED
176 pud_init(new);
177 #endif
178 }
179
180 pud = pud_offset(p4d, addr);
181 if (pud_none(*pud)) {
182 pmd_t *new;
183
184 new = memblock_alloc_low(PAGE_SIZE, PAGE_SIZE);
185 pud_populate(&init_mm, pud, new);
186 #ifndef __PAGETABLE_PMD_FOLDED
187 pmd_init(new);
188 #endif
189 }
190
191 pmd = pmd_offset(pud, addr);
192 if (pmd_none(*pmd)) {
193 pte_t *new;
194
195 new = memblock_alloc_low(PAGE_SIZE, PAGE_SIZE);
196 pmd_populate_kernel(&init_mm, pmd, new);
197 }
198
199 return pte_offset_kernel(pmd, addr);
200 }
201
--
0-DAY CI Kernel Test Service
https://01.org/lkp
next prev parent reply other threads:[~2022-08-15 18:53 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-08-15 12:46 [PATCH] LoongArch: Use TLB for ioremap() Huacai Chen
2022-08-15 18:29 ` kernel test robot
2022-08-15 18:29 ` kernel test robot [this message]
2022-08-17 0:41 ` kernel test robot
2022-08-21 13:23 ` Christoph Hellwig
2022-08-22 2:19 ` Huacai Chen
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=202208160237.kJBIqapz-lkp@intel.com \
--to=lkp@intel.com \
--cc=arnd@arndb.de \
--cc=chenhuacai@kernel.org \
--cc=chenhuacai@loongson.cn \
--cc=guoren@kernel.org \
--cc=jiaxun.yang@flygoat.com \
--cc=kbuild-all@lists.01.org \
--cc=kernel@xen0n.name \
--cc=linux-arch@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=lixuefeng@loongson.cn \
--cc=loongarch@lists.linux.dev \
/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