Re: 17.08.1 patches review and test

[Yuanhan Liu <yliu@fridaylinux.org>]

On Mon, Dec 04, 2017 at 08:40:03PM +0000, Patil, Harish wrote:
> 
> 
> -----Original Message-----
> From: dev <dev-bounces@dpdk.org> on behalf of Yuanhan Liu
> <yliu@fridaylinux.org>
> Date: Monday, November 27, 2017 at 4:21 AM
> To: dpdk stable <stable@dpdk.org>
> Cc: "dev@dpdk.org" <dev@dpdk.org>, "Xu, Qian Q" <qian.q.xu@intel.com>
> Subject: [dpdk-dev] 17.08.1 patches review and test
> 
> >Hi all,
> >
> >Here is a list of patches targeted for stable release 17.08.1. Please
> >help review and test. The planned date for the final release is 7th,
> >Dec. Before that, please shout if anyone has objections with these
> >patches being applied.
> >
> >These patches are located at branch 17.08 of dpdk-stable repo:
> >    http://dpdk.org/browse/dpdk-stable/
> >
> >Thanks.
> >
> >        --yliu
> >
> >---
> >Aaron Conole (1):
> >      net/enic: fix assignment
> >
> >Ajit Khaparde (28):
> >      net/bnxt: fix HWRM macros and locking
> >      net/bnxt: use 64-bits of address for VLAN table
> >      net/bnxt: fix an issue with group id calculation
> >      net/bnxt: fix calculation of number of pools
> >      net/bnxt: handle multi queue mode properly
> >      net/bnxt: fix Rx handling and buffer allocation logic
> >      net/bnxt: fix an issue with broadcast traffic
> >      net/bnxt: fix usage of VMDq flags
> >      net/bnxt: set checksum offload flags correctly
> >      net/bnxt: update status of Rx IP/L4 CKSUM
> >      net/bnxt: fix config RSS update
> >      net/bnxt: set the hash key size
> >      net/bnxt: fix per queue stats display in xstats
> >      net/bnxt: fix interrupt handler
> >      net/bnxt: fix number of MAC addresses for VMDq
> >      net/bnxt: fix the association of a MACVLAN per VNIC
> >      net/bnxt: fix Tx offload capability
> >      net/bnxt: fix Rx offload capability
> >      net/bnxt: handle Rx multi queue creation properly
> >      net/bnxt: remove redundant code parsing pool map
> >      net/bnxt: fix a bit shift operation
> >      net/bnxt: fix a potential null pointer dereference
> >      net/bnxt: fix a potential null pointer dereference
> >      net/bnxt: fix a pointer deref before null check
> >      net/bnxt: fix an unused value
> >      net/bnxt: check VLANs from pool map only for VMDq
> >      net/bnxt: do not set hash type unnecessarily
> >      net/bnxt: fix VLAN spoof configuration
> >
> >Akhil Goyal (2):
> >      test/crypto: fix dpaa2 sec macros and definitions
> >      net/dpaa2: set queues after reconfiguration
> >
> >Alejandro Lucero (2):
> >      net/nfp: fix RSS
> >      net/nfp: fix Rx interrupt when multiqueue
> >
> >Alok Makhariya (2):
> >      crypto/dpaa2_sec: remove ICV memset on decryption side
> >      crypto/dpaa2_sec: add check for segmented buffer
> >
> >Anatoly Burakov (1):
> >      vfio: fix secondary process initialization
> >
> >Andrey Chilikin (1):
> >      net/i40e: fix flexible payload configuration
> >
> >Aviad Yehezkel (4):
> >      examples/ipsec-secgw: fix crypto device mapping
> >      examples/ipsec-secgw: fix session creation
> >      examples/ipsec-secgw: fix AAD length setting
> >      app/testpmd: fix build without ixgbe and bnxt PMDs
> >
> >Beilei Xing (1):
> >      net/i40e: fix VF device stop issue
> >
> >Chas Williams (1):
> >      net/vmxnet3: fix memory leak when releasing queues
> >
> >Congwen Zhang (1):
> >      net/cxgbe: fix memory leak
> >
> >Daniel Mrzyglod (3):
> >      net/virtio: fix untrusted scalar value
> >      app/testpmd: fix DDP package filesize detection
> >      net/bonding: fix default aggregator mode to stable
> >
> >David Harton (2):
> >      net/vmxnet3: fix MAC address set
> >      net/i40e: fix i40evf MAC filter table
> >
> >Ferruh Yigit (4):
> >      ethdev: fix ABI version
> >      ethdev: revert use port name from device structure
> >      igb_uio: remove device reset in open
> >      net/qede: fix icc build
> >
> >Gaetan Rivet (1):
> >      net/failsafe: fix errno set on command execution
> >
> >Gowrishankar Muthukrishnan (1):
> >      net/bonding: support bifurcated driver in eal
> >
> >Guduri Prathyusha (2):
> >      examples/l3fwd: fix NEON instructions
> >      examples/l3fwd: fix aliasing in port grouping
> >
> >Harish Patil (2):
> >      net/qede: fix supported packet types
> >      net/qede: fix to re-enable LRO during device start
> >
> >Hemant Agrawal (3):
> >      net/dpaa2: fix the Tx handling of non HW pool bufs
> >      examples/l2fwd-crypto: fix uninitialized errno value
> >      app/crypto-perf: fix uninitialized errno value
> >
> >Ian Stokes (1):
> >      cryptodev: fix build with -Ofast
> >
> >Ivan Malov (2):
> >      net/sfc: specify correct scale table size on Rx start
> >      net/sfc: fix unused variable in RSS-agnostic build
> >
> >Jacek Piasecki (1):
> >      examples/vhost_scsi: fix product id string termination
> >
> >Jasvinder Singh (1):
> >      examples/qos_sched: fix uninitialized config
> >
> >Jerin Jacob (1):
> >      timer: use 64-bit specific code on more platforms
> >
> >Jianbo Liu (1):
> >      net/i40e: fix Rx packets number for NEON
> >
> >Jiayu Hu (1):
> >      gro: fix typo in map file
> >
> >Jingjing Wu (4):
> >      net/i40e: fix interrupt throttling setting in PF
> >      net/i40e: fix memory leak if VF init fails
> >      net/i40e: fix variable assignment
> >      net/i40e: fix VF initialization error
> >
> >John Daley (3):
> >      net/enic: fix multi-process operation
> >      net/enic: fix packet loss after MTU change
> >      net/enic: fix TSO for packets greater than 9208 bytes
> >
> >John Miller (1):
> >      net/ark: fix loop counter
> >
> >Kuba Kozak (2):
> >      vfio: fix close unchecked file descriptor
> >      vhost: check poll error code
> >
> >Li Han (1):
> >      app/testpmd: fix invalid port id parameters
> >
> >Lukasz Majczak (1):
> >      eal: fix auxv open check for ARM and PPC
> >
> >Mark Spender (1):
> >      net/sfc/base: fix default RSS context check on Siena
> >
> >Matan Azrad (7):
> >      net/failsafe: fix Tx sub device deactivating
> >      net/mlx5: fix locking in xstats functions
> >      net/failsafe: fix parameters parsing
> >      net/mlx5: fix probe failure report
> >      net/failsafe: fix adding MAC error report miss
> >      app/testpmd: fix forward port ids setting
> >      net/failsafe: fix Rx clean race
> >
> >Maxime Coquelin (1):
> >      vhost: fix dereferencing invalid pointer after realloc
> >
> >Michal Jastrzebski (1):
> >      net/vmxnet3: fix dereference before null check
> >
> >Nikhil Rao (1):
> >      eal/x86: fix atomic cmpset
> >
> >Nirmoy Das (1):
> >      kni: fix SLE version detection
> >
> >Nélio Laranjeiro (4):
> >      net/mlx5: fix clang build
> >      net/mlx5: fix SSE Rx support verification
> >      net/mlx5: fix clang compilation error
> >      app/testpmd: fix RSS structure initialisation
> >
> >Olivier Matz (12):
> >      net: fix inner L2 length in packet type parser
> >      net/bnxt: fix compilation with -Og
> >      net/qede: fix compilation with -Og
> >      app/test-crypto-perf: fix memory leak
> >      app/test-crypto-perf: fix compilation with -Og
> >      net/virtio: revert not claiming LRO support
> >      net/virtio: revert not claiming IP checksum offload
> >      net/virtio: fix log levels in configure
> >      net/virtio: fix mbuf port for simple Rx function
> >      net/virtio: fix queue setup consistency
> >      net/virtio: fix compilation with -Og
> >      lpm6: fix compilation with -Og
> >
> >Ophir Munk (3):
> >      net/tap: fix flow and port commands
> >      net/failsafe: fix VLAN stripping configuration
> >      app/testpmd: fix forwarding between non consecutive ports
> >
> >Pablo de Lara (8):
> >      hash: fix eviction counter
> >      crypto/aesni_gcm: fix zero data operation
> >      app/crypto-perf: fix packet length check
> >      app/crypto-perf: parse AEAD data from vectors
> >      crypto/openssl: fix AEAD parameters
> >      examples/l2fwd-crypto: fix physical address setting
> >      crypto/qat: fix HMAC supported digest sizes
> >      app/testpmd: fix topology error message
> >
> >Patrick MacArthur (1):
> >      eal: copy raw strings taken from command line
> >
> >Phil Yang (1):
> >      app/testpmd: fix quitting in container
> >
> >Qi Zhang (4):
> >      net/i40e: fix flow control watermark mismatch
> >      net/i40e: fix packet count for PF
> >      net/i40e: fix mbuf free in vector Tx
> >      net/i40e: fix mirror with firmware 6.0
> >
> >Rami Rosen (1):
> >      net/kni: remove driver struct forward declaration
> >
> >Rasesh Mody (4):
> >      net/qede/base: fix to use a passed ptt handle
> >      net/qede/base: fix return code to align with FW
> >      net/qede: remove duplicate includes
> >      net/qede/base: fix division by zero
> >
> >Raslan Darawsheh (2):
> >      net/failsafe: fix failsafe bus uninit return value
> >      net/failsafe: fix PCI devices init
> >
> >RongQiang Xie (2):
> >      net/enic: fix possible null pointer dereference
> >      net/qede: fix possible null pointer dereference
> >
> >Sebastian Basierski (3):
> >      net/vmxnet3: fix unintentional integer overflow
> >      net/virtio-user: fix TAP name string termination
> >      net/virtio: check error on setting non block flag
> >
> >Sergio Gonzalez Monroy (1):
> >      crypto/aesni_mb: fix invalid session error
> >
> >Shahaf Shuler (5):
> >      net/mlx5: fix num seg assumption in SSE Tx
> >      net/mlx5: fix Tx stats error counter definition
> >      net/mlx5: fix Tx stats error counter logic
> >      net/mlx5: fix TSO segment size verification
> >      net/mlx5: fix packet type flags for Ethernet only frame
> >
> >Stefan Baranoff (1):
> >      net/pcap: fix memory leak in dumper open
> >
> >Stephen Hemminger (1):
> >      eal: initialize logging before bus
> >
> >Tiwei Bie (1):
> >      net/virtio: flush Rx queues on start
> >
> >Tomasz Duszynski (3):
> >      drivers/crypto: use snprintf return value correctly
> >      examples/ipsec-secgw: fix IP version check
> >      examples/ipsec-secgw: fix IPv6 payload length
> >
> >Tomasz Kulasek (3):
> >      net/bonding: fix slaves capacity check
> >      net/i40e: fix assignment of enum values
> >      net/bonding: fix check slaves link properties
> >
> >Vipin Varghese (1):
> >      net/tap: fix unregistering callback with invalid fd
> >
> >Wei Dai (8):
> >      net/ixgbe: fix mapping of user priority to TC
> >      net/ixgbe: fix adding a mirror rule
> >      net/i40e: fix mirror rule reset when port is closed
> >      net/ixgbe: fix Rx queue interrupt mapping in VF
> >      net/ixgbe: fix VFIO interrupt mapping in VF
> >      net/ixgbe: fix PF DCB info
> >      app/testpmd: fix mapping of user priority to DCB TC
> >      net/i40e: fix VFIO interrupt mapping in VF
> >
> >Wei Zhao (4):
> >      net/ixgbe: fix MAC VLAN filter fail problem
> >      net/i40e: fix clear xstats bug in VF
> >      app/testpmd: fix packet throughput after stats reset
> >      net/ixgbe: fix filter parser for L2 tunnel
> >
> >Wenzhuo Lu (7):
> >      net/i40e: fix TM node parameter checking
> >      net/i40e: fix TM level capability getting
> >      net/ixgbe: fix TM node parameter checking
> >      net/ixgbe: fix TM level capability getting
> >      net/i40e: fix not supporting NULL TM profile
> >      net/ixgbe: fix not supporting NULL TM profile
> >      net/i40e: fix parent when adding TM node
> >
> >Xiaoyun Li (2):
> >      net/i40e: fix PF notify issue when VF is not up
> >      net/igb: fix Rx interrupt with VFIO and MSI-X
> >
> >Xueming Li (5):
> >      net/mlx5: fix tunnel offload detection
> >      mem: fix malloc debug config
> >      mem: fix malloc element free in debug mode
> >      examples/l2fwd_fork: fix message pool init
> >      examples/multi_process: fix received message length
> >
> >Yi Yang (1):
> >      service: fix build with gcc 4.9
> >
> >Yong Wang (4):
> >      net/liquidio: fix uninitialized variable
> >      net/igb: fix memcpy length
> >      net/i40e: fix uninitialized variable
> >      net/ixgbe: fix uninitialized variable
> >
> >Yongseok Koh (3):
> >      net/mlx5: fix calculating TSO inline size
> >      net/mlx5: fix overflow of Rx SW ring
> >      net/mlx5: fix tunneled TCP/UDP packet type
> >
> >Zhiyong Yang (2):
> >      test: fix assignment operation
> >      net/virtio: fix Tx packet length stats
> 
> 
> Hi Yuanhan,
> I found the following two patches marked for stable release is not picked
> up here.
> 
> commit f07aa795c92ac6a9460d25ab2e9d74379ed8e5eb
> Author: Harish Patil <harish.patil@cavium.com>
> Date:   Tue Nov 7 00:34:18 2017 -0800
> 
>     net/qede: disable per-VF Tx switching feature
>     
>     Provide a knob to control per-VF Tx switching feature by adding a
> config
>     option, CONFIG_RTE_LIBRTE_QEDE_VF_TX_SWITCH. By default, it will be
> kept
>     in disabled state for better performance with small sized frames.
>     
>     Fixes: 2ea6f76aff40 ("qede: add core driver")
>     Cc: stable@dpdk.org
>     
>     Signed-off-by: Harish Patil <harish.patil@cavium.com>
> 
> commit 1282943aa05b2f211bd1b831b9d2962859323063
> Author: Harish Patil <harish.patil@cavium.com>
> Date:   Wed Nov 8 22:52:20 2017 -0800
> 
>     net/qede: fix default config option
>     
>     Restore the default configuration as in previous releases and
>     add a debug msg.
>     
>     Fixes: f07aa795c92a ("net/qede: disable per-VF Tx switching feature")
>     Cc: stable@dpdk.org
>     
>     Signed-off-by: Harish Patil <harish.patil@cavium.com>
>     Signed-off-by: Rasesh Mody <rasesh.mody@cavium.com>
> 
> 
> 
> We need to include these two patches as well.

Now they were included. Thanks!

	--yliu