Received-SPF: pass (google.com: domain of linux-wireless-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Date:   Fri, 20 Nov 2020 14:48:24 -0800
From:   Jakub Kicinski <kuba@kernel.org>
To:     Johannes Berg <johannes@sipsolutions.net>
Cc:     davem@davemloft.net, netdev@vger.kernel.org, andrew@lunn.ch,
        mkubecek@suse.cz, linux-rdma@vger.kernel.org,
        linux-wireless@vger.kernel.org
Subject: Re: [PATCH net-next] net: don't include ethtool.h from netdevice.h
Message-ID: <20201120144824.2fc3eda0@kicinski-fedora-pc1c0hjn.dhcp.thefacebook.com>
In-Reply-To: <9b722328efd90104d0c0819f530c296213c19450.camel@sipsolutions.net>
References: <20201120221328.1422925-1-kuba@kernel.org>
        <9b722328efd90104d0c0819f530c296213c19450.camel@sipsolutions.net>
MIME-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit
Precedence: bulk

On Fri, 20 Nov 2020 23:28:28 +0100 Johannes Berg wrote:
> On Fri, 2020-11-20 at 14:13 -0800, Jakub Kicinski wrote:
> > linux/netdevice.h is included in very many places, touching any
> > of its dependecies causes large incremental builds.
> > 
> > Drop the linux/ethtool.h include, linux/netdevice.h just needs
> > a forward declaration of struct ethtool_ops.
> > 
> > Fix all the places which made use of this implicit include.  
> 
> >  include/net/cfg80211.h                                   | 1 +  
> 
> Sounds good to me, thanks. Will still cause all wireless drivers to
> rebuild this way though. Maybe I'll see later if something can be done
> about that.
> 
> Acked-by: Johannes Berg <johannes@sipsolutions.net>

Same for RDMA, sadly. Both need the firmware version string length.

One simple way to narrow things down would be to include only the uapi
version. Or we can duplicate the define, AFAIR duplicate defines are ok
as long as they are identical.

But not sure it's worth it, I'm happy enough with the 2x reduction in
objects getting rebuilt :)