netdev - Re: [PATCH 1/1] Revert "net: fec: Ensure clocks are enabled while using mdio bus"

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

Message-ID: <1439544994.13210.45.camel@pengutronix.de>
Date:	Fri, 14 Aug 2015 11:36:34 +0200
From:	Lucas Stach <l.stach@...gutronix.de>
To:	Peter Chen <peter.chen@...escale.com>
Cc:	"davem@...emloft.net" <davem@...emloft.net>,
	"netdev@...r.kernel.org" <netdev@...r.kernel.org>,
	Duan Andy <fugang.duan@...escale.com>,
	"shawn.guo@...aro.org" <shawn.guo@...aro.org>,
	Fabio Estevam <Fabio.Estevam@...escale.com>,
	"tyler.baker@...aro.org" <tyler.baker@...aro.org>,
	Andrew Lunn <andrew@...n.ch>
Subject: Re: [PATCH 1/1] Revert "net: fec: Ensure clocks are enabled while
 using mdio bus"

Am Freitag, den 14.08.2015, 16:14 +0800 schrieb Peter Chen:
> On Fri, Aug 14, 2015 at 10:27:33AM +0200, Lucas Stach wrote:
> > Am Freitag, den 14.08.2015, 08:25 +0000 schrieb Peter Chen:
> > >  
> > > > 
> > > > Am Freitag, den 14.08.2015, 13:47 +0800 schrieb Peter Chen:
> > > > > It causes the i.mx6sx sdb board hang when using nfsroot during boots
> > > > > up at v4.2-rc6.
> > > > >
> > > > > This reverts commit 8fff755e9f8d0f70a595e79f248695ce6aef5cc3.
> > > > >
> > > > > Cc: netdev@...r.kernel.org
> > > > > Cc: Fugang Duan <B38611@...escale.com>
> > > > > Cc: shawn.guo@...aro.org
> > > > > Cc: fabio.estevam@...escale.com
> > > > > Cc: tyler.baker@...aro.org
> > > > > Cc: Lucas Stach <l.stach@...gutronix.de>
> > > > > Cc: Andrew Lunn <andrew@...n.ch>
> > > > > Signed-off-by: Peter Chen <peter.chen@...escale.com>
> > > > > ---
> > > > >
> > > > > According to Fugang Duan, the i.mx series has different clock control
> > > > > sequence among SoCs, this patch may only consider certain SoCs.
> > > > >
> > > > Sorry, but NACK.
> > > > 
> > > > Please test current mainline (what will become v4.2-rc7). There is already a
> > > > patch in that fixes i.MX27 and probably fixes the same problem on i.MX6SX.
> > > > 
> > > 
> > > Would you help point to me which commit and at which tree?
> > > 
> > Mainline, so Linus Torvalds tree.
> > 
> > http://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=14d2b7c1a96ef37eb571599c73d4a1a606b964d6
> 
> It fixes my imx6sx-sdb board. It is interesting that there was
> no problem for some platforms, but with problem for others.
> Your fix is a common runtime PM fix.
> 
That's because on i.MX6Q/DL IPG and AHB clock are the same clock, on
i.MX27 and apparently i.MX6SX they are different, so those chips fail if
RPM is disabling the clock at the wrong point in time.

> Again, why we need this as a bug-fix, not but as new feature for next
> rc1?

It fixes MDIO attached switches on Vybrid, but you are right this
probably could have waited until the next merge window. But this is the
wrong question to ask after we got in all the fixes to keep things from
regressing.

Seeing that people test those things pretty late (the original broken
patch got in with -rc2!) moving things to the next merge window would
just have people complaining during the v4.3 RC phase, instead of now
during the 4.2 RC phase.

Takeaway for everyone involved: test things more thoroughly and earlier.

Regards,
Lucas

-- 
Pengutronix e.K.             | Lucas Stach                 |
Industrial Linux Solutions   | http://www.pengutronix.de/  |

--
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