From mboxrd@z Thu Jan 1 00:00:00 1970 From: "J. Bruce Fields" Subject: Re: [PATCH 1/1] silence-call-timeout-printk Date: Thu, 27 Mar 2008 16:23:31 -0400 Message-ID: <20080327202331.GJ32540@fieldses.org> References: <1206643752-11132-1-git-send-email-aglo@umich.edu> <20080327195445.GI32540@fieldses.org> <1206648229.15396.2.camel@heimdal.trondhjem.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Olga Kornievskaia , linux-nfs@vger.kernel.org To: Trond Myklebust Return-path: Received: from mail.fieldses.org ([66.93.2.214]:57695 "EHLO fieldses.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756991AbYC0UXf (ORCPT ); Thu, 27 Mar 2008 16:23:35 -0400 In-Reply-To: <1206648229.15396.2.camel-rJ7iovZKK19ZJLDQqaL3InhyD016LWXt@public.gmane.org> Sender: linux-nfs-owner@vger.kernel.org List-ID: On Thu, Mar 27, 2008 at 04:03:49PM -0400, Trond Myklebust wrote: > On Thu, 2008-03-27 at 15:54 -0400, J. Bruce Fields wrote: > > On Thu, Mar 27, 2008 at 02:49:12PM -0400, Olga Kornievskaia wrote: > > > When the client's callback server goes away, the server's callback client > > > tries to contact the server and times out. For nfsd, it is beneficial to > > > printout a message when the client is unable to contact the server. For > > > the callback server, the same message is printed yet it is really not > > > an error. Thus we need a way to silence the message for the callback > > > and yet print it for other cases. > > Bring, back, > Bring, back, > Oh, bring back my cl_chatty to me... > > http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commitdiff;h=f518e35aec984036903c1003e867f833747a9d79 OK, OK, so, shall we apply a patch that basically just reverts Chuck's patch there, except that it inverts the meaning and uses the name cl_quiet instead of cl_chatty? (On the theory that since the verbose case is the common case, it's best to let this thing be initialized to 0 and only make the callback code mess with it?) Or are you nostalgic for cl_chatty? --b.