From owner-svn-src-all@FreeBSD.ORG Fri May 22 10:34:01 2009 Return-Path: Delivered-To: svn-src-all@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 6EE50106566B; Fri, 22 May 2009 10:34:01 +0000 (UTC) (envelope-from des@des.no) Received: from tim.des.no (tim.des.no [194.63.250.121]) by mx1.freebsd.org (Postfix) with ESMTP id 2CF168FC1B; Fri, 22 May 2009 10:34:00 +0000 (UTC) (envelope-from des@des.no) Received: from ds4.des.no (cm-84.215.252.34.getinternet.no [84.215.252.34]) by smtp.des.no (Postfix) with ESMTP id CFE486D41E; Fri, 22 May 2009 12:33:59 +0200 (CEST) Received: by ds4.des.no (Postfix, from userid 1001) id C746A844BF; Fri, 22 May 2009 12:33:59 +0200 (CEST) From: =?utf-8?Q?Dag-Erling_Sm=C3=B8rgrav?= To: Rick Macklem References: <200905201858.n4KIw7Fc040619@svn.freebsd.org> Date: Fri, 22 May 2009 12:33:59 +0200 In-Reply-To: <200905201858.n4KIw7Fc040619@svn.freebsd.org> (Rick Macklem's message of "Wed, 20 May 2009 18:58:07 +0000 (UTC)") Message-ID: <86r5yhzaso.fsf@ds4.des.no> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/23.0.92 (berkeley-unix) MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org Subject: Re: svn commit: r192463 - head/sys/fs/nfsserver X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: "SVN commit messages for the entire src tree \(except for " user" and " projects" \)" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 22 May 2009 10:34:01 -0000 Rick Macklem writes: > Log: > Although it should never happen, all the nfsv4 server can do > when it runs out of clientids is reboot. I had replaced cpu_reboot() > with printf(), since cpu_reboot() doesn't exist for sparc64. > This change replaces the printf() with panic(), so the reboot > would occur for this highly unlikely occurrence. Regardless of how improbable this is, wouldn't it be better (and simpler) to just log an error message and deny further mount requests? DES --=20 Dag-Erling Sm=C3=B8rgrav - des@des.no