Re: [dpdk-dev] [PATCH] compress/isal: fix getting information about CPU

2019-03-29 Thread Akhil Goyal
, 2019 3:01 PM >> To: Akhil Goyal ; Cel, TomaszX >> ; dev@dpdk.org >> Cc: sta...@dpdk.org; Trahe, Fiona >> Subject: RE: [dpdk-dev] [PATCH] compress/isal: fix getting information about >> CPU >> >> Hi Tomasz, >> >> This patch makes sense to ch

Re: [dpdk-dev] [PATCH] compress/isal: fix getting information about CPU

2019-03-29 Thread Daly, Lee
c: sta...@dpdk.org; Trahe, Fiona > Subject: RE: [dpdk-dev] [PATCH] compress/isal: fix getting information about > CPU > > Hi Tomasz, > > This patch makes sense to check does the CPU have the capability of > certain instructions before Adding it to the dev_info flags. I t

Re: [dpdk-dev] [PATCH] compress/isal: fix getting information about CPU

2019-03-27 Thread Daly, Lee
Hi Tomasz, This patch makes sense to check does the CPU have the capability of certain instructions before Adding it to the dev_info flags. I think one more addition should be made in isal_compress_pmd.c When checking what compression level should be used , decided by the AVX instructi

Re: [dpdk-dev] [PATCH] compress/isal: fix getting information about CPU

2019-03-27 Thread Akhil Goyal
Hi Fiona/Lee, Could you please review this patch. Thanks, Akhil On 3/7/2019 4:58 PM, Tomasz Cel wrote: > This patch adds query about CPU features > > Fixes: 53a9baa98c36 ("compress/isal: add basic PMD ops") > Cc: sta...@dpdk.org > > Signed-off-by: Tomasz Cel > --- > drivers/compress/isal/isal

[dpdk-dev] [PATCH] compress/isal: fix getting information about CPU

2019-03-07 Thread Tomasz Cel
This patch adds query about CPU features Fixes: 53a9baa98c36 ("compress/isal: add basic PMD ops") Cc: sta...@dpdk.org Signed-off-by: Tomasz Cel --- drivers/compress/isal/isal_compress_pmd_ops.c | 16 1 file changed, 12 insertions(+), 4 deletions(-) diff --git a/drivers/compres