Re: [dpdk-dev] [PATCH v5] compress/qat: enable dynamic huffman encoding

2018-10-24 Thread Thomas Monjalon
25/10/2018 02:40, Trahe, Fiona: > I'll send a v6. > On which OS/compiler did this fail to compile? It fails on i686 of course. I already sent some general notes in the past, explaining how %lx is most of the time wrong. %PRIu64 is the right specifier. https://www.mail-archive.com/dev@dpdk.org/ms

Re: [dpdk-dev] [PATCH v5] compress/qat: enable dynamic huffman encoding

2018-10-24 Thread Trahe, Fiona
tal, ArkadiuszX > Subject: Re: [dpdk-dev] [PATCH v5] compress/qat: enable dynamic huffman > encoding > > 24/10/2018 16:15, Akhil Goyal: > > > > On 10/18/2018 2:18 AM, Fiona Trahe wrote: > > > Enable dynamic huffman encoding in the QAT comp PMD. > > >

Re: [dpdk-dev] [PATCH v5] compress/qat: enable dynamic huffman encoding

2018-10-24 Thread Thomas Monjalon
24/10/2018 16:15, Akhil Goyal: > > On 10/18/2018 2:18 AM, Fiona Trahe wrote: > > Enable dynamic huffman encoding in the QAT comp PMD. > > > > Signed-off-by: Tomasz Jozwiak > > Signed-off-by: Fiona Trahe > > Acked-by: Arek Kusztal > > --- > Applied to dpdk-next-crypto error: format ‘%lx’ expects a

Re: [dpdk-dev] [PATCH v5] compress/qat: enable dynamic huffman encoding

2018-10-24 Thread Akhil Goyal
On 10/18/2018 2:18 AM, Fiona Trahe wrote: > Enable dynamic huffman encoding in the QAT comp PMD. > > Signed-off-by: Tomasz Jozwiak > Signed-off-by: Fiona Trahe > Acked-by: Arek Kusztal > --- Applied to dpdk-next-crypto Thanks

[dpdk-dev] [PATCH v5] compress/qat: enable dynamic huffman encoding

2018-10-17 Thread Fiona Trahe
Enable dynamic huffman encoding in the QAT comp PMD. Signed-off-by: Tomasz Jozwiak Signed-off-by: Fiona Trahe Acked-by: Arek Kusztal --- v5 changes: - update release note v4 changes: - use #define for IM buffer logs, disable all by default and use param for clearer hexdump length. v3 cha