All of lore.kernel.org
 help / color / mirror / Atom feed
From: kbuild test robot <lkp@intel.com>
To: Matt Fleming <matt@codeblueprint.co.uk>
Cc: kbuild-all@01.org, linux-efi@vger.kernel.org,
	linux-kernel@vger.kernel.org,
	Ard Biesheuvel <ard.biesheuvel@linaro.org>,
	Tony Luck <tony.luck@gmail.com>,
	Matt Fleming <matt@codeblueprint.co.uk>,
	Mark Salter <msalter@redhat.com>,
	Leif Lindholm <leif.lindholm@linaro.org>
Subject: Re: [PATCH 1/2] efi: Iterate over efi->memmap in for_each_efi_memory_desc
Date: Mon, 11 Apr 2016 21:24:03 +0800	[thread overview]
Message-ID: <201604112150.cvcV2wEO%fengguang.wu@intel.com> (raw)
In-Reply-To: <1460379809-18490-2-git-send-email-matt@codeblueprint.co.uk>

[-- Attachment #1: Type: text/plain, Size: 19828 bytes --]

Hi Matt,

[auto build test ERROR on efi/next]
[cannot apply to v4.6-rc3 next-20160411]
[if your patch is applied to the wrong git tree, please drop us a note to help improving the system]

url:    https://github.com/0day-ci/linux/commits/Matt-Fleming/efi-Delete-global-memmap-variable/20160411-210837
base:   https://git.kernel.org/pub/scm/linux/kernel/git/mfleming/efi.git next
config: x86_64-allyesconfig (attached as .config)
reproduce:
        # save the attached .config to linux build tree
        make ARCH=x86_64 

Note: the linux-review/Matt-Fleming/efi-Delete-global-memmap-variable/20160411-210837 HEAD 8fc82e8aa63ce89d26d143cce41530319212f7d8 builds fine.
      It only hurts bisectibility.

All error/warnings (new ones prefixed by >>):

   In file included from arch/x86/platform/efi/quirks.c:8:0:
   arch/x86/platform/efi/quirks.c: In function 'efi_reserve_boot_services':
   include/linux/efi.h:973:37: error: invalid type argument of '->' (have 'struct efi')
     for_each_efi_memory_desc_in_map(efi->memmap, md)
                                        ^
   include/linux/efi.h:962:15: note: in definition of macro 'for_each_efi_memory_desc_in_map'
     for ((md) = (m)->map;         \
                  ^
>> arch/x86/platform/efi/quirks.c:200:2: note: in expansion of macro 'for_each_efi_memory_desc'
     for_each_efi_memory_desc(md) {
     ^
   include/linux/efi.h:973:37: error: invalid type argument of '->' (have 'struct efi')
     for_each_efi_memory_desc_in_map(efi->memmap, md)
                                        ^
   include/linux/efi.h:963:38: note: in definition of macro 'for_each_efi_memory_desc_in_map'
          (md) <= (efi_memory_desc_t *)((m)->map_end - (m)->desc_size); \
                                         ^
>> arch/x86/platform/efi/quirks.c:200:2: note: in expansion of macro 'for_each_efi_memory_desc'
     for_each_efi_memory_desc(md) {
     ^
   include/linux/efi.h:973:37: error: invalid type argument of '->' (have 'struct efi')
     for_each_efi_memory_desc_in_map(efi->memmap, md)
                                        ^
   include/linux/efi.h:963:53: note: in definition of macro 'for_each_efi_memory_desc_in_map'
          (md) <= (efi_memory_desc_t *)((m)->map_end - (m)->desc_size); \
                                                        ^
>> arch/x86/platform/efi/quirks.c:200:2: note: in expansion of macro 'for_each_efi_memory_desc'
     for_each_efi_memory_desc(md) {
     ^
   include/linux/efi.h:973:37: error: invalid type argument of '->' (have 'struct efi')
     for_each_efi_memory_desc_in_map(efi->memmap, md)
                                        ^
   include/linux/efi.h:964:30: note: in definition of macro 'for_each_efi_memory_desc_in_map'
          (md) = (void *)(md) + (m)->desc_size)
                                 ^
>> arch/x86/platform/efi/quirks.c:200:2: note: in expansion of macro 'for_each_efi_memory_desc'
     for_each_efi_memory_desc(md) {
     ^
   arch/x86/platform/efi/quirks.c: In function 'efi_free_boot_services':
   include/linux/efi.h:973:37: error: invalid type argument of '->' (have 'struct efi')
     for_each_efi_memory_desc_in_map(efi->memmap, md)
                                        ^
   include/linux/efi.h:962:15: note: in definition of macro 'for_each_efi_memory_desc_in_map'
     for ((md) = (m)->map;         \
                  ^
   arch/x86/platform/efi/quirks.c:254:2: note: in expansion of macro 'for_each_efi_memory_desc'
     for_each_efi_memory_desc(md) {
     ^
   include/linux/efi.h:973:37: error: invalid type argument of '->' (have 'struct efi')
     for_each_efi_memory_desc_in_map(efi->memmap, md)
                                        ^
   include/linux/efi.h:963:38: note: in definition of macro 'for_each_efi_memory_desc_in_map'
          (md) <= (efi_memory_desc_t *)((m)->map_end - (m)->desc_size); \
                                         ^
   arch/x86/platform/efi/quirks.c:254:2: note: in expansion of macro 'for_each_efi_memory_desc'
     for_each_efi_memory_desc(md) {
     ^
   include/linux/efi.h:973:37: error: invalid type argument of '->' (have 'struct efi')
     for_each_efi_memory_desc_in_map(efi->memmap, md)
                                        ^
   include/linux/efi.h:963:53: note: in definition of macro 'for_each_efi_memory_desc_in_map'
          (md) <= (efi_memory_desc_t *)((m)->map_end - (m)->desc_size); \
                                                        ^
   arch/x86/platform/efi/quirks.c:254:2: note: in expansion of macro 'for_each_efi_memory_desc'
     for_each_efi_memory_desc(md) {
     ^
   include/linux/efi.h:973:37: error: invalid type argument of '->' (have 'struct efi')
     for_each_efi_memory_desc_in_map(efi->memmap, md)
                                        ^
   include/linux/efi.h:964:30: note: in definition of macro 'for_each_efi_memory_desc_in_map'
          (md) = (void *)(md) + (m)->desc_size)
                                 ^
   arch/x86/platform/efi/quirks.c:254:2: note: in expansion of macro 'for_each_efi_memory_desc'
     for_each_efi_memory_desc(md) {
     ^
--
   In file included from arch/x86/platform/efi/efi.c:35:0:
   arch/x86/platform/efi/efi.c: In function 'efi_find_mirror':
   include/linux/efi.h:973:37: error: invalid type argument of '->' (have 'struct efi')
     for_each_efi_memory_desc_in_map(efi->memmap, md)
                                        ^
   include/linux/efi.h:962:15: note: in definition of macro 'for_each_efi_memory_desc_in_map'
     for ((md) = (m)->map;         \
                  ^
>> arch/x86/platform/efi/efi.c:125:2: note: in expansion of macro 'for_each_efi_memory_desc'
     for_each_efi_memory_desc(md) {
     ^
   include/linux/efi.h:973:37: error: invalid type argument of '->' (have 'struct efi')
     for_each_efi_memory_desc_in_map(efi->memmap, md)
                                        ^
   include/linux/efi.h:963:38: note: in definition of macro 'for_each_efi_memory_desc_in_map'
          (md) <= (efi_memory_desc_t *)((m)->map_end - (m)->desc_size); \
                                         ^
>> arch/x86/platform/efi/efi.c:125:2: note: in expansion of macro 'for_each_efi_memory_desc'
     for_each_efi_memory_desc(md) {
     ^
   include/linux/efi.h:973:37: error: invalid type argument of '->' (have 'struct efi')
     for_each_efi_memory_desc_in_map(efi->memmap, md)
                                        ^
   include/linux/efi.h:963:53: note: in definition of macro 'for_each_efi_memory_desc_in_map'
          (md) <= (efi_memory_desc_t *)((m)->map_end - (m)->desc_size); \
                                                        ^
>> arch/x86/platform/efi/efi.c:125:2: note: in expansion of macro 'for_each_efi_memory_desc'
     for_each_efi_memory_desc(md) {
     ^
   include/linux/efi.h:973:37: error: invalid type argument of '->' (have 'struct efi')
     for_each_efi_memory_desc_in_map(efi->memmap, md)
                                        ^
   include/linux/efi.h:964:30: note: in definition of macro 'for_each_efi_memory_desc_in_map'
          (md) = (void *)(md) + (m)->desc_size)
                                 ^
>> arch/x86/platform/efi/efi.c:125:2: note: in expansion of macro 'for_each_efi_memory_desc'
     for_each_efi_memory_desc(md) {
     ^
   arch/x86/platform/efi/efi.c: In function 'do_add_efi_memmap':
   include/linux/efi.h:973:37: error: invalid type argument of '->' (have 'struct efi')
     for_each_efi_memory_desc_in_map(efi->memmap, md)
                                        ^
   include/linux/efi.h:962:15: note: in definition of macro 'for_each_efi_memory_desc_in_map'
     for ((md) = (m)->map;         \
                  ^
   arch/x86/platform/efi/efi.c:150:2: note: in expansion of macro 'for_each_efi_memory_desc'
     for_each_efi_memory_desc(md) {
     ^
   include/linux/efi.h:973:37: error: invalid type argument of '->' (have 'struct efi')
     for_each_efi_memory_desc_in_map(efi->memmap, md)
                                        ^
   include/linux/efi.h:963:38: note: in definition of macro 'for_each_efi_memory_desc_in_map'
          (md) <= (efi_memory_desc_t *)((m)->map_end - (m)->desc_size); \
                                         ^
   arch/x86/platform/efi/efi.c:150:2: note: in expansion of macro 'for_each_efi_memory_desc'
     for_each_efi_memory_desc(md) {
     ^
   include/linux/efi.h:973:37: error: invalid type argument of '->' (have 'struct efi')
     for_each_efi_memory_desc_in_map(efi->memmap, md)
                                        ^
   include/linux/efi.h:963:53: note: in definition of macro 'for_each_efi_memory_desc_in_map'
          (md) <= (efi_memory_desc_t *)((m)->map_end - (m)->desc_size); \
                                                        ^
   arch/x86/platform/efi/efi.c:150:2: note: in expansion of macro 'for_each_efi_memory_desc'
     for_each_efi_memory_desc(md) {
     ^
   include/linux/efi.h:973:37: error: invalid type argument of '->' (have 'struct efi')
     for_each_efi_memory_desc_in_map(efi->memmap, md)
                                        ^
   include/linux/efi.h:964:30: note: in definition of macro 'for_each_efi_memory_desc_in_map'
          (md) = (void *)(md) + (m)->desc_size)
                                 ^
   arch/x86/platform/efi/efi.c:150:2: note: in expansion of macro 'for_each_efi_memory_desc'
     for_each_efi_memory_desc(md) {
     ^
   arch/x86/platform/efi/efi.c: In function 'efi_print_memmap':
   include/linux/efi.h:973:37: error: invalid type argument of '->' (have 'struct efi')
     for_each_efi_memory_desc_in_map(efi->memmap, md)
                                        ^
   include/linux/efi.h:962:15: note: in definition of macro 'for_each_efi_memory_desc_in_map'
     for ((md) = (m)->map;         \
                  ^
   arch/x86/platform/efi/efi.c:229:2: note: in expansion of macro 'for_each_efi_memory_desc'
     for_each_efi_memory_desc(md) {
     ^
   include/linux/efi.h:973:37: error: invalid type argument of '->' (have 'struct efi')
     for_each_efi_memory_desc_in_map(efi->memmap, md)
                                        ^
   include/linux/efi.h:963:38: note: in definition of macro 'for_each_efi_memory_desc_in_map'
          (md) <= (efi_memory_desc_t *)((m)->map_end - (m)->desc_size); \
                                         ^
   arch/x86/platform/efi/efi.c:229:2: note: in expansion of macro 'for_each_efi_memory_desc'
     for_each_efi_memory_desc(md) {
     ^
   include/linux/efi.h:973:37: error: invalid type argument of '->' (have 'struct efi')
     for_each_efi_memory_desc_in_map(efi->memmap, md)
                                        ^
   include/linux/efi.h:963:53: note: in definition of macro 'for_each_efi_memory_desc_in_map'
          (md) <= (efi_memory_desc_t *)((m)->map_end - (m)->desc_size); \
                                                        ^
   arch/x86/platform/efi/efi.c:229:2: note: in expansion of macro 'for_each_efi_memory_desc'
     for_each_efi_memory_desc(md) {
     ^
   include/linux/efi.h:973:37: error: invalid type argument of '->' (have 'struct efi')
     for_each_efi_memory_desc_in_map(efi->memmap, md)
                                        ^
   include/linux/efi.h:964:30: note: in definition of macro 'for_each_efi_memory_desc_in_map'
          (md) = (void *)(md) + (m)->desc_size)
                                 ^
   arch/x86/platform/efi/efi.c:229:2: note: in expansion of macro 'for_each_efi_memory_desc'
     for_each_efi_memory_desc(md) {
     ^
   arch/x86/platform/efi/efi.c: In function 'runtime_code_page_mkexec':
   include/linux/efi.h:973:37: error: invalid type argument of '->' (have 'struct efi')
     for_each_efi_memory_desc_in_map(efi->memmap, md)
                                        ^
   include/linux/efi.h:962:15: note: in definition of macro 'for_each_efi_memory_desc_in_map'
     for ((md) = (m)->map;         \
                  ^
   arch/x86/platform/efi/efi.c:550:2: note: in expansion of macro 'for_each_efi_memory_desc'
     for_each_efi_memory_desc(md) {
     ^
   include/linux/efi.h:973:37: error: invalid type argument of '->' (have 'struct efi')
     for_each_efi_memory_desc_in_map(efi->memmap, md)
                                        ^
   include/linux/efi.h:963:38: note: in definition of macro 'for_each_efi_memory_desc_in_map'
          (md) <= (efi_memory_desc_t *)((m)->map_end - (m)->desc_size); \
                                         ^
   arch/x86/platform/efi/efi.c:550:2: note: in expansion of macro 'for_each_efi_memory_desc'
     for_each_efi_memory_desc(md) {
     ^
   include/linux/efi.h:973:37: error: invalid type argument of '->' (have 'struct efi')
     for_each_efi_memory_desc_in_map(efi->memmap, md)
                                        ^
   include/linux/efi.h:963:53: note: in definition of macro 'for_each_efi_memory_desc_in_map'
          (md) <= (efi_memory_desc_t *)((m)->map_end - (m)->desc_size); \
                                                        ^
   arch/x86/platform/efi/efi.c:550:2: note: in expansion of macro 'for_each_efi_memory_desc'
     for_each_efi_memory_desc(md) {
     ^
   include/linux/efi.h:973:37: error: invalid type argument of '->' (have 'struct efi')
     for_each_efi_memory_desc_in_map(efi->memmap, md)
                                        ^
   include/linux/efi.h:964:30: note: in definition of macro 'for_each_efi_memory_desc_in_map'
          (md) = (void *)(md) + (m)->desc_size)
                                 ^
   arch/x86/platform/efi/efi.c:550:2: note: in expansion of macro 'for_each_efi_memory_desc'
     for_each_efi_memory_desc(md) {
     ^
>> arch/x86/platform/efi/efi.c:547:8: warning: unused variable 'p' [-Wunused-variable]
     void *p;
           ^
   In file included from arch/x86/platform/efi/efi.c:35:0:
   arch/x86/platform/efi/efi.c: In function 'efi_merge_regions':
   include/linux/efi.h:973:37: error: invalid type argument of '->' (have 'struct efi')
     for_each_efi_memory_desc_in_map(efi->memmap, md)
                                        ^
   include/linux/efi.h:962:15: note: in definition of macro 'for_each_efi_memory_desc_in_map'
     for ((md) = (m)->map;         \
                  ^
   arch/x86/platform/efi/efi.c:600:2: note: in expansion of macro 'for_each_efi_memory_desc'
     for_each_efi_memory_desc(md) {
     ^
   include/linux/efi.h:973:37: error: invalid type argument of '->' (have 'struct efi')
     for_each_efi_memory_desc_in_map(efi->memmap, md)
                                        ^
   include/linux/efi.h:963:38: note: in definition of macro 'for_each_efi_memory_desc_in_map'
          (md) <= (efi_memory_desc_t *)((m)->map_end - (m)->desc_size); \
                                         ^
   arch/x86/platform/efi/efi.c:600:2: note: in expansion of macro 'for_each_efi_memory_desc'
     for_each_efi_memory_desc(md) {
     ^
   include/linux/efi.h:973:37: error: invalid type argument of '->' (have 'struct efi')
     for_each_efi_memory_desc_in_map(efi->memmap, md)
                                        ^
   include/linux/efi.h:963:53: note: in definition of macro 'for_each_efi_memory_desc_in_map'
          (md) <= (efi_memory_desc_t *)((m)->map_end - (m)->desc_size); \
                                                        ^
   arch/x86/platform/efi/efi.c:600:2: note: in expansion of macro 'for_each_efi_memory_desc'
     for_each_efi_memory_desc(md) {
     ^
   include/linux/efi.h:973:37: error: invalid type argument of '->' (have 'struct efi')
     for_each_efi_memory_desc_in_map(efi->memmap, md)
                                        ^
   include/linux/efi.h:964:30: note: in definition of macro 'for_each_efi_memory_desc_in_map'
          (md) = (void *)(md) + (m)->desc_size)
                                 ^
   arch/x86/platform/efi/efi.c:600:2: note: in expansion of macro 'for_each_efi_memory_desc'
     for_each_efi_memory_desc(md) {
     ^
   arch/x86/platform/efi/efi.c:597:8: warning: unused variable 'p' [-Wunused-variable]
     void *p;
           ^
   In file included from arch/x86/platform/efi/efi.c:35:0:
   arch/x86/platform/efi/efi.c: In function 'save_runtime_map':
   include/linux/efi.h:973:37: error: invalid type argument of '->' (have 'struct efi')
     for_each_efi_memory_desc_in_map(efi->memmap, md)
                                        ^
   include/linux/efi.h:962:15: note: in definition of macro 'for_each_efi_memory_desc_in_map'
     for ((md) = (m)->map;         \
                  ^
   arch/x86/platform/efi/efi.c:650:2: note: in expansion of macro 'for_each_efi_memory_desc'
     for_each_efi_memory_desc(md) {
     ^
   include/linux/efi.h:973:37: error: invalid type argument of '->' (have 'struct efi')
     for_each_efi_memory_desc_in_map(efi->memmap, md)
                                        ^
   include/linux/efi.h:963:38: note: in definition of macro 'for_each_efi_memory_desc_in_map'
          (md) <= (efi_memory_desc_t *)((m)->map_end - (m)->desc_size); \
                                         ^
   arch/x86/platform/efi/efi.c:650:2: note: in expansion of macro 'for_each_efi_memory_desc'
     for_each_efi_memory_desc(md) {
     ^
   include/linux/efi.h:973:37: error: invalid type argument of '->' (have 'struct efi')
     for_each_efi_memory_desc_in_map(efi->memmap, md)
                                        ^
   include/linux/efi.h:963:53: note: in definition of macro 'for_each_efi_memory_desc_in_map'
          (md) <= (efi_memory_desc_t *)((m)->map_end - (m)->desc_size); \
                                                        ^
   arch/x86/platform/efi/efi.c:650:2: note: in expansion of macro 'for_each_efi_memory_desc'
     for_each_efi_memory_desc(md) {
     ^
   include/linux/efi.h:973:37: error: invalid type argument of '->' (have 'struct efi')
     for_each_efi_memory_desc_in_map(efi->memmap, md)
                                        ^
   include/linux/efi.h:964:30: note: in definition of macro 'for_each_efi_memory_desc_in_map'
          (md) = (void *)(md) + (m)->desc_size)
                                 ^
   arch/x86/platform/efi/efi.c:650:2: note: in expansion of macro 'for_each_efi_memory_desc'
     for_each_efi_memory_desc(md) {
     ^
   arch/x86/platform/efi/efi.c: In function 'kexec_enter_virtual_mode':
   include/linux/efi.h:973:37: error: invalid type argument of '->' (have 'struct efi')
     for_each_efi_memory_desc_in_map(efi->memmap, md)
                                        ^
   include/linux/efi.h:962:15: note: in definition of macro 'for_each_efi_memory_desc_in_map'
     for ((md) = (m)->map;         \
                  ^
   arch/x86/platform/efi/efi.c:829:2: note: in expansion of macro 'for_each_efi_memory_desc'
     for_each_efi_memory_desc(md) {
     ^
   include/linux/efi.h:973:37: error: invalid type argument of '->' (have 'struct efi')
     for_each_efi_memory_desc_in_map(efi->memmap, md)
                                        ^
   include/linux/efi.h:963:38: note: in definition of macro 'for_each_efi_memory_desc_in_map'
          (md) <= (efi_memory_desc_t *)((m)->map_end - (m)->desc_size); \
                                         ^
   arch/x86/platform/efi/efi.c:829:2: note: in expansion of macro 'for_each_efi_memory_desc'
     for_each_efi_memory_desc(md) {
     ^
..

vim +259 drivers/firmware/efi/efi.c

   253	 * physicall address, determine if it exists within an EFI Memory Map entry,
   254	 * and if so, populate the supplied memory descriptor with the appropriate
   255	 * data.
   256	 */
   257	int __init efi_mem_desc_lookup(u64 phys_addr, efi_memory_desc_t *out_md)
   258	{
 > 259		struct efi_memory_map *map = &efi.memmap;
   260		phys_addr_t p, e;
   261	
   262		if (!efi_enabled(EFI_MEMMAP)) {

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

[-- Attachment #2: .config.gz --]
[-- Type: application/octet-stream, Size: 53377 bytes --]

WARNING: multiple messages have this Message-ID (diff)
From: kbuild test robot <lkp-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
Cc: kbuild-all-JC7UmRfGjtg@public.gmane.org,
	linux-efi-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	Ard Biesheuvel
	<ard.biesheuvel-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>,
	Tony Luck <tony.luck-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
	Matt Fleming
	<matt-mF/unelCI9GS6iBeEJttW/XRex20P6io@public.gmane.org>,
	Mark Salter <msalter-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>,
	Leif Lindholm
	<leif.lindholm-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
Subject: Re: [PATCH 1/2] efi: Iterate over efi->memmap in for_each_efi_memory_desc
Date: Mon, 11 Apr 2016 21:24:03 +0800	[thread overview]
Message-ID: <201604112150.cvcV2wEO%fengguang.wu@intel.com> (raw)
In-Reply-To: <1460379809-18490-2-git-send-email-matt-mF/unelCI9GS6iBeEJttW/XRex20P6io@public.gmane.org>

[-- Attachment #1: Type: text/plain, Size: 19828 bytes --]

Hi Matt,

[auto build test ERROR on efi/next]
[cannot apply to v4.6-rc3 next-20160411]
[if your patch is applied to the wrong git tree, please drop us a note to help improving the system]

url:    https://github.com/0day-ci/linux/commits/Matt-Fleming/efi-Delete-global-memmap-variable/20160411-210837
base:   https://git.kernel.org/pub/scm/linux/kernel/git/mfleming/efi.git next
config: x86_64-allyesconfig (attached as .config)
reproduce:
        # save the attached .config to linux build tree
        make ARCH=x86_64 

Note: the linux-review/Matt-Fleming/efi-Delete-global-memmap-variable/20160411-210837 HEAD 8fc82e8aa63ce89d26d143cce41530319212f7d8 builds fine.
      It only hurts bisectibility.

All error/warnings (new ones prefixed by >>):

   In file included from arch/x86/platform/efi/quirks.c:8:0:
   arch/x86/platform/efi/quirks.c: In function 'efi_reserve_boot_services':
   include/linux/efi.h:973:37: error: invalid type argument of '->' (have 'struct efi')
     for_each_efi_memory_desc_in_map(efi->memmap, md)
                                        ^
   include/linux/efi.h:962:15: note: in definition of macro 'for_each_efi_memory_desc_in_map'
     for ((md) = (m)->map;         \
                  ^
>> arch/x86/platform/efi/quirks.c:200:2: note: in expansion of macro 'for_each_efi_memory_desc'
     for_each_efi_memory_desc(md) {
     ^
   include/linux/efi.h:973:37: error: invalid type argument of '->' (have 'struct efi')
     for_each_efi_memory_desc_in_map(efi->memmap, md)
                                        ^
   include/linux/efi.h:963:38: note: in definition of macro 'for_each_efi_memory_desc_in_map'
          (md) <= (efi_memory_desc_t *)((m)->map_end - (m)->desc_size); \
                                         ^
>> arch/x86/platform/efi/quirks.c:200:2: note: in expansion of macro 'for_each_efi_memory_desc'
     for_each_efi_memory_desc(md) {
     ^
   include/linux/efi.h:973:37: error: invalid type argument of '->' (have 'struct efi')
     for_each_efi_memory_desc_in_map(efi->memmap, md)
                                        ^
   include/linux/efi.h:963:53: note: in definition of macro 'for_each_efi_memory_desc_in_map'
          (md) <= (efi_memory_desc_t *)((m)->map_end - (m)->desc_size); \
                                                        ^
>> arch/x86/platform/efi/quirks.c:200:2: note: in expansion of macro 'for_each_efi_memory_desc'
     for_each_efi_memory_desc(md) {
     ^
   include/linux/efi.h:973:37: error: invalid type argument of '->' (have 'struct efi')
     for_each_efi_memory_desc_in_map(efi->memmap, md)
                                        ^
   include/linux/efi.h:964:30: note: in definition of macro 'for_each_efi_memory_desc_in_map'
          (md) = (void *)(md) + (m)->desc_size)
                                 ^
>> arch/x86/platform/efi/quirks.c:200:2: note: in expansion of macro 'for_each_efi_memory_desc'
     for_each_efi_memory_desc(md) {
     ^
   arch/x86/platform/efi/quirks.c: In function 'efi_free_boot_services':
   include/linux/efi.h:973:37: error: invalid type argument of '->' (have 'struct efi')
     for_each_efi_memory_desc_in_map(efi->memmap, md)
                                        ^
   include/linux/efi.h:962:15: note: in definition of macro 'for_each_efi_memory_desc_in_map'
     for ((md) = (m)->map;         \
                  ^
   arch/x86/platform/efi/quirks.c:254:2: note: in expansion of macro 'for_each_efi_memory_desc'
     for_each_efi_memory_desc(md) {
     ^
   include/linux/efi.h:973:37: error: invalid type argument of '->' (have 'struct efi')
     for_each_efi_memory_desc_in_map(efi->memmap, md)
                                        ^
   include/linux/efi.h:963:38: note: in definition of macro 'for_each_efi_memory_desc_in_map'
          (md) <= (efi_memory_desc_t *)((m)->map_end - (m)->desc_size); \
                                         ^
   arch/x86/platform/efi/quirks.c:254:2: note: in expansion of macro 'for_each_efi_memory_desc'
     for_each_efi_memory_desc(md) {
     ^
   include/linux/efi.h:973:37: error: invalid type argument of '->' (have 'struct efi')
     for_each_efi_memory_desc_in_map(efi->memmap, md)
                                        ^
   include/linux/efi.h:963:53: note: in definition of macro 'for_each_efi_memory_desc_in_map'
          (md) <= (efi_memory_desc_t *)((m)->map_end - (m)->desc_size); \
                                                        ^
   arch/x86/platform/efi/quirks.c:254:2: note: in expansion of macro 'for_each_efi_memory_desc'
     for_each_efi_memory_desc(md) {
     ^
   include/linux/efi.h:973:37: error: invalid type argument of '->' (have 'struct efi')
     for_each_efi_memory_desc_in_map(efi->memmap, md)
                                        ^
   include/linux/efi.h:964:30: note: in definition of macro 'for_each_efi_memory_desc_in_map'
          (md) = (void *)(md) + (m)->desc_size)
                                 ^
   arch/x86/platform/efi/quirks.c:254:2: note: in expansion of macro 'for_each_efi_memory_desc'
     for_each_efi_memory_desc(md) {
     ^
--
   In file included from arch/x86/platform/efi/efi.c:35:0:
   arch/x86/platform/efi/efi.c: In function 'efi_find_mirror':
   include/linux/efi.h:973:37: error: invalid type argument of '->' (have 'struct efi')
     for_each_efi_memory_desc_in_map(efi->memmap, md)
                                        ^
   include/linux/efi.h:962:15: note: in definition of macro 'for_each_efi_memory_desc_in_map'
     for ((md) = (m)->map;         \
                  ^
>> arch/x86/platform/efi/efi.c:125:2: note: in expansion of macro 'for_each_efi_memory_desc'
     for_each_efi_memory_desc(md) {
     ^
   include/linux/efi.h:973:37: error: invalid type argument of '->' (have 'struct efi')
     for_each_efi_memory_desc_in_map(efi->memmap, md)
                                        ^
   include/linux/efi.h:963:38: note: in definition of macro 'for_each_efi_memory_desc_in_map'
          (md) <= (efi_memory_desc_t *)((m)->map_end - (m)->desc_size); \
                                         ^
>> arch/x86/platform/efi/efi.c:125:2: note: in expansion of macro 'for_each_efi_memory_desc'
     for_each_efi_memory_desc(md) {
     ^
   include/linux/efi.h:973:37: error: invalid type argument of '->' (have 'struct efi')
     for_each_efi_memory_desc_in_map(efi->memmap, md)
                                        ^
   include/linux/efi.h:963:53: note: in definition of macro 'for_each_efi_memory_desc_in_map'
          (md) <= (efi_memory_desc_t *)((m)->map_end - (m)->desc_size); \
                                                        ^
>> arch/x86/platform/efi/efi.c:125:2: note: in expansion of macro 'for_each_efi_memory_desc'
     for_each_efi_memory_desc(md) {
     ^
   include/linux/efi.h:973:37: error: invalid type argument of '->' (have 'struct efi')
     for_each_efi_memory_desc_in_map(efi->memmap, md)
                                        ^
   include/linux/efi.h:964:30: note: in definition of macro 'for_each_efi_memory_desc_in_map'
          (md) = (void *)(md) + (m)->desc_size)
                                 ^
>> arch/x86/platform/efi/efi.c:125:2: note: in expansion of macro 'for_each_efi_memory_desc'
     for_each_efi_memory_desc(md) {
     ^
   arch/x86/platform/efi/efi.c: In function 'do_add_efi_memmap':
   include/linux/efi.h:973:37: error: invalid type argument of '->' (have 'struct efi')
     for_each_efi_memory_desc_in_map(efi->memmap, md)
                                        ^
   include/linux/efi.h:962:15: note: in definition of macro 'for_each_efi_memory_desc_in_map'
     for ((md) = (m)->map;         \
                  ^
   arch/x86/platform/efi/efi.c:150:2: note: in expansion of macro 'for_each_efi_memory_desc'
     for_each_efi_memory_desc(md) {
     ^
   include/linux/efi.h:973:37: error: invalid type argument of '->' (have 'struct efi')
     for_each_efi_memory_desc_in_map(efi->memmap, md)
                                        ^
   include/linux/efi.h:963:38: note: in definition of macro 'for_each_efi_memory_desc_in_map'
          (md) <= (efi_memory_desc_t *)((m)->map_end - (m)->desc_size); \
                                         ^
   arch/x86/platform/efi/efi.c:150:2: note: in expansion of macro 'for_each_efi_memory_desc'
     for_each_efi_memory_desc(md) {
     ^
   include/linux/efi.h:973:37: error: invalid type argument of '->' (have 'struct efi')
     for_each_efi_memory_desc_in_map(efi->memmap, md)
                                        ^
   include/linux/efi.h:963:53: note: in definition of macro 'for_each_efi_memory_desc_in_map'
          (md) <= (efi_memory_desc_t *)((m)->map_end - (m)->desc_size); \
                                                        ^
   arch/x86/platform/efi/efi.c:150:2: note: in expansion of macro 'for_each_efi_memory_desc'
     for_each_efi_memory_desc(md) {
     ^
   include/linux/efi.h:973:37: error: invalid type argument of '->' (have 'struct efi')
     for_each_efi_memory_desc_in_map(efi->memmap, md)
                                        ^
   include/linux/efi.h:964:30: note: in definition of macro 'for_each_efi_memory_desc_in_map'
          (md) = (void *)(md) + (m)->desc_size)
                                 ^
   arch/x86/platform/efi/efi.c:150:2: note: in expansion of macro 'for_each_efi_memory_desc'
     for_each_efi_memory_desc(md) {
     ^
   arch/x86/platform/efi/efi.c: In function 'efi_print_memmap':
   include/linux/efi.h:973:37: error: invalid type argument of '->' (have 'struct efi')
     for_each_efi_memory_desc_in_map(efi->memmap, md)
                                        ^
   include/linux/efi.h:962:15: note: in definition of macro 'for_each_efi_memory_desc_in_map'
     for ((md) = (m)->map;         \
                  ^
   arch/x86/platform/efi/efi.c:229:2: note: in expansion of macro 'for_each_efi_memory_desc'
     for_each_efi_memory_desc(md) {
     ^
   include/linux/efi.h:973:37: error: invalid type argument of '->' (have 'struct efi')
     for_each_efi_memory_desc_in_map(efi->memmap, md)
                                        ^
   include/linux/efi.h:963:38: note: in definition of macro 'for_each_efi_memory_desc_in_map'
          (md) <= (efi_memory_desc_t *)((m)->map_end - (m)->desc_size); \
                                         ^
   arch/x86/platform/efi/efi.c:229:2: note: in expansion of macro 'for_each_efi_memory_desc'
     for_each_efi_memory_desc(md) {
     ^
   include/linux/efi.h:973:37: error: invalid type argument of '->' (have 'struct efi')
     for_each_efi_memory_desc_in_map(efi->memmap, md)
                                        ^
   include/linux/efi.h:963:53: note: in definition of macro 'for_each_efi_memory_desc_in_map'
          (md) <= (efi_memory_desc_t *)((m)->map_end - (m)->desc_size); \
                                                        ^
   arch/x86/platform/efi/efi.c:229:2: note: in expansion of macro 'for_each_efi_memory_desc'
     for_each_efi_memory_desc(md) {
     ^
   include/linux/efi.h:973:37: error: invalid type argument of '->' (have 'struct efi')
     for_each_efi_memory_desc_in_map(efi->memmap, md)
                                        ^
   include/linux/efi.h:964:30: note: in definition of macro 'for_each_efi_memory_desc_in_map'
          (md) = (void *)(md) + (m)->desc_size)
                                 ^
   arch/x86/platform/efi/efi.c:229:2: note: in expansion of macro 'for_each_efi_memory_desc'
     for_each_efi_memory_desc(md) {
     ^
   arch/x86/platform/efi/efi.c: In function 'runtime_code_page_mkexec':
   include/linux/efi.h:973:37: error: invalid type argument of '->' (have 'struct efi')
     for_each_efi_memory_desc_in_map(efi->memmap, md)
                                        ^
   include/linux/efi.h:962:15: note: in definition of macro 'for_each_efi_memory_desc_in_map'
     for ((md) = (m)->map;         \
                  ^
   arch/x86/platform/efi/efi.c:550:2: note: in expansion of macro 'for_each_efi_memory_desc'
     for_each_efi_memory_desc(md) {
     ^
   include/linux/efi.h:973:37: error: invalid type argument of '->' (have 'struct efi')
     for_each_efi_memory_desc_in_map(efi->memmap, md)
                                        ^
   include/linux/efi.h:963:38: note: in definition of macro 'for_each_efi_memory_desc_in_map'
          (md) <= (efi_memory_desc_t *)((m)->map_end - (m)->desc_size); \
                                         ^
   arch/x86/platform/efi/efi.c:550:2: note: in expansion of macro 'for_each_efi_memory_desc'
     for_each_efi_memory_desc(md) {
     ^
   include/linux/efi.h:973:37: error: invalid type argument of '->' (have 'struct efi')
     for_each_efi_memory_desc_in_map(efi->memmap, md)
                                        ^
   include/linux/efi.h:963:53: note: in definition of macro 'for_each_efi_memory_desc_in_map'
          (md) <= (efi_memory_desc_t *)((m)->map_end - (m)->desc_size); \
                                                        ^
   arch/x86/platform/efi/efi.c:550:2: note: in expansion of macro 'for_each_efi_memory_desc'
     for_each_efi_memory_desc(md) {
     ^
   include/linux/efi.h:973:37: error: invalid type argument of '->' (have 'struct efi')
     for_each_efi_memory_desc_in_map(efi->memmap, md)
                                        ^
   include/linux/efi.h:964:30: note: in definition of macro 'for_each_efi_memory_desc_in_map'
          (md) = (void *)(md) + (m)->desc_size)
                                 ^
   arch/x86/platform/efi/efi.c:550:2: note: in expansion of macro 'for_each_efi_memory_desc'
     for_each_efi_memory_desc(md) {
     ^
>> arch/x86/platform/efi/efi.c:547:8: warning: unused variable 'p' [-Wunused-variable]
     void *p;
           ^
   In file included from arch/x86/platform/efi/efi.c:35:0:
   arch/x86/platform/efi/efi.c: In function 'efi_merge_regions':
   include/linux/efi.h:973:37: error: invalid type argument of '->' (have 'struct efi')
     for_each_efi_memory_desc_in_map(efi->memmap, md)
                                        ^
   include/linux/efi.h:962:15: note: in definition of macro 'for_each_efi_memory_desc_in_map'
     for ((md) = (m)->map;         \
                  ^
   arch/x86/platform/efi/efi.c:600:2: note: in expansion of macro 'for_each_efi_memory_desc'
     for_each_efi_memory_desc(md) {
     ^
   include/linux/efi.h:973:37: error: invalid type argument of '->' (have 'struct efi')
     for_each_efi_memory_desc_in_map(efi->memmap, md)
                                        ^
   include/linux/efi.h:963:38: note: in definition of macro 'for_each_efi_memory_desc_in_map'
          (md) <= (efi_memory_desc_t *)((m)->map_end - (m)->desc_size); \
                                         ^
   arch/x86/platform/efi/efi.c:600:2: note: in expansion of macro 'for_each_efi_memory_desc'
     for_each_efi_memory_desc(md) {
     ^
   include/linux/efi.h:973:37: error: invalid type argument of '->' (have 'struct efi')
     for_each_efi_memory_desc_in_map(efi->memmap, md)
                                        ^
   include/linux/efi.h:963:53: note: in definition of macro 'for_each_efi_memory_desc_in_map'
          (md) <= (efi_memory_desc_t *)((m)->map_end - (m)->desc_size); \
                                                        ^
   arch/x86/platform/efi/efi.c:600:2: note: in expansion of macro 'for_each_efi_memory_desc'
     for_each_efi_memory_desc(md) {
     ^
   include/linux/efi.h:973:37: error: invalid type argument of '->' (have 'struct efi')
     for_each_efi_memory_desc_in_map(efi->memmap, md)
                                        ^
   include/linux/efi.h:964:30: note: in definition of macro 'for_each_efi_memory_desc_in_map'
          (md) = (void *)(md) + (m)->desc_size)
                                 ^
   arch/x86/platform/efi/efi.c:600:2: note: in expansion of macro 'for_each_efi_memory_desc'
     for_each_efi_memory_desc(md) {
     ^
   arch/x86/platform/efi/efi.c:597:8: warning: unused variable 'p' [-Wunused-variable]
     void *p;
           ^
   In file included from arch/x86/platform/efi/efi.c:35:0:
   arch/x86/platform/efi/efi.c: In function 'save_runtime_map':
   include/linux/efi.h:973:37: error: invalid type argument of '->' (have 'struct efi')
     for_each_efi_memory_desc_in_map(efi->memmap, md)
                                        ^
   include/linux/efi.h:962:15: note: in definition of macro 'for_each_efi_memory_desc_in_map'
     for ((md) = (m)->map;         \
                  ^
   arch/x86/platform/efi/efi.c:650:2: note: in expansion of macro 'for_each_efi_memory_desc'
     for_each_efi_memory_desc(md) {
     ^
   include/linux/efi.h:973:37: error: invalid type argument of '->' (have 'struct efi')
     for_each_efi_memory_desc_in_map(efi->memmap, md)
                                        ^
   include/linux/efi.h:963:38: note: in definition of macro 'for_each_efi_memory_desc_in_map'
          (md) <= (efi_memory_desc_t *)((m)->map_end - (m)->desc_size); \
                                         ^
   arch/x86/platform/efi/efi.c:650:2: note: in expansion of macro 'for_each_efi_memory_desc'
     for_each_efi_memory_desc(md) {
     ^
   include/linux/efi.h:973:37: error: invalid type argument of '->' (have 'struct efi')
     for_each_efi_memory_desc_in_map(efi->memmap, md)
                                        ^
   include/linux/efi.h:963:53: note: in definition of macro 'for_each_efi_memory_desc_in_map'
          (md) <= (efi_memory_desc_t *)((m)->map_end - (m)->desc_size); \
                                                        ^
   arch/x86/platform/efi/efi.c:650:2: note: in expansion of macro 'for_each_efi_memory_desc'
     for_each_efi_memory_desc(md) {
     ^
   include/linux/efi.h:973:37: error: invalid type argument of '->' (have 'struct efi')
     for_each_efi_memory_desc_in_map(efi->memmap, md)
                                        ^
   include/linux/efi.h:964:30: note: in definition of macro 'for_each_efi_memory_desc_in_map'
          (md) = (void *)(md) + (m)->desc_size)
                                 ^
   arch/x86/platform/efi/efi.c:650:2: note: in expansion of macro 'for_each_efi_memory_desc'
     for_each_efi_memory_desc(md) {
     ^
   arch/x86/platform/efi/efi.c: In function 'kexec_enter_virtual_mode':
   include/linux/efi.h:973:37: error: invalid type argument of '->' (have 'struct efi')
     for_each_efi_memory_desc_in_map(efi->memmap, md)
                                        ^
   include/linux/efi.h:962:15: note: in definition of macro 'for_each_efi_memory_desc_in_map'
     for ((md) = (m)->map;         \
                  ^
   arch/x86/platform/efi/efi.c:829:2: note: in expansion of macro 'for_each_efi_memory_desc'
     for_each_efi_memory_desc(md) {
     ^
   include/linux/efi.h:973:37: error: invalid type argument of '->' (have 'struct efi')
     for_each_efi_memory_desc_in_map(efi->memmap, md)
                                        ^
   include/linux/efi.h:963:38: note: in definition of macro 'for_each_efi_memory_desc_in_map'
          (md) <= (efi_memory_desc_t *)((m)->map_end - (m)->desc_size); \
                                         ^
   arch/x86/platform/efi/efi.c:829:2: note: in expansion of macro 'for_each_efi_memory_desc'
     for_each_efi_memory_desc(md) {
     ^
..

vim +259 drivers/firmware/efi/efi.c

   253	 * physicall address, determine if it exists within an EFI Memory Map entry,
   254	 * and if so, populate the supplied memory descriptor with the appropriate
   255	 * data.
   256	 */
   257	int __init efi_mem_desc_lookup(u64 phys_addr, efi_memory_desc_t *out_md)
   258	{
 > 259		struct efi_memory_map *map = &efi.memmap;
   260		phys_addr_t p, e;
   261	
   262		if (!efi_enabled(EFI_MEMMAP)) {

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

[-- Attachment #2: .config.gz --]
[-- Type: application/octet-stream, Size: 53377 bytes --]

  reply	other threads:[~2016-04-11 13:25 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-04-11 13:03 [PATCH 0/2] efi: Delete global 'memmap' variable Matt Fleming
2016-04-11 13:03 ` Matt Fleming
2016-04-11 13:03 ` [PATCH 1/2] efi: Iterate over efi->memmap in for_each_efi_memory_desc Matt Fleming
2016-04-11 13:03   ` Matt Fleming
2016-04-11 13:24   ` kbuild test robot [this message]
2016-04-11 13:24     ` kbuild test robot
2016-04-11 13:27   ` kbuild test robot
2016-04-11 13:27     ` kbuild test robot
2016-04-11 13:03 ` [PATCH 2/2] efi: Remove global 'memmap' Matt Fleming
2016-04-11 13:03   ` Matt Fleming
2016-04-11 13:17   ` Ard Biesheuvel
2016-04-12 20:01     ` Matt Fleming
2016-04-12 20:01       ` Matt Fleming
2016-04-13  8:12       ` Ard Biesheuvel
2016-04-13  8:12         ` Ard Biesheuvel
2016-04-13 10:24         ` Matt Fleming
2016-04-13 10:24           ` Matt Fleming

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=201604112150.cvcV2wEO%fengguang.wu@intel.com \
    --to=lkp@intel.com \
    --cc=ard.biesheuvel@linaro.org \
    --cc=kbuild-all@01.org \
    --cc=leif.lindholm@linaro.org \
    --cc=linux-efi@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=matt@codeblueprint.co.uk \
    --cc=msalter@redhat.com \
    --cc=tony.luck@gmail.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.