On Tue, Jul 16, 2024 at 02:34:07AM +0000, Liu, Yuan1 wrote: > > -----Original Message----- > > From: Yichen Wang <[email protected]> > > Sent: Tuesday, July 16, 2024 6:13 AM > > To: Peter Xu <[email protected]>; Fabiano Rosas <[email protected]>; Paolo > > Bonzini <[email protected]>; Daniel P. Berrangé <[email protected]>; > > Eduardo Habkost <[email protected]>; Marc-André Lureau > > <[email protected]>; Thomas Huth <[email protected]>; Philippe > > Mathieu-Daudé <[email protected]>; Eric Blake <[email protected]>; Markus > > Armbruster <[email protected]>; Laurent Vivier <[email protected]>; qemu- > > [email protected] > > Cc: Hao Xiang <[email protected]>; Liu, Yuan1 <[email protected]>; > > Zou, Nanhai <[email protected]>; Ho-Ren (Jack) Chuang > > <[email protected]>; Wang, Yichen <[email protected]> > > Subject: [PATCH v6 1/5] docs/migration: add qatzip compression feature > > > > From: Yuan Liu <[email protected]> > > > > add Intel QATzip compression method introduction > > > > Signed-off-by: Yuan Liu <[email protected]> > > Reviewed-by: Nanhai Zou <[email protected]> > > Reviewed-by: Peter Xu <[email protected]> > > Reviewed-by: Yichen Wang <[email protected]> > > --- > > docs/devel/migration/features.rst | 1 + > > docs/devel/migration/qatzip-compression.rst | 251 ++++++++++++++++++++ > > 2 files changed, 252 insertions(+) > > create mode 100644 docs/devel/migration/qatzip-compression.rst > > > > diff --git a/docs/devel/migration/features.rst > > b/docs/devel/migration/features.rst > > index 58f8fd9e16..8f431d52f9 100644 > > --- a/docs/devel/migration/features.rst > > +++ b/docs/devel/migration/features.rst > > @@ -14,3 +14,4 @@ Migration has plenty of features to support different > > use cases. > > CPR > > qpl-compression > > uadk-compression > > + qatzip-compression > > diff --git a/docs/devel/migration/qatzip-compression.rst > > b/docs/devel/migration/qatzip-compression.rst > > new file mode 100644 > > index 0000000000..72fa3e2826 > > --- /dev/null > > +++ b/docs/devel/migration/qatzip-compression.rst > > @@ -0,0 +1,251 @@ > > +================== > > +QATzip Compression > > +================== > > +In scenarios with limited network bandwidth, the ``QATzip`` solution can > > help > > +users save a lot of host CPU resources by accelerating compression and > > +decompression through the Intel QuickAssist Technology(``QAT``) hardware. > > Hi Yichen > > Thanks for adding the part of Performance Testing with QATzip, I wonder if we > can remove Performance Testing with QATzip part and directly add the following > content. > > Here, we use a typical example of limited bandwidth to illustrate the > advantages > of QATzip. If the user is interested in qatzip, he still needs to verify the > performance > by himself. > > +The following test was conducted using 8 multifd channels and 10Gbps network > +bandwidth. The results show that, compared to zstd, ``QATzip`` significantly > +saves CPU resources on the sender and reduces migration time. Compared to the > +uncompressed solution, ``QATzip`` greatly improves the dirty page processing > +capability, indicated by the Pages per Second metric, and also reduces the > +total migration time. > + > +:: > + > + VM Configuration: 16 vCPU and 64G memory > + VM Workload: all vCPUs are idle and 54G memory is filled with Silesia > data. > + QAT Devices: 4 > + |-----------|--------|---------|----------|----------|------|------| > + |8 Channels |Total |down |throughput|pages per | send | recv | > + | |time(ms)|time(ms) |(mbps) |second | cpu %| cpu% | > + |-----------|--------|---------|----------|----------|------|------| > + |qatzip | 16630| 28| 10467| 2940235| 160| 360| > + |-----------|--------|---------|----------|----------|------|------| > + |zstd | 20165| 24| 8579| 2391465| 810| 340| > + |-----------|--------|---------|----------|----------|------|------| > + |none | 46063| 40| 10848| 330240| 45| 85| > + |-----------|--------|---------|----------|----------|------|------|
Yes this looks much simpler and better. The 10GBps test isn't that useful at least, especially with nocomp numbers absent. I didn't say when looking previously, but it'll be better to clarify the numbers. Yuan, thanks so much for reviewing all the relevant patches. It's very helpful to us. -- Peter Xu
