ACK/Cmnt: [SRU][j/gke][f/gke][PATCH 0/1] Make new NFS cache behaviour opt-in

[Tim Gardner]

On 6/1/23 1:09 PM, Khalid Elmously wrote:
> BugLink: https://bugs.launchpad.net/bugs/2022098
> 
> The fix to LP #2003053 has caused massively increased NFS server access in some use-cases, which caused severe performance degradation to the point of being unusable.
> 
> The solution to this issue, at least temporarily and at least for linux-gke, is to make the new behaviour optional using the "nfs_fasc=1" module parameter. Without this parameter specified, (or specified as =0) will keep the old behaviour.
> 
> 
> Thanks Thadeu for the feedback and the original inspiration for this fix.
> 
> 
> Testing:
>   - Used a slightly modified version of this patch (which simply logs whether the new or the old behaviour is taking effect) on an NFS client to confirm that 'modprobe nfs' and 'modprobe nfs nfs_fasc=0' maintain the old behaviour, and that 'modprobe nfs nfs_fasc=1' implements the new cache behaviour.
> 
> 
> Regresion potential:
>   - Regression potential is considered low considering the scope of the change and is limited to NFS only.
> 
> 
> 
> 
> Khalid Elmously (1):
>    UBUNTU: SAUCE: Make NFS file-access stale cache behaviour opt-in
> 
>   fs/nfs/dir.c | 29 +++++++++++++++++++++--------
>   1 file changed, 21 insertions(+), 8 deletions(-)
> 
Acked-by: Tim Gardner <tim.gardner at canonical.com>

+1 for applying to master kernels focal through mantic.
-- 
-----------
Tim Gardner
Canonical, Inc