On Tue, 2009-08-04 at 12:32 +0200, ext OGAWA Hirofumi wrote: > Jörg Schummer <ext-jorg.2.schummer@xxxxxxxxx> writes: > > > On Mon, 2009-07-27 at 13:47 +0200, ext OGAWA Hirofumi wrote: > >> Jörg Schummer <ext-jorg.2.schummer@xxxxxxxxx> writes: > > > >> BTW, the patch has several bugs. fat_get_label_entry() doesn't check > >> IS_FREE(), is it right? fat_create_label_entry() doesn't initialize all > >> timestamp in the case of msdos. spin_lock() usage is wrong. and more... > > > > After reviewing the patch again, I did notice some additional problems > > (I suppose that was the "and more..."). If and when I have time again, I > > might take a closer look at it again. > > > > Thanks for your help until now! Hello, on the off-chance that somebody would like to use this mechanism, I did a 3rd take of the patch. I tried to fix the problems of the previous patch and additionally added the functionality to correct the root dir timestamp's mtime and atime somewhat based on contained directory entries' ctime. Any comments are welcome, but especially I'd like to ask if it is safe / good / doable to call remove_inode_hash(inode); fat_attach(inode, i_pos); insert_inode_hash(inode); inside fat_write_inode . Thanks for any feedback! Jörg -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
