Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933463AbaDBWQ2 (ORCPT ); Wed, 2 Apr 2014 18:16:28 -0400 Received: from out4-smtp.messagingengine.com ([66.111.4.28]:42297 "EHLO out4-smtp.messagingengine.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932972AbaDBWQZ (ORCPT ); Wed, 2 Apr 2014 18:16:25 -0400 X-Sasl-enc: TP/P941uzVPI54yAGAe4+070NqESyz0nc6d09rDZ7KLM 1396476980 Date: Wed, 2 Apr 2014 15:18:42 -0700 From: Greg KH To: Andrew Morton Cc: Steven Rostedt , LKML , Linus Torvalds , Thomas Gleixner , "H. Peter Anvin" , Borislav Petkov , Ingo Molnar , Mel Gorman , Kay Sievers Subject: Re: [RFC PATCH] cmdline: Hide "debug" from /proc/cmdline Message-ID: <20140402221842.GA30420@kroah.com> References: <20140402144219.4cafbe37@gandalf.local.home> <20140402120440.1644d6a6486c554a33e584bc@linux-foundation.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20140402120440.1644d6a6486c554a33e584bc@linux-foundation.org> 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 Wed, Apr 02, 2014 at 12:04:40PM -0700, Andrew Morton wrote: > On Wed, 2 Apr 2014 14:42:19 -0400 Steven Rostedt wrote: > > > It has come to our attention that a system running a specific user > > space init program will not boot if you add "debug" to the kernel > > command line. What happens is that the user space tool parses the > > kernel command line, and if it sees "debug" it will spit out so much > > information that the system fails to boot. This basically renders the > > "debug" option for the kernel useless. > > > > This bug has been reported to the developers of said tool > > here: > > > > https://bugs.freedesktop.org/show_bug.cgi?id=76935 > > > > The response is: > > > > "Generic terms are generic, not the first user owns them." > > > > That is, the "debug" statement on the *kernel* command line is not > > owned by the kernel just because it was the first user of it, and > > they refuse to fix their bug. > > > > Well, my response is, we OWN the kernel command line, and as such, we > > can keep the users from seeing stuff on it if we so choose. And with > > that, I propose this patch, which hides "debug" from /proc/cmdline, > > such that we don't have to worry about tools parsing for it and causing > > hardship for those trying to debug the kernel. > > > > I had to check the date on this but surprisingly, it's all post > April 1. > > --- a/fs/read_write.c~a > +++ a/fs/read_write.c > @@ -513,6 +513,8 @@ SYSCALL_DEFINE3(read, unsigned int, fd, > struct fd f = fdget_pos(fd); > ssize_t ret = -EBADF; > > + BUG_ON(!strcmp(current->comm, "systemd")); > + > if (f.file) { > loff_t pos = file_pos_read(f.file); > ret = vfs_read(f.file, buf, count, &pos); > _ Sure, bring a box up and the first user that logs in you will crash the machine? That's going to be fun to explain to users. Unless Linux doesn't care about local users anymore. I understand the sediment, but really people... -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/