[dpdk-stable] please help backporting some patches to LTS release 17.11.5

Yongseok Koh yskoh at mellanox.com
Wed Dec 19 21:00:27 CET 2018


> On Dec 18, 2018, at 12:36 AM, Cao, Yahui <yahui.cao at intel.com> wrote:
> 
> Hi Yongseok,
> 
> The commit f457e9007 (net/virtio-user: do not stop stopped device again) isn't needed to be backported to 17.11.5 release. As it was to fix a harmless error message 
> "vhost_kernel_set_backend(): VHOST_NET_SET_BACKEND fails, Operation not permitted
> vhost_kernel_set_backend(): VHOST_NET_SET_BACKEND fails, Operation not permitted" 
> when application(like testpmd) calling rte_eth_dev_close() and then calling rte_dev_remove().  Also, it depends on commit 12ecb2f63 (net/virtio-user: support memory hotplug), which is a feature commit.

Thanks, I dropped it from the list.

Yongseok

> 
> 
> -----Original Message-----
> From: Yongseok Koh [mailto:yskoh at mellanox.com] 
> Sent: Friday, November 30, 2018 7:44 AM
> To: dpdk stable <stable at dpdk.org>
> Cc: Ajit Khaparde <ajit.khaparde at broadcom.com>; Akhil Goyal <akhil.goyal at nxp.com>; Alejandro Lucero <alejandro.lucero at netronome.com>; Alex Smith <alex.smith at example.com>; Ali Alnubani <alialnu at mellanox.com>; Allain Legacy <allain.legacy at windriver.com>; Burakov, Anatoly <anatoly.burakov at intel.com>; Andrew Rybchenko <arybchenko at solarflare.com>; Andy Moreton <amoreton at solarflare.com>; Xing, Beilei <beilei.xing at intel.com>; Iremonger, Bernard <bernard.iremonger at intel.com>; Brian Russell <brussell at brocade.com>; Richardson, Bruce <bruce.richardson at intel.com>; Chao Zhu <chaozhu at linux.vnet.ibm.com>; Damjan Marion <damarion at cisco.com>; David Marchand <david.marchand at redhat.com>; David Wilder <dwilder at us.ibm.com>; Eric Zhang <eric.zhang at windriver.com>; Zhang, Roy Fan <roy.fan.zhang at intel.com>; Yigit, Ferruh <ferruh.yigit at intel.com>; Gaetan Rivet <gaetan.rivet at 6wind.com>; Gautam Dawar <gdawar at solarflare.com>; Gavin Hu <gavin.hu at arm.com>; Van Haaren, Harry <harry.van.haaren at intel.com>; Honnappa Nagarahalli <honnappa.nagarahalli at arm.com>; Ian Dolzhansky <ian.dolzhansky at oktetlabs.ru>; Igor Romanov <igor.romanov at oktetlabs.ru>; Ilya Maximets <i.maximets at samsung.com>; Ivan Malov <ivan.malov at oktetlabs.ru>; Jerin Jacob <jerin.jacob at caviumnetworks.com>; Hu, Jiayu <jiayu.hu at intel.com>; Mcnamara, John <john.mcnamara at intel.com>; Joyce Kong <joyce.kong at arm.com>; Junxiao Shi <git at mail1.yoursunny.com>; Wiles, Keith <keith.wiles at intel.com>; Ananyev, Konstantin <konstantin.ananyev at intel.com>; Daly, Lee <lee.daly at intel.com>; Luca Boccassi <bluca at debian.org>; Kovacevic, Marko <marko.kovacevic at intel.com>; Martin Harvey <mharvey at solarflare.com>; Maxime Coquelin <maxime.coquelin at redhat.com>; Somarowthu, Naga SureshX <naga.sureshx.somarowthu at intel.com>; Neil Horman <nhorman at tuxdriver.com>; Rao, Nikhil <nikhil.rao at intel.com>; Ola Liljedahl <ola.liljedahl at arm.com>; Poornima, PallantlaX <pallantlax.poornima at intel.com>; Pradeep Satyanarayana <pradeep at us.ibm.com>; Zhang, Qi Z <qi.z.zhang at intel.com>; Rasesh Mody <rasesh.mody at cavium.com>; Pattan, Reshma <reshma.pattan at intel.com>; Shahaf Shuler <shahafs at mellanox.com>; Shahed Shaikh <shahed.shaikh at cavium.com>; Shreyansh Jain <shreyansh.jain at nxp.com>; Somnath Kotur <somnath.kotur at broadcom.com>; Stephen Hemminger <stephen at networkplumber.org>; Steve Capper <steve.capper at arm.com>; Thomas Monjalon <thomas at monjalon.net>; Zhang, Tianfei <tianfei.zhang at intel.com>; Bie, Tiwei <tiwei.bie at intel.com>; Zhao1, Wei <wei.zhao1 at intel.com>; Lu, Wenzhuo <wenzhuo.lu at intel.com>; Xiaoxin Peng <xiaoxin.peng at broadcom.com>; Wu, Yanglong <yanglong.wu at intel.com>; Wang, Yipeng1 <yipeng1.wang at intel.com>; Yan, Zhirun <zhirun.yan at intel.com>
> Subject: please help backporting some patches to LTS release 17.11.5
> 
> Hi commit authors (and maintainers),
> 
> I didn't manage to apply following commits from upstream to stable branch 17.11:
> conflict happens. Or, even if a patch could be applied cleanly, some of them caused compilation errors.
> 
> I'm wondering can the authors check the following list and backport those patches belong to you?
> 
> FYI, branch 17.11 is located at tree:
>   git://dpdk.org/dpdk-stable
> 
> It'd be great if you could do that in one or two weeks. Also, please add a heading line like below before the commit log body:
>    [ backported from upstream commit xxx ]
> 
> Example: https://emea01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fdpdk.org%2Fbrowse%2Fdpdk-stable%2Fcommit%2F%3Fh%3D16.07%26id%3Dc4831394c7d1944d8ec27d52c22997f20d19718e&data=02%7C01%7Cyskoh%40mellanox.com%7C6a7e6e58a1a04274eb1a08d664c3ef7f%7Ca652971c7d2e4d9ba6a4d149256f461b%7C0%7C0%7C636807190038715645&sdata=rYqax0BZJsze0cg0yewwsUHXHgVNgIRBl7SkiCU99wE%3D&reserved=0
> 
> Also please mention the target LTS in the subject line, as we have more than one at the same time, for example:
> 
>    [PATCH 17.11] foo/bar: fix baz
> 
> With git send-email, this can be achieved by appending the parameter:
> 
>    --subject-prefix='17.11'
> 
> Please let me know if you have any comments, say, need more time, or it's worthless to packport it. And please send it to "stable at dpdk.org", but not "dev at dpdk.org".
> 
> Thanks,
> Yongseok
> 
> ---
> 80ad678ae  Ajit Khaparde    net/bnxt: fix MTU setting
> 6f93b2aa2  Ajit Khaparde    net/bnxt: remove excess log messages
> 5bf10cdb6  Ajit Khaparde    net/bnxt: set a VNIC as default only once
> 0de9eb613  Alejandro Lucero mem: rename DMA mask check with proper prefix
> 7712a1c66  Alejandro Lucero net/nfp: fix live MAC changes not supported
> 0c15f3c01  Ali Alnubani     net/mlx5: fix initialization of struct members
> 416aa7f1c  Andy Moreton     net/sfc/base: fix ID retrieval in v3 licensing
> 054d1be48  Beilei Xing      net/i40e: fix Rx instability with vector mode
> 3c4c76cf1  Damjan Marion    net/i40e: fix 25G AOC and ACC cable detection on XXV710
> 6b062d56b  David Wilder     mem: fix anonymous mapping on Power9
> 1db4d2330  Eric Zhang       net/virtio-user: check negotiated features before set
> 1fd1aa0c9  Fan Zhang        crypto/aesni_mb: fix queue pair free
> 5b4557ec8  Ferruh Yigit     app/testpmd: fix displaying RSS hash functions
> 335e57d35  Ferruh Yigit     doc: add missing shared library versions to release notes
> a8b52dafe  Ferruh Yigit     doc: fix vhost library version in release notes
> 520dd9923  Ferruh Yigit     drivers: prefix global variables with module name
> 975735834  Ferruh Yigit     fix global variable issues
> c06aee624  Ferruh Yigit     net/bnxt: fix uninitialized variable access
> c8b506e4b  Ferruh Yigit     service: fix possible null access
> ed42d1845  Gautam Dawar     net/sfc/base: fix out of bounds read when dereferencing sdup
> dbdbc4a2e  Honnappa Nagarahalli hash: fix key store element alignment
> c3a210a26  Ian Dolzhansky   net/failsafe: add checks for deferred queue setup
> 6b35f4d88  Igor Romanov     net/failsafe: fix crash on slave queue release
> 0408f47ba  Ilya Maximets    net/ixgbe: fix busy polling while fiber link update
> 916193dda  Ilya Maximets    net/ixgbe: stop link setup alarm handler before start
> 705dced4a  Ilya Maximets    net/virtio: avoid annoying IOPL error log
> 0d7853a4d  Ilya Maximets    vhost-user: drop connection on message handling failures
> 453e1d4ba  Ivan Malov       net/sfc: allow to query RSS key and HF in isolated mode
> 662e38224  Ivan Malov       net/sfc: allow to query RSS key and HF when RSS is disabled
> 5205c436a  Ivan Malov       net/sfc: do not skip RSS configuration step on reconfigure
> c23e46594  Jerin Jacob      mbuf: fix offload flag name and list
> 729199397  Jiayu Hu         vhost: fix corner case for enqueue operation
> 9b47ed438  Joyce Kong       config: enable more than 128 cores for arm64
> f49d7e9a9  Junxiao Shi      cryptodev: fix pool element size for undefined operation
> 6f4ae7f5d  Lee Daly         net/bonding: fix possible silent failure in configuration
> f9b0d1902  Luca Boccassi    net/virtio: register/unregister intr handler on start/stop
> f394c63d2  Martin Harvey    net/sfc/base: fix PreFAST warnings because of unused return
> 57b4d90b5  Maxime Coquelin  vhost: fix payload size of reply
> ee1caebc4  Naga Suresh Somarowthu test/kni: check module dependency
> 5835b7d37  Pallantla Poornima test/kni: fix module miss fallback
> 188d0bda2  Qi Zhang         net/i40e/base: fix partition id calculation for X722
> 54f4a22dc  Qi Zhang         net/i40e/base: improve the polling mechanism
> b72611a27  Qi Zhang         net/i40e/base: read LLDP config area with correct endianness
> 853c78249  Rasesh Mody      net/bnx2x: fix dynamic logging
> ba7eeb035  Rasesh Mody      net/bnx2x: fix logging to include device name
> a9b58b15e  Rasesh Mody      net/bnx2x: fix to add PHY lock
> 39ae74caa  Rasesh Mody      net/bnx2x: fix VF link state update
> cb7199279  Rasesh Mody      net/qede/base: fix to handle stag update event
> d469f6a5e  Shahaf Shuler    net/mlx5: add representor specific statistics
> 1a611fdaf  Shahaf Shuler    net/mlx5: support missing counter in extended statistics
> 797ce8eec  Shahed Shaikh    net/qede/base: fix MFW FLR flow
> 7ffef6292  Shahed Shaikh    net/qede: fix flow director for IPv6 filter
> 318d7da31  Shahed Shaikh    net/qede: fix Rx buffer size calculation
> 49d3978d5  Shahed Shaikh    net/qede: fix Tx tunnel offload support mask
> 51fafb89a  Somnath Kotur    net/bnxt: get rid of ff pools and use VNIC info array
> 987e0def5  Stephen Hemminger doc: fix spelling in PMD guides
> cc02c9771  Stephen Hemminger net/tap: fix file descriptor leak on error
> 8f3ea0a03  Thomas Monjalon  devtools: fix printing subject of checked patch
> ff37ca5d3  Thomas Monjalon  devtools: use a common prefix for temporary files f6a12685a  Thomas Monjalon  ethdev: fix doxygen comments of shared data fields
> 43d162bc1  Thomas Monjalon  fix dpdk.org URLs c5f21bdae  Thomas Monjalon  fix indentation in symbol maps
> 1a7a39eb3  Tiwei Bie        net/vhost: fix parameters string
> 66908eff1  Tiwei Bie        net/virtio: add missing supported features
> 74dc6746a  Tiwei Bie        net/virtio-user: do not reset owner when driver resets
> f457e9007  Tiwei Bie        net/virtio-user: do not stop stopped device again
> aa28ec5d2  Wenzhuo Lu       ethdev: fix invalid configuration after failure
> a83090006  Xiaoxin Peng     net/bnxt: reduce polling interval for valid bit
> ff30a020b  Yanglong Wu      net/ixgbe: fix TDH register write
> 2d28bb5dd  Yipeng Wang      hash: remove unnecessary pause
> 545db54c7  Yongseok Koh     net/mlx5: optimize Rx buffer replenishment threshold
> d31c84443  Zhirun Yan       net/avf: fix Tx offload mask



More information about the stable mailing list