netdev - Re: config NET

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>] [day] [month] [year] [list]

Date:	Wed, 09 Nov 2011 01:28:18 -0800
From:	Jeff Kirsher <jeffrey.t.kirsher@...el.com>
To:	David Miller <davem@...emloft.net>
Cc:	"pebolle@...cali.nl" <pebolle@...cali.nl>,
	netdev <netdev@...r.kernel.org>
Subject: Re: config NET_ETHERNET

On Tue, 2011-11-08 at 21:15 -0800, David Miller wrote:
> From: Paul Bolle <pebolle@...cali.nl>
> Date: Wed, 09 Nov 2011 01:41:15 +0100
> 
> > 0) The Kconfig symbol NET_ETHERNET got dropped in commit f860b0522f
> > ("drivers/net: Kconfig and Makefile cleanup").
> > 
> > 1) However, there are still a number of references to that symbol
> and to
> > the macro CONFIG_NET_ETHERNET:
> >     $ git grep -n "NET_ETHERNET\>"  | grep -v defconfig:
> >     arch/cris/arch-v10/drivers/Kconfig:6:     select NET_ETHERNET
> >     arch/cris/arch-v32/drivers/Kconfig:6:     select NET_ETHERNET
> >     drivers/s390/net/Kconfig:7:       depends on CCW && NETDEVICES
> && (NET_ETHERNET || TR || FDDI)
> >     drivers/s390/net/lcs.c:53:#if !defined(CONFIG_NET_ETHERNET) && \
> >     drivers/s390/net/lcs.c:1637:#ifdef CONFIG_NET_ETHERNET
> >     drivers/s390/net/lcs.c:2169:#ifdef CONFIG_NET_ETHERNET
> > 
> > 2) It's trivial to draft a patch which drops both that Kconfig
> symbol
> > and the macro (and everything that could be considered dead code by
> > now). But perhaps this needs a more sophisticated solution.
> 
> The best thing to do for now is to switch these references over
> to use just plain "ETHERNET" and that's what I'll commit into
> the net tree.
> 
> --------------------
> [PATCH] net: Fix references to deleted NET_ETHERNET Kconfig setting.
> 
> Change them over to plain "ETHERNET"
> 
> Reported-by: Paul Bolle <pebolle@...cali.nl>
> Signed-off-by: David S. Miller <davem@...emloft.net>
> ---
>  arch/cris/arch-v10/drivers/Kconfig |    2 +-
>  arch/cris/arch-v32/drivers/Kconfig |    2 +-
>  drivers/s390/net/Kconfig           |    2 +-
>  drivers/s390/net/lcs.c             |    6 +++---
>  4 files changed, 6 insertions(+), 6 deletions(-) 

Acked-by: Jeff Kirsher <jeffrey.t.kirsher@...el.com>

Download attachment "signature.asc" of type "application/pgp-signature" (837 bytes)