Received-SPF: pass (google.com: domain of linux-nfs-owner@vger.kernel.org designates 23.128.96.37 as permitted sender) client-ip=23.128.96.37;
Date:   Fri, 15 Sep 2023 11:57:43 -0400
From:   Chuck Lever <chuck.lever@oracle.com>
To:     Dai Ngo <dai.ngo@oracle.com>
Cc:     jlayton@kernel.org, linux-nfs@vger.kernel.org
Subject: Re: [PATCH v2 0/2] NFSD: use CB_GETATTR to handle GETATTR conflict
 with write delegation
Message-ID: <ZQR+95ZeXn1pLivt@tissot.1015granger.net>
References: <1694648301-26746-1-git-send-email-dai.ngo@oracle.com>
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <1694648301-26746-1-git-send-email-dai.ngo@oracle.com>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?J6BYwPhB9cCPKuy02sN5dYVo89ZONdj4yFDEfuuHhxDcredc/eOV9z5Vzmv8?=
 =?us-ascii?Q?EXWp4x3qEas0qYMEecAeRn5dqyTVe8eBkaDIFp/cWiDIUMLxTGScH7B+0rFt?=
 =?us-ascii?Q?W4e9XTeRrF8abWwd6INzMv2r/9uxgSAET0np2AVgVZRlB+L7uXEoxkzJwtle?=
 =?us-ascii?Q?Tuyqj6yOcg18ZcfIoK4Z/Sp9JF/Cjmo0TPyy7csG+aWXmeOdAKR5ZVbwFgww?=
 =?us-ascii?Q?KuExyv2eKxhb/TuhRkoWjv9WhhNDuBJBhy+IQYolUkUm2K7qYfxvFvHEJYma?=
 =?us-ascii?Q?t7bcrQVhSo00MfvspCK57qjCwo6HM5g6OKVLBNNz8+qcDtgS1LAsB2yG3/lm?=
 =?us-ascii?Q?k6bgR5DMQgEPm+/DidrhaWmjCs84qRYDdMmIL6m5mMEmmo/o1kazwzA9+Xzs?=
 =?us-ascii?Q?DqywdE/nKvNiIa7JRJYppdPxQjW4WzzQYL6K1xT7tzikQOFsyRYAxIbiQUB/?=
 =?us-ascii?Q?WBQLJYnBI2dKm9cuNIyg3v5sosMMvhaFxtHuB6PZXugO0oCu+k4+djcl+QMw?=
 =?us-ascii?Q?41qnHODxS1J17V73pBhaYYvqhwY6uKJjyG4wKaIKzr1kSsDsnAxFCP/0HWSK?=
 =?us-ascii?Q?fz/LnAiA224ZBkcVXPmu5KYQ9Cnq5gKWXxJR4Ra6wmKWxD/Mo7zluFNoQgf8?=
 =?us-ascii?Q?LSnp9WS8sywWctkJJuGJAneqiUS9jB9s1NYuDh0UWuDuH+dAgzF2TTasqNjs?=
 =?us-ascii?Q?gnsMBOwXvbLvo2AU5tDijxnedzjrmC9KF2BqKb8h5cpSfVelkwu8cKd+lNOA?=
 =?us-ascii?Q?gYTkcHCn0G8AIw6qxYj+NuXh27RJ2JuxkQGWkpZ0VcdQoYVO0plRy/WY/lGt?=
 =?us-ascii?Q?cUFOCaCbRGJDkgSRdONhsNftlYeyVTk3F9n+Xwdi6SrgxmGgUZuNmcnpsW7b?=
 =?us-ascii?Q?sMLjgI8kjCqlc6k2DtUYWauaiHqAkHG6s/j8CcIFh9LuY+avTuZGsWMX+ZeJ?=
 =?us-ascii?Q?R73MQCZVNh+AU7iXwW/Q2BeISIBtN+jWEzFsCbBX/o1wELwtjt//DxJIMajK?=
 =?us-ascii?Q?2I2aP99RYmWQ6QobU4G0Zrhmv6xWXyoSE5i0L7yw++UcX/ULGm/FW2btvLY5?=
 =?us-ascii?Q?HENs3WZTGx6plfPGY7CmVyAS5Ll1SLX9gzip7VDnjSTqLk1tuc6tCtF5SLaD?=
 =?us-ascii?Q?bSY+R3sOF9ZPSTZ+eR2gdE6/UVBtCHi3X1tMyyFBHh2kOM4hmdh93PLt1Yh5?=
 =?us-ascii?Q?EzVUaMY9njHWrRPt8FEZuPsK837RajtEdh9TtYWKbai5SEr95kA4cmA1q8ew?=
 =?us-ascii?Q?2bfnj+0LeiDicQUCs1EIlQ9vh6Q/u7cEK6XnGRcMtD2fRNQrxhTfLHpjaWW7?=
 =?us-ascii?Q?IHwig+yHXQY86eksqiIG1Irg59H69ctHck7NzWAANtn4db0XiQ0B6xwkUDQf?=
 =?us-ascii?Q?IMDrFs97YNEie3mdKJ89OUBri1FjBFBSTe7yxyiDMkQ6GY1qxPCAL3BG0bFI?=
 =?us-ascii?Q?zztvpCGqlnZoMTXTqT2HSbqZw8L2k63OPSC0qVHmmJJWSXHzX8Ca8UfBIpHY?=
 =?us-ascii?Q?VjXcrr/sjMHUQ00vFF28dv9lsG/A4cOtEYCztEVLQ+tOtdISPq+CQMDyb69j?=
 =?us-ascii?Q?JjkBp1YQf4SsdGYvmgLfUeG33wBFSwrKAXsQxe/lf1OHg48VpHB8Sy/55i60?=
 =?us-ascii?Q?uA=3D=3D?=
