Received: by 2002:a25:1985:0:0:0:0:0 with SMTP id 127csp493860ybz; Wed, 29 Apr 2020 04:10:22 -0700 (PDT) X-Google-Smtp-Source: APiQypJTeKM0+/nXsUTxilWNSg7PSd7ChG3cRG0QmF6AqdFKcE9kOC0PbWd7PBBqnDN5+3Ruavzr X-Received: by 2002:a17:906:1dce:: with SMTP id v14mr2147965ejh.244.1588158621988; Wed, 29 Apr 2020 04:10:21 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1588158621; cv=none; d=google.com; s=arc-20160816; b=is3S+vvaU1ZfV5OjYqnMuR9+Ze6Mq2HlG2Qzw2upmCSc/aoWbPRLwFFSZ6bC/0COzJ uQlyMH6z1v78N5Wr/GtrmxuiVsZBr/OnB4pd/3rUQFzgtahtS01zxd0r8hU5MDe0Y7UO l3o1m2pQffxbePAg+EcJkD3T9x3E0YHwo2i6bdVDdQH7Xx73ObQ2lSZO8gvcbxDmsNjY wS4NuSLgh30CdabMnbzNx88Q/ZCyFkSgznaYw2vH7U/PmDcD50Wyw1WQQpvYWQnsgX/w n6RrrcHtT7h3MryV7dAbisJjT3/P1r7gFxYKVUbgNJ41WGiz11TubcVrhotdEa6DYZmX 0Axg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:in-reply-to:content-disposition :mime-version:references:message-id:subject:cc:to:from:date :dkim-signature; bh=azkwK7kCyzu9TJzT40ryl2Bq1YKytVIxT2fz6tK4FSQ=; b=WEKC4YveHUcUbO2GtUhryW2mS25BdB36d51LpkX1cDdzQmBTNmN7KMYuisqRMdUIZn 1guXdmMsHMXZZs9SI2CXXHSxfrDO44TjUO6lx7evfSD5bU+vgO8RWPS+WNfvsGtH3B5P xBrAfLY2S1HCMu3clBgmcBHItlt0V0oB6Xjv9ECwPLluglwQtMdOA0UVPzexSY6AWLkO XX8gJH5378OE59jj+Y9Bx434OB8TcLhKoQ+uHW/zu5YKJArgF5re1YZC98uH3RLxzraC Bpq14kKpJxR5Tl22Qwbn2Q2185wMJgiAMyu76j0lGQIsXbKr7m6f3Y97izInOt65WHTn CHYw== ARC-Authentication-Results: i=1; mx.google.com; dkim=fail header.i=@infradead.org header.s=bombadil.20170209 header.b=iQruzPw7; spf=pass (google.com: domain of linux-ext4-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-ext4-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id p24si3521618eju.406.2020.04.29.04.09.57; Wed, 29 Apr 2020 04:10:21 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-ext4-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=fail header.i=@infradead.org header.s=bombadil.20170209 header.b=iQruzPw7; spf=pass (google.com: domain of linux-ext4-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-ext4-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726669AbgD2LJy (ORCPT + 99 others); Wed, 29 Apr 2020 07:09:54 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:58192 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-FAIL-OK-FAIL) by vger.kernel.org with ESMTP id S1726524AbgD2LJx (ORCPT ); Wed, 29 Apr 2020 07:09:53 -0400 Received: from bombadil.infradead.org (bombadil.infradead.org [IPv6:2607:7c80:54:e::133]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id DA0A0C03C1AD for ; Wed, 29 Apr 2020 04:09:53 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=infradead.org; s=bombadil.20170209; h=In-Reply-To:Content-Type:MIME-Version :References:Message-ID:Subject:Cc:To:From:Date:Sender:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description; bh=azkwK7kCyzu9TJzT40ryl2Bq1YKytVIxT2fz6tK4FSQ=; b=iQruzPw7Gr5yYsSt+fpH0L5ei8 uQQrBpjQfk+OxabFAwwQ0f0jGDcfHEHpLJg8NbfsRvVK+IXQLUR8nE2jQdJsOrdKk99fmQU2u7r87 d/LwUrqTKS8O+VtmCZWRw7L1ciRVv6GP9cHb5V94aIaFIvr7+2qgvH2HSQQ1z+5eK+utdCu6lwJrj p8lipLzrZyHt/2wzb1EQkSeC1TNTyOhZulkMYIAV/Z5KjjsivxzaPjHcnZQZHUYmeCF0wpLZV9KwR YpR6tT1Ynq+tVLGAzUxx2s/GEW1BVndVL1FkHABMYS7tjhw/QUw49XvbLaKsejNE7yMOvNA8wdAYD tcuVSXOw==; Received: from hch by bombadil.infradead.org with local (Exim 4.92.3 #3 (Red Hat Linux)) id 1jTkb5-0002Rp-Rr; Wed, 29 Apr 2020 11:09:47 +0000 Date: Wed, 29 Apr 2020 04:09:47 -0700 From: Christoph Hellwig To: Ian Kent Cc: Lukas Czerner , Christoph Hellwig , linux-ext4@vger.kernel.org, dhowells@redhat.com, viro@zeniv.linux.org.uk Subject: Re: [PATCH v2 05/17] ext4: Allow sb to be NULL in ext4_msg() Message-ID: <20200429110947.GA6290@infradead.org> References: <20200428164536.462-1-lczerner@redhat.com> <20200428164536.462-6-lczerner@redhat.com> <20200428164808.GA3632@infradead.org> <20200428165747.ondq7nbn4ol3j3lu@work> <4f6598e12127cc6e80af7dde5149220d92b07b11.camel@themaw.net> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <4f6598e12127cc6e80af7dde5149220d92b07b11.camel@themaw.net> X-SRS-Rewrite: SMTP reverse-path rewritten from by bombadil.infradead.org. See http://www.infradead.org/rpr.html Sender: linux-ext4-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-ext4@vger.kernel.org On Wed, Apr 29, 2020 at 10:53:00AM +0800, Ian Kent wrote: > The mount-API log macros tend to cause user confusion because they > often lead to unexpected log messages. > > We're seeing that now with bugs logged due to unexpected messages > resulting from the NFS mount-API conversion. > > I'd recommend mostly avoiding using the macros until there's been > time to reconsider how they should work, after all fsopen() and > friends will still get errno errors just not the passed string > description. And when is that time going to be? Should we convert existing users back and remove that functionality if it doesn't work properly?