From: kernel test robot <lkp@intel.com>
To: Bjorn Helgaas <helgaas@kernel.org>
Cc: kbuild-all@lists.01.org, linux-pci@vger.kernel.org,
Hans de Goede <hdegoede@redhat.com>,
Mika Westerberg <mika.westerberg@linux.intel.com>
Subject: [helgaas-pci:for-linus 3/3] arch/x86/kernel/resource.c:51:18: error: missing terminating " character
Date: Thu, 10 Mar 2022 15:12:42 +0800 [thread overview]
Message-ID: <202203101540.DAEpd9Mt-lkp@intel.com> (raw)
tree: https://git.kernel.org/pub/scm/linux/kernel/git/helgaas/pci.git for-linus
head: 3018c5d59f923be2fa2d91c4ce02c8d97a37edce
commit: 3018c5d59f923be2fa2d91c4ce02c8d97a37edce [3/3] x86/PCI: Preserve host bridge windows completely covered by E820
config: x86_64-randconfig-a002 (https://download.01.org/0day-ci/archive/20220310/202203101540.DAEpd9Mt-lkp@intel.com/config)
compiler: gcc-9 (Debian 9.3.0-22) 9.3.0
reproduce (this is a W=1 build):
# https://git.kernel.org/pub/scm/linux/kernel/git/helgaas/pci.git/commit/?id=3018c5d59f923be2fa2d91c4ce02c8d97a37edce
git remote add helgaas-pci https://git.kernel.org/pub/scm/linux/kernel/git/helgaas/pci.git
git fetch --no-tags helgaas-pci for-linus
git checkout 3018c5d59f923be2fa2d91c4ce02c8d97a37edce
# save the config file to linux build tree
mkdir build_dir
make W=1 O=build_dir ARCH=x86_64 SHELL=/bin/bash
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
All errors (new ones prefixed by >>):
arch/x86/kernel/resource.c: In function 'remove_e820_regions':
arch/x86/kernel/resource.c:51:18: warning: missing terminating " character
51 | dev_info(dev, "resource %pR fully covered by e820 entry
| ^
arch/x86/kernel/resource.c:52:24: warning: missing terminating " character
52 | [mem %#010Lx-%#010Lx]\n",
| ^
In file included from arch/x86/kernel/resource.c:2:
>> arch/x86/kernel/resource.c:51:18: error: missing terminating " character
51 | dev_info(dev, "resource %pR fully covered by e820 entry
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/dev_printk.h:110:16: note: in definition of macro 'dev_printk_index_wrap'
110 | _p_func(dev, fmt, ##__VA_ARGS__); \
| ^~~
include/linux/dev_printk.h:150:51: note: in expansion of macro 'dev_fmt'
150 | dev_printk_index_wrap(_dev_info, KERN_INFO, dev, dev_fmt(fmt), ##__VA_ARGS__)
| ^~~~~~~
arch/x86/kernel/resource.c:51:4: note: in expansion of macro 'dev_info'
51 | dev_info(dev, "resource %pR fully covered by e820 entry
| ^~~~~~~~
>> arch/x86/kernel/resource.c:52:1: error: expected expression before '[' token
52 | [mem %#010Lx-%#010Lx]\n",
| ^
include/linux/dev_printk.h:110:16: note: in definition of macro 'dev_printk_index_wrap'
110 | _p_func(dev, fmt, ##__VA_ARGS__); \
| ^~~
include/linux/dev_printk.h:150:51: note: in expansion of macro 'dev_fmt'
150 | dev_printk_index_wrap(_dev_info, KERN_INFO, dev, dev_fmt(fmt), ##__VA_ARGS__)
| ^~~~~~~
arch/x86/kernel/resource.c:51:4: note: in expansion of macro 'dev_info'
51 | dev_info(dev, "resource %pR fully covered by e820 entry
| ^~~~~~~~
>> arch/x86/kernel/resource.c:52:2: error: 'mem' undeclared (first use in this function); did you mean 'sem'?
52 | [mem %#010Lx-%#010Lx]\n",
| ^~~
include/linux/dev_printk.h:110:16: note: in definition of macro 'dev_printk_index_wrap'
110 | _p_func(dev, fmt, ##__VA_ARGS__); \
| ^~~
include/linux/dev_printk.h:150:51: note: in expansion of macro 'dev_fmt'
150 | dev_printk_index_wrap(_dev_info, KERN_INFO, dev, dev_fmt(fmt), ##__VA_ARGS__)
| ^~~~~~~
arch/x86/kernel/resource.c:51:4: note: in expansion of macro 'dev_info'
51 | dev_info(dev, "resource %pR fully covered by e820 entry
| ^~~~~~~~
arch/x86/kernel/resource.c:52:2: note: each undeclared identifier is reported only once for each function it appears in
52 | [mem %#010Lx-%#010Lx]\n",
| ^~~
include/linux/dev_printk.h:110:16: note: in definition of macro 'dev_printk_index_wrap'
110 | _p_func(dev, fmt, ##__VA_ARGS__); \
| ^~~
include/linux/dev_printk.h:150:51: note: in expansion of macro 'dev_fmt'
150 | dev_printk_index_wrap(_dev_info, KERN_INFO, dev, dev_fmt(fmt), ##__VA_ARGS__)
| ^~~~~~~
arch/x86/kernel/resource.c:51:4: note: in expansion of macro 'dev_info'
51 | dev_info(dev, "resource %pR fully covered by e820 entry
| ^~~~~~~~
>> arch/x86/kernel/resource.c:52:7: error: stray '#' in program
52 | [mem %#010Lx-%#010Lx]\n",
| ^
include/linux/dev_printk.h:110:16: note: in definition of macro 'dev_printk_index_wrap'
110 | _p_func(dev, fmt, ##__VA_ARGS__); \
| ^~~
include/linux/dev_printk.h:150:51: note: in expansion of macro 'dev_fmt'
150 | dev_printk_index_wrap(_dev_info, KERN_INFO, dev, dev_fmt(fmt), ##__VA_ARGS__)
| ^~~~~~~
arch/x86/kernel/resource.c:51:4: note: in expansion of macro 'dev_info'
51 | dev_info(dev, "resource %pR fully covered by e820 entry
| ^~~~~~~~
>> arch/x86/kernel/resource.c:52:8: error: invalid suffix "Lx" on integer constant
52 | [mem %#010Lx-%#010Lx]\n",
| ^~~~~
include/linux/dev_printk.h:110:16: note: in definition of macro 'dev_printk_index_wrap'
110 | _p_func(dev, fmt, ##__VA_ARGS__); \
| ^~~
include/linux/dev_printk.h:150:51: note: in expansion of macro 'dev_fmt'
150 | dev_printk_index_wrap(_dev_info, KERN_INFO, dev, dev_fmt(fmt), ##__VA_ARGS__)
| ^~~~~~~
arch/x86/kernel/resource.c:51:4: note: in expansion of macro 'dev_info'
51 | dev_info(dev, "resource %pR fully covered by e820 entry
| ^~~~~~~~
arch/x86/kernel/resource.c:52:15: error: stray '#' in program
52 | [mem %#010Lx-%#010Lx]\n",
| ^
include/linux/dev_printk.h:110:16: note: in definition of macro 'dev_printk_index_wrap'
110 | _p_func(dev, fmt, ##__VA_ARGS__); \
| ^~~
include/linux/dev_printk.h:150:51: note: in expansion of macro 'dev_fmt'
150 | dev_printk_index_wrap(_dev_info, KERN_INFO, dev, dev_fmt(fmt), ##__VA_ARGS__)
| ^~~~~~~
arch/x86/kernel/resource.c:51:4: note: in expansion of macro 'dev_info'
51 | dev_info(dev, "resource %pR fully covered by e820 entry
| ^~~~~~~~
arch/x86/kernel/resource.c:52:16: error: invalid suffix "Lx" on integer constant
52 | [mem %#010Lx-%#010Lx]\n",
| ^~~~~
include/linux/dev_printk.h:110:16: note: in definition of macro 'dev_printk_index_wrap'
110 | _p_func(dev, fmt, ##__VA_ARGS__); \
| ^~~
include/linux/dev_printk.h:150:51: note: in expansion of macro 'dev_fmt'
150 | dev_printk_index_wrap(_dev_info, KERN_INFO, dev, dev_fmt(fmt), ##__VA_ARGS__)
| ^~~~~~~
arch/x86/kernel/resource.c:51:4: note: in expansion of macro 'dev_info'
51 | dev_info(dev, "resource %pR fully covered by e820 entry
| ^~~~~~~~
arch/x86/kernel/resource.c:52:22: error: stray '\' in program
52 | [mem %#010Lx-%#010Lx]\n",
| ^
include/linux/dev_printk.h:110:16: note: in definition of macro 'dev_printk_index_wrap'
110 | _p_func(dev, fmt, ##__VA_ARGS__); \
| ^~~
include/linux/dev_printk.h:150:51: note: in expansion of macro 'dev_fmt'
150 | dev_printk_index_wrap(_dev_info, KERN_INFO, dev, dev_fmt(fmt), ##__VA_ARGS__)
| ^~~~~~~
arch/x86/kernel/resource.c:51:4: note: in expansion of macro 'dev_info'
51 | dev_info(dev, "resource %pR fully covered by e820 entry
| ^~~~~~~~
>> arch/x86/kernel/resource.c:52:23: error: expected ')' before 'n'
52 | [mem %#010Lx-%#010Lx]\n",
| ^
include/linux/dev_printk.h:110:16: note: in definition of macro 'dev_printk_index_wrap'
110 | _p_func(dev, fmt, ##__VA_ARGS__); \
| ^~~
include/linux/dev_printk.h:150:51: note: in expansion of macro 'dev_fmt'
150 | dev_printk_index_wrap(_dev_info, KERN_INFO, dev, dev_fmt(fmt), ##__VA_ARGS__)
| ^~~~~~~
arch/x86/kernel/resource.c:51:4: note: in expansion of macro 'dev_info'
51 | dev_info(dev, "resource %pR fully covered by e820 entry
| ^~~~~~~~
arch/x86/kernel/resource.c:52:24: error: missing terminating " character
52 | [mem %#010Lx-%#010Lx]\n",
| ^~
include/linux/dev_printk.h:110:16: note: in definition of macro 'dev_printk_index_wrap'
110 | _p_func(dev, fmt, ##__VA_ARGS__); \
| ^~~
include/linux/dev_printk.h:150:51: note: in expansion of macro 'dev_fmt'
150 | dev_printk_index_wrap(_dev_info, KERN_INFO, dev, dev_fmt(fmt), ##__VA_ARGS__)
| ^~~~~~~
arch/x86/kernel/resource.c:51:4: note: in expansion of macro 'dev_info'
51 | dev_info(dev, "resource %pR fully covered by e820 entry
| ^~~~~~~~
vim +51 arch/x86/kernel/resource.c
26
27 void remove_e820_regions(struct device *dev, struct resource *avail)
28 {
29 struct resource orig = *avail;
30 int i;
31 struct e820_entry *entry;
32 u64 e820_start, e820_end;
33
34 if (!(avail->flags & IORESOURCE_MEM))
35 return;
36
37 for (i = 0; i < e820_table->nr_entries; i++) {
38 entry = &e820_table->entries[i];
39 e820_start = entry->addr;
40 e820_end = entry->addr + entry->size - 1;
41
42 /*
43 * If an E820 entry covers just part of the resource, we
44 * assume E820 is telling us about something like host
45 * bridge register space that is unavailable for PCI
46 * devices. But if it covers the *entire* resource, it's
47 * more likely just telling us that this is MMIO space, and
48 * that doesn't need to be removed.
49 */
50 if (e820_start <= avail->start && avail->end <= e820_end) {
> 51 dev_info(dev, "resource %pR fully covered by e820 entry
> 52 [mem %#010Lx-%#010Lx]\n",
53 avail, e820_start, e820_end);
54
55 continue;
56 }
57
58 resource_clip(avail, e820_start, e820_end);
59 if (orig.start != avail->start || orig.end != avail->end) {
60 dev_info(dev, "clipped %pR to %pR for e820 entry [mem %#010Lx-%#010Lx]\n",
61 &orig, avail, e820_start, e820_end);
62 orig = *avail;
63 }
64 }
65 }
66
---
0-DAY CI Kernel Test Service
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
reply other threads:[~2022-03-10 7:13 UTC|newest]
Thread overview: [no followups] expand[flat|nested] mbox.gz Atom feed
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=202203101540.DAEpd9Mt-lkp@intel.com \
--to=lkp@intel.com \
--cc=hdegoede@redhat.com \
--cc=helgaas@kernel.org \
--cc=kbuild-all@lists.01.org \
--cc=linux-pci@vger.kernel.org \
--cc=mika.westerberg@linux.intel.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.