Hi zhong,

Thank you for the patch! Yet something to improve:

[auto build test ERROR on char-misc/char-misc-testing]
[also build test ERROR on v4.19-rc5 next-20180926]
[if your patch is applied to the wrong git tree, please drop us a note to help 
improve the system]

url:    
https://github.com/0day-ci/linux/commits/zhong-jiang/misc-genwqe-remove-duplicated-include-and-order-header-files-alphabetically-in-card_utils-c/20180927-005512
config: x86_64-randconfig-s3-09270441 (attached as .config)
compiler: gcc-7 (Debian 7.3.0-1) 7.3.0
reproduce:
        # save the attached .config to linux build tree
        make ARCH=x86_64 

All errors (new ones prefixed by >>):

   In file included from drivers/misc/genwqe/card_utils.c:25:0:
   arch/x86/include/asm/pgtable.h:51:8: error: unknown type name 'spinlock_t'
    extern spinlock_t pgd_lock;
           ^~~~~~~~~~
   In file included from drivers/misc/genwqe/card_utils.c:25:0:
   arch/x86/include/asm/pgtable.h: In function 'read_pkru':
   arch/x86/include/asm/pgtable.h:130:6: error: implicit declaration of 
function 'boot_cpu_has'; did you mean 'get_cpu_mask'? 
[-Werror=implicit-function-declaration]
     if (boot_cpu_has(X86_FEATURE_OSPKE))
         ^~~~~~~~~~~~
         get_cpu_mask
>> arch/x86/include/asm/pgtable.h:131:10: error: implicit declaration of 
>> function '__read_pkru'; did you mean 'read_pkru'? 
>> [-Werror=implicit-function-declaration]
      return __read_pkru();
             ^~~~~~~~~~~
             read_pkru
   arch/x86/include/asm/pgtable.h: In function 'write_pkru':
>> arch/x86/include/asm/pgtable.h:138:3: error: implicit declaration of 
>> function '__write_pkru'; did you mean 'write_pkru'? 
>> [-Werror=implicit-function-declaration]
      __write_pkru(pkru);
      ^~~~~~~~~~~~
      write_pkru
   arch/x86/include/asm/pgtable.h: In function 'pti_set_user_pgtbl':
   arch/x86/include/asm/pgtable.h:677:7: error: implicit declaration of 
function 'static_cpu_has'; did you mean 'static_key_false'? 
[-Werror=implicit-function-declaration]
     if (!static_cpu_has(X86_FEATURE_PTI))
          ^~~~~~~~~~~~~~
          static_key_false
   In file included from arch/x86/include/asm/processor.h:24:0,
                    from arch/x86/include/asm/pgtable_64.h:14,
                    from arch/x86/include/asm/pgtable.h:694,
                    from drivers/misc/genwqe/card_utils.c:25:
   arch/x86/include/asm/special_insns.h: At top level:
   arch/x86/include/asm/special_insns.h:122:19: error: conflicting types for 
'__read_pkru'
    static inline u32 __read_pkru(void)
                      ^~~~~~~~~~~
   In file included from drivers/misc/genwqe/card_utils.c:25:0:
   arch/x86/include/asm/pgtable.h:131:10: note: previous implicit declaration 
of '__read_pkru' was here
      return __read_pkru();
             ^~~~~~~~~~~
   In file included from arch/x86/include/asm/processor.h:24:0,
                    from arch/x86/include/asm/pgtable_64.h:14,
                    from arch/x86/include/asm/pgtable.h:694,
                    from drivers/misc/genwqe/card_utils.c:25:
   arch/x86/include/asm/special_insns.h:127:20: warning: conflicting types for 
'__write_pkru'
    static inline void __write_pkru(u32 pkru)
                       ^~~~~~~~~~~~
   arch/x86/include/asm/special_insns.h:127:20: error: static declaration of 
'__write_pkru' follows non-static declaration
   In file included from drivers/misc/genwqe/card_utils.c:25:0:
   arch/x86/include/asm/pgtable.h:138:3: note: previous implicit declaration of 
'__write_pkru' was here
      __write_pkru(pkru);
      ^~~~~~~~~~~~
   cc1: some warnings being treated as errors

vim +131 arch/x86/include/asm/pgtable.h

8405b122 include/asm-x86/pgtable.h      Jeremy Fitzhardinge 2008-01-30   50  
e3ed910d include/asm-x86/pgtable.h      Jeremy Fitzhardinge 2008-01-30  @51  
extern spinlock_t pgd_lock;
e3ed910d include/asm-x86/pgtable.h      Jeremy Fitzhardinge 2008-01-30   52  
extern struct list_head pgd_list;
8405b122 include/asm-x86/pgtable.h      Jeremy Fitzhardinge 2008-01-30   53  
617d34d9 arch/x86/include/asm/pgtable.h Jeremy Fitzhardinge 2010-09-21   54  
extern struct mm_struct *pgd_page_get_mm(struct page *page);
617d34d9 arch/x86/include/asm/pgtable.h Jeremy Fitzhardinge 2010-09-21   55  
21729f81 arch/x86/include/asm/pgtable.h Tom Lendacky        2017-07-17   56  
extern pmdval_t early_pmd_flags;
21729f81 arch/x86/include/asm/pgtable.h Tom Lendacky        2017-07-17   57  
54321d94 arch/x86/include/asm/pgtable.h Jeremy Fitzhardinge 2009-02-11   58  
#ifdef CONFIG_PARAVIRT
54321d94 arch/x86/include/asm/pgtable.h Jeremy Fitzhardinge 2009-02-11   59  
#include <asm/paravirt.h>
54321d94 arch/x86/include/asm/pgtable.h Jeremy Fitzhardinge 2009-02-11   60  
#else  /* !CONFIG_PARAVIRT */
54321d94 arch/x86/include/asm/pgtable.h Jeremy Fitzhardinge 2009-02-11   61  
#define set_pte(ptep, pte)         native_set_pte(ptep, pte)
54321d94 arch/x86/include/asm/pgtable.h Jeremy Fitzhardinge 2009-02-11   62  
#define set_pte_at(mm, addr, ptep, pte)    native_set_pte_at(mm, addr, ptep, 
pte)
54321d94 arch/x86/include/asm/pgtable.h Jeremy Fitzhardinge 2009-02-11   63  
54321d94 arch/x86/include/asm/pgtable.h Jeremy Fitzhardinge 2009-02-11   64  
#define set_pte_atomic(ptep, pte)                                  \
54321d94 arch/x86/include/asm/pgtable.h Jeremy Fitzhardinge 2009-02-11   65     
native_set_pte_atomic(ptep, pte)
54321d94 arch/x86/include/asm/pgtable.h Jeremy Fitzhardinge 2009-02-11   66  
54321d94 arch/x86/include/asm/pgtable.h Jeremy Fitzhardinge 2009-02-11   67  
#define set_pmd(pmdp, pmd)         native_set_pmd(pmdp, pmd)
54321d94 arch/x86/include/asm/pgtable.h Jeremy Fitzhardinge 2009-02-11   68  
f2a6a705 arch/x86/include/asm/pgtable.h Kirill A. Shutemov  2017-03-17   69  
#ifndef __PAGETABLE_P4D_FOLDED
54321d94 arch/x86/include/asm/pgtable.h Jeremy Fitzhardinge 2009-02-11   70  
#define set_pgd(pgdp, pgd)         native_set_pgd(pgdp, pgd)
ed7588d5 arch/x86/include/asm/pgtable.h Kirill A. Shutemov  2018-05-18   71  
#define pgd_clear(pgd)                     (pgtable_l5_enabled() ? 
native_pgd_clear(pgd) : 0)
54321d94 arch/x86/include/asm/pgtable.h Jeremy Fitzhardinge 2009-02-11   72  
#endif
54321d94 arch/x86/include/asm/pgtable.h Jeremy Fitzhardinge 2009-02-11   73  
f2a6a705 arch/x86/include/asm/pgtable.h Kirill A. Shutemov  2017-03-17   74  
#ifndef set_p4d
f2a6a705 arch/x86/include/asm/pgtable.h Kirill A. Shutemov  2017-03-17   75  # 
define set_p4d(p4dp, p4d)                native_set_p4d(p4dp, p4d)
f2a6a705 arch/x86/include/asm/pgtable.h Kirill A. Shutemov  2017-03-17   76  
#endif
f2a6a705 arch/x86/include/asm/pgtable.h Kirill A. Shutemov  2017-03-17   77  
f2a6a705 arch/x86/include/asm/pgtable.h Kirill A. Shutemov  2017-03-17   78  
#ifndef __PAGETABLE_PUD_FOLDED
f2a6a705 arch/x86/include/asm/pgtable.h Kirill A. Shutemov  2017-03-17   79  
#define p4d_clear(p4d)                     native_p4d_clear(p4d)
f2a6a705 arch/x86/include/asm/pgtable.h Kirill A. Shutemov  2017-03-17   80  
#endif
f2a6a705 arch/x86/include/asm/pgtable.h Kirill A. Shutemov  2017-03-17   81  
54321d94 arch/x86/include/asm/pgtable.h Jeremy Fitzhardinge 2009-02-11   82  
#ifndef set_pud
54321d94 arch/x86/include/asm/pgtable.h Jeremy Fitzhardinge 2009-02-11   83  # 
define set_pud(pudp, pud)                native_set_pud(pudp, pud)
54321d94 arch/x86/include/asm/pgtable.h Jeremy Fitzhardinge 2009-02-11   84  
#endif
54321d94 arch/x86/include/asm/pgtable.h Jeremy Fitzhardinge 2009-02-11   85  
d0f33ac9 arch/x86/include/asm/pgtable.h Arnd Bergmann       2017-03-16   86  
#ifndef __PAGETABLE_PUD_FOLDED
54321d94 arch/x86/include/asm/pgtable.h Jeremy Fitzhardinge 2009-02-11   87  
#define pud_clear(pud)                     native_pud_clear(pud)
54321d94 arch/x86/include/asm/pgtable.h Jeremy Fitzhardinge 2009-02-11   88  
#endif
54321d94 arch/x86/include/asm/pgtable.h Jeremy Fitzhardinge 2009-02-11   89  
54321d94 arch/x86/include/asm/pgtable.h Jeremy Fitzhardinge 2009-02-11   90  
#define pte_clear(mm, addr, ptep)  native_pte_clear(mm, addr, ptep)
54321d94 arch/x86/include/asm/pgtable.h Jeremy Fitzhardinge 2009-02-11   91  
#define pmd_clear(pmd)                     native_pmd_clear(pmd)
54321d94 arch/x86/include/asm/pgtable.h Jeremy Fitzhardinge 2009-02-11   92  
54321d94 arch/x86/include/asm/pgtable.h Jeremy Fitzhardinge 2009-02-11   93  
#define pgd_val(x) native_pgd_val(x)
54321d94 arch/x86/include/asm/pgtable.h Jeremy Fitzhardinge 2009-02-11   94  
#define __pgd(x)   native_make_pgd(x)
54321d94 arch/x86/include/asm/pgtable.h Jeremy Fitzhardinge 2009-02-11   95  
f2a6a705 arch/x86/include/asm/pgtable.h Kirill A. Shutemov  2017-03-17   96  
#ifndef __PAGETABLE_P4D_FOLDED
f2a6a705 arch/x86/include/asm/pgtable.h Kirill A. Shutemov  2017-03-17   97  
#define p4d_val(x) native_p4d_val(x)
f2a6a705 arch/x86/include/asm/pgtable.h Kirill A. Shutemov  2017-03-17   98  
#define __p4d(x)   native_make_p4d(x)
f2a6a705 arch/x86/include/asm/pgtable.h Kirill A. Shutemov  2017-03-17   99  
#endif
f2a6a705 arch/x86/include/asm/pgtable.h Kirill A. Shutemov  2017-03-17  100  
54321d94 arch/x86/include/asm/pgtable.h Jeremy Fitzhardinge 2009-02-11  101  
#ifndef __PAGETABLE_PUD_FOLDED
54321d94 arch/x86/include/asm/pgtable.h Jeremy Fitzhardinge 2009-02-11  102  
#define pud_val(x) native_pud_val(x)
54321d94 arch/x86/include/asm/pgtable.h Jeremy Fitzhardinge 2009-02-11  103  
#define __pud(x)   native_make_pud(x)
54321d94 arch/x86/include/asm/pgtable.h Jeremy Fitzhardinge 2009-02-11  104  
#endif
54321d94 arch/x86/include/asm/pgtable.h Jeremy Fitzhardinge 2009-02-11  105  
54321d94 arch/x86/include/asm/pgtable.h Jeremy Fitzhardinge 2009-02-11  106  
#ifndef __PAGETABLE_PMD_FOLDED
54321d94 arch/x86/include/asm/pgtable.h Jeremy Fitzhardinge 2009-02-11  107  
#define pmd_val(x) native_pmd_val(x)
54321d94 arch/x86/include/asm/pgtable.h Jeremy Fitzhardinge 2009-02-11  108  
#define __pmd(x)   native_make_pmd(x)
54321d94 arch/x86/include/asm/pgtable.h Jeremy Fitzhardinge 2009-02-11  109  
#endif
54321d94 arch/x86/include/asm/pgtable.h Jeremy Fitzhardinge 2009-02-11  110  
54321d94 arch/x86/include/asm/pgtable.h Jeremy Fitzhardinge 2009-02-11  111  
#define pte_val(x) native_pte_val(x)
54321d94 arch/x86/include/asm/pgtable.h Jeremy Fitzhardinge 2009-02-11  112  
#define __pte(x)   native_make_pte(x)
54321d94 arch/x86/include/asm/pgtable.h Jeremy Fitzhardinge 2009-02-11  113  
224101ed arch/x86/include/asm/pgtable.h Jeremy Fitzhardinge 2009-02-18  114  
#define arch_end_context_switch(prev)      do {} while(0)
224101ed arch/x86/include/asm/pgtable.h Jeremy Fitzhardinge 2009-02-18  115  
54321d94 arch/x86/include/asm/pgtable.h Jeremy Fitzhardinge 2009-02-11  116  
#endif     /* CONFIG_PARAVIRT */
54321d94 arch/x86/include/asm/pgtable.h Jeremy Fitzhardinge 2009-02-11  117  
8405b122 include/asm-x86/pgtable.h      Jeremy Fitzhardinge 2008-01-30  118  /*
4614139c include/asm-x86/pgtable.h      Jeremy Fitzhardinge 2008-01-30  119   * 
The following only work if pte_present() is true.
4614139c include/asm-x86/pgtable.h      Jeremy Fitzhardinge 2008-01-30  120   * 
Undefined behaviour if not..
4614139c include/asm-x86/pgtable.h      Jeremy Fitzhardinge 2008-01-30  121   */
3cbaeafe include/asm-x86/pgtable.h      Joe Perches         2008-03-23  122  
static inline int pte_dirty(pte_t pte)
3cbaeafe include/asm-x86/pgtable.h      Joe Perches         2008-03-23  123  {
a15af1c9 include/asm-x86/pgtable.h      Jeremy Fitzhardinge 2008-05-26  124     
return pte_flags(pte) & _PAGE_DIRTY;
3cbaeafe include/asm-x86/pgtable.h      Joe Perches         2008-03-23  125  }
3cbaeafe include/asm-x86/pgtable.h      Joe Perches         2008-03-23  126  
a927cb83 arch/x86/include/asm/pgtable.h Dave Hansen         2016-02-12  127  
a927cb83 arch/x86/include/asm/pgtable.h Dave Hansen         2016-02-12  128  
static inline u32 read_pkru(void)
a927cb83 arch/x86/include/asm/pgtable.h Dave Hansen         2016-02-12  129  {
a927cb83 arch/x86/include/asm/pgtable.h Dave Hansen         2016-02-12 @130     
if (boot_cpu_has(X86_FEATURE_OSPKE))
a927cb83 arch/x86/include/asm/pgtable.h Dave Hansen         2016-02-12 @131     
        return __read_pkru();
a927cb83 arch/x86/include/asm/pgtable.h Dave Hansen         2016-02-12  132     
return 0;
a927cb83 arch/x86/include/asm/pgtable.h Dave Hansen         2016-02-12  133  }
a927cb83 arch/x86/include/asm/pgtable.h Dave Hansen         2016-02-12  134  
9e90199c arch/x86/include/asm/pgtable.h Xiao Guangrong      2016-03-22  135  
static inline void write_pkru(u32 pkru)
9e90199c arch/x86/include/asm/pgtable.h Xiao Guangrong      2016-03-22  136  {
9e90199c arch/x86/include/asm/pgtable.h Xiao Guangrong      2016-03-22  137     
if (boot_cpu_has(X86_FEATURE_OSPKE))
9e90199c arch/x86/include/asm/pgtable.h Xiao Guangrong      2016-03-22 @138     
        __write_pkru(pkru);
9e90199c arch/x86/include/asm/pgtable.h Xiao Guangrong      2016-03-22  139  }
9e90199c arch/x86/include/asm/pgtable.h Xiao Guangrong      2016-03-22  140  

:::::: The code at line 131 was first introduced by commit
:::::: a927cb83f3300bcb1ae18672e58029acddd18b33 x86/mm/pkeys: Add functions to 
fetch PKRU

:::::: TO: Dave Hansen <[email protected]>
:::::: CC: Ingo Molnar <[email protected]>

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

Attachment: .config.gz
Description: application/gzip

Reply via email to