All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jim Rees <rees@umich.edu>
To: Srinivas KANDAGATLA <srinivas.kandagatla@st.com>
Cc: Chuck Lever <chuck.lever@oracle.com>,
	linux-nfs@vger.kernel.org, akpm@linux-foundation.org,
	mingo@redhat.com, neilb@suse.de, stuart.menefy@st.com,
	bharrosh@panasas.com, linux@razik.name
Subject: Re: [PATCH 3.3.0-rc1 1/2] do_mounts: Change the nfs-mount retry min max delays.
Date: Fri, 3 Feb 2012 08:16:36 -0500	[thread overview]
Message-ID: <20120203131636.GA8220@umich.edu> (raw)
In-Reply-To: <4F2BB2C0.9060207@st.com>

It seems confusing to change the meaning of NFSROOT_TIMEOUT_MIN.  It used to
be the timeout minimum, as the name implies, but now it's the timeout
minimum plus one.  I understand that the actual timeout includes the time
spent in mount, but I would expect NFSROOT_TIMEOUT_MIN to just be the
initial loop sleep time.  Maybe this just needs a comment.

      reply	other threads:[~2012-02-03 13:16 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-01-26 14:42 [PATCH 3.3.0-rc1 1/2] do_mounts: Change the nfs-mount retry min max delays Srinivas KANDAGATLA
2012-01-26 14:55 ` Jim Rees
2012-01-26 15:06   ` Srinivas KANDAGATLA
2012-01-26 15:35 ` Chuck Lever
2012-01-27  8:10   ` Srinivas KANDAGATLA
2012-01-27 12:34     ` Jim Rees
2012-01-30  7:46       ` Srinivas KANDAGATLA
2012-02-03 10:11     ` Srinivas KANDAGATLA
2012-02-03 13:16       ` Jim Rees [this message]

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=20120203131636.GA8220@umich.edu \
    --to=rees@umich.edu \
    --cc=akpm@linux-foundation.org \
    --cc=bharrosh@panasas.com \
    --cc=chuck.lever@oracle.com \
    --cc=linux-nfs@vger.kernel.org \
    --cc=linux@razik.name \
    --cc=mingo@redhat.com \
    --cc=neilb@suse.de \
    --cc=srinivas.kandagatla@st.com \
    --cc=stuart.menefy@st.com \
    /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.