From: kbuild test robot <lkp@intel.com>
To: Matthew Garrett <matthewgarrett@google.com>
Cc: kbuild-all@lists.01.org, linux-efi@vger.kernel.org,
ard.biesheuvel@linaro.org, x86@kernel.org,
linux-pci@vger.kernel.org, linux-kernel@vger.kernel.org,
Matthew Garrett <matthewgarrett@google.com>,
Matthew Garrett <mjg59@google.com>
Subject: Re: [PATCH] [EFI,PCI] Allow disabling PCI busmastering on bridges during boot
Date: Thu, 5 Dec 2019 21:04:59 +0800 [thread overview]
Message-ID: <201912052123.ASO2WGsj%lkp@intel.com> (raw)
In-Reply-To: <20191203004043.174977-1-matthewgarrett@google.com>
[-- Attachment #1: Type: text/plain, Size: 6663 bytes --]
Hi Matthew,
I love your patch! Yet something to improve:
[auto build test ERROR on v5.4-rc8]
[also build test ERROR on linux/master]
[cannot apply to efi/next linus/master next-20191203]
[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/Matthew-Garrett/Allow-disabling-PCI-busmastering-on-bridges-during-boot/20191203-084747
base: af42d3466bdc8f39806b26f593604fdc54140bcb
config: arm-multi_v7_defconfig (attached as .config)
compiler: arm-linux-gnueabi-gcc (GCC) 7.5.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=7.5.0 make.cross ARCH=arm
If you fix the issue, kindly add following tag
Reported-by: kbuild test robot <lkp@intel.com>
All errors (new ones prefixed by >>):
In file included from drivers/firmware/efi/libstub/pci.c:12:0:
drivers/firmware/efi/libstub/pci.c: In function 'efi_pci_disable_bridge_busmaster':
>> arch/arm/include/asm/efi.h:53:33: error: 'sys_table_arg' undeclared (first use in this function); did you mean 'sys_table'?
#define efi_call_early(f, ...) sys_table_arg->boottime->f(__VA_ARGS__)
^
drivers/firmware/efi/libstub/pci.c:29:11: note: in expansion of macro 'efi_call_early'
status = efi_call_early(locate_handle,
^~~~~~~~~~~~~~
arch/arm/include/asm/efi.h:53:33: note: each undeclared identifier is reported only once for each function it appears in
#define efi_call_early(f, ...) sys_table_arg->boottime->f(__VA_ARGS__)
^
drivers/firmware/efi/libstub/pci.c:29:11: note: in expansion of macro 'efi_call_early'
status = efi_call_early(locate_handle,
^~~~~~~~~~~~~~
>> arch/arm/include/asm/efi.h:62:3: error: called object is not a function or function pointer
((protocol##_t *)instance)->f(instance, ##__VA_ARGS__)
~^~~~~~~~~~~~~~~~~~~~~~~~~
drivers/firmware/efi/libstub/pci.c:62:12: note: in expansion of macro 'efi_call_proto'
status = efi_call_proto(efi_pci_io_protocol, pci.read, pci,
^~~~~~~~~~~~~~
>> arch/arm/include/asm/efi.h:62:3: error: called object is not a function or function pointer
((protocol##_t *)instance)->f(instance, ##__VA_ARGS__)
~^~~~~~~~~~~~~~~~~~~~~~~~~
drivers/firmware/efi/libstub/pci.c:70:12: note: in expansion of macro 'efi_call_proto'
status = efi_call_proto(efi_pci_io_protocol, pci.read, pci,
^~~~~~~~~~~~~~
>> arch/arm/include/asm/efi.h:62:3: error: called object is not a function or function pointer
((protocol##_t *)instance)->f(instance, ##__VA_ARGS__)
~^~~~~~~~~~~~~~~~~~~~~~~~~
drivers/firmware/efi/libstub/pci.c:78:12: note: in expansion of macro 'efi_call_proto'
status = efi_call_proto(efi_pci_io_protocol, pci.write, pci,
^~~~~~~~~~~~~~
--
In file included from drivers/firmware//efi/libstub/pci.c:12:0:
drivers/firmware//efi/libstub/pci.c: In function 'efi_pci_disable_bridge_busmaster':
>> arch/arm/include/asm/efi.h:53:33: error: 'sys_table_arg' undeclared (first use in this function); did you mean 'sys_table'?
#define efi_call_early(f, ...) sys_table_arg->boottime->f(__VA_ARGS__)
^
drivers/firmware//efi/libstub/pci.c:29:11: note: in expansion of macro 'efi_call_early'
status = efi_call_early(locate_handle,
^~~~~~~~~~~~~~
arch/arm/include/asm/efi.h:53:33: note: each undeclared identifier is reported only once for each function it appears in
#define efi_call_early(f, ...) sys_table_arg->boottime->f(__VA_ARGS__)
^
drivers/firmware//efi/libstub/pci.c:29:11: note: in expansion of macro 'efi_call_early'
status = efi_call_early(locate_handle,
^~~~~~~~~~~~~~
>> arch/arm/include/asm/efi.h:62:3: error: called object is not a function or function pointer
((protocol##_t *)instance)->f(instance, ##__VA_ARGS__)
~^~~~~~~~~~~~~~~~~~~~~~~~~
drivers/firmware//efi/libstub/pci.c:62:12: note: in expansion of macro 'efi_call_proto'
status = efi_call_proto(efi_pci_io_protocol, pci.read, pci,
^~~~~~~~~~~~~~
>> arch/arm/include/asm/efi.h:62:3: error: called object is not a function or function pointer
((protocol##_t *)instance)->f(instance, ##__VA_ARGS__)
~^~~~~~~~~~~~~~~~~~~~~~~~~
drivers/firmware//efi/libstub/pci.c:70:12: note: in expansion of macro 'efi_call_proto'
status = efi_call_proto(efi_pci_io_protocol, pci.read, pci,
^~~~~~~~~~~~~~
>> arch/arm/include/asm/efi.h:62:3: error: called object is not a function or function pointer
((protocol##_t *)instance)->f(instance, ##__VA_ARGS__)
~^~~~~~~~~~~~~~~~~~~~~~~~~
drivers/firmware//efi/libstub/pci.c:78:12: note: in expansion of macro 'efi_call_proto'
status = efi_call_proto(efi_pci_io_protocol, pci.write, pci,
^~~~~~~~~~~~~~
vim +53 arch/arm/include/asm/efi.h
81a0bc39ea1960 Roy Franz 2015-09-23 52
81a0bc39ea1960 Roy Franz 2015-09-23 @53 #define efi_call_early(f, ...) sys_table_arg->boottime->f(__VA_ARGS__)
fc37206427ce38 Ard Biesheuvel 2016-04-25 54 #define __efi_call_early(f, ...) f(__VA_ARGS__)
6d0ca4a47bf8cb David Howells 2017-02-06 55 #define efi_call_runtime(f, ...) sys_table_arg->runtime->f(__VA_ARGS__)
fc37206427ce38 Ard Biesheuvel 2016-04-25 56 #define efi_is_64bit() (false)
81a0bc39ea1960 Roy Franz 2015-09-23 57
c4db9c1e8c70bc Lukas Wunner 2018-07-20 58 #define efi_table_attr(table, attr, instance) \
c4db9c1e8c70bc Lukas Wunner 2018-07-20 59 ((table##_t *)instance)->attr
c4db9c1e8c70bc Lukas Wunner 2018-07-20 60
3552fdf29f01e5 Lukas Wunner 2016-11-12 61 #define efi_call_proto(protocol, f, instance, ...) \
3552fdf29f01e5 Lukas Wunner 2016-11-12 @62 ((protocol##_t *)instance)->f(instance, ##__VA_ARGS__)
3552fdf29f01e5 Lukas Wunner 2016-11-12 63
:::::: The code at line 53 was first introduced by commit
:::::: 81a0bc39ea1960bbf8ece6a895d7cfd2d9efa28a ARM: add UEFI stub support
:::::: TO: Roy Franz <roy.franz@linaro.org>
:::::: CC: ard <ard.biesheuvel@linaro.org>
---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org Intel Corporation
[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 48397 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] [EFI, PCI] Allow disabling PCI busmastering on bridges during boot
Date: Thu, 05 Dec 2019 21:04:59 +0800 [thread overview]
Message-ID: <201912052123.ASO2WGsj%lkp@intel.com> (raw)
In-Reply-To: <20191203004043.174977-1-matthewgarrett@google.com>
[-- Attachment #1: Type: text/plain, Size: 6781 bytes --]
Hi Matthew,
I love your patch! Yet something to improve:
[auto build test ERROR on v5.4-rc8]
[also build test ERROR on linux/master]
[cannot apply to efi/next linus/master next-20191203]
[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/Matthew-Garrett/Allow-disabling-PCI-busmastering-on-bridges-during-boot/20191203-084747
base: af42d3466bdc8f39806b26f593604fdc54140bcb
config: arm-multi_v7_defconfig (attached as .config)
compiler: arm-linux-gnueabi-gcc (GCC) 7.5.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=7.5.0 make.cross ARCH=arm
If you fix the issue, kindly add following tag
Reported-by: kbuild test robot <lkp@intel.com>
All errors (new ones prefixed by >>):
In file included from drivers/firmware/efi/libstub/pci.c:12:0:
drivers/firmware/efi/libstub/pci.c: In function 'efi_pci_disable_bridge_busmaster':
>> arch/arm/include/asm/efi.h:53:33: error: 'sys_table_arg' undeclared (first use in this function); did you mean 'sys_table'?
#define efi_call_early(f, ...) sys_table_arg->boottime->f(__VA_ARGS__)
^
drivers/firmware/efi/libstub/pci.c:29:11: note: in expansion of macro 'efi_call_early'
status = efi_call_early(locate_handle,
^~~~~~~~~~~~~~
arch/arm/include/asm/efi.h:53:33: note: each undeclared identifier is reported only once for each function it appears in
#define efi_call_early(f, ...) sys_table_arg->boottime->f(__VA_ARGS__)
^
drivers/firmware/efi/libstub/pci.c:29:11: note: in expansion of macro 'efi_call_early'
status = efi_call_early(locate_handle,
^~~~~~~~~~~~~~
>> arch/arm/include/asm/efi.h:62:3: error: called object is not a function or function pointer
((protocol##_t *)instance)->f(instance, ##__VA_ARGS__)
~^~~~~~~~~~~~~~~~~~~~~~~~~
drivers/firmware/efi/libstub/pci.c:62:12: note: in expansion of macro 'efi_call_proto'
status = efi_call_proto(efi_pci_io_protocol, pci.read, pci,
^~~~~~~~~~~~~~
>> arch/arm/include/asm/efi.h:62:3: error: called object is not a function or function pointer
((protocol##_t *)instance)->f(instance, ##__VA_ARGS__)
~^~~~~~~~~~~~~~~~~~~~~~~~~
drivers/firmware/efi/libstub/pci.c:70:12: note: in expansion of macro 'efi_call_proto'
status = efi_call_proto(efi_pci_io_protocol, pci.read, pci,
^~~~~~~~~~~~~~
>> arch/arm/include/asm/efi.h:62:3: error: called object is not a function or function pointer
((protocol##_t *)instance)->f(instance, ##__VA_ARGS__)
~^~~~~~~~~~~~~~~~~~~~~~~~~
drivers/firmware/efi/libstub/pci.c:78:12: note: in expansion of macro 'efi_call_proto'
status = efi_call_proto(efi_pci_io_protocol, pci.write, pci,
^~~~~~~~~~~~~~
--
In file included from drivers/firmware//efi/libstub/pci.c:12:0:
drivers/firmware//efi/libstub/pci.c: In function 'efi_pci_disable_bridge_busmaster':
>> arch/arm/include/asm/efi.h:53:33: error: 'sys_table_arg' undeclared (first use in this function); did you mean 'sys_table'?
#define efi_call_early(f, ...) sys_table_arg->boottime->f(__VA_ARGS__)
^
drivers/firmware//efi/libstub/pci.c:29:11: note: in expansion of macro 'efi_call_early'
status = efi_call_early(locate_handle,
^~~~~~~~~~~~~~
arch/arm/include/asm/efi.h:53:33: note: each undeclared identifier is reported only once for each function it appears in
#define efi_call_early(f, ...) sys_table_arg->boottime->f(__VA_ARGS__)
^
drivers/firmware//efi/libstub/pci.c:29:11: note: in expansion of macro 'efi_call_early'
status = efi_call_early(locate_handle,
^~~~~~~~~~~~~~
>> arch/arm/include/asm/efi.h:62:3: error: called object is not a function or function pointer
((protocol##_t *)instance)->f(instance, ##__VA_ARGS__)
~^~~~~~~~~~~~~~~~~~~~~~~~~
drivers/firmware//efi/libstub/pci.c:62:12: note: in expansion of macro 'efi_call_proto'
status = efi_call_proto(efi_pci_io_protocol, pci.read, pci,
^~~~~~~~~~~~~~
>> arch/arm/include/asm/efi.h:62:3: error: called object is not a function or function pointer
((protocol##_t *)instance)->f(instance, ##__VA_ARGS__)
~^~~~~~~~~~~~~~~~~~~~~~~~~
drivers/firmware//efi/libstub/pci.c:70:12: note: in expansion of macro 'efi_call_proto'
status = efi_call_proto(efi_pci_io_protocol, pci.read, pci,
^~~~~~~~~~~~~~
>> arch/arm/include/asm/efi.h:62:3: error: called object is not a function or function pointer
((protocol##_t *)instance)->f(instance, ##__VA_ARGS__)
~^~~~~~~~~~~~~~~~~~~~~~~~~
drivers/firmware//efi/libstub/pci.c:78:12: note: in expansion of macro 'efi_call_proto'
status = efi_call_proto(efi_pci_io_protocol, pci.write, pci,
^~~~~~~~~~~~~~
vim +53 arch/arm/include/asm/efi.h
81a0bc39ea1960 Roy Franz 2015-09-23 52
81a0bc39ea1960 Roy Franz 2015-09-23 @53 #define efi_call_early(f, ...) sys_table_arg->boottime->f(__VA_ARGS__)
fc37206427ce38 Ard Biesheuvel 2016-04-25 54 #define __efi_call_early(f, ...) f(__VA_ARGS__)
6d0ca4a47bf8cb David Howells 2017-02-06 55 #define efi_call_runtime(f, ...) sys_table_arg->runtime->f(__VA_ARGS__)
fc37206427ce38 Ard Biesheuvel 2016-04-25 56 #define efi_is_64bit() (false)
81a0bc39ea1960 Roy Franz 2015-09-23 57
c4db9c1e8c70bc Lukas Wunner 2018-07-20 58 #define efi_table_attr(table, attr, instance) \
c4db9c1e8c70bc Lukas Wunner 2018-07-20 59 ((table##_t *)instance)->attr
c4db9c1e8c70bc Lukas Wunner 2018-07-20 60
3552fdf29f01e5 Lukas Wunner 2016-11-12 61 #define efi_call_proto(protocol, f, instance, ...) \
3552fdf29f01e5 Lukas Wunner 2016-11-12 @62 ((protocol##_t *)instance)->f(instance, ##__VA_ARGS__)
3552fdf29f01e5 Lukas Wunner 2016-11-12 63
:::::: The code at line 53 was first introduced by commit
:::::: 81a0bc39ea1960bbf8ece6a895d7cfd2d9efa28a ARM: add UEFI stub support
:::::: TO: Roy Franz <roy.franz@linaro.org>
:::::: CC: ard <ard.biesheuvel@linaro.org>
---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org Intel Corporation
[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 48397 bytes --]
next prev parent reply other threads:[~2019-12-05 13:05 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-12-03 0:40 [PATCH] [EFI,PCI] Allow disabling PCI busmastering on bridges during boot Matthew Garrett
2019-12-03 0:42 ` Matthew Garrett
2019-12-03 11:54 ` Ard Biesheuvel
2019-12-03 13:38 ` Laszlo Ersek
2019-12-03 19:36 ` Matthew Garrett
2019-12-03 19:40 ` Matthew Garrett
2019-12-04 7:11 ` Laszlo Ersek
2019-12-04 19:29 ` Matthew Garrett
2019-12-03 15:30 ` Andy Lutomirski
2019-12-03 16:33 ` Ard Biesheuvel
2019-12-03 19:41 ` Matthew Garrett
2019-12-04 19:50 ` Andy Lutomirski
2019-12-04 19:56 ` Matthew Garrett
2019-12-12 15:46 ` Ard Biesheuvel
2019-12-13 21:24 ` Matthew Garrett
2019-12-03 18:23 ` kbuild test robot
2019-12-03 18:23 ` [PATCH] [EFI, PCI] " kbuild test robot
2019-12-05 13:04 ` kbuild test robot [this message]
2019-12-05 13:04 ` kbuild test robot
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=201912052123.ASO2WGsj%lkp@intel.com \
--to=lkp@intel.com \
--cc=ard.biesheuvel@linaro.org \
--cc=kbuild-all@lists.01.org \
--cc=linux-efi@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pci@vger.kernel.org \
--cc=matthewgarrett@google.com \
--cc=mjg59@google.com \
--cc=x86@kernel.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.