Thread (29 messages) 29 messages, 5 authors, 2023-06-21

Re: [PATCH net-next v2 03/15] idpf: add controlq init and reset checks

From: Jakub Kicinski <kuba@kernel.org>
Date: 2023-06-21 19:14:41

On Wed, 21 Jun 2023 12:05:54 -0700 Linga, Pavan Kumar wrote:
quoted
quoted
+void *idpf_alloc_dma_mem(struct idpf_hw *hw, struct idpf_dma_mem *mem, u64 size)
+{
+	struct idpf_adapter *adapter = hw->back;
+	size_t sz = ALIGN(size, 4096);
+
+	mem->va = dma_alloc_coherent(&adapter->pdev->dev, sz,
+				     &mem->pa, GFP_KERNEL | __GFP_ZERO);  
DMA API always zeros memory, I thought cocci warns about this
Did you run cocci checks?
I ran cocci check using the command "make -j8 
M=drivers/net/ethernet/intel/idpf/ C=1 CHECK="scripts/coccicheck" 
CONFIG_IDPF=m &>>err.log" but didn't see any hits and not sure if this 
is the right command to see the warning. Will fix it anyways.
Ugh, disappointing, looks like people how were chasing these flags
throughout the tree couldn't be bothered to upstream the cocci check.
Or even document it, it seems? :| See 750afb08ca7131 as the initial
commit, FWIW, sorry to misdirect towards coccicheck.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help