[Patch v5] net/netvsc: fix number Tx queues > Rx queues

Ferruh Yigit ferruh.yigit at amd.com
Fri Oct 18 02:04:42 CEST 2024


On 10/18/2024 12:02 AM, Stephen Hemminger wrote:
> On Thu, 17 Oct 2024 22:38:49 +0000
> Long Li <longli at microsoft.com> wrote:
> 
>>> Subject: Re: [Patch v5] net/netvsc: fix number Tx queues > Rx queues
>>>
>>> On Thu, 17 Oct 2024 12:20:29 -0700
>>> longli at linuxonhyperv.com wrote:
>>>   
>>>> +static void
>>>> +hn_rx_queue_free_common(struct hn_rx_queue *rxq) {
>>>> +	if (!rxq)
>>>> +		return;
>>>> +
>>>> +	rte_free(rxq->rxbuf_info);
>>>> +	rte_free(rxq->event_buf);
>>>> +	rte_free(rxq);
>>>> +}  
>>>
>>> Minor nit, DPDK style is for the bracket on the next line.
>>> Checkpatch will complain about this.  
>>
>> Is it okay to take the patch as is, or should I send a v6 to have it fixed?
> 
> It is ok as is, but followup to fix the minor stuff like this would be good.
>

I will fix this one while merging.


More information about the dev mailing list