Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754115Ab1C1NPW (ORCPT ); Mon, 28 Mar 2011 09:15:22 -0400 Received: from rcsinet10.oracle.com ([148.87.113.121]:30525 "EHLO rcsinet10.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753155Ab1C1NPS convert rfc822-to-8bit (ORCPT ); Mon, 28 Mar 2011 09:15:18 -0400 Subject: Re: [PATCH] NFS: Revert NFSROOT default mount options Mime-Version: 1.0 (Apple Message framework v1084) Content-Type: text/plain; charset=us-ascii From: Chuck Lever In-Reply-To: Date: Mon, 28 Mar 2011 09:15:12 -0400 Cc: linux-nfs@vger.kernel.org, linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8BIT Message-Id: References: <20110325141947.3236.83222.stgit@matisse.1015granger.net> To: Belisko Marek X-Mailer: Apple Mail (2.1084) X-Source-IP: acsmt358.oracle.com [141.146.40.158] X-Auth-Type: Internal IP X-CT-RefId: str=0001.0A090209.4D9089E3.01FB,ss=1,fgs=0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3436 Lines: 105 On Mar 28, 2011, at 1:09 AM, Belisko Marek wrote: > On Fri, Mar 25, 2011 at 3:23 PM, Chuck Lever wrote: >> Marek Belisko reports that recent attempts >> to fix regressions in NFSROOT have broken his configuration: >> >>> After update from 2.6.38-rc8 to 2.6.38 is mounting rootfs over nfs not possible. >>> Log: >>> VFS: Mounted root (nfs filesystem) on device 0:14. >>> Freeing init memory: 132K >>> nfs: server 10.146.1.21 not responding, still trying >>> nfs: server 10.146.1.21 not responding, still trying >>> >>> This is never ending. I make short bisect (not too much commits >>> between versions) >>> and bad commit was reported: 53d4737580535e073963b91ce87d4216e434fab5 >>> >>> NFS: NFSROOT should default to "proto=udp" >>> >>> I've tested on mini2440 board (DM9000, static IP). >>> Is there some missing option or something else to be checked? >> >> An examination of a network trace captured during the failure shows >> that the mount is actually succeeding, but that the client is not >> seeing READ replies larger than 16KB. >> >> Last fall we removed the ad hoc mount option parser in fs/nfs/nfsroot.c >> in favor of using the main parser in fs/nfs/super.c (see commit >> 56463e50 "NFS: Use super.c for NFSROOT mount option parsing"). That >> commit changed the default NFSROOT mount options to be the same as >> those employed by user space mounts. As it turns out, that change is >> not tolerated by many embedded systems. >> >> Subsequently, commit 53d47375 introduced a clean way to specify >> default mount options for NFSROOT, so we can restore the traditional >> defaults for NFSROOT easily now: >> >> vers=2,udp,rsize=4096,wsize=4096 >> >> This should revert the changed default NFSROOT mount options >> introduced with commit 56463e50. > Work fine with below patch. > You can add my Tested by: Marek Belisko Will do, thanks for your testing. >> >> Signed-off-by: Chuck Lever >> --- >> >> Hi- >> >> Please test this, as I no longer have an NFSROOT rig here. >> >> Can some responsible maintainer type see that this gets into >> linux-next quickly for broader testing? >> >> fs/nfs/nfsroot.c | 2 +- >> 1 files changed, 1 insertions(+), 1 deletions(-) >> >> diff --git a/fs/nfs/nfsroot.c b/fs/nfs/nfsroot.c >> index c541093..c4744e1 100644 >> --- a/fs/nfs/nfsroot.c >> +++ b/fs/nfs/nfsroot.c >> @@ -87,7 +87,7 @@ >> #define NFS_ROOT "/tftpboot/%s" >> >> /* Default NFSROOT mount options. */ >> -#define NFS_DEF_OPTIONS "udp" >> +#define NFS_DEF_OPTIONS "vers=2,udp,rsize=4096,wsize=4096" >> >> /* Parameters passed from the kernel command line */ >> static char nfs_root_parms[256] __initdata = ""; >> >> > > regards, > > marek > > -- > as simple and primitive as possible > ------------------------------------------------- > Marek Belisko - OPEN-NANDRA > Freelance Developer > > Ruska Nova Ves 219 | Presov, 08005 Slovak Republic > Tel: +421 915 052 184 > skype: marekwhite > icq: 290551086 > web: http://open-nandra.com -- Chuck Lever chuck[dot]lever[at]oracle[dot]com -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/