From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754739AbdAaBCg (ORCPT ); Mon, 30 Jan 2017 20:02:36 -0500 Received: from casper.infradead.org ([85.118.1.10]:55730 "EHLO casper.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751239AbdAaBCb (ORCPT ); Mon, 30 Jan 2017 20:02:31 -0500 Date: Tue, 31 Jan 2017 01:00:59 +0000 (GMT) From: James Simmons To: e@mwanda cc: Greg Kroah-Hartman , devel@driverdev.osuosl.org, Andreas Dilger , Oleg Drokin , Bruno Faccini , Linux Kernel Mailing List , Lustre Development List Subject: Re: [PATCH 13/60] staging: lustre: obdclass: health_check to report unhealthy upon LBUG In-Reply-To: <20170130120352.GC6881@mwanda> Message-ID: References: <1485648328-2141-1-git-send-email-jsimmons@infradead.org> <1485648328-2141-14-git-send-email-jsimmons@infradead.org> <20170130120352.GC6881@mwanda> User-Agent: Alpine 2.20 (LFD 67 2015-01-07) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20170131_010100_058407_F1E36905 X-CRM114-Status: GOOD ( 19.92 ) X-Spam-Score: 0.2 (/) X-Spam-Report: SpamAssassin version 3.4.1 on casper.infradead.org summary: Content analysis details: (0.2 points, 5.0 required) pts rule name description ---- ---------------------- -------------------------------------------------- 2.1 TO_MALFORMED To: has a malformed address -0.0 NO_RELAYS Informational: message was not relayed via SMTP -1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1% [score: 0.0000] Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org > Wat? > > I'm sorry but this patch makes no sense at all. > > On Sat, Jan 28, 2017 at 07:04:41PM -0500, James Simmons wrote: > > From: Bruno Faccini > > > > When a LBUG has occurred, without panic_on_lbug being set, > > health_check sysfs file must return an unhealthy state. > > Why? Its a patch being applied out of order issue. Originally this patch was written before this was moved to sysfs and the original code didn't return right after printing "LBUG". The move to sysfs changed this behavior. I will fix up the out of tree code in this case. > > > > Signed-off-by: Bruno Faccini > > Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-7486 > > Reviewed-on: http://review.whamcloud.com/17981 > > Reviewed-by: Bobi Jam > > Reviewed-by: Niu Yawei > > Reviewed-by: James Simmons > > Reviewed-by: Oleg Drokin > > Signed-off-by: James Simmons > > --- > > drivers/staging/lustre/lustre/obdclass/linux/linux-module.c | 6 ++++-- > > 1 file changed, 4 insertions(+), 2 deletions(-) > > > > diff --git a/drivers/staging/lustre/lustre/obdclass/linux/linux-module.c b/drivers/staging/lustre/lustre/obdclass/linux/linux-module.c > > index 22e6d1f..ef25db6 100644 > > --- a/drivers/staging/lustre/lustre/obdclass/linux/linux-module.c > > +++ b/drivers/staging/lustre/lustre/obdclass/linux/linux-module.c > > @@ -224,8 +224,10 @@ static ssize_t pinger_show(struct kobject *kobj, struct attribute *attr, > > int i; > > size_t len = 0; > > > > - if (libcfs_catastrophe) > > - return sprintf(buf, "LBUG\n"); > > + if (libcfs_catastrophe) { > > + len = sprintf(buf, "LBUG\n"); > > This line is dead code, now. > > > + healthy = false; > > + } > > > > regards, > dan carpenter > >