Re: [PATCH -next] init/Kconfig: fix unmet direct dependencies

From: Ren Zhijie
Date: Thu Sep 29 2022 - 02:48:55 EST



在 2022/9/28 17:32, Arnd Bergmann 写道:
On Wed, Sep 28, 2022, at 11:14 AM, Sebastian Andrzej Siewior wrote:
On 2022-09-28 09:20:42 [+0200], Lukas Bulwahn wrote:
Couldn't this become a depends?
It could also be a depends (to resolve the warning).

It is just the question whether:

When PROC_FS is not set, should the CHECKPOINT_RESTORE still be
visible as a config option to add (and then automatically add
PROC_FS)? Then select is right here.
then CHECKPOINT_RESTORE is the only option selecting PROC_FS while
everyone else depends on it _or_ avoids using it in the absence of
PROC_FS.
Right, we should not mix 'select' and 'depends on' for the same
symbol, as that leads to circular dependencies and general
confusion.

If there is no way to use CHECKPOINT_RESTORE without procfs,
then the symbol should just not be visible (it will still show
up with the dependency when one searches in menuconfig).
Force-enabling a major subsystem like procfs from another
symbol is not a good solution.

Agree that,  i will fix it in patch v2.

Thanks,

Ren

Arnd
.