X-MS-Exchange-AntiSpam-ExternalHop-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-ExternalHop-MessageData-0: Co5g/E7zZw0O6AoeGsDcLh5JD84hALWCcmp5VUYU4T1oY6X8DC+Vl4IVXh0LLw08PpirJB0b3h85Jytn7gkUm1KBkTe6o5E18wdpAGXsyttMdKajJ2olfnDXbI0fsKnXzMCvKFXPHPL1Rqd1R07sbFp7+Qyaz9AOnw8WB++C4pMmJ7C4KCiFj38GMYwMhU1Nhv9VlnqTalnArnmJibAv0CUZHs9QsNc8PXn+CE3cY5knD70w+eEJoUnrTjYs/tWtr7nNajVIjVZqqikhpf97cRICEnog3CpuKhPL1JDiNVeOmSMYCwyN9bQPhJEEemvvT18Jbk8lDayaAORu4edxn4IhmMWM0vZBvhngGQe5qT1/mEFQKaJM5iBg2yFE4DsDJbZkkM8v78tphcsXQcdkmhZs7UuefVdytd1MZT/R1fD3tLBynNYgMCfHDsECgjipgHslUP2enzKBaFt67ZQOx3pBxMTWT4MNDyGO13LoUrg1KRB1EoJ9DSXyE9n321Ey4VN8Kt8aEKiU7xk1c7AovP1a7m9uShrJTau8cS67Lb0AU4zbbUOVZM+eZyagD7TsbmDsU5X+sVAn14f9KEPzg/yyHdcAeJIGLzy8N+encFhYGUxNsmcrLxrBZmQN7HFINjc+TpibEjKTK34LUT4jNr6OOhiIzBhGD9mQUZ9uTCrpIE5E8xvftAC5aDjzqGvxVAR/IyMEsK8MGnbbssUBX1RjYs+XwSQEOn7SWzcp2UShfDWAovM4P83x9KRH6EcKPYoz9YVh0H794Wm7BHm4oigFQcRjrj6w0xlHwhKAHn4=
X-OriginatorOrg: oracle.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 0e4fdde6-8f5d-4496-bbad-08dbb604812f
X-MS-Exchange-CrossTenant-AuthSource: BN0PR10MB5128.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 15 Sep 2023 15:57:46.7521
 (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: zsG7yPq/nV0rM/ncJ+C4MeHUBaS+QUhJes/oDTBOstZouYVYphrV0jqnVgx3NPYa41bpFfhnj1JY5ZTFCgUtgw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SJ1PR10MB5954
X-Proofpoint-Virus-Version: vendor=baseguard
 engine=ICAP:2.0.267,Aquarius:18.0.980,Hydra:6.0.601,FMLib:17.11.176.26
 definitions=2023-09-15_12,2023-09-15_01,2023-05-22_02
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 malwarescore=0 mlxscore=0 bulkscore=0
 adultscore=0 phishscore=0 spamscore=0 mlxlogscore=694 suspectscore=0
 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2308100000
 definitions=main-2309150142
X-Proofpoint-GUID: bkURgmJJgsBx0td-M1kBqU2yp4UrQQHw
X-Proofpoint-ORIG-GUID: bkURgmJJgsBx0td-M1kBqU2yp4UrQQHw
X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_BLOCKED,
        RCVD_IN_MSPIKE_H5,RCVD_IN_MSPIKE_WL,SPF_HELO_NONE,SPF_NONE
        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
X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (snail.vger.email [0.0.0.0]); Fri, 15 Sep 2023 08:58:32 -0700 (PDT)

On Wed, Sep 13, 2023 at 04:38:19PM -0700, Dai Ngo wrote:
> Currently GETATTR conflict with a write delegation is handled by
> recalling the delegation before replying to the GETATTR.
> 
> This patch series add supports for CB_GETATTR callback to get the latest
> change_info and size information of the file from the client that holds
> the delegation to reply to the GETATTR from the second client.
> 
> v2:
>   . fix kernel test robot report of missing function parameter description
>     of nfsd4_deleg_getattr_conflict()

This series has been applied to nfsd-next for broader testing.
Thank you, Dai!

-- 
Chuck Lever