All of lore.kernel.org
 help / color / mirror / Atom feed
From: kernel test robot <lkp@intel.com>
To: Philip Cox <Philip.Cox@amd.com>
Cc: Gerrit Cr <Gerrit.Cr@amd.com>,
	kbuild-all@lists.01.org, Felix Kuehling <Felix.Kuehling@amd.com>,
	dri-devel@lists.freedesktop.org,
	Rajneesh Bhardwaj <rajneesh.bhardwaj@amd.com>
Subject: [radeon-alex:amd-20.45 1587/2427] drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gfx_v10.c:882:6: warning: no previous prototype for 'kgd_gfx_v10_set_address_watch'
Date: Tue, 15 Dec 2020 13:50:02 +0800	[thread overview]
Message-ID: <202012151358.rsGfl3ws-lkp@intel.com> (raw)

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

tree:   git://people.freedesktop.org/~agd5f/linux.git amd-20.45
head:   a3950d94b046fb206e58fd3ec717f071c0203ba3
commit: abdc393ab8265df9db5d9d64eb33ed1c33e9d95c [1587/2427] drm/amdkfd: Add gfx10 address watch support
config: powerpc-allyesconfig (attached as .config)
compiler: powerpc64-linux-gcc (GCC) 9.3.0
reproduce (this is a W=1 build):
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        git remote add radeon-alex git://people.freedesktop.org/~agd5f/linux.git
        git fetch --no-tags radeon-alex amd-20.45
        git checkout abdc393ab8265df9db5d9d64eb33ed1c33e9d95c
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=powerpc 

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>

