Re: [PATCH net v2 06/14] net: pse-pd: check the PI state under the lock when setting a power limit

[Carlo Szelinsky <github@szelinsky.de>]

On Fri, 23 Oct 2026 08:13:34 +0000, netdev-bot+sashiko@kernel.org wrote:
> [Severity: High]
> Do admin_state_enabled and isr_pd_detected really mean the PI is powered?

Kory, a question about pending PIs. A PI can be admin enabled with a
PD detected but without power: when there was not enough budget when
it was enabled or when the PD was detected, or when a PI with higher
priority took its budget. Such a PI passes the check in
pse_pi_update_pw_budget(), so setting a power limit reserves budget
for it if the limit fits. When the PI is powered later,
pse_pi_allocate_pw_budget_static_prio() sets pw_allocated_mW again,
and the budget reserved before is never freed.

This was the same before my patch. Should set_pw_limit skip a pending
PI, for example with pse_pi_is_admin_enable_pending()? If yes, I would
send that as a separate patch.

Carlo