Received: by 2002:a05:6a10:a852:0:0:0:0 with SMTP id d18csp195929pxy; Fri, 7 May 2021 00:47:58 -0700 (PDT) X-Google-Smtp-Source: ABdhPJyoc1e4uZMtmI4xZDrx6SIiSZ1Rro3JBxcIRkWGr2wq+04YJNWgJLwLkQZgY1Re6yys5Gk0 X-Received: by 2002:aa7:d5c9:: with SMTP id d9mr9835711eds.153.1620373678522; Fri, 07 May 2021 00:47:58 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1620373678; cv=none; d=google.com; s=arc-20160816; b=hQNC35+HYxqB30TBT0WmikCsG23mNUH2OXpTkc8C+LkkIKRX0UqDbF4icqZinrBpiC Wozd2zgBxVn88wacNqW+IW0SIz/2ii7o5k2jrEjTK/9GAi4n6erqdXuuhwsa5iVZLJqp N9XsBz0AuCCGzjmbmkwwt2kL1iBDrRsnD1xv+qIz1AoAuhD4MJhS+PB1yDN5PgFniBFD 4BStnbKBhrD0Ntk/ziAUZ5MN8EeyvVu+nSKYyMl++PD6Hy8LnJkwE9qIudhyLBwpNU6p qtPgeeG+HGHjRS5Xi76EHWkBI0DG3V8jMPtC+h9hCV+bIsgcBzCiSutf4dof93pp8TQW 0QLg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:in-reply-to:content-disposition:mime-version :references:message-id:subject:cc:to:from:date:dkim-signature; bh=yhgio/eBb2HNtTdxqYR2YXXTVe7upvPzsVWjxcm3SOA=; b=JZ3tRJig9bqKNoK2wx5F1Cyn4UkGgCOjRSGZM9pf93ppPSMJY8N0OhGhOdtcIbZuyZ 2xYhX9oRTgxp/s9hEZeLu7lYHJoxc/88Cb/0WXvhWIsCa1ePfLQHCiVbX5x5RNXpN3s0 Rn+tsCWtwOEKxhJLz8FkIqRZTlJhZ94qCrHGweFocQ08un7kDbOYBkFLOsyn8aCC7NvS QiDxLIwhHX8viOZiHqq/EzAk655zsef0sx9KU+Gt5ApPl3VzQWQolN1GvKZHDlCea4AB kTnR5FzjjShRrInVWxPkUYONHIBLcXKzZhbDoTXrUB1pGVSsQK9ADPVwol54LwHpinfs TYhA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=Ca3dThI7; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id sb14si4517787ejb.322.2021.05.07.00.47.34; Fri, 07 May 2021 00:47:58 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=Ca3dThI7; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S234800AbhEGGeo (ORCPT + 99 others); Fri, 7 May 2021 02:34:44 -0400 Received: from mail.kernel.org ([198.145.29.99]:47016 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S234791AbhEGGe0 (ORCPT ); Fri, 7 May 2021 02:34:26 -0400 Received: by mail.kernel.org (Postfix) with ESMTPSA id 4B98C6112F; Fri, 7 May 2021 06:33:27 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1620369207; bh=BpAPmiejLFpAyTHujOzMNbWbBmKEXC5DKEs1SQmHxIo=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=Ca3dThI7CoAz/6b6jxiTn3EDabPzD3RPFff82F5AJAVgQdK+S9HWXEDWFgdF4R0yE e4FUDEpNcWPsqSd9OBXNPV9BWHNMIoVzGY3WO29tfL1zTS1GFiVL+JGxVoJqJTx+yv qk16Hoc9sRgy5CtFKbmPo7yElKP9Bs52lGanr17GfkSur7LQ6R3VaoMjhLxFLHcCdF yzuei+YcuyYw3R/VSgtq47qeHQ/CKczYCvvAgfTxuWUb2C5XiQggDH3ciGP3WOCwvH 2BbwKygdZm2D2nEQZl8QuuxLBiCaH+w+PZzW4WGdHY4g3q3ZRIxB2MdrDHkxApxbxo SfjZ270CPe95A== Date: Thu, 6 May 2021 23:33:25 -0700 From: Eric Biggers To: Chao Yu Cc: jaegeuk@kernel.org, linux-kernel@vger.kernel.org, linux-f2fs-devel@lists.sourceforge.net Subject: Re: [f2fs-dev] [PATCH v2] f2fs: reduce expensive checkpoint trigger frequency Message-ID: References: <20210425011053.44436-1-yuchao0@huawei.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20210425011053.44436-1-yuchao0@huawei.com> Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Sun, Apr 25, 2021 at 09:10:53AM +0800, Chao Yu wrote: > + if (!error && S_ISDIR(inode->i_mode) && f2fs_encrypted_file(inode) && > + f2fs_is_checkpointed_node(sbi, inode->i_ino)) > + f2fs_add_ino_entry(sbi, inode->i_ino, ENC_DIR_INO); This will never be true, since S_ISDIR() and f2fs_encrypted_file() are logically contradictory (as f2fs_encrypted_file() only returns true when S_ISREG()). How did you test this change? - Eric