在 2023/4/14 21:53, David Howells 写道:
Jia Zhu <zhujia.zj@xxxxxxxxxxxxx> wrote:+int cachefiles_ondemand_restore(struct cachefiles_cache *cache, char *args) +{ + struct cachefiles_req *req; + + XA_STATE(xas, &cache->reqs, 0); + + if (!test_bit(CACHEFILES_ONDEMAND_MODE, &cache->flags)) + return -EOPNOTSUPP; + + /* + * Reset the requests to CACHEFILES_REQ_NEW state, so that the + * requests have been processed halfway before the crash of the + * user daemon could be reprocessed after the recovery. + */ + xas_lock(&xas); + xas_for_each(&xas, req, ULONG_MAX) + xas_set_mark(&xas, CACHEFILES_REQ_NEW); + xas_unlock(&xas); + + wake_up_all(&cache->daemon_pollwq); + return 0; +}Should there be a check to see if this is needed? David
I've considered whether to add a check here, since the user could invoke 'restore' through ioctl at any time. If 'restore' is called without user daemon crashing. Then the req being processed by the user daemon will be reset to CACHEFILES_REQ_NEW and could be re-read and processed by the user daemon. For OPEN req: The user daemon will ignore this repeated req since the related anonymous fd is the same. For READ req: The user daemon will read the same part of the data again and write it to fscache (this will not cause any error) For CLOSE req: Actually the user daemon will close this anonymous fd, but fortunately [PATCH 3/5] could handle this case by reopening it automatically. Thus "restore" could be called at any time without error, checks might introduce extra race conditions. Jia
-- Linux-cachefs mailing list Linux-cachefs@xxxxxxxxxx https://listman.redhat.com/mailman/listinfo/linux-cachefs
