On Wed, Sep 28, 2022, at 11:14 AM, Sebastian Andrzej Siewior wrote:
On 2022-09-28 09:20:42 [+0200], Lukas Bulwahn wrote:Right, we should not mix 'select' and 'depends on' for the same
…Couldn't this become a depends?It could also be a depends (to resolve the warning).
It is just the question whether:then CHECKPOINT_RESTORE is the only option selecting PROC_FS while
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.
everyone else depends on it _or_ avoids using it in the absence of
PROC_FS.
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.
Arnd
.