Hi Rodrigo,
I love your patch! Perhaps something to improve:
[auto build test WARNING on linus/master]
[also build test WARNING on v5.14-rc1 next-20210713]
[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]
url:
https://github.com/0day-ci/linux/commits/Rodrigo-Siqueira/drm-amd-display...
base:
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
7fef2edf7cc753b51f7ccc74993971b0a9c81eca
config: x86_64-randconfig-a014-20210713 (attached as .config)
compiler: gcc-9 (Debian 9.3.0-22) 9.3.0
reproduce (this is a W=1 build):
#
https://github.com/0day-ci/linux/commit/86e864484b1781d5a294660fe92f5ba96...
git remote add linux-review
https://github.com/0day-ci/linux
git fetch --no-tags linux-review
Rodrigo-Siqueira/drm-amd-display-Base-changes-for-isolating-FPU-operation-in-a-single-place/20210713-220840
git checkout 86e864484b1781d5a294660fe92f5ba9664d7612
# save the attached .config to linux build tree
make W=1 ARCH=x86_64
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All warnings (new ones prefixed by >>):
drivers/gpu/drm/amd/amdgpu/../display/amdgpu_dm/dc_fpu.c: In function
'dc_assert_fp_enabled':
> drivers/gpu/drm/amd/amdgpu/../display/amdgpu_dm/dc_fpu.c:57:7:
warning: variable 'pcpu' set but not used [-Wunused-but-set-variable]
57 | int *pcpu, depth = 0;
| ^~~~
vim +/pcpu +57 drivers/gpu/drm/amd/amdgpu/../display/amdgpu_dm/dc_fpu.c
43
44 /**
45 * dc_assert_fp_enabled - Check if FPU protection is enabled
46 *
47 * This function tells if the code is already under FPU protection or not. A
48 * function that works as an API for a set of FPU operations can use this
49 * function for checking if the caller invoked it after DC_FP_START(). For
50 * example, take a look at dcn2x.c file.
51 *
52 * Return:
53 * Return true if we already enabled FPU protection, otherwise return false.
54 */
55 inline bool dc_assert_fp_enabled(void)
56 {
57 int *pcpu, depth = 0;
58
59 pcpu = get_cpu_ptr(&fpu_recursion_depth);
60 depth = this_cpu_read(fpu_recursion_depth);
61 put_cpu_ptr(&fpu_recursion_depth);
62
63 return depth > 1;
64 }
65
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org