diff options
author | David Woodhouse <dwmw2@infradead.org> | 2006-05-22 10:23:10 -0400 |
---|---|---|
committer | David Woodhouse <dwmw2@infradead.org> | 2006-05-22 10:23:10 -0400 |
commit | fcb7578719529898aef9edce8e409e457a1c2d15 (patch) | |
tree | 58022b22d175b69e4b267f2f19ab26d93f7d4910 /fs/jffs2/erase.c | |
parent | a1b563d652b54647ffacb2d6edf7859d3e97a723 (diff) |
[JFFS2] Extend jffs2_link_node_ref() to link into per-inode list too.
Let's avoid the potential for forgetting to set ref->next_in_ino, by doing
it within jffs2_link_node_ref() instead.
This highlights the ugliness of what we're currently doing with
xattr_datum and xattr_ref structures -- we should find a nicer way of
dealing with that.
Signed-off-by: David Woodhouse <dwmw2@infradead.org>
Diffstat (limited to 'fs/jffs2/erase.c')
-rw-r--r-- | fs/jffs2/erase.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/fs/jffs2/erase.c b/fs/jffs2/erase.c index f677d6950fd4..0fc19a2fb5d9 100644 --- a/fs/jffs2/erase.c +++ b/fs/jffs2/erase.c | |||
@@ -410,10 +410,9 @@ static void jffs2_mark_erased_block(struct jffs2_sb_info *c, struct jffs2_eraseb | |||
410 | /* Everything else got zeroed before the erase */ | 410 | /* Everything else got zeroed before the erase */ |
411 | jeb->free_size = c->sector_size; | 411 | jeb->free_size = c->sector_size; |
412 | 412 | ||
413 | marker_ref->next_in_ino = NULL; | ||
414 | marker_ref->flash_offset = jeb->offset | REF_NORMAL; | 413 | marker_ref->flash_offset = jeb->offset | REF_NORMAL; |
415 | 414 | ||
416 | jffs2_link_node_ref(c, jeb, marker_ref, c->cleanmarker_size); | 415 | jffs2_link_node_ref(c, jeb, marker_ref, c->cleanmarker_size, NULL); |
417 | } | 416 | } |
418 | 417 | ||
419 | spin_lock(&c->erase_completion_lock); | 418 | spin_lock(&c->erase_completion_lock); |