[dpdk-stable] [EXT] patch 'net/qede: fix performance bottleneck in Rx path' has been queued to LTS release 17.11.6

Yongseok Koh yskoh at mellanox.com
Tue Mar 12 23:03:14 CET 2019


> On Mar 12, 2019, at 10:04 AM, Shahed Shaikh <shshaikh at marvell.com> wrote:
> 
>> -----Original Message-----
>> From: Yongseok Koh <yskoh at mellanox.com>
>> Sent: Friday, March 8, 2019 11:18 PM
>> To: Shahed Shaikh <shshaikh at marvell.com>
>> Cc: Rasesh Mody <rmody at marvell.com>; dpdk stable <stable at dpdk.org>
>> Subject: [EXT] patch 'net/qede: fix performance bottleneck in Rx path' has been
>> queued to LTS release 17.11.6
>> Hi,
>> 
>> FYI, your patch has been queued to LTS release 17.11.6
>> 
>> Note it hasn't been pushed to https://emea01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fdpdk.org%2Fbrowse%2Fdpdk-stable&data=02%7C01%7Cyskoh%40mellanox.com%7C65bf75b6f82140e58fda08d6a70cce02%7Ca652971c7d2e4d9ba6a4d149256f461b%7C0%7C0%7C636880070771455895&sdata=jZGcy3fhfhLg0GAEz1leT%2F3%2Bw%2FaXYyzujmd14PQ2PYg%3D&reserved=0 yet.
>> It will be pushed if I get no objection by 03/13/19. So please shout if anyone has
>> objection.
> 
> Hi,
> 
> We recently found that this patch introduces a regression for which I have just sent out a fix.
> https://emea01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fpatchwork.dpdk.org%2Fpatch%2F51134%2F&data=02%7C01%7Cyskoh%40mellanox.com%7C65bf75b6f82140e58fda08d6a70cce02%7Ca652971c7d2e4d9ba6a4d149256f461b%7C0%7C0%7C636880070771455895&sdata=kFuiOeXlRoZv0%2Fsn3ds7cSJmrKRlyOMPIMbk7r5pWKw%3D&reserved=0   ("net/qede: fix receive packet drop")
> 
> How do we handle such situation? Will you also pull above fix once it gets accepted or ignore current patch for 17.11.6 and consider for next release?
> Please advise.

>From looking at the new patch, I'm okay to merge it for 17.11.6 as long as it is acked.
Please let me know if it is acked and merged, then I'll squash it to the original fix.
Until then, I'll hold the original one - 'net/qede: fix performance bottleneck in Rx path'
Also try to drop the Cc:stable tag so that it won't get queued again for 17.11.7.

Thanks,
Yongseok



More information about the stable mailing list