From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-7.2 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS,USER_AGENT_SANE_1 autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 7A9C2C07E95 for ; Wed, 7 Jul 2021 15:19:39 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 6A57161CC3 for ; Wed, 7 Jul 2021 15:19:39 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232310AbhGGPWS (ORCPT ); Wed, 7 Jul 2021 11:22:18 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:51870 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232273AbhGGPWR (ORCPT ); Wed, 7 Jul 2021 11:22:17 -0400 Received: from fieldses.org (fieldses.org [IPv6:2600:3c00:e000:2f7::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 46083C061574; Wed, 7 Jul 2021 08:19:37 -0700 (PDT) Received: by fieldses.org (Postfix, from userid 2815) id C57DB581C; Wed, 7 Jul 2021 11:19:36 -0400 (EDT) DKIM-Filter: OpenDKIM Filter v2.11.0 fieldses.org C57DB581C DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=fieldses.org; s=default; t=1625671176; bh=6dAsIZ5lza0rz/s9WS3udlrYH4HxjUOA7/i1ESqKdT4=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=vY+gZbtFbrXsjt2QtKn/H3Gv+61kOYspLJ6bwHNpeK332pXhZqRFEXPjZ5XVnjiab YS7owiZJt/DEcx/8cfet87riJHlkLUu1tVZj9R2vZhPKQ2+NdcZnLkj9xO8NGXKMBb GMHFqJKEgHpOxmJ6Nu24KgOdlNVyTkSYSdqCbgjk= Date: Wed, 7 Jul 2021 11:19:36 -0400 From: "J. Bruce Fields" To: Greg KH Cc: Jeff Layton , Desmond Cheong Zhi Xi , viro@zeniv.linux.org.uk, linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org, skhan@linuxfoundation.org, linux-kernel-mentees@lists.linuxfoundation.org, syzbot+e6d5398a02c516ce5e70@syzkaller.appspotmail.com Subject: Re: [PATCH v2 1/2] fcntl: fix potential deadlocks for &fown_struct.lock Message-ID: <20210707151936.GB9911@fieldses.org> References: <20210707023548.15872-1-desmondcheongzx@gmail.com> <20210707023548.15872-2-desmondcheongzx@gmail.com> <14633c3be87286d811263892375f2dfa9a8ed40a.camel@kernel.org> <4dda1cad6348fced5fcfcb6140186795ed07d948.camel@kernel.org> <20210707135129.GA9446@fieldses.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.21 (2010-09-15) Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Jul 07, 2021 at 05:06:45PM +0200, Greg KH wrote: > On Wed, Jul 07, 2021 at 09:51:29AM -0400, J. Bruce Fields wrote: > > On Wed, Jul 07, 2021 at 07:40:47AM -0400, Jeff Layton wrote: > > > On Wed, 2021-07-07 at 12:51 +0200, Greg KH wrote: > > > > On Wed, Jul 07, 2021 at 06:44:42AM -0400, Jeff Layton wrote: > > > > > On Wed, 2021-07-07 at 08:05 +0200, Greg KH wrote: > > > > > > On Wed, Jul 07, 2021 at 10:35:47AM +0800, Desmond Cheong Zhi Xi wrote: > > > > > > > + WARN_ON_ONCE(irqs_disabled()); > > > > > > > > > > > > If this triggers, you just rebooted the box :( > > > > > > > > > > > > Please never do this, either properly handle the problem and return an > > > > > > error, or do not check for this. It is not any type of "fix" at all, > > > > > > and at most, a debugging aid while you work on the root problem. > > > > > > > > > > > > thanks, > > > > > > > > > > > > greg k-h > > > > > > > > > > Wait, what? Why would testing for irqs being disabled and throwing a > > > > > WARN_ON in that case crash the box? > > > > > > > > If panic-on-warn is enabled, which is a common setting for systems these > > > > days. > > > > > > Ok, that makes some sense. > > > > Wait, I don't get it. > > > > How are we supposed to decide when to use WARN, when to use BUG, and > > when to panic? Do we really want to treat them all as equivalent? And > > who exactly is turning on panic-on-warn? > > You never use WARN or BUG, unless the system is so messed up that you > can not possibly recover from the issue. I've heard similar advice for BUG before, but this is the first I've heard it for WARN. Do we have any guidelines for how to choose between WARN and BUG? --b. From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-5.0 required=3.0 tests=BAYES_00,DKIM_INVALID, DKIM_SIGNED,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE, SPF_PASS,URIBL_RED,USER_AGENT_SANE_1 autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 7123BC07E95 for ; Wed, 7 Jul 2021 15:19:49 +0000 (UTC) Received: from smtp2.osuosl.org (smtp2.osuosl.org [140.211.166.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id DDCED601FC for ; Wed, 7 Jul 2021 15:19:48 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org DDCED601FC Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=fieldses.org Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=linux-kernel-mentees-bounces@lists.linuxfoundation.org Received: from localhost (localhost [127.0.0.1]) by smtp2.osuosl.org (Postfix) with ESMTP id ABE3140214; Wed, 7 Jul 2021 15:19:48 +0000 (UTC) X-Virus-Scanned: amavisd-new at osuosl.org Received: from smtp2.osuosl.org ([127.0.0.1]) by localhost (smtp2.osuosl.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id o_og472-yMAE; Wed, 7 Jul 2021 15:19:45 +0000 (UTC) Received: from lists.linuxfoundation.org (lf-lists.osuosl.org [IPv6:2605:bc80:3010:104::8cd3:938]) by smtp2.osuosl.org (Postfix) with ESMTPS id E410B401CE; Wed, 7 Jul 2021 15:19:44 +0000 (UTC) Received: from lf-lists.osuosl.org (localhost [127.0.0.1]) by lists.linuxfoundation.org (Postfix) with ESMTP id C27A8C001A; Wed, 7 Jul 2021 15:19:44 +0000 (UTC) Received: from smtp1.osuosl.org (smtp1.osuosl.org [IPv6:2605:bc80:3010::138]) by lists.linuxfoundation.org (Postfix) with ESMTP id 3BF2AC000E for ; Wed, 7 Jul 2021 15:19:44 +0000 (UTC) Received: from localhost (localhost [127.0.0.1]) by smtp1.osuosl.org (Postfix) with ESMTP id 1EBF681A2B for ; Wed, 7 Jul 2021 15:19:44 +0000 (UTC) X-Virus-Scanned: amavisd-new at osuosl.org Authentication-Results: smtp1.osuosl.org (amavisd-new); dkim=pass (1024-bit key) header.d=fieldses.org Received: from smtp1.osuosl.org ([127.0.0.1]) by localhost (smtp1.osuosl.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id bMMkxOTlE9yD for ; Wed, 7 Jul 2021 15:19:39 +0000 (UTC) X-Greylist: from auto-whitelisted by SQLgrey-1.8.0 Received: from fieldses.org (fieldses.org [IPv6:2600:3c00:e000:2f7::1]) by smtp1.osuosl.org (Postfix) with ESMTPS id 1BEA681919 for ; Wed, 7 Jul 2021 15:19:38 +0000 (UTC) Received: by fieldses.org (Postfix, from userid 2815) id C57DB581C; Wed, 7 Jul 2021 11:19:36 -0400 (EDT) DKIM-Filter: OpenDKIM Filter v2.11.0 fieldses.org C57DB581C DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=fieldses.org; s=default; t=1625671176; bh=6dAsIZ5lza0rz/s9WS3udlrYH4HxjUOA7/i1ESqKdT4=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=vY+gZbtFbrXsjt2QtKn/H3Gv+61kOYspLJ6bwHNpeK332pXhZqRFEXPjZ5XVnjiab YS7owiZJt/DEcx/8cfet87riJHlkLUu1tVZj9R2vZhPKQ2+NdcZnLkj9xO8NGXKMBb GMHFqJKEgHpOxmJ6Nu24KgOdlNVyTkSYSdqCbgjk= Date: Wed, 7 Jul 2021 11:19:36 -0400 From: "J. Bruce Fields" To: Greg KH Subject: Re: [PATCH v2 1/2] fcntl: fix potential deadlocks for &fown_struct.lock Message-ID: <20210707151936.GB9911@fieldses.org> References: <20210707023548.15872-1-desmondcheongzx@gmail.com> <20210707023548.15872-2-desmondcheongzx@gmail.com> <14633c3be87286d811263892375f2dfa9a8ed40a.camel@kernel.org> <4dda1cad6348fced5fcfcb6140186795ed07d948.camel@kernel.org> <20210707135129.GA9446@fieldses.org> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.21 (2010-09-15) Cc: syzbot+e6d5398a02c516ce5e70@syzkaller.appspotmail.com, Jeff Layton , linux-kernel@vger.kernel.org, viro@zeniv.linux.org.uk, linux-fsdevel@vger.kernel.org, Desmond Cheong Zhi Xi , linux-kernel-mentees@lists.linuxfoundation.org X-BeenThere: linux-kernel-mentees@lists.linuxfoundation.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: linux-kernel-mentees-bounces@lists.linuxfoundation.org Sender: "Linux-kernel-mentees" On Wed, Jul 07, 2021 at 05:06:45PM +0200, Greg KH wrote: > On Wed, Jul 07, 2021 at 09:51:29AM -0400, J. Bruce Fields wrote: > > On Wed, Jul 07, 2021 at 07:40:47AM -0400, Jeff Layton wrote: > > > On Wed, 2021-07-07 at 12:51 +0200, Greg KH wrote: > > > > On Wed, Jul 07, 2021 at 06:44:42AM -0400, Jeff Layton wrote: > > > > > On Wed, 2021-07-07 at 08:05 +0200, Greg KH wrote: > > > > > > On Wed, Jul 07, 2021 at 10:35:47AM +0800, Desmond Cheong Zhi Xi wrote: > > > > > > > + WARN_ON_ONCE(irqs_disabled()); > > > > > > > > > > > > If this triggers, you just rebooted the box :( > > > > > > > > > > > > Please never do this, either properly handle the problem and return an > > > > > > error, or do not check for this. It is not any type of "fix" at all, > > > > > > and at most, a debugging aid while you work on the root problem. > > > > > > > > > > > > thanks, > > > > > > > > > > > > greg k-h > > > > > > > > > > Wait, what? Why would testing for irqs being disabled and throwing a > > > > > WARN_ON in that case crash the box? > > > > > > > > If panic-on-warn is enabled, which is a common setting for systems these > > > > days. > > > > > > Ok, that makes some sense. > > > > Wait, I don't get it. > > > > How are we supposed to decide when to use WARN, when to use BUG, and > > when to panic? Do we really want to treat them all as equivalent? And > > who exactly is turning on panic-on-warn? > > You never use WARN or BUG, unless the system is so messed up that you > can not possibly recover from the issue. I've heard similar advice for BUG before, but this is the first I've heard it for WARN. Do we have any guidelines for how to choose between WARN and BUG? --b. _______________________________________________ Linux-kernel-mentees mailing list Linux-kernel-mentees@lists.linuxfoundation.org https://lists.linuxfoundation.org/mailman/listinfo/linux-kernel-mentees