From patchwork Thu Jun 18 14:20:40 2015 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Steve Dickson X-Patchwork-Id: 6638251 Return-Path: X-Original-To: patchwork-linux-nfs@patchwork.kernel.org Delivered-To: patchwork-parsemail@patchwork2.web.kernel.org Received: from mail.kernel.org (mail.kernel.org [198.145.29.136]) by patchwork2.web.kernel.org (Postfix) with ESMTP id 60C34C0433 for ; Thu, 18 Jun 2015 14:20:50 +0000 (UTC) Received: from mail.kernel.org (localhost [127.0.0.1]) by mail.kernel.org (Postfix) with ESMTP id D098D207D1 for ; Thu, 18 Jun 2015 14:20:44 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 78921206D8 for ; Thu, 18 Jun 2015 14:20:43 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753976AbbFROUm (ORCPT ); Thu, 18 Jun 2015 10:20:42 -0400 Received: from mx1.redhat.com ([209.132.183.28]:60215 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753148AbbFROUl (ORCPT ); Thu, 18 Jun 2015 10:20:41 -0400 Received: from int-mx09.intmail.prod.int.phx2.redhat.com (int-mx09.intmail.prod.int.phx2.redhat.com [10.5.11.22]) by mx1.redhat.com (Postfix) with ESMTPS id A68CA38AAE6; Thu, 18 Jun 2015 14:20:41 +0000 (UTC) Received: from smallhat.boston.devel.redhat.com (smallhat.boston.devel.redhat.com [10.19.60.65]) by int-mx09.intmail.prod.int.phx2.redhat.com (8.14.4/8.14.4) with ESMTP id t5IEKePI031696; Thu, 18 Jun 2015 10:20:40 -0400 Message-ID: <5582D3B8.7090409@RedHat.com> Date: Thu, 18 Jun 2015 10:20:40 -0400 From: Steve Dickson User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.7.0 MIME-Version: 1.0 To: Trond Myklebust , Anders Blomdell CC: Linux NFS Mailing List Subject: Re: Mount regression between 4.0.4 client and 2.6.35 server References: <55827B8D.3070103@control.lth.se> <5582B960.7000301@control.lth.se> In-Reply-To: X-Scanned-By: MIMEDefang 2.68 on 10.5.11.22 Sender: linux-nfs-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-nfs@vger.kernel.org X-Spam-Status: No, score=-7.2 required=5.0 tests=BAYES_00, RCVD_IN_DNSWL_HI, RP_MATCHES_RCVD, UNPARSEABLE_RELAY autolearn=unavailable version=3.3.1 X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on mail.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP Hello, BTW, I asked Anders to post this patch to get this discussion going. On 06/18/2015 08:53 AM, Trond Myklebust wrote: > On Thu, Jun 18, 2015 at 8:28 AM, Anders Blomdell > wrote: >> On 2015-06-18 13:49, Trond Myklebust wrote: >>> On Thu, Jun 18, 2015 at 4:04 AM, Anders Blomdell >>> wrote: >>>> >>>> I have a problem with a 4.0.4 client refusing to mount from a 2.6.35 server >>>> due to NFS4ERR_INVAL returned during nfs4_discover_server_trunking. See >>>> https://bugzilla.redhat.com/show_bug.cgi?id=1228272. >>> >>> >>> Why should we change the clients if the server is in clear and obvious >>> violation of the spec? >> In order to make clients work with servers that worked well with previous versions >> of nfs-utils, the cultprit probably being commit f9802988 that bumped the default >> autonegotion version to 4.2, what the patch does is only to negotiate a lower version >> in case of errors, and hence making 1.3.2 working with servers that worked with >> 1.3.1 (that only tried version 4[.0]). >> >> Will probably save some people some time. > > This is what /etc/nfsmount.conf is for. We don't fix clients that are > working correctly according to the protocol spec. > Using /etc/nfsmount.conf does not scale very well when there is a large number clients involved... When we bumped up the auto-negotiation to 4.2, I thinking there would be issues with legacy servers. But I was thinking obscure servers like AIX would break, not old Linux servers and possibly Solaris servers (I'm trying to check that now) Inlining the patch: Now I'm not sure this is the right to handle this but I do think it is a good idea to make the auto-negation mounting code smarter to handle these legacy servers. steved. --- To unsubscribe from this list: send the line "unsubscribe linux-nfs" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html --- utils/mount/stropts.c.orig 2015-06-18 09:51:02.091148891 +0200 +++ utils/mount/stropts.c 2015-06-18 09:48:56.859970023 +0200 @@ -838,6 +838,10 @@ check_result: return result; switch (errno) { + case EIO: + /* Fix to handle nfs4_discover_server_trunking returning + * EIO in case where nfs server returns NFS4ERR_INVAL, + * see https://bugzilla.redhat.com/show_bug.cgi?id=1228272 */ case EPROTONOSUPPORT: /* A clear indication that the server or our * client does not support NFS version 4 and minor */