Received-SPF: pass (google.com: domain of linux-nfs-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   Chuck Lever III <chuck.lever@oracle.com>
To:     Neil Brown <neilb@suse.de>
CC:     Chuck Lever <cel@kernel.org>,
        Linux NFS Mailing List <linux-nfs@vger.kernel.org>
Subject: Re: [PATCH RFC 0/4] Encode NFSv4 attributes via a branch table
Thread-Topic: [PATCH RFC 0/4] Encode NFSv4 attributes via a branch table
Thread-Index: AQHZqvMKgPkCgxCJ3UKxuAN3nOfCL6+nfTiAgACZHoCAAH/SAIAAGaKA
Date:   Mon, 3 Jul 2023 23:05:33 +0000
Message-ID: <15AEF33D-CA17-41C9-80D3-B89673804D7B@oracle.com>
References: <168808788945.7728.6965361432016501208.stgit@manet.1015granger.net>
 <168835968730.8939.17203263812842647260@noble.neil.brown.name>
 <F0F2C0F7-3F73-4713-BB37-661463646CC5@oracle.com>
 <168842001897.8939.938340124592787737@noble.neil.brown.name>
In-Reply-To: <168842001897.8939.938340124592787737@noble.neil.brown.name>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?AIG1nYNIsOoxefYCtQDxpaKzI9WpQPwtd36c0d3jGgFkTwacY4AK3skVuS5X?=
 =?us-ascii?Q?RN1UDpHtSOgwNvGWENz6WoZlXA0D0GwfKNwaFKOpoLiBhuNTC9cnLJI1U+G9?=
 =?us-ascii?Q?EWZKX+VirT4j4I0yFuw7Ec+zqAnUAzWLvkb/57UvJCG47Yda1R7xx2e3WMDR?=
 =?us-ascii?Q?jeB3q3yofYDqv6bpc4M9EYwPaLyc584eXRjT8HPAX4mpcaSbZCcfk1yBmMHn?=
 =?us-ascii?Q?di7mRn6wt+ptUtHcw9/LIk4QMVxqGuR+W1SxyWCt+AZiN/T2yHvgBNxgKOWs?=
 =?us-ascii?Q?0pdkPyBqPWBiPRqlX+z0YXUMPNE+MK6IiP+x/p62yRgkRR9nM7eb29LYtj48?=
 =?us-ascii?Q?doxWxuOTWYUerAc4Po+8KBhFNx1oU7Z48j1ZtnEH8iPGvVz+H4v5F8ll1LWX?=
 =?us-ascii?Q?xJ5LLoGvlZ3QZ+RTZ+GSU79asNt7jCCdE+bg6Nh2jee1iAxSayD9Om41J80V?=
 =?us-ascii?Q?rSho1+pyNc3qasAZzMuTp0IoFFUwSkY7x4HvLSet2a2kaTFqhn/cIzaDdxJ6?=
 =?us-ascii?Q?D53P2jSJ0QkKwOagb/Mg+h41fZ4uk0rO3lUSkSmdI/XHB9K2SEHEg3XlQ2xp?=
 =?us-ascii?Q?q4z8pKfWvk2d6sxNF1mJR2uxBXn7ZG7SYh6dkGAp1y/gzphQyIvWvPxRGpJ+?=
 =?us-ascii?Q?2YXvviNTjyPRIBsB1E3K71I9z7DdCw+B6cP/fvrOpc6EcNOM0TUO4VyjDg/v?=
 =?us-ascii?Q?9CasINeKkWN5oGE8f+nNWj8OhzgJ0aVkHt9+to6v0qQi5KPCMty8RNuJ9Hky?=
 =?us-ascii?Q?bZu+gbCC0UjJTdzNYcmirnsvxACVFZ6U/TQL8gz0bepRKCgMGhvjrnzb6RSg?=
 =?us-ascii?Q?LskPUV25WVg13kdvGDpjYD8lbqh79kEDvyb3Ypdb025ddwFjno3jsFR9cOm5?=
 =?us-ascii?Q?6KWhAM1l/4EzTGX+gRDmmlMGQucmtox+qrmCVJrMsHwG51nwKn8hy4UmeZUO?=
 =?us-ascii?Q?jwwtSxPkp7NH6ygxouTSvS3nusmYel5iWPQbfc513nSxJEdfRRq9qb1LID/F?=
 =?us-ascii?Q?UefzQKydi+1/KBIvq0ODdE1zdE7lYS/mLJDqTRg0uZ9rl+OP8W27qjqC2Zwz?=
 =?us-ascii?Q?phNjeL62cvJnNAX4xFeAp9K10ddKndsgH5rN+T9VbT5H1odyFu23heKktnxI?=
 =?us-ascii?Q?HV9i2xEW3Ged5ff9nRb9I3DSOp5fotcUx1GNrBl6MHD9Y7pc4pWY38pdbbRR?=
 =?us-ascii?Q?Vtc49wfxzoiihBaizhvchVz9vDbgsEn8ptVKVZequ9rul2Ld/v1JVZb7JG73?=
 =?us-ascii?Q?7Agf+cavHbYWQdp7w7rLiTbyNrUNm77fBP71yQHB+HsJzLc/5SOrOipAU4qS?=
 =?us-ascii?Q?5CGqhGuFNPdJ6USCm9EaY/M1B5+Ok+GE1HYRYM8fNZHXEF9u3H2J3bNHbS9d?=
 =?us-ascii?Q?Wiu0yB3/3zfWH52U5dnQIteF+83DEVsEnNa0JaVang9sHwqvHeDSnuy3rDKW?=
 =?us-ascii?Q?y3y/m1wBMTUWHb0rsAH6C+gTMR6FzOvi9Bpov+Wgm5fdHwhaiOVLI5kIjcM5?=
 =?us-ascii?Q?9DMwDWiDEjHA7VvQKVYuxd66Z0AUAZk0pBArUk9N8mMECVizNV+3wRl65K6G?=
 =?us-ascii?Q?cZrqHbdi/oTNOBKfAHngdgYs2kqfLr7AC+2EdjHNk5RM+8NgzssAq8fYB8UI?=
 =?us-ascii?Q?wA=3D=3D?=
Content-Type: text/plain; charset="us-ascii"
Content-ID: <38126A0C94A83B4F83CFC8F99E260E37@namprd10.prod.outlook.com>
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-ExternalHop-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-ExternalHop-MessageData-0: Qe1nBpzJKahlZpAc05YFc/gIvgII1YiaKfvpCsgUwve5paIlwHXmQhjsS+ZbR39qWyYMHu9PouWOB+PqqvVr4Gz76r2UWISfMx++ssVdTSmKZpPcQYkJ3PELIZt5z1a43Xtt2pRMxgSmgzzw7aYu2m5jiZ6J8J/Tc45GnqfaIPZyIqFMnrq2j5PKpFtNnHNhYYhCr4AvVV8hG12SX/c/iUGUqFQuP9AmhWylBNLPcGSJoVnUsSESnvs7VkAPgKJlpTFI4352kNRmiL79PRNbR0agntEcf/aNV+ATTljnT63EvAVttj0F0PCXJIOfI/K2QtDwC268FcET9znXdVg/PnPsrCPqtSfFA0ZSIA/SRhOasTPCyFuQ9tRMNPI7vQDqDok7evu9hKO2iRCsPZBKFXzmGx9lfnU9G1rQmA6Xca52JsnR1EqiUHL+6gtLmWiuxEkSucLAuJ0FC9g/MTEAu7b68y3WpSO8+sregHuXWnk0YnYaslTjwT36AcSbC9ba9HF+/aOuFTH5lBmMuKXtqsfMuBesNkP6Z3DP2jgfKHGzAzUvMBzqJnAnAetJ/85CMQoAIsgSVFzrNPvkjAQ4SMXN2nOKuhTTiPWVJHSLYiAwvRW7PoUMYpb7VcS8yuvPRwXEnZpVqs0GxrNP8E0RPPWWtWRJaZGKhnNkKW4XuBzvZzq/AFq7TLCyq0c3BiKAaYDfI2OvZw0+joDPPB7pUGz4GkP7ynTLNTpFmAV751HGo8oDGRXT7k/2CkaFJ07B0LNMTcZHaMoZmhu2tnJYXN78+JptilBJ9GL7Cjw90yB2Ipunhu8efeXZfhNhycaE
X-OriginatorOrg: oracle.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: BN0PR10MB5128.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 08432ac1-b650-49ee-638e-08db7c1a0167
X-MS-Exchange-CrossTenant-originalarrivaltime: 03 Jul 2023 23:05:33.7593
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 4e2c6054-71cb-48f1-bd6c-3a9705aca71b
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: 3XvtHxqt8WVA2eQnns+pDdGoBCf0AMPulBkM7BY1pGNNZsT65zbtBC9beVADwyvh7UWWpVkRrF3rRMFpv7fIrw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CH3PR10MB7562
X-Proofpoint-Virus-Version: vendor=baseguard
 engine=ICAP:2.0.254,Aquarius:18.0.957,Hydra:6.0.591,FMLib:17.11.176.26
 definitions=2023-07-03_15,2023-06-30_01,2023-05-22_02
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 suspectscore=0 phishscore=0 bulkscore=0
 malwarescore=0 mlxlogscore=825 adultscore=0 spamscore=0 mlxscore=0
 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2305260000
 definitions=main-2307030209
X-Proofpoint-GUID: E-aR-jH0ZZb0vCW2TVwuz9Jrg-rayrnF
X-Proofpoint-ORIG-GUID: E-aR-jH0ZZb0vCW2TVwuz9Jrg-rayrnF
X-Spam-Status: No, score=-2.8 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_LOW,
        RCVD_IN_MSPIKE_H5,RCVD_IN_MSPIKE_WL,SPF_HELO_NONE,SPF_NONE,
        T_SCC_BODY_TEXT_LINE autolearn=ham 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: <linux-nfs.vger.kernel.org>
X-Mailing-List: linux-nfs@vger.kernel.org


> On Jul 3, 2023, at 5:33 PM, NeilBrown <neilb@suse.de> wrote:
>=20
> On Mon, 03 Jul 2023, Chuck Lever III wrote:
>>=20
>> - It's not the efficiency of encode_fattr() that is the issue,
>>  it's the frequency of its use. That's something the server
>>  can't do much about.
>=20
> There probably needs a protocol revision to improve this.  I imagine a
> GETATTR request including a CTIME value with the implication that if the
> CTIME hasn't changed, then there is no need to return any attributes.

You can precede the GETATTR with an NVERIFY operation.

https://datatracker.ietf.org/doc/html/rfc8881#name-operation-17-nverify-ver=
ify


--
Chuck Lever