diff options
author | Arjan van de Ven <arjan@infradead.org> | 2006-01-14 16:21:31 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-01-14 21:27:15 -0500 |
commit | f16623569e2ec055b489462315f5296f153d119b (patch) | |
tree | daef533c4ba906c0618795c0f4b8584db33268ef | |
parent | 652050aec936fdd70ed9cbce1cd1ef30a7c9d117 (diff) |
[PATCH] Mark some key VFS functions as __always_inline
Mark a few VFS functions as mandatory inline (based on Al Viro's request);
these must be inline due to stack usage issues during a recursive loop that
happens during the recursive symlink resolution (symlink to a symlink to a
symlink ..)
This patch at this point does not change behavior and is for documentation
purposes only (but this changes later in the series)
Signed-off-by: Arjan van de Ven <arjan@infradead.org>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
-rw-r--r-- | fs/namei.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/fs/namei.c b/fs/namei.c index 7bde381fa490..33fb5bd34a81 100644 --- a/fs/namei.c +++ b/fs/namei.c | |||
@@ -486,7 +486,7 @@ static struct dentry * real_lookup(struct dentry * parent, struct qstr * name, s | |||
486 | static int __emul_lookup_dentry(const char *, struct nameidata *); | 486 | static int __emul_lookup_dentry(const char *, struct nameidata *); |
487 | 487 | ||
488 | /* SMP-safe */ | 488 | /* SMP-safe */ |
489 | static inline int | 489 | static __always_inline int |
490 | walk_init_root(const char *name, struct nameidata *nd) | 490 | walk_init_root(const char *name, struct nameidata *nd) |
491 | { | 491 | { |
492 | read_lock(¤t->fs->lock); | 492 | read_lock(¤t->fs->lock); |
@@ -504,7 +504,7 @@ walk_init_root(const char *name, struct nameidata *nd) | |||
504 | return 1; | 504 | return 1; |
505 | } | 505 | } |
506 | 506 | ||
507 | static inline int __vfs_follow_link(struct nameidata *nd, const char *link) | 507 | static __always_inline int __vfs_follow_link(struct nameidata *nd, const char *link) |
508 | { | 508 | { |
509 | int res = 0; | 509 | int res = 0; |
510 | char *name; | 510 | char *name; |
@@ -544,7 +544,7 @@ struct path { | |||
544 | struct dentry *dentry; | 544 | struct dentry *dentry; |
545 | }; | 545 | }; |
546 | 546 | ||
547 | static inline int __do_follow_link(struct path *path, struct nameidata *nd) | 547 | static __always_inline int __do_follow_link(struct path *path, struct nameidata *nd) |
548 | { | 548 | { |
549 | int error; | 549 | int error; |
550 | void *cookie; | 550 | void *cookie; |
@@ -690,7 +690,7 @@ int follow_down(struct vfsmount **mnt, struct dentry **dentry) | |||
690 | return 0; | 690 | return 0; |
691 | } | 691 | } |
692 | 692 | ||
693 | static inline void follow_dotdot(struct nameidata *nd) | 693 | static __always_inline void follow_dotdot(struct nameidata *nd) |
694 | { | 694 | { |
695 | while(1) { | 695 | while(1) { |
696 | struct vfsmount *parent; | 696 | struct vfsmount *parent; |