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:     Jeff Layton <jlayton@kernel.org>
CC:     Linux NFS Mailing List <linux-nfs@vger.kernel.org>
Subject: Re: [PATCH v3 0/7] Wait for DELEGRETURN before returning
 NFS4ERR_DELAY
Thread-Topic: [PATCH v3 0/7] Wait for DELEGRETURN before returning
 NFS4ERR_DELAY
Thread-Index: AQHYqy4gRT0K7GpiEkGiJB7utcU4AK2lWIgAgAAVxAA=
Date:   Mon, 8 Aug 2022 20:06:23 +0000
Message-ID: <3D17DE4C-73B9-4635-A102-72955D19CCDE@oracle.com>
References: <165996657035.2637.4745479232455341597.stgit@manet.1015granger.net>
 <55bc0656a841cc1229d2b1594d4f9eeabfd00ae5.camel@kernel.org>
In-Reply-To: <55bc0656a841cc1229d2b1594d4f9eeabfd00ae5.camel@kernel.org>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?xD4OeIQkOCdPofApc4NI/vFuSWSGjJQvJmF2si+Ewv7ehJ+MX0xAuM1k8OYk?=
 =?us-ascii?Q?xKLVN9AxaAtQk4NaTjP1Jhevjxzb11uPfk3g3MkU7h8tJ9izXA5Dy6KGwRxY?=
 =?us-ascii?Q?YEwUl3W6chmQND6DVEQXbgOe3EXSKX60lAKS3OzgPh4lM89xBz86op8bOSgw?=
 =?us-ascii?Q?MDsfPlZUu70ofek1j0Ld+MOQ7h6Cdpu/DbhPauABCkeYbRlDa+qs18rq1xiS?=
 =?us-ascii?Q?GOaOznUD5zo5rBjQZ6V2r44ahpBHkuahGZlYaqpumb55RMoeKt0qsMq2kPs9?=
 =?us-ascii?Q?vNus2jyBUqusxCSCX80NKF2SEutpq+JP5xeef44Si1UHtaLgA7TmBv9iAA6a?=
 =?us-ascii?Q?lNjg6O5wRQI3TKzxHXpqLg04h2Lq4e2dYA3yqAQ+PIZhnVfAXTbIi5TShUkU?=
 =?us-ascii?Q?lrI+X0cQYtVoJ/7ddLTV0CpJpFrPZ69cugM82YShTrQ9DCoghgORxqffDGse?=
 =?us-ascii?Q?Tl0Yygx0tkeuKwmrBtxxqwgadWY4poTYl3zoadKOh8wfrVZSOoxs/4nzNaEY?=
 =?us-ascii?Q?G3myF4Cs2vb0omt1x2orxELlAN3jSF84oYt3uuAI3s8JZyBmcEdY8SqbXgG1?=
 =?us-ascii?Q?jMO0dgxwm3Vsak/nEboL9ePWuJmwrJQ3w1CD8smYbvv4kGdQc0e/W52Atfac?=
 =?us-ascii?Q?aQ6Lb73We+yaWvC+94bOHVSJeFnHmvtC18zJTRu6DMblcr06Dh2LcXHkmgY4?=
 =?us-ascii?Q?iX4Yk+APIk6VkolRuRizey50NvB8Lk+FYkUS8cXa8uB1wfG6cA4ob2gWV5Ed?=
 =?us-ascii?Q?w4+ktADutFvMIMGcp02iv/owWU/1r2cTO3aP9jbOnkO30rH7bLb1Cghmvbgg?=
 =?us-ascii?Q?oDNyyLjX/NMZ6WTe0JPwDl+PPiwejXH1IJJxTkvKfYux7K+2EPGoeRPLDe5A?=
 =?us-ascii?Q?Ku3fexc99iKnLC99pQXXEHDmDsXJ8oWrwYMwchO7R7e7xgPiXUi0P6dD4hIi?=
 =?us-ascii?Q?g83OVWyobxyWR52ItOce057kJ/PjwwUnc5Jii0S6TBMBnRlZW0DHKmx4/fEo?=
 =?us-ascii?Q?nnKsOFx5002bqauC1Q8iYNCcgO61p4CbSVJ4R9Rfc41LJ3V8/J4cJUWIMhhT?=
 =?us-ascii?Q?YisXOXXbaQdm9oX6OB6QZus0ESiEVcnU1Bm5iOJktMQN/881owiO86+qwqLP?=
 =?us-ascii?Q?JR9iK79n9h8728U5JGJiiYduWyK7pedEWtzCY0Ghdhp9SB5KKQBj7SphYDoC?=
 =?us-ascii?Q?NSJtHy+h64Sql87yNk4riB4MbXAqlpswDkrA71E57tG7KeqhIhHPsF3AOYq2?=
 =?us-ascii?Q?DtWgwKTE/T5RFFjdmwsXM38b8O/da3uhRs8UfIXZ0orOoDLIohu6mGnPzwyQ?=
 =?us-ascii?Q?zF91JLu0j9r1XKDdeu1kzGzqz/6D4cyxg+UqeoGLYNeIkgI/krY+TpZxeQmZ?=
 =?us-ascii?Q?dpf+kQ0YaujHbaf5fC6Mt4bL2UFsX5IzWON9escnWonqWN5ouUnaZ5/Nl5V+?=
 =?us-ascii?Q?SiR8ytbZl0VrmQ1pM377y3UDVUcrW1EVHKRzBGdpDGGIdloJdRlJhs+Tdy8S?=
 =?us-ascii?Q?g8S7A1gQwx0KdtxOq4J8fMNQ4ArjVTAid9Yajjpf3xrFrWtrM9+rbznA2cls?=
 =?us-ascii?Q?wbQNntHd7eg7AexAbb2/8xOmNtmcQp8NQrDAithX6bHiGL6vH5kgnLvuC24C?=
 =?us-ascii?Q?zQ=3D=3D?=
