Received: by 2002:a05:6358:4e97:b0:b3:742d:4702 with SMTP id ce23csp1029590rwb; Thu, 11 Aug 2022 14:31:30 -0700 (PDT) X-Google-Smtp-Source: AA6agR4mjHuOF0MC2ylsmdJpHkOZpTX/pZACRxO7zVue9Pj8Ryu6YKdkgAf0gtV2TTjaLAxGKPbl X-Received: by 2002:aa7:954d:0:b0:52e:b22c:14a2 with SMTP id w13-20020aa7954d000000b0052eb22c14a2mr931351pfq.45.1660253490148; Thu, 11 Aug 2022 14:31:30 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1660253490; cv=none; d=google.com; s=arc-20160816; b=JN+E+8v75AWX48Kqvb3SH8FP+xHC/1ezy/VfLb0/O4EXp/FOOIQTw12FOdqUPb64Iv eB3um6mkPhoCO1Sr8Ien3bRJuFNaSAF8oykUt9Ya8JOprv00f2pOh5LnPDQgPo0FrOJM GvUNXBkyU7xqm/UTFu2RXU6NOeK/fE809F+0BAb42EGpO0PdlK9065r0gzBOWk9wfDj8 G4h9K1ZBpYJewD7g9dZEsF5tE3RtcOGYg3T/Q8Zz8ZxKHGpT0dZHf6GLkLoo5hsD/pYd jgEjwmU9WLsuaNabV0D3i9hzH1sV3dtUoFDG9udfeB2jjGzUK42cpoII7u8HoeJ74fjD Woxw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:cc:to:subject:message-id:date:from:in-reply-to :references:mime-version:dkim-signature; bh=VBVguebNsed4JHqBLdZ7/rFrHjcbNJWah0iJWMXOSv4=; b=Q8cjFEyFKVT/dEfjgZDFFWqMjX6mlrSVZRePryW08OU/hJGJYgQC4FeP8c69+1tkqI 87HR7eJBXjTu/+DOEKygCJixih6cEb9+pg2vhNI9qWEKvclKEXVaneAM0T06rWa/6uzs Y7OpZvASscDp9uiMVJ4d0v8r+bbAZsfCfvvCnV1s9FQO2aMt4NNOcspFGOhhJ/Xbkuny VFcf3gF372mGOIZ1Bz5NyjfFIAR3wNUPbTiI1LJEw6wKz8WXZYSn1vIu5SnYq6c68eTI V9hOxCRj6h5hOxc9THxseu9dRAfp0qYAt1A6O6O00plrvuH6Creg49FcniNlJ3fcrJ2l 3iPw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linux-foundation.org header.s=google header.b=NFG1HQSM; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id b15-20020a170902d88f00b0016efb38cf39si269616plz.141.2022.08.11.14.31.15; Thu, 11 Aug 2022 14:31:30 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@linux-foundation.org header.s=google header.b=NFG1HQSM; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229591AbiHKV35 (ORCPT + 99 others); Thu, 11 Aug 2022 17:29:57 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:35544 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S235221AbiHKV3z (ORCPT ); Thu, 11 Aug 2022 17:29:55 -0400 Received: from mail-ej1-x62a.google.com (mail-ej1-x62a.google.com [IPv6:2a00:1450:4864:20::62a]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 506F29D8E8 for ; Thu, 11 Aug 2022 14:29:53 -0700 (PDT) Received: by mail-ej1-x62a.google.com with SMTP id kb8so35701708ejc.4 for ; Thu, 11 Aug 2022 14:29:53 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linux-foundation.org; s=google; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:from:to:cc; bh=VBVguebNsed4JHqBLdZ7/rFrHjcbNJWah0iJWMXOSv4=; b=NFG1HQSM0O6Jf7SBD7BNXBhIFpeZiANKK67mUTxpyGt9xhZstsMvYLuTfwX1HANa9X P1oHlk3r5nM94hBzmuLNwwWxmK2sitEZhvHeDJhawn2Mne+EPZJq5eWghnbFL18KKCdw kuQlXz3dGIvTtpgMfta6fN3M/1ES2zGHXd+AA= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:x-gm-message-state:from:to:cc; bh=VBVguebNsed4JHqBLdZ7/rFrHjcbNJWah0iJWMXOSv4=; b=2tviItqGlpPLxZEd38VN8BjJeblIsA6OfDW2VEFpZ+UyIe5gnJxbubUXMooK6lveyI jMbehWG902tSOp7BFge4oPNbVVKbwVXfCuN+mFHuhNmnbxuDqCUYCmjLxcYIHYg1iQPt bvKIlyGkOgFCQHfiWUBoXvf2KAPJf1pGuC0TG2c6iBBcPlk0TKP/8WpIxaDGMkU6OL5p J2EiBm2UwY1jtYoF6DDY5HQSNya4Fy9epTziK04nAOvfGNnDe3No8eYQx28R7IEezMoD +Jv7BYb1owBcV4hdbDLLVVSrryhfDP244FZMpmocV1kjU3QElSKxdQVRcOOlrECk2uWN JtNw== X-Gm-Message-State: ACgBeo1SUzCGz2A93RLVO/5I+vkxMstUEdxegy5pcx7XU2Eqqcdm1Tv9 RSKMX7ArAr6rJoBCI8jnZLUzvH07gUOrzOUR X-Received: by 2002:a17:907:7f1d:b0:732:fa14:2012 with SMTP id qf29-20020a1709077f1d00b00732fa142012mr660735ejc.652.1660253391714; Thu, 11 Aug 2022 14:29:51 -0700 (PDT) Received: from mail-wr1-f46.google.com (mail-wr1-f46.google.com. [209.85.221.46]) by smtp.gmail.com with ESMTPSA id gv41-20020a1709072be900b00730ed9e4ce1sm76816ejc.198.2022.08.11.14.29.50 for (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Thu, 11 Aug 2022 14:29:51 -0700 (PDT) Received: by mail-wr1-f46.google.com with SMTP id bv3so22668446wrb.5 for ; Thu, 11 Aug 2022 14:29:50 -0700 (PDT) X-Received: by 2002:a05:6000:178d:b0:222:c7ad:2d9a with SMTP id e13-20020a056000178d00b00222c7ad2d9amr431723wrg.274.1660253390641; Thu, 11 Aug 2022 14:29:50 -0700 (PDT) MIME-Version: 1.0 References: <20220811152446.281723-1-idryomov@gmail.com> In-Reply-To: From: Linus Torvalds Date: Thu, 11 Aug 2022 14:29:34 -0700 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [GIT PULL] Ceph updates for 5.20-rc1 To: Ilya Dryomov Cc: ceph-devel@vger.kernel.org, linux-kernel@vger.kernel.org, Jeff Layton , Al Viro , Matthew Wilcox Content-Type: text/plain; charset="UTF-8" X-Spam-Status: No, score=-1.8 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS, RCVD_IN_DNSWL_NONE,SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=no autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Aug 11, 2022 at 1:56 PM Ilya Dryomov wrote: > > On Thu, Aug 11, 2022 at 10:04 PM Linus Torvalds > wrote: > > > > What's the point of warning about bogus folios more than once? That's > > a debug warning - if it hits even once, that's already "uhhuh, > > something is bad". Showing the warning more than once is likely just > > going to cause more problems, not give you more information. > > Xiubo and Jeff used it to track down some issues between netfs library > and folio code that have been randomly plaguing our automated tests for > a couple of releases. We already knew that there were issues in that > area and the actual occurrences mattered. This was done in cooperation > with Willy and, since he was involved and this is a no-impact change, > I didn't think twice. I don't mind the warning. I mind the "more than ONCE" part. If it's a "this shouldn't happen, but if it ever happens I want to know about it" situation, then the ONCE variant should be what you want. And that variant already existed, and adding a new and inferior macro seems to just have been pointless. As to dcache issues - if you really don't get an ack from Al, you can at least make me aware of it before-hand. That's one of the files that I at least personally care about, and while I would much prefer an ack from Al for anything that touches it, at least I'll likely be less unhappy about changes if I was made aware of them ahead of time, instead of seeing a pull request that suddenly changes that file. Linus