Bug#561843: ethtool/mii-tool report incorrect/missing values on Marvell 88E1011

2010-08-01 Thread Moritz Muehlenhoff
reassign 561843 ethtool
thanks

On Sun, Dec 20, 2009 at 06:55:27PM +, Ben Hutchings wrote:
 On Sun, 2009-12-20 at 19:36 +0100, Kurt Roeckx wrote:
  On Sun, Dec 20, 2009 at 04:54:42PM +, Ben Hutchings wrote:
It also ways:
Link partner advertised auto-negotiation: No
Auto-negotiation: on

Those also seem to conflict and it really should advertise it.
   [...]
   
   No, Auto-negotiation: on does not mean auto-negotiation was
   successful.
  
  But the link parter does support auto-negotiation, while
  it says no.
 
 That's really a bug in ethtool; it should say not reported.  I'll
 submit a patch.

You patch has been merged and it present in the 2.6.34 from unstable.

Reassigning, I'll leave it to ethtool maintainers to actually close it.

Cheers,
Moritz



-- 
To UNSUBSCRIBE, email to debian-kernel-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/20100801211832.ga26...@galadriel.inutil.org



Processed: Re: Bug#561843: ethtool/mii-tool report incorrect/missing values on Marvell 88E1011

2010-08-01 Thread Debian Bug Tracking System
Processing commands for cont...@bugs.debian.org:

 reassign 561843 ethtool
Bug #561843 [src:linux-2.6] ethtool/mii-tool report incorrect/missing values on 
Marvell 88E1011
Bug reassigned from package 'src:linux-2.6' to 'ethtool'.
Bug No longer marked as found in versions linux-2.6/2.6.32-2.
 thanks
Stopping processing here.

Please contact me if you need assistance.
-- 
561843: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=561843
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems


-- 
To UNSUBSCRIBE, email to debian-kernel-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: 
http://lists.debian.org/handler.s.c.12806975189421.transcr...@bugs.debian.org



Bug#561843: ethtool/mii-tool report incorrect/missing values on Marvell 88E1011

2009-12-20 Thread Kurt Roeckx
Source: linux-2.6
Version: 2.6.32-2

Hi,

I get this:
# ethtool eth0 |grep -i pause
Advertised pause frame use: No
Link partner advertised pause frame use: No
# ethtool -a eth0
Pause parameters for eth0:
Autonegotiate:  on
RX: on
TX: on

Those seem to conflict with each other.  I assume this is a kernel
bug.

It also reports:
Link partner advertised link modes:  Not reported

It would be nice that it could report it and that atleast seems
to be supported with other cards.  As I understand this, this
should be supported since a 2.6.31 kernel.

It also ways:
Link partner advertised auto-negotiation: No
Auto-negotiation: on

Those also seem to conflict and it really should advertise it.
But it might be related to not reporting the link modes and
ethtool should show that instead?

On the other hand, mii-tool reports this:
# mii-tool -v
eth0: negotiated 100baseTx-FD flow-control, link ok
  product info: Yukon 88E1011 rev 5
  basic mode:   autonegotiation enabled
  basic status: autonegotiation complete, link ok
  capabilities: 1000baseT-HD 1000baseT-FD 100baseTx-FD 100baseTx-HD 10baseT-FD 
10baseT-HD
  advertising:  100baseTx-FD 100baseTx-HD 10baseT-FD 10baseT-HD flow-control
  link partner: 1000baseT-HD 1000baseT-FD 100baseTx-FD 100baseTx-HD 10baseT-FD 
10baseT-HD flow-control

The link partner is only a 100 mbit switch, so it shouldn't
advertise 1000.  It looks like the 2 last lines are switched.

lspci -v says:
00:0a.0 Ethernet controller: Marvell Technology Group Ltd. 88E8001 Gigabit 
Ethernet Controller (rev 13)
Subsystem: ASUSTeK Computer Inc. Marvell 88E8001 Gigabit Ethernet 
Controller (Asus)
Flags: bus master, 66MHz, medium devsel, latency 64, IRQ 17
Memory at fdd0 (32-bit, non-prefetchable) [size=16K]
I/O ports at b000 [size=256]
Expansion ROM at fdc0 [disabled] [size=128K]
Capabilities: [48] Power Management version 2
Capabilities: [50] Vital Product Data
Kernel driver in use: skge


