netdev - Re: [PATCH net-next V3 3/5] devlink: Introduce error burst period for health reporter

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <20250815122337.4aa50105@kernel.org>
Date: Fri, 15 Aug 2025 12:23:37 -0700
From: Jakub Kicinski <kuba@...nel.org>
To: Tariq Toukan <tariqt@...dia.com>
Cc: Jiri Pirko <jiri@...dia.com>, Jiri Pirko <jiri@...nulli.us>, Eric
 Dumazet <edumazet@...gle.com>, Paolo Abeni <pabeni@...hat.com>, Andrew Lunn
 <andrew+netdev@...n.ch>, "David S. Miller" <davem@...emloft.net>, Donald
 Hunter <donald.hunter@...il.com>, Jonathan Corbet <corbet@....net>, Brett
 Creeley <brett.creeley@....com>, Michael Chan <michael.chan@...adcom.com>,
 Pavan Chebbi <pavan.chebbi@...adcom.com>, "Cai Huoqing"
 <cai.huoqing@...ux.dev>, Tony Nguyen <anthony.l.nguyen@...el.com>, Przemek
 Kitszel <przemyslaw.kitszel@...el.com>, Sunil Goutham
 <sgoutham@...vell.com>, Linu Cherian <lcherian@...vell.com>, Geetha
 sowjanya <gakula@...vell.com>, Jerin Jacob <jerinj@...vell.com>, hariprasad
 <hkelam@...vell.com>, Subbaraya Sundeep <sbhatta@...vell.com>, Saeed
 Mahameed <saeedm@...dia.com>, Leon Romanovsky <leon@...nel.org>, Mark Bloch
 <mbloch@...dia.com>, Ido Schimmel <idosch@...dia.com>, Petr Machata
 <petrm@...dia.com>, Manish Chopra <manishc@...vell.com>,
 <netdev@...r.kernel.org>, <linux-kernel@...r.kernel.org>,
 <linux-doc@...r.kernel.org>, <intel-wired-lan@...ts.osuosl.org>,
 <linux-rdma@...r.kernel.org>, "Gal Pressman" <gal@...dia.com>, Dragos
 Tatulea <dtatulea@...dia.com>, "Shahar Shitrit" <shshitrit@...dia.com>
Subject: Re: [PATCH net-next V3 3/5] devlink: Introduce error burst period
 for health reporter

On Wed, 13 Aug 2025 21:55:47 +0300 Tariq Toukan wrote:
> + * @default_error_burst_period: default time (in msec) for
> + *				error recoveries before
> + *				starting the grace period

FWIW you don't have to indent the continuation lines this much,
one tab is enough to make it clear that it's a continuation.

> +static bool
> +devlink_health_reporter_burst_period_active(struct devlink_health_reporter *reporter)

Not sure what it means for a period to be active.

devlink_health_reporter_in_burst()