From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andrew Morton Subject: Re: [PATCH] init/do_mounts.c : Create /root if it does not exits Date: Wed, 30 Nov 2011 12:55:55 -0800 Message-ID: <20111130125555.85ac54e7.akpm@linux-foundation.org> References: Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Cc: , linux-fsdevel@vger.kernel.org, Al Viro To: yanhong Return-path: Received: from mail.linuxfoundation.org ([140.211.169.12]:54772 "EHLO mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753331Ab1K3Uz4 (ORCPT ); Wed, 30 Nov 2011 15:55:56 -0500 In-Reply-To: Sender: linux-fsdevel-owner@vger.kernel.org List-ID: On Sat, 26 Nov 2011 08:34:18 +0000 yanhong wrote: > > From: YanHong > > If someone supplies an initramfs without /root in it, and we fail to execute rdinit, we will try to mount root device and fail, for the mount point does not exits. > > But we get error message "VFS: Cannot open root device". It's confusing. > > We can give more detailed error message, or we can go further: if /root does not exits, create one. > I really don't know enough about initramfs usage to know if this is a good or bad thing. Can anyone else comment? > --- a/init/do_mounts.c > +++ b/init/do_mounts.c > @@ -350,6 +350,9 @@ void __init mount_block_root(char *name, int flags) > const char *b = name; > #endif > > + if (sys_access((const char __user *) "/root", 0) != 0) > + sys_mkdir((const char __user *) "/root", 0700); > + > get_fs_names(fs_names); > retry: > for (p = fs_names; *p; p += strlen(p)+1) { I suppose we could remove the sys_access() check and just run mkdir(), which will fail to do anything if /root already exists.