From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753930AbaE3AHW (ORCPT ); Thu, 29 May 2014 20:07:22 -0400 Received: from mx1.redhat.com ([209.132.183.28]:15674 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751809AbaE3AHT (ORCPT ); Thu, 29 May 2014 20:07:19 -0400 Date: Thu, 29 May 2014 20:06:49 -0400 From: Dave Jones To: Dave Chinner Cc: Linus Torvalds , Jens Axboe , Minchan Kim , Linux Kernel Mailing List , Andrew Morton , linux-mm , "H. Peter Anvin" , Ingo Molnar , Peter Zijlstra , Mel Gorman , Rik van Riel , Johannes Weiner , Hugh Dickins , Rusty Russell , "Michael S. Tsirkin" , Dave Hansen , Steven Rostedt Subject: Re: [RFC 2/2] x86_64: expand kernel stack to 16K Message-ID: <20140530000649.GA3477@redhat.com> Mail-Followup-To: Dave Jones , Dave Chinner , Linus Torvalds , Jens Axboe , Minchan Kim , Linux Kernel Mailing List , Andrew Morton , linux-mm , "H. Peter Anvin" , Ingo Molnar , Peter Zijlstra , Mel Gorman , Rik van Riel , Johannes Weiner , Hugh Dickins , Rusty Russell , "Michael S. Tsirkin" , Dave Hansen , Steven Rostedt References: <1401260039-18189-1-git-send-email-minchan@kernel.org> <1401260039-18189-2-git-send-email-minchan@kernel.org> <20140528223142.GO8554@dastard> <20140529013007.GF6677@dastard> <20140529072633.GH6677@dastard> <20140529235308.GA14410@dastard> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20140529235308.GA14410@dastard> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, May 30, 2014 at 09:53:08AM +1000, Dave Chinner wrote: > That sounds like a plan. Perhaps it would be useful to add a > WARN_ON_ONCE(stack_usage > 8k) (or some other arbitrary depth beyond > 8k) so that we get some indication that we're hitting a deep stack > but the system otherwise keeps functioning. That gives us some > motivation to keep stack usage down but isn't a fatal problem like > it is now.... We have check_stack_usage() and DEBUG_STACK_USAGE for this. Though it needs some tweaking if we move to 16K I gave it a try yesterday, and noticed a spew of noisy warnings as soon as I gave it a workload to chew on. (Moreso than usual with 8K stacks) Dave