Re: [PATCH 0/6] nilfs2: remove own inode hash table

2010-08-24 Thread Nick Piggin
On Sun, Aug 22, 2010 at 07:05:00PM +0900, Ryusuke Konishi wrote: This is a patchset to remove own inode hash table from nilfs. The current version of nilfs uses inode not only to manage regular files, directories, symlinks but also for some types of metadata and for caching file blocks

Re: [PATCH 0/6] nilfs2: remove own inode hash table

2010-08-24 Thread Christoph Hellwig
On Tue, Aug 24, 2010 at 05:48:44PM +1000, Nick Piggin wrote: The reason I broke nilfs2 is because it duplicates a lot of the generic inode initialisation code. This really should go in core code because nilfs2 does not own the generic inode fields. It just needs some helper function to do

Re: [PATCH 0/6] nilfs2: remove own inode hash table

2010-08-24 Thread Al Viro
On Wed, Aug 25, 2010 at 01:20:12AM +0900, Ryusuke Konishi wrote: On Tue, 24 Aug 2010 11:32:14 -0400, Christoph Hellwig wrote: On Tue, Aug 24, 2010 at 09:58:23PM +0900, Ryusuke Konishi wrote: Actually, I recently wrote a patchset to unify all super block instances on one device into one

Re: [PATCH 0/6] nilfs2: remove own inode hash table

2010-08-24 Thread Ryusuke Konishi
On Tue, 24 Aug 2010 17:42:16 +0100, Al Viro wrote: On Wed, Aug 25, 2010 at 01:20:12AM +0900, Ryusuke Konishi wrote: So, my implementation issue here is how to free dentries and inodes of snapshots like shrink_dcache_for_umount(sb) does for a single namespace. Use d_obtain_alias() to

[PATCH 0/6] nilfs2: remove own inode hash table

2010-08-22 Thread Ryusuke Konishi
This is a patchset to remove own inode hash table from nilfs. The current version of nilfs uses inode not only to manage regular files, directories, symlinks but also for some types of metadata and for caching file blocks relocated by GC. The last type of inodes is called gc-inodes, and nilfs