Re: [patch-2.4.0-test5-pre3] jffs_delete_inode() cleanup

From: David Woodhouse (dwmw2@passion.cygnus.co.uk)
Date: Thu Jul 20 2000 - 09:14:11 EST


On Thu, 20 Jul 2000, Tigran Aivazian wrote:

> Hi David,
>
> I don't think jffs_delete_inode() needs a big kernel lock.
>
> Regards,
> Tigran
>
> --- linux/fs/jffs/inode-v23.c Fri Jul 14 20:52:15 2000
> +++ work/fs/jffs/inode-v23.c Thu Jul 20 13:05:14 2000
> @@ -1527,14 +1527,8 @@
> {
>
> D3(printk("jffs_delete_inode(): inode->i_ino == %lu\n", inode->i_ino));
> -
> - lock_kernel();
> -
> inode->i_size = 0;
> -
> clear_inode(inode);
> -
> - unlock_kernel();
> }
>
> void
>
>

Thanks,

Is that in the CVS tree or was it added (by Al?) into 2.4?

Please could you Cc all JFFS changes to the jffs-dev list.

I believe you have CVS access so you can commit it yourself, if and when
the link to my machine comes back up.

--
dwmw2

- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.rutgers.edu Please read the FAQ at http://www.tux.org/lkml/



This archive was generated by hypermail 2b29 : Sun Jul 23 2000 - 21:00:13 EST