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
.