RE: release candidate 22.07-rc4

["Jiang, YuX" <yux.jiang@intel.com>]

> -----Original Message-----
> From: Jiang, YuX <yux.jiang@intel.com>
> Sent: Thursday, July 14, 2022 4:18 PM
> To: Xia, Chenbo <chenbo.xia@intel.com>; Thomas Monjalon
> <thomas@monjalon.net>; dev (dev@dpdk.org) <dev@dpdk.org>; Maxime
> Coquelin <maxime.coquelin@redhat.com>
> Cc: Devlin, Michelle <michelle.devlin@intel.com>; Mcnamara, John
> <john.mcnamara@intel.com>; Ferruh Yigit <ferruh.yigit@xilinx.com>
> Subject: RE: release candidate 22.07-rc4
> 
> > -----Original Message-----
> > From: Xia, Chenbo <chenbo.xia@intel.com>
> > Sent: Tuesday, July 12, 2022 5:45 PM
> > To: Jiang, YuX <yux.jiang@intel.com>; Thomas Monjalon
> > <thomas@monjalon.net>; dev (dev@dpdk.org) <dev@dpdk.org>; Maxime
> > Coquelin <maxime.coquelin@redhat.com>
> > Cc: Devlin, Michelle <michelle.devlin@intel.com>; Mcnamara, John
> > <john.mcnamara@intel.com>; Ferruh Yigit <ferruh.yigit@xilinx.com>
> > Subject: RE: release candidate 22.07-rc4
> >
> > > -----Original Message-----
> > > From: Jiang, YuX <yux.jiang@intel.com>
> > > Sent: Tuesday, July 12, 2022 5:33 PM
> > > To: Thomas Monjalon <thomas@monjalon.net>; dev (dev@dpdk.org)
> > > <dev@dpdk.org>
> > > Cc: Devlin, Michelle <michelle.devlin@intel.com>; Mcnamara, John
> > > <john.mcnamara@intel.com>; Ferruh Yigit <ferruh.yigit@xilinx.com>;
> > > Xia, Chenbo <chenbo.xia@intel.com>
> > > Subject: RE: release candidate 22.07-rc4
> > >
> > > > -----Original Message-----
> > > > From: Thomas Monjalon <thomas@monjalon.net>
> > > > Sent: Tuesday, July 12, 2022 6:37 AM
> > > > To: announce@dpdk.org
> > > > Subject: release candidate 22.07-rc4
> > > >
> > > > A new DPDK release candidate is ready for testing:
> > > > 	https://git.dpdk.org/dpdk/tag/?id=v22.07-rc4
> > > >
> > > > There are 22 new patches in this snapshot.
> > > >
> > > > Release notes:
> > > > 	https://doc.dpdk.org/guides/rel_notes/release_22_07.html
> > > >
> > > > This is the last release candidate.
> > > > Only documentation should be updated before the release.
> > > >
> > > > Reviews of deprecation notices are required:
> > > > https://patches.dpdk.org/bundle/dmarchand/deprecation_notices/
> > > >
> > > > You may share some release validation results by replying to this
> > > message at
> > > > dev@dpdk.org and by adding tested hardware in the release notes.
> > > >
> > > > Please think about sharing your roadmap now for DPDK 22.11.
> > > >
> > > > Thank you everyone
> > > >
> > > Hi
> > >
> > > Update the test status for Intel part. Till now dpdk22.07-rc4
> > > regression is running smooth.
> > > But we find one high issue about "[dpdk-
> > >
> >
> 22.07]loopback_virtio_user_server_mode/server_mode_reconnect_with_s
> > pli
> > > t_al
> > > l_path_payload_check: the first packet payload is different from the
> > > ohter packets in the same queue", Bad commit id is
> > > cd79d1b030bd69df31fd401d1b48c23df0f71d00(which is used to fix the
> > > performance drop of split ring)
> >
> > I just found the potential root cause, Maxime helped confirm it and
> > patch will be out soon.
> >
> > Thanks,
> > Chenbo
> >
> All,
> 
> Update the test status for Intel part. Till now dpdk22.07-rc4 regression is
> almost finished, two new bugs are found.
> The high issue about "[dpdk-
> 22.07]loopback_virtio_user_server_mode/server_mode_reconnect_with_s
> plit_all_path_payload_check: the first packet payload is different from the
> ohter packets in the same queue".
> 	It has fix link:
> http://patchwork.dpdk.org/project/dpdk/patch/20220712100447.104162-1-
> maxime.coquelin@redhat.com/, Intel validation team has completed
> validtion test for the patch, no new issue is found.