Content-Type: text/plain; charset="us-ascii"
Content-ID: <867C4BF97E8E264183F5E00EFA6CDFBC@namprd10.prod.outlook.com>
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
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: 526de93e-2500-48ff-bd5f-08da797977bd
X-MS-Exchange-CrossTenant-originalarrivaltime: 08 Aug 2022 20:06:23.2805
 (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: +66MpZkkIlX6NOgkM4dqkdz/5y8/ZLem9PYkFHbW+RtzU7GWq05YXoAn1Q+XS2ZP7K6s3ed9iBatx5zUXiqMyA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BYAPR10MB3031
X-Proofpoint-Virus-Version: vendor=baseguard
 engine=ICAP:2.0.205,Aquarius:18.0.883,Hydra:6.0.517,FMLib:17.11.122.1
 definitions=2022-08-08_13,2022-08-08_01,2022-06-22_01
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 mlxscore=0 bulkscore=0 mlxlogscore=906
 spamscore=0 suspectscore=0 adultscore=0 phishscore=0 malwarescore=0
 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2206140000
 definitions=main-2208080088
X-Proofpoint-GUID: BcXj5wYs882bXKWtj_wjLTztLCyb4vj4
X-Proofpoint-ORIG-GUID: BcXj5wYs882bXKWtj_wjLTztLCyb4vj4
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_H2,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 Aug 8, 2022, at 2:48 PM, Jeff Layton <jlayton@kernel.org> wrote:
>=20
> On Mon, 2022-08-08 at 09:52 -0400, Chuck Lever wrote:
>> This RFC series adds logic to the Linux NFS server to make it wait a
>> few moments for clients to return a delegation before replying with
>> NFS4ERR_DELAY. There are two main benefits:
>>=20
>> - This improves responsiveness when a delegated file is accessed
>> from another client
>> - This removes an unnecessary latency if a client has neglected to
>> return a delegation before attempting a RENAME or SETATTR
>>=20
>> This series is incomplete:
>> - There are likely other operations that can benefit (eg. OPEN)

> Does REMOVE need similar treatment? Does the Apple client return a
> delegation before attempting to unlink?

It's certainly plausible that REMOVE might trigger a delegation recall,
but I haven't yet seen this happen on the wire.


--
Chuck Lever