From: Shiraz Hashim <shiraz.hashim@st.com>
To: Chuck Lever <chuck.lever@oracle.com>
Cc: Brian Downing <bdowning@lavos.net>,
Deepak SIKRI <deepak.sikri@st.com>,
Armando VISCONTI <armando.visconti@st.com>,
Trond Myklebust <Trond.Myklebust@netapp.com>,
"netdev@vger.kernel.org" <netdev@vger.kernel.org>,
Linux NFS Mailing List <linux-nfs@vger.kernel.org>,
Viresh KUMAR <viresh.kumar@st.com>,
Peppe CAVALLARO <peppe.cavallaro@st.com>,
amitgoel <amit.goel@st.com>
Subject: Re: STMMAC driver: NFS Problem on 2.6.37
Date: Thu, 24 Feb 2011 19:06:27 +0530 [thread overview]
Message-ID: <20110224133627.GO920@DLHLAP0379> (raw)
In-Reply-To: <5CC01953-376D-46FC-99DA-C282332FA40F@oracle.com>
On Thu, Feb 10, 2011 at 05:26:16AM +0800, Chuck Lever wrote:
>
> On Feb 9, 2011, at 3:58 PM, Brian Downing wrote:
>
> > On Wed, Feb 09, 2011 at 03:12:22PM -0500, Chuck Lever wrote:
> >> Based on your console logs, I see that the working case uses UDP to
> >> contact the server's mountd, but the failing case uses TCP. You can
> >> try explicitly specifying "proto=udp" to force the use of UDP, to test
> >> this theory.
> >
> > This does indeed make it work again for me, thanks!
> >
> >> Meanwhile, the patch description explicitly states that the default
> >> mount option settings have changed. Does it make sense to change the
> >> default behavior of NFSROOT mounts to use UDP again? I don't see
> >> another way to make this process more reliable across NIC
> >> initialization. If this is considered a regression, we can make a
> >> patch for 2.6.38-rc and 2.6.37.
> >
> > I only use nfsroot for development, so I don't have a terribly strong
> > opinion. I would point out though that the default u-boot parameters
> > for nfsrooting a lot of boards will no longer work at this point, so if
> > it's not patched to work again without specifying nfs options I think
> > there should at least be a note in the documentation and possibly a
> > "maybe try proto=udp?" console message on failure.
> >
> > I assume it's not feasable to either wait until the chosen interface's
> > link is ready before trying to mount nfsroot, or retrying TCP-based
> > connections a little bit more aggressively/at all?
>
> Our goal is to use the same mount logic for both normal user
> space mounts and for NFSROOT (that was the purpose of the patch
> series this particular patch comes from). It's
> exceptionally difficult to add a special case for retrying TCP
> connections here, as that would change the behavior of user
> space mounts, which often want to fail quickly, and don't need
> to worry about NIC initialization.
>
> Sounds like the right thing to do is restore the default UDP behavior. I'll cook up a patch.
Is there some patch available for this now.
There is one more observation (on 2.6.37), when I pass
nfsroot=$(ip):$(rootpath),udp , then it works fine.
If I pass proto=udp then it doesn't work. Is there any difference
between the two methods ?
--
regards
Shiraz
next prev parent reply other threads:[~2011-02-24 14:11 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-01-13 9:09 STMMAC driver: NFS Problem on 2.6.37 deepaksi
2011-01-13 9:09 ` deepaksi
2011-01-13 11:48 ` Peppe CAVALLARO
2011-01-13 11:48 ` Peppe CAVALLARO
2011-01-13 15:07 ` Chuck Lever
2011-01-13 15:07 ` Chuck Lever
2011-01-13 18:28 ` Armando Visconti
2011-01-13 18:28 ` Armando Visconti
2011-01-14 9:56 ` deepaksi
2011-01-14 15:35 ` Chuck Lever
2011-01-14 15:35 ` Chuck Lever
[not found] ` <4D3EBA54.4020308@st.com>
2011-01-25 18:04 ` Chuck Lever
2011-01-25 18:04 ` Chuck Lever
2011-01-28 12:43 ` Shiraz Hashim
2011-01-28 12:43 ` Shiraz Hashim
2011-01-28 16:58 ` Chuck Lever
2011-02-09 20:01 ` Brian Downing
2011-02-09 20:12 ` Chuck Lever
2011-02-09 20:12 ` Chuck Lever
2011-02-09 20:58 ` Brian Downing
2011-02-09 21:26 ` Chuck Lever
2011-02-09 21:26 ` Chuck Lever
2011-02-24 13:36 ` Shiraz Hashim [this message]
2011-02-24 18:33 ` Chuck Lever
2011-02-24 18:33 ` Chuck Lever
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=20110224133627.GO920@DLHLAP0379 \
--to=shiraz.hashim@st.com \
--cc=Trond.Myklebust@netapp.com \
--cc=amit.goel@st.com \
--cc=armando.visconti@st.com \
--cc=bdowning@lavos.net \
--cc=chuck.lever@oracle.com \
--cc=deepak.sikri@st.com \
--cc=linux-nfs@vger.kernel.org \
--cc=netdev@vger.kernel.org \
--cc=peppe.cavallaro@st.com \
--cc=viresh.kumar@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.