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

Trahe, Fiona fiona.trahe at intel.com
Thu Oct 25 02:40:49 CEST 2018


I'll send a v6.
On which OS/compiler did this fail to compile?

> -----Original Message-----
> From: Thomas Monjalon [mailto:thomas at monjalon.net]
> Sent: Wednesday, October 24, 2018 2:30 PM
> To: Trahe, Fiona <fiona.trahe at intel.com>
> Cc: dev at dpdk.org; Akhil Goyal <akhil.goyal at nxp.com>; Jozwiak, TomaszX <tomaszx.jozwiak at intel.com>;
> Kusztal, ArkadiuszX <arkadiuszx.kusztal at intel.com>
> 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.
> > >
> > > Signed-off-by: Tomasz Jozwiak<tomaszx.jozwiak at intel.com>
> > > Signed-off-by: Fiona Trahe<fiona.trahe at intel.com>
> > > Acked-by: Arek Kusztal<arkadiuszx.kusztal at intel.com>
> > > ---
> > Applied to dpdk-next-crypto
> 
> error: format ‘%lx’ expects argument of type ‘long unsigned int’, but argument 7 has type ‘phys_addr_t’
> {aka ‘long long unsigned int’} [-Werror=format=]
>     "%s(): " fmt "\n", __func__, ## args)
>     ^~~~~~~~
> drivers/compress/qat/qat_comp_pmd.c:194:2: note: in expansion of macro ‘QAT_LOG’
>   QAT_LOG(DEBUG, "Memzone %s: addr = %p, phys = %lx, size required %d,"
> 
> 
> One more comment about the release notes, please try to integrate
> compressdev features between cryptodev and eventdev.
> 



More information about the dev mailing list