netdev - Re: [PATCH] pds_core: fix linking without CONFIG_DEBUG

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Mon, 01 May 2023 21:34:54 +0200
From:   "Arnd Bergmann" <arnd@...db.de>
To:     "Simon Horman" <simon.horman@...igine.com>,
        "Arnd Bergmann" <arnd@...nel.org>
Cc:     "Shannon Nelson" <shannon.nelson@....com>,
        "Brett Creeley" <brett.creeley@....com>,
        "David S . Miller" <davem@...emloft.net>,
        "Eric Dumazet" <edumazet@...gle.com>,
        "Jakub Kicinski" <kuba@...nel.org>,
        "Paolo Abeni" <pabeni@...hat.com>, Netdev <netdev@...r.kernel.org>,
        linux-kernel@...r.kernel.org,
        "Conor Dooley" <conor.dooley@...rochip.com>,
        "Philipp Zabel" <p.zabel@...gutronix.de>
Subject: Re: [PATCH] pds_core: fix linking without CONFIG_DEBUG_FS

On Mon, May 1, 2023, at 17:41, Simon Horman wrote:
> On Mon, May 01, 2023 at 05:06:14PM +0200, Arnd Bergmann wrote:
>
> While exercising this I noticed that building pds_core fails
> if either CONFIG_AUXILIARY_BUS or NET_DEVLINK are not enabled.
>
> I think the solution is for PFS_CORE to select both CONFIG_AUXILIARY_BUS
> and NET_DEVLINK.

Makes sense. I just double-checked the other uses of these symbols
to see if they should be 'select' or 'depends on', and you are
right that selecting them is the correct solution.

There are two instances of 'depends on CONFIG_AUXILIARY_BUS'
in drivers/reset that both should be 'select' as well, since
this is not a user-visible symbol.

       Arnd