[linux-next:master 9762/10134] drivers/gpu/drm/amd/amdgpu/sdma_v7_0.c:235: warning: Function parameter or struct member 'job' not described in 'sdma_v7_0_ring_emit_ib'

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



tree:   https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master
head:   f68868ba718e30594165879cc3020607165b0761
commit: b412351e91bde3a6ec546b598087bdc3fcc458ee [9762/10134] drm/amdgpu: Add sdma v7_0 ip block support (v7)
config: loongarch-allmodconfig (https://download.01.org/0day-ci/archive/20240501/202405012302.cOj1BNI5-lkp@xxxxxxxxx/config)
compiler: loongarch64-linux-gcc (GCC) 13.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20240501/202405012302.cOj1BNI5-lkp@xxxxxxxxx/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@xxxxxxxxx>
| Closes: https://lore.kernel.org/oe-kbuild-all/202405012302.cOj1BNI5-lkp@xxxxxxxxx/

All warnings (new ones prefixed by >>):

>> drivers/gpu/drm/amd/amdgpu/sdma_v7_0.c:235: warning: Function parameter or struct member 'job' not described in 'sdma_v7_0_ring_emit_ib'
>> drivers/gpu/drm/amd/amdgpu/sdma_v7_0.c:235: warning: Function parameter or struct member 'flags' not described in 'sdma_v7_0_ring_emit_ib'
>> drivers/gpu/drm/amd/amdgpu/sdma_v7_0.c:269: warning: Excess function parameter 'job' description in 'sdma_v7_0_ring_emit_mem_sync'
>> drivers/gpu/drm/amd/amdgpu/sdma_v7_0.c:269: warning: Excess function parameter 'ib' description in 'sdma_v7_0_ring_emit_mem_sync'
>> drivers/gpu/drm/amd/amdgpu/sdma_v7_0.c:324: warning: Function parameter or struct member 'addr' not described in 'sdma_v7_0_ring_emit_fence'
>> drivers/gpu/drm/amd/amdgpu/sdma_v7_0.c:324: warning: Function parameter or struct member 'seq' not described in 'sdma_v7_0_ring_emit_fence'
>> drivers/gpu/drm/amd/amdgpu/sdma_v7_0.c:324: warning: Function parameter or struct member 'flags' not described in 'sdma_v7_0_ring_emit_fence'
>> drivers/gpu/drm/amd/amdgpu/sdma_v7_0.c:324: warning: Excess function parameter 'fence' description in 'sdma_v7_0_ring_emit_fence'
>> drivers/gpu/drm/amd/amdgpu/sdma_v7_0.c:915: warning: Function parameter or struct member 'timeout' not described in 'sdma_v7_0_ring_test_ib'
>> drivers/gpu/drm/amd/amdgpu/sdma_v7_0.c:1043: warning: Function parameter or struct member 'value' not described in 'sdma_v7_0_vm_write_pte'
>> drivers/gpu/drm/amd/amdgpu/sdma_v7_0.c:1043: warning: Excess function parameter 'addr' description in 'sdma_v7_0_vm_write_pte'
>> drivers/gpu/drm/amd/amdgpu/sdma_v7_0.c:1043: warning: Excess function parameter 'flags' description in 'sdma_v7_0_vm_write_pte'
>> drivers/gpu/drm/amd/amdgpu/sdma_v7_0.c:1095: warning: Function parameter or struct member 'ring' not described in 'sdma_v7_0_ring_pad_ib'
>> drivers/gpu/drm/amd/amdgpu/sdma_v7_0.c:1147: warning: Function parameter or struct member 'vmid' not described in 'sdma_v7_0_ring_emit_vm_flush'
>> drivers/gpu/drm/amd/amdgpu/sdma_v7_0.c:1147: warning: Function parameter or struct member 'pd_addr' not described in 'sdma_v7_0_ring_emit_vm_flush'
>> drivers/gpu/drm/amd/amdgpu/sdma_v7_0.c:1147: warning: Excess function parameter 'vm' description in 'sdma_v7_0_ring_emit_vm_flush'
>> drivers/gpu/drm/amd/amdgpu/sdma_v7_0.c:1555: warning: Function parameter or struct member 'ib' not described in 'sdma_v7_0_emit_copy_buffer'
>> drivers/gpu/drm/amd/amdgpu/sdma_v7_0.c:1555: warning: Excess function parameter 'ring' description in 'sdma_v7_0_emit_copy_buffer'
>> drivers/gpu/drm/amd/amdgpu/sdma_v7_0.c:1581: warning: Function parameter or struct member 'ib' not described in 'sdma_v7_0_emit_fill_buffer'
>> drivers/gpu/drm/amd/amdgpu/sdma_v7_0.c:1581: warning: Excess function parameter 'ring' description in 'sdma_v7_0_emit_fill_buffer'


vim +235 drivers/gpu/drm/amd/amdgpu/sdma_v7_0.c

   222	
   223	/**
   224	 * sdma_v7_0_ring_emit_ib - Schedule an IB on the DMA engine
   225	 *
   226	 * @ring: amdgpu ring pointer
   227	 * @ib: IB object to schedule
   228	 *
   229	 * Schedule an IB in the DMA ring.
   230	 */
   231	static void sdma_v7_0_ring_emit_ib(struct amdgpu_ring *ring,
   232					   struct amdgpu_job *job,
   233					   struct amdgpu_ib *ib,
   234					   uint32_t flags)
 > 235	{
   236		unsigned vmid = AMDGPU_JOB_GET_VMID(job);
   237		uint64_t csa_mc_addr = amdgpu_sdma_get_csa_mc_addr(ring, vmid);
   238	
   239		/* An IB packet must end on a 8 DW boundary--the next dword
   240		 * must be on a 8-dword boundary. Our IB packet below is 6
   241		 * dwords long, thus add x number of NOPs, such that, in
   242		 * modular arithmetic,
   243		 * wptr + 6 + x = 8k, k >= 0, which in C is,
   244		 * (wptr + 6 + x) % 8 = 0.
   245		 * The expression below, is a solution of x.
   246		 */
   247		sdma_v7_0_ring_insert_nop(ring, (2 - lower_32_bits(ring->wptr)) & 7);
   248	
   249		amdgpu_ring_write(ring, SDMA_PKT_COPY_LINEAR_HEADER_OP(SDMA_OP_INDIRECT) |
   250				  SDMA_PKT_INDIRECT_HEADER_VMID(vmid & 0xf));
   251		/* base must be 32 byte aligned */
   252		amdgpu_ring_write(ring, lower_32_bits(ib->gpu_addr) & 0xffffffe0);
   253		amdgpu_ring_write(ring, upper_32_bits(ib->gpu_addr));
   254		amdgpu_ring_write(ring, ib->length_dw);
   255		amdgpu_ring_write(ring, lower_32_bits(csa_mc_addr));
   256		amdgpu_ring_write(ring, upper_32_bits(csa_mc_addr));
   257	}
   258	
   259	/**
   260	 * sdma_v7_0_ring_emit_mem_sync - flush the IB by graphics cache rinse
   261	 *
   262	 * @ring: amdgpu ring pointer
   263	 * @job: job to retrieve vmid from
   264	 * @ib: IB object to schedule
   265	 *
   266	 * flush the IB by graphics cache rinse.
   267	 */
   268	static void sdma_v7_0_ring_emit_mem_sync(struct amdgpu_ring *ring)
 > 269	{
   270		uint32_t gcr_cntl = SDMA_GCR_GL2_INV | SDMA_GCR_GL2_WB | SDMA_GCR_GLM_INV |
   271			SDMA_GCR_GL1_INV | SDMA_GCR_GLV_INV | SDMA_GCR_GLK_INV |
   272			SDMA_GCR_GLI_INV(1);
   273	
   274		/* flush entire cache L0/L1/L2, this can be optimized by performance requirement */
   275		amdgpu_ring_write(ring, SDMA_PKT_COPY_LINEAR_HEADER_OP(SDMA_OP_GCR_REQ));
   276		amdgpu_ring_write(ring, SDMA_PKT_GCR_REQ_PAYLOAD1_BASE_VA_31_7(0));
   277		amdgpu_ring_write(ring, SDMA_PKT_GCR_REQ_PAYLOAD2_GCR_CONTROL_15_0(gcr_cntl) |
   278				  SDMA_PKT_GCR_REQ_PAYLOAD2_BASE_VA_47_32(0));
   279		amdgpu_ring_write(ring, SDMA_PKT_GCR_REQ_PAYLOAD3_LIMIT_VA_31_7(0) |
   280				  SDMA_PKT_GCR_REQ_PAYLOAD3_GCR_CONTROL_18_16(gcr_cntl >> 16));
   281		amdgpu_ring_write(ring, SDMA_PKT_GCR_REQ_PAYLOAD4_LIMIT_VA_47_32(0) |
   282				  SDMA_PKT_GCR_REQ_PAYLOAD4_VMID(0));
   283	}
   284	
   285	
   286	/**
   287	 * sdma_v7_0_ring_emit_hdp_flush - emit an hdp flush on the DMA ring
   288	 *
   289	 * @ring: amdgpu ring pointer
   290	 *
   291	 * Emit an hdp flush packet on the requested DMA ring.
   292	 */
   293	static void sdma_v7_0_ring_emit_hdp_flush(struct amdgpu_ring *ring)
   294	{
   295		struct amdgpu_device *adev = ring->adev;
   296		u32 ref_and_mask = 0;
   297		const struct nbio_hdp_flush_reg *nbio_hf_reg = adev->nbio.hdp_flush_reg;
   298	
   299		ref_and_mask = nbio_hf_reg->ref_and_mask_sdma0 << ring->me;
   300	
   301		amdgpu_ring_write(ring, SDMA_PKT_COPY_LINEAR_HEADER_OP(SDMA_OP_POLL_REGMEM) |
   302				  SDMA_PKT_POLL_REGMEM_HEADER_HDP_FLUSH(1) |
   303				  SDMA_PKT_POLL_REGMEM_HEADER_FUNC(3)); /* == */
   304		amdgpu_ring_write(ring, (adev->nbio.funcs->get_hdp_flush_done_offset(adev)) << 2);
   305		amdgpu_ring_write(ring, (adev->nbio.funcs->get_hdp_flush_req_offset(adev)) << 2);
   306		amdgpu_ring_write(ring, ref_and_mask); /* reference */
   307		amdgpu_ring_write(ring, ref_and_mask); /* mask */
   308		amdgpu_ring_write(ring, SDMA_PKT_POLL_REGMEM_DW5_RETRY_COUNT(0xfff) |
   309				  SDMA_PKT_POLL_REGMEM_DW5_INTERVAL(10)); /* retry count, poll interval */
   310	}
   311	
   312	/**
   313	 * sdma_v7_0_ring_emit_fence - emit a fence on the DMA ring
   314	 *
   315	 * @ring: amdgpu ring pointer
   316	 * @fence: amdgpu fence object
   317	 *
   318	 * Add a DMA fence packet to the ring to write
   319	 * the fence seq number and DMA trap packet to generate
   320	 * an interrupt if needed.
   321	 */
   322	static void sdma_v7_0_ring_emit_fence(struct amdgpu_ring *ring, u64 addr, u64 seq,
   323					      unsigned flags)
 > 324	{
   325		bool write64bit = flags & AMDGPU_FENCE_FLAG_64BIT;
   326		/* write the fence */
   327		amdgpu_ring_write(ring, SDMA_PKT_COPY_LINEAR_HEADER_OP(SDMA_OP_FENCE) |
   328				  SDMA_PKT_FENCE_HEADER_MTYPE(0x3)); /* Ucached(UC) */
   329		/* zero in first two bits */
   330		BUG_ON(addr & 0x3);
   331		amdgpu_ring_write(ring, lower_32_bits(addr));
   332		amdgpu_ring_write(ring, upper_32_bits(addr));
   333		amdgpu_ring_write(ring, lower_32_bits(seq));
   334	
   335		/* optionally write high bits as well */
   336		if (write64bit) {
   337			addr += 4;
   338			amdgpu_ring_write(ring, SDMA_PKT_COPY_LINEAR_HEADER_OP(SDMA_OP_FENCE) |
   339					  SDMA_PKT_FENCE_HEADER_MTYPE(0x3));
   340			/* zero in first two bits */
   341			BUG_ON(addr & 0x3);
   342			amdgpu_ring_write(ring, lower_32_bits(addr));
   343			amdgpu_ring_write(ring, upper_32_bits(addr));
   344			amdgpu_ring_write(ring, upper_32_bits(seq));
   345		}
   346	
   347		if (flags & AMDGPU_FENCE_FLAG_INT) {
   348			uint32_t ctx = ring->is_mes_queue ?
   349				(ring->hw_queue_id | AMDGPU_FENCE_MES_QUEUE_FLAG) : 0;
   350			/* generate an interrupt */
   351			amdgpu_ring_write(ring, SDMA_PKT_COPY_LINEAR_HEADER_OP(SDMA_OP_TRAP));
   352			amdgpu_ring_write(ring, SDMA_PKT_TRAP_INT_CONTEXT_INT_CONTEXT(ctx));
   353		}
   354	}
   355	

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Bugtraq]     [Linux OMAP]     [Linux MIPS]     [eCos]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux