Jeffle Xu <jefflexu@xxxxxxxxxxxxxxxxx> wrote: > +static int init_close_req(struct cachefiles_req *req, void *private) "cachefiles_" prefix please. > + /* > + * It's possible if the cookie looking up phase failed before READ > + * request has ever been sent. > + */ What "it" is possible? You might want to say "It's possible that the cookie..." > + if (fd == 0) > + return -ENOENT; 0 is a valid fd. David
