Received: by 2002:a05:6a10:d5a5:0:0:0:0 with SMTP id gn37csp420650pxb; Wed, 6 Oct 2021 07:47:13 -0700 (PDT) X-Google-Smtp-Source: ABdhPJwKK3FvdUq9S+E4ph3Moh0EgQEiQQpMXhaZKACZeshdLdorHXoq1GSd8yVdNudug5s/T4+Q X-Received: by 2002:aa7:dd90:: with SMTP id g16mr32193413edv.199.1633531633759; Wed, 06 Oct 2021 07:47:13 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1633531633; cv=none; d=google.com; s=arc-20160816; b=Km0pVoQiSnpZLgqfm8TnjlchZYkFBHzMOzlr6rauz2CmPmqCmSAhOu3jHTA2+oNPl3 ObFBTBSf3trDCMf1Mmm2TXKT1jQYKW7tPzwBur87U4GzWyY89I6bXGBQpjz+YxLc5+kr 6fpR5iDw8r2R3ks3gCmvoHpJcCTysF3o7oBUCvvt/iRDi96FuzXRwrUe/Jk9TyvrpxaD QeScLk9Y1qiIJOLsBb6iJr63rGFAElDxNi5X6M2RwI1+6Caz1r0dN2cPYBXol3Yp/L5F fAyj5lXO3AIatwqZuw6+WPS4EnZJDgqozeZ+rPkHtOicM2u6LDXvv1EhpV4jzULKL2b+ jlaQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:user-agent:in-reply-to:content-disposition :mime-version:references:message-id:subject:cc:to:from:date :dkim-signature:dkim-filter; bh=elalU8AnPHAif8h0i3/TwTZFw8RzuZYbaHtXhUET0Vg=; b=eMjm/TDbNB3E3fDqo8n3H+OQW71OJAjKTy/i6sw/btFd6vrTw07voAlROklE9V5fZr zHGGkoXKuMg/9JfKkBc4Z/E7Xls4k+uVAebycRNIxU4VaRQwLGhWnHfOsv3+zE0/TLI0 qAaDVFQjDgrAlxOMfcLsUnKPqowHB8qoM/FNHKN3h8EC0d6pynHMuxvOu3MmDgscInsv pxHIEp+knTf8ldoae+b9SroBh9dyGoOgwe58noFuts8Y5oprgx1jbrH82ohMQ6T9LYnL ALhpRciohvySGETDJEfNMNXIf2Ca0ITR2JtfX0epUfnP9etkLkqNXkgr5kDhEvnniu6v bGqA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@fieldses.org header.s=default header.b=GsZil5tC; spf=pass (google.com: domain of linux-nfs-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-nfs-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id 30si26498931ejl.658.2021.10.06.07.46.49; Wed, 06 Oct 2021 07:47:13 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-nfs-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@fieldses.org header.s=default header.b=GsZil5tC; spf=pass (google.com: domain of linux-nfs-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-nfs-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S238226AbhJFOsi (ORCPT + 99 others); Wed, 6 Oct 2021 10:48:38 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:41678 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231403AbhJFOsi (ORCPT ); Wed, 6 Oct 2021 10:48:38 -0400 Received: from fieldses.org (fieldses.org [IPv6:2600:3c00:e000:2f7::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id DF181C061749 for ; Wed, 6 Oct 2021 07:46:45 -0700 (PDT) Received: by fieldses.org (Postfix, from userid 2815) id 4BE02703F; Wed, 6 Oct 2021 10:46:45 -0400 (EDT) DKIM-Filter: OpenDKIM Filter v2.11.0 fieldses.org 4BE02703F DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=fieldses.org; s=default; t=1633531605; bh=elalU8AnPHAif8h0i3/TwTZFw8RzuZYbaHtXhUET0Vg=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=GsZil5tCj9DtsV9oz8cCvrznJEbG2zS0ZwBmSnsbwD9D8qqYbTpci9IL+sB3Pr4ia 2HSb1oJpgiduPfhcG7hgcgMjWDwEDsyUtqc51QMwsJVEGvJQxewMFC7hP/TO2vD0cU kaBB/8uHKKxa7dq4qh+S2HlIt/LN0mBZwDTwf2I8= Date: Wed, 6 Oct 2021 10:46:45 -0400 From: "J. Bruce Fields" To: Benjamin Coddington Cc: chuck.lever@oracle.com, linux-nfs@vger.kernel.org Subject: Re: [PATCH] NFSD: Keep existing listners on portlist error Message-ID: <20211006144645.GB15343@fieldses.org> References: <45b916f1aa3fb7c059a574f61188a8f2f615410e.1633529847.git.bcodding@redhat.com> <20211006143335.GA15343@fieldses.org> <053E93B5-0DFB-4A20-9742-F3894E2BE224@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <053E93B5-0DFB-4A20-9742-F3894E2BE224@redhat.com> User-Agent: Mutt/1.5.21 (2010-09-15) Precedence: bulk List-ID: X-Mailing-List: linux-nfs@vger.kernel.org On Wed, Oct 06, 2021 at 10:43:14AM -0400, Benjamin Coddington wrote: > On 6 Oct 2021, at 10:33, J. Bruce Fields wrote: > > >On Wed, Oct 06, 2021 at 10:18:05AM -0400, Benjamin Coddington wrote: > >>If nfsd has existing listening sockets without any processes, > >>then an error > >>returned from svc_create_xprt() for an additional transport will > >>remove > >>those existing listeners. We're seeing this in practice when > >>userspace > >>attempts to create rpcrdma transports without having the rpcrdma > >>modules > >>present before creating nfsd kernel processes. Fix this by > >>checking for > >>existing sockets before callingn nfsd_destroy(). > > > >That seems like an improvement. > > > >I'm curious, though, what the rpc.nfsd behavior is on partial failure. > >And what do we want it to be? > > > >If a user runs rpc.nfsd expecting it to start up tcp and rdma, but > >rdma > >fails, do we want rpc.nfsd to succeed or fail? Should it exit > >with nfsd > >running or not? > > I lean toward having it fail - but I think that's a different patch for > rpc.nfsd. Right now rpc.nfsd exists without error, but you end up > without > any listeners at all. > > Do you want a patch for rpc.nfsd instead? I think we need the kernel fix. I agree that it's weird to shut down the server on a failure to add a listener--better to leave usrespace to decide if it wants to do that. --b.