From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from out1-smtp.messagingengine.com ([66.111.4.25]:33473 "EHLO out1-smtp.messagingengine.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753214AbdDLNAb (ORCPT ); Wed, 12 Apr 2017 09:00:31 -0400 Date: Wed, 12 Apr 2017 15:00:21 +0200 From: Greg KH To: Martin Brandenburg Cc: hubcap@omnibond.com, stable@vger.kernel.org, linux-fsdevel@vger.kernel.org Subject: Re: [PATCH 2/3] orangefs: Dan Carpenter influenced cleanups... Message-ID: <20170412130021.GA10858@kroah.com> References: <20170411164128.8401-1-martin@omnibond.com> <20170411164128.8401-3-martin@omnibond.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20170411164128.8401-3-martin@omnibond.com> Sender: stable-owner@vger.kernel.org List-ID: On Tue, Apr 11, 2017 at 12:41:27PM -0400, Martin Brandenburg wrote: > From: Mike Marshall > > commit 05973c2efb40122f2a9ecde2d065f7ea5068d024 upstream. > > This patch is simlar to one Dan Carpenter sent me, cleans > up some return codes and whitespace errors. There was one > place where he thought inserting an error message into > the ring buffer might be too chatty, I hope I convinced him > othewise. As a consolation I changed a truly chatty > error message in another location into a debug message, > system-admins had already yelled at me about that one... > > Signed-off-by: Mike Marshall > > I have removed the return code and whitespace fixes as they do not cause > a real bug. I keep the removal of an error message. This error shows > up when a process dies before the OrangeFS userspace client responds. > This happens all the time on production systems and fills up the ring > buffer. No, please keep patches identical to what is in Linus's tree. That way you don't mess something up, and any future patches over the next 2+ years the kernel tree will be around apply cleanly. Almost every single time we "modify" a patch from what is in Linus's tree it is broken. I've taken the original patch here for 4.9 and 4.10 stable trees. thanks, greg k-h