This patch has been merged into 22.07 main branch, and test passed by Intel Validation team.

> Other one issue about
> "vswitch_sample_cbdma:perf_pvp_vhost_async_operation_with_cbdma_2
> _channels: virtio-user will drop all the packets when sending large 1280 Byte
> packets".
> 	It has fix link:
> https://patches.dpdk.org/project/dpdk/patch/20220714051106.1134222-1-
> wenwux.ma@intel.com/ and tested-by by Intel, but no acked-by yet.
> 
As discussed by Chenbo and maintainer in yesterday's release meeting, this patch should have minor impact, so the fix could be moved to next release.

BR
Yu Jiang

> # Basic Intel(R) NIC testing
> * Build or compile:
>  *Build: cover the build test combination with latest GCC/Clang version and
> the popular OS revision such as Ubuntu20.04, Ubuntu22.04, Fedora36,
> RHEL8.6 etc.
>   - All test passed.
> 
> * PF/VF(i40e, ixgbe): test scenarios including PF/VF-
> RTE_FLOW/TSO/Jumboframe/checksum offload/VLAN/VXLAN, etc.
> 	- All test done. No new issue is found
> 
> * PF/VF(ice): test scenarios including Switch features/Package
> Management/Flow Director/Advanced Tx/Advanced RSS/ACL/DCF/Flexible
> Descriptor, etc.
> 	- All test done. No new issue is found.
> 
> * Intel NIC single core/NIC performance: test scenarios including PF/VF single
> core performance test, RFC2544 Zero packet loss performance test, etc.
> 	- All test done. No new issue is found.
> 
> # Basic cryptodev and virtio testing
> * Virtio: both function and performance test are covered. Such as
> PVP/Virtio_loopback/virtio-user loopback/virtio-net VM2VM perf
> testing/VMAWARE ESXI 7.0u3, etc.
> 	- All test done. Find two new bugs as above.
> 
> * Cryptodev:
>  *Function test: test scenarios including Cryptodev API testing/CompressDev
> ISA-L/QAT/ZLIB PMD Testing/FIPS, etc.
> 	- All test done. No new new isue is found.
>  *Performance test: test scenarios including Throughput
> Performance/Cryptodev Latency, etc.
> 	- All test done. No new new isue is found.
> 
> Best regards,
> Yu Jiang
> 
> > >
> > > commit cd79d1b030bd69df31fd401d1b48c23df0f71d00 (HEAD)
> > > Author: Xuan Ding <xuan.ding@intel.com>
> > > Date:   Thu Jul 7 06:55:13 2022 +0000
> > >
> > >     vhost: fix unnecessary dirty page logging
> > >
> > >     The dirty page logging is only required in vhost enqueue direction for
> > >     live migration. This patch removes the unnecessary dirty page logging
> > >     in vhost dequeue direction. Otherwise, it will result in a performance
> > >     drop. Some if-else judgements are also optimized to improve
> > > performance.
> > >
> > >     Fixes: 6d823bb302c7 ("vhost: prepare sync for descriptor to mbuf
> > > refactoring")
> > >     Fixes: b6eee3e83402 ("vhost: fix sync dequeue offload")
> > >
> > >     Signed-off-by: Xuan Ding <xuan.ding@intel.com>
> > >     Reviewed-by: Jiayu Hu <jiayu.hu@intel.com>
> > >     Reviewed-by: Chenbo Xia <chenbo.xia@intel.com>
> > >     Reviewed-by: Maxime Coquelin <maxime.coquelin@redhat.com>
> > >     Tested-by: Xingguang He <xingguang.he@intel.com>
> > >
> > > Best regards,
> > > Yu Jiang