Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Subject: Re: [PATCH v2] net: ethernet: ti: davinci_cpdma: fix warning "device
 driver frees DMA memory with different size"
To:     David Miller <davem@davemloft.net>
CC:     <netdev@vger.kernel.org>, <ivan.khoronzhuk@linaro.org>,
        <nsekhar@ti.com>, <linux-kernel@vger.kernel.org>
References: <20191204165029.9264-1-grygorii.strashko@ti.com>
 <20191204.123718.1152659362924451799.davem@davemloft.net>
From:   Grygorii Strashko <grygorii.strashko@ti.com>
Message-ID: <0c6b88b2-31b1-11f0-7baa-1ecd5f4b6644@ti.com>
Date:   Thu, 5 Dec 2019 12:48:46 +0200
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101
 Thunderbird/60.9.0
MIME-Version: 1.0
In-Reply-To: <20191204.123718.1152659362924451799.davem@davemloft.net>
Content-Type: text/plain; charset="utf-8"; format=flowed
Content-Language: en-US
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk


On 04/12/2019 22:37, David Miller wrote:
> From: Grygorii Strashko <grygorii.strashko@ti.com>
> Date: Wed, 4 Dec 2019 18:50:29 +0200
> 
>> @@ -1018,7 +1018,7 @@ static int cpdma_chan_submit_si(struct submit_info *si)
>>   	struct cpdma_chan		*chan = si->chan;
>>   	struct cpdma_ctlr		*ctlr = chan->ctlr;
>>   	int				len = si->len;
>> -	int				swlen = len;
>> +	int				swlen;
>>   	struct cpdma_desc __iomem	*desc;
>>   	dma_addr_t			buffer;
>>   	u32				mode;
>> @@ -1040,6 +1040,7 @@ static int cpdma_chan_submit_si(struct submit_info *si)
>>   		chan->stats.runt_transmit_buff++;
>>   	}
>>   
>> +	swlen = len;
>>   	mode = CPDMA_DESC_OWNER | CPDMA_DESC_SOP | CPDMA_DESC_EOP;
>>   	cpdma_desc_to_port(chan, mode, si->directed);
>>   
>> -- 
>> 2.17.1
>>
> 
> Now there is no reason to keep a separate swlen variable.
> 
> The integral value is always consumed as the length before the descriptor bits
> are added to it.
> 
> Therefore you can just use 'len' everywhere in this function now.
> 

Sry, but seems i can't, at least i can't just drop swlen.

Below in this function:
	writel_relaxed(0, &desc->hw_next);
	writel_relaxed(buffer, &desc->hw_buffer);
	writel_relaxed(len, &desc->hw_len);
	writel_relaxed(mode | len, &desc->hw_mode);
^^ here the "len" should be use

	writel_relaxed((uintptr_t)si->token, &desc->sw_token);
	writel_relaxed(buffer, &desc->sw_buffer);
	writel_relaxed(swlen, &desc->sw_len);
^^ and here "len"|CPDMA_DMA_EXT_MAP if (si->data_dma) [1]

	desc_read(desc, sw_len);

so additional if statement has to be added at [1] if "swlen" is dropped

-- 
Best regards,
grygorii