<div dir="ltr">Hi,<div><br></div><div>Is there any other work/changes required for this patch to be submitted?</div><div><pre style="white-space:pre-wrap;color:rgb(0,0,0)"><span style="color:rgb(34,34,34);font-family:Arial,Helvetica,sans-serif;white-space:normal">Thanks</span></pre></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Oct 4, 2022 at 7:53 PM Andrew Rybchenko <<a href="mailto:andrew.rybchenko@oktetlabs.ru">andrew.rybchenko@oktetlabs.ru</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 8/8/22 13:39, Joyce Kong wrote:<br>
> Hi Huzaifa,<br>
> <br>
> This patch looks good to me.<br>
> And would you please help review my memif patches?<br>
> <a href="https://patches.dpdk.org/project/dpdk/cover/20220701102815.1444223-1-joyce.kong@arm.com/" rel="noreferrer" target="_blank">https://patches.dpdk.org/project/dpdk/cover/20220701102815.1444223-1-joyce.kong@arm.com/</a><br>
> <br>
> Thanks,<br>
> Joyce<br>
> <br>
>> -----Original Message-----<br>
>> From: huzaifa.rahman <<a href="mailto:huzaifa.rahman@emumba.com" target="_blank">huzaifa.rahman@emumba.com</a>><br>
>> Sent: Tuesday, July 26, 2022 6:16 PM<br>
>> To: <a href="mailto:jgrajcia@cisco.com" target="_blank">jgrajcia@cisco.com</a><br>
>> Cc: <a href="mailto:dev@dpdk.org" target="_blank">dev@dpdk.org</a>; huzaifa.rahman <<a href="mailto:huzaifa.rahman@emumba.com" target="_blank">huzaifa.rahman@emumba.com</a>><br>
>> Subject: [PATCH] memif: memif driver does not crashes when there's<br>
>> different N of TX and RX queues<br>
> net/memif: fix memif crash with different Tx Rx queues<br>
> <br>
>><br>
>> Bugzilla ID: 734<br>
>><br>
>> there's a bug in memif_stats_get() function due to confusion between C2S<br>
>> (client->server) and S2C (server->client) rings, causing a crash if there's a<br>
>> different number of RX and TX queues.<br>
>><br>
>> this is fixed by selectiing the correct rings for RX and TX i.e for RX, S2C rings<br>
>> are selected and for TX, C2S rings are selected.<br>
>><br>
> Fixes: 09c7e63a71f9 ("net/memif: introduce memory interface PMD")<br>
> Cc: <a href="mailto:stable@dpdk.org" target="_blank">stable@dpdk.org</a><br>
> <br>
>> Signed-off-by: huzaifa.rahman <<a href="mailto:huzaifa.rahman@emumba.com" target="_blank">huzaifa.rahman@emumba.com</a>><br>
> Reviewed-by: Joyce Kong <<a href="mailto:joyce.kong@arm.com" target="_blank">joyce.kong@arm.com</a>><br>
<br>
Fixed above on applying.<br>
<br>
Applied to dpdk-next-net/main, thanks.<br>
<br>
<br>
</blockquote></div>