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

[Yongseok Koh <yskoh@mellanox.com>]

> On Dec 18, 2018, at 12:36 AM, Cao, Yahui <yahui.cao@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@mellanox.com] 
> Sent: Friday, November 30, 2018 7:44 AM
> To: dpdk stable <stable@dpdk.org>
> Cc: Ajit Khaparde <ajit.khaparde@broadcom.com>; Akhil Goyal <akhil.goyal@nxp.com>; Alejandro Lucero <alejandro.lucero@netronome.com>; Alex Smith <alex.smith@example.com>; Ali Alnubani <alialnu@mellanox.com>; Allain Legacy <allain.legacy@windriver.com>; Burakov, Anatoly <anatoly.burakov@intel.com>; Andrew Rybchenko <arybchenko@solarflare.com>; Andy Moreton <amoreton@solarflare.com>; Xing, Beilei <beilei.xing@intel.com>; Iremonger, Bernard <bernard.iremonger@intel.com>; Brian Russell <brussell@brocade.com>; Richardson, Bruce <bruce.richardson@intel.com>; Chao Zhu <chaozhu@linux.vnet.ibm.com>; Damjan Marion <damarion@cisco.com>; David Marchand <david.marchand@redhat.com>; David Wilder <dwilder@us.ibm.com>; Eric Zhang <eric.zhang@windriver.com>; Zhang, Roy Fan <roy.fan.zhang@intel.com>; Yigit, Ferruh <ferruh.yigit@intel.com>; Gaetan Rivet <gaetan.rivet@6wind.com>; Gautam Dawar <gdawar@solarflare.com>; Gavin Hu <gavin.hu@arm.com>; Van Haaren, Harry <harry.van.haaren@intel.com>; Honnappa Nagarahalli <honnappa.nagarahalli@arm.com>; Ian Dolzhansky <ian.dolzhansky@oktetlabs.ru>; Igor Romanov <igor.romanov@oktetlabs.ru>; Ilya Maximets <i.maximets@samsung.com>; Ivan Malov <ivan.malov@oktetlabs.ru>; Jerin Jacob <jerin.jacob@caviumnetworks.com>; Hu, Jiayu <jiayu.hu@intel.com>; Mcnamara, John <john.mcnamara@intel.com>; Joyce Kong <joyce.kong@arm.com>; Junxiao Shi <git@mail1.yoursunny.com>; Wiles, Keith <keith.wiles@intel.com>; Ananyev, Konstantin <konstantin.ananyev@intel.com>; Daly, Lee <lee.daly@intel.com>; Luca Boccassi <bluca@debian.org>; Kovacevic, Marko <marko.kovacevic@intel.com>; Martin Harvey <mharvey@solarflare.com>; Maxime Coquelin <maxime.coquelin@redhat.com>; Somarowthu, Naga SureshX <naga.sureshx.somarowthu@intel.com>; Neil Horman <nhorman@tuxdriver.com>; Rao, Nikhil <nikhil.rao@intel.com>; Ola Liljedahl <ola.liljedahl@arm.com>; Poornima, PallantlaX <pallantlax.poornima@intel.com>; Pradeep Satyanarayana <pradeep@us.ibm.com>; Zhang, Qi Z <qi.z.zhang@intel.com>; Rasesh Mody <rasesh.mody@cavium.com>; Pattan, Reshma <reshma.pattan@intel.com>; Shahaf Shuler <shahafs@mellanox.com>; Shahed Shaikh <shahed.shaikh@cavium.com>; Shreyansh Jain <shreyansh.jain@nxp.com>; Somnath Kotur <somnath.kotur@broadcom.com>; Stephen Hemminger <stephen@networkplumber.org>; Steve Capper <steve.capper@arm.com>; Thomas Monjalon <thomas@monjalon.net>; Zhang, Tianfei <tianfei.zhang@intel.com>; Bie, Tiwei <tiwei.bie@intel.com>; Zhao1, Wei <wei.zhao1@intel.com>; Lu, Wenzhuo <wenzhuo.lu@intel.com>; Xiaoxin Peng <xiaoxin.peng@broadcom.com>; Wu, Yanglong <yanglong.wu@intel.com>; Wang, Yipeng1 <yipeng1.wang@intel.com>; Yan, Zhirun <zhirun.yan@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&amp;data=02%7C01%7Cyskoh%40mellanox.com%7C6a7e6e58a1a04274eb1a08d664c3ef7f%7Ca652971c7d2e4d9ba6a4d149256f461b%7C0%7C0%7C636807190038715645&amp;sdata=rYqax0BZJsze0cg0yewwsUHXHgVNgIRBl7SkiCU99wE%3D&amp;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@dpdk.org", but not "dev@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