<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
<style type="text/css" style="display:none;"> P {margin-top:0;margin-bottom:0;} </style>
</head>
<body dir="ltr">
<div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);" class="elementToProof">
As per the DPDK contribution guidelines, </div>
<div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);" class="elementToProof">
<span style="font-family: Calibri, Helvetica, sans-serif; text-align: left; display: inline !important; color: rgb(64, 64, 64); background-color: rgb(252, 252, 252);" class="ContentPasted0">Ideally, the review should be done within 1 week of patch submission
to the mailing list.</span><br>
</div>
<div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);" class="elementToProof">
<span style="font-family: Lato, proxima-nova, "Helvetica Neue", Arial, sans-serif; text-align: left; display: inline !important; color: rgb(64, 64, 64); background-color: rgb(252, 252, 252);" class="ContentPasted0 ContentPasted1"><a href="https://doc.dpdk.org/guides/contributing/patches.html" id="OWA06b300eb-43c8-2071-27d6-a599fb463a42" class="OWAAutoLink" data-loopstyle="linkonly"><span style="font-family: Calibri, Helvetica, sans-serif;">https://doc.dpdk.org/guides/contributing/patches.html</span></a><br>
</span></div>
<div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);" class="elementToProof">
<span style="font-family: Lato, proxima-nova, "Helvetica Neue", Arial, sans-serif; text-align: left; display: inline !important; color: rgb(64, 64, 64); background-color: rgb(252, 252, 252);" class="ContentPasted0"><br>
</span></div>
<div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);" class="elementToProof">
<span style="font-family: Calibri, Helvetica, sans-serif; text-align: left; display: inline !important; color: rgb(64, 64, 64); background-color: rgb(252, 252, 252);" class="ContentPasted0">Surprised to see this review request ping message the very next day
of the patch.</span></div>
<div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);" class="elementToProof">
<span style="font-family: Calibri, Helvetica, sans-serif; text-align: left; display: inline !important; color: rgb(64, 64, 64); background-color: rgb(252, 252, 252);" class="ContentPasted0">Better to follow the contribution guidelines.</span></div>
<div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);" class="elementToProof">
<span style="font-family: Lato, proxima-nova, "Helvetica Neue", Arial, sans-serif; text-align: left; display: inline !important; color: rgb(64, 64, 64); background-color: rgb(252, 252, 252);" class="ContentPasted0"><br>
</span></div>
<div id="appendonsend"></div>
<div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);" class="elementToProof">
-Harish</div>
<hr tabindex="-1" style="display:inline-block; width:98%">
<div id="divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" style="font-size: 11pt; color: rgb(0, 0, 0);"><b>From:</b> Jerin Jacob <jerinjacobk@gmail.com><br>
<b>Sent:</b> Tuesday, September 19, 2023 9:59 PM<br>
<b>To:</b> Rahul Bhansali <rbhansali@marvell.com><br>
<b>Cc:</b> dev@dpdk.org <dev@dpdk.org>; Naga Harish K, S V <s.v.naga.harish.k@intel.com>; Jerin Jacob <jerinj@marvell.com>; Kundapura, Ganapati <Ganapati.Kundapura@intel.com>; stable@dpdk.org <stable@dpdk.org><br>
<b>Subject:</b> Re: [PATCH] eventdev/eth_rx: fix timestamp field register in mbuf</font>
<div> </div>
</div>
<div class="BodyFragment"><font size="2"><span style="font-size:11pt">
<div class="PlainText">On Mon, Sep 18, 2023 at 6:46 PM Rahul Bhansali <rbhansali@marvell.com> wrote:<br>
><br>
> For eventdev internal port, timestamp dynamic field registration<br>
> in mbuf is not required as that will be done from net device.<br>
> For SW eventdev, Rx timestamp field registration will be<br>
> done during Rx queue add operation as per device capabilities and<br>
> offload configuration.<br>
><br>
> Fixes: 83ab470d1259 ("eventdev/eth_rx: use timestamp as dynamic mbuf field")<br>
> Cc: stable@dpdk.org<br>
><br>
> Signed-off-by: Rahul Bhansali <rbhansali@marvell.com><br>
<br>
Ping for review from maintainer.<br>
</div>
</span></font></div>
</body>
</html>