diff options
author | Kent Overstreet <kmo@daterainc.com> | 2014-02-27 20:51:12 -0500 |
---|---|---|
committer | Kent Overstreet <kmo@daterainc.com> | 2014-03-18 15:24:54 -0400 |
commit | 3a2fd9d5090b83aab85378a846fa10f39b0b5aa7 (patch) | |
tree | 13862a06f381010ba9716441d4f844d953472670 /drivers/md/bcache/extents.c | |
parent | 2531d9ee61fa08a5a9ab8f002c50779888d232c7 (diff) |
bcache: Kill bucket->gc_gen
gc_gen was a temporary used to recalculate last_gc, but since we only need
bucket->last_gc when gc isn't running (gc_mark_valid = 1), we can just update
last_gc directly.
Signed-off-by: Kent Overstreet <kmo@daterainc.com>
Diffstat (limited to 'drivers/md/bcache/extents.c')
-rw-r--r-- | drivers/md/bcache/extents.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/drivers/md/bcache/extents.c b/drivers/md/bcache/extents.c index 82d5e3288a6c..35887330b49d 100644 --- a/drivers/md/bcache/extents.c +++ b/drivers/md/bcache/extents.c | |||
@@ -194,9 +194,9 @@ err: | |||
194 | mutex_unlock(&b->c->bucket_lock); | 194 | mutex_unlock(&b->c->bucket_lock); |
195 | bch_extent_to_text(buf, sizeof(buf), k); | 195 | bch_extent_to_text(buf, sizeof(buf), k); |
196 | btree_bug(b, | 196 | btree_bug(b, |
197 | "inconsistent btree pointer %s: bucket %zi pin %i prio %i gen %i last_gc %i mark %llu gc_gen %i", | 197 | "inconsistent btree pointer %s: bucket %zi pin %i prio %i gen %i last_gc %i mark %llu", |
198 | buf, PTR_BUCKET_NR(b->c, k, i), atomic_read(&g->pin), | 198 | buf, PTR_BUCKET_NR(b->c, k, i), atomic_read(&g->pin), |
199 | g->prio, g->gen, g->last_gc, GC_MARK(g), g->gc_gen); | 199 | g->prio, g->gen, g->last_gc, GC_MARK(g)); |
200 | return true; | 200 | return true; |
201 | } | 201 | } |
202 | 202 | ||
@@ -515,9 +515,9 @@ err: | |||
515 | mutex_unlock(&b->c->bucket_lock); | 515 | mutex_unlock(&b->c->bucket_lock); |
516 | bch_extent_to_text(buf, sizeof(buf), k); | 516 | bch_extent_to_text(buf, sizeof(buf), k); |
517 | btree_bug(b, | 517 | btree_bug(b, |
518 | "inconsistent extent pointer %s:\nbucket %zu pin %i prio %i gen %i last_gc %i mark %llu gc_gen %i", | 518 | "inconsistent extent pointer %s:\nbucket %zu pin %i prio %i gen %i last_gc %i mark %llu", |
519 | buf, PTR_BUCKET_NR(b->c, k, ptr), atomic_read(&g->pin), | 519 | buf, PTR_BUCKET_NR(b->c, k, ptr), atomic_read(&g->pin), |
520 | g->prio, g->gen, g->last_gc, GC_MARK(g), g->gc_gen); | 520 | g->prio, g->gen, g->last_gc, GC_MARK(g)); |
521 | return true; | 521 | return true; |
522 | } | 522 | } |
523 | 523 | ||