From mboxrd@z Thu Jan 1 00:00:00 1970 From: Cong Wang Subject: Re: [PATCH 0/7 respin] network,nfs,fstab-sys stuff Date: Thu, 02 Feb 2012 16:27:37 +0800 Message-ID: <4F2A48F9.90201@gmail.com> References: <20120202075944.GA12408@darkstar.nay.redhat.com> Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Return-path: DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=message-id:date:from:user-agent:mime-version:to:cc:subject :references:in-reply-to:content-type:content-transfer-encoding; bh=ksRUhNKvapAW5FyFA8xglCFOnjhY7MsMk+9lQMBJUCU=; b=SIdq9RCL6IQJ+vghJZOkNmVVv2jyRYlZjeYHj2Sn5HLln5esoAxT97gdac9fgpKcK9 juL94oDVJo9XK0fil5sXlnFPBJfmQkpg+xZv7xZPu9isF6Z6Hf81wSq5h6q1GJpbDx9/ Se6xPZSyIur6YAzeKPrPLbdt21L16T/HmxUy4= In-Reply-To: <20120202075944.GA12408-4/PLUo9XfK+sDdueE5tM26fLeoKvNuZc@public.gmane.org> Sender: initramfs-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-ID: Content-Type: text/plain; charset="us-ascii"; format="flowed" To: Dave Young Cc: initramfs-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, harald-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org On 02/02/2012 03:59 PM, Dave Young wrote: > This series deal with network startup without netroot and nfsmount > in fstab-sys. Patch 1/7, 2/7 and 3/7 is same with previous ones, > I just send them for easily tracking. > > [PATCH 1/7] allow ifup bring up network manually even without netroot > [PATCH 2/7] auto setup network without netroot > [PATCH 3/7] fix fstab-sys module check > [PATCH 4/7] add nobody group for rpc.idmapd > [PATCH 5/7] source dracut-lib.sh in nfs cleanup script > [PATCH 6/7] split nfs rpc daemons startup script > [PATCH 7/7] do not check dev node exist for fstab-sys mounting 3- 7 patches all look good to me now. I still keep my opinion on 2/7, I think we'd better not add more cmdline unless we have to, until now, you still don't prove "we have to do that". :) > > Changes from last version are: > 4/7: new fix for nfs > 5/7: new fix for nfs > 6/7: start nfs-start-rpc.sh in pre-udev hook > 7/7: remove fsck check for nfs due to fsck_able already do this for us. It would be nicer to put these into per patch. Thanks.