<meta http-equiv="Content-Type" content="text/html; charset=GB18030"><span style="font-family: -apple-system, "system-ui", "Segoe UI", Helvetica, Arial, sans-serif, "Apple Color Emoji", "Segoe UI Emoji"; font-size: 16px;">Yes, you are correct, it should be the commit 2a7d0b872f79 that introduced the issue. Thank you for the correction.</span><div> </div><div><sign signid="1"><hr align="left" style="margin: 0 0 10px 0;border: 0;border-bottom:1px solid #E4E5E6;height:0;line-height:0;font-size:0;padding: 20px 0 0 0;width: 50px;"/><div style='font-size:14px;font-family:Verdana;color:#000;'><a class="xm_write_card" id="in_alias" style="white-space: normal; display: inline-block; text-decoration: none !important;font-family: -apple-system,BlinkMacSystemFont,PingFang SC,Microsoft YaHei;" href="https://wx.mail.qq.com/home/index?t=readmail_businesscard_midpage&nocheck=true&name=+&icon=http%3A%2F%2Fthirdqq.qlogo.cn%2Fg%3Fb%3Dsdk%26k%3Dl6L5mOH7tXsb6y2YpXib93w%26s%3D100%26t%3D1556762297%3Frand%3D1647395337&mail=1819846787%40qq.com&code=" target="_blank"><table style="white-space: normal;table-layout: fixed; padding-right: 20px;" contenteditable="false" cellpadding="0" cellspacing="0"><tbody><tr valign="top"><td style="width: 40px;min-width: 40px; padding-top:10px"><div style="width: 38px; height: 38px; border: 1px #FFF solid; border-radius:50%; margin: 0;vertical-align: top;box-shadow: 0 0 10px 0 rgba(127,152,178,0.14);"><img src="http://thirdqq.qlogo.cn/g?b=sdk&k=l6L5mOH7tXsb6y2YpXib93w&s=100&t=1556762297?rand=1647395337" style="width:100%;height:100%;border-radius:50%;pointer-events: none;"  /></div></td><td style="padding: 10px 0 8px 10px;"><div class="businessCard_name" style="font-size: 14px;color: #33312E;line-height: 20px; padding-bottom: 2px; margin:0;font-weight: 500;"> </div><div class="businessCard_mail" style="font-size: 12px;color: #999896;line-height: 18px; margin:0;">1819846787@qq.com</div></td></tr></tbody></table></a></div></sign></div><div> </div><div><br></div><div><br></div><div style="font-size: 12px;font-family: Arial Narrow;padding:2px 0 2px 0;">------------------ 原始邮件 ------------------</div><div style="font-size: 12px;background:#efefef;padding:8px;"><div><b>发件人:</b> "David Marchand"<david.marchand@redhat.com>; </div><div><b>发送时间:</b> 2024年2月1日(星期四) 晚上9:14</div><div><b>收件人:</b> " ShaoWei Sun"<1819846787@qq.com>; "ciara.power"<ciara.power@intel.com>; </div><div><b >抄送:</b> "dev"<dev@dpdk.org>; "Bruce Richardson"<bruce.richardson@intel.com>; "Chengwen Feng"<fengchengwen@huawei.com>; </div><div><b>主题:</b> Re: [PATCH] [v3]lib/telemetry:fix telemetry conns leak in case of socket write fail</div></div><div><br></div><style>/* 邮件内部图片支持调起预览 */img[image-inside-content='1'] {cursor: pointer;}</style>On Tue, Jan 30, 2024 at 2:57 AM Shaowei Sun <<a href="mailto:1819846787@qq.com" rel="noopener" target="_blank">1819846787@qq.com</a>> wrote:<br>><br>> Telemetry can only create 10 conns by default, each of which is processed<br>> by a thread.<br>><br>> When a thread fails to write using socket, the thread will end directly<br>> without reducing the total number of conns.<br>><br>> This will result in the machine running for a long time, and if there are<br>> 10 failures, the telemetry will be unavailable<br>><br>> Fixes: 6dd571fd07c3 ("telemetry: introduce new functionality")<br>><br>> Signed-off-by: Shaowei Sun <<a href="mailto:1819846787@qq.com" rel="noopener" target="_blank">1819846787@qq.com</a>><br>> Acked-by: Bruce Richardson <<a href="mailto:bruce.richardson@intel.com" rel="noopener" target="_blank">bruce.richardson@intel.com</a>><br>> Acked-by: Ciara Power <<a href="mailto:ciara.power@intel.com" rel="noopener" target="_blank">ciara.power@intel.com</a>><br>> Acked-by: Chengwen Feng <<a href="mailto:fengchengwen@huawei.com" rel="noopener" target="_blank">fengchengwen@huawei.com</a>><br><br>Thanks for the fix.<br>As far as I can see, the limiting of the number of connections (which<br>here results in a DoS on the telemetry socket) was added in commit<br>2a7d0b872f79 ("telemetry: add upper limit on connections").<br><br>If you confirm this is indeed this commit that introduced the issue, I<br>will fix the Fixes: tag myself when applying.<br><br><br>-- <br>David Marchand<br><style type="text/css">.qmbox style, .qmbox script, .qmbox head, .qmbox link, .qmbox meta {display: none !important;}</style>