<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=us-ascii">
<style type="text/css" style="display:none;"> P {margin-top:0;margin-bottom:0;} </style>
</head>
<body dir="ltr">
<div style="font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 11pt; color: rgb(0, 0, 0);" class="elementToProof">
Hey Akhil,</div>
<div style="font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 11pt; color: rgb(0, 0, 0);" class="elementToProof">
<br>
</div>
<div style="font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 11pt; color: rgb(0, 0, 0);" class="elementToProof">
I pushed a reworked v2 patch based on the comments below: <a class="OWAAutoLink" id="LPlnk401803" href="https://patches.dpdk.org/project/dpdk/list/?series=36288">
https://patches.dpdk.org/project/dpdk/list/?series=36288</a></div>
<div style="font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 11pt; color: rgb(0, 0, 0);" class="elementToProof">
<br>
Even though I included --in-reply-to to the older series it seems to have created a new series. Is that expected or is the command I used incorrect? The command I used was:</div>
<div id="Signature">
<div style="font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 11pt; color: rgb(0, 0, 0);">
<br>
git send-email --to-cmd ./devtools/get-maintainer.sh --cc dev@dpdk.org --thread --no-chain-reply-to --in-reply-to=20250924161918.2211290-1-svaze@qti.qualcomm.com *.patch</div>
<div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 11pt; color: rgb(0, 0, 0);">
<br>
</div>
<div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 11pt; color: rgb(0, 0, 0);">
Thanks<br>
Sameer Vaze</div>
</div>
<div id="appendonsend"></div>
<hr style="display:inline-block;width:98%" tabindex="-1">
<div id="divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" style="font-size:11pt" color="#000000"><b>From:</b> Akhil Goyal <gakhil@marvell.com><br>
<b>Sent:</b> Friday, October 3, 2025 1:27 AM<br>
<b>To:</b> Sameer Vaze <svaze@qti.qualcomm.com>; Fan Zhang <fanzhang.oss@gmail.com>; Ashish Gupta <ashishg@marvell.com><br>
<b>Cc:</b> dev@dpdk.org <dev@dpdk.org><br>
<b>Subject:</b> RE: [EXTERNAL] [PATCH 1/3] compressdev: support for dictionaries and PDCP checksum</font>
<div> </div>
</div>
<div class="BodyFragment"><font size="2"><span style="font-size:11pt;">
<div class="PlainText">WARNING: This email originated from outside of Qualcomm. Please be wary of any links or attachments, and do not enable macros.<br>
<br>
> Sorry for the various series I pushed. I attempted in reply to but it seemed to<br>
> create a new series. If I am making changes to one commit in a series, do I need<br>
> to resend all commits with an updated version? In such a case do I reply to the<br>
> series or the specific change I am updating?<br>
Yes reply to the series. And all patches need to be resent.<br>
<br>
><br>
> Regarding the comments on adding UDC as a new algo: UDC uses raw deflate and<br>
> is not a separate algo in itself. Support for pre defined dictionaries is already<br>
> supported by zlib and is not UDC specific per se.<br>
><br>
In that case also, these new params in xform should be part of alg specific params - rte_comp_deflate_params.<br>
<br>
Please note: we need to close the RC1 by 5th Oct. Please send the rework asap.<br>
<br>
Thanks.<br>
</div>
</span></font></div>
</body>
</html>