From mboxrd@z Thu Jan 1 00:00:00 1970 From: Yu Zhiguo Subject: Re: [PATCH] xencommons: mount xenfs directly if xenfs it not be loaded Date: Tue, 22 Jun 2010 09:18:56 +0800 Message-ID: <4C200F80.5030306@cn.fujitsu.com> References: <4C1C744F.8030903@cn.fujitsu.com> <4C1C7B88.1000907@cn.fujitsu.com> <4C1F34D9.4000006@goop.org> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <4C1F34D9.4000006@goop.org> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xensource.com Errors-To: xen-devel-bounces@lists.xensource.com To: Jeremy Fitzhardinge Cc: "xen-devel@lists.xensource.com" , Keir Fraser List-Id: xen-devel@lists.xenproject.org Hi Jeremy Jeremy Fitzhardinge wrote: >> > > It should just be sufficient to explicitly mount /proc/xen, which will > trigger a modprobe if necessary. > > J > OK, I fixed it, new patch is here. Just deleted check for xenfs support before mount xenfs. ------------------ If xenfs is compiled as a module, now cannot complete 'service xencommons start', mount xenfs directly can fix it, it can trigger modprobe xenfs. Signed-off-by: Yu Zhiguo diff -r bd3bf925f4ce -r 6c8c32876b30 tools/hotplug/Linux/init.d/xencommons --- a/tools/hotplug/Linux/init.d/xencommons Tue Jun 22 17:02:30 2010 +0800 +++ b/tools/hotplug/Linux/init.d/xencommons Tue Jun 22 17:21:44 2010 +0800 @@ -27,7 +27,6 @@ if test "x$1" = xstart && \ test -d /proc/xen && \ ! test -d /proc/xen/capabilities && \ - grep ' xenfs$' /proc/filesystems >/dev/null && \ ! grep '^xenfs ' /proc/mounts >/dev/null; then mount -t xenfs xenfs /proc/xen