Re: [PATCH] driver core: avoid repeatedly printing the same 'Fixed dependency' log

[sashiko-bot@kernel.org]

Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
- [Medium] Moving `pr_info()` into `__fwnode_link_cycle()` causes log spam and highly misleading dependency cycle logs during initial cycle detection.
--

commit b7864d7a5daadca2b750e1e3f78ba9cec6f247d5
Author: Xu Yang <xu.yang_2@nxp.com>

driver core: avoid repeatedly printing the same 'Fixed dependency' log

This commit attempts to deduplicate "Fixed dependency cycle" logs by moving
the print statement into __fwnode_link_cycle() and gating it behind the
FWLINK_FLAG_CYCLE flag check.

> diff --git a/drivers/base/core.c b/drivers/base/core.c
> --- a/drivers/base/core.c
> +++ b/drivers/base/core.c
> @@ -127,7 +127,13 @@ static void __fwnode_link_cycle(struct fwnode_link *link)
>  {
>  	pr_debug("%pfwf: cycle: depends on %pfwf\n",
>  		 link->consumer, link->supplier);
> +
> +	if (link->flags & FWLINK_FLAG_CYCLE)
> +		return;
> +
>  	link->flags |= FWLINK_FLAG_CYCLE;
> +	pr_info("%pfwf: Fixed dependency cycle(s) with %pfwf\n",
> +		link->consumer, link->supplier);
>  }

When moving this pr_info() into __fwnode_link_cycle(), does this cause the
log to be printed for every intermediate edge in the cycle?

__fw_devlink_relax_cycles() unwinds the recursion and calls
__fwnode_link_cycle() for each link involved in the cycle:

__fw_devlink_relax_cycles() {
	...
	list_for_each_entry(link, &sup_handle->suppliers, c_hook) {
		if (link->flags & FWLINK_FLAG_IGNORE)
			continue;

		if (__fw_devlink_relax_cycles(con_handle, link->supplier)) {
			__fwnode_link_cycle(link);
			ret = true;
		}
	}
	...
}

Because of this recursive unwinding, won't this change print the
"Fixed dependency cycle(s)" message for every node involved in a newly
detected cycle, rather than just the specific link that closed it?

Could this lead to more confusing log spam by flagging older, valid
dependencies as the source of the cycle?

[ ... ]

> @@ -2140,8 +2146,6 @@ static int fw_devlink_create_devlink(struct device *con,
>  	if (__fw_devlink_relax_cycles(link->consumer, sup_handle)) {
>  		__fwnode_link_cycle(link);
>  		pr_debug("----- cycle: end -----\n");
> -		pr_info("%pfwf: Fixed dependency cycle(s) with %pfwf\n",
> -			link->consumer, sup_handle);
>  	}

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20260509111728.4123331-1-xu.yang_2@nxp.com?part=1