From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 80DDB19925D; Wed, 5 Jun 2024 11:53:40 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1717588420; cv=none; b=Wf7VVfUD3KJGU3pWXyN3rEvQlrdsuuwexzGPsHrWIYR2M6hMxTvMP9rI0OxtKLIbOPKTpvJqxbqjvFoljPCuhkNTgH0JwN71rgeblWUTb6wYGxSJRz2ztB+A5v2Ns4U61F8LY/X3Uly/BJLi/J87CaVkYdXAz9SkelaPcuRKEEM= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1717588420; c=relaxed/simple; bh=2ISdze8NAMVbO4NGcT3IQZ72Nfo8NiD6Q5dZiOhhYFc=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=f6HEgXAwWUre1L5UVDQG5nRVF4LngT1lXUaAF9ayBrEGzcIEMjCcF/hrrPsGfN0ySa205hmhKget3FTc944Fj3xdwyVc9B5QTiZ3RcPckWYzMQc82OnrdfgZyO6/vQ2e/VUMnNbV0cyQ+EK7apZnAMxhDyXyHT8grk4edNB833c= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=s8ms4B1k; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="s8ms4B1k" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 7B19FC4AF07; Wed, 5 Jun 2024 11:53:39 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1717588420; bh=2ISdze8NAMVbO4NGcT3IQZ72Nfo8NiD6Q5dZiOhhYFc=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=s8ms4B1kKZQXlDDHmRQOnw2cqe/kYgkcQbpbM2lALPcU3UPlu40m4TlMRUTey4mQx zJMDxrmc+6BfggjGIr8foipaJddDTqvu5Mn0Mm6fGzvNFQ4inP2h2A68IfnsgEfFhO Y2kEIdc31t3xYrOK0mqP61i78tpNlT5WvMbCD8SMioH65tlJp+YcuaRVsLDU3Wn8E0 v0hTuySblOYj4DcwUIQqfb9UQsLCdopMgRYIh4xacP2WclLjocySZU1GoOHmgJjnay mItdbx9n63SI9wYCHXO4n5cOpfyzsRJgGFWF+wNDi4obCvoLZbuEwnk2H8woLRJgsy egmOIB6p9spgQ== From: Sasha Levin To: linux-kernel@vger.kernel.org, stable@vger.kernel.org Cc: Joao Pinto , Joao Pinto , Vinod Koul , Sasha Levin , Eugeniy.Paltsev@synopsys.com, dmaengine@vger.kernel.org Subject: [PATCH AUTOSEL 6.1 03/12] Avoid hw_desc array overrun in dw-axi-dmac Date: Wed, 5 Jun 2024 07:53:08 -0400 Message-ID: <20240605115334.2963803-3-sashal@kernel.org> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20240605115334.2963803-1-sashal@kernel.org> References: <20240605115334.2963803-1-sashal@kernel.org> Precedence: bulk X-Mailing-List: stable@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-stable: review X-Patchwork-Hint: Ignore X-stable-base: Linux 6.1.92 Content-Transfer-Encoding: 8bit From: Joao Pinto [ Upstream commit 333e11bf47fa8d477db90e2900b1ed3c9ae9b697 ] I have a use case where nr_buffers = 3 and in which each descriptor is composed by 3 segments, resulting in the DMA channel descs_allocated to be 9. Since axi_desc_put() handles the hw_desc considering the descs_allocated, this scenario would result in a kernel panic (hw_desc array will be overrun). To fix this, the proposal is to add a new member to the axi_dma_desc structure, where we keep the number of allocated hw_descs (axi_desc_alloc()) and use it in axi_desc_put() to handle the hw_desc array correctly. Additionally I propose to remove the axi_chan_start_first_queued() call after completing the transfer, since it was identified that unbalance can occur (started descriptors can be interrupted and transfer ignored due to DMA channel not being enabled). Signed-off-by: Joao Pinto Link: https://lore.kernel.org/r/1711536564-12919-1-git-send-email-jpinto@synopsys.com Signed-off-by: Vinod Koul Signed-off-by: Sasha Levin --- drivers/dma/dw-axi-dmac/dw-axi-dmac-platform.c | 6 ++---- drivers/dma/dw-axi-dmac/dw-axi-dmac.h | 1 + 2 files changed, 3 insertions(+), 4 deletions(-) diff --git a/drivers/dma/dw-axi-dmac/dw-axi-dmac-platform.c b/drivers/dma/dw-axi-dmac/dw-axi-dmac-platform.c index 152c5d98524d7..7596864bf8bb2 100644 --- a/drivers/dma/dw-axi-dmac/dw-axi-dmac-platform.c +++ b/drivers/dma/dw-axi-dmac/dw-axi-dmac-platform.c @@ -250,6 +250,7 @@ static struct axi_dma_desc *axi_desc_alloc(u32 num) kfree(desc); return NULL; } + desc->nr_hw_descs = num; return desc; } @@ -276,7 +277,7 @@ static struct axi_dma_lli *axi_desc_get(struct axi_dma_chan *chan, static void axi_desc_put(struct axi_dma_desc *desc) { struct axi_dma_chan *chan = desc->chan; - int count = atomic_read(&chan->descs_allocated); + int count = desc->nr_hw_descs; struct axi_dma_hw_desc *hw_desc; int descs_put; @@ -1087,9 +1088,6 @@ static void axi_chan_block_xfer_complete(struct axi_dma_chan *chan) /* Remove the completed descriptor from issued list before completing */ list_del(&vd->node); vchan_cookie_complete(vd); - - /* Submit queued descriptors after processing the completed ones */ - axi_chan_start_first_queued(chan); } out: diff --git a/drivers/dma/dw-axi-dmac/dw-axi-dmac.h b/drivers/dma/dw-axi-dmac/dw-axi-dmac.h index e9d5eb0fd5948..764427a66f5e8 100644 --- a/drivers/dma/dw-axi-dmac/dw-axi-dmac.h +++ b/drivers/dma/dw-axi-dmac/dw-axi-dmac.h @@ -103,6 +103,7 @@ struct axi_dma_desc { u32 completed_blocks; u32 length; u32 period_len; + u32 nr_hw_descs; }; struct axi_dma_chan_config { -- 2.43.0