<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);">
Hi,</div>
<div class="elementToProof" style="font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 11pt; color: rgb(0, 0, 0);">
Actually looks like we're going with another workaround, so I might not be needing my original patch.</div>
<div class="elementToProof" style="font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 11pt; color: rgb(0, 0, 0);">
Also, I do not understand how my patch (not printing a log) makes a test fail - I couldn't find that test in DPDK to verify. I'm not that familiar though..</div>
<div class="elementToProof" style="font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 11pt; color: rgb(0, 0, 0);">
<br>
</div>
<div class="elementToProof" style="font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 11pt; color: rgb(0, 0, 0);">
Yogev</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> Maayan Kashani <mkashani@nvidia.com><br>
<b>Sent:</b> Wednesday, June 4, 2025 10:02<br>
<b>To:</b> Stephen Hemminger <stephen@networkplumber.org>; Yogev Chaimovich <yogev@cgstowernetworks.com><br>
<b>Cc:</b> Dariusz Sosnowski <dsosnowski@nvidia.com>; Slava Ovsiienko <viacheslavo@nvidia.com>; Bing Zhao <bingz@nvidia.com>; Ori Kam <orika@nvidia.com>; Suanming Mou <suanmingm@nvidia.com>; Matan Azrad <matan@nvidia.com>; dev@dpdk.org <dev@dpdk.org><br>
<b>Subject:</b> RE: [PATCH] net/mlx5: fix transceiver warning when not exist</font>
<div> </div>
</div>
<div class="BodyFragment"><font size="2"><span style="font-size:11pt;">
<div class="PlainText">Hi, Yogev,<br>
Thank you for contributing,<br>
Can you fix the failing CI TEST?<br>
<a href="https://patches.dpdk.org/project/dpdk/patch/20250506133107.4432-1-yogev@cgstowernetworks.com/">https://patches.dpdk.org/project/dpdk/patch/20250506133107.4432-1-yogev@cgstowernetworks.com/</a><br>
<br>
Regards,<br>
Maayan Kashani<br>
<br>
> -----Original Message-----<br>
> From: Stephen Hemminger <stephen@networkplumber.org><br>
> Sent: Tuesday, 20 May 2025 21:49<br>
> To: Yogev Chaimovich <yogev@cgstowernetworks.com><br>
> Cc: Dariusz Sosnowski <dsosnowski@nvidia.com>; Slava Ovsiienko<br>
> <viacheslavo@nvidia.com>; Bing Zhao <bingz@nvidia.com>; Ori Kam<br>
> <orika@nvidia.com>; Suanming Mou <suanmingm@nvidia.com>; Matan<br>
> Azrad <matan@nvidia.com>; dev@dpdk.org<br>
> Subject: Re: [PATCH] net/mlx5: fix transceiver warning when not exist<br>
> <br>
> External email: Use caution opening links or attachments<br>
> <br>
> <br>
> On Mon, 5 May 2025 12:14:51 +0000<br>
> Yogev Chaimovich <yogev@cgstowernetworks.com> wrote:<br>
> <br>
> > From ffe666291bc531b89620cc28a7b45fa209700f35 Mon Sep 17 00:00:00<br>
> 2001<br>
> > From: yogev <yogev@cgstowernetworks.com><br>
> > Date: Sun, 4 May 2025 23:11:46 +0300<br>
> > Subject: [PATCH] net/mlx5: fix transceiver warning when not exist<br>
> ><br>
> > In case no transceiver exist when trying to get the transceiver's info<br>
> > There should not be an error message from the library.<br>
> > The caller may print an appropriate error message using the relevant<br>
> rte_errno if required.<br>
> ><br>
> > Signed-off-by: yogev <yogev@cgstowernetworks.com><br>
> <br>
> Signed off by is a legal requirement and must use your full name.<br>
> As in:<br>
> Signed-off-by: Yogev Chaimovich <yogev@cgstowernetworks.com><br>
</div>
</span></font></div>
</body>
</html>