linux-kernel - Re: [tip:x86/urgent] x86/microcode/amd: Tone down printk(), don' t treat a missing firmware file as an error

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]

Message-ID: <20131113021136.GA31650@kroah.com>
Date:	Wed, 13 Nov 2013 11:11:36 +0900
From:	Greg Kroah-Hartman <gregkh@...uxfoundation.org>
To:	Ingo Molnar <mingo@...nel.org>
Cc:	Borislav Petkov <bp@...e.de>, hpa@...or.com,
	linux-kernel@...r.kernel.org, stable@...r.kernel.org,
	tglx@...utronix.de, trenn@...e.de,
	linux-tip-commits@...r.kernel.org
Subject: Re: [tip:x86/urgent] x86/microcode/amd: Tone down printk(), don' t
 treat a missing firmware file as an error

On Wed, Nov 13, 2013 at 12:10:08AM +0100, Ingo Molnar wrote:
> 
> * Borislav Petkov <bp@...e.de> wrote:
> 
> > (fixed stable@...r).
> > 
> > On Tue, Nov 12, 2013 at 11:45:11PM +0100, Borislav Petkov wrote:
> > > On Tue, Nov 12, 2013 at 01:58:00PM -0800, tip-bot for Thomas Renninger wrote:
> > > > Commit-ID:  11f918d3e2d3861b6931e97b3aa778e4984935aa
> > > > Gitweb:     http://git.kernel.org/tip/11f918d3e2d3861b6931e97b3aa778e4984935aa
> > > > Author:     Thomas Renninger <trenn@...e.de>
> > > > AuthorDate: Tue, 12 Nov 2013 17:39:43 +0100
> > > > Committer:  Ingo Molnar <mingo@...nel.org>
> > > > CommitDate: Tue, 12 Nov 2013 22:03:49 +0100
> > > > 
> > > > x86/microcode/amd: Tone down printk(), don't treat a missing firmware file as an error
> > > > 
> > > > Do it the same way as done in microcode_intel.c: use pr_debug()
> > > > for missing firmware files.
> > > > 
> > > > There seem to be CPUs out there for which no microcode update
> > > > has been submitted to kernel-firmware repo yet resulting in
> > > > scary sounding error messages in dmesg:
> > > > 
> > > >   microcode: failed to load file amd-ucode/microcode_amd_fam16h.bin
> > > > 
> > > > Signed-off-by: Thomas Renninger <trenn@...e.de>
> > > > Acked-by: Borislav Petkov <bp@...e.de>
> > > > Cc: <stable@...nel.org>
> > > 
> > > Shouldn't that be <stable@...r.kernel.org> ?
> > 
> > Yes, it should:
> > 
> > Final-Recipient: rfc822; stable@...nel.org
> > Action: failed
> > Status: 5.0.0
> > Diagnostic-Code: X-Postfix; host mail.kernel.org[198.145.19.201] said: 550
> >     5.1.1 <stable@...nel.org>: Recipient address rejected: User unknown in
> >     local recipient table (in reply to RCPT TO command)
> 
> Hm, that's really weird, I've been using <stable@...nel.org> for years and 
> the commits do get picked up. I also never saw such a mailer failure.
> 
> In any case I've changed my pre-cooked alias to <stable@...r.kernel.org>, 
> but still I'm wondering why <stable@...nel.org> seems to be working in 
> practice - maybe Greg is picking up such commits as well, not via an email 
> flow but via scripting?

Yes, my scripts pick up both, as this is a very common mistake (the
address did change after kernel.org got rebuilt, so you were right in
remembering this as the original address to use.)

So no worries, I can handle both.

thanks,

greg k-h
--
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/