Re: [Yaffs] delete inode on 2.6.14

Michael Trimarchi wrote:
> I think the yaffs_delete_inode on yaffs must be correct on new kernel.
...
> The new one:

It's helpful to use the patch format for discussing code changes.
...

>        if (inode->i_data.nrpages)
>                truncate_inode_pages(&inode->i_data, 0);

...
> Is it correct?

yaffs2 CVS module has similar code, should be added to the deprecated
yaffs module as well if maintainers agree:

#if (LINUX_VERSION_CODE > KERNEL_VERSION(2,6,13))
         truncate_inode_pages (&inode->i_data, 0);
#endif

The check for inode->i_data.nrpages is not necessary, the callee already
does this, and it may not be safe if the implementation changes in the
future. Thanks,

--
Todd