netdev - Re: [patch 1/6] 8139too: force media setting cleanup

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

Message-ID: <46560567.5060708@garzik.org>
Date:	Thu, 24 May 2007 17:36:39 -0400
From:	Jeff Garzik <jeff@...zik.org>
To:	akpm@...ux-foundation.org
CC:	leewkb@...oo.com, netdev <netdev@...r.kernel.org>,
	Linux Kernel Mailing List <linux-kernel@...r.kernel.org>
Subject: Re: [patch 1/6] 8139too: force media setting cleanup

akpm@...ux-foundation.org wrote:
> From: Bernard Lee <leewkb@...oo.com>
> 
> Setting bit 4 & 5 alone in 8139too module media option does not really
> force 100Mbps full-duplex mode.  When media option bit 0-3 is cleared,
> 8139too module does not force media setting.  Therefore, bit 0-3 requires
> to be set for bit 4 & 5 to take effect.  The hidden bit 0-3 setting is not
> stated in module description.
> 
> It can be fixed by changing rtl8139_private structure default_port bitfield
> from 4-bit to 6-bit.
> 
> Besides, module media bit 9 is a duplicate of bit 4 (full-duplex).  It is
> suggested that bit 9 is freed.  A remark is added to module description
> that bit 0 can be used to force setting.  It helps to clarify 10Mbps
> half-duplex mode.
> 
> Signed-off-by: Bernard Lee <leewkb@...oo.com>
> Cc: Jeff Garzik <jeff@...zik.org>
> Signed-off-by: Andrew Morton <akpm@...ux-foundation.org>
> ---
> 
>  drivers/net/8139too.c |    8 ++++----
>  1 files changed, 4 insertions(+), 4 deletions(-)

(re-replying, just to refresh everybody on the status)

The status of this patch is HOLD, and has been for months.

AFAICS this is a behavior change that differs from other Becker-derived 
drivers.  I would suggest looking at other drivers.

Furthermore, people should be using ethtool to set media.  Rather than 
changing the behavior of this module parameter to be non-standard, I 
feel it is better to (a) leave it as is or (b) remove it completely.

	Jeff




-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html