From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751503AbdAaKBu (ORCPT ); Tue, 31 Jan 2017 05:01:50 -0500 Received: from mail.skyhub.de ([78.46.96.112]:51629 "EHLO mail.skyhub.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750809AbdAaKBl (ORCPT ); Tue, 31 Jan 2017 05:01:41 -0500 Date: Tue, 31 Jan 2017 11:01:22 +0100 From: Borislav Petkov To: Ingo Molnar Cc: Mike Galbraith , Andrey Ryabinin , Thomas Gleixner , LKML , "H. Peter Anvin" Subject: Re: [PATCH] x86/microcode: Do not access the initrd after it has been freed Message-ID: <20170131100122.r6ubk5z3t2vm3hxl@pd.tnic> References: <3e4b6f03-384d-411e-5243-9d3b0595d5cb@virtuozzo.com> <20170125172321.fpy6cswq3ibqvfgo@pd.tnic> <16165aad-5990-51f9-4dc7-c4b68597a4d5@virtuozzo.com> <20170125192336.kmgxmwcffdo2gxrf@pd.tnic> <20170126165833.evjemhbqzaepirxo@pd.tnic> <7d07442a-c64e-797b-c8c4-ef453df30cde@virtuozzo.com> <20170127090915.zh7ct7cnvx2ds7qk@pd.tnic> <20170130084632.GA25091@gmail.com> <20170130093527.syjz7yldgndml7qb@pd.tnic> <20170131074355.GA17636@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <20170131074355.GA17636@gmail.com> User-Agent: NeoMutt/20161014 (1.7.1) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Jan 31, 2017 at 08:43:55AM +0100, Ingo Molnar wrote: > (Cc:-ed Mike as this could explain his early boot crash/hang? > Mike: please try -tip f18a8a0143b1 that I just pushed out. ) One other thing to try, Mike, is boot with "dis_ucode_ldr". See whether that makes it go away. > Nice - I've updated the branch with your resolution. Could you please > double-check the double checked resolution? That's like quadruple-checked :-) > So the diff between your resolution and mine is attached below - now fpu/core.c > changes, so I'm not sure why fpu/core.c is in your diff? Hmm, maybe I did the diff wrong? So this is how I generated it: $ git diff tip/x86/microcode~1..tip/x86/microcode meaning that I want to see what the merge commit changed. That's why this pulled in the fpu/core.c hunk which is new in urgent. In any case, the merge looks ok to me, I'll be testing it anyway in the coming days so I'll let you know if anything goes nuts. Thanks! -- Regards/Gruss, Boris. Good mailing practices for 400: avoid top-posting and trim the reply.