All warnings (new ones prefixed by >>):

   drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gfx_v10.c:763:6: warning: no previous prototype for 'kgd_gfx_v10_enable_debug_trap' [-Wmissing-prototypes]
     763 | void kgd_gfx_v10_enable_debug_trap(struct kgd_dev *kgd,
         |      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gfx_v10.c:792:6: warning: no previous prototype for 'kgd_gfx_v10_disable_debug_trap' [-Wmissing-prototypes]
     792 | void kgd_gfx_v10_disable_debug_trap(struct kgd_dev *kgd)
         |      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gfx_v10.c:803:5: warning: no previous prototype for 'kgd_gfx_v10_set_wave_launch_trap_override' [-Wmissing-prototypes]
     803 | int kgd_gfx_v10_set_wave_launch_trap_override(struct kgd_dev *kgd,
         |     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gfx_v10.c:852:6: warning: no previous prototype for 'kgd_gfx_v10_set_wave_launch_mode' [-Wmissing-prototypes]
     852 | void kgd_gfx_v10_set_wave_launch_mode(struct kgd_dev *kgd,
         |      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>> drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gfx_v10.c:882:6: warning: no previous prototype for 'kgd_gfx_v10_set_address_watch' [-Wmissing-prototypes]
     882 | void kgd_gfx_v10_set_address_watch(struct kgd_dev *kgd,
         |      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>> drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gfx_v10.c:942:6: warning: no previous prototype for 'kgd_gfx_v10_clear_address_watch' [-Wmissing-prototypes]
     942 | void kgd_gfx_v10_clear_address_watch(struct kgd_dev *kgd,
         |      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gfx_v10.c:967:6: warning: no previous prototype for 'kgd_gfx_v10_get_iq_wait_times' [-Wmissing-prototypes]
     967 | void kgd_gfx_v10_get_iq_wait_times(struct kgd_dev *kgd,
         |      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gfx_v10.c:976:6: warning: no previous prototype for 'kgd_gfx_v10_build_grace_period_packet_info' [-Wmissing-prototypes]
     976 | void kgd_gfx_v10_build_grace_period_packet_info(struct kgd_dev *kgd,
         |      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   In file included from drivers/gpu/drm/amd/display/dc/dc_types.h:33,
                    from drivers/gpu/drm/amd/display/dc/dm_services_types.h:30,
                    from drivers/gpu/drm/amd/include/dm_pp_interface.h:26,
                    from drivers/gpu/drm/amd/amdgpu/amdgpu.h:64,
                    from drivers/gpu/drm/amd/backport/include/kcl/kcl_amdgpu.h:6,
                    from drivers/gpu/drm/amd/backport/backport.h:18,
                    from <command-line>:
   drivers/gpu/drm/amd/display/include/fixed31_32.h:76:32: warning: 'dc_fixpt_ln2_div_2' defined but not used [-Wunused-const-variable=]
      76 | static const struct fixed31_32 dc_fixpt_ln2_div_2 = { 1488522236LL };
         |                                ^~~~~~~~~~~~~~~~~~
   drivers/gpu/drm/amd/display/include/fixed31_32.h:75:32: warning: 'dc_fixpt_ln2' defined but not used [-Wunused-const-variable=]
      75 | static const struct fixed31_32 dc_fixpt_ln2 = { 2977044471LL };
         |                                ^~~~~~~~~~~~
   drivers/gpu/drm/amd/display/include/fixed31_32.h:74:32: warning: 'dc_fixpt_e' defined but not used [-Wunused-const-variable=]
      74 | static const struct fixed31_32 dc_fixpt_e = { 11674931555LL };
         |                                ^~~~~~~~~~
   drivers/gpu/drm/amd/display/include/fixed31_32.h:73:32: warning: 'dc_fixpt_two_pi' defined but not used [-Wunused-const-variable=]
      73 | static const struct fixed31_32 dc_fixpt_two_pi = { 26986075409LL };
         |                                ^~~~~~~~~~~~~~~
   drivers/gpu/drm/amd/display/include/fixed31_32.h:72:32: warning: 'dc_fixpt_pi' defined but not used [-Wunused-const-variable=]
      72 | static const struct fixed31_32 dc_fixpt_pi = { 13493037705LL };
         |                                ^~~~~~~~~~~
   drivers/gpu/drm/amd/display/include/fixed31_32.h:67:32: warning: 'dc_fixpt_zero' defined but not used [-Wunused-const-variable=]
      67 | static const struct fixed31_32 dc_fixpt_zero = { 0 };
         |                                ^~~~~~~~~~~~~

vim +/kgd_gfx_v10_set_address_watch +882 drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gfx_v10.c

   881	
 > 882	void kgd_gfx_v10_set_address_watch(struct kgd_dev *kgd,
   883						uint64_t watch_address,
   884						uint32_t watch_address_mask,
   885						uint32_t watch_id,
   886						uint32_t watch_mode,
   887						uint32_t debug_vmid)
   888	{
   889		struct amdgpu_device *adev = get_amdgpu_device(kgd);
   890		uint32_t watch_address_high;
   891		uint32_t watch_address_low;
   892		uint32_t watch_address_cntl;
   893	
   894		watch_address_cntl = 0;
   895	
   896		watch_address_low = lower_32_bits(watch_address);
   897		watch_address_high = upper_32_bits(watch_address) & 0xffff;
   898	
   899		watch_address_cntl = REG_SET_FIELD(watch_address_cntl,
   900				TCP_WATCH0_CNTL,
   901				VMID,
   902				debug_vmid);
   903		watch_address_cntl = REG_SET_FIELD(watch_address_cntl,
   904				TCP_WATCH0_CNTL,
   905				MODE,
   906				watch_mode);
   907		watch_address_cntl = REG_SET_FIELD(watch_address_cntl,
   908				TCP_WATCH0_CNTL,
   909				MASK,
   910				watch_address_mask >> 6);
   911	
   912		/* Turning off this watch point until we set all the registers */
   913		watch_address_cntl = REG_SET_FIELD(watch_address_cntl,
   914				TCP_WATCH0_CNTL,
   915				VALID,
   916				0);
   917	
   918		WREG32((SOC15_REG_OFFSET(GC, 0, mmTCP_WATCH0_CNTL) +
   919				(watch_id * TCP_WATCH_STRIDE)),
   920				watch_address_cntl);
   921	
   922		WREG32((SOC15_REG_OFFSET(GC, 0, mmTCP_WATCH0_ADDR_H) +
   923				(watch_id * TCP_WATCH_STRIDE)),
   924				watch_address_high);
   925	
   926		WREG32((SOC15_REG_OFFSET(GC, 0, mmTCP_WATCH0_ADDR_L) +
   927				(watch_id * TCP_WATCH_STRIDE)),
   928				watch_address_low);
   929	
   930		/* Enable the watch point */
   931		watch_address_cntl = REG_SET_FIELD(watch_address_cntl,
   932				TCP_WATCH0_CNTL,
   933				VALID,
   934				1);
   935	
   936		WREG32((SOC15_REG_OFFSET(GC, 0, mmTCP_WATCH0_CNTL) +
   937				(watch_id * TCP_WATCH_STRIDE)),
   938				watch_address_cntl);
   939	
   940	}
   941	
 > 942	void kgd_gfx_v10_clear_address_watch(struct kgd_dev *kgd,
   943						uint32_t watch_id)
   944	{
   945		struct amdgpu_device *adev = get_amdgpu_device(kgd);
   946	
   947		uint32_t watch_address_cntl;
   948	
   949		watch_address_cntl = 0;
   950	
   951		WREG32((SOC15_REG_OFFSET(GC, 0, mmTCP_WATCH0_CNTL) +
   952				(watch_id * TCP_WATCH_STRIDE)),
   953				watch_address_cntl);
   954	}
   955	

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org

[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 65816 bytes --]

[-- Attachment #3: Type: text/plain, Size: 160 bytes --]

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

WARNING: multiple messages have this Message-ID (diff)
From: kernel test robot <lkp@intel.com>
To: kbuild-all@lists.01.org
Subject: [radeon-alex:amd-20.45 1587/2427] drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gfx_v10.c:882:6: warning: no previous prototype for 'kgd_gfx_v10_set_address_watch'
Date: Tue, 15 Dec 2020 13:50:02 +0800	[thread overview]
Message-ID: <202012151358.rsGfl3ws-lkp@intel.com> (raw)

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

tree:   git://people.freedesktop.org/~agd5f/linux.git amd-20.45
head:   a3950d94b046fb206e58fd3ec717f071c0203ba3
commit: abdc393ab8265df9db5d9d64eb33ed1c33e9d95c [1587/2427] drm/amdkfd: Add gfx10 address watch support
config: powerpc-allyesconfig (attached as .config)
compiler: powerpc64-linux-gcc (GCC) 9.3.0
reproduce (this is a W=1 build):
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        git remote add radeon-alex git://people.freedesktop.org/~agd5f/linux.git
        git fetch --no-tags radeon-alex amd-20.45
        git checkout abdc393ab8265df9db5d9d64eb33ed1c33e9d95c
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=powerpc 

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>

All warnings (new ones prefixed by >>):

   drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gfx_v10.c:763:6: warning: no previous prototype for 'kgd_gfx_v10_enable_debug_trap' [-Wmissing-prototypes]
     763 | void kgd_gfx_v10_enable_debug_trap(struct kgd_dev *kgd,
         |      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gfx_v10.c:792:6: warning: no previous prototype for 'kgd_gfx_v10_disable_debug_trap' [-Wmissing-prototypes]
     792 | void kgd_gfx_v10_disable_debug_trap(struct kgd_dev *kgd)
         |      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gfx_v10.c:803:5: warning: no previous prototype for 'kgd_gfx_v10_set_wave_launch_trap_override' [-Wmissing-prototypes]
     803 | int kgd_gfx_v10_set_wave_launch_trap_override(struct kgd_dev *kgd,
         |     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gfx_v10.c:852:6: warning: no previous prototype for 'kgd_gfx_v10_set_wave_launch_mode' [-Wmissing-prototypes]
     852 | void kgd_gfx_v10_set_wave_launch_mode(struct kgd_dev *kgd,
         |      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>> drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gfx_v10.c:882:6: warning: no previous prototype for 'kgd_gfx_v10_set_address_watch' [-Wmissing-prototypes]
     882 | void kgd_gfx_v10_set_address_watch(struct kgd_dev *kgd,
         |      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>> drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gfx_v10.c:942:6: warning: no previous prototype for 'kgd_gfx_v10_clear_address_watch' [-Wmissing-prototypes]
     942 | void kgd_gfx_v10_clear_address_watch(struct kgd_dev *kgd,
         |      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gfx_v10.c:967:6: warning: no previous prototype for 'kgd_gfx_v10_get_iq_wait_times' [-Wmissing-prototypes]
     967 | void kgd_gfx_v10_get_iq_wait_times(struct kgd_dev *kgd,
         |      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gfx_v10.c:976:6: warning: no previous prototype for 'kgd_gfx_v10_build_grace_period_packet_info' [-Wmissing-prototypes]
     976 | void kgd_gfx_v10_build_grace_period_packet_info(struct kgd_dev *kgd,
         |      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   In file included from drivers/gpu/drm/amd/display/dc/dc_types.h:33,
                    from drivers/gpu/drm/amd/display/dc/dm_services_types.h:30,
                    from drivers/gpu/drm/amd/include/dm_pp_interface.h:26,
                    from drivers/gpu/drm/amd/amdgpu/amdgpu.h:64,
                    from drivers/gpu/drm/amd/backport/include/kcl/kcl_amdgpu.h:6,
                    from drivers/gpu/drm/amd/backport/backport.h:18,
                    from <command-line>:
   drivers/gpu/drm/amd/display/include/fixed31_32.h:76:32: warning: 'dc_fixpt_ln2_div_2' defined but not used [-Wunused-const-variable=]
      76 | static const struct fixed31_32 dc_fixpt_ln2_div_2 = { 1488522236LL };
         |                                ^~~~~~~~~~~~~~~~~~
   drivers/gpu/drm/amd/display/include/fixed31_32.h:75:32: warning: 'dc_fixpt_ln2' defined but not used [-Wunused-const-variable=]
      75 | static const struct fixed31_32 dc_fixpt_ln2 = { 2977044471LL };
         |                                ^~~~~~~~~~~~
   drivers/gpu/drm/amd/display/include/fixed31_32.h:74:32: warning: 'dc_fixpt_e' defined but not used [-Wunused-const-variable=]
      74 | static const struct fixed31_32 dc_fixpt_e = { 11674931555LL };
         |                                ^~~~~~~~~~
   drivers/gpu/drm/amd/display/include/fixed31_32.h:73:32: warning: 'dc_fixpt_two_pi' defined but not used [-Wunused-const-variable=]
      73 | static const struct fixed31_32 dc_fixpt_two_pi = { 26986075409LL };
         |                                ^~~~~~~~~~~~~~~
   drivers/gpu/drm/amd/display/include/fixed31_32.h:72:32: warning: 'dc_fixpt_pi' defined but not used [-Wunused-const-variable=]
      72 | static const struct fixed31_32 dc_fixpt_pi = { 13493037705LL };
         |                                ^~~~~~~~~~~
   drivers/gpu/drm/amd/display/include/fixed31_32.h:67:32: warning: 'dc_fixpt_zero' defined but not used [-Wunused-const-variable=]
      67 | static const struct fixed31_32 dc_fixpt_zero = { 0 };
         |                                ^~~~~~~~~~~~~

vim +/kgd_gfx_v10_set_address_watch +882 drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gfx_v10.c

   881	
 > 882	void kgd_gfx_v10_set_address_watch(struct kgd_dev *kgd,
   883						uint64_t watch_address,
   884						uint32_t watch_address_mask,
   885						uint32_t watch_id,
   886						uint32_t watch_mode,
   887						uint32_t debug_vmid)
   888	{
   889		struct amdgpu_device *adev = get_amdgpu_device(kgd);
   890		uint32_t watch_address_high;
   891		uint32_t watch_address_low;
   892		uint32_t watch_address_cntl;
   893	
   894		watch_address_cntl = 0;
   895	
   896		watch_address_low = lower_32_bits(watch_address);
   897		watch_address_high = upper_32_bits(watch_address) & 0xffff;
   898	
   899		watch_address_cntl = REG_SET_FIELD(watch_address_cntl,
   900				TCP_WATCH0_CNTL,
   901				VMID,
   902				debug_vmid);
   903		watch_address_cntl = REG_SET_FIELD(watch_address_cntl,
   904				TCP_WATCH0_CNTL,
   905				MODE,
   906				watch_mode);
   907		watch_address_cntl = REG_SET_FIELD(watch_address_cntl,
   908				TCP_WATCH0_CNTL,
   909				MASK,
   910				watch_address_mask >> 6);
   911	
   912		/* Turning off this watch point until we set all the registers */
   913		watch_address_cntl = REG_SET_FIELD(watch_address_cntl,
   914				TCP_WATCH0_CNTL,
   915				VALID,
   916				0);
   917	
   918		WREG32((SOC15_REG_OFFSET(GC, 0, mmTCP_WATCH0_CNTL) +
   919				(watch_id * TCP_WATCH_STRIDE)),
   920				watch_address_cntl);
   921	
   922		WREG32((SOC15_REG_OFFSET(GC, 0, mmTCP_WATCH0_ADDR_H) +
   923				(watch_id * TCP_WATCH_STRIDE)),
   924				watch_address_high);
   925	
   926		WREG32((SOC15_REG_OFFSET(GC, 0, mmTCP_WATCH0_ADDR_L) +
   927				(watch_id * TCP_WATCH_STRIDE)),
   928				watch_address_low);
   929	
   930		/* Enable the watch point */
   931		watch_address_cntl = REG_SET_FIELD(watch_address_cntl,
   932				TCP_WATCH0_CNTL,
   933				VALID,
   934				1);
   935	
   936		WREG32((SOC15_REG_OFFSET(GC, 0, mmTCP_WATCH0_CNTL) +
   937				(watch_id * TCP_WATCH_STRIDE)),
   938				watch_address_cntl);
   939	
   940	}
   941	
 > 942	void kgd_gfx_v10_clear_address_watch(struct kgd_dev *kgd,
   943						uint32_t watch_id)
   944	{
   945		struct amdgpu_device *adev = get_amdgpu_device(kgd);
   946	
   947		uint32_t watch_address_cntl;
   948	
   949		watch_address_cntl = 0;
   950	
   951		WREG32((SOC15_REG_OFFSET(GC, 0, mmTCP_WATCH0_CNTL) +
   952				(watch_id * TCP_WATCH_STRIDE)),
   953				watch_address_cntl);
   954	}
   955	

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org

[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 65816 bytes --]

             reply	other threads:[~2020-12-15  5:50 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-12-15  5:50 kernel test robot [this message]
2020-12-15  5:50 ` [radeon-alex:amd-20.45 1587/2427] drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gfx_v10.c:882:6: warning: no previous prototype for 'kgd_gfx_v10_set_address_watch' kernel test robot

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=202012151358.rsGfl3ws-lkp@intel.com \
    --to=lkp@intel.com \
    --cc=Felix.Kuehling@amd.com \
    --cc=Gerrit.Cr@amd.com \
    --cc=Philip.Cox@amd.com \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=kbuild-all@lists.01.org \
    --cc=rajneesh.bhardwaj@amd.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.