From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753807AbbL2Smv (ORCPT ); Tue, 29 Dec 2015 13:42:51 -0500 Received: from bombadil.infradead.org ([198.137.202.9]:46231 "EHLO bombadil.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753798AbbL2Smm (ORCPT ); Tue, 29 Dec 2015 13:42:42 -0500 Subject: Re: [PATCH 1/1] kernel booting messages, first line To: Alexandru GOIA , "linux-kernel@vger.kernel.org" References: Cc: Tejun Heo From: Randy Dunlap Message-ID: <5682D420.3030207@infradead.org> Date: Tue, 29 Dec 2015 10:42:40 -0800 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 12/28/15 17:31, Alexandru GOIA wrote: > > For -rc8 , > > Hoping that someone DO cares about the aesthetic of kernel messages at boot : > > > What it does : first line of booting kernel, as seen in dmesg(1), will be the Linux kernel signature > (version, compiler, compiler hostname). > > What it solves : it appers ahead of cgroup messages at booting. > > > This ugliness, since 3.10.x kernel. > > > So : > > Instead of the unaesthetic lines : > > Initializing cgroup subsys cpuset > Initializing cgroup subsys cpu > Initializing cgroup subsys cpuacct > Linux version 4 ... > > > ... We will have : > > > Linux version 4 ... > Initializing cgroup subsys cpuset > Initializing cgroup subsys cpu > Initializing cgroup subsys cpuacct > > > As normal . > > > In file linux-4.4-rc7/init/main.c : > > > > --- main.c 2015-12-28 04:17:37.000000000 +0200 > +++ main+.c 2015-12-29 03:18:59.896820046 +0200 > @@ -512,6 +512,7 @@ > * Set up the the initial canary ASAP: > */ > boot_init_stack_canary(); > + pr_notice("%s", linux_banner); > > cgroup_init_early(); > > @@ -524,7 +525,6 @@ > */ > boot_cpu_init(); > page_address_init(); > - pr_notice("%s", linux_banner); > setup_arch(&command_line); > mm_init_cpumask(&init_mm); > setup_command_line(command_line); > > > > > Signed-off-by: Alexandru Goia -- I like the patch. Could you make it a proper kernel patch, please? thanks, -- ~Randy