From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Miller Subject: Re: [PATCH net] net: mvneta: fix the Rx desc buffer DMA unmapping Date: Wed, 19 Sep 2018 21:25:41 -0700 (PDT) Message-ID: <20180919.212541.1251609826389428684.davem@davemloft.net> References: <20180919132906.26868-1-antoine.tenart@bootlin.com> Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit Cc: yelena@marvell.com, netdev@vger.kernel.org, linux-kernel@vger.kernel.org, thomas.petazzoni@bootlin.com, maxime.chevallier@bootlin.com, gregory.clement@bootlin.com, miquel.raynal@bootlin.com, nadavh@marvell.com, stefanc@marvell.com, ymarkman@marvell.com, mw@semihalf.com To: antoine.tenart@bootlin.com Return-path: Received: from shards.monkeyblade.net ([23.128.96.9]:34450 "EHLO shards.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726254AbeITKHF (ORCPT ); Thu, 20 Sep 2018 06:07:05 -0400 In-Reply-To: <20180919132906.26868-1-antoine.tenart@bootlin.com> Sender: netdev-owner@vger.kernel.org List-ID: From: Antoine Tenart Date: Wed, 19 Sep 2018 15:29:06 +0200 > With CONFIG_DMA_API_DEBUG enabled we now get a warning when using the > mvneta driver: > > mvneta d0030000.ethernet: DMA-API: device driver frees DMA memory with > wrong function [device address=0x000000001165b000] [size=4096 bytes] > [mapped as page] [unmapped as single] > > This is because when using the s/w buffer management, the Rx descriptor > buffer is mapped with dma_map_page but unmapped with dma_unmap_single. > This patch fixes this by using the right unmapping function. > > Fixes: 562e2f467e71 ("net: mvneta: Improve the buffer allocation method for SWBM") > Signed-off-by: Antoine Tenart Applied.