Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752672AbcJDHDa (ORCPT ); Tue, 4 Oct 2016 03:03:30 -0400 Received: from mail-pf0-f170.google.com ([209.85.192.170]:35338 "EHLO mail-pf0-f170.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752029AbcJDHD3 (ORCPT ); Tue, 4 Oct 2016 03:03:29 -0400 Date: Tue, 04 Oct 2016 00:03:25 -0700 From: Raymond Jennings Subject: Re: BUG_ON() in workingset_node_shadows_dec() triggers To: Linus Torvalds Cc: Andrew Morton , Johannes Weiner , Antonio SJ Musumeci , Miklos Szeredi , Linux Kernel Mailing List , stable Message-Id: <1475564605.7361.11@smtp.gmail.com> In-Reply-To: References: <20161003210729.e4c39a86b8e8e4d103715c7d@linux-foundation.org> X-Mailer: geary/0.11.1 MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8; format=flowed Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1303 Lines: 40 On Mon, Oct 3, 2016 at 9:12 PM, Linus Torvalds wrote: > On Mon, Oct 3, 2016 at 9:07 PM, Andrew Morton > wrote: >> >> Well, it's a VM_BUG_ON and few people run with CONFIG_DEBUG_VM. > > Ehh. If by "few people" you mean "pretty much everybody", you'd be > right, but your choice of wording would be somewhat misleading, > wouldn't you say? > > Hint: here's a line from the standard Fedora kernel config: > > CONFIG_DEBUG_VM=y > > so *no*. VM_BUG_ON() is no less deadly than a regular BUG_ON(). It > just allows some people to build smaller kernels, but apparently > distro people would rather have debugging than save a few kB of RAM. > > The VM debvugging code has VM_WARN_ON() and VM_WARN_ON_ONCE() for > people who want to get a "oops, my assumptions were wrong" > > Killing machines because somebody made an assumption that was wrong > is not ok. > > Killing the machine is ok if we have a situation where there literally > is no other choice. For the curious: This would include situations like 1. The kernel is confused and further processing would result in undefined behavior (like bluesmoke detecting PCC for example) 2. Security hazards where we'd leak stuff if we don't shut down. ? > Linus