diff options
| author | Steven Whitehouse <swhiteho@redhat.com> | 2012-04-16 19:40:56 +0400 |
|---|---|---|
| committer | Steven Whitehouse <swhiteho@redhat.com> | 2012-04-24 19:44:36 +0400 |
| commit | c50b91c4bd511dfe844e2aa7be429b6b88406353 (patch) | |
| tree | 0a9584ec238036b599bb717db5bd1a7df5c616ca /fs/gfs2/aops.c | |
| parent | dad30e9031c5927c30b402f73ac57ffbe09dc9ee (diff) | |
| download | linux-c50b91c4bd511dfe844e2aa7be429b6b88406353.tar.xz | |
GFS2: Remove bd_list_tr
This is another clean up in the logging code. This per-transaction
list was largely unused. Its main function was to ensure that the
number of buffers in a transaction was correct, however that counter
was only used to check the number of buffers in the bd_list_tr, plus
an assert at the end of each transaction. With the assert now changed
to use the calculated buffer counts, we can remove both bd_list_tr and
its associated counter.
This should make the code easier to understand as well as shrinking
a couple of structures.
Signed-off-by: Steven Whitehouse <swhiteho@redhat.com>
Diffstat (limited to 'fs/gfs2/aops.c')
| -rw-r--r-- | fs/gfs2/aops.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/fs/gfs2/aops.c b/fs/gfs2/aops.c index 56dc1f0084ba..a627b3ee5c9f 100644 --- a/fs/gfs2/aops.c +++ b/fs/gfs2/aops.c @@ -1084,7 +1084,6 @@ int gfs2_releasepage(struct page *page, gfp_t gfp_mask) bd = bh->b_private; if (bd) { gfs2_assert_warn(sdp, bd->bd_bh == bh); - gfs2_assert_warn(sdp, list_empty(&bd->bd_list_tr)); if (!list_empty(&bd->bd_le.le_list)) { if (!buffer_pinned(bh)) list_del_init(&bd->bd_le.le_list); |