Re: [PATCH] net: move ethtool include to where its used

[David Miller <davem@davemloft.net>]

From: Mike Frysinger <vapier@gentoo.org>
Date: Fri,  3 Feb 2012 06:17:02 -0500

> The linux/mii.h headers has long been standalone for userspace.  But a
> recent commit added linux/ethtool.h to the include list even when it
> isn't used.  Since we only need this with __KERNEL__, move the include
> down to the right spot.  Now userland no longer needs to pull this in.
> 
> Signed-off-by: Mike Frysinger <vapier@gentoo.org>

I don't understand what the problem is.

linux/ethtool.h is a safely exported header file for userspace,
so if linux/mii.h now starts to include it this should make no
difference whatsoever for userspace unless userspace does something
bogus which in turn would mean it's almost certainly userspace
which is at fault here no the kernel headers.

You don't describe the exact nature of the failure nor have you shown
a specific actual failure, so we can only guess what the real problem
is.  And that's another thing that makes this patch submission bogus.

This change seems arbitrary and not necessary, therefore I am not
applying it.