Changman Lee <[email protected]> writes: >> > +void update_inode(struct inode *inode, struct page *node_page) >> > +{ >> > + struct f2fs_node *rn; >> > + struct f2fs_inode *ri; >> > + >> > + wait_on_page_writeback(node_page); >> > + >> > + rn = page_address(node_page); >> > + ri = &(rn->i); >> > + >> > + ri->i_mode = cpu_to_le16(inode->i_mode); >> > + ri->i_uid = cpu_to_le32(inode->i_uid); >> > + ri->i_gid = cpu_to_le32(inode->i_gid); >> And make this: >> i_uid_write(inode, le32_to_cpu(ri->i_uid)); >> i_gid_write(inode, le32_to_cpu(ri->i_gid)); >> > > You mean this. Right? :) > ri->i_uid = cpu_to_le32(i_uid_read(inode)); > ri->i_gid = cpu_to_le32(i_gid_read(inode));
Yeah. ;) I did have that one backwards. Eric -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
