* [PATCH] FS-Cache: make check_consistency callback return int
@ 2016-05-20 10:53 Yan, Zheng
2016-05-31 20:36 ` David Howells
0 siblings, 1 reply; 2+ messages in thread
From: Yan, Zheng @ 2016-05-20 10:53 UTC (permalink / raw)
To: linux-kernel; +Cc: dhowells, linux-cachefs, Yan, Zheng
__fscache_check_consistency() calls check_consistency() callback
and return the callback's return value. But the return type of
check_consistency() is bool. So __fscache_check_consistency()
return 1 if the cache is inconsistent. This is inconsistent with
the document.
Signed-off-by: "Yan, Zheng" <zyan@redhat.com>
---
fs/cachefiles/interface.c | 2 +-
include/linux/fscache-cache.h | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/fs/cachefiles/interface.c b/fs/cachefiles/interface.c
index 861d611..ce5f345 100644
--- a/fs/cachefiles/interface.c
+++ b/fs/cachefiles/interface.c
@@ -380,7 +380,7 @@ static void cachefiles_sync_cache(struct fscache_cache *_cache)
* check if the backing cache is updated to FS-Cache
* - called by FS-Cache when evaluates if need to invalidate the cache
*/
-static bool cachefiles_check_consistency(struct fscache_operation *op)
+static int cachefiles_check_consistency(struct fscache_operation *op)
{
struct cachefiles_object *object;
struct cachefiles_cache *cache;
diff --git a/include/linux/fscache-cache.h b/include/linux/fscache-cache.h
index 604e152..13ba552 100644
--- a/include/linux/fscache-cache.h
+++ b/include/linux/fscache-cache.h
@@ -241,7 +241,7 @@ struct fscache_cache_ops {
/* check the consistency between the backing cache and the FS-Cache
* cookie */
- bool (*check_consistency)(struct fscache_operation *op);
+ int (*check_consistency)(struct fscache_operation *op);
/* store the updated auxiliary data on an object */
void (*update_object)(struct fscache_object *object);
--
2.5.5
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] FS-Cache: make check_consistency callback return int
2016-05-20 10:53 [PATCH] FS-Cache: make check_consistency callback return int Yan, Zheng
@ 2016-05-31 20:36 ` David Howells
0 siblings, 0 replies; 2+ messages in thread
From: David Howells @ 2016-05-31 20:36 UTC (permalink / raw)
To: Yan, Zheng; +Cc: dhowells, linux-kernel, linux-cachefs
Yan, Zheng <zyan@redhat.com> wrote:
> __fscache_check_consistency() calls check_consistency() callback
> and return the callback's return value. But the return type of
> check_consistency() is bool. So __fscache_check_consistency()
> return 1 if the cache is inconsistent. This is inconsistent with
> the document.
Acked-by: David Howells <dhowells@redhat.com>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2016-05-31 20:36 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-05-20 10:53 [PATCH] FS-Cache: make check_consistency callback return int Yan, Zheng
2016-05-31 20:36 ` David Howells
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).