From mboxrd@z Thu Jan 1 00:00:00 1970 From: Albert ARIBAUD Date: Mon, 3 Sep 2012 18:50:14 +0200 Subject: [U-Boot] [PATCH] mtest: Disable dcache during test In-Reply-To: <356588499.3415554.1346682315106.JavaMail.root@advansee.com> References: <20120902163023.6AC19203B2F@gemini.denx.de> <356588499.3415554.1346682315106.JavaMail.root@advansee.com> Message-ID: <20120903185014.624f6939@lilith> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi Beno?t, On Mon, 3 Sep 2012 16:25:15 +0200 (CEST), Beno?t Th?baudeau wrote: > Dear Wolfgang Denk, > > On Sunday, September 2, 2012 6:30:23 PM, Wolfgang Denk wrote: > > Dear Beno?t Th?baudeau, > > > > In message > > <1725235724.2300239.1344694624384.JavaMail.root@advansee.com> you > > wrote: > > > > > > On 08/11/2012 05:18 AM, Mike Frysinger wrote: > > ... > > > > if you want dcache disabled, then why don't you run `dcache off` > > > > first ? i > > > > think it's useful to be able to do both, and forcing it one way > > > > is > > > > wrong. > > > > > > > > thus, NAK from me. > > > > -mike > > > > > > Because you will very likely trust mtest and forget about running > > > `dcache off` > > > first, so you may then be happy about falsely positive mtest > > > results. Moreover, > > > I can't find any sense or usefulness in running mtest with dcache > > > enabled. > > > > I agree with Mike. > > > > "UNIX was not designed to stop you from doing stupid things, > > because that would also stop you from doing clever things." - > > Doug Gwyn > > > > So NAK from me, too. > > OK, but do you agree with the following that Mike and me agreed on > after that? > http://lists.denx.de/pipermail/u-boot/2012-August/130650.html > http://lists.denx.de/pipermail/u-boot/2012-August/130726.html > http://patchwork.ozlabs.org/patch/176909/ I did already reply to this, but since the agreement is brought back, I think I should re-state my opinion: such a warning line will most likely be overlooked, thus has little value. People using mtest should know that they must check/set dcache state before running mtest. > Best regards, > Beno?t Amicalement, -- Albert.