Re: bug: mutex_lock() in interrupt conntext via phy_stop() in gianfar

2008-07-24 Thread Sebastian Siewior
* Benjamin Herrenschmidt | 2008-07-24 08:12:48 [+1000]: >On Mon, 2008-07-21 at 17:57 -0500, Nate Case wrote: >> On Fri, 2008-07-18 at 14:10 +0200, Sebastian Siewior wrote: >> > Commit 35b5f6b1a aka [PHYLIB: Locking fixes for PHY I/O potentially >> > sleeping] >> > changed the phydev->lock from sp

Re: bug: mutex_lock() in interrupt conntext via phy_stop() in gianfar

2008-07-23 Thread Benjamin Herrenschmidt
On Mon, 2008-07-21 at 17:57 -0500, Nate Case wrote: > On Fri, 2008-07-18 at 14:10 +0200, Sebastian Siewior wrote: > > Commit 35b5f6b1a aka [PHYLIB: Locking fixes for PHY I/O potentially > > sleeping] > > changed the phydev->lock from spinlock into a mutex. Now, the following > > code path got trig

Re: bug: mutex_lock() in interrupt conntext via phy_stop() in gianfar

2008-07-22 Thread Sebastian Siewior
* Nate Case | 2008-07-21 17:57:08 [-0500]: >On Fri, 2008-07-18 at 14:10 +0200, Sebastian Siewior wrote: >> Commit 35b5f6b1a aka [PHYLIB: Locking fixes for PHY I/O potentially sleeping] >> changed the phydev->lock from spinlock into a mutex. Now, the following >> code path got triggered while NFS w

Re: bug: mutex_lock() in interrupt conntext via phy_stop() in gianfar

2008-07-22 Thread Wolfram Sang
Hi, On Fri, Jul 18, 2008 at 02:10:08PM +0200, Sebastian Siewior wrote: > Commit 35b5f6b1a aka [PHYLIB: Locking fixes for PHY I/O potentially sleeping] > changed the phydev->lock from spinlock into a mutex. Now, the following > code path got triggered while NFS was unavailable: [...] > I found out

Re: bug: mutex_lock() in interrupt conntext via phy_stop() in gianfar

2008-07-21 Thread Nate Case
On Fri, 2008-07-18 at 14:10 +0200, Sebastian Siewior wrote: > Commit 35b5f6b1a aka [PHYLIB: Locking fixes for PHY I/O potentially sleeping] > changed the phydev->lock from spinlock into a mutex. Now, the following > code path got triggered while NFS was unavailable: > [snip] > |[ 194.864733] BUG:

bug: mutex_lock() in interrupt conntext via phy_stop() in gianfar

2008-07-18 Thread Sebastian Siewior
Commit 35b5f6b1a aka [PHYLIB: Locking fixes for PHY I/O potentially sleeping] changed the phydev->lock from spinlock into a mutex. Now, the following code path got triggered while NFS was unavailable: |[ 21.287359] nfs: server 10.11.3.47 not responding, still trying |[ 38.891373] nfs: server 1