All of lore.kernel.org
 help / color / mirror / Atom feed
From: Seewer Philippe <philippe.seewer-omB+W0Dpw2o@public.gmane.org>
To: Harald Hoyer <harald-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
Cc: initramfs-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Subject: Re: [PATCH 08/10] put back the nfs mount in the udev event
Date: Mon, 6 Jul 2009 10:39:17 +0200	[thread overview]
Message-ID: <4A51B835.1050302@bfh.ch> (raw)
In-Reply-To: <1246639520-3094-9-git-send-email-harald-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>

Harald Hoyer wrote:
> it solves the following case:
> 
> root=/dev/nfs nfsroot=server:/path
> 
> - the server could be reachable on any interface
> - any interface can get an IP by dhcp
> - only one IP is allowed to mount the root

Thanks.

For the record, some further explanations:

If netroot is in "guess-the-interface-mode", netroot calls the
root-handler for each interface that has a useable IP configuration. If
the root-handler fails, the interface is deconfigured so that the next
interface netroot tries does not suffer from side-effects.

This is accomplished by depending on mount-handlers having an exit-code
telling us whether mounting (would) succeed or not. For devicefile based 
network mounts, this isn't a problem since we need to call some sort of
client-utility that makes the device nodes available (or not if
contacting the server fails). For network-filesystems like NFS that do
not use devicefiles, the actual mount call is needed.


> ---
>  modules.d/95nfs/nfsroot |   11 +++++------
>  1 files changed, 5 insertions(+), 6 deletions(-)
> 
> diff --git a/modules.d/95nfs/nfsroot b/modules.d/95nfs/nfsroot
> index 5f5b08f..0add977 100755
> --- a/modules.d/95nfs/nfsroot
> +++ b/modules.d/95nfs/nfsroot
> @@ -142,9 +142,9 @@ if [ "$nfs" = "nfs4" ]; then
>      [ -z "$(pidof rpc.idmapd)" ] && rpc.idmapd
>  
>      # XXX Should we loop here?
> -    echo mount -t nfs4 -o$options${nfslock+,$nfslock} \
> -	$server:$path $NEWROOT > /mount/01-$$-nfs4.sh
> -    [ -e /dev/root ] || >/dev/root
> +    mount -t nfs4 -o$options${nfslock+,$nfslock} \
> +	$server:$path $NEWROOT \
> +	&& { [ -e /dev/root ] || >/dev/root ; }
>  else
>      # NFSv{2,3} doesn't support using locks as it requires a helper to transfer
>      # the rpcbind state to the new root
> @@ -152,7 +152,6 @@ else
>          warn "Locks unsupported on NFSv{2,3}, using nolock" 1>&2
>  
>      # XXX Should we loop here?
> -    echo mount -t nfs -o$options${options:+,}nolock $server:$path $NEWROOT \
> -        > /mount/01-$$-nfs.sh
> -    [ -e /dev/root ] || >/dev/root
> +    mount -t nfs -o$options${options:+,}nolock $server:$path $NEWROOT \
> +	&& { [ -e /dev/root ] || >/dev/root ; }
>  fi
--
To unsubscribe from this list: send the line "unsubscribe initramfs" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

  parent reply	other threads:[~2009-07-06  8:39 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-07-03 16:45 [PATCH 00/10] *** SUBJECT HERE *** Harald Hoyer
     [not found] ` <1246639520-3094-1-git-send-email-harald-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2009-07-03 16:45   ` [PATCH 01/10] add binutiles requirement to specfile (because of nm) Harald Hoyer
2009-07-03 16:45   ` [PATCH 02/10] output everything to /dev/kmesg and add dmesg for the emergency_shell Harald Hoyer
     [not found]     ` <1246639520-3094-3-git-send-email-harald-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2009-07-03 18:11       ` David Dillow
     [not found]         ` <1246644686.13823.5.camel-1q1vX8mYZiGLUyTwlgNVppKKF0rrzTr+@public.gmane.org>
2009-07-03 18:14           ` Harald Hoyer
2009-07-03 16:45   ` [PATCH 03/10] Defer mount to the real mount loop Harald Hoyer
     [not found]     ` <1246639520-3094-4-git-send-email-harald-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2009-07-04 12:29       ` Seewer Philippe
     [not found]         ` <4A4F4B41.2060205-omB+W0Dpw2o@public.gmane.org>
2009-07-13 10:21           ` Harald Hoyer
2009-07-03 16:45   ` [PATCH 04/10] remove 50plymouth-pre0.7 module Harald Hoyer
2009-07-03 16:45   ` [PATCH 05/10] add firmware packages to be required by the dracut-generic package Harald Hoyer
2009-07-03 16:45   ` [PATCH 06/10] add rm to be installed for initqueue Harald Hoyer
2009-07-03 16:45   ` [PATCH 07/10] fail iscsiroot, if iscsistart fails Harald Hoyer
2009-07-03 16:45   ` [PATCH 08/10] put back the nfs mount in the udev event Harald Hoyer
     [not found]     ` <1246639520-3094-9-git-send-email-harald-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2009-07-06  8:39       ` Seewer Philippe [this message]
2009-07-03 16:45   ` [PATCH 09/10] initqueue now loops until /dev/root exists or root is mounted Harald Hoyer
     [not found]     ` <1246639520-3094-10-git-send-email-harald-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2009-07-06  8:54       ` Seewer Philippe
     [not found]         ` <4A51BBE0.6030603-omB+W0Dpw2o@public.gmane.org>
2009-07-13  9:53           ` Harald Hoyer
2009-07-03 16:45   ` [PATCH 10/10] add "rdshell" command line argument Harald Hoyer
     [not found]     ` <1246639520-3094-11-git-send-email-harald-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2009-07-06  1:53       ` Jeremy Katz
     [not found]         ` <20090706015313.GA70037-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2009-07-13 10:29           ` Harald Hoyer
2009-07-13 10:30           ` Harald Hoyer

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4A51B835.1050302@bfh.ch \
    --to=philippe.seewer-omb+w0dpw2o@public.gmane.org \
    --cc=harald-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org \
    --cc=initramfs-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.