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  PHC 
Open Source and information security mailing list archives
 
Hash Suite for Android: free password hash cracker in your pocket
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date:	Tue, 11 Mar 2014 00:06:29 +0100
From:	Paul Bolle <pebolle@...cali.nl>
To:	Thomas Gleixner <tglx@...utronix.de>
Cc:	"H. Peter Anvin" <hpa@...or.com>,
	Jerome Oufella <jerome.oufella@...oirfairelinux.com>,
	Julian Wollrath <jwollrath@....de>, x86@...nel.org,
	linux-kernel@...r.kernel.org
Subject: Re: [RESEND] Fast TSC calibration fails with v3.14-rc1 and later

On Mon, 2014-03-10 at 21:50 +0100, Thomas Gleixner wrote:
> On Mon, 10 Mar 2014, Paul Bolle wrote:
> > So could you please consider downgrading this message?
> >
> > Yes, this will make regressions less visible, although I'm unsure we're
> > actually seeing a regression here. But I think that this is better than
> > scaring people about a situation they can't easily do something about,
> > if at all. Especially since fast TSC calibration failures, apparently,
> > can be worked around.
> 
> If something works fine from version X up to v3.13 and then suddenly
> fails, then we can safely ignore it because there is a work around or
> fallback? And just shrug and say: Oh, it's not a regression.

That's not what I said, not at all. Saying that "I'm unsure we're
actually seeing a regression" is not shrugging it off.
 
> Dammit no. We want to know WHY!

Here I am one and half year after reporting seeing this error. I'm not
aware of anyone reporting it before me. Anyhow, I don't think things
were working fine up to v3.13. And in my report (I'm not going to link
to it again) I suggested to downgrade this message. My exact words were:
    So, without even understanding why tsc calibration is needed, it does
    look unnecessary to print "Fast TSC calibration failed" at error level.
    If that's correct, I'd be happy to submit the trivial patch to downgrade
    it to (say) informational level.

Which was fine with hpa (added in CC).

I actually submitted that patch, sent a reminder, and Jerome sent a
similar patch. Most of that was directly sent to you too. I don't think
we got any reaction whatsoever. And, still, suggesting this is not a
regression and reiterating that this could also be printed at something
else than error level somehow provoked your outburst.

I'd like to discuss a way forward. But could you please _try_ to
understand why I said what I said in this discussion?


Paul Bolle

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Powered by blists - more mailing lists