Received: by 2002:ac0:a582:0:0:0:0:0 with SMTP id m2-v6csp2436439imm; Thu, 18 Oct 2018 14:46:03 -0700 (PDT) X-Google-Smtp-Source: ACcGV637pTpEes+sOZvv54UADWK2zDFalouTyygFOx2fJVuOrYH0Ev6/b9TeQorPM6fIaF4NinA2 X-Received: by 2002:a62:2e02:: with SMTP id u2-v6mr33032008pfu.134.1539899163594; Thu, 18 Oct 2018 14:46:03 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1539899163; cv=none; d=google.com; s=arc-20160816; b=giAnVWIXwHQxHCacjp6tbj7/fRuCh9DtElJZNDF9ovOd2Q4hRLgYJHRhh8pfQ5v2N0 rLaCr6TnUlw7s3DgRQznNaYQHZXnK0QLBXXM8i3wRyCgvDizOvEUAk5eJh/eKBydqm6G QoeB8AtD5MHXIjF+4kxsK8weWIWkYbneWiePhy0ghfh9qhTMMC+swRpDRGonJZ33bNe+ 7AExZbqfuboRM4zCEY2B1gfsK5e8c+JEq3oyR03gdTA+hepvfoqIBNDElGc623OxNQhT RpPjotdsDOEpdLrCHwpOqLDpTQbFEQJefp5Qm+K/GvjZWggX0+qBID/VjXUbuiYbrRIo RVXw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:in-reply-to :content-disposition:mime-version:references:message-id:subject:cc :to:from:date:dkim-signature; bh=cKO6C4CuFmlcQMHod2k6MX0M8iJXUgxmWJyTyPMmdH0=; b=RwOT0cA04A2ydOgcnfeFK2CtEc+bGNl9YzkBhsLvQFrODAA5u/xYFvqbxAGMO9k+gt asNfYuOm5vM4kZDFyLKKxXX+TsJ4Z6QcdItGK3OqYmd6+A10pznXDHaQBRp5ZXPnrqHE FP59i5fN3TJMFvw3gJvWVumITQzWr+60uSjvrjqpSJYRFObw+XwyRvCbVuMsoplolbG6 bmeWHsjwArHlH1u4kwIyC6UWYsLpVdbZrW8sdBO30tNLeVWD6sGjeFeMZhCKw8pcckze yAh+M3jaViDrrjz3eG1MI7AmzTPXbt11hxaiEJpNDTRuwhqKyszI+MCexMpOw4v1WYoV br5A== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@omnibond-com.20150623.gappssmtp.com header.s=20150623 header.b=HnfyWpv6; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id p70-v6si7416371pgp.53.2018.10.18.14.45.45; Thu, 18 Oct 2018 14:46:03 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=pass header.i=@omnibond-com.20150623.gappssmtp.com header.s=20150623 header.b=HnfyWpv6; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726627AbeJSFsT (ORCPT + 99 others); Fri, 19 Oct 2018 01:48:19 -0400 Received: from mail-yb1-f193.google.com ([209.85.219.193]:34512 "EHLO mail-yb1-f193.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725751AbeJSFsT (ORCPT ); Fri, 19 Oct 2018 01:48:19 -0400 Received: by mail-yb1-f193.google.com with SMTP id 184-v6so12484549ybg.1 for ; Thu, 18 Oct 2018 14:45:21 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=omnibond-com.20150623.gappssmtp.com; s=20150623; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to:user-agent; bh=cKO6C4CuFmlcQMHod2k6MX0M8iJXUgxmWJyTyPMmdH0=; b=HnfyWpv6P/ai9KVTY/w8rHoTFouxeKSrPlJtTFlB+lGN/YR3dfMyedG8I3NhM9OUtM kcxcpzBUVfKMazA1cRvRUSX0K6vR4bFrA10adWv0PLABT/QYe8HQLtuIX2GC1pknLRER /utDiGFztGnOPqX+wqFSj5w4TasZFC0+Df1Bi+/dDvOMVEJJBKQMH84QsPrhHBpJ1deY IgtcEoPBM/kixs/R4r8NT/0qL2UGALHB6ha2q+sTMo/vyy+omisg6/CdE0FIQ6uPh0/D UvrMetVPlNsKjzSuG1PVy7eCtNmj+UVINa/qBpCm1MLSStdo3F7BB/CGrHBacsdQuzX+ g9yA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to:user-agent; bh=cKO6C4CuFmlcQMHod2k6MX0M8iJXUgxmWJyTyPMmdH0=; b=marXTr8lHnzyWVWAJcVqf8WPNBNrGQX3Ss7GaPxxbanv0766jb7WFvSwsXuB/k8xWh lp/JaKFwTIMLjwjsx7GA57SNvX6fsz04f7FFCd1AfkQlNs1JNJhWkg6HNrobhuDoEdCl 0qhITHTvdAYFA2AV/sNZKzQgAC8f2Q3+qvgFFHR0rpPzvDuGeualv3dQDCKKHqFVTZbQ lbmTUVcUX1olaNMpsfw0+O+s4Nm041l5foo77tM47HSiFbelKbTFrLqDUWDJw7+yytOl wKOvDj5Xv54lUk9MSPp2QLSiV5K9yKFK39qZ/3d/twTS+WofklQdUqJNZ5Z7H2lOOeYt nROw== X-Gm-Message-State: ABuFfoh02xItQSgOoaaRqu8UShQ2vv/hrXMi+xly/eQTkE+3g8CbxOBz HUAli/reCdXBOrMZRvVNGygtfg== X-Received: by 2002:a25:67d6:: with SMTP id b205-v6mr3049621ybc.521.1539899121109; Thu, 18 Oct 2018 14:45:21 -0700 (PDT) Received: from t480s.mkb.name (47-49-164-163.static.spbg.sc.charter.com. [47.49.164.163]) by smtp.gmail.com with ESMTPSA id x64-v6sm5978439ywx.103.2018.10.18.14.45.19 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Thu, 18 Oct 2018 14:45:19 -0700 (PDT) Date: Thu, 18 Oct 2018 17:45:17 -0400 From: martin@omnibond.com To: Miklos Szeredi Cc: linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org, linux-api@vger.kernel.org, David Howells , Michael Kerrisk , Andreas Dilger , Florian Weimer , Amir Goldstein , Mike Marshall Subject: Re: [PATCH 1/3] orangefs: fix request_mask misuse Message-ID: <20181018214517.GA44581@t480s.mkb.name> References: <20181018131125.6303-1-mszeredi@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20181018131125.6303-1-mszeredi@redhat.com> User-Agent: Mutt/1.9.4 (2018-02-28) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Oct 18, 2018 at 03:11:23PM +0200, Miklos Szeredi wrote: > Orangefs only handles STATX_BASIC_STATS in its getattr implementation, so > mask off all other flags. Not doing so results in statx(2) forcing a > refresh of cached attributes on any other requested flag (i.e. STATX_BTIME > currently) due to the following test in orangefs_inode_getattr(): > > (request_mask & orangefs_inode->getattr_mask) == request_mask > > Also clean up gratuitous uses of STATX_ALL. > > Signed-off-by: Miklos Szeredi > Cc: Mike Marshall > Cc: Martin Brandenburg Reviewed-by: Martin Brandenburg > --- > fs/orangefs/inode.c | 7 ++++--- > 1 file changed, 4 insertions(+), 3 deletions(-) > > diff --git a/fs/orangefs/inode.c b/fs/orangefs/inode.c > index 31932879b716..bd7f15a831dc 100644 > --- a/fs/orangefs/inode.c > +++ b/fs/orangefs/inode.c > @@ -256,7 +256,8 @@ int orangefs_getattr(const struct path *path, struct kstat *stat, > "orangefs_getattr: called on %pd\n", > path->dentry); > > - ret = orangefs_inode_getattr(inode, 0, 0, request_mask); > + ret = orangefs_inode_getattr(inode, 0, 0, > + request_mask & STATX_BASIC_STATS); > if (ret == 0) { > generic_fillattr(inode, stat); > > @@ -408,7 +409,7 @@ struct inode *orangefs_iget(struct super_block *sb, > if (!inode || !(inode->i_state & I_NEW)) > return inode; > > - error = orangefs_inode_getattr(inode, 1, 1, STATX_ALL); > + error = orangefs_inode_getattr(inode, 1, 1, STATX_BASIC_STATS); > if (error) { > iget_failed(inode); > return ERR_PTR(error); > @@ -453,7 +454,7 @@ struct inode *orangefs_new_inode(struct super_block *sb, struct inode *dir, > orangefs_set_inode(inode, ref); > inode->i_ino = hash; /* needed for stat etc */ > > - error = orangefs_inode_getattr(inode, 1, 1, STATX_ALL); > + error = orangefs_inode_getattr(inode, 1, 1, STATX_BASIC_STATS); > if (error) > goto out_iput; > > -- > 2.14.3 >