On 11/10/2022 10:02 AM, Huzaifa Rahman wrote:
Hi,
Is there any other work/changes required for this patch to be submitted?
Hi Huzaifa,
Patch seems already merged by Andrew and pulled to main repo:
https://git.dpdk.org/dpdk/commit/?id=231435a5e6c7fa915697d8f84a91b44176
So it will in oncoming 22.11 release.
Thanks
On Tue, Oct 4, 2022 at 7:53 PM Andrew Rybchenko
<[email protected] <mailto:[email protected]>>
wrote:
On 8/8/22 13:39, Joyce Kong wrote:
> Hi Huzaifa,
>
> This patch looks good to me.
> And would you please help review my memif patches?
>
https://patches.dpdk.org/project/dpdk/cover/[email protected]/
<https://patches.dpdk.org/project/dpdk/cover/[email protected]/>
>
> Thanks,
> Joyce
>
>> -----Original Message-----
>> From: huzaifa.rahman <[email protected]
<mailto:[email protected]>>
>> Sent: Tuesday, July 26, 2022 6:16 PM
>> To: [email protected] <mailto:[email protected]>
>> Cc: [email protected] <mailto:[email protected]>; huzaifa.rahman
<[email protected] <mailto:[email protected]>>
>> Subject: [PATCH] memif: memif driver does not crashes when there's
>> different N of TX and RX queues
> net/memif: fix memif crash with different Tx Rx queues
>
>>
>> Bugzilla ID: 734
>>
>> there's a bug in memif_stats_get() function due to confusion
between C2S
>> (client->server) and S2C (server->client) rings, causing a crash
if there's a
>> different number of RX and TX queues.
>>
>> this is fixed by selectiing the correct rings for RX and TX i.e
for RX, S2C rings
>> are selected and for TX, C2S rings are selected.
>>
> Fixes: 09c7e63a71f9 ("net/memif: introduce memory interface PMD")
> Cc: [email protected] <mailto:[email protected]>
>
>> Signed-off-by: huzaifa.rahman <[email protected]
<mailto:[email protected]>>
> Reviewed-by: Joyce Kong <[email protected]
<mailto:[email protected]>>
Fixed above on applying.
Applied to dpdk-next-net/main, thanks.
