Re: [PATCH] cifs: cache dirent names for cached directories

[Enzo Matsumiya <ematsumiya@xxxxxxx>]

Just another nitpick

On 05/04, Ronnie Sahlberg wrote:
<snip>
> @@ -776,7 +791,8 @@ int open_cached_dir(unsigned int xid, struct cifs_tcon *tcon,
> 	struct cifs_fid *pfid;
> 	struct dentry *dentry;
>
> -	if (tcon->nohandlecache)
> +	if (tcon == NULL || tcon->nohandlecache ||
> +	    is_smb1_server(tcon->ses->server))
> 		return -ENOTSUPP;
>
> 	if (cifs_sb->root == NULL)

This last hunk looks unrelated to the original topic. Could it be sent
as a separate patch please? This helps tracking when doing
backports/bisects.


Cheers,

Enzo