On 2017/7/1 上午4:42, bca...@lists.ewheeler.net wrote:
> From: Tang Junhui <tang.jun...@zte.com.cn>
>
> Some missed IOs are not counted into cache_misses, this patch fix this
> issue.
Could you please explain more about,
- which kind of missed I/O are mot counted
- where cache_missed is located
This will help the patch to be more understandable.
>
> Signed-off-by: tang.junhui <tang.jun...@zte.com.cn>
> Reviewed-by: Eric Wheeler <bca...@linux.ewheeler.net>
> Cc: sta...@vger.kernel.org
[snip]
> @@ -758,7 +760,7 @@ static void cached_dev_read_done_bh(struct closure *cl)
> struct cached_dev *dc = container_of(s->d, struct cached_dev, disk);
>
> bch_mark_cache_accounting(s->iop.c, s->d,
> - !s->cache_miss, s->iop.bypass);
> + !s->cache_missed, s->iop.bypass);
> trace_bcache_read(s->orig_bio, !s->cache_miss, s->iop.bypass);
Should the above line be changed to,
trace_bcache_read(s->orig_bio, !s->cache_missed, s->iop.bypass);
as well ?
[snip]
Thanks.
--
Coly Li
