[Tux3] logmap inode fix

[OGAWA Hirofumi]

Daniel Phillips <phillips at phunq.net> writes:

> On Tuesday 06 January 2009 23:37, OGAWA Hirofumi wrote:
>> Daniel Phillips <phillips at phunq.net> writes:
>> 
>> > The iattr passed here seem to be uninitialized:
>> >
>> > 	sbi->logmap = tux_new_inode(sbi->rootdir, &iattr, 0);
>> >
>> > Why not pass &(struct iattr){ }?
>> 
>> I think "static iattr" is more prefer, it would be in .bss, and it would
>> be zeroed by initialization of program.
>
> Ah, I overlooked the static.  Thanks, it looks good.

I forget to add insert_inode_hash() after tux_new_inode(). Without hash,
it may work. Well, __mark_inode_dirty() will not dirty unhashed inode.
However, logmap may not need it.

I'll think about it.
-- 
OGAWA Hirofumi <hirofumi at mail.parknet.co.jp>

_______________________________________________
Tux3 mailing list
Tux3 at tux3.org
http://mailman.tux3.org/cgi-bin/mailman/listinfo/tux3