From: kernel test robot <lkp@intel.com>
To: kbuild-all@lists.01.org
Subject: Re: [PATCH v4 3/4] ACPI: APEI: Reserve the MCFG address for quirk ECAM implementation
Date: Fri, 29 Oct 2021 20:38:56 +0800 [thread overview]
Message-ID: <202110292036.bo3sAvRO-lkp@intel.com> (raw)
In-Reply-To: <20211027081312.53682-1-xuesong.chen@linux.alibaba.com>
[-- Attachment #1: Type: text/plain, Size: 3920 bytes --]
Hi Xuesong,
Thank you for the patch! Yet something to improve:
[auto build test ERROR on helgaas-pci/next]
[also build test ERROR on rafael-pm/linux-next tip/x86/core arm64/for-next/core v5.15-rc7 next-20211029]
[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]
url: https://github.com/0day-ci/linux/commits/Xuesong-Chen/PCI-MCFG-Consolidate-the-separate-PCI-MCFG-table-entry-list/20211027-171229
base: https://git.kernel.org/pub/scm/linux/kernel/git/helgaas/pci.git next
config: i386-allyesconfig (attached as .config)
compiler: gcc-9 (Debian 9.3.0-22) 9.3.0
reproduce (this is a W=1 build):
# https://github.com/0day-ci/linux/commit/8fce66d9da6f8e55c5cf0dda4a13dba6bd51492d
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Xuesong-Chen/PCI-MCFG-Consolidate-the-separate-PCI-MCFG-table-entry-list/20211027-171229
git checkout 8fce66d9da6f8e55c5cf0dda4a13dba6bd51492d
# save the attached .config to linux build tree
make W=1 ARCH=i386
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 >>):
>> drivers/acpi/apei/apei-base.c:453:5: error: no previous prototype for 'remove_quirk_mcfg_res' [-Werror=missing-prototypes]
453 | int remove_quirk_mcfg_res(struct apei_resources *mcfg_res)
| ^~~~~~~~~~~~~~~~~~~~~
drivers/acpi/apei/apei-base.c:804:12: error: no previous prototype for 'arch_apei_enable_cmcff' [-Werror=missing-prototypes]
804 | int __weak arch_apei_enable_cmcff(struct acpi_hest_header *hest_hdr,
| ^~~~~~~~~~~~~~~~~~~~~~
drivers/acpi/apei/apei-base.c:811:13: error: no previous prototype for 'arch_apei_report_mem_error' [-Werror=missing-prototypes]
811 | void __weak arch_apei_report_mem_error(int sev,
| ^~~~~~~~~~~~~~~~~~~~~~~~~~
cc1: all warnings being treated as errors
vim +/remove_quirk_mcfg_res +453 drivers/acpi/apei/apei-base.c
451
452 #ifdef CONFIG_PCI
> 453 int remove_quirk_mcfg_res(struct apei_resources *mcfg_res)
454 {
455 #ifdef CONFIG_PCI_QUIRKS
456 int rc = 0;
457 struct apei_resources quirk_res;
458
459 apei_resources_init(&quirk_res);
460 rc = apei_res_add(&quirk_res.iomem, pci_quirk_mcfg_res.start,
461 resource_size(&pci_quirk_mcfg_res));
462 if (rc)
463 return rc;
464
465 return apei_resources_sub(mcfg_res, &quirk_res);
466 #else
467 return 0;
468 #endif
469 }
470 extern struct list_head pci_mmcfg_list;
471 static int apei_filter_mcfg_addr(struct apei_resources *res,
472 struct apei_resources *mcfg_res)
473 {
474 int rc = 0;
475 struct pci_mmcfg_region *cfg;
476
477 if (list_empty(&pci_mmcfg_list))
478 return 0;
479
480 apei_resources_init(mcfg_res);
481 list_for_each_entry(cfg, &pci_mmcfg_list, list) {
482 rc = apei_res_add(&mcfg_res->iomem, cfg->res.start,
483 resource_size(&cfg->res));
484 if (rc)
485 return rc;
486 }
487
488 /* remove the pci quirk mcfg resource if any from the mcfg_res */
489 rc = remove_quirk_mcfg_res(mcfg_res);
490 if (rc)
491 return rc;
492
493 /* filter the mcfg resource from current APEI's */
494 return apei_resources_sub(res, mcfg_res);
495 }
496 #else
497 static inline int apei_filter_mcfg_addr(struct apei_resources *res,
498 struct apei_resources *mcfg_res)
499 {
500 return 0;
501 }
502 #endif
503
---
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: 66055 bytes --]
next prev parent reply other threads:[~2021-10-29 12:38 UTC|newest]
Thread overview: 38+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-10-19 4:49 [PATCH v3 0/2] PCI MCFG consolidation and APEI resource filterin Xuesong Chen
2021-10-19 4:49 ` Xuesong Chen
2021-10-19 15:12 ` Bjorn Helgaas
2021-10-19 15:12 ` Bjorn Helgaas
2021-10-20 2:28 ` Xuesong Chen
2021-10-20 2:28 ` Xuesong Chen
2021-10-27 8:10 ` [PATCH v4 0/4] PCI MCFG consolidation and APEI resource filtering Xuesong Chen
2021-10-27 8:10 ` Xuesong Chen
2021-10-27 8:12 ` [PATCH v4 1/4] PCI: MCFG: Consolidate the separate PCI MCFG table entry list Xuesong Chen
2021-10-27 8:12 ` Xuesong Chen
2021-10-27 8:12 ` [PATCH v4 2/4] ACPI: APEI: Filter the PCI MCFG address with an arch-agnostic method Xuesong Chen
2021-10-27 8:12 ` Xuesong Chen
2021-10-27 8:13 ` [PATCH v4 3/4] ACPI: APEI: Reserve the MCFG address for quirk ECAM implementation Xuesong Chen
2021-10-27 8:13 ` Xuesong Chen
2021-10-27 22:46 ` kernel test robot
2021-10-27 22:46 ` kernel test robot
2021-10-27 22:46 ` kernel test robot
2021-11-01 17:15 ` Bjorn Helgaas
2021-11-01 17:15 ` Bjorn Helgaas
2021-11-04 2:30 ` [kbuild-all] " Rong Chen
2021-11-04 2:30 ` Rong Chen
2021-10-27 22:46 ` [RFC PATCH] ACPI: APEI: remove_quirk_mcfg_res() can be static kernel test robot
2021-10-28 4:13 ` [PATCH v4 3/4] ACPI: APEI: Reserve the MCFG address for quirk ECAM implementation kernel test robot
2021-10-29 12:38 ` kernel test robot [this message]
2021-10-27 8:13 ` [PATCH v4 4/4] PCI: MCFG: Add the MCFG entry parse log message Xuesong Chen
2021-10-27 8:13 ` Xuesong Chen
2021-11-01 2:18 ` [PATCH v4 0/4] PCI MCFG consolidation and APEI resource filtering Xuesong Chen
2021-11-01 2:18 ` Xuesong Chen
2021-11-01 9:36 ` Will Deacon
2021-11-01 9:36 ` Will Deacon
2021-11-01 12:12 ` Xuesong Chen
2021-11-01 12:12 ` Xuesong Chen
2021-11-01 12:22 ` Borislav Petkov
2021-11-01 12:22 ` Borislav Petkov
2021-11-01 13:32 ` Xuesong Chen
2021-11-01 13:32 ` Xuesong Chen
2021-11-01 13:55 ` Borislav Petkov
2021-11-01 13:55 ` Borislav Petkov
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=202110292036.bo3sAvRO-lkp@intel.com \
--to=lkp@intel.com \
--cc=kbuild-all@lists.01.org \
/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.