Re: [PATCH 1/1] Add AVX512 support for xbzrle_encode_buffer function

2022-07-22 Thread Daniel P . Berrangé
On Fri, Jul 22, 2022 at 02:23:13AM +, Zhao, Zhou wrote: > 1 we need gcc machined options to tirgger gcc enable SIMD > instructions(https://gcc.gnu.org/onlinedocs/gcc/x86-Options.html). > Or it will trigger a compile error. This is the reason why we > change the config file. Take a look at util

RE: [PATCH 1/1] Add AVX512 support for xbzrle_encode_buffer function

2022-07-21 Thread Zhao, Zhou
nt to save cpu time , we'd better check it in compile time. -Original Message- From: Daniel P. Berrangé Sent: Thursday, July 21, 2022 11:11 PM To: Xu, Ling1 Cc: qemu-devel@nongnu.org; quint...@redhat.com; dgilb...@redhat.com; Zhao, Zhou ; Jin, Jun I Subject: Re: [PATCH 1/1] Add AVX

Re: [PATCH 1/1] Add AVX512 support for xbzrle_encode_buffer function

2022-07-21 Thread Daniel P . Berrangé
On Thu, Jul 21, 2022 at 04:02:49PM +, Zhao, Zhou wrote: > Hi dainel: > Cause our code depend on intel intrinsics lib implement. And > this lib depend on macro like " AVX512BW ". This macro need > compile time check to enable some machine options . if you only > use that utility to do runtime

Re: [PATCH 1/1] Add AVX512 support for xbzrle_encode_buffer function

2022-07-21 Thread Dr. David Alan Gilbert
t; From: Daniel P. Berrangé > Sent: Thursday, July 21, 2022 11:11 PM > To: Xu, Ling1 > Cc: qemu-devel@nongnu.org; quint...@redhat.com; dgilb...@redhat.com; Zhao, > Zhou ; Jin, Jun I > Subject: Re: [PATCH 1/1] Add AVX512 support for xbzrle_encode_buffer function > > On

Re: [PATCH 1/1] Add AVX512 support for xbzrle_encode_buffer function

2022-07-21 Thread Daniel P . Berrangé
On Thu, Jul 21, 2022 at 06:31:47PM +0800, ling xu wrote: > This commit adds AVX512 implementation of xbzrle_encode_buffer function to > accelerate xbzrle encoding speed. Compared with C version of > xbzrle_encode_buffer function, > AVX512 version can achieve almost 60%-70% performance improvement