Kurt




-- 
To UNSUBSCRIBE, email to debian-kernel-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#561843: ethtool/mii-tool report incorrect/missing values on Marvell 88E1011

2009-12-20 Thread Kurt Roeckx
On Sun, Dec 20, 2009 at 05:27:33PM +0100, Kurt Roeckx wrote:
 On the other hand, mii-tool reports this:
 # mii-tool -v
 eth0: negotiated 100baseTx-FD flow-control, link ok
   product info: Yukon 88E1011 rev 5
   basic mode:   autonegotiation enabled
   basic status: autonegotiation complete, link ok
   capabilities: 1000baseT-HD 1000baseT-FD 100baseTx-FD 100baseTx-HD 
 10baseT-FD 10baseT-HD
   advertising:  100baseTx-FD 100baseTx-HD 10baseT-FD 10baseT-HD flow-control
   link partner: 1000baseT-HD 1000baseT-FD 100baseTx-FD 100baseTx-HD 
 10baseT-FD 10baseT-HD flow-control
 
 The link partner is only a 100 mbit switch, so it shouldn't
 advertise 1000.  It looks like the 2 last lines are switched.

This part is part is probably #511395


Kurt




-- 
To UNSUBSCRIBE, email to debian-kernel-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#561843: ethtool/mii-tool report incorrect/missing values on Marvell 88E1011

2009-12-20 Thread Ben Hutchings
On Sun, 2009-12-20 at 17:27 +0100, Kurt Roeckx wrote:
 Source: linux-2.6
 Version: 2.6.32-2
 
 Hi,
 
 I get this:
 # ethtool eth0 |grep -i pause
 Advertised pause frame use: No
 Link partner advertised pause frame use: No
 # ethtool -a eth0
 Pause parameters for eth0:
 Autonegotiate:  on
 RX: on
 TX: on
 
 Those seem to conflict with each other.  I assume this is a kernel
 bug.

Driver bug.  A lot of drivers do not report specific pause frame
auto-negotiation flags.

 It also reports:
 Link partner advertised link modes:  Not reported
 
 It would be nice that it could report it and that atleast seems
 to be supported with other cards.  As I understand this, this
 should be supported since a 2.6.31 kernel.

It depends on the driver.  I provided patches for the generic MII code
but many drivers do not use that and do not yet provide any information
about link partner advertising.

 It also ways:
 Link partner advertised auto-negotiation: No
 Auto-negotiation: on
 
 Those also seem to conflict and it really should advertise it.
[...]

No, Auto-negotiation: on does not mean auto-negotiation was
successful.

Ben.

-- 
Ben Hutchings
Once a job is fouled up, anything done to improve it makes it worse.


signature.asc
Description: This is a digitally signed message part


Bug#561843: ethtool/mii-tool report incorrect/missing values on Marvell 88E1011

2009-12-20 Thread Kurt Roeckx
On Sun, Dec 20, 2009 at 04:54:42PM +, Ben Hutchings wrote:
  It also ways:
  Link partner advertised auto-negotiation: No
  Auto-negotiation: on
  
  Those also seem to conflict and it really should advertise it.
 [...]
 
 No, Auto-negotiation: on does not mean auto-negotiation was
 successful.

But the link parter does support auto-negotiation, while
it says no.


Kurt




-- 
To UNSUBSCRIBE, email to debian-kernel-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#561843: ethtool/mii-tool report incorrect/missing values on Marvell 88E1011

2009-12-20 Thread Ben Hutchings
On Sun, 2009-12-20 at 19:36 +0100, Kurt Roeckx wrote:
 On Sun, Dec 20, 2009 at 04:54:42PM +, Ben Hutchings wrote:
   It also ways:
   Link partner advertised auto-negotiation: No
   Auto-negotiation: on
   
   Those also seem to conflict and it really should advertise it.
  [...]
  
  No, Auto-negotiation: on does not mean auto-negotiation was
  successful.
 
 But the link parter does support auto-negotiation, while
 it says no.

That's really a bug in ethtool; it should say not reported.  I'll
submit a patch.

Ben.

-- 
Ben Hutchings
Once a job is fouled up, anything done to improve it makes it worse.


signature.asc
Description: This is a digitally signed message part