Hi Sameer, 

Thanks for your patience and I apologize for not reaching out yesterday. 

Normally, if you want a retest you send an email on your patch requesting one per the instructions here and it automatically gets picked up by the Community Lab CI testing system: https://core.dpdk.org/testing/#requesting-a-patch-retest

In this case I will just trigger the retest directly since I'm the Community Lab manager. The failure in question should be gone in approx 30 minutes. 

Have a good weekend.

On Fri, Sep 26, 2025 at 12:15 PM Sameer Vaze <svaze@qti.qualcomm.com> wrote:

Hey Patrick,

Were you able to fix the issue with the traffic generator? Is there documentation on what steps need to be done to request a rerun?

Thanks
Sameer Vaze

---

From: Patrick Robb <probb@iol.unh.edu>
Sent: Wednesday, September 24, 2025 2:24 PM
To: Sameer Vaze <svaze@qti.qualcomm.com>
Cc: Fan Zhang <fanzhang.oss@gmail.com>; Ashish Gupta <ashishg@marvell.com>; dev@dpdk.org <dev@dpdk.org>
Subject: Re: [PATCH 1/3] compressdev: support for dictionaries and PDCP checksum

 

WARNING: This email originated from outside of Qualcomm. Please be wary of any links or attachments, and do not enable macros.

The CI testing failure for this series is due to an issue with the traffic generator that is unrelated to the patch. I will resolve it tomorrow and do a test rerun.

On Wed, Sep 24, 2025 at 12:19 PM Sameer Vaze <svaze@qti.qualcomm.com> wrote:

Adds definitions for PDCP checksums and apis to pass in
dictionaries

Signed-off-by: Sameer Vaze <svaze@qti.qualcomm.com>
---
 lib/compressdev/rte_comp.h | 31 +++++++++++++++++++++++++++++++
 1 file changed, 31 insertions(+)

diff --git a/lib/compressdev/rte_comp.h b/lib/compressdev/rte_comp.h
index 96d9b276dd..169d3d960e 100644
--- a/lib/compressdev/rte_comp.h
+++ b/lib/compressdev/rte_comp.h
@@ -101,6 +101,10 @@ enum rte_comp_op_status {
         * is not an error case. Output data up to op.produced can be used and
         * next op in the stream should continue on from op.consumed+1.
         */
+       RTE_COMP_OP_STATUS_CHECK_SUM_VALIDATION_FAILED,
+       /**< Checksum validation failed. Either calculated does checksum not match
+        * the one provided or there was an error calculating the checksum
+        */
 };

 /** Compression Algorithms */
@@ -166,6 +170,10 @@ enum rte_comp_checksum_type {
        /**< Generates a xxHash-32 checksum, as used by LZ4.
         * https://github.com/Cyan4973/xxHash/blob/dev/doc/xxhash_spec.md
         */
+       RTE_COMP_CHECKSUM_3GPP_PDCP_UDC,
+       /**< Generates checksum as defined under Uplink Data Compression
+        * checksum as defined in the 3GPP PDCP specification
+        */
 };

 /** Compression Huffman Type - used by DEFLATE algorithm */
@@ -201,6 +209,11 @@ enum rte_comp_flush_flag {
         */
 };

+#define        DEFLATE_MAX_WINDOW_SIZE (1ULL << 15)
+
+#define        DEFLATE_MIN_WINDOW_SIZE         (1ULL << 8)
+
+
 /** Compression transform types */
 enum rte_comp_xform_type {
        RTE_COMP_COMPRESS,
@@ -305,6 +318,15 @@ struct rte_comp_compress_xform {
        /**< Hash algorithm to be used with compress operation. Hash is always
         * done on plaintext.
         */
+       uint8_t *dictionary;
+       /**<
+        * Pointer to memory containing dictionary to be used for inflate
+        * and deflate operations
+        */
+       uint16_t dictionary_len;
+       /**<
+        * Length of dictionary to be used
+        */
 };

 /**
@@ -328,6 +350,15 @@ struct rte_comp_decompress_xform {
        /**< Hash algorithm to be used with decompress operation. Hash is always
         * done on plaintext.
         */
+       uint8_t *dictionary;
+       /**<
+        * Pointer to memory containing dictionary to be used for inflate
+        * and deflate operations
+        */
+       uint16_t dictionary_len;
+       /**<
+        * Length of dictionary to be used
+        */
 };

 /**
--
2.31.1