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 Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 8F91BEB64DA for ; Wed, 5 Jul 2023 20:26:52 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S233718AbjGEU0v (ORCPT ); Wed, 5 Jul 2023 16:26:51 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:41184 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233619AbjGEU0r (ORCPT ); Wed, 5 Jul 2023 16:26:47 -0400 Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.129.124]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id C523E1990 for ; Wed, 5 Jul 2023 13:26:06 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1688588766; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=t45r9A2YvfoSJv2GZo8UC8AtO/lxGa/2A2TsFaxOzFQ=; b=KvBPxlmM7mEvVLMsbCX/DAIj6BbKqGnVDhcdDKOY5frryJUrPW+4hqwqhS92uWh3DpKW3S hGldEEDgW8vwHY/fdrTBK2RpXHf5ZW7b2YHBVzH/oPXUDerM/7l7oAs/nuzz1RM81YElis CvSBJ/Ty6HOiEPf4gJ7AteLAfw5OeJc= Received: from mail-pf1-f197.google.com (mail-pf1-f197.google.com [209.85.210.197]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-177-jB6jzi-MOFOlgPVUCuK0Ug-1; Wed, 05 Jul 2023 16:26:04 -0400 X-MC-Unique: jB6jzi-MOFOlgPVUCuK0Ug-1 Received: by mail-pf1-f197.google.com with SMTP id d2e1a72fcca58-67106f598b1so83813b3a.2 for ; Wed, 05 Jul 2023 13:26:04 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1688588764; x=1691180764; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=t45r9A2YvfoSJv2GZo8UC8AtO/lxGa/2A2TsFaxOzFQ=; b=BIWf7F4r8ZglUD2xlKipYDytIyX3jCNy3uNt0iflcrUout8O+yE9CQg4uyItL2fMdQ YiHZCqUIekpFuQjXSHhe4V15u62WqW+zuGlouG2MwO4h9jOMRhl2UqDgKjX/opm3h+CZ XMHR3kyx+TWbb1Vo1s16qlSdK3brT/EJNosQO5f4XRlanZIVDMI46S75rNKUsWc5/cSG rTg0q2QyR0tKAjJLEP8lpP3+oPARnD+RoBoC5HGQOt8O88du4FmqYbgQVsc+SWqB1FYm ST1bv+mm1/w9wvkmsDwDeLr47G6NcKuFK2nR8OxtnWJUUbGheOQyllBAspoJV8lFsBWY 5uLA== X-Gm-Message-State: AC+VfDynTcU3V+ZLZHxKITDio3atzvqiJCFfme3dcih/aKt5lOA4zkEE ZrOedu/zXC+RhzK40dZU82uuxwGsCoEAmTrQgnzqpW8TzcNYFRzea3ZP1yu1KXpYGdEOa4ZCffQ QcTvVNzq++TvaoGIu71kkBjeMo6VhbRiOOewiPnwL X-Received: by 2002:a05:6a21:6d88:b0:122:e4f:25c7 with SMTP id wl8-20020a056a216d8800b001220e4f25c7mr15659569pzb.31.1688588763762; Wed, 05 Jul 2023 13:26:03 -0700 (PDT) X-Google-Smtp-Source: ACHHUZ6ds4xvbUG+ZYaDvJ2apGx91OQ4qB0akGNWWFL3LaWKhMz96IxFpmQ2sEuPoCI3p2ih7fuYhkf9fNjQ5oeaAq0= X-Received: by 2002:a05:6a21:6d88:b0:122:e4f:25c7 with SMTP id wl8-20020a056a216d8800b001220e4f25c7mr15659535pzb.31.1688588763446; Wed, 05 Jul 2023 13:26:03 -0700 (PDT) MIME-Version: 1.0 References: <20230609125023.399942-1-jlayton@kernel.org> <20230609125023.399942-8-jlayton@kernel.org> In-Reply-To: From: Andreas Gruenbacher Date: Wed, 5 Jul 2023 22:25:51 +0200 Message-ID: Subject: Re: [PATCH 7/9] gfs2: update ctime when quota is updated To: Jeff Layton Cc: Christian Brauner , Al Viro , Brad Warrum , Ritu Agarwal , Arnd Bergmann , Greg Kroah-Hartman , Ian Kent , "Tigran A. Aivazian" , Jeremy Kerr , Ard Biesheuvel , Namjae Jeon , Sungjong Seo , Bob Peterson , Steve French , Paulo Alcantara , Ronnie Sahlberg , Shyam Prasad N , Tom Talpey , John Johansen , Paul Moore , James Morris , "Serge E. Hallyn" , Ruihan Li , Suren Baghdasaryan , Sebastian Reichel , Wolfram Sang , linux-kernel@vger.kernel.org, linux-usb@vger.kernel.org, autofs@vger.kernel.org, linux-efi@vger.kernel.org, linux-fsdevel@vger.kernel.org, cluster-devel@redhat.com, linux-cifs@vger.kernel.org, samba-technical@lists.samba.org, apparmor@lists.ubuntu.com, linux-security-module@vger.kernel.org Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Jun 12, 2023 at 12:36=E2=80=AFPM Jeff Layton w= rote: > On Fri, 2023-06-09 at 18:44 +0200, Andreas Gruenbacher wrote: > > Jeff, > > > > On Fri, Jun 9, 2023 at 2:50=E2=80=AFPM Jeff Layton = wrote: > > > Signed-off-by: Jeff Layton > > > --- > > > fs/gfs2/quota.c | 2 +- > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > > > diff --git a/fs/gfs2/quota.c b/fs/gfs2/quota.c > > > index 1ed17226d9ed..6d283e071b90 100644 > > > --- a/fs/gfs2/quota.c > > > +++ b/fs/gfs2/quota.c > > > @@ -869,7 +869,7 @@ static int gfs2_adjust_quota(struct gfs2_inode *i= p, loff_t loc, > > > size =3D loc + sizeof(struct gfs2_quota); > > > if (size > inode->i_size) > > > i_size_write(inode, size); > > > - inode->i_mtime =3D inode->i_atime =3D current_time(in= ode); > > > + inode->i_mtime =3D inode->i_atime =3D inode->i_ctime = =3D current_time(inode); > > > > I don't think we need to worry about the ctime of the quota inode as > > that inode is internal to the filesystem only. > > > > Thanks Andreas. I'll plan to drop this patch from the series for now. > > Does updating the mtime and atime here serve any purpose, or should > those also be removed? If you plan to keep the a/mtime updates then I'd > still suggest updating the ctime for consistency's sake. It shouldn't > cost anything extra to do so since you're dirtying the inode below > anyway. Yes, good point actually, we should keep things consistent for simplicity. Would you add this back in if you do another posting? Thanks, Andreas > Thanks! > > > > mark_inode_dirty(inode); > > > set_bit(QDF_REFRESH, &qd->qd_flags); > > > } > > > -- > > > 2.40.1 > > > > > > > Thanks, > > Andreas > > > > -- > Jeff Layton >