netdev - Re: [Intel-wired-lan] [PATCH iwl-next 1/2] ice: add link_down

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <355fc4f1-0116-4028-a455-ec76772f19b3@molgen.mpg.de>
Date: Mon, 12 May 2025 11:27:09 +0200
From: Paul Menzel <pmenzel@...gen.mpg.de>
To: Martyna Szapar-Mudlaw <martyna.szapar-mudlaw@...ux.intel.com>
Cc: intel-wired-lan@...ts.osuosl.org, netdev@...r.kernel.org
Subject: Re: [Intel-wired-lan] [PATCH iwl-next 1/2] ice: add link_down_events
 statistic

Dear Martyna,


Thank you for your reply.

Am 14.04.25 um 15:12 schrieb Szapar-Mudlaw, Martyna:

> On 4/9/2025 2:08 PM, Paul Menzel wrote:

>> Am 09.04.25 um 13:36 schrieb Martyna Szapar-Mudlaw:
>>> Introduce a new ethtool statistic to ice driver, `link_down_events`,
>>> to track the number of times the link transitions from up to down.
>>> This counter can help diagnose issues related to link stability,
>>> such as port flapping or unexpected link drops.
>>>
>>> The counter increments when a link-down event occurs and is exposed
>>> via ethtool stats as `link_down_events.nic`.
>>
>> It’d be great if you pasted an example output.
> 
> In v2 (which I just submitted) the generic ethtool statistic is used for 
> this, instead of driver specific, so I guess no need to paste the 
> example output now.

I think it’s always good also as a reference how to test the patch. I 
just saw your v3. Should you resent, it’d be great if you added the 
example output.

[…]


Kind regards,

Paul