netdev - RE: phylib: kernel oops on resume with 2.6.33-rc4

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

Date:	Sun, 17 Jan 2010 13:58:39 +0530
From:	"Nori, Sekhar" <nsekhar@...com>
To:	"avorontsov@...mvista.com" <avorontsov@...mvista.com>
CC:	"netdev@...r.kernel.org" <netdev@...r.kernel.org>
Subject: RE: phylib: kernel oops on resume with 2.6.33-rc4

On Sun, Jan 17, 2010 at 04:07:03, Anton Vorontsov wrote:
> Hello,
>
> On Sat, Jan 16, 2010 at 09:38:34PM +0530, Nori, Sekhar wrote:
> > Hello all,
> >
> > I am hitting a kernel oops when resuming from suspend-to-RAM on 2.6.33-rc4
> > kernel.
> >
> > The MAC is TI DaVinci EMAC and PHY is SMSC LAN8710. However, I also hit the
> > issue even when using generic PHY driver (CONFIG_SMSC_PHY turned off).
> >
> > On reverting commit 541cd3e "phylib: Fix deadlock on resume" which was
> > introduced in 2.6.33-rc4, the issue doesn't come about. So far, I haven't
> > figured out how this particular commit is causing the oops. Full oops report
> > is attached. Appreciate any help in debugging this.
>
> It seems phy_start_machine() needlessly reinitialized work
> struct. This is OK for ucc_geth and gianfar driver because
> these drivers do not detach PHYs on suspend, but TI davinci
> does. :-/
>
> Can you try the patch down below?

This seems to have fixed the problem. Thanks! I hope this can be
included for the next rc. FWIW,

Reported-and-Tested-by: Sekhar Nori <nsekhar@...com>

Thanks,
Sekhar