Commit 961f3c89 authored by Mauro Carvalho Chehab's avatar Mauro Carvalho Chehab Committed by Jonathan Corbet

fs: fix kernel-doc markups

Two markups are at the wrong place. Kernel-doc only
support having the comment just before the identifier.

Also, some identifiers have different names between their
prototypes and the kernel-doc markup.
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab+huawei@kernel.org>
Link: https://lore.kernel.org/r/96b1e1b388600ab092331f6c4e88ff8e8779ce6c.1610610937.git.mchehab+huawei@kernel.orgSigned-off-by: default avatarJonathan Corbet <corbet@lwn.net>
parent 3de990b0
...@@ -456,23 +456,6 @@ static void d_lru_shrink_move(struct list_lru_one *lru, struct dentry *dentry, ...@@ -456,23 +456,6 @@ static void d_lru_shrink_move(struct list_lru_one *lru, struct dentry *dentry,
list_lru_isolate_move(lru, &dentry->d_lru, list); list_lru_isolate_move(lru, &dentry->d_lru, list);
} }
/**
* d_drop - drop a dentry
* @dentry: dentry to drop
*
* d_drop() unhashes the entry from the parent dentry hashes, so that it won't
* be found through a VFS lookup any more. Note that this is different from
* deleting the dentry - d_delete will try to mark the dentry negative if
* possible, giving a successful _negative_ lookup, while d_drop will
* just make the cache lookup fail.
*
* d_drop() is used mainly for stuff that wants to invalidate a dentry for some
* reason (NFS timeouts or autofs deletes).
*
* __d_drop requires dentry->d_lock
* ___d_drop doesn't mark dentry as "unhashed"
* (dentry->d_hash.pprev will be LIST_POISON2, not NULL).
*/
static void ___d_drop(struct dentry *dentry) static void ___d_drop(struct dentry *dentry)
{ {
struct hlist_bl_head *b; struct hlist_bl_head *b;
...@@ -501,6 +484,24 @@ void __d_drop(struct dentry *dentry) ...@@ -501,6 +484,24 @@ void __d_drop(struct dentry *dentry)
} }
EXPORT_SYMBOL(__d_drop); EXPORT_SYMBOL(__d_drop);
/**
* d_drop - drop a dentry
* @dentry: dentry to drop
*
* d_drop() unhashes the entry from the parent dentry hashes, so that it won't
* be found through a VFS lookup any more. Note that this is different from
* deleting the dentry - d_delete will try to mark the dentry negative if
* possible, giving a successful _negative_ lookup, while d_drop will
* just make the cache lookup fail.
*
* d_drop() is used mainly for stuff that wants to invalidate a dentry for some
* reason (NFS timeouts or autofs deletes).
*
* __d_drop requires dentry->d_lock
*
* ___d_drop doesn't mark dentry as "unhashed"
* (dentry->d_hash.pprev will be LIST_POISON2, not NULL).
*/
void d_drop(struct dentry *dentry) void d_drop(struct dentry *dentry)
{ {
spin_lock(&dentry->d_lock); spin_lock(&dentry->d_lock);
...@@ -996,20 +997,6 @@ struct dentry *d_find_any_alias(struct inode *inode) ...@@ -996,20 +997,6 @@ struct dentry *d_find_any_alias(struct inode *inode)
} }
EXPORT_SYMBOL(d_find_any_alias); EXPORT_SYMBOL(d_find_any_alias);
/**
* d_find_alias - grab a hashed alias of inode
* @inode: inode in question
*
* If inode has a hashed alias, or is a directory and has any alias,
* acquire the reference to alias and return it. Otherwise return NULL.
* Notice that if inode is a directory there can be only one alias and
* it can be unhashed only if it has no children, or if it is the root
* of a filesystem, or if the directory was renamed and d_revalidate
* was the first vfs operation to notice.
*
* If the inode has an IS_ROOT, DCACHE_DISCONNECTED alias, then prefer
* any other hashed alias over that one.
*/
static struct dentry *__d_find_alias(struct inode *inode) static struct dentry *__d_find_alias(struct inode *inode)
{ {
struct dentry *alias; struct dentry *alias;
...@@ -1029,6 +1016,20 @@ static struct dentry *__d_find_alias(struct inode *inode) ...@@ -1029,6 +1016,20 @@ static struct dentry *__d_find_alias(struct inode *inode)
return NULL; return NULL;
} }
/**
* d_find_alias - grab a hashed alias of inode
* @inode: inode in question
*
* If inode has a hashed alias, or is a directory and has any alias,
* acquire the reference to alias and return it. Otherwise return NULL.
* Notice that if inode is a directory there can be only one alias and
* it can be unhashed only if it has no children, or if it is the root
* of a filesystem, or if the directory was renamed and d_revalidate
* was the first vfs operation to notice.
*
* If the inode has an IS_ROOT, DCACHE_DISCONNECTED alias, then prefer
* any other hashed alias over that one.
*/
struct dentry *d_find_alias(struct inode *inode) struct dentry *d_find_alias(struct inode *inode)
{ {
struct dentry *de = NULL; struct dentry *de = NULL;
......
...@@ -1493,7 +1493,7 @@ struct inode *find_inode_rcu(struct super_block *sb, unsigned long hashval, ...@@ -1493,7 +1493,7 @@ struct inode *find_inode_rcu(struct super_block *sb, unsigned long hashval,
EXPORT_SYMBOL(find_inode_rcu); EXPORT_SYMBOL(find_inode_rcu);
/** /**
* find_inode_by_rcu - Find an inode in the inode cache * find_inode_by_ino_rcu - Find an inode in the inode cache
* @sb: Super block of file system to search * @sb: Super block of file system to search
* @ino: The inode number to match * @ino: The inode number to match
* *
...@@ -1777,7 +1777,7 @@ static int update_time(struct inode *inode, struct timespec64 *time, int flags) ...@@ -1777,7 +1777,7 @@ static int update_time(struct inode *inode, struct timespec64 *time, int flags)
} }
/** /**
* touch_atime - update the access time * atime_needs_update - update the access time
* @path: the &struct path to update * @path: the &struct path to update
* @inode: inode to update * @inode: inode to update
* *
......
...@@ -669,7 +669,8 @@ void seq_puts(struct seq_file *m, const char *s) ...@@ -669,7 +669,8 @@ void seq_puts(struct seq_file *m, const char *s)
EXPORT_SYMBOL(seq_puts); EXPORT_SYMBOL(seq_puts);
/** /**
* A helper routine for putting decimal numbers without rich format of printf(). * seq_put_decimal_ull_width - A helper routine for putting decimal numbers
* without rich format of printf().
* only 'unsigned long long' is supported. * only 'unsigned long long' is supported.
* @m: seq_file identifying the buffer to which data should be written * @m: seq_file identifying the buffer to which data should be written
* @delimiter: a string which is printed before the number * @delimiter: a string which is printed before the number
...@@ -1044,7 +1045,7 @@ struct hlist_node *seq_hlist_next_rcu(void *v, ...@@ -1044,7 +1045,7 @@ struct hlist_node *seq_hlist_next_rcu(void *v,
EXPORT_SYMBOL(seq_hlist_next_rcu); EXPORT_SYMBOL(seq_hlist_next_rcu);
/** /**
* seq_hlist_start_precpu - start an iteration of a percpu hlist array * seq_hlist_start_percpu - start an iteration of a percpu hlist array
* @head: pointer to percpu array of struct hlist_heads * @head: pointer to percpu array of struct hlist_heads
* @cpu: pointer to cpu "cursor" * @cpu: pointer to cpu "cursor"
* @pos: start position of sequence * @pos: start position of sequence
......
...@@ -1718,12 +1718,6 @@ int freeze_super(struct super_block *sb) ...@@ -1718,12 +1718,6 @@ int freeze_super(struct super_block *sb)
} }
EXPORT_SYMBOL(freeze_super); EXPORT_SYMBOL(freeze_super);
/**
* thaw_super -- unlock filesystem
* @sb: the super to thaw
*
* Unlocks the filesystem and marks it writeable again after freeze_super().
*/
static int thaw_super_locked(struct super_block *sb) static int thaw_super_locked(struct super_block *sb)
{ {
int error; int error;
...@@ -1759,6 +1753,12 @@ static int thaw_super_locked(struct super_block *sb) ...@@ -1759,6 +1753,12 @@ static int thaw_super_locked(struct super_block *sb)
return 0; return 0;
} }
/**
* thaw_super -- unlock filesystem
* @sb: the super to thaw
*
* Unlocks the filesystem and marks it writeable again after freeze_super().
*/
int thaw_super(struct super_block *sb) int thaw_super(struct super_block *sb)
{ {
down_write(&sb->s_umount); down_write(&sb->s_umount);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment