Hi,
I know this is not a patch but a change in the old yaffs project for 
the peaple
that want to use yaffs on new kernel. The test on inode->i_data.nrpages 
is only
done for decrease function call.

Regards
Michael

Quoting Todd Poynor <tpoynor@mvista.com>:

> 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
>



----------------------------------------------------------------
This message was sent using IMP, the Internet Messaging Program.