On 4/11/22 8:44 PM, David Howells wrote:
> Jeffle Xu <jeffl...@linux.alibaba.com> wrote:
> 
>>      /*
>>       * Uninstall anon_fd to the cachefiles object, so that no further
>>       * associated requests will get enqueued.
>>       */
> 
> "Uninstall anon_fd from..."?

Okay, will be fixed.

> 
>> +static int init_read_req(struct cachefiles_req *req, void *private)
> 
> Prefix with "cachefiles_" please (or "cachefiles_ondemand_").

Alright.

Thanks for reviewing.

-- 
Thanks,
Jeffle

--
Linux-cachefs mailing list
Linux-cachefs@redhat.com
https://listman.redhat.com/mailman/listinfo/linux-cachefs

Reply via email to