diff options
author | KaiGai Kohei <kaigai@ak.jp.nec.com> | 2006-06-10 21:35:15 -0400 |
---|---|---|
committer | David Woodhouse <dwmw2@infradead.org> | 2006-06-27 11:16:26 -0400 |
commit | c9f700f840bd481b3e01fcad1ba8da01794a6726 (patch) | |
tree | 786f4a6a3f3a2d619ae4230a62f95361fcd3245a | |
parent | 6d4f8224d4ee065e0e3186cc554468d735e6015d (diff) |
[JFFS2][XATTR] using 'delete marker' for xdatum/xref deletion
- When xdatum is removed, a new xdatum with 'delete marker' is
written. (version==0xffffffff means 'delete marker')
- When xref is removed, a new xref with 'delete marker' is written.
(odd-numbered xseqno means 'delete marker')
- delete_xattr_(datum/xref)_delay() are new deletion functions
are added. We can only use them if we can detect the target
obsolete xdatum/xref as a orphan or errir one.
(e.g when inode deletion, or detecting crc error)
[1/3] jffs2-xattr-v6-01-delete_marker.patch
Signed-off-by: KaiGai Kohei <kaigai@ak.jp.nec.com>
Signed-off-by: David Woodhouse <dwmw2@infradead.org>
-rw-r--r-- | fs/jffs2/erase.c | 19 | ||||
-rw-r--r-- | fs/jffs2/gc.c | 5 | ||||
-rw-r--r-- | fs/jffs2/jffs2_fs_sb.h | 3 | ||||
-rw-r--r-- | fs/jffs2/malloc.c | 2 | ||||
-rw-r--r-- | fs/jffs2/nodemgmt.c | 21 | ||||
-rw-r--r-- | fs/jffs2/scan.c | 57 | ||||
-rw-r--r-- | fs/jffs2/summary.c | 39 | ||||
-rw-r--r-- | fs/jffs2/xattr.c | 736 | ||||
-rw-r--r-- | fs/jffs2/xattr.h | 40 | ||||
-rw-r--r-- | include/linux/jffs2.h | 1 |
10 files changed, 584 insertions, 339 deletions
diff --git a/fs/jffs2/erase.c b/fs/jffs2/erase.c index 1862e8bc101d..1644e3408852 100644 --- a/fs/jffs2/erase.c +++ b/fs/jffs2/erase.c | |||
@@ -230,7 +230,6 @@ static inline void jffs2_remove_node_refs_from_ino_list(struct jffs2_sb_info *c, | |||
230 | at the end of the linked list. Stash it and continue | 230 | at the end of the linked list. Stash it and continue |
231 | from the beginning of the list */ | 231 | from the beginning of the list */ |
232 | ic = (struct jffs2_inode_cache *)(*prev); | 232 | ic = (struct jffs2_inode_cache *)(*prev); |
233 | BUG_ON(ic->class != RAWNODE_CLASS_INODE_CACHE); | ||
234 | prev = &ic->nodes; | 233 | prev = &ic->nodes; |
235 | continue; | 234 | continue; |
236 | } | 235 | } |
@@ -254,7 +253,8 @@ static inline void jffs2_remove_node_refs_from_ino_list(struct jffs2_sb_info *c, | |||
254 | 253 | ||
255 | /* PARANOIA */ | 254 | /* PARANOIA */ |
256 | if (!ic) { | 255 | if (!ic) { |
257 | printk(KERN_WARNING "inode_cache not found in remove_node_refs()!!\n"); | 256 | JFFS2_WARNING("inode_cache/xattr_datum/xattr_ref" |
257 | " not found in remove_node_refs()!!\n"); | ||
258 | return; | 258 | return; |
259 | } | 259 | } |
260 | 260 | ||
@@ -279,8 +279,19 @@ static inline void jffs2_remove_node_refs_from_ino_list(struct jffs2_sb_info *c, | |||
279 | printk("\n"); | 279 | printk("\n"); |
280 | }); | 280 | }); |
281 | 281 | ||
282 | if (ic->nodes == (void *)ic && ic->nlink == 0) | 282 | switch (ic->class) { |
283 | jffs2_del_ino_cache(c, ic); | 283 | #ifdef CONFIG_JFFS2_FS_XATTR |
284 | case RAWNODE_CLASS_XATTR_DATUM: | ||
285 | jffs2_release_xattr_datum(c, (struct jffs2_xattr_datum *)ic); | ||
286 | break; | ||
287 | case RAWNODE_CLASS_XATTR_REF: | ||
288 | jffs2_release_xattr_ref(c, (struct jffs2_xattr_ref *)ic); | ||
289 | break; | ||
290 | #endif | ||
291 | default: | ||
292 | if (ic->nodes == (void *)ic && ic->nlink == 0) | ||
293 | jffs2_del_ino_cache(c, ic); | ||
294 | } | ||
284 | } | 295 | } |
285 | 296 | ||
286 | void jffs2_free_jeb_node_refs(struct jffs2_sb_info *c, struct jffs2_eraseblock *jeb) | 297 | void jffs2_free_jeb_node_refs(struct jffs2_sb_info *c, struct jffs2_eraseblock *jeb) |
diff --git a/fs/jffs2/gc.c b/fs/jffs2/gc.c index 477c526d638b..f59b147661c9 100644 --- a/fs/jffs2/gc.c +++ b/fs/jffs2/gc.c | |||
@@ -275,13 +275,12 @@ int jffs2_garbage_collect_pass(struct jffs2_sb_info *c) | |||
275 | * We can decide whether this node is inode or xattr by ic->class. */ | 275 | * We can decide whether this node is inode or xattr by ic->class. */ |
276 | if (ic->class == RAWNODE_CLASS_XATTR_DATUM | 276 | if (ic->class == RAWNODE_CLASS_XATTR_DATUM |
277 | || ic->class == RAWNODE_CLASS_XATTR_REF) { | 277 | || ic->class == RAWNODE_CLASS_XATTR_REF) { |
278 | BUG_ON(raw->next_in_ino != (void *)ic); | ||
279 | spin_unlock(&c->erase_completion_lock); | 278 | spin_unlock(&c->erase_completion_lock); |
280 | 279 | ||
281 | if (ic->class == RAWNODE_CLASS_XATTR_DATUM) { | 280 | if (ic->class == RAWNODE_CLASS_XATTR_DATUM) { |
282 | ret = jffs2_garbage_collect_xattr_datum(c, (struct jffs2_xattr_datum *)ic); | 281 | ret = jffs2_garbage_collect_xattr_datum(c, (struct jffs2_xattr_datum *)ic, raw); |
283 | } else { | 282 | } else { |
284 | ret = jffs2_garbage_collect_xattr_ref(c, (struct jffs2_xattr_ref *)ic); | 283 | ret = jffs2_garbage_collect_xattr_ref(c, (struct jffs2_xattr_ref *)ic, raw); |
285 | } | 284 | } |
286 | goto release_sem; | 285 | goto release_sem; |
287 | } | 286 | } |
diff --git a/fs/jffs2/jffs2_fs_sb.h b/fs/jffs2/jffs2_fs_sb.h index 935fec1b1201..b98594992eed 100644 --- a/fs/jffs2/jffs2_fs_sb.h +++ b/fs/jffs2/jffs2_fs_sb.h | |||
@@ -119,8 +119,11 @@ struct jffs2_sb_info { | |||
119 | #ifdef CONFIG_JFFS2_FS_XATTR | 119 | #ifdef CONFIG_JFFS2_FS_XATTR |
120 | #define XATTRINDEX_HASHSIZE (57) | 120 | #define XATTRINDEX_HASHSIZE (57) |
121 | uint32_t highest_xid; | 121 | uint32_t highest_xid; |
122 | uint32_t highest_xseqno; | ||
122 | struct list_head xattrindex[XATTRINDEX_HASHSIZE]; | 123 | struct list_head xattrindex[XATTRINDEX_HASHSIZE]; |
123 | struct list_head xattr_unchecked; | 124 | struct list_head xattr_unchecked; |
125 | struct list_head xattr_dead_list; | ||
126 | struct jffs2_xattr_ref *xref_dead_list; | ||
124 | struct jffs2_xattr_ref *xref_temp; | 127 | struct jffs2_xattr_ref *xref_temp; |
125 | struct rw_semaphore xattr_sem; | 128 | struct rw_semaphore xattr_sem; |
126 | uint32_t xdatum_mem_usage; | 129 | uint32_t xdatum_mem_usage; |
diff --git a/fs/jffs2/malloc.c b/fs/jffs2/malloc.c index 4889d0700c0e..8310c95478e9 100644 --- a/fs/jffs2/malloc.c +++ b/fs/jffs2/malloc.c | |||
@@ -291,6 +291,7 @@ struct jffs2_xattr_datum *jffs2_alloc_xattr_datum(void) | |||
291 | 291 | ||
292 | memset(xd, 0, sizeof(struct jffs2_xattr_datum)); | 292 | memset(xd, 0, sizeof(struct jffs2_xattr_datum)); |
293 | xd->class = RAWNODE_CLASS_XATTR_DATUM; | 293 | xd->class = RAWNODE_CLASS_XATTR_DATUM; |
294 | xd->node = (void *)xd; | ||
294 | INIT_LIST_HEAD(&xd->xindex); | 295 | INIT_LIST_HEAD(&xd->xindex); |
295 | return xd; | 296 | return xd; |
296 | } | 297 | } |
@@ -309,6 +310,7 @@ struct jffs2_xattr_ref *jffs2_alloc_xattr_ref(void) | |||
309 | 310 | ||
310 | memset(ref, 0, sizeof(struct jffs2_xattr_ref)); | 311 | memset(ref, 0, sizeof(struct jffs2_xattr_ref)); |
311 | ref->class = RAWNODE_CLASS_XATTR_REF; | 312 | ref->class = RAWNODE_CLASS_XATTR_REF; |
313 | ref->node = (void *)ref; | ||
312 | return ref; | 314 | return ref; |
313 | } | 315 | } |
314 | 316 | ||
diff --git a/fs/jffs2/nodemgmt.c b/fs/jffs2/nodemgmt.c index 8bedfd2ff689..01594a2256eb 100644 --- a/fs/jffs2/nodemgmt.c +++ b/fs/jffs2/nodemgmt.c | |||
@@ -684,19 +684,26 @@ void jffs2_mark_node_obsolete(struct jffs2_sb_info *c, struct jffs2_raw_node_ref | |||
684 | spin_lock(&c->erase_completion_lock); | 684 | spin_lock(&c->erase_completion_lock); |
685 | 685 | ||
686 | ic = jffs2_raw_ref_to_ic(ref); | 686 | ic = jffs2_raw_ref_to_ic(ref); |
687 | /* It seems we should never call jffs2_mark_node_obsolete() for | ||
688 | XATTR nodes.... yet. Make sure we notice if/when we change | ||
689 | that :) */ | ||
690 | BUG_ON(ic->class != RAWNODE_CLASS_INODE_CACHE); | ||
691 | for (p = &ic->nodes; (*p) != ref; p = &((*p)->next_in_ino)) | 687 | for (p = &ic->nodes; (*p) != ref; p = &((*p)->next_in_ino)) |
692 | ; | 688 | ; |
693 | 689 | ||
694 | *p = ref->next_in_ino; | 690 | *p = ref->next_in_ino; |
695 | ref->next_in_ino = NULL; | 691 | ref->next_in_ino = NULL; |
696 | 692 | ||
697 | if (ic->nodes == (void *)ic && ic->nlink == 0) | 693 | switch (ic->class) { |
698 | jffs2_del_ino_cache(c, ic); | 694 | #ifdef CONFIG_JFFS2_FS_XATTR |
699 | 695 | case RAWNODE_CLASS_XATTR_DATUM: | |
696 | jffs2_release_xattr_datum(c, (struct jffs2_xattr_datum *)ic); | ||
697 | break; | ||
698 | case RAWNODE_CLASS_XATTR_REF: | ||
699 | jffs2_release_xattr_ref(c, (struct jffs2_xattr_ref *)ic); | ||
700 | break; | ||
701 | #endif | ||
702 | default: | ||
703 | if (ic->nodes == (void *)ic && ic->nlink == 0) | ||
704 | jffs2_del_ino_cache(c, ic); | ||
705 | break; | ||
706 | } | ||
700 | spin_unlock(&c->erase_completion_lock); | 707 | spin_unlock(&c->erase_completion_lock); |
701 | } | 708 | } |
702 | 709 | ||
diff --git a/fs/jffs2/scan.c b/fs/jffs2/scan.c index 61618080b86f..79638f56c5ea 100644 --- a/fs/jffs2/scan.c +++ b/fs/jffs2/scan.c | |||
@@ -317,20 +317,25 @@ static int jffs2_scan_xattr_node(struct jffs2_sb_info *c, struct jffs2_erasebloc | |||
317 | struct jffs2_summary *s) | 317 | struct jffs2_summary *s) |
318 | { | 318 | { |
319 | struct jffs2_xattr_datum *xd; | 319 | struct jffs2_xattr_datum *xd; |
320 | uint32_t totlen, crc; | 320 | uint32_t xid, version, totlen, crc; |
321 | int err; | 321 | int err; |
322 | 322 | ||
323 | crc = crc32(0, rx, sizeof(struct jffs2_raw_xattr) - 4); | 323 | crc = crc32(0, rx, sizeof(struct jffs2_raw_xattr) - 4); |
324 | if (crc != je32_to_cpu(rx->node_crc)) { | 324 | if (crc != je32_to_cpu(rx->node_crc)) { |
325 | if (je32_to_cpu(rx->node_crc) != 0xffffffff) | 325 | JFFS2_WARNING("node CRC failed at %#08x, read=%#08x, calc=%#08x\n", |
326 | JFFS2_WARNING("node CRC failed at %#08x, read=%#08x, calc=%#08x\n", | 326 | ofs, je32_to_cpu(rx->node_crc), crc); |
327 | ofs, je32_to_cpu(rx->node_crc), crc); | ||
328 | if ((err = jffs2_scan_dirty_space(c, jeb, je32_to_cpu(rx->totlen)))) | 327 | if ((err = jffs2_scan_dirty_space(c, jeb, je32_to_cpu(rx->totlen)))) |
329 | return err; | 328 | return err; |
330 | return 0; | 329 | return 0; |
331 | } | 330 | } |
332 | 331 | ||
333 | totlen = PAD(sizeof(*rx) + rx->name_len + 1 + je16_to_cpu(rx->value_len)); | 332 | xid = je32_to_cpu(rx->xid); |
333 | version = je32_to_cpu(rx->version); | ||
334 | |||
335 | totlen = sizeof(struct jffs2_raw_xattr); | ||
336 | if (version != XDATUM_DELETE_MARKER) | ||
337 | totlen += rx->name_len + 1 + je16_to_cpu(rx->value_len); | ||
338 | totlen = PAD(totlen); | ||
334 | if (totlen != je32_to_cpu(rx->totlen)) { | 339 | if (totlen != je32_to_cpu(rx->totlen)) { |
335 | JFFS2_WARNING("node length mismatch at %#08x, read=%u, calc=%u\n", | 340 | JFFS2_WARNING("node length mismatch at %#08x, read=%u, calc=%u\n", |
336 | ofs, je32_to_cpu(rx->totlen), totlen); | 341 | ofs, je32_to_cpu(rx->totlen), totlen); |
@@ -339,22 +344,24 @@ static int jffs2_scan_xattr_node(struct jffs2_sb_info *c, struct jffs2_erasebloc | |||
339 | return 0; | 344 | return 0; |
340 | } | 345 | } |
341 | 346 | ||
342 | xd = jffs2_setup_xattr_datum(c, je32_to_cpu(rx->xid), je32_to_cpu(rx->version)); | 347 | xd = jffs2_setup_xattr_datum(c, xid, version); |
343 | if (IS_ERR(xd)) { | 348 | if (IS_ERR(xd)) |
344 | if (PTR_ERR(xd) == -EEXIST) { | ||
345 | if ((err = jffs2_scan_dirty_space(c, jeb, PAD(je32_to_cpu(rx->totlen))))) | ||
346 | return err; | ||
347 | return 0; | ||
348 | } | ||
349 | return PTR_ERR(xd); | 349 | return PTR_ERR(xd); |
350 | } | ||
351 | xd->xprefix = rx->xprefix; | ||
352 | xd->name_len = rx->name_len; | ||
353 | xd->value_len = je16_to_cpu(rx->value_len); | ||
354 | xd->data_crc = je32_to_cpu(rx->data_crc); | ||
355 | 350 | ||
356 | xd->node = jffs2_link_node_ref(c, jeb, ofs | REF_PRISTINE, totlen, NULL); | 351 | if (xd->version > version) { |
357 | /* FIXME */ xd->node->next_in_ino = (void *)xd; | 352 | struct jffs2_raw_node_ref *raw |
353 | = jffs2_link_node_ref(c, jeb, ofs | REF_PRISTINE, totlen, NULL); | ||
354 | raw->next_in_ino = xd->node->next_in_ino; | ||
355 | xd->node->next_in_ino = raw; | ||
356 | } else { | ||
357 | xd->version = version; | ||
358 | xd->xprefix = rx->xprefix; | ||
359 | xd->name_len = rx->name_len; | ||
360 | xd->value_len = je16_to_cpu(rx->value_len); | ||
361 | xd->data_crc = je32_to_cpu(rx->data_crc); | ||
362 | |||
363 | jffs2_link_node_ref(c, jeb, ofs | REF_PRISTINE, totlen, (void *)xd); | ||
364 | } | ||
358 | 365 | ||
359 | if (jffs2_sum_active()) | 366 | if (jffs2_sum_active()) |
360 | jffs2_sum_add_xattr_mem(s, rx, ofs - jeb->offset); | 367 | jffs2_sum_add_xattr_mem(s, rx, ofs - jeb->offset); |
@@ -373,9 +380,8 @@ static int jffs2_scan_xref_node(struct jffs2_sb_info *c, struct jffs2_eraseblock | |||
373 | 380 | ||
374 | crc = crc32(0, rr, sizeof(*rr) - 4); | 381 | crc = crc32(0, rr, sizeof(*rr) - 4); |
375 | if (crc != je32_to_cpu(rr->node_crc)) { | 382 | if (crc != je32_to_cpu(rr->node_crc)) { |
376 | if (je32_to_cpu(rr->node_crc) != 0xffffffff) | 383 | JFFS2_WARNING("node CRC failed at %#08x, read=%#08x, calc=%#08x\n", |
377 | JFFS2_WARNING("node CRC failed at %#08x, read=%#08x, calc=%#08x\n", | 384 | ofs, je32_to_cpu(rr->node_crc), crc); |
378 | ofs, je32_to_cpu(rr->node_crc), crc); | ||
379 | if ((err = jffs2_scan_dirty_space(c, jeb, PAD(je32_to_cpu(rr->totlen))))) | 385 | if ((err = jffs2_scan_dirty_space(c, jeb, PAD(je32_to_cpu(rr->totlen))))) |
380 | return err; | 386 | return err; |
381 | return 0; | 387 | return 0; |
@@ -395,6 +401,7 @@ static int jffs2_scan_xref_node(struct jffs2_sb_info *c, struct jffs2_eraseblock | |||
395 | return -ENOMEM; | 401 | return -ENOMEM; |
396 | 402 | ||
397 | /* BEFORE jffs2_build_xattr_subsystem() called, | 403 | /* BEFORE jffs2_build_xattr_subsystem() called, |
404 | * and AFTER xattr_ref is marked as a dead xref, | ||
398 | * ref->xid is used to store 32bit xid, xd is not used | 405 | * ref->xid is used to store 32bit xid, xd is not used |
399 | * ref->ino is used to store 32bit inode-number, ic is not used | 406 | * ref->ino is used to store 32bit inode-number, ic is not used |
400 | * Thoes variables are declared as union, thus using those | 407 | * Thoes variables are declared as union, thus using those |
@@ -404,11 +411,13 @@ static int jffs2_scan_xref_node(struct jffs2_sb_info *c, struct jffs2_eraseblock | |||
404 | */ | 411 | */ |
405 | ref->ino = je32_to_cpu(rr->ino); | 412 | ref->ino = je32_to_cpu(rr->ino); |
406 | ref->xid = je32_to_cpu(rr->xid); | 413 | ref->xid = je32_to_cpu(rr->xid); |
414 | ref->xseqno = je32_to_cpu(rr->xseqno); | ||
415 | if (ref->xseqno > c->highest_xseqno) | ||
416 | c->highest_xseqno = (ref->xseqno & ~XREF_DELETE_MARKER); | ||
407 | ref->next = c->xref_temp; | 417 | ref->next = c->xref_temp; |
408 | c->xref_temp = ref; | 418 | c->xref_temp = ref; |
409 | 419 | ||
410 | ref->node = jffs2_link_node_ref(c, jeb, ofs | REF_PRISTINE, PAD(je32_to_cpu(rr->totlen)), NULL); | 420 | jffs2_link_node_ref(c, jeb, ofs | REF_PRISTINE, PAD(je32_to_cpu(rr->totlen)), (void *)ref); |
411 | /* FIXME */ ref->node->next_in_ino = (void *)ref; | ||
412 | 421 | ||
413 | if (jffs2_sum_active()) | 422 | if (jffs2_sum_active()) |
414 | jffs2_sum_add_xref_mem(s, rr, ofs - jeb->offset); | 423 | jffs2_sum_add_xref_mem(s, rr, ofs - jeb->offset); |
diff --git a/fs/jffs2/summary.c b/fs/jffs2/summary.c index 0b02fc79e4d1..c430f1d217e2 100644 --- a/fs/jffs2/summary.c +++ b/fs/jffs2/summary.c | |||
@@ -310,8 +310,6 @@ int jffs2_sum_add_kvec(struct jffs2_sb_info *c, const struct kvec *invecs, | |||
310 | #ifdef CONFIG_JFFS2_FS_XATTR | 310 | #ifdef CONFIG_JFFS2_FS_XATTR |
311 | case JFFS2_NODETYPE_XATTR: { | 311 | case JFFS2_NODETYPE_XATTR: { |
312 | struct jffs2_sum_xattr_mem *temp; | 312 | struct jffs2_sum_xattr_mem *temp; |
313 | if (je32_to_cpu(node->x.version) == 0xffffffff) | ||
314 | return 0; | ||
315 | temp = kmalloc(sizeof(struct jffs2_sum_xattr_mem), GFP_KERNEL); | 313 | temp = kmalloc(sizeof(struct jffs2_sum_xattr_mem), GFP_KERNEL); |
316 | if (!temp) | 314 | if (!temp) |
317 | goto no_mem; | 315 | goto no_mem; |
@@ -327,10 +325,6 @@ int jffs2_sum_add_kvec(struct jffs2_sb_info *c, const struct kvec *invecs, | |||
327 | } | 325 | } |
328 | case JFFS2_NODETYPE_XREF: { | 326 | case JFFS2_NODETYPE_XREF: { |
329 | struct jffs2_sum_xref_mem *temp; | 327 | struct jffs2_sum_xref_mem *temp; |
330 | |||
331 | if (je32_to_cpu(node->r.ino) == 0xffffffff | ||
332 | && je32_to_cpu(node->r.xid) == 0xffffffff) | ||
333 | return 0; | ||
334 | temp = kmalloc(sizeof(struct jffs2_sum_xref_mem), GFP_KERNEL); | 328 | temp = kmalloc(sizeof(struct jffs2_sum_xref_mem), GFP_KERNEL); |
335 | if (!temp) | 329 | if (!temp) |
336 | goto no_mem; | 330 | goto no_mem; |
@@ -483,22 +477,20 @@ static int jffs2_sum_process_sum_data(struct jffs2_sb_info *c, struct jffs2_eras | |||
483 | 477 | ||
484 | xd = jffs2_setup_xattr_datum(c, je32_to_cpu(spx->xid), | 478 | xd = jffs2_setup_xattr_datum(c, je32_to_cpu(spx->xid), |
485 | je32_to_cpu(spx->version)); | 479 | je32_to_cpu(spx->version)); |
486 | if (IS_ERR(xd)) { | 480 | if (IS_ERR(xd)) |
487 | if (PTR_ERR(xd) == -EEXIST) { | ||
488 | /* a newer version of xd exists */ | ||
489 | if ((err = jffs2_scan_dirty_space(c, jeb, je32_to_cpu(spx->totlen)))) | ||
490 | return err; | ||
491 | sp += JFFS2_SUMMARY_XATTR_SIZE; | ||
492 | break; | ||
493 | } | ||
494 | JFFS2_NOTICE("allocation of xattr_datum failed\n"); | ||
495 | return PTR_ERR(xd); | 481 | return PTR_ERR(xd); |
482 | if (xd->version > je32_to_cpu(spx->version)) { | ||
483 | /* node is not the newest one */ | ||
484 | struct jffs2_raw_node_ref *raw | ||
485 | = sum_link_node_ref(c, jeb, je32_to_cpu(spx->offset) | REF_UNCHECKED, | ||
486 | PAD(je32_to_cpu(spx->totlen)), NULL); | ||
487 | raw->next_in_ino = xd->node->next_in_ino; | ||
488 | xd->node->next_in_ino = raw; | ||
489 | } else { | ||
490 | xd->version = je32_to_cpu(spx->version); | ||
491 | sum_link_node_ref(c, jeb, je32_to_cpu(spx->offset) | REF_UNCHECKED, | ||
492 | PAD(je32_to_cpu(spx->totlen)), (void *)xd); | ||
496 | } | 493 | } |
497 | |||
498 | xd->node = sum_link_node_ref(c, jeb, je32_to_cpu(spx->offset) | REF_UNCHECKED, | ||
499 | PAD(je32_to_cpu(spx->totlen)), NULL); | ||
500 | /* FIXME */ xd->node->next_in_ino = (void *)xd; | ||
501 | |||
502 | *pseudo_random += je32_to_cpu(spx->xid); | 494 | *pseudo_random += je32_to_cpu(spx->xid); |
503 | sp += JFFS2_SUMMARY_XATTR_SIZE; | 495 | sp += JFFS2_SUMMARY_XATTR_SIZE; |
504 | 496 | ||
@@ -519,14 +511,11 @@ static int jffs2_sum_process_sum_data(struct jffs2_sb_info *c, struct jffs2_eras | |||
519 | JFFS2_NOTICE("allocation of xattr_datum failed\n"); | 511 | JFFS2_NOTICE("allocation of xattr_datum failed\n"); |
520 | return -ENOMEM; | 512 | return -ENOMEM; |
521 | } | 513 | } |
522 | ref->ino = 0xfffffffe; | ||
523 | ref->xid = 0xfffffffd; | ||
524 | ref->next = c->xref_temp; | 514 | ref->next = c->xref_temp; |
525 | c->xref_temp = ref; | 515 | c->xref_temp = ref; |
526 | 516 | ||
527 | ref->node = sum_link_node_ref(c, jeb, je32_to_cpu(spr->offset) | REF_UNCHECKED, | 517 | sum_link_node_ref(c, jeb, je32_to_cpu(spr->offset) | REF_UNCHECKED, |
528 | PAD(sizeof(struct jffs2_raw_xref)), NULL); | 518 | PAD(sizeof(struct jffs2_raw_xref)), (void *)ref); |
529 | /* FIXME */ ref->node->next_in_ino = (void *)ref; | ||
530 | 519 | ||
531 | *pseudo_random += ref->node->flash_offset; | 520 | *pseudo_random += ref->node->flash_offset; |
532 | sp += JFFS2_SUMMARY_XREF_SIZE; | 521 | sp += JFFS2_SUMMARY_XREF_SIZE; |
diff --git a/fs/jffs2/xattr.c b/fs/jffs2/xattr.c index 2d82e250be34..03871ab7c26d 100644 --- a/fs/jffs2/xattr.c +++ b/fs/jffs2/xattr.c | |||
@@ -23,18 +23,15 @@ | |||
23 | * xattr_datum_hashkey(xprefix, xname, xvalue, xsize) | 23 | * xattr_datum_hashkey(xprefix, xname, xvalue, xsize) |
24 | * is used to calcurate xdatum hashkey. The reminder of hashkey into XATTRINDEX_HASHSIZE is | 24 | * is used to calcurate xdatum hashkey. The reminder of hashkey into XATTRINDEX_HASHSIZE is |
25 | * the index of the xattr name/value pair cache (c->xattrindex). | 25 | * the index of the xattr name/value pair cache (c->xattrindex). |
26 | * is_xattr_datum_unchecked(c, xd) | ||
27 | * returns 1, if xdatum contains any unchecked raw nodes. if all raw nodes are not | ||
28 | * unchecked, it returns 0. | ||
26 | * unload_xattr_datum(c, xd) | 29 | * unload_xattr_datum(c, xd) |
27 | * is used to release xattr name/value pair and detach from c->xattrindex. | 30 | * is used to release xattr name/value pair and detach from c->xattrindex. |
28 | * reclaim_xattr_datum(c) | 31 | * reclaim_xattr_datum(c) |
29 | * is used to reclaim xattr name/value pairs on the xattr name/value pair cache when | 32 | * is used to reclaim xattr name/value pairs on the xattr name/value pair cache when |
30 | * memory usage by cache is over c->xdatum_mem_threshold. Currentry, this threshold | 33 | * memory usage by cache is over c->xdatum_mem_threshold. Currentry, this threshold |
31 | * is hard coded as 32KiB. | 34 | * is hard coded as 32KiB. |
32 | * delete_xattr_datum_node(c, xd) | ||
33 | * is used to delete a jffs2 node is dominated by xdatum. When EBS(Erase Block Summary) is | ||
34 | * enabled, it overwrites the obsolete node by myself. | ||
35 | * delete_xattr_datum(c, xd) | ||
36 | * is used to delete jffs2_xattr_datum object. It must be called with 0-value of reference | ||
37 | * counter. (It means how many jffs2_xattr_ref object refers this xdatum.) | ||
38 | * do_verify_xattr_datum(c, xd) | 35 | * do_verify_xattr_datum(c, xd) |
39 | * is used to load the xdatum informations without name/value pair from the medium. | 36 | * is used to load the xdatum informations without name/value pair from the medium. |
40 | * It's necessary once, because those informations are not collected during mounting | 37 | * It's necessary once, because those informations are not collected during mounting |
@@ -53,8 +50,13 @@ | |||
53 | * is used to write xdatum to medium. xd->version will be incremented. | 50 | * is used to write xdatum to medium. xd->version will be incremented. |
54 | * create_xattr_datum(c, xprefix, xname, xvalue, xsize) | 51 | * create_xattr_datum(c, xprefix, xname, xvalue, xsize) |
55 | * is used to create new xdatum and write to medium. | 52 | * is used to create new xdatum and write to medium. |
53 | * delete_xattr_datum_delay(c, xd) | ||
54 | * is used to delete a xdatum without 'delete marker'. It has a possibility to detect | ||
55 | * orphan xdatum on next mounting. | ||
56 | * delete_xattr_datum(c, xd) | ||
57 | * is used to delete a xdatum with 'delete marker'. Calling jffs2_reserve_space() is | ||
58 | * necessary before this function. | ||
56 | * -------------------------------------------------- */ | 59 | * -------------------------------------------------- */ |
57 | |||
58 | static uint32_t xattr_datum_hashkey(int xprefix, const char *xname, const char *xvalue, int xsize) | 60 | static uint32_t xattr_datum_hashkey(int xprefix, const char *xname, const char *xvalue, int xsize) |
59 | { | 61 | { |
60 | int name_len = strlen(xname); | 62 | int name_len = strlen(xname); |
@@ -62,6 +64,22 @@ static uint32_t xattr_datum_hashkey(int xprefix, const char *xname, const char * | |||
62 | return crc32(xprefix, xname, name_len) ^ crc32(xprefix, xvalue, xsize); | 64 | return crc32(xprefix, xname, name_len) ^ crc32(xprefix, xvalue, xsize); |
63 | } | 65 | } |
64 | 66 | ||
67 | static int is_xattr_datum_unchecked(struct jffs2_sb_info *c, struct jffs2_xattr_datum *xd) | ||
68 | { | ||
69 | struct jffs2_raw_node_ref *raw; | ||
70 | int rc = 0; | ||
71 | |||
72 | spin_lock(&c->erase_completion_lock); | ||
73 | for (raw=xd->node; raw != (void *)xd; raw=raw->next_in_ino) { | ||
74 | if (ref_flags(raw) == REF_UNCHECKED) { | ||
75 | rc = 1; | ||
76 | break; | ||
77 | } | ||
78 | } | ||
79 | spin_unlock(&c->erase_completion_lock); | ||
80 | return rc; | ||
81 | } | ||
82 | |||
65 | static void unload_xattr_datum(struct jffs2_sb_info *c, struct jffs2_xattr_datum *xd) | 83 | static void unload_xattr_datum(struct jffs2_sb_info *c, struct jffs2_xattr_datum *xd) |
66 | { | 84 | { |
67 | /* must be called under down_write(xattr_sem) */ | 85 | /* must be called under down_write(xattr_sem) */ |
@@ -107,80 +125,39 @@ static void reclaim_xattr_datum(struct jffs2_sb_info *c) | |||
107 | before, c->xdatum_mem_usage, before - c->xdatum_mem_usage); | 125 | before, c->xdatum_mem_usage, before - c->xdatum_mem_usage); |
108 | } | 126 | } |
109 | 127 | ||
110 | static void delete_xattr_datum_node(struct jffs2_sb_info *c, struct jffs2_xattr_datum *xd) | ||
111 | { | ||
112 | /* must be called under down_write(xattr_sem) */ | ||
113 | struct jffs2_raw_xattr rx; | ||
114 | size_t length; | ||
115 | int rc; | ||
116 | |||
117 | if (!xd->node) { | ||
118 | JFFS2_WARNING("xdatum (xid=%u) is removed twice.\n", xd->xid); | ||
119 | return; | ||
120 | } | ||
121 | if (jffs2_sum_active()) { | ||
122 | memset(&rx, 0xff, sizeof(struct jffs2_raw_xattr)); | ||
123 | rc = jffs2_flash_read(c, ref_offset(xd->node), | ||
124 | sizeof(struct jffs2_unknown_node), | ||
125 | &length, (char *)&rx); | ||
126 | if (rc || length != sizeof(struct jffs2_unknown_node)) { | ||
127 | JFFS2_ERROR("jffs2_flash_read()=%d, req=%zu, read=%zu at %#08x\n", | ||
128 | rc, sizeof(struct jffs2_unknown_node), | ||
129 | length, ref_offset(xd->node)); | ||
130 | } | ||
131 | rc = jffs2_flash_write(c, ref_offset(xd->node), sizeof(rx), | ||
132 | &length, (char *)&rx); | ||
133 | if (rc || length != sizeof(struct jffs2_raw_xattr)) { | ||
134 | JFFS2_ERROR("jffs2_flash_write()=%d, req=%zu, wrote=%zu ar %#08x\n", | ||
135 | rc, sizeof(rx), length, ref_offset(xd->node)); | ||
136 | } | ||
137 | } | ||
138 | spin_lock(&c->erase_completion_lock); | ||
139 | xd->node->next_in_ino = NULL; | ||
140 | spin_unlock(&c->erase_completion_lock); | ||
141 | jffs2_mark_node_obsolete(c, xd->node); | ||
142 | xd->node = NULL; | ||
143 | } | ||
144 | |||
145 | static void delete_xattr_datum(struct jffs2_sb_info *c, struct jffs2_xattr_datum *xd) | ||
146 | { | ||
147 | /* must be called under down_write(xattr_sem) */ | ||
148 | BUG_ON(xd->refcnt); | ||
149 | |||
150 | unload_xattr_datum(c, xd); | ||
151 | if (xd->node) { | ||
152 | delete_xattr_datum_node(c, xd); | ||
153 | xd->node = NULL; | ||
154 | } | ||
155 | jffs2_free_xattr_datum(xd); | ||
156 | } | ||
157 | |||
158 | static int do_verify_xattr_datum(struct jffs2_sb_info *c, struct jffs2_xattr_datum *xd) | 128 | static int do_verify_xattr_datum(struct jffs2_sb_info *c, struct jffs2_xattr_datum *xd) |
159 | { | 129 | { |
160 | /* must be called under down_write(xattr_sem) */ | 130 | /* must be called under down_write(xattr_sem) */ |
161 | struct jffs2_eraseblock *jeb; | 131 | struct jffs2_eraseblock *jeb; |
132 | struct jffs2_raw_node_ref *raw; | ||
162 | struct jffs2_raw_xattr rx; | 133 | struct jffs2_raw_xattr rx; |
163 | size_t readlen; | 134 | size_t readlen; |
164 | uint32_t crc, totlen; | 135 | uint32_t crc, offset, totlen; |
165 | int rc; | 136 | int rc; |
166 | 137 | ||
167 | BUG_ON(!xd->node); | 138 | spin_lock(&c->erase_completion_lock); |
168 | BUG_ON(ref_flags(xd->node) != REF_UNCHECKED); | 139 | offset = ref_offset(xd->node); |
140 | if (ref_flags(xd->node) == REF_PRISTINE) | ||
141 | goto complete; | ||
142 | spin_unlock(&c->erase_completion_lock); | ||
169 | 143 | ||
170 | rc = jffs2_flash_read(c, ref_offset(xd->node), sizeof(rx), &readlen, (char *)&rx); | 144 | rc = jffs2_flash_read(c, offset, sizeof(rx), &readlen, (char *)&rx); |
171 | if (rc || readlen != sizeof(rx)) { | 145 | if (rc || readlen != sizeof(rx)) { |
172 | JFFS2_WARNING("jffs2_flash_read()=%d, req=%zu, read=%zu at %#08x\n", | 146 | JFFS2_WARNING("jffs2_flash_read()=%d, req=%zu, read=%zu at %#08x\n", |
173 | rc, sizeof(rx), readlen, ref_offset(xd->node)); | 147 | rc, sizeof(rx), readlen, offset); |
174 | return rc ? rc : -EIO; | 148 | return rc ? rc : -EIO; |
175 | } | 149 | } |
176 | crc = crc32(0, &rx, sizeof(rx) - 4); | 150 | crc = crc32(0, &rx, sizeof(rx) - 4); |
177 | if (crc != je32_to_cpu(rx.node_crc)) { | 151 | if (crc != je32_to_cpu(rx.node_crc)) { |
178 | if (je32_to_cpu(rx.node_crc) != 0xffffffff) | 152 | JFFS2_ERROR("node CRC failed at %#08x, read=%#08x, calc=%#08x\n", |
179 | JFFS2_ERROR("node CRC failed at %#08x, read=%#08x, calc=%#08x\n", | 153 | offset, je32_to_cpu(rx.hdr_crc), crc); |
180 | ref_offset(xd->node), je32_to_cpu(rx.hdr_crc), crc); | 154 | xd->flags |= JFFS2_XFLAGS_INVALID; |
181 | return EIO; | 155 | return EIO; |
182 | } | 156 | } |
183 | totlen = PAD(sizeof(rx) + rx.name_len + 1 + je16_to_cpu(rx.value_len)); | 157 | totlen = sizeof(rx); |
158 | if (xd->version != XDATUM_DELETE_MARKER) | ||
159 | totlen += rx.name_len + 1 + je16_to_cpu(rx.value_len); | ||
160 | totlen = PAD(totlen); | ||
184 | if (je16_to_cpu(rx.magic) != JFFS2_MAGIC_BITMASK | 161 | if (je16_to_cpu(rx.magic) != JFFS2_MAGIC_BITMASK |
185 | || je16_to_cpu(rx.nodetype) != JFFS2_NODETYPE_XATTR | 162 | || je16_to_cpu(rx.nodetype) != JFFS2_NODETYPE_XATTR |
186 | || je32_to_cpu(rx.totlen) != totlen | 163 | || je32_to_cpu(rx.totlen) != totlen |
@@ -188,11 +165,12 @@ static int do_verify_xattr_datum(struct jffs2_sb_info *c, struct jffs2_xattr_dat | |||
188 | || je32_to_cpu(rx.version) != xd->version) { | 165 | || je32_to_cpu(rx.version) != xd->version) { |
189 | JFFS2_ERROR("inconsistent xdatum at %#08x, magic=%#04x/%#04x, " | 166 | JFFS2_ERROR("inconsistent xdatum at %#08x, magic=%#04x/%#04x, " |
190 | "nodetype=%#04x/%#04x, totlen=%u/%u, xid=%u/%u, version=%u/%u\n", | 167 | "nodetype=%#04x/%#04x, totlen=%u/%u, xid=%u/%u, version=%u/%u\n", |
191 | ref_offset(xd->node), je16_to_cpu(rx.magic), JFFS2_MAGIC_BITMASK, | 168 | offset, je16_to_cpu(rx.magic), JFFS2_MAGIC_BITMASK, |
192 | je16_to_cpu(rx.nodetype), JFFS2_NODETYPE_XATTR, | 169 | je16_to_cpu(rx.nodetype), JFFS2_NODETYPE_XATTR, |
193 | je32_to_cpu(rx.totlen), totlen, | 170 | je32_to_cpu(rx.totlen), totlen, |
194 | je32_to_cpu(rx.xid), xd->xid, | 171 | je32_to_cpu(rx.xid), xd->xid, |
195 | je32_to_cpu(rx.version), xd->version); | 172 | je32_to_cpu(rx.version), xd->version); |
173 | xd->flags |= JFFS2_XFLAGS_INVALID; | ||
196 | return EIO; | 174 | return EIO; |
197 | } | 175 | } |
198 | xd->xprefix = rx.xprefix; | 176 | xd->xprefix = rx.xprefix; |
@@ -200,14 +178,17 @@ static int do_verify_xattr_datum(struct jffs2_sb_info *c, struct jffs2_xattr_dat | |||
200 | xd->value_len = je16_to_cpu(rx.value_len); | 178 | xd->value_len = je16_to_cpu(rx.value_len); |
201 | xd->data_crc = je32_to_cpu(rx.data_crc); | 179 | xd->data_crc = je32_to_cpu(rx.data_crc); |
202 | 180 | ||
203 | /* This JFFS2_NODETYPE_XATTR node is checked */ | ||
204 | jeb = &c->blocks[ref_offset(xd->node) / c->sector_size]; | ||
205 | totlen = PAD(je32_to_cpu(rx.totlen)); | ||
206 | |||
207 | spin_lock(&c->erase_completion_lock); | 181 | spin_lock(&c->erase_completion_lock); |
208 | c->unchecked_size -= totlen; c->used_size += totlen; | 182 | complete: |
209 | jeb->unchecked_size -= totlen; jeb->used_size += totlen; | 183 | for (raw=xd->node; raw != (void *)xd; raw=raw->next_in_ino) { |
210 | xd->node->flash_offset = ref_offset(xd->node) | REF_PRISTINE; | 184 | jeb = &c->blocks[ref_offset(raw) / c->sector_size]; |
185 | totlen = PAD(ref_totlen(c, jeb, raw)); | ||
186 | if (ref_flags(raw) == REF_UNCHECKED) { | ||
187 | c->unchecked_size -= totlen; c->used_size += totlen; | ||
188 | jeb->unchecked_size -= totlen; jeb->used_size += totlen; | ||
189 | } | ||
190 | raw->flash_offset = ref_offset(raw) | ((xd->node==raw) ? REF_PRISTINE : REF_NORMAL); | ||
191 | } | ||
211 | spin_unlock(&c->erase_completion_lock); | 192 | spin_unlock(&c->erase_completion_lock); |
212 | 193 | ||
213 | /* unchecked xdatum is chained with c->xattr_unchecked */ | 194 | /* unchecked xdatum is chained with c->xattr_unchecked */ |
@@ -227,7 +208,6 @@ static int do_load_xattr_datum(struct jffs2_sb_info *c, struct jffs2_xattr_datum | |||
227 | uint32_t crc, length; | 208 | uint32_t crc, length; |
228 | int i, ret, retry = 0; | 209 | int i, ret, retry = 0; |
229 | 210 | ||
230 | BUG_ON(!xd->node); | ||
231 | BUG_ON(ref_flags(xd->node) != REF_PRISTINE); | 211 | BUG_ON(ref_flags(xd->node) != REF_PRISTINE); |
232 | BUG_ON(!list_empty(&xd->xindex)); | 212 | BUG_ON(!list_empty(&xd->xindex)); |
233 | retry: | 213 | retry: |
@@ -253,6 +233,7 @@ static int do_load_xattr_datum(struct jffs2_sb_info *c, struct jffs2_xattr_datum | |||
253 | " at %#08x, read: 0x%08x calculated: 0x%08x\n", | 233 | " at %#08x, read: 0x%08x calculated: 0x%08x\n", |
254 | ref_offset(xd->node), xd->data_crc, crc); | 234 | ref_offset(xd->node), xd->data_crc, crc); |
255 | kfree(data); | 235 | kfree(data); |
236 | xd->flags |= JFFS2_XFLAGS_INVALID; | ||
256 | return EIO; | 237 | return EIO; |
257 | } | 238 | } |
258 | 239 | ||
@@ -286,16 +267,13 @@ static int load_xattr_datum(struct jffs2_sb_info *c, struct jffs2_xattr_datum *x | |||
286 | * rc > 0 : Unrecoverable error, this node should be deleted. | 267 | * rc > 0 : Unrecoverable error, this node should be deleted. |
287 | */ | 268 | */ |
288 | int rc = 0; | 269 | int rc = 0; |
289 | BUG_ON(xd->xname); | 270 | |
290 | if (!xd->node) | 271 | if (xd->xname) |
272 | return 0; | ||
273 | if (xd->flags & JFFS2_XFLAGS_INVALID) | ||
291 | return EIO; | 274 | return EIO; |
292 | if (unlikely(ref_flags(xd->node) != REF_PRISTINE)) { | 275 | if (unlikely(is_xattr_datum_unchecked(c, xd))) |
293 | rc = do_verify_xattr_datum(c, xd); | 276 | rc = do_verify_xattr_datum(c, xd); |
294 | if (rc > 0) { | ||
295 | list_del_init(&xd->xindex); | ||
296 | delete_xattr_datum_node(c, xd); | ||
297 | } | ||
298 | } | ||
299 | if (!rc) | 277 | if (!rc) |
300 | rc = do_load_xattr_datum(c, xd); | 278 | rc = do_load_xattr_datum(c, xd); |
301 | return rc; | 279 | return rc; |
@@ -304,36 +282,43 @@ static int load_xattr_datum(struct jffs2_sb_info *c, struct jffs2_xattr_datum *x | |||
304 | static int save_xattr_datum(struct jffs2_sb_info *c, struct jffs2_xattr_datum *xd) | 282 | static int save_xattr_datum(struct jffs2_sb_info *c, struct jffs2_xattr_datum *xd) |
305 | { | 283 | { |
306 | /* must be called under down_write(xattr_sem) */ | 284 | /* must be called under down_write(xattr_sem) */ |
307 | struct jffs2_raw_node_ref *raw; | ||
308 | struct jffs2_raw_xattr rx; | 285 | struct jffs2_raw_xattr rx; |
309 | struct kvec vecs[2]; | 286 | struct kvec vecs[2]; |
310 | size_t length; | 287 | size_t length; |
311 | int rc, totlen; | 288 | int rc, totlen, nvecs = 1; |
312 | uint32_t phys_ofs = write_ofs(c); | 289 | uint32_t phys_ofs = write_ofs(c); |
313 | 290 | ||
314 | BUG_ON(!xd->xname); | 291 | BUG_ON(is_xattr_datum_dead(xd) || (xd->flags & JFFS2_XFLAGS_INVALID) |
292 | ? !!xd->xname : !xd->xname); | ||
315 | 293 | ||
316 | vecs[0].iov_base = ℞ | 294 | vecs[0].iov_base = ℞ |
317 | vecs[0].iov_len = PAD(sizeof(rx)); | 295 | vecs[0].iov_len = totlen = sizeof(rx); |
318 | vecs[1].iov_base = xd->xname; | 296 | if (!is_xattr_datum_dead(xd) && !(xd->flags & JFFS2_XFLAGS_INVALID)) { |
319 | vecs[1].iov_len = xd->name_len + 1 + xd->value_len; | 297 | nvecs++; |
320 | totlen = vecs[0].iov_len + vecs[1].iov_len; | 298 | vecs[1].iov_base = xd->xname; |
321 | 299 | vecs[1].iov_len = xd->name_len + 1 + xd->value_len; | |
300 | totlen += vecs[1].iov_len; | ||
301 | } | ||
322 | /* Setup raw-xattr */ | 302 | /* Setup raw-xattr */ |
303 | memset(&rx, 0, sizeof(rx)); | ||
323 | rx.magic = cpu_to_je16(JFFS2_MAGIC_BITMASK); | 304 | rx.magic = cpu_to_je16(JFFS2_MAGIC_BITMASK); |
324 | rx.nodetype = cpu_to_je16(JFFS2_NODETYPE_XATTR); | 305 | rx.nodetype = cpu_to_je16(JFFS2_NODETYPE_XATTR); |
325 | rx.totlen = cpu_to_je32(PAD(totlen)); | 306 | rx.totlen = cpu_to_je32(PAD(totlen)); |
326 | rx.hdr_crc = cpu_to_je32(crc32(0, &rx, sizeof(struct jffs2_unknown_node) - 4)); | 307 | rx.hdr_crc = cpu_to_je32(crc32(0, &rx, sizeof(struct jffs2_unknown_node) - 4)); |
327 | 308 | ||
328 | rx.xid = cpu_to_je32(xd->xid); | 309 | rx.xid = cpu_to_je32(xd->xid); |
329 | rx.version = cpu_to_je32(++xd->version); | 310 | if (!is_xattr_datum_dead(xd) && !(xd->flags & JFFS2_XFLAGS_INVALID)) { |
330 | rx.xprefix = xd->xprefix; | 311 | rx.version = cpu_to_je32(++xd->version); |
331 | rx.name_len = xd->name_len; | 312 | rx.xprefix = xd->xprefix; |
332 | rx.value_len = cpu_to_je16(xd->value_len); | 313 | rx.name_len = xd->name_len; |
333 | rx.data_crc = cpu_to_je32(crc32(0, vecs[1].iov_base, vecs[1].iov_len)); | 314 | rx.value_len = cpu_to_je16(xd->value_len); |
315 | rx.data_crc = cpu_to_je32(crc32(0, vecs[1].iov_base, vecs[1].iov_len)); | ||
316 | } else { | ||
317 | rx.version = cpu_to_je32(XDATUM_DELETE_MARKER); | ||
318 | } | ||
334 | rx.node_crc = cpu_to_je32(crc32(0, &rx, sizeof(struct jffs2_raw_xattr) - 4)); | 319 | rx.node_crc = cpu_to_je32(crc32(0, &rx, sizeof(struct jffs2_raw_xattr) - 4)); |
335 | 320 | ||
336 | rc = jffs2_flash_writev(c, vecs, 2, phys_ofs, &length, 0); | 321 | rc = jffs2_flash_writev(c, vecs, nvecs, phys_ofs, &length, 0); |
337 | if (rc || totlen != length) { | 322 | if (rc || totlen != length) { |
338 | JFFS2_WARNING("jffs2_flash_writev()=%d, req=%u, wrote=%zu, at %#08x\n", | 323 | JFFS2_WARNING("jffs2_flash_writev()=%d, req=%u, wrote=%zu, at %#08x\n", |
339 | rc, totlen, length, phys_ofs); | 324 | rc, totlen, length, phys_ofs); |
@@ -343,14 +328,8 @@ static int save_xattr_datum(struct jffs2_sb_info *c, struct jffs2_xattr_datum *x | |||
343 | 328 | ||
344 | return rc; | 329 | return rc; |
345 | } | 330 | } |
346 | |||
347 | /* success */ | 331 | /* success */ |
348 | raw = jffs2_add_physical_node_ref(c, phys_ofs | REF_PRISTINE, PAD(totlen), NULL); | 332 | jffs2_add_physical_node_ref(c, phys_ofs | REF_PRISTINE, PAD(totlen), (void *)xd); |
349 | /* FIXME */ raw->next_in_ino = (void *)xd; | ||
350 | |||
351 | if (xd->node) | ||
352 | delete_xattr_datum_node(c, xd); | ||
353 | xd->node = raw; | ||
354 | 333 | ||
355 | dbg_xattr("success on saving xdatum (xid=%u, version=%u, xprefix=%u, xname='%s')\n", | 334 | dbg_xattr("success on saving xdatum (xid=%u, version=%u, xprefix=%u, xname='%s')\n", |
356 | xd->xid, xd->version, xd->xprefix, xd->xname); | 335 | xd->xid, xd->version, xd->xprefix, xd->xname); |
@@ -426,6 +405,39 @@ static struct jffs2_xattr_datum *create_xattr_datum(struct jffs2_sb_info *c, | |||
426 | return xd; | 405 | return xd; |
427 | } | 406 | } |
428 | 407 | ||
408 | static void delete_xattr_datum_delay(struct jffs2_sb_info *c, struct jffs2_xattr_datum *xd) | ||
409 | { | ||
410 | /* must be called under down_write(xattr_sem) */ | ||
411 | BUG_ON(xd->refcnt); | ||
412 | |||
413 | unload_xattr_datum(c, xd); | ||
414 | set_xattr_datum_dead(xd); | ||
415 | spin_lock(&c->erase_completion_lock); | ||
416 | list_add(&xd->xindex, &c->xattr_dead_list); | ||
417 | spin_unlock(&c->erase_completion_lock); | ||
418 | JFFS2_NOTICE("xdatum(xid=%u) was removed without delete marker. " | ||
419 | "An orphan xdatum may be detected on next mounting.\n", xd->xid); | ||
420 | } | ||
421 | |||
422 | static void delete_xattr_datum(struct jffs2_sb_info *c, struct jffs2_xattr_datum *xd) | ||
423 | { | ||
424 | /* must be called under jffs2_reserve_space() and down_write(xattr_sem) */ | ||
425 | int rc; | ||
426 | BUG_ON(xd->refcnt); | ||
427 | |||
428 | unload_xattr_datum(c, xd); | ||
429 | set_xattr_datum_dead(xd); | ||
430 | rc = save_xattr_datum(c, xd); | ||
431 | if (rc) { | ||
432 | JFFS2_NOTICE("xdatum(xid=%u) was removed without delete marker. " | ||
433 | "An orphan xdatum may be detected on next mounting.\n", | ||
434 | xd->xid); | ||
435 | } | ||
436 | spin_lock(&c->erase_completion_lock); | ||
437 | list_add(&xd->xindex, &c->xattr_dead_list); | ||
438 | spin_unlock(&c->erase_completion_lock); | ||
439 | } | ||
440 | |||
429 | /* -------- xref related functions ------------------ | 441 | /* -------- xref related functions ------------------ |
430 | * verify_xattr_ref(c, ref) | 442 | * verify_xattr_ref(c, ref) |
431 | * is used to load xref information from medium. Because summary data does not | 443 | * is used to load xref information from medium. Because summary data does not |
@@ -450,25 +462,29 @@ static struct jffs2_xattr_datum *create_xattr_datum(struct jffs2_sb_info *c, | |||
450 | static int verify_xattr_ref(struct jffs2_sb_info *c, struct jffs2_xattr_ref *ref) | 462 | static int verify_xattr_ref(struct jffs2_sb_info *c, struct jffs2_xattr_ref *ref) |
451 | { | 463 | { |
452 | struct jffs2_eraseblock *jeb; | 464 | struct jffs2_eraseblock *jeb; |
465 | struct jffs2_raw_node_ref *raw; | ||
453 | struct jffs2_raw_xref rr; | 466 | struct jffs2_raw_xref rr; |
454 | size_t readlen; | 467 | size_t readlen; |
455 | uint32_t crc, totlen; | 468 | uint32_t crc, offset, totlen; |
456 | int rc; | 469 | int rc; |
457 | 470 | ||
458 | BUG_ON(ref_flags(ref->node) != REF_UNCHECKED); | 471 | spin_lock(&c->erase_completion_lock); |
472 | if (ref_flags(ref->node) != REF_UNCHECKED) | ||
473 | goto complete; | ||
474 | offset = ref_offset(ref->node); | ||
475 | spin_unlock(&c->erase_completion_lock); | ||
459 | 476 | ||
460 | rc = jffs2_flash_read(c, ref_offset(ref->node), sizeof(rr), &readlen, (char *)&rr); | 477 | rc = jffs2_flash_read(c, offset, sizeof(rr), &readlen, (char *)&rr); |
461 | if (rc || sizeof(rr) != readlen) { | 478 | if (rc || sizeof(rr) != readlen) { |
462 | JFFS2_WARNING("jffs2_flash_read()=%d, req=%zu, read=%zu, at %#08x\n", | 479 | JFFS2_WARNING("jffs2_flash_read()=%d, req=%zu, read=%zu, at %#08x\n", |
463 | rc, sizeof(rr), readlen, ref_offset(ref->node)); | 480 | rc, sizeof(rr), readlen, offset); |
464 | return rc ? rc : -EIO; | 481 | return rc ? rc : -EIO; |
465 | } | 482 | } |
466 | /* obsolete node */ | 483 | /* obsolete node */ |
467 | crc = crc32(0, &rr, sizeof(rr) - 4); | 484 | crc = crc32(0, &rr, sizeof(rr) - 4); |
468 | if (crc != je32_to_cpu(rr.node_crc)) { | 485 | if (crc != je32_to_cpu(rr.node_crc)) { |
469 | if (je32_to_cpu(rr.node_crc) != 0xffffffff) | 486 | JFFS2_ERROR("node CRC failed at %#08x, read=%#08x, calc=%#08x\n", |
470 | JFFS2_ERROR("node CRC failed at %#08x, read=%#08x, calc=%#08x\n", | 487 | offset, je32_to_cpu(rr.node_crc), crc); |
471 | ref_offset(ref->node), je32_to_cpu(rr.node_crc), crc); | ||
472 | return EIO; | 488 | return EIO; |
473 | } | 489 | } |
474 | if (je16_to_cpu(rr.magic) != JFFS2_MAGIC_BITMASK | 490 | if (je16_to_cpu(rr.magic) != JFFS2_MAGIC_BITMASK |
@@ -476,22 +492,28 @@ static int verify_xattr_ref(struct jffs2_sb_info *c, struct jffs2_xattr_ref *ref | |||
476 | || je32_to_cpu(rr.totlen) != PAD(sizeof(rr))) { | 492 | || je32_to_cpu(rr.totlen) != PAD(sizeof(rr))) { |
477 | JFFS2_ERROR("inconsistent xref at %#08x, magic=%#04x/%#04x, " | 493 | JFFS2_ERROR("inconsistent xref at %#08x, magic=%#04x/%#04x, " |
478 | "nodetype=%#04x/%#04x, totlen=%u/%zu\n", | 494 | "nodetype=%#04x/%#04x, totlen=%u/%zu\n", |
479 | ref_offset(ref->node), je16_to_cpu(rr.magic), JFFS2_MAGIC_BITMASK, | 495 | offset, je16_to_cpu(rr.magic), JFFS2_MAGIC_BITMASK, |
480 | je16_to_cpu(rr.nodetype), JFFS2_NODETYPE_XREF, | 496 | je16_to_cpu(rr.nodetype), JFFS2_NODETYPE_XREF, |
481 | je32_to_cpu(rr.totlen), PAD(sizeof(rr))); | 497 | je32_to_cpu(rr.totlen), PAD(sizeof(rr))); |
482 | return EIO; | 498 | return EIO; |
483 | } | 499 | } |
484 | ref->ino = je32_to_cpu(rr.ino); | 500 | ref->ino = je32_to_cpu(rr.ino); |
485 | ref->xid = je32_to_cpu(rr.xid); | 501 | ref->xid = je32_to_cpu(rr.xid); |
486 | 502 | ref->xseqno = je32_to_cpu(rr.xseqno); | |
487 | /* fixup superblock/eraseblock info */ | 503 | if (ref->xseqno > c->highest_xseqno) |
488 | jeb = &c->blocks[ref_offset(ref->node) / c->sector_size]; | 504 | c->highest_xseqno = (ref->xseqno & ~XREF_DELETE_MARKER); |
489 | totlen = PAD(sizeof(rr)); | ||
490 | 505 | ||
491 | spin_lock(&c->erase_completion_lock); | 506 | spin_lock(&c->erase_completion_lock); |
492 | c->unchecked_size -= totlen; c->used_size += totlen; | 507 | complete: |
493 | jeb->unchecked_size -= totlen; jeb->used_size += totlen; | 508 | for (raw=ref->node; raw != (void *)ref; raw=raw->next_in_ino) { |
494 | ref->node->flash_offset = ref_offset(ref->node) | REF_PRISTINE; | 509 | jeb = &c->blocks[ref_offset(raw) / c->sector_size]; |
510 | totlen = PAD(ref_totlen(c, jeb, raw)); | ||
511 | if (ref_flags(raw) == REF_UNCHECKED) { | ||
512 | c->unchecked_size -= totlen; c->used_size += totlen; | ||
513 | jeb->unchecked_size -= totlen; jeb->used_size += totlen; | ||
514 | } | ||
515 | raw->flash_offset = ref_offset(raw) | ((ref->node==raw) ? REF_PRISTINE : REF_NORMAL); | ||
516 | } | ||
495 | spin_unlock(&c->erase_completion_lock); | 517 | spin_unlock(&c->erase_completion_lock); |
496 | 518 | ||
497 | dbg_xattr("success on verifying xref (ino=%u, xid=%u) at %#08x\n", | 519 | dbg_xattr("success on verifying xref (ino=%u, xid=%u) at %#08x\n", |
@@ -499,58 +521,12 @@ static int verify_xattr_ref(struct jffs2_sb_info *c, struct jffs2_xattr_ref *ref | |||
499 | return 0; | 521 | return 0; |
500 | } | 522 | } |
501 | 523 | ||
502 | static void delete_xattr_ref_node(struct jffs2_sb_info *c, struct jffs2_xattr_ref *ref) | ||
503 | { | ||
504 | struct jffs2_raw_xref rr; | ||
505 | size_t length; | ||
506 | int rc; | ||
507 | |||
508 | if (jffs2_sum_active()) { | ||
509 | memset(&rr, 0xff, sizeof(rr)); | ||
510 | rc = jffs2_flash_read(c, ref_offset(ref->node), | ||
511 | sizeof(struct jffs2_unknown_node), | ||
512 | &length, (char *)&rr); | ||
513 | if (rc || length != sizeof(struct jffs2_unknown_node)) { | ||
514 | JFFS2_ERROR("jffs2_flash_read()=%d, req=%zu, read=%zu at %#08x\n", | ||
515 | rc, sizeof(struct jffs2_unknown_node), | ||
516 | length, ref_offset(ref->node)); | ||
517 | } | ||
518 | rc = jffs2_flash_write(c, ref_offset(ref->node), sizeof(rr), | ||
519 | &length, (char *)&rr); | ||
520 | if (rc || length != sizeof(struct jffs2_raw_xref)) { | ||
521 | JFFS2_ERROR("jffs2_flash_write()=%d, req=%zu, wrote=%zu at %#08x\n", | ||
522 | rc, sizeof(rr), length, ref_offset(ref->node)); | ||
523 | } | ||
524 | } | ||
525 | spin_lock(&c->erase_completion_lock); | ||
526 | ref->node->next_in_ino = NULL; | ||
527 | spin_unlock(&c->erase_completion_lock); | ||
528 | jffs2_mark_node_obsolete(c, ref->node); | ||
529 | ref->node = NULL; | ||
530 | } | ||
531 | |||
532 | static void delete_xattr_ref(struct jffs2_sb_info *c, struct jffs2_xattr_ref *ref) | ||
533 | { | ||
534 | /* must be called under down_write(xattr_sem) */ | ||
535 | struct jffs2_xattr_datum *xd; | ||
536 | |||
537 | BUG_ON(!ref->node); | ||
538 | delete_xattr_ref_node(c, ref); | ||
539 | |||
540 | xd = ref->xd; | ||
541 | xd->refcnt--; | ||
542 | if (!xd->refcnt) | ||
543 | delete_xattr_datum(c, xd); | ||
544 | jffs2_free_xattr_ref(ref); | ||
545 | } | ||
546 | |||
547 | static int save_xattr_ref(struct jffs2_sb_info *c, struct jffs2_xattr_ref *ref) | 524 | static int save_xattr_ref(struct jffs2_sb_info *c, struct jffs2_xattr_ref *ref) |
548 | { | 525 | { |
549 | /* must be called under down_write(xattr_sem) */ | 526 | /* must be called under down_write(xattr_sem) */ |
550 | struct jffs2_raw_node_ref *raw; | ||
551 | struct jffs2_raw_xref rr; | 527 | struct jffs2_raw_xref rr; |
552 | size_t length; | 528 | size_t length; |
553 | uint32_t phys_ofs = write_ofs(c); | 529 | uint32_t xseqno, phys_ofs = write_ofs(c); |
554 | int ret; | 530 | int ret; |
555 | 531 | ||
556 | rr.magic = cpu_to_je16(JFFS2_MAGIC_BITMASK); | 532 | rr.magic = cpu_to_je16(JFFS2_MAGIC_BITMASK); |
@@ -558,8 +534,16 @@ static int save_xattr_ref(struct jffs2_sb_info *c, struct jffs2_xattr_ref *ref) | |||
558 | rr.totlen = cpu_to_je32(PAD(sizeof(rr))); | 534 | rr.totlen = cpu_to_je32(PAD(sizeof(rr))); |
559 | rr.hdr_crc = cpu_to_je32(crc32(0, &rr, sizeof(struct jffs2_unknown_node) - 4)); | 535 | rr.hdr_crc = cpu_to_je32(crc32(0, &rr, sizeof(struct jffs2_unknown_node) - 4)); |
560 | 536 | ||
561 | rr.ino = cpu_to_je32(ref->ic->ino); | 537 | xseqno = (c->highest_xseqno += 2); |
562 | rr.xid = cpu_to_je32(ref->xd->xid); | 538 | if (is_xattr_ref_dead(ref)) { |
539 | xseqno |= XREF_DELETE_MARKER; | ||
540 | rr.ino = cpu_to_je32(ref->ino); | ||
541 | rr.xid = cpu_to_je32(ref->xid); | ||
542 | } else { | ||
543 | rr.ino = cpu_to_je32(ref->ic->ino); | ||
544 | rr.xid = cpu_to_je32(ref->xd->xid); | ||
545 | } | ||
546 | rr.xseqno = cpu_to_je32(xseqno); | ||
563 | rr.node_crc = cpu_to_je32(crc32(0, &rr, sizeof(rr) - 4)); | 547 | rr.node_crc = cpu_to_je32(crc32(0, &rr, sizeof(rr) - 4)); |
564 | 548 | ||
565 | ret = jffs2_flash_write(c, phys_ofs, sizeof(rr), &length, (char *)&rr); | 549 | ret = jffs2_flash_write(c, phys_ofs, sizeof(rr), &length, (char *)&rr); |
@@ -572,12 +556,9 @@ static int save_xattr_ref(struct jffs2_sb_info *c, struct jffs2_xattr_ref *ref) | |||
572 | 556 | ||
573 | return ret; | 557 | return ret; |
574 | } | 558 | } |
575 | 559 | /* success */ | |
576 | raw = jffs2_add_physical_node_ref(c, phys_ofs | REF_PRISTINE, PAD(sizeof(rr)), NULL); | 560 | ref->xseqno = xseqno; |
577 | /* FIXME */ raw->next_in_ino = (void *)ref; | 561 | jffs2_add_physical_node_ref(c, phys_ofs | REF_PRISTINE, PAD(sizeof(rr)), (void *)ref); |
578 | if (ref->node) | ||
579 | delete_xattr_ref_node(c, ref); | ||
580 | ref->node = raw; | ||
581 | 562 | ||
582 | dbg_xattr("success on saving xref (ino=%u, xid=%u)\n", ref->ic->ino, ref->xd->xid); | 563 | dbg_xattr("success on saving xref (ino=%u, xid=%u)\n", ref->ic->ino, ref->xd->xid); |
583 | 564 | ||
@@ -610,22 +591,116 @@ static struct jffs2_xattr_ref *create_xattr_ref(struct jffs2_sb_info *c, struct | |||
610 | return ref; /* success */ | 591 | return ref; /* success */ |
611 | } | 592 | } |
612 | 593 | ||
594 | static void delete_xattr_ref_delay(struct jffs2_sb_info *c, struct jffs2_xattr_ref *ref) | ||
595 | { | ||
596 | /* must be called under down_write(xattr_sem) */ | ||
597 | struct jffs2_xattr_datum *xd; | ||
598 | |||
599 | set_xattr_ref_dead(ref); | ||
600 | xd = ref->xd; | ||
601 | ref->ino = ref->ic->ino; | ||
602 | ref->xid = ref->xd->xid; | ||
603 | spin_lock(&c->erase_completion_lock); | ||
604 | ref->next = c->xref_dead_list; | ||
605 | c->xref_dead_list = ref; | ||
606 | spin_unlock(&c->erase_completion_lock); | ||
607 | |||
608 | JFFS2_NOTICE("xref(ino=%u, xid=%u) was removed without delete marker. " | ||
609 | "An orphan xref may be detected on next mounting.\n", | ||
610 | ref->ino, ref->xid); | ||
611 | |||
612 | if (!--xd->refcnt) | ||
613 | delete_xattr_datum_delay(c, xd); | ||
614 | } | ||
615 | |||
616 | static int delete_xattr_ref(struct jffs2_sb_info *c, struct jffs2_xattr_ref *ref, int enforce) | ||
617 | { | ||
618 | /* must be called under jffs2_reserve_space() and down_write(xattr_sem) */ | ||
619 | struct jffs2_inode_cache *ic; | ||
620 | struct jffs2_xattr_datum *xd; | ||
621 | uint32_t length; | ||
622 | int rc; | ||
623 | |||
624 | set_xattr_ref_dead(ref); | ||
625 | ic = ref->ic; | ||
626 | xd = ref->xd; | ||
627 | ref->ino = ic->ino; | ||
628 | ref->xid = xd->xid; | ||
629 | rc = save_xattr_ref(c, ref); | ||
630 | if (rc) { | ||
631 | if (!enforce) { | ||
632 | clr_xattr_ref_dead(ref); | ||
633 | ref->ic = ic; | ||
634 | ref->xd = xd; | ||
635 | return rc; | ||
636 | } | ||
637 | JFFS2_WARNING("could not write delete marker of xref(ino=%u, xid=%u). " | ||
638 | "An orphan xref may be detected on next mounting.\n", | ||
639 | ref->ic->ino, ref->xd->xid); | ||
640 | } | ||
641 | spin_lock(&c->erase_completion_lock); | ||
642 | ref->next = c->xref_dead_list; | ||
643 | c->xref_dead_list = ref; | ||
644 | spin_unlock(&c->erase_completion_lock); | ||
645 | |||
646 | xd->refcnt--; | ||
647 | if (xd->refcnt) | ||
648 | return 0; | ||
649 | |||
650 | /* delete xdatum */ | ||
651 | unload_xattr_datum(c, xd); | ||
652 | up_write(&c->xattr_sem); | ||
653 | jffs2_complete_reservation(c); | ||
654 | |||
655 | rc = jffs2_reserve_space(c, PAD(sizeof(struct jffs2_raw_xattr)), &length, | ||
656 | ALLOC_DELETION, JFFS2_SUMMARY_XATTR_SIZE); | ||
657 | if (rc) { | ||
658 | down(&c->alloc_sem); | ||
659 | down_write(&c->xattr_sem); | ||
660 | delete_xattr_datum_delay(c, xd); | ||
661 | } else { | ||
662 | down_write(&c->xattr_sem); | ||
663 | delete_xattr_datum(c, xd); | ||
664 | } | ||
665 | return 0; | ||
666 | } | ||
667 | |||
613 | void jffs2_xattr_delete_inode(struct jffs2_sb_info *c, struct jffs2_inode_cache *ic) | 668 | void jffs2_xattr_delete_inode(struct jffs2_sb_info *c, struct jffs2_inode_cache *ic) |
614 | { | 669 | { |
615 | /* It's called from jffs2_clear_inode() on inode removing. | 670 | /* It's called from jffs2_clear_inode() on inode removing. |
616 | When an inode with XATTR is removed, those XATTRs must be removed. */ | 671 | When an inode with XATTR is removed, those XATTRs must be removed. */ |
617 | struct jffs2_xattr_ref *ref, *_ref; | 672 | struct jffs2_xattr_ref *ref; |
673 | uint32_t length; | ||
674 | int rc, retry; | ||
618 | 675 | ||
619 | if (!ic || ic->nlink > 0) | 676 | if (!ic || ic->nlink > 0) |
620 | return; | 677 | return; |
621 | 678 | ||
679 | down_read(&c->xattr_sem); | ||
680 | if (!ic->xref) { | ||
681 | up_read(&c->xattr_sem); | ||
682 | return; | ||
683 | } | ||
684 | up_read(&c->xattr_sem); | ||
685 | retry: | ||
686 | rc = jffs2_reserve_space(c, PAD(sizeof(struct jffs2_raw_xref)), &length, | ||
687 | ALLOC_DELETION, JFFS2_SUMMARY_XREF_SIZE); | ||
622 | down_write(&c->xattr_sem); | 688 | down_write(&c->xattr_sem); |
623 | for (ref = ic->xref; ref; ref = _ref) { | 689 | if (ic->xref) { |
624 | _ref = ref->next; | 690 | ref = ic->xref; |
625 | delete_xattr_ref(c, ref); | 691 | ic->xref = ref->next; |
692 | if (rc) { | ||
693 | delete_xattr_ref_delay(c, ref); | ||
694 | } else { | ||
695 | delete_xattr_ref(c, ref, 1); | ||
696 | } | ||
626 | } | 697 | } |
627 | ic->xref = NULL; | 698 | retry = ic->xref ? 1 : 0; |
628 | up_write(&c->xattr_sem); | 699 | up_write(&c->xattr_sem); |
700 | if (!rc) | ||
701 | jffs2_complete_reservation(c); | ||
702 | if (retry) | ||
703 | goto retry; | ||
629 | } | 704 | } |
630 | 705 | ||
631 | void jffs2_xattr_free_inode(struct jffs2_sb_info *c, struct jffs2_inode_cache *ic) | 706 | void jffs2_xattr_free_inode(struct jffs2_sb_info *c, struct jffs2_inode_cache *ic) |
@@ -655,7 +730,7 @@ static int check_xattr_ref_inode(struct jffs2_sb_info *c, struct jffs2_inode_cac | |||
655 | * duplicate name/value pairs. If duplicate name/value pair would be found, | 730 | * duplicate name/value pairs. If duplicate name/value pair would be found, |
656 | * one will be removed. | 731 | * one will be removed. |
657 | */ | 732 | */ |
658 | struct jffs2_xattr_ref *ref, *cmp, **pref; | 733 | struct jffs2_xattr_ref *ref, *cmp, **pref, **pcmp; |
659 | int rc = 0; | 734 | int rc = 0; |
660 | 735 | ||
661 | if (likely(ic->flags & INO_FLAGS_XATTR_CHECKED)) | 736 | if (likely(ic->flags & INO_FLAGS_XATTR_CHECKED)) |
@@ -668,27 +743,32 @@ static int check_xattr_ref_inode(struct jffs2_sb_info *c, struct jffs2_inode_cac | |||
668 | rc = load_xattr_datum(c, ref->xd); | 743 | rc = load_xattr_datum(c, ref->xd); |
669 | if (unlikely(rc > 0)) { | 744 | if (unlikely(rc > 0)) { |
670 | *pref = ref->next; | 745 | *pref = ref->next; |
671 | delete_xattr_ref(c, ref); | 746 | delete_xattr_ref_delay(c, ref); |
672 | goto retry; | 747 | goto retry; |
673 | } else if (unlikely(rc < 0)) | 748 | } else if (unlikely(rc < 0)) |
674 | goto out; | 749 | goto out; |
675 | } | 750 | } |
676 | for (cmp=ref->next, pref=&ref->next; cmp; pref=&cmp->next, cmp=cmp->next) { | 751 | for (cmp=ref->next, pcmp=&ref->next; cmp; pcmp=&cmp->next, cmp=cmp->next) { |
677 | if (!cmp->xd->xname) { | 752 | if (!cmp->xd->xname) { |
678 | ref->xd->flags |= JFFS2_XFLAGS_BIND; | 753 | ref->xd->flags |= JFFS2_XFLAGS_BIND; |
679 | rc = load_xattr_datum(c, cmp->xd); | 754 | rc = load_xattr_datum(c, cmp->xd); |
680 | ref->xd->flags &= ~JFFS2_XFLAGS_BIND; | 755 | ref->xd->flags &= ~JFFS2_XFLAGS_BIND; |
681 | if (unlikely(rc > 0)) { | 756 | if (unlikely(rc > 0)) { |
682 | *pref = cmp->next; | 757 | *pcmp = cmp->next; |
683 | delete_xattr_ref(c, cmp); | 758 | delete_xattr_ref_delay(c, cmp); |
684 | goto retry; | 759 | goto retry; |
685 | } else if (unlikely(rc < 0)) | 760 | } else if (unlikely(rc < 0)) |
686 | goto out; | 761 | goto out; |
687 | } | 762 | } |
688 | if (ref->xd->xprefix == cmp->xd->xprefix | 763 | if (ref->xd->xprefix == cmp->xd->xprefix |
689 | && !strcmp(ref->xd->xname, cmp->xd->xname)) { | 764 | && !strcmp(ref->xd->xname, cmp->xd->xname)) { |
690 | *pref = cmp->next; | 765 | if (ref->xseqno > cmp->xseqno) { |
691 | delete_xattr_ref(c, cmp); | 766 | *pcmp = cmp->next; |
767 | delete_xattr_ref_delay(c, cmp); | ||
768 | } else { | ||
769 | *pref = ref->next; | ||
770 | delete_xattr_ref_delay(c, ref); | ||
771 | } | ||
692 | goto retry; | 772 | goto retry; |
693 | } | 773 | } |
694 | } | 774 | } |
@@ -719,9 +799,13 @@ void jffs2_init_xattr_subsystem(struct jffs2_sb_info *c) | |||
719 | for (i=0; i < XATTRINDEX_HASHSIZE; i++) | 799 | for (i=0; i < XATTRINDEX_HASHSIZE; i++) |
720 | INIT_LIST_HEAD(&c->xattrindex[i]); | 800 | INIT_LIST_HEAD(&c->xattrindex[i]); |
721 | INIT_LIST_HEAD(&c->xattr_unchecked); | 801 | INIT_LIST_HEAD(&c->xattr_unchecked); |
802 | INIT_LIST_HEAD(&c->xattr_dead_list); | ||
803 | c->xref_dead_list = NULL; | ||
722 | c->xref_temp = NULL; | 804 | c->xref_temp = NULL; |
723 | 805 | ||
724 | init_rwsem(&c->xattr_sem); | 806 | init_rwsem(&c->xattr_sem); |
807 | c->highest_xid = 0; | ||
808 | c->highest_xseqno = 0; | ||
725 | c->xdatum_mem_usage = 0; | 809 | c->xdatum_mem_usage = 0; |
726 | c->xdatum_mem_threshold = 32 * 1024; /* Default 32KB */ | 810 | c->xdatum_mem_threshold = 32 * 1024; /* Default 32KB */ |
727 | } | 811 | } |
@@ -751,7 +835,11 @@ void jffs2_clear_xattr_subsystem(struct jffs2_sb_info *c) | |||
751 | _ref = ref->next; | 835 | _ref = ref->next; |
752 | jffs2_free_xattr_ref(ref); | 836 | jffs2_free_xattr_ref(ref); |
753 | } | 837 | } |
754 | c->xref_temp = NULL; | 838 | |
839 | for (ref=c->xref_dead_list; ref; ref = _ref) { | ||
840 | _ref = ref->next; | ||
841 | jffs2_free_xattr_ref(ref); | ||
842 | } | ||
755 | 843 | ||
756 | for (i=0; i < XATTRINDEX_HASHSIZE; i++) { | 844 | for (i=0; i < XATTRINDEX_HASHSIZE; i++) { |
757 | list_for_each_entry_safe(xd, _xd, &c->xattrindex[i], xindex) { | 845 | list_for_each_entry_safe(xd, _xd, &c->xattrindex[i], xindex) { |
@@ -761,64 +849,120 @@ void jffs2_clear_xattr_subsystem(struct jffs2_sb_info *c) | |||
761 | jffs2_free_xattr_datum(xd); | 849 | jffs2_free_xattr_datum(xd); |
762 | } | 850 | } |
763 | } | 851 | } |
852 | |||
853 | list_for_each_entry_safe(xd, _xd, &c->xattr_dead_list, xindex) { | ||
854 | list_del(&xd->xindex); | ||
855 | jffs2_free_xattr_datum(xd); | ||
856 | } | ||
764 | } | 857 | } |
765 | 858 | ||
859 | #define XREF_TMPHASH_SIZE (128) | ||
766 | void jffs2_build_xattr_subsystem(struct jffs2_sb_info *c) | 860 | void jffs2_build_xattr_subsystem(struct jffs2_sb_info *c) |
767 | { | 861 | { |
768 | struct jffs2_xattr_ref *ref, *_ref; | 862 | struct jffs2_xattr_ref *ref, *_ref; |
863 | struct jffs2_xattr_ref *xref_tmphash[XREF_TMPHASH_SIZE]; | ||
769 | struct jffs2_xattr_datum *xd, *_xd; | 864 | struct jffs2_xattr_datum *xd, *_xd; |
770 | struct jffs2_inode_cache *ic; | 865 | struct jffs2_inode_cache *ic; |
771 | int i, xdatum_count =0, xdatum_unchecked_count = 0, xref_count = 0; | 866 | struct jffs2_raw_node_ref *raw; |
867 | int i, xdatum_count = 0, xdatum_unchecked_count = 0, xref_count = 0; | ||
772 | 868 | ||
773 | BUG_ON(!(c->flags & JFFS2_SB_FLAG_BUILDING)); | 869 | BUG_ON(!(c->flags & JFFS2_SB_FLAG_BUILDING)); |
870 | /* Phase.1 : Drop dead xdatum */ | ||
871 | for (i=0; i < XATTRINDEX_HASHSIZE; i++) { | ||
872 | list_for_each_entry_safe(xd, _xd, &c->xattrindex[i], xindex) { | ||
873 | BUG_ON(xd->node == (void *)xd); | ||
874 | if (is_xattr_datum_dead(xd)) { | ||
875 | list_del_init(&xd->xindex); | ||
876 | list_add(&xd->xindex, &c->xattr_unchecked); | ||
877 | } | ||
878 | } | ||
879 | } | ||
774 | 880 | ||
775 | /* Phase.1 */ | 881 | /* Phase.2 : Merge same xref */ |
882 | for (i=0; i < XREF_TMPHASH_SIZE; i++) | ||
883 | xref_tmphash[i] = NULL; | ||
776 | for (ref=c->xref_temp; ref; ref=_ref) { | 884 | for (ref=c->xref_temp; ref; ref=_ref) { |
885 | struct jffs2_xattr_ref *tmp; | ||
886 | |||
777 | _ref = ref->next; | 887 | _ref = ref->next; |
778 | /* checking REF_UNCHECKED nodes */ | ||
779 | if (ref_flags(ref->node) != REF_PRISTINE) { | 888 | if (ref_flags(ref->node) != REF_PRISTINE) { |
780 | if (verify_xattr_ref(c, ref)) { | 889 | if (verify_xattr_ref(c, ref)) { |
781 | delete_xattr_ref_node(c, ref); | 890 | BUG_ON(ref->node->next_in_ino != (void *)ref); |
891 | ref->node->next_in_ino = NULL; | ||
892 | jffs2_mark_node_obsolete(c, ref->node); | ||
782 | jffs2_free_xattr_ref(ref); | 893 | jffs2_free_xattr_ref(ref); |
783 | continue; | 894 | continue; |
784 | } | 895 | } |
785 | } | 896 | } |
786 | /* At this point, ref->xid and ref->ino contain XID and inode number. | 897 | |
787 | ref->xd and ref->ic are not valid yet. */ | 898 | i = (ref->ino ^ ref->xid) % XREF_TMPHASH_SIZE; |
788 | xd = jffs2_find_xattr_datum(c, ref->xid); | 899 | for (tmp=xref_tmphash[i]; tmp; tmp=tmp->next) { |
789 | ic = jffs2_get_ino_cache(c, ref->ino); | 900 | if (tmp->ino == ref->ino && tmp->xid == ref->xid) |
790 | if (!xd || !ic) { | 901 | break; |
791 | if (ref_flags(ref->node) != REF_UNCHECKED) | 902 | } |
792 | JFFS2_WARNING("xref(ino=%u, xid=%u) is orphan. \n", | 903 | if (tmp) { |
793 | ref->ino, ref->xid); | 904 | raw = ref->node; |
794 | delete_xattr_ref_node(c, ref); | 905 | if (ref->xseqno > tmp->xseqno) { |
906 | tmp->xseqno = ref->xseqno; | ||
907 | raw->next_in_ino = tmp->node; | ||
908 | tmp->node = raw; | ||
909 | } else { | ||
910 | raw->next_in_ino = tmp->node->next_in_ino; | ||
911 | tmp->node->next_in_ino = raw; | ||
912 | } | ||
795 | jffs2_free_xattr_ref(ref); | 913 | jffs2_free_xattr_ref(ref); |
796 | continue; | 914 | continue; |
915 | } else { | ||
916 | ref->next = xref_tmphash[i]; | ||
917 | xref_tmphash[i] = ref; | ||
797 | } | 918 | } |
798 | ref->xd = xd; | ||
799 | ref->ic = ic; | ||
800 | xd->refcnt++; | ||
801 | ref->next = ic->xref; | ||
802 | ic->xref = ref; | ||
803 | xref_count++; | ||
804 | } | 919 | } |
805 | c->xref_temp = NULL; | 920 | c->xref_temp = NULL; |
806 | /* After this, ref->xid/ino are NEVER used. */ | ||
807 | 921 | ||
808 | /* Phase.2 */ | 922 | /* Phase.3 : Bind xref with inode_cache and xattr_datum */ |
923 | for (i=0; i < XREF_TMPHASH_SIZE; i++) { | ||
924 | for (ref=xref_tmphash[i]; ref; ref=_ref) { | ||
925 | _ref = ref->next; | ||
926 | if (is_xattr_ref_dead(ref)) { | ||
927 | ref->next = c->xref_dead_list; | ||
928 | c->xref_dead_list = ref; | ||
929 | continue; | ||
930 | } | ||
931 | /* At this point, ref->xid and ref->ino contain XID and inode number. | ||
932 | ref->xd and ref->ic are not valid yet. */ | ||
933 | xd = jffs2_find_xattr_datum(c, ref->xid); | ||
934 | ic = jffs2_get_ino_cache(c, ref->ino); | ||
935 | if (!xd || !ic) { | ||
936 | JFFS2_WARNING("xref(ino=%u, xid=%u, xseqno=%u) is orphan. \n", | ||
937 | ref->ino, ref->xid, ref->xseqno); | ||
938 | set_xattr_ref_dead(ref); | ||
939 | ref->next = c->xref_dead_list; | ||
940 | c->xref_dead_list = ref; | ||
941 | continue; | ||
942 | } | ||
943 | ref->xd = xd; | ||
944 | ref->ic = ic; | ||
945 | xd->refcnt++; | ||
946 | ref->next = ic->xref; | ||
947 | ic->xref = ref; | ||
948 | xref_count++; | ||
949 | } | ||
950 | } | ||
951 | |||
952 | /* Phase.4 : Link unchecked xdatum to xattr_unchecked list */ | ||
809 | for (i=0; i < XATTRINDEX_HASHSIZE; i++) { | 953 | for (i=0; i < XATTRINDEX_HASHSIZE; i++) { |
810 | list_for_each_entry_safe(xd, _xd, &c->xattrindex[i], xindex) { | 954 | list_for_each_entry_safe(xd, _xd, &c->xattrindex[i], xindex) { |
811 | list_del_init(&xd->xindex); | 955 | list_del_init(&xd->xindex); |
812 | if (!xd->refcnt) { | 956 | if (!xd->refcnt) { |
813 | if (ref_flags(xd->node) != REF_UNCHECKED) | 957 | JFFS2_WARNING("orphan xdatum(xid=%u, version=%u)\n", |
814 | JFFS2_WARNING("orphan xdatum(xid=%u, version=%u) at %#08x\n", | 958 | xd->xid, xd->version); |
815 | xd->xid, xd->version, ref_offset(xd->node)); | 959 | set_xattr_datum_dead(xd); |
816 | delete_xattr_datum(c, xd); | 960 | list_add(&xd->xindex, &c->xattr_unchecked); |
817 | continue; | 961 | continue; |
818 | } | 962 | } |
819 | if (ref_flags(xd->node) != REF_PRISTINE) { | 963 | if (is_xattr_datum_unchecked(c, xd)) { |
820 | dbg_xattr("unchecked xdatum(xid=%u) at %#08x\n", | 964 | dbg_xattr("unchecked xdatum(xid=%u, version=%u)\n", |
821 | xd->xid, ref_offset(xd->node)); | 965 | xd->xid, xd->version); |
822 | list_add(&xd->xindex, &c->xattr_unchecked); | 966 | list_add(&xd->xindex, &c->xattr_unchecked); |
823 | xdatum_unchecked_count++; | 967 | xdatum_unchecked_count++; |
824 | } | 968 | } |
@@ -833,28 +977,18 @@ void jffs2_build_xattr_subsystem(struct jffs2_sb_info *c) | |||
833 | struct jffs2_xattr_datum *jffs2_setup_xattr_datum(struct jffs2_sb_info *c, | 977 | struct jffs2_xattr_datum *jffs2_setup_xattr_datum(struct jffs2_sb_info *c, |
834 | uint32_t xid, uint32_t version) | 978 | uint32_t xid, uint32_t version) |
835 | { | 979 | { |
836 | struct jffs2_xattr_datum *xd, *_xd; | 980 | struct jffs2_xattr_datum *xd; |
837 | 981 | ||
838 | _xd = jffs2_find_xattr_datum(c, xid); | 982 | xd = jffs2_find_xattr_datum(c, xid); |
839 | if (_xd) { | 983 | if (!xd) { |
840 | dbg_xattr("duplicate xdatum (xid=%u, version=%u/%u) at %#08x\n", | 984 | xd = jffs2_alloc_xattr_datum(); |
841 | xid, version, _xd->version, ref_offset(_xd->node)); | 985 | if (!xd) |
842 | if (version < _xd->version) | 986 | return ERR_PTR(-ENOMEM); |
843 | return ERR_PTR(-EEXIST); | 987 | xd->xid = xid; |
844 | } | 988 | xd->version = version; |
845 | xd = jffs2_alloc_xattr_datum(); | 989 | if (xd->xid > c->highest_xid) |
846 | if (!xd) | 990 | c->highest_xid = xd->xid; |
847 | return ERR_PTR(-ENOMEM); | 991 | list_add_tail(&xd->xindex, &c->xattrindex[xid % XATTRINDEX_HASHSIZE]); |
848 | xd->xid = xid; | ||
849 | xd->version = version; | ||
850 | if (xd->xid > c->highest_xid) | ||
851 | c->highest_xid = xd->xid; | ||
852 | list_add_tail(&xd->xindex, &c->xattrindex[xid % XATTRINDEX_HASHSIZE]); | ||
853 | |||
854 | if (_xd) { | ||
855 | list_del_init(&_xd->xindex); | ||
856 | delete_xattr_datum_node(c, _xd); | ||
857 | jffs2_free_xattr_datum(_xd); | ||
858 | } | 992 | } |
859 | return xd; | 993 | return xd; |
860 | } | 994 | } |
@@ -945,7 +1079,7 @@ ssize_t jffs2_listxattr(struct dentry *dentry, char *buffer, size_t size) | |||
945 | rc = load_xattr_datum(c, xd); | 1079 | rc = load_xattr_datum(c, xd); |
946 | if (unlikely(rc > 0)) { | 1080 | if (unlikely(rc > 0)) { |
947 | *pref = ref->next; | 1081 | *pref = ref->next; |
948 | delete_xattr_ref(c, ref); | 1082 | delete_xattr_ref_delay(c, ref); |
949 | goto retry; | 1083 | goto retry; |
950 | } else if (unlikely(rc < 0)) | 1084 | } else if (unlikely(rc < 0)) |
951 | goto out; | 1085 | goto out; |
@@ -1006,7 +1140,7 @@ int do_jffs2_getxattr(struct inode *inode, int xprefix, const char *xname, | |||
1006 | rc = load_xattr_datum(c, xd); | 1140 | rc = load_xattr_datum(c, xd); |
1007 | if (unlikely(rc > 0)) { | 1141 | if (unlikely(rc > 0)) { |
1008 | *pref = ref->next; | 1142 | *pref = ref->next; |
1009 | delete_xattr_ref(c, ref); | 1143 | delete_xattr_ref_delay(c, ref); |
1010 | goto retry; | 1144 | goto retry; |
1011 | } else if (unlikely(rc < 0)) { | 1145 | } else if (unlikely(rc < 0)) { |
1012 | goto out; | 1146 | goto out; |
@@ -1069,7 +1203,7 @@ int do_jffs2_setxattr(struct inode *inode, int xprefix, const char *xname, | |||
1069 | rc = load_xattr_datum(c, xd); | 1203 | rc = load_xattr_datum(c, xd); |
1070 | if (unlikely(rc > 0)) { | 1204 | if (unlikely(rc > 0)) { |
1071 | *pref = ref->next; | 1205 | *pref = ref->next; |
1072 | delete_xattr_ref(c, ref); | 1206 | delete_xattr_ref_delay(c, ref); |
1073 | goto retry; | 1207 | goto retry; |
1074 | } else if (unlikely(rc < 0)) | 1208 | } else if (unlikely(rc < 0)) |
1075 | goto out; | 1209 | goto out; |
@@ -1081,8 +1215,7 @@ int do_jffs2_setxattr(struct inode *inode, int xprefix, const char *xname, | |||
1081 | } | 1215 | } |
1082 | if (!buffer) { | 1216 | if (!buffer) { |
1083 | *pref = ref->next; | 1217 | *pref = ref->next; |
1084 | delete_xattr_ref(c, ref); | 1218 | rc = delete_xattr_ref(c, ref, 0); |
1085 | rc = 0; | ||
1086 | goto out; | 1219 | goto out; |
1087 | } | 1220 | } |
1088 | goto found; | 1221 | goto found; |
@@ -1094,7 +1227,7 @@ int do_jffs2_setxattr(struct inode *inode, int xprefix, const char *xname, | |||
1094 | goto out; | 1227 | goto out; |
1095 | } | 1228 | } |
1096 | if (!buffer) { | 1229 | if (!buffer) { |
1097 | rc = -EINVAL; | 1230 | rc = -ENODATA; |
1098 | goto out; | 1231 | goto out; |
1099 | } | 1232 | } |
1100 | found: | 1233 | found: |
@@ -1110,16 +1243,15 @@ int do_jffs2_setxattr(struct inode *inode, int xprefix, const char *xname, | |||
1110 | request = PAD(sizeof(struct jffs2_raw_xref)); | 1243 | request = PAD(sizeof(struct jffs2_raw_xref)); |
1111 | rc = jffs2_reserve_space(c, request, &length, | 1244 | rc = jffs2_reserve_space(c, request, &length, |
1112 | ALLOC_NORMAL, JFFS2_SUMMARY_XREF_SIZE); | 1245 | ALLOC_NORMAL, JFFS2_SUMMARY_XREF_SIZE); |
1246 | down_write(&c->xattr_sem); | ||
1113 | if (rc) { | 1247 | if (rc) { |
1114 | JFFS2_WARNING("jffs2_reserve_space()=%d, request=%u\n", rc, request); | 1248 | JFFS2_WARNING("jffs2_reserve_space()=%d, request=%u\n", rc, request); |
1115 | down_write(&c->xattr_sem); | ||
1116 | xd->refcnt--; | 1249 | xd->refcnt--; |
1117 | if (!xd->refcnt) | 1250 | if (!xd->refcnt) |
1118 | delete_xattr_datum(c, xd); | 1251 | delete_xattr_datum_delay(c, xd); |
1119 | up_write(&c->xattr_sem); | 1252 | up_write(&c->xattr_sem); |
1120 | return rc; | 1253 | return rc; |
1121 | } | 1254 | } |
1122 | down_write(&c->xattr_sem); | ||
1123 | if (ref) | 1255 | if (ref) |
1124 | *pref = ref->next; | 1256 | *pref = ref->next; |
1125 | newref = create_xattr_ref(c, ic, xd); | 1257 | newref = create_xattr_ref(c, ic, xd); |
@@ -1131,9 +1263,21 @@ int do_jffs2_setxattr(struct inode *inode, int xprefix, const char *xname, | |||
1131 | rc = PTR_ERR(newref); | 1263 | rc = PTR_ERR(newref); |
1132 | xd->refcnt--; | 1264 | xd->refcnt--; |
1133 | if (!xd->refcnt) | 1265 | if (!xd->refcnt) |
1134 | delete_xattr_datum(c, xd); | 1266 | delete_xattr_datum_delay(c, xd); |
1135 | } else if (ref) { | 1267 | } else if (ref) { |
1136 | delete_xattr_ref(c, ref); | 1268 | up_write(&c->xattr_sem); |
1269 | jffs2_complete_reservation(c); | ||
1270 | |||
1271 | rc = jffs2_reserve_space(c, request, &length, | ||
1272 | ALLOC_DELETION, JFFS2_SUMMARY_XREF_SIZE); | ||
1273 | down_write(&c->xattr_sem); | ||
1274 | if (rc) { | ||
1275 | JFFS2_WARNING("jffs2_reserve_space()=%d, request=%u\n", rc, request); | ||
1276 | delete_xattr_ref_delay(c, ref); | ||
1277 | up_write(&c->xattr_sem); | ||
1278 | return 0; | ||
1279 | } | ||
1280 | delete_xattr_ref(c, ref, 1); | ||
1137 | } | 1281 | } |
1138 | out: | 1282 | out: |
1139 | up_write(&c->xattr_sem); | 1283 | up_write(&c->xattr_sem); |
@@ -1142,38 +1286,37 @@ int do_jffs2_setxattr(struct inode *inode, int xprefix, const char *xname, | |||
1142 | } | 1286 | } |
1143 | 1287 | ||
1144 | /* -------- garbage collector functions ------------- | 1288 | /* -------- garbage collector functions ------------- |
1145 | * jffs2_garbage_collect_xattr_datum(c, xd) | 1289 | * jffs2_garbage_collect_xattr_datum(c, xd, raw) |
1146 | * is used to move xdatum into new node. | 1290 | * is used to move xdatum into new node. |
1147 | * jffs2_garbage_collect_xattr_ref(c, ref) | 1291 | * jffs2_garbage_collect_xattr_ref(c, ref, raw) |
1148 | * is used to move xref into new node. | 1292 | * is used to move xref into new node. |
1149 | * jffs2_verify_xattr(c) | 1293 | * jffs2_verify_xattr(c) |
1150 | * is used to call do_verify_xattr_datum() before garbage collecting. | 1294 | * is used to call do_verify_xattr_datum() before garbage collecting. |
1151 | * -------------------------------------------------- */ | 1295 | * -------------------------------------------------- */ |
1152 | int jffs2_garbage_collect_xattr_datum(struct jffs2_sb_info *c, struct jffs2_xattr_datum *xd) | 1296 | int jffs2_garbage_collect_xattr_datum(struct jffs2_sb_info *c, struct jffs2_xattr_datum *xd, |
1297 | struct jffs2_raw_node_ref *raw) | ||
1153 | { | 1298 | { |
1154 | uint32_t totlen, length, old_ofs; | 1299 | uint32_t totlen, length, old_ofs; |
1155 | int rc = -EINVAL; | 1300 | int rc = 0; |
1156 | 1301 | ||
1157 | down_write(&c->xattr_sem); | 1302 | down_write(&c->xattr_sem); |
1158 | BUG_ON(!xd->node); | 1303 | if (xd->node != raw) |
1304 | goto out; | ||
1305 | if (is_xattr_datum_dead(xd) && (raw->next_in_ino == (void *)xd)) | ||
1306 | goto out; | ||
1159 | 1307 | ||
1160 | old_ofs = ref_offset(xd->node); | 1308 | old_ofs = ref_offset(xd->node); |
1161 | totlen = ref_totlen(c, c->gcblock, xd->node); | 1309 | totlen = ref_totlen(c, c->gcblock, xd->node); |
1162 | if (totlen < sizeof(struct jffs2_raw_xattr)) | ||
1163 | goto out; | ||
1164 | 1310 | ||
1165 | if (!xd->xname) { | 1311 | if (!is_xattr_datum_dead(xd)) { |
1166 | rc = load_xattr_datum(c, xd); | 1312 | rc = load_xattr_datum(c, xd); |
1167 | if (unlikely(rc > 0)) { | 1313 | if (unlikely(rc < 0)) |
1168 | delete_xattr_datum_node(c, xd); | ||
1169 | rc = 0; | ||
1170 | goto out; | ||
1171 | } else if (unlikely(rc < 0)) | ||
1172 | goto out; | 1314 | goto out; |
1173 | } | 1315 | } |
1316 | |||
1174 | rc = jffs2_reserve_space_gc(c, totlen, &length, JFFS2_SUMMARY_XATTR_SIZE); | 1317 | rc = jffs2_reserve_space_gc(c, totlen, &length, JFFS2_SUMMARY_XATTR_SIZE); |
1175 | if (rc || length < totlen) { | 1318 | if (rc) { |
1176 | JFFS2_WARNING("jffs2_reserve_space()=%d, request=%u\n", rc, totlen); | 1319 | JFFS2_WARNING("jffs2_reserve_space_gc()=%d, request=%u\n", rc, totlen); |
1177 | rc = rc ? rc : -EBADFD; | 1320 | rc = rc ? rc : -EBADFD; |
1178 | goto out; | 1321 | goto out; |
1179 | } | 1322 | } |
@@ -1182,27 +1325,33 @@ int jffs2_garbage_collect_xattr_datum(struct jffs2_sb_info *c, struct jffs2_xatt | |||
1182 | dbg_xattr("xdatum (xid=%u, version=%u) GC'ed from %#08x to %08x\n", | 1325 | dbg_xattr("xdatum (xid=%u, version=%u) GC'ed from %#08x to %08x\n", |
1183 | xd->xid, xd->version, old_ofs, ref_offset(xd->node)); | 1326 | xd->xid, xd->version, old_ofs, ref_offset(xd->node)); |
1184 | out: | 1327 | out: |
1328 | if (!rc) | ||
1329 | jffs2_mark_node_obsolete(c, raw); | ||
1185 | up_write(&c->xattr_sem); | 1330 | up_write(&c->xattr_sem); |
1186 | return rc; | 1331 | return rc; |
1187 | } | 1332 | } |
1188 | 1333 | ||
1189 | 1334 | ||
1190 | int jffs2_garbage_collect_xattr_ref(struct jffs2_sb_info *c, struct jffs2_xattr_ref *ref) | 1335 | int jffs2_garbage_collect_xattr_ref(struct jffs2_sb_info *c, struct jffs2_xattr_ref *ref, |
1336 | struct jffs2_raw_node_ref *raw) | ||
1191 | { | 1337 | { |
1192 | uint32_t totlen, length, old_ofs; | 1338 | uint32_t totlen, length, old_ofs; |
1193 | int rc = -EINVAL; | 1339 | int rc = 0; |
1194 | 1340 | ||
1195 | down_write(&c->xattr_sem); | 1341 | down_write(&c->xattr_sem); |
1196 | BUG_ON(!ref->node); | 1342 | BUG_ON(!ref->node); |
1197 | 1343 | ||
1344 | if (ref->node != raw) | ||
1345 | goto out; | ||
1346 | if (is_xattr_ref_dead(ref) && (raw->next_in_ino == (void *)ref)) | ||
1347 | goto out; | ||
1348 | |||
1198 | old_ofs = ref_offset(ref->node); | 1349 | old_ofs = ref_offset(ref->node); |
1199 | totlen = ref_totlen(c, c->gcblock, ref->node); | 1350 | totlen = ref_totlen(c, c->gcblock, ref->node); |
1200 | if (totlen != sizeof(struct jffs2_raw_xref)) | ||
1201 | goto out; | ||
1202 | 1351 | ||
1203 | rc = jffs2_reserve_space_gc(c, totlen, &length, JFFS2_SUMMARY_XREF_SIZE); | 1352 | rc = jffs2_reserve_space_gc(c, totlen, &length, JFFS2_SUMMARY_XREF_SIZE); |
1204 | if (rc || length < totlen) { | 1353 | if (rc) { |
1205 | JFFS2_WARNING("%s: jffs2_reserve_space() = %d, request = %u\n", | 1354 | JFFS2_WARNING("%s: jffs2_reserve_space_gc() = %d, request = %u\n", |
1206 | __FUNCTION__, rc, totlen); | 1355 | __FUNCTION__, rc, totlen); |
1207 | rc = rc ? rc : -EBADFD; | 1356 | rc = rc ? rc : -EBADFD; |
1208 | goto out; | 1357 | goto out; |
@@ -1212,6 +1361,8 @@ int jffs2_garbage_collect_xattr_ref(struct jffs2_sb_info *c, struct jffs2_xattr_ | |||
1212 | dbg_xattr("xref (ino=%u, xid=%u) GC'ed from %#08x to %08x\n", | 1361 | dbg_xattr("xref (ino=%u, xid=%u) GC'ed from %#08x to %08x\n", |
1213 | ref->ic->ino, ref->xd->xid, old_ofs, ref_offset(ref->node)); | 1362 | ref->ic->ino, ref->xd->xid, old_ofs, ref_offset(ref->node)); |
1214 | out: | 1363 | out: |
1364 | if (!rc) | ||
1365 | jffs2_mark_node_obsolete(c, raw); | ||
1215 | up_write(&c->xattr_sem); | 1366 | up_write(&c->xattr_sem); |
1216 | return rc; | 1367 | return rc; |
1217 | } | 1368 | } |
@@ -1219,20 +1370,59 @@ int jffs2_garbage_collect_xattr_ref(struct jffs2_sb_info *c, struct jffs2_xattr_ | |||
1219 | int jffs2_verify_xattr(struct jffs2_sb_info *c) | 1370 | int jffs2_verify_xattr(struct jffs2_sb_info *c) |
1220 | { | 1371 | { |
1221 | struct jffs2_xattr_datum *xd, *_xd; | 1372 | struct jffs2_xattr_datum *xd, *_xd; |
1373 | struct jffs2_eraseblock *jeb; | ||
1374 | struct jffs2_raw_node_ref *raw; | ||
1375 | uint32_t totlen; | ||
1222 | int rc; | 1376 | int rc; |
1223 | 1377 | ||
1224 | down_write(&c->xattr_sem); | 1378 | down_write(&c->xattr_sem); |
1225 | list_for_each_entry_safe(xd, _xd, &c->xattr_unchecked, xindex) { | 1379 | list_for_each_entry_safe(xd, _xd, &c->xattr_unchecked, xindex) { |
1226 | rc = do_verify_xattr_datum(c, xd); | 1380 | rc = do_verify_xattr_datum(c, xd); |
1227 | if (rc == 0) { | 1381 | if (rc < 0) |
1228 | list_del_init(&xd->xindex); | 1382 | continue; |
1229 | break; | 1383 | list_del_init(&xd->xindex); |
1230 | } else if (rc > 0) { | 1384 | spin_lock(&c->erase_completion_lock); |
1231 | list_del_init(&xd->xindex); | 1385 | for (raw=xd->node; raw != (void *)xd; raw=raw->next_in_ino) { |
1232 | delete_xattr_datum_node(c, xd); | 1386 | if (ref_flags(raw) != REF_UNCHECKED) |
1387 | continue; | ||
1388 | jeb = &c->blocks[ref_offset(raw) / c->sector_size]; | ||
1389 | totlen = PAD(ref_totlen(c, jeb, raw)); | ||
1390 | c->unchecked_size -= totlen; c->used_size += totlen; | ||
1391 | jeb->unchecked_size -= totlen; jeb->used_size += totlen; | ||
1392 | raw->flash_offset = ref_offset(raw) | ||
1393 | | ((xd->node == (void *)raw) ? REF_PRISTINE : REF_NORMAL); | ||
1233 | } | 1394 | } |
1395 | if (is_xattr_datum_dead(xd)) | ||
1396 | list_add(&xd->xindex, &c->xattr_dead_list); | ||
1397 | spin_unlock(&c->erase_completion_lock); | ||
1234 | } | 1398 | } |
1235 | up_write(&c->xattr_sem); | 1399 | up_write(&c->xattr_sem); |
1236 | |||
1237 | return list_empty(&c->xattr_unchecked) ? 1 : 0; | 1400 | return list_empty(&c->xattr_unchecked) ? 1 : 0; |
1238 | } | 1401 | } |
1402 | |||
1403 | void jffs2_release_xattr_datum(struct jffs2_sb_info *c, struct jffs2_xattr_datum *xd) | ||
1404 | { | ||
1405 | /* must be called under spin_lock(&c->erase_completion_lock) */ | ||
1406 | if (xd->node != (void *)xd) | ||
1407 | return; | ||
1408 | |||
1409 | list_del(&xd->xindex); | ||
1410 | jffs2_free_xattr_datum(xd); | ||
1411 | } | ||
1412 | |||
1413 | void jffs2_release_xattr_ref(struct jffs2_sb_info *c, struct jffs2_xattr_ref *ref) | ||
1414 | { | ||
1415 | /* must be called under spin_lock(&c->erase_completion_lock) */ | ||
1416 | struct jffs2_xattr_ref *tmp, **ptmp; | ||
1417 | |||
1418 | if (ref->node != (void *)ref) | ||
1419 | return; | ||
1420 | |||
1421 | for (tmp=c->xref_dead_list, ptmp=&c->xref_dead_list; tmp; ptmp=&tmp->next, tmp=tmp->next) { | ||
1422 | if (ref == tmp) { | ||
1423 | *ptmp = tmp->next; | ||
1424 | jffs2_free_xattr_ref(ref); | ||
1425 | break; | ||
1426 | } | ||
1427 | } | ||
1428 | } | ||
diff --git a/fs/jffs2/xattr.h b/fs/jffs2/xattr.h index 2c199856c582..06ab7b880212 100644 --- a/fs/jffs2/xattr.h +++ b/fs/jffs2/xattr.h | |||
@@ -16,6 +16,7 @@ | |||
16 | 16 | ||
17 | #define JFFS2_XFLAGS_HOT (0x01) /* This datum is HOT */ | 17 | #define JFFS2_XFLAGS_HOT (0x01) /* This datum is HOT */ |
18 | #define JFFS2_XFLAGS_BIND (0x02) /* This datum is not reclaimed */ | 18 | #define JFFS2_XFLAGS_BIND (0x02) /* This datum is not reclaimed */ |
19 | #define JFFS2_XFLAGS_INVALID (0x80) /* This datum contains crc error */ | ||
19 | 20 | ||
20 | struct jffs2_xattr_datum | 21 | struct jffs2_xattr_datum |
21 | { | 22 | { |
@@ -23,7 +24,7 @@ struct jffs2_xattr_datum | |||
23 | struct jffs2_raw_node_ref *node; | 24 | struct jffs2_raw_node_ref *node; |
24 | uint8_t class; | 25 | uint8_t class; |
25 | uint8_t flags; | 26 | uint8_t flags; |
26 | uint16_t xprefix; /* see JFFS2_XATTR_PREFIX_* */ | 27 | uint16_t xprefix; /* see JFFS2_XATTR_PREFIX_* */ |
27 | 28 | ||
28 | struct list_head xindex; /* chained from c->xattrindex[n] */ | 29 | struct list_head xindex; /* chained from c->xattrindex[n] */ |
29 | uint32_t refcnt; /* # of xattr_ref refers this */ | 30 | uint32_t refcnt; /* # of xattr_ref refers this */ |
@@ -47,6 +48,7 @@ struct jffs2_xattr_ref | |||
47 | uint8_t flags; /* Currently unused */ | 48 | uint8_t flags; /* Currently unused */ |
48 | u16 unused; | 49 | u16 unused; |
49 | 50 | ||
51 | uint32_t xseqno; | ||
50 | union { | 52 | union { |
51 | struct jffs2_inode_cache *ic; /* reference to jffs2_inode_cache */ | 53 | struct jffs2_inode_cache *ic; /* reference to jffs2_inode_cache */ |
52 | uint32_t ino; /* only used in scanning/building */ | 54 | uint32_t ino; /* only used in scanning/building */ |
@@ -58,6 +60,34 @@ struct jffs2_xattr_ref | |||
58 | struct jffs2_xattr_ref *next; /* chained from ic->xref_list */ | 60 | struct jffs2_xattr_ref *next; /* chained from ic->xref_list */ |
59 | }; | 61 | }; |
60 | 62 | ||
63 | #define XDATUM_DELETE_MARKER (0xffffffff) | ||
64 | #define XREF_DELETE_MARKER (0x00000001) | ||
65 | static inline int is_xattr_datum_dead(struct jffs2_xattr_datum *xd) | ||
66 | { | ||
67 | return (xd->version == XDATUM_DELETE_MARKER); | ||
68 | } | ||
69 | |||
70 | static inline void set_xattr_datum_dead(struct jffs2_xattr_datum *xd) | ||
71 | { | ||
72 | xd->version = XDATUM_DELETE_MARKER; | ||
73 | } | ||
74 | |||
75 | static inline int is_xattr_ref_dead(struct jffs2_xattr_ref *ref) | ||
76 | { | ||
77 | return ((ref->xseqno & XREF_DELETE_MARKER) != 0); | ||
78 | } | ||
79 | |||
80 | static inline void set_xattr_ref_dead(struct jffs2_xattr_ref *ref) | ||
81 | { | ||
82 | ref->xseqno |= XREF_DELETE_MARKER; | ||
83 | } | ||
84 | |||
85 | static inline void clr_xattr_ref_dead(struct jffs2_xattr_ref *ref) | ||
86 | { | ||
87 | ref->xseqno &= ~XREF_DELETE_MARKER; | ||
88 | } | ||
89 | |||
90 | |||
61 | #ifdef CONFIG_JFFS2_FS_XATTR | 91 | #ifdef CONFIG_JFFS2_FS_XATTR |
62 | 92 | ||
63 | extern void jffs2_init_xattr_subsystem(struct jffs2_sb_info *c); | 93 | extern void jffs2_init_xattr_subsystem(struct jffs2_sb_info *c); |
@@ -70,9 +100,13 @@ extern struct jffs2_xattr_datum *jffs2_setup_xattr_datum(struct jffs2_sb_info *c | |||
70 | extern void jffs2_xattr_delete_inode(struct jffs2_sb_info *c, struct jffs2_inode_cache *ic); | 100 | extern void jffs2_xattr_delete_inode(struct jffs2_sb_info *c, struct jffs2_inode_cache *ic); |
71 | extern void jffs2_xattr_free_inode(struct jffs2_sb_info *c, struct jffs2_inode_cache *ic); | 101 | extern void jffs2_xattr_free_inode(struct jffs2_sb_info *c, struct jffs2_inode_cache *ic); |
72 | 102 | ||
73 | extern int jffs2_garbage_collect_xattr_datum(struct jffs2_sb_info *c, struct jffs2_xattr_datum *xd); | 103 | extern int jffs2_garbage_collect_xattr_datum(struct jffs2_sb_info *c, struct jffs2_xattr_datum *xd, |
74 | extern int jffs2_garbage_collect_xattr_ref(struct jffs2_sb_info *c, struct jffs2_xattr_ref *ref); | 104 | struct jffs2_raw_node_ref *raw); |
105 | extern int jffs2_garbage_collect_xattr_ref(struct jffs2_sb_info *c, struct jffs2_xattr_ref *ref, | ||
106 | struct jffs2_raw_node_ref *raw); | ||
75 | extern int jffs2_verify_xattr(struct jffs2_sb_info *c); | 107 | extern int jffs2_verify_xattr(struct jffs2_sb_info *c); |
108 | extern void jffs2_release_xattr_datum(struct jffs2_sb_info *c, struct jffs2_xattr_datum *xd); | ||
109 | extern void jffs2_release_xattr_ref(struct jffs2_sb_info *c, struct jffs2_xattr_ref *ref); | ||
76 | 110 | ||
77 | extern int do_jffs2_getxattr(struct inode *inode, int xprefix, const char *xname, | 111 | extern int do_jffs2_getxattr(struct inode *inode, int xprefix, const char *xname, |
78 | char *buffer, size_t size); | 112 | char *buffer, size_t size); |
diff --git a/include/linux/jffs2.h b/include/linux/jffs2.h index c6f70660b371..c9c760700bc3 100644 --- a/include/linux/jffs2.h +++ b/include/linux/jffs2.h | |||
@@ -186,6 +186,7 @@ struct jffs2_raw_xref | |||
186 | jint32_t hdr_crc; | 186 | jint32_t hdr_crc; |
187 | jint32_t ino; /* inode number */ | 187 | jint32_t ino; /* inode number */ |
188 | jint32_t xid; /* XATTR identifier number */ | 188 | jint32_t xid; /* XATTR identifier number */ |
189 | jint32_t xseqno; /* xref sequencial number */ | ||
189 | jint32_t node_crc; | 190 | jint32_t node_crc; |
190 | } __attribute__((packed)); | 191 | } __attribute__((packed)); |
191 | 192 | ||