From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ingo Molnar Subject: Re: [4.9-rc3] BUG: unable to handle kernel paging request at ffffc900144dfc60 Date: Thu, 3 Nov 2016 07:32:23 +0100 Message-ID: <20161103063223.GA3267@gmail.com> References: <201611012336.IAC18714.VLMOQSHOFtOFJF@I-love.SAKURA.ne.jp> <201611021950.FEJ34368.HFFJOOMLtQOVSF@I-love.SAKURA.ne.jp> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org To: Linus Torvalds Cc: Tetsuo Handa , Peter Zijlstra , Ingo Molnar , Andrew Lutomirski , the arch/x86 maintainers , Linux Kernel Mailing List , Brian Gerst , Borislav Petkov , Jann Horn , Linux API , Kees Cook , Tycho Andersen List-Id: linux-api@vger.kernel.org * Linus Torvalds wrote: > On Wed, Nov 2, 2016 at 4:50 AM, Tetsuo Handa > wrote: > >> > >> So I _think_ the fix is to just remove this. Perhaps something like > >> the attached? Adding scheduler people since this is in their code.. > > > > That is not sufficient, for another Oops occurs inside stack_not_used(). > > Since I don't want to break stack_not_used(), can we tolerate nested > > try_get_task_stack() usage and protect the whole sched_show_task()? > > Sure, looks good to me. > > But as Andy says, we should probably apply my patch too, just to get > rid of the useless hex printk and the ugly ifdeffery. Agreed, and I have applied both fixes to sched/urgent. Thanks, Ingo