Re: [PATCH] alx: add a simple AR816x/AR817x device driver

[Johannes Berg <johannes@sipsolutions.net>]

On Fri, 2013-06-14 at 00:24 +0200, Johannes Stezenbach wrote:

> [    0.939189] INFO: trying to register non-static key.
> [    0.939787] the code is fine but needs lockdep annotation.
> [    0.940387] turning off the locking correctness validator.
> [    0.940992] Pid: 1, comm: swapper/0 Not tainted 3.9.6 #2
> [    0.941589] Call Trace:
> [    0.942179]  [<ffffffff8108cead>] __lock_acquire.isra.24+0x710/0xa47
> [    0.942781]  [<ffffffff8108d87f>] lock_acquire+0xc0/0x142

I *think* this is probably the missing spin_lock_init I had forgotten.

> and after "ifconfig eth0 up":
> 
> [  539.482153] alx 0000:03:00.0: irq 45 for MSI/MSI-X
> [  539.482249] IPv6: ADDRCONF(NETDEV_UP): eth0: link is not ready
> [  539.483169] alx 0000:03:00.0 eth0: NIC Up: 1 Gbps Full
> [  539.483354] IPv6: ADDRCONF(NETDEV_CHANGE): eth0: link becomes ready
> [  544.801438] ------------[ cut here ]------------
> [  544.801458] WARNING: at net/sched/sch_generic.c:255 dev_watchdog+0x19b/0x201()
> [  544.801462] Hardware name: System Product Name
> [  544.801464] NETDEV WATCHDOG: eth0 (alx): transmit queue 0 timed out

Hmm, this is odd. I might just be forgetting to set something.

> 03:00.0 Ethernet controller: Qualcomm Atheros AR8161 Gigabit Ethernet (rev 08)

OTOH, I have a 8171, so maybe I messed up with my simplifications or it
simply doesn't work.

> Any idea what to try?

Does it still pass traffic after that? Or is it actually dead?

johannes