On 10/18/2024 12:02 AM, Stephen Hemminger wrote:
> On Thu, 17 Oct 2024 22:38:49 +0000
> Long Li <lon...@microsoft.com> wrote:
> 
>>> Subject: Re: [Patch v5] net/netvsc: fix number Tx queues > Rx queues
>>>
>>> On Thu, 17 Oct 2024 12:20:29 -0700
>>> lon...@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.

Reply via email to