From: kernel test robot <lkp@intel.com>
To: Maarten Lankhorst <dev@lankhorst.se>,
intel-xe@lists.freedesktop.org, intel-gfx@lists.freedesktop.org
Cc: oe-kbuild-all@lists.linux.dev
Subject: Re: [i915-rt v4 19/20] drm/i915/display: Do not take uncore lock in i915_get_vblank_counter
Date: Sat, 20 Dec 2025 16:20:46 +0100 [thread overview]
Message-ID: <202512201653.RtK7HcqE-lkp@intel.com> (raw)
In-Reply-To: <20251218163408.97508-20-dev@lankhorst.se>
Hi Maarten,
kernel test robot noticed the following build errors:
[auto build test ERROR on drm-i915/for-linux-next]
[also build test ERROR on drm-i915/for-linux-next-fixes drm-xe/drm-xe-next drm-tip/drm-tip linus/master v6.19-rc1 next-20251219]
[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/Maarten-Lankhorst/drm-i915-display-Fix-intel_lpe_audio_irq_handler-for-PREEMPT-RT/20251219-044020
base: https://gitlab.freedesktop.org/drm/i915/kernel.git for-linux-next
patch link: https://lore.kernel.org/r/20251218163408.97508-20-dev%40lankhorst.se
patch subject: [i915-rt v4 19/20] drm/i915/display: Do not take uncore lock in i915_get_vblank_counter
config: x86_64-rhel-9.4 (https://download.01.org/0day-ci/archive/20251220/202512201653.RtK7HcqE-lkp@intel.com/config)
compiler: gcc-14 (Debian 14.2.0-19) 14.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20251220/202512201653.RtK7HcqE-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/202512201653.RtK7HcqE-lkp@intel.com/
All error/warnings (new ones prefixed by >>):
In file included from drivers/gpu/drm/i915/gt/intel_engine_types.h:26,
from drivers/gpu/drm/i915/gt/intel_context_types.h:17,
from drivers/gpu/drm/i915/gem/i915_gem_context_types.h:19,
from drivers/gpu/drm/i915/gem/i915_gem_context.h:9,
from drivers/gpu/drm/i915/i915_driver.c:75:
drivers/gpu/drm/i915/intel_uncore.h: In function 'intel_uncore_read64_2x32_fw':
>> drivers/gpu/drm/i915/intel_uncore.h:455:19: error: expected '=', ',', ';', 'asm' or '__attribute__' before 'lower'
455 | u32 upper lower, old_upper, loop = 0;
| ^~~~~
>> drivers/gpu/drm/i915/intel_uncore.h:455:19: error: 'lower' undeclared (first use in this function); did you mean 'tolower'?
455 | u32 upper lower, old_upper, loop = 0;
| ^~~~~
| tolower
drivers/gpu/drm/i915/intel_uncore.h:455:19: note: each undeclared identifier is reported only once for each function it appears in
>> drivers/gpu/drm/i915/intel_uncore.h:455:26: error: 'old_upper' undeclared (first use in this function); did you mean 'si_upper'?
455 | u32 upper lower, old_upper, loop = 0;
| ^~~~~~~~~
| si_upper
>> drivers/gpu/drm/i915/intel_uncore.h:455:24: warning: left-hand operand of comma expression has no effect [-Wunused-value]
455 | u32 upper lower, old_upper, loop = 0;
| ^
>> drivers/gpu/drm/i915/intel_uncore.h:455:37: error: 'loop' undeclared (first use in this function)
455 | u32 upper lower, old_upper, loop = 0;
| ^~~~
drivers/gpu/drm/i915/intel_uncore.h:455:35: warning: left-hand operand of comma expression has no effect [-Wunused-value]
455 | u32 upper lower, old_upper, loop = 0;
| ^
>> drivers/gpu/drm/i915/intel_uncore.h:456:9: error: 'upper' undeclared (first use in this function)
456 | upper = intel_uncore_read_fw(uncoree, upper_reg);
| ^~~~~
>> drivers/gpu/drm/i915/intel_uncore.h:456:38: error: 'uncoree' undeclared (first use in this function); did you mean 'uncore'?
456 | upper = intel_uncore_read_fw(uncoree, upper_reg);
| ^~~~~~~
drivers/gpu/drm/i915/intel_uncore.h:425:55: note: in definition of macro 'intel_uncore_read_fw'
425 | #define intel_uncore_read_fw(...) __raw_uncore_read32(__VA_ARGS__)
| ^~~~~~~~~~~
>> drivers/gpu/drm/i915/intel_uncore.h:456:47: error: 'upper_reg' undeclared (first use in this function)
456 | upper = intel_uncore_read_fw(uncoree, upper_reg);
| ^~~~~~~~~
drivers/gpu/drm/i915/intel_uncore.h:425:55: note: in definition of macro 'intel_uncore_read_fw'
425 | #define intel_uncore_read_fw(...) __raw_uncore_read32(__VA_ARGS__)
| ^~~~~~~~~~~
>> drivers/gpu/drm/i915/intel_uncore.h:459:54: error: 'lower_reg' undeclared (first use in this function)
459 | lower = intel_uncore_read_fw(uncore, lower_reg);
| ^~~~~~~~~
drivers/gpu/drm/i915/intel_uncore.h:425:55: note: in definition of macro 'intel_uncore_read_fw'
425 | #define intel_uncore_read_fw(...) __raw_uncore_read32(__VA_ARGS__)
| ^~~~~~~~~~~
drivers/gpu/drm/i915/intel_uncore.h: In function 'intel_uncore_read64_2x32':
>> drivers/gpu/drm/i915/intel_uncore.h:483:15: error: too many arguments to function 'intel_uncore_read64_2x32_fw'
483 | ret = intel_uncore_read64_2x32_fw(uncore, lower_reg, upper_reg);
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/gpu/drm/i915/intel_uncore.h:453:1: note: declared here
453 | intel_uncore_read64_2x32_fw(struct intel_uncore *uncore)
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~
--
In file included from drivers/gpu/drm/i915/gt/intel_engine_types.h:26,
from drivers/gpu/drm/i915/gt/intel_context_types.h:17,
from drivers/gpu/drm/i915/gem/i915_gem_context_types.h:19,
from drivers/gpu/drm/i915/gem/i915_gem_context.h:9,
from drivers/gpu/drm/i915/i915_drm_client.c:14:
drivers/gpu/drm/i915/intel_uncore.h: In function 'intel_uncore_read64_2x32_fw':
>> drivers/gpu/drm/i915/intel_uncore.h:455:19: error: expected '=', ',', ';', 'asm' or '__attribute__' before 'lower'
455 | u32 upper lower, old_upper, loop = 0;
| ^~~~~
drivers/gpu/drm/i915/intel_uncore.h:455:19: error: 'lower' undeclared (first use in this function); did you mean 'islower'?
455 | u32 upper lower, old_upper, loop = 0;
| ^~~~~
| islower
drivers/gpu/drm/i915/intel_uncore.h:455:19: note: each undeclared identifier is reported only once for each function it appears in
>> drivers/gpu/drm/i915/intel_uncore.h:455:26: error: 'old_upper' undeclared (first use in this function); did you mean 'si_upper'?
455 | u32 upper lower, old_upper, loop = 0;
| ^~~~~~~~~
| si_upper
>> drivers/gpu/drm/i915/intel_uncore.h:455:24: warning: left-hand operand of comma expression has no effect [-Wunused-value]
455 | u32 upper lower, old_upper, loop = 0;
| ^
>> drivers/gpu/drm/i915/intel_uncore.h:455:37: error: 'loop' undeclared (first use in this function)
455 | u32 upper lower, old_upper, loop = 0;
| ^~~~
drivers/gpu/drm/i915/intel_uncore.h:455:35: warning: left-hand operand of comma expression has no effect [-Wunused-value]
455 | u32 upper lower, old_upper, loop = 0;
| ^
>> drivers/gpu/drm/i915/intel_uncore.h:456:9: error: 'upper' undeclared (first use in this function)
456 | upper = intel_uncore_read_fw(uncoree, upper_reg);
| ^~~~~
>> drivers/gpu/drm/i915/intel_uncore.h:456:38: error: 'uncoree' undeclared (first use in this function); did you mean 'uncore'?
456 | upper = intel_uncore_read_fw(uncoree, upper_reg);
| ^~~~~~~
drivers/gpu/drm/i915/intel_uncore.h:425:55: note: in definition of macro 'intel_uncore_read_fw'
425 | #define intel_uncore_read_fw(...) __raw_uncore_read32(__VA_ARGS__)
| ^~~~~~~~~~~
>> drivers/gpu/drm/i915/intel_uncore.h:456:47: error: 'upper_reg' undeclared (first use in this function)
456 | upper = intel_uncore_read_fw(uncoree, upper_reg);
| ^~~~~~~~~
drivers/gpu/drm/i915/intel_uncore.h:425:55: note: in definition of macro 'intel_uncore_read_fw'
425 | #define intel_uncore_read_fw(...) __raw_uncore_read32(__VA_ARGS__)
| ^~~~~~~~~~~
>> drivers/gpu/drm/i915/intel_uncore.h:459:54: error: 'lower_reg' undeclared (first use in this function)
459 | lower = intel_uncore_read_fw(uncore, lower_reg);
| ^~~~~~~~~
drivers/gpu/drm/i915/intel_uncore.h:425:55: note: in definition of macro 'intel_uncore_read_fw'
425 | #define intel_uncore_read_fw(...) __raw_uncore_read32(__VA_ARGS__)
| ^~~~~~~~~~~
drivers/gpu/drm/i915/intel_uncore.h: In function 'intel_uncore_read64_2x32':
>> drivers/gpu/drm/i915/intel_uncore.h:483:15: error: too many arguments to function 'intel_uncore_read64_2x32_fw'
483 | ret = intel_uncore_read64_2x32_fw(uncore, lower_reg, upper_reg);
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/gpu/drm/i915/intel_uncore.h:453:1: note: declared here
453 | intel_uncore_read64_2x32_fw(struct intel_uncore *uncore)
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~
--
In file included from drivers/gpu/drm/i915/gt/intel_engine_types.h:26,
from drivers/gpu/drm/i915/gt/intel_context_types.h:17,
from drivers/gpu/drm/i915/gem/i915_gem_context_types.h:19,
from drivers/gpu/drm/i915/i915_drv.h:40,
from drivers/gpu/drm/i915/i915_ioctl.c:8:
drivers/gpu/drm/i915/intel_uncore.h: In function 'intel_uncore_read64_2x32_fw':
>> drivers/gpu/drm/i915/intel_uncore.h:455:19: error: expected '=', ',', ';', 'asm' or '__attribute__' before 'lower'
455 | u32 upper lower, old_upper, loop = 0;
| ^~~~~
drivers/gpu/drm/i915/intel_uncore.h:455:19: error: 'lower' undeclared (first use in this function); did you mean 'islower'?
455 | u32 upper lower, old_upper, loop = 0;
| ^~~~~
| islower
drivers/gpu/drm/i915/intel_uncore.h:455:19: note: each undeclared identifier is reported only once for each function it appears in
>> drivers/gpu/drm/i915/intel_uncore.h:455:26: error: 'old_upper' undeclared (first use in this function); did you mean 'si_upper'?
455 | u32 upper lower, old_upper, loop = 0;
| ^~~~~~~~~
| si_upper
>> drivers/gpu/drm/i915/intel_uncore.h:455:24: warning: left-hand operand of comma expression has no effect [-Wunused-value]
455 | u32 upper lower, old_upper, loop = 0;
| ^
>> drivers/gpu/drm/i915/intel_uncore.h:455:37: error: 'loop' undeclared (first use in this function)
455 | u32 upper lower, old_upper, loop = 0;
| ^~~~
drivers/gpu/drm/i915/intel_uncore.h:455:35: warning: left-hand operand of comma expression has no effect [-Wunused-value]
455 | u32 upper lower, old_upper, loop = 0;
| ^
>> drivers/gpu/drm/i915/intel_uncore.h:456:9: error: 'upper' undeclared (first use in this function)
456 | upper = intel_uncore_read_fw(uncoree, upper_reg);
| ^~~~~
>> drivers/gpu/drm/i915/intel_uncore.h:456:38: error: 'uncoree' undeclared (first use in this function); did you mean 'uncore'?
456 | upper = intel_uncore_read_fw(uncoree, upper_reg);
| ^~~~~~~
drivers/gpu/drm/i915/intel_uncore.h:425:55: note: in definition of macro 'intel_uncore_read_fw'
425 | #define intel_uncore_read_fw(...) __raw_uncore_read32(__VA_ARGS__)
| ^~~~~~~~~~~
>> drivers/gpu/drm/i915/intel_uncore.h:456:47: error: 'upper_reg' undeclared (first use in this function)
456 | upper = intel_uncore_read_fw(uncoree, upper_reg);
| ^~~~~~~~~
drivers/gpu/drm/i915/intel_uncore.h:425:55: note: in definition of macro 'intel_uncore_read_fw'
425 | #define intel_uncore_read_fw(...) __raw_uncore_read32(__VA_ARGS__)
| ^~~~~~~~~~~
>> drivers/gpu/drm/i915/intel_uncore.h:459:54: error: 'lower_reg' undeclared (first use in this function)
459 | lower = intel_uncore_read_fw(uncore, lower_reg);
| ^~~~~~~~~
drivers/gpu/drm/i915/intel_uncore.h:425:55: note: in definition of macro 'intel_uncore_read_fw'
425 | #define intel_uncore_read_fw(...) __raw_uncore_read32(__VA_ARGS__)
| ^~~~~~~~~~~
drivers/gpu/drm/i915/intel_uncore.h: In function 'intel_uncore_read64_2x32':
>> drivers/gpu/drm/i915/intel_uncore.h:483:15: error: too many arguments to function 'intel_uncore_read64_2x32_fw'
483 | ret = intel_uncore_read64_2x32_fw(uncore, lower_reg, upper_reg);
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/gpu/drm/i915/intel_uncore.h:453:1: note: declared here
453 | intel_uncore_read64_2x32_fw(struct intel_uncore *uncore)
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/gpu/drm/i915/intel_uncore.h: In function 'intel_uncore_read64_2x32_fw':
>> drivers/gpu/drm/i915/intel_uncore.h:464:1: warning: control reaches end of non-void function [-Wreturn-type]
464 | }
| ^
--
In file included from drivers/gpu/drm/i915/display/intel_de.h:12,
from drivers/gpu/drm/i915/display/hsw_ips.c:13:
drivers/gpu/drm/i915/intel_uncore.h: In function 'intel_uncore_read64_2x32_fw':
>> drivers/gpu/drm/i915/intel_uncore.h:455:19: error: expected '=', ',', ';', 'asm' or '__attribute__' before 'lower'
455 | u32 upper lower, old_upper, loop = 0;
| ^~~~~
>> drivers/gpu/drm/i915/intel_uncore.h:455:19: error: 'lower' undeclared (first use in this function); did you mean 'tolower'?
455 | u32 upper lower, old_upper, loop = 0;
| ^~~~~
| tolower
drivers/gpu/drm/i915/intel_uncore.h:455:19: note: each undeclared identifier is reported only once for each function it appears in
>> drivers/gpu/drm/i915/intel_uncore.h:455:26: error: 'old_upper' undeclared (first use in this function); did you mean 'si_upper'?
455 | u32 upper lower, old_upper, loop = 0;
| ^~~~~~~~~
| si_upper
>> drivers/gpu/drm/i915/intel_uncore.h:455:24: warning: left-hand operand of comma expression has no effect [-Wunused-value]
455 | u32 upper lower, old_upper, loop = 0;
| ^
>> drivers/gpu/drm/i915/intel_uncore.h:455:37: error: 'loop' undeclared (first use in this function)
455 | u32 upper lower, old_upper, loop = 0;
| ^~~~
drivers/gpu/drm/i915/intel_uncore.h:455:35: warning: left-hand operand of comma expression has no effect [-Wunused-value]
455 | u32 upper lower, old_upper, loop = 0;
| ^
>> drivers/gpu/drm/i915/intel_uncore.h:456:9: error: 'upper' undeclared (first use in this function)
456 | upper = intel_uncore_read_fw(uncoree, upper_reg);
| ^~~~~
>> drivers/gpu/drm/i915/intel_uncore.h:456:38: error: 'uncoree' undeclared (first use in this function); did you mean 'uncore'?
456 | upper = intel_uncore_read_fw(uncoree, upper_reg);
| ^~~~~~~
drivers/gpu/drm/i915/intel_uncore.h:425:55: note: in definition of macro 'intel_uncore_read_fw'
425 | #define intel_uncore_read_fw(...) __raw_uncore_read32(__VA_ARGS__)
| ^~~~~~~~~~~
>> drivers/gpu/drm/i915/intel_uncore.h:456:47: error: 'upper_reg' undeclared (first use in this function)
456 | upper = intel_uncore_read_fw(uncoree, upper_reg);
| ^~~~~~~~~
drivers/gpu/drm/i915/intel_uncore.h:425:55: note: in definition of macro 'intel_uncore_read_fw'
425 | #define intel_uncore_read_fw(...) __raw_uncore_read32(__VA_ARGS__)
| ^~~~~~~~~~~
>> drivers/gpu/drm/i915/intel_uncore.h:459:54: error: 'lower_reg' undeclared (first use in this function)
459 | lower = intel_uncore_read_fw(uncore, lower_reg);
| ^~~~~~~~~
drivers/gpu/drm/i915/intel_uncore.h:425:55: note: in definition of macro 'intel_uncore_read_fw'
425 | #define intel_uncore_read_fw(...) __raw_uncore_read32(__VA_ARGS__)
| ^~~~~~~~~~~
drivers/gpu/drm/i915/intel_uncore.h: In function 'intel_uncore_read64_2x32':
>> drivers/gpu/drm/i915/intel_uncore.h:483:15: error: too many arguments to function 'intel_uncore_read64_2x32_fw'
483 | ret = intel_uncore_read64_2x32_fw(uncore, lower_reg, upper_reg);
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/gpu/drm/i915/intel_uncore.h:453:1: note: declared here
453 | intel_uncore_read64_2x32_fw(struct intel_uncore *uncore)
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/gpu/drm/i915/display/intel_de.h: In function 'intel_de_read64_2x32_fw':
>> drivers/gpu/drm/i915/display/intel_de.h:70:16: error: too many arguments to function 'intel_uncore_read64_2x32_fw'
70 | return intel_uncore_read64_2x32_fw(__to_uncore(display),
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/gpu/drm/i915/intel_uncore.h:453:1: note: declared here
453 | intel_uncore_read64_2x32_fw(struct intel_uncore *uncore)
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~
--
In file included from drivers/gpu/drm/i915/gt/intel_engine_types.h:26,
from drivers/gpu/drm/i915/gt/intel_context_types.h:17,
from drivers/gpu/drm/i915/gem/i915_gem_context_types.h:19,
from drivers/gpu/drm/i915/i915_drv.h:40,
from drivers/gpu/drm/i915/display/i9xx_wm.c:10:
drivers/gpu/drm/i915/intel_uncore.h: In function 'intel_uncore_read64_2x32_fw':
>> drivers/gpu/drm/i915/intel_uncore.h:455:19: error: expected '=', ',', ';', 'asm' or '__attribute__' before 'lower'
455 | u32 upper lower, old_upper, loop = 0;
| ^~~~~
drivers/gpu/drm/i915/intel_uncore.h:455:19: error: 'lower' undeclared (first use in this function); did you mean 'islower'?
455 | u32 upper lower, old_upper, loop = 0;
| ^~~~~
| islower
drivers/gpu/drm/i915/intel_uncore.h:455:19: note: each undeclared identifier is reported only once for each function it appears in
>> drivers/gpu/drm/i915/intel_uncore.h:455:26: error: 'old_upper' undeclared (first use in this function); did you mean 'si_upper'?
455 | u32 upper lower, old_upper, loop = 0;
| ^~~~~~~~~
| si_upper
>> drivers/gpu/drm/i915/intel_uncore.h:455:24: warning: left-hand operand of comma expression has no effect [-Wunused-value]
455 | u32 upper lower, old_upper, loop = 0;
| ^
>> drivers/gpu/drm/i915/intel_uncore.h:455:37: error: 'loop' undeclared (first use in this function)
455 | u32 upper lower, old_upper, loop = 0;
| ^~~~
drivers/gpu/drm/i915/intel_uncore.h:455:35: warning: left-hand operand of comma expression has no effect [-Wunused-value]
455 | u32 upper lower, old_upper, loop = 0;
| ^
>> drivers/gpu/drm/i915/intel_uncore.h:456:9: error: 'upper' undeclared (first use in this function)
456 | upper = intel_uncore_read_fw(uncoree, upper_reg);
| ^~~~~
>> drivers/gpu/drm/i915/intel_uncore.h:456:38: error: 'uncoree' undeclared (first use in this function); did you mean 'uncore'?
456 | upper = intel_uncore_read_fw(uncoree, upper_reg);
| ^~~~~~~
drivers/gpu/drm/i915/intel_uncore.h:425:55: note: in definition of macro 'intel_uncore_read_fw'
425 | #define intel_uncore_read_fw(...) __raw_uncore_read32(__VA_ARGS__)
| ^~~~~~~~~~~
>> drivers/gpu/drm/i915/intel_uncore.h:456:47: error: 'upper_reg' undeclared (first use in this function)
456 | upper = intel_uncore_read_fw(uncoree, upper_reg);
| ^~~~~~~~~
drivers/gpu/drm/i915/intel_uncore.h:425:55: note: in definition of macro 'intel_uncore_read_fw'
425 | #define intel_uncore_read_fw(...) __raw_uncore_read32(__VA_ARGS__)
| ^~~~~~~~~~~
>> drivers/gpu/drm/i915/intel_uncore.h:459:54: error: 'lower_reg' undeclared (first use in this function)
459 | lower = intel_uncore_read_fw(uncore, lower_reg);
| ^~~~~~~~~
drivers/gpu/drm/i915/intel_uncore.h:425:55: note: in definition of macro 'intel_uncore_read_fw'
425 | #define intel_uncore_read_fw(...) __raw_uncore_read32(__VA_ARGS__)
| ^~~~~~~~~~~
drivers/gpu/drm/i915/intel_uncore.h: In function 'intel_uncore_read64_2x32':
>> drivers/gpu/drm/i915/intel_uncore.h:483:15: error: too many arguments to function 'intel_uncore_read64_2x32_fw'
483 | ret = intel_uncore_read64_2x32_fw(uncore, lower_reg, upper_reg);
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/gpu/drm/i915/intel_uncore.h:453:1: note: declared here
453 | intel_uncore_read64_2x32_fw(struct intel_uncore *uncore)
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~
In file included from drivers/gpu/drm/i915/display/i9xx_wm.c:16:
drivers/gpu/drm/i915/display/intel_de.h: In function 'intel_de_read64_2x32_fw':
>> drivers/gpu/drm/i915/display/intel_de.h:70:16: error: too many arguments to function 'intel_uncore_read64_2x32_fw'
70 | return intel_uncore_read64_2x32_fw(__to_uncore(display),
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/gpu/drm/i915/intel_uncore.h:453:1: note: declared here
453 | intel_uncore_read64_2x32_fw(struct intel_uncore *uncore)
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~
--
In file included from drivers/gpu/drm/i915/gt/intel_engine_types.h:26,
from drivers/gpu/drm/i915/gt/intel_context_types.h:17,
from drivers/gpu/drm/i915/gem/i915_gem_context_types.h:19,
from drivers/gpu/drm/i915/i915_drv.h:40,
from drivers/gpu/drm/i915/display/intel_vblank.c:11:
drivers/gpu/drm/i915/intel_uncore.h: In function 'intel_uncore_read64_2x32_fw':
>> drivers/gpu/drm/i915/intel_uncore.h:455:19: error: expected '=', ',', ';', 'asm' or '__attribute__' before 'lower'
455 | u32 upper lower, old_upper, loop = 0;
| ^~~~~
drivers/gpu/drm/i915/intel_uncore.h:455:19: error: 'lower' undeclared (first use in this function); did you mean 'islower'?
455 | u32 upper lower, old_upper, loop = 0;
| ^~~~~
| islower
drivers/gpu/drm/i915/intel_uncore.h:455:19: note: each undeclared identifier is reported only once for each function it appears in
>> drivers/gpu/drm/i915/intel_uncore.h:455:26: error: 'old_upper' undeclared (first use in this function); did you mean 'si_upper'?
455 | u32 upper lower, old_upper, loop = 0;
| ^~~~~~~~~
| si_upper
>> drivers/gpu/drm/i915/intel_uncore.h:455:24: warning: left-hand operand of comma expression has no effect [-Wunused-value]
455 | u32 upper lower, old_upper, loop = 0;
| ^
>> drivers/gpu/drm/i915/intel_uncore.h:455:37: error: 'loop' undeclared (first use in this function)
455 | u32 upper lower, old_upper, loop = 0;
| ^~~~
drivers/gpu/drm/i915/intel_uncore.h:455:35: warning: left-hand operand of comma expression has no effect [-Wunused-value]
455 | u32 upper lower, old_upper, loop = 0;
| ^
>> drivers/gpu/drm/i915/intel_uncore.h:456:9: error: 'upper' undeclared (first use in this function)
456 | upper = intel_uncore_read_fw(uncoree, upper_reg);
| ^~~~~
>> drivers/gpu/drm/i915/intel_uncore.h:456:38: error: 'uncoree' undeclared (first use in this function); did you mean 'uncore'?
456 | upper = intel_uncore_read_fw(uncoree, upper_reg);
| ^~~~~~~
drivers/gpu/drm/i915/intel_uncore.h:425:55: note: in definition of macro 'intel_uncore_read_fw'
425 | #define intel_uncore_read_fw(...) __raw_uncore_read32(__VA_ARGS__)
| ^~~~~~~~~~~
>> drivers/gpu/drm/i915/intel_uncore.h:456:47: error: 'upper_reg' undeclared (first use in this function)
456 | upper = intel_uncore_read_fw(uncoree, upper_reg);
| ^~~~~~~~~
drivers/gpu/drm/i915/intel_uncore.h:425:55: note: in definition of macro 'intel_uncore_read_fw'
425 | #define intel_uncore_read_fw(...) __raw_uncore_read32(__VA_ARGS__)
| ^~~~~~~~~~~
>> drivers/gpu/drm/i915/intel_uncore.h:459:54: error: 'lower_reg' undeclared (first use in this function)
459 | lower = intel_uncore_read_fw(uncore, lower_reg);
| ^~~~~~~~~
drivers/gpu/drm/i915/intel_uncore.h:425:55: note: in definition of macro 'intel_uncore_read_fw'
425 | #define intel_uncore_read_fw(...) __raw_uncore_read32(__VA_ARGS__)
| ^~~~~~~~~~~
drivers/gpu/drm/i915/intel_uncore.h: In function 'intel_uncore_read64_2x32':
>> drivers/gpu/drm/i915/intel_uncore.h:483:15: error: too many arguments to function 'intel_uncore_read64_2x32_fw'
483 | ret = intel_uncore_read64_2x32_fw(uncore, lower_reg, upper_reg);
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/gpu/drm/i915/intel_uncore.h:453:1: note: declared here
453 | intel_uncore_read64_2x32_fw(struct intel_uncore *uncore)
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~
In file included from drivers/gpu/drm/i915/display/intel_vblank.c:14:
drivers/gpu/drm/i915/display/intel_de.h: In function 'intel_de_read64_2x32_fw':
>> drivers/gpu/drm/i915/display/intel_de.h:70:16: error: too many arguments to function 'intel_uncore_read64_2x32_fw'
70 | return intel_uncore_read64_2x32_fw(__to_uncore(display),
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/gpu/drm/i915/intel_uncore.h:453:1: note: declared here
453 | intel_uncore_read64_2x32_fw(struct intel_uncore *uncore)
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/gpu/drm/i915/intel_uncore.h: In function 'intel_uncore_read64_2x32_fw':
>> drivers/gpu/drm/i915/intel_uncore.h:464:1: warning: control reaches end of non-void function [-Wreturn-type]
464 | }
| ^
..
vim +455 drivers/gpu/drm/i915/intel_uncore.h
451
452 static inline u64
453 intel_uncore_read64_2x32_fw(struct intel_uncore *uncore)
454 {
> 455 u32 upper lower, old_upper, loop = 0;
> 456 upper = intel_uncore_read_fw(uncoree, upper_reg);
457 do {
458 old_upper = upper;
> 459 lower = intel_uncore_read_fw(uncore, lower_reg);
460 upper = intel_uncore_read_fw(uncore, upper_reg);
461 } while (upper != old_upper && loop++ < 2);
462
463 return (u64)upper << 32 | lower;
> 464 }
465
466 static inline u64
467 intel_uncore_read64_2x32(struct intel_uncore *uncore,
468 i915_reg_t lower_reg, i915_reg_t upper_reg)
469 {
470 enum forcewake_domains fw_domains;
471 unsigned long flags;
472 u64 ret;
473
474 fw_domains = intel_uncore_forcewake_for_reg(uncore, lower_reg,
475 FW_REG_READ);
476
477 fw_domains |= intel_uncore_forcewake_for_reg(uncore, upper_reg,
478 FW_REG_READ);
479
480 spin_lock_irqsave(&uncore->lock, flags);
481 intel_uncore_forcewake_get__locked(uncore, fw_domains);
482
> 483 ret = intel_uncore_read64_2x32_fw(uncore, lower_reg, upper_reg);
484
485 intel_uncore_forcewake_put__locked(uncore, fw_domains);
486 spin_unlock_irqrestore(&uncore->lock, flags);
487
488 return ret;
489 }
490
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
next prev parent reply other threads:[~2025-12-20 15:21 UTC|newest]
Thread overview: 32+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-12-18 16:33 [i915-rt v4 00/20] drm/i915/display: All patches to make PREEMPT_RT work on i915 + xe Maarten Lankhorst
2025-12-18 16:33 ` [i915-rt v4 01/20] drm/i915/display: Fix intel_lpe_audio_irq_handler for PREEMPT-RT Maarten Lankhorst
2025-12-18 16:33 ` [i915-rt v4 02/20] drm/i915/display: Make get_vblank_counter use intel_de_read_fw() Maarten Lankhorst
2025-12-18 16:33 ` [i915-rt v4 03/20] drm/i915/display: Use intel_de_write_fw in intel_pipe_fastset Maarten Lankhorst
2025-12-18 16:33 ` [i915-rt v4 04/20] drm/i915/display: Make set_pipeconf use the fw variants Maarten Lankhorst
2025-12-18 16:33 ` [i915-rt v4 05/20] drm/i915/display: Move vblank put until after critical section Maarten Lankhorst
2025-12-18 16:33 ` [i915-rt v4 06/20] drm/i915/display: Remove locking from intel_vblank_evade " Maarten Lankhorst
2025-12-18 16:33 ` [i915-rt v4 07/20] drm/i915/display: Handle vlv dsi workaround in scanline_in_safe_range too Maarten Lankhorst
2025-12-18 16:33 ` [i915-rt v4 08/20] drm/i915/display: Make icl_dsi_frame_update use _fw too Maarten Lankhorst
2025-12-18 16:33 ` [i915-rt v4 09/20] drm/i915/display: Enable interrupts earlier on PREEMPT_RT Maarten Lankhorst
2025-12-18 16:33 ` [i915-rt v4 10/20] drm/i915: Use preempt_disable/enable_rt() where recommended Maarten Lankhorst
2025-12-18 16:33 ` [i915-rt v4 11/20] PREEMPT_RT injection Maarten Lankhorst
2025-12-18 16:34 ` [i915-rt v4 12/20] drm/i915/display: Use intel_de_read_fw in colorops Maarten Lankhorst
2025-12-18 16:34 ` [i915-rt v4 13/20] drm/i915/gt: Use spin_lock_irq() instead of local_irq_disable() + spin_lock() Maarten Lankhorst
2025-12-18 16:34 ` [i915-rt v4 14/20] drm/i915: Drop the irqs_disabled() check Maarten Lankhorst
2025-12-18 16:34 ` [i915-rt v4 15/20] drm/i915/guc: Consider also RCU depth in busy loop Maarten Lankhorst
2025-12-18 16:34 ` [i915-rt v4 16/20] Revert "drm/i915: Depend on !PREEMPT_RT." Maarten Lankhorst
2025-12-18 16:34 ` [i915-rt v4 17/20] drm/i915/gt: Fix selftests on PREEMPT_RT Maarten Lankhorst
2025-12-18 16:34 ` [i915-rt v4 18/20] drm/i915/gt: Set stop_timeout() correctly on PREEMPT-RT Maarten Lankhorst
2025-12-18 16:34 ` [i915-rt v4 19/20] drm/i915/display: Do not take uncore lock in i915_get_vblank_counter Maarten Lankhorst
2025-12-18 19:33 ` [i915-rt v4.1] " Maarten Lankhorst
2025-12-20 15:20 ` kernel test robot [this message]
2025-12-20 15:43 ` [i915-rt v4 19/20] " kernel test robot
2025-12-21 0:35 ` kernel test robot
2025-12-18 16:34 ` [i915-rt v4 20/20] drm/i915/display: Remove uncore lock from vlv_atomic_update_fifo Maarten Lankhorst
2025-12-18 21:06 ` [i915-rt v4.1] " Maarten Lankhorst
2025-12-18 21:50 ` [i915-rt v4.2] " Maarten Lankhorst
2025-12-20 18:23 ` [i915-rt v4 20/20] " kernel test robot
2025-12-18 17:42 ` ✗ Fi.CI.BUILD: failure for drm/i915/display: All patches to make PREEMPT_RT work on i915 + xe. (rev5) Patchwork
2025-12-18 20:08 ` ✗ Fi.CI.BUILD: failure for drm/i915/display: All patches to make PREEMPT_RT work on i915 + xe. (rev6) Patchwork
2025-12-18 21:28 ` ✗ Fi.CI.BUILD: failure for drm/i915/display: All patches to make PREEMPT_RT work on i915 + xe. (rev7) Patchwork
2025-12-18 23:31 ` ✗ i915.CI.BAT: failure for drm/i915/display: All patches to make PREEMPT_RT work on i915 + xe. (rev8) Patchwork
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=202512201653.RtK7HcqE-lkp@intel.com \
--to=lkp@intel.com \
--cc=dev@lankhorst.se \
--cc=intel-gfx@lists.freedesktop.org \
--cc=intel-xe@lists.freedesktop.org \
--cc=oe-kbuild-all@lists.linux.dev \
/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