From: kernel test robot <lkp@intel.com>
To: Jani Nikula <jani.nikula@intel.com>, dri-devel@lists.freedesktop.org
Cc: oe-kbuild-all@lists.linux.dev, intel-gfx@lists.freedesktop.org,
jani.nikula@intel.com, Daniel Vetter <daniel@ffwll.ch>,
David Airlie <airlied@gmail.com>,
Maarten Lankhorst <maarten.lankhorst@linux.intel.com>,
Maxime Ripard <mripard@kernel.org>,
Thomas Zimmermann <tzimmermann@suse.de>,
Masahiro Yamada <masahiroy@kernel.org>
Subject: Re: [PATCH 2/2] drm: ensure drm headers are self-contained and pass kernel-doc
Date: Tue, 27 Aug 2024 00:05:06 +0800 [thread overview]
Message-ID: <202408262328.MknfI1MV-lkp@intel.com> (raw)
In-Reply-To: <20240823141110.3431423-2-jani.nikula@intel.com>
Hi Jani,
kernel test robot noticed the following build errors:
[auto build test ERROR on drm/drm-next]
[also build test ERROR on drm-misc/drm-misc-next next-20240826]
[cannot apply to linus/master v6.11-rc5]
[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#_base_tree_information]
url: https://github.com/intel-lab-lkp/linux/commits/Jani-Nikula/drm-ensure-drm-headers-are-self-contained-and-pass-kernel-doc/20240826-134953
base: git://anongit.freedesktop.org/drm/drm drm-next
patch link: https://lore.kernel.org/r/20240823141110.3431423-2-jani.nikula%40intel.com
patch subject: [PATCH 2/2] drm: ensure drm headers are self-contained and pass kernel-doc
config: sparc-allmodconfig (https://download.01.org/0day-ci/archive/20240826/202408262328.MknfI1MV-lkp@intel.com/config)
compiler: sparc64-linux-gcc (GCC) 14.1.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20240826/202408262328.MknfI1MV-lkp@intel.com/reproduce)
If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202408262328.MknfI1MV-lkp@intel.com/
All errors (new ones prefixed by >>):
In file included from arch/sparc/include/asm/pgtable_64.h:16,
from arch/sparc/include/asm/pgtable.h:5,
from include/linux/pgtable.h:6,
from include/drm/ttm/ttm_caching.h:28,
from <command-line>:
include/asm-generic/pgtable-nop4d.h:9:18: error: unknown type name 'pgd_t'; did you mean 'pid_t'?
9 | typedef struct { pgd_t pgd; } p4d_t;
| ^~~~~
| pid_t
include/asm-generic/pgtable-nop4d.h:21:28: error: unknown type name 'pgd_t'; did you mean 'p4d_t'?
21 | static inline int pgd_none(pgd_t pgd) { return 0; }
| ^~~~~
| p4d_t
include/asm-generic/pgtable-nop4d.h:22:27: error: unknown type name 'pgd_t'; did you mean 'p4d_t'?
22 | static inline int pgd_bad(pgd_t pgd) { return 0; }
| ^~~~~
| p4d_t
include/asm-generic/pgtable-nop4d.h:23:31: error: unknown type name 'pgd_t'; did you mean 'p4d_t'?
23 | static inline int pgd_present(pgd_t pgd) { return 1; }
| ^~~~~
| p4d_t
include/asm-generic/pgtable-nop4d.h:24:30: error: unknown type name 'pgd_t'; did you mean 'p4d_t'?
24 | static inline void pgd_clear(pgd_t *pgd) { }
| ^~~~~
| p4d_t
include/asm-generic/pgtable-nop4d.h:35:33: error: unknown type name 'pgd_t'; did you mean 'p4d_t'?
35 | static inline p4d_t *p4d_offset(pgd_t *pgd, unsigned long address)
| ^~~~~
| p4d_t
In file included from arch/sparc/include/asm/page.h:8,
from arch/sparc/include/asm/pgtable_64.h:23:
include/linux/pgtable.h: In function 'pud_offset':
arch/sparc/include/asm/page_64.h:79:29: error: request for member 'pgd' in something not a structure or union
79 | #define pgd_val(x) ((x).pgd)
| ^
arch/sparc/include/asm/page_64.h:147:60: note: in definition of macro '__va'
147 | #define __va(x) ((void *)((unsigned long) (x) + PAGE_OFFSET))
| ^
include/asm-generic/pgtable-nop4d.h:40:50: note: in expansion of macro 'pgd_val'
40 | #define p4d_val(x) (pgd_val((x).pgd))
| ^~~~~~~
arch/sparc/include/asm/pgtable_64.h:863:25: note: in expansion of macro 'p4d_val'
863 | ((pud_t *) __va(p4d_val(p4d)))
| ^~~~~~~
include/linux/pgtable.h:133:16: note: in expansion of macro 'p4d_pgtable'
133 | return p4d_pgtable(*p4d) + pud_index(address);
| ^~~~~~~~~~~
include/linux/pgtable.h: In function 'pmd_off':
include/linux/pgtable.h:165:38: error: implicit declaration of function 'p4d_offset'; did you mean 'pud_offset'? [-Wimplicit-function-declaration]
165 | return pmd_offset(pud_offset(p4d_offset(pgd_offset(mm, va), va), va), va);
| ^~~~~~~~~~
| pud_offset
>> include/linux/pgtable.h:165:38: error: passing argument 1 of 'pud_offset' makes pointer from integer without a cast [-Wint-conversion]
165 | return pmd_offset(pud_offset(p4d_offset(pgd_offset(mm, va), va), va), va);
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
| |
| int
include/linux/pgtable.h:131:40: note: expected 'p4d_t *' but argument is of type 'int'
131 | static inline pud_t *pud_offset(p4d_t *p4d, unsigned long address)
| ~~~~~~~^~~
include/linux/pgtable.h: In function 'pmd_off_k':
include/linux/pgtable.h:170:38: error: passing argument 1 of 'pud_offset' makes pointer from integer without a cast [-Wint-conversion]
170 | return pmd_offset(pud_offset(p4d_offset(pgd_offset_k(va), va), va), va);
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
| |
| int
include/linux/pgtable.h:131:40: note: expected 'p4d_t *' but argument is of type 'int'
131 | static inline pud_t *pud_offset(p4d_t *p4d, unsigned long address)
| ~~~~~~~^~~
include/linux/pgtable.h: In function 'p4d_same':
arch/sparc/include/asm/page_64.h:79:29: error: request for member 'pgd' in something not a structure or union
79 | #define pgd_val(x) ((x).pgd)
| ^
include/asm-generic/pgtable-nop4d.h:40:50: note: in expansion of macro 'pgd_val'
40 | #define p4d_val(x) (pgd_val((x).pgd))
| ^~~~~~~
include/linux/pgtable.h:1042:16: note: in expansion of macro 'p4d_val'
1042 | return p4d_val(p4d_a) == p4d_val(p4d_b);
| ^~~~~~~
arch/sparc/include/asm/page_64.h:79:29: error: request for member 'pgd' in something not a structure or union
79 | #define pgd_val(x) ((x).pgd)
| ^
include/asm-generic/pgtable-nop4d.h:40:50: note: in expansion of macro 'pgd_val'
40 | #define p4d_val(x) (pgd_val((x).pgd))
| ^~~~~~~
include/linux/pgtable.h:1042:34: note: in expansion of macro 'p4d_val'
1042 | return p4d_val(p4d_a) == p4d_val(p4d_b);
| ^~~~~~~
include/linux/pgtable.h:1040:34: warning: parameter 'p4d_a' set but not used [-Wunused-but-set-parameter]
1040 | static inline int p4d_same(p4d_t p4d_a, p4d_t p4d_b)
| ~~~~~~^~~~~
include/linux/pgtable.h:1040:47: warning: parameter 'p4d_b' set but not used [-Wunused-but-set-parameter]
1040 | static inline int p4d_same(p4d_t p4d_a, p4d_t p4d_b)
| ~~~~~~^~~~~
include/linux/pgtable.h: In function 'pgd_none_or_clear_bad':
include/linux/pgtable.h:1240:13: error: implicit declaration of function 'pgd_none'; did you mean 'p4d_none'? [-Wimplicit-function-declaration]
1240 | if (pgd_none(*pgd))
| ^~~~~~~~
| p4d_none
In file included from arch/sparc/include/asm/pgtable_64.h:17:
include/linux/pgtable.h:1242:22: error: implicit declaration of function 'pgd_bad'; did you mean 'pmd_bad'? [-Wimplicit-function-declaration]
1242 | if (unlikely(pgd_bad(*pgd))) {
| ^~~~~~~
include/linux/compiler.h:77:45: note: in definition of macro 'unlikely'
77 | # define unlikely(x) __builtin_expect(!!(x), 0)
| ^
include/linux/pgtable.h: In function 'p4d_none_or_clear_bad':
arch/sparc/include/asm/page_64.h:79:29: error: request for member 'pgd' in something not a structure or union
79 | #define pgd_val(x) ((x).pgd)
| ^
include/asm-generic/pgtable-nop4d.h:40:50: note: in expansion of macro 'pgd_val'
40 | #define p4d_val(x) (pgd_val((x).pgd))
| ^~~~~~~
arch/sparc/include/asm/pgtable_64.h:813:43: note: in expansion of macro 'p4d_val'
813 | #define p4d_none(p4d) (!p4d_val(p4d))
| ^~~~~~~
include/linux/pgtable.h:1251:13: note: in expansion of macro 'p4d_none'
1251 | if (p4d_none(*p4d))
| ^~~~~~~~
arch/sparc/include/asm/page_64.h:79:29: error: request for member 'pgd' in something not a structure or union
79 | #define pgd_val(x) ((x).pgd)
| ^
include/linux/compiler.h:77:45: note: in definition of macro 'unlikely'
77 | # define unlikely(x) __builtin_expect(!!(x), 0)
| ^
include/asm-generic/pgtable-nop4d.h:40:50: note: in expansion of macro 'pgd_val'
40 | #define p4d_val(x) (pgd_val((x).pgd))
| ^~~~~~~
arch/sparc/include/asm/pgtable_64.h:815:42: note: in expansion of macro 'p4d_val'
815 | #define p4d_bad(p4d) (p4d_val(p4d) & ~PAGE_MASK)
| ^~~~~~~
include/linux/pgtable.h:1253:22: note: in expansion of macro 'p4d_bad'
1253 | if (unlikely(p4d_bad(*p4d))) {
| ^~~~~~~
vim +/pud_offset +165 include/linux/pgtable.h
974b9b2c68f3d3 Mike Rapoport 2020-06-08 155
e05c7b1f2bc4b7 Mike Rapoport 2020-06-08 156 /*
e05c7b1f2bc4b7 Mike Rapoport 2020-06-08 157 * In many cases it is known that a virtual address is mapped at PMD or PTE
e05c7b1f2bc4b7 Mike Rapoport 2020-06-08 158 * level, so instead of traversing all the page table levels, we can get a
e05c7b1f2bc4b7 Mike Rapoport 2020-06-08 159 * pointer to the PMD entry in user or kernel page table or translate a virtual
e05c7b1f2bc4b7 Mike Rapoport 2020-06-08 160 * address to the pointer in the PTE in the kernel page tables with simple
e05c7b1f2bc4b7 Mike Rapoport 2020-06-08 161 * helpers.
e05c7b1f2bc4b7 Mike Rapoport 2020-06-08 162 */
e05c7b1f2bc4b7 Mike Rapoport 2020-06-08 163 static inline pmd_t *pmd_off(struct mm_struct *mm, unsigned long va)
e05c7b1f2bc4b7 Mike Rapoport 2020-06-08 164 {
e05c7b1f2bc4b7 Mike Rapoport 2020-06-08 @165 return pmd_offset(pud_offset(p4d_offset(pgd_offset(mm, va), va), va), va);
e05c7b1f2bc4b7 Mike Rapoport 2020-06-08 166 }
e05c7b1f2bc4b7 Mike Rapoport 2020-06-08 167
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
next prev parent reply other threads:[~2024-08-26 16:05 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-08-23 14:11 [PATCH 1/2] drm/ttm: fix kernel-doc typo for @trylock_only Jani Nikula
2024-08-23 14:11 ` [PATCH 2/2] drm: ensure drm headers are self-contained and pass kernel-doc Jani Nikula
2024-08-26 16:05 ` kernel test robot [this message]
2024-08-26 22:28 ` kernel test robot
2024-08-23 14:12 ` [PATCH 1/2] drm/ttm: fix kernel-doc typo for @trylock_only Christian König
2024-08-26 10:20 ` Jani Nikula
2024-08-23 14:44 ` ✗ Fi.CI.CHECKPATCH: warning for series starting with [1/2] " Patchwork
2024-08-23 14:51 ` ✓ Fi.CI.BAT: success " Patchwork
2024-08-24 13:38 ` ✗ Fi.CI.IGT: failure " Patchwork
-- strict thread matches above, loose matches on Subject: below --
2025-01-22 14:41 [PATCH 0/2] drm: add header tests Jani Nikula
2025-01-22 14:41 ` [PATCH 2/2] drm: ensure drm headers are self-contained and pass kernel-doc Jani Nikula
2025-03-02 16:01 ` Masahiro Yamada
2025-03-03 10:02 ` Jani Nikula
2025-03-03 12:59 ` Masahiro Yamada
2025-03-03 13:52 ` Jani Nikula
2025-03-04 18:05 ` Masahiro Yamada
2025-03-05 13:59 ` Maxime Ripard
2025-03-08 17:05 ` Masahiro Yamada
2025-03-13 10:59 ` Jani Nikula
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=202408262328.MknfI1MV-lkp@intel.com \
--to=lkp@intel.com \
--cc=airlied@gmail.com \
--cc=daniel@ffwll.ch \
--cc=dri-devel@lists.freedesktop.org \
--cc=intel-gfx@lists.freedesktop.org \
--cc=jani.nikula@intel.com \
--cc=maarten.lankhorst@linux.intel.com \
--cc=masahiroy@kernel.org \
--cc=mripard@kernel.org \
--cc=oe-kbuild-all@lists.linux.dev \
--cc=tzimmermann@suse.de \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox