lkml.org 
[lkml]   [2022]   [Mar]   [18]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: [PATCH v4 13/20] netfs: Add a netfs inode context
Date
Jeff Layton <jlayton@kernel.org> wrote:

> > +static inline bool netfs_is_cache_enabled(struct netfs_i_context *ctx)
> > +{
> > +#if IS_ENABLED(CONFIG_FSCACHE)
> > + struct fscache_cookie *cookie = ctx->cache;
> > +
> > + return fscache_cookie_valid(cookie) && cookie->cache_priv &&
> > + fscache_cookie_enabled(cookie);
>
>
> As you mentioned in the other thread, it may be cleaner to move the
> cookie->cache_priv check into fscache_cookie_enabled. Is there ever a
> case where you'd need to separate the two checks?

I'm not sure, but I'd prefer not to do it in this series as it would affect
NFS plus some other operations, so will need retesting thoroughly. I'd prefer
to defer it.

David

\
 
 \ /
  Last update: 2022-03-18 15:49    [W:3.068 / U:0.316 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site