>-----Original Message-----
>From: De Lara Guarch, Pablo <[email protected]>
>Sent: 24 July 2018 03:33
>To: Verma, Shally <[email protected]>
>Cc: [email protected]; Athreya, Narayana Prasad
><[email protected]>; Challa, Mahipal
><[email protected]>; Gupta, Ashish <[email protected]>; Sahu,
>Sunila <[email protected]>
>Subject: RE: [PATCH v4 2/5] compress/zlib: add device PMD ops
>
>External Email
>
>> -----Original Message-----
>> From: Shally Verma [mailto:[email protected]]
>> Sent: Monday, July 23, 2018 3:51 PM
>> To: De Lara Guarch, Pablo <[email protected]>
>> Cc: [email protected]; [email protected];
>> [email protected]; Ashish Gupta
>> <[email protected]>; Sunila Sahu
>> <[email protected]>
>> Subject: [PATCH v4 2/5] compress/zlib: add device PMD ops
>>
>> From: Ashish Gupta <[email protected]>
>>
>> Implement device configure and queue pair setup PMD ops
>>
>> Signed-off-by: Sunila Sahu <[email protected]>
>> Signed-off-by: Shally Verma <[email protected]>
>> Signed-off-by: Ashish Gupta <[email protected]>
>
>...
>
>> --- /dev/null
>> +++ b/drivers/compress/zlib/zlib_pmd_ops.c
>> @@ -0,0 +1,238 @@
>> +/* SPDX-License-Identifier: BSD-3-Clause
>> + * Copyright(c) 2018 Cavium Networks
>> + */
>> +
>> +#include <string.h>
>> +
>> +#include <rte_common.h>
>> +#include <rte_malloc.h>
>> +
>> +#include "zlib_pmd_private.h"
>> +
>> +static const struct rte_compressdev_capabilities zlib_pmd_capabilities[] = {
>> + { /* Deflate */
>> + .algo = RTE_COMP_ALGO_DEFLATE,
>> + .comp_feature_flags =
>> (RTE_COMP_FF_NONCOMPRESSED_BLOCKS |
>> + RTE_COMP_FF_HUFFMAN_FIXED |
>> + RTE_COMP_FF_HUFFMAN_DYNAMIC |
>> +
>> RTE_COMP_FF_OOP_SGL_IN_SGL_OUT),
>
>As said in the document patch, I think you should add the other two SGL cases
>here.
Would give thought to it however, prefer to hold back on their support until we
evaluate them with proper testcases.
Thanks
Shally