On Fri, 01 Dec 2006 09:48:36 -0500 Jeff Layton wrote:
> This patch is a proof of concept. It works, but still needs a bit of
> polish before it's ready for submission. First, the problems:
>
>
> This patch is a first step at correcting these problems. This adds 2 new
> functions, an idr_register and idr_unregister. Filesystems can call
> idr_register at inode creation time, and then at deletion time, we'll
> automatically unregister them.
s/idr_/iunique_/
> This patch also adds a new s_generation counter to the superblock.
> Because i_ino's can be reused so quickly, we don't want NFS getting
> confused when it happens. When iunique_register is called, we'll assign
> the s_generation value to the i_generation, and then increment it to
> help ensure that we get different filehandles.
>
> There are some things that need to be cleaned up, of course:
>
> - error handling for the idr calls
>
> - recheck all the possible places where the inode should be unhashed
>
> - don't attempt to remove inodes with values <100
Please explain that one. (May be obvious to some, but not to me.)
> - convert other filesystems
>
> - remove the static counter from new_inode and (maybe) eliminate iunique
>
> Comments and suggestions appreciated.
Better to post patches inline (for review) rather than as attachments.
Some (mostly style) comments on the patch:
+ rv = idr_pre_get(&inode->i_sb->s_inode_ids, GFP_KERNEL);
+ if (! rv)
+ return -ENOMEM;
if (!rv)
+ rv = idr_get_new_above(&inode->i_sb->s_inode_ids, inode,
+ max_reserved+1, (int *) &inode->i_ino);
max_reserved + 1,
+}
+
+EXPORT_SYMBOL(iunique_register);
No need for the extra blank line after the function closing
brace. Just put the EXPORT_SYMBOL immediately on the next line.
(in multiple places)
@@ -1681,6 +1688,8 @@ extern void inode_init_once(struct inode
extern void iput(struct inode *);
extern struct inode * igrab(struct inode *);
extern ino_t iunique(struct super_block *, ino_t);
+extern int iunique_register(struct inode *, int);
+extern void iunique_unregister(struct inode *);
extern int inode_needs_sync(struct inode *inode);
extern void generic_delete_inode(struct inode *inode);
extern void generic_drop_inode(struct inode *inode);
Some of these have a parameter name, some don't.
Having a (meaningful) parameter name is strongly preferred.
---
~Randy
-
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/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Stuff]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
[Linux Resources]
