2013-11-24 22:54:58 +08:00
|
|
|
/*
|
|
|
|
* fs/kernfs/dir.c - kernfs directory implementation
|
|
|
|
*
|
|
|
|
* Copyright (c) 2001-3 Patrick Mochel
|
|
|
|
* Copyright (c) 2007 SUSE Linux Products GmbH
|
|
|
|
* Copyright (c) 2007, 2013 Tejun Heo <tj@kernel.org>
|
|
|
|
*
|
|
|
|
* This file is released under the GPLv2.
|
|
|
|
*/
|
2013-11-29 03:54:33 +08:00
|
|
|
|
2014-02-04 03:02:55 +08:00
|
|
|
#include <linux/sched.h>
|
2013-11-29 03:54:33 +08:00
|
|
|
#include <linux/fs.h>
|
|
|
|
#include <linux/namei.h>
|
|
|
|
#include <linux/idr.h>
|
|
|
|
#include <linux/slab.h>
|
|
|
|
#include <linux/security.h>
|
|
|
|
#include <linux/hash.h>
|
|
|
|
|
|
|
|
#include "kernfs-internal.h"
|
|
|
|
|
2013-12-12 03:11:57 +08:00
|
|
|
DEFINE_MUTEX(kernfs_mutex);
|
2014-02-08 02:32:07 +08:00
|
|
|
static DEFINE_SPINLOCK(kernfs_rename_lock); /* kn->parent and ->name */
|
|
|
|
static char kernfs_pr_cont_buf[PATH_MAX]; /* protected by rename_lock */
|
2013-11-29 03:54:33 +08:00
|
|
|
|
2013-12-12 03:11:54 +08:00
|
|
|
#define rb_to_kn(X) rb_entry((X), struct kernfs_node, rb)
|
2013-11-29 03:54:33 +08:00
|
|
|
|
2014-02-04 03:03:00 +08:00
|
|
|
static bool kernfs_active(struct kernfs_node *kn)
|
|
|
|
{
|
|
|
|
lockdep_assert_held(&kernfs_mutex);
|
|
|
|
return atomic_read(&kn->active) >= 0;
|
|
|
|
}
|
|
|
|
|
2014-02-04 03:02:59 +08:00
|
|
|
static bool kernfs_lockdep(struct kernfs_node *kn)
|
|
|
|
{
|
|
|
|
#ifdef CONFIG_DEBUG_LOCK_ALLOC
|
|
|
|
return kn->flags & KERNFS_LOCKDEP;
|
|
|
|
#else
|
|
|
|
return false;
|
|
|
|
#endif
|
|
|
|
}
|
|
|
|
|
2014-02-08 02:32:07 +08:00
|
|
|
static int kernfs_name_locked(struct kernfs_node *kn, char *buf, size_t buflen)
|
|
|
|
{
|
|
|
|
return strlcpy(buf, kn->parent ? kn->name : "/", buflen);
|
|
|
|
}
|
|
|
|
|
|
|
|
static char * __must_check kernfs_path_locked(struct kernfs_node *kn, char *buf,
|
|
|
|
size_t buflen)
|
|
|
|
{
|
|
|
|
char *p = buf + buflen;
|
|
|
|
int len;
|
|
|
|
|
|
|
|
*--p = '\0';
|
|
|
|
|
|
|
|
do {
|
|
|
|
len = strlen(kn->name);
|
|
|
|
if (p - buf < len + 1) {
|
|
|
|
buf[0] = '\0';
|
|
|
|
p = NULL;
|
|
|
|
break;
|
|
|
|
}
|
|
|
|
p -= len;
|
|
|
|
memcpy(p, kn->name, len);
|
|
|
|
*--p = '/';
|
|
|
|
kn = kn->parent;
|
|
|
|
} while (kn && kn->parent);
|
|
|
|
|
|
|
|
return p;
|
|
|
|
}
|
|
|
|
|
|
|
|
/**
|
|
|
|
* kernfs_name - obtain the name of a given node
|
|
|
|
* @kn: kernfs_node of interest
|
|
|
|
* @buf: buffer to copy @kn's name into
|
|
|
|
* @buflen: size of @buf
|
|
|
|
*
|
|
|
|
* Copies the name of @kn into @buf of @buflen bytes. The behavior is
|
|
|
|
* similar to strlcpy(). It returns the length of @kn's name and if @buf
|
|
|
|
* isn't long enough, it's filled upto @buflen-1 and nul terminated.
|
|
|
|
*
|
|
|
|
* This function can be called from any context.
|
|
|
|
*/
|
|
|
|
int kernfs_name(struct kernfs_node *kn, char *buf, size_t buflen)
|
|
|
|
{
|
|
|
|
unsigned long flags;
|
|
|
|
int ret;
|
|
|
|
|
|
|
|
spin_lock_irqsave(&kernfs_rename_lock, flags);
|
|
|
|
ret = kernfs_name_locked(kn, buf, buflen);
|
|
|
|
spin_unlock_irqrestore(&kernfs_rename_lock, flags);
|
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
|
2015-08-19 05:54:55 +08:00
|
|
|
/**
|
|
|
|
* kernfs_path_len - determine the length of the full path of a given node
|
|
|
|
* @kn: kernfs_node of interest
|
|
|
|
*
|
|
|
|
* The returned length doesn't include the space for the terminating '\0'.
|
|
|
|
*/
|
|
|
|
size_t kernfs_path_len(struct kernfs_node *kn)
|
|
|
|
{
|
|
|
|
size_t len = 0;
|
|
|
|
unsigned long flags;
|
|
|
|
|
|
|
|
spin_lock_irqsave(&kernfs_rename_lock, flags);
|
|
|
|
|
|
|
|
do {
|
|
|
|
len += strlen(kn->name) + 1;
|
|
|
|
kn = kn->parent;
|
|
|
|
} while (kn && kn->parent);
|
|
|
|
|
|
|
|
spin_unlock_irqrestore(&kernfs_rename_lock, flags);
|
|
|
|
|
|
|
|
return len;
|
|
|
|
}
|
|
|
|
|
2014-02-08 02:32:07 +08:00
|
|
|
/**
|
|
|
|
* kernfs_path - build full path of a given node
|
|
|
|
* @kn: kernfs_node of interest
|
|
|
|
* @buf: buffer to copy @kn's name into
|
|
|
|
* @buflen: size of @buf
|
|
|
|
*
|
|
|
|
* Builds and returns the full path of @kn in @buf of @buflen bytes. The
|
|
|
|
* path is built from the end of @buf so the returned pointer usually
|
|
|
|
* doesn't match @buf. If @buf isn't long enough, @buf is nul terminated
|
|
|
|
* and %NULL is returned.
|
|
|
|
*/
|
|
|
|
char *kernfs_path(struct kernfs_node *kn, char *buf, size_t buflen)
|
|
|
|
{
|
|
|
|
unsigned long flags;
|
|
|
|
char *p;
|
|
|
|
|
|
|
|
spin_lock_irqsave(&kernfs_rename_lock, flags);
|
|
|
|
p = kernfs_path_locked(kn, buf, buflen);
|
|
|
|
spin_unlock_irqrestore(&kernfs_rename_lock, flags);
|
|
|
|
return p;
|
|
|
|
}
|
2014-02-12 22:29:50 +08:00
|
|
|
EXPORT_SYMBOL_GPL(kernfs_path);
|
2014-02-08 02:32:07 +08:00
|
|
|
|
|
|
|
/**
|
|
|
|
* pr_cont_kernfs_name - pr_cont name of a kernfs_node
|
|
|
|
* @kn: kernfs_node of interest
|
|
|
|
*
|
|
|
|
* This function can be called from any context.
|
|
|
|
*/
|
|
|
|
void pr_cont_kernfs_name(struct kernfs_node *kn)
|
|
|
|
{
|
|
|
|
unsigned long flags;
|
|
|
|
|
|
|
|
spin_lock_irqsave(&kernfs_rename_lock, flags);
|
|
|
|
|
|
|
|
kernfs_name_locked(kn, kernfs_pr_cont_buf, sizeof(kernfs_pr_cont_buf));
|
|
|
|
pr_cont("%s", kernfs_pr_cont_buf);
|
|
|
|
|
|
|
|
spin_unlock_irqrestore(&kernfs_rename_lock, flags);
|
|
|
|
}
|
|
|
|
|
|
|
|
/**
|
|
|
|
* pr_cont_kernfs_path - pr_cont path of a kernfs_node
|
|
|
|
* @kn: kernfs_node of interest
|
|
|
|
*
|
|
|
|
* This function can be called from any context.
|
|
|
|
*/
|
|
|
|
void pr_cont_kernfs_path(struct kernfs_node *kn)
|
|
|
|
{
|
|
|
|
unsigned long flags;
|
|
|
|
char *p;
|
|
|
|
|
|
|
|
spin_lock_irqsave(&kernfs_rename_lock, flags);
|
|
|
|
|
|
|
|
p = kernfs_path_locked(kn, kernfs_pr_cont_buf,
|
|
|
|
sizeof(kernfs_pr_cont_buf));
|
|
|
|
if (p)
|
|
|
|
pr_cont("%s", p);
|
|
|
|
else
|
|
|
|
pr_cont("<name too long>");
|
|
|
|
|
|
|
|
spin_unlock_irqrestore(&kernfs_rename_lock, flags);
|
|
|
|
}
|
|
|
|
|
|
|
|
/**
|
|
|
|
* kernfs_get_parent - determine the parent node and pin it
|
|
|
|
* @kn: kernfs_node of interest
|
|
|
|
*
|
|
|
|
* Determines @kn's parent, pins and returns it. This function can be
|
|
|
|
* called from any context.
|
|
|
|
*/
|
|
|
|
struct kernfs_node *kernfs_get_parent(struct kernfs_node *kn)
|
|
|
|
{
|
|
|
|
struct kernfs_node *parent;
|
|
|
|
unsigned long flags;
|
|
|
|
|
|
|
|
spin_lock_irqsave(&kernfs_rename_lock, flags);
|
|
|
|
parent = kn->parent;
|
|
|
|
kernfs_get(parent);
|
|
|
|
spin_unlock_irqrestore(&kernfs_rename_lock, flags);
|
|
|
|
|
|
|
|
return parent;
|
|
|
|
}
|
|
|
|
|
2013-11-29 03:54:33 +08:00
|
|
|
/**
|
2013-12-12 03:11:58 +08:00
|
|
|
* kernfs_name_hash
|
2013-11-29 03:54:33 +08:00
|
|
|
* @name: Null terminated string to hash
|
|
|
|
* @ns: Namespace tag to hash
|
|
|
|
*
|
|
|
|
* Returns 31 bit hash of ns + name (so it fits in an off_t )
|
|
|
|
*/
|
2013-12-12 03:11:58 +08:00
|
|
|
static unsigned int kernfs_name_hash(const char *name, const void *ns)
|
2013-11-29 03:54:33 +08:00
|
|
|
{
|
|
|
|
unsigned long hash = init_name_hash();
|
|
|
|
unsigned int len = strlen(name);
|
|
|
|
while (len--)
|
|
|
|
hash = partial_name_hash(*name++, hash);
|
|
|
|
hash = (end_name_hash(hash) ^ hash_ptr((void *)ns, 31));
|
|
|
|
hash &= 0x7fffffffU;
|
|
|
|
/* Reserve hash numbers 0, 1 and INT_MAX for magic directory entries */
|
2014-03-06 00:10:52 +08:00
|
|
|
if (hash < 2)
|
2013-11-29 03:54:33 +08:00
|
|
|
hash += 2;
|
|
|
|
if (hash >= INT_MAX)
|
|
|
|
hash = INT_MAX - 1;
|
|
|
|
return hash;
|
|
|
|
}
|
|
|
|
|
2013-12-12 03:11:58 +08:00
|
|
|
static int kernfs_name_compare(unsigned int hash, const char *name,
|
|
|
|
const void *ns, const struct kernfs_node *kn)
|
2013-11-29 03:54:33 +08:00
|
|
|
{
|
2014-12-06 06:41:33 +08:00
|
|
|
if (hash < kn->hash)
|
|
|
|
return -1;
|
|
|
|
if (hash > kn->hash)
|
|
|
|
return 1;
|
|
|
|
if (ns < kn->ns)
|
|
|
|
return -1;
|
|
|
|
if (ns > kn->ns)
|
|
|
|
return 1;
|
2013-12-12 03:11:54 +08:00
|
|
|
return strcmp(name, kn->name);
|
2013-11-29 03:54:33 +08:00
|
|
|
}
|
|
|
|
|
2013-12-12 03:11:58 +08:00
|
|
|
static int kernfs_sd_compare(const struct kernfs_node *left,
|
|
|
|
const struct kernfs_node *right)
|
2013-11-29 03:54:33 +08:00
|
|
|
{
|
2013-12-12 03:11:58 +08:00
|
|
|
return kernfs_name_compare(left->hash, left->name, left->ns, right);
|
2013-11-29 03:54:33 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
/**
|
2013-12-12 03:11:58 +08:00
|
|
|
* kernfs_link_sibling - link kernfs_node into sibling rbtree
|
2013-12-12 03:11:53 +08:00
|
|
|
* @kn: kernfs_node of interest
|
2013-11-29 03:54:33 +08:00
|
|
|
*
|
2013-12-12 03:11:53 +08:00
|
|
|
* Link @kn into its sibling rbtree which starts from
|
2013-12-12 03:11:54 +08:00
|
|
|
* @kn->parent->dir.children.
|
2013-11-29 03:54:33 +08:00
|
|
|
*
|
|
|
|
* Locking:
|
2013-12-12 03:11:57 +08:00
|
|
|
* mutex_lock(kernfs_mutex)
|
2013-11-29 03:54:33 +08:00
|
|
|
*
|
|
|
|
* RETURNS:
|
|
|
|
* 0 on susccess -EEXIST on failure.
|
|
|
|
*/
|
2013-12-12 03:11:58 +08:00
|
|
|
static int kernfs_link_sibling(struct kernfs_node *kn)
|
2013-11-29 03:54:33 +08:00
|
|
|
{
|
2013-12-12 03:11:54 +08:00
|
|
|
struct rb_node **node = &kn->parent->dir.children.rb_node;
|
2013-11-29 03:54:33 +08:00
|
|
|
struct rb_node *parent = NULL;
|
|
|
|
|
|
|
|
while (*node) {
|
2013-12-12 03:11:53 +08:00
|
|
|
struct kernfs_node *pos;
|
2013-11-29 03:54:33 +08:00
|
|
|
int result;
|
|
|
|
|
2013-12-12 03:11:53 +08:00
|
|
|
pos = rb_to_kn(*node);
|
2013-11-29 03:54:33 +08:00
|
|
|
parent = *node;
|
2013-12-12 03:11:58 +08:00
|
|
|
result = kernfs_sd_compare(kn, pos);
|
2013-11-29 03:54:33 +08:00
|
|
|
if (result < 0)
|
2013-12-12 03:11:54 +08:00
|
|
|
node = &pos->rb.rb_left;
|
2013-11-29 03:54:33 +08:00
|
|
|
else if (result > 0)
|
2013-12-12 03:11:54 +08:00
|
|
|
node = &pos->rb.rb_right;
|
2013-11-29 03:54:33 +08:00
|
|
|
else
|
|
|
|
return -EEXIST;
|
|
|
|
}
|
2014-04-17 17:52:10 +08:00
|
|
|
|
2013-11-29 03:54:33 +08:00
|
|
|
/* add new node and rebalance the tree */
|
2013-12-12 03:11:54 +08:00
|
|
|
rb_link_node(&kn->rb, parent, node);
|
|
|
|
rb_insert_color(&kn->rb, &kn->parent->dir.children);
|
2014-04-17 17:52:10 +08:00
|
|
|
|
|
|
|
/* successfully added, account subdir number */
|
|
|
|
if (kernfs_type(kn) == KERNFS_DIR)
|
|
|
|
kn->parent->dir.subdirs++;
|
|
|
|
|
2013-11-29 03:54:33 +08:00
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
|
|
|
/**
|
2013-12-12 03:11:58 +08:00
|
|
|
* kernfs_unlink_sibling - unlink kernfs_node from sibling rbtree
|
2013-12-12 03:11:53 +08:00
|
|
|
* @kn: kernfs_node of interest
|
2013-11-29 03:54:33 +08:00
|
|
|
*
|
kernfs: restructure removal path to fix possible premature return
The recursive nature of kernfs_remove() means that, even if
kernfs_remove() is not allowed to be called multiple times on the same
node, there may be race conditions between removal of parent and its
descendants. While we can claim that kernfs_remove() shouldn't be
called on one of the descendants while the removal of an ancestor is
in progress, such rule is unnecessarily restrictive and very difficult
to enforce. It's better to simply allow invoking kernfs_remove() as
the caller sees fit as long as the caller ensures that the node is
accessible.
The current behavior in such situations is broken. Whoever enters
removal path first takes the node off the hierarchy and then
deactivates. Following removers either return as soon as it notices
that it's not the first one or can't even find the target node as it
has already been removed from the hierarchy. In both cases, the
following removers may finish prematurely while the nodes which should
be removed and drained are still being processed by the first one.
This patch restructures so that multiple removers, whether through
recursion or direction invocation, always follow the following rules.
* When there are multiple concurrent removers, only one puts the base
ref.
* Regardless of which one puts the base ref, all removers are blocked
until the target node is fully deactivated and removed.
To achieve the above, removal path now first marks all descendants
including self REMOVED and then deactivates and unlinks leftmost
descendant one-by-one. kernfs_deactivate() is called directly from
__kernfs_removal() and drops and regrabs kernfs_mutex for each
descendant to drain active refs. As this means that multiple removers
can enter kernfs_deactivate() for the same node, the function is
updated so that it can handle multiple deactivators of the same node -
only one actually deactivates but all wait till drain completion.
The restructured removal path guarantees that a removed node gets
unlinked only after the node is deactivated and drained. Combined
with proper multiple deactivator handling, this guarantees that any
invocation of kernfs_remove() returns only after the node itself and
all its descendants are deactivated, drained and removed.
v2: Draining separated into a separate loop (used to be in the same
loop as unlink) and done from __kernfs_deactivate(). This is to
allow exposing deactivation as a separate interface later.
Root node removal was broken in v1 patch. Fixed.
v3: Revert most of v2 except for root node removal fix and
simplification of KERNFS_REMOVED setting loop.
v4: Refreshed on top of ("kernfs: make kernfs_deactivate() honor
KERNFS_LOCKDEP flag").
Signed-off-by: Tejun Heo <tj@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-02-04 03:02:56 +08:00
|
|
|
* Try to unlink @kn from its sibling rbtree which starts from
|
|
|
|
* kn->parent->dir.children. Returns %true if @kn was actually
|
|
|
|
* removed, %false if @kn wasn't on the rbtree.
|
2013-11-29 03:54:33 +08:00
|
|
|
*
|
|
|
|
* Locking:
|
2013-12-12 03:11:57 +08:00
|
|
|
* mutex_lock(kernfs_mutex)
|
2013-11-29 03:54:33 +08:00
|
|
|
*/
|
kernfs: restructure removal path to fix possible premature return
The recursive nature of kernfs_remove() means that, even if
kernfs_remove() is not allowed to be called multiple times on the same
node, there may be race conditions between removal of parent and its
descendants. While we can claim that kernfs_remove() shouldn't be
called on one of the descendants while the removal of an ancestor is
in progress, such rule is unnecessarily restrictive and very difficult
to enforce. It's better to simply allow invoking kernfs_remove() as
the caller sees fit as long as the caller ensures that the node is
accessible.
The current behavior in such situations is broken. Whoever enters
removal path first takes the node off the hierarchy and then
deactivates. Following removers either return as soon as it notices
that it's not the first one or can't even find the target node as it
has already been removed from the hierarchy. In both cases, the
following removers may finish prematurely while the nodes which should
be removed and drained are still being processed by the first one.
This patch restructures so that multiple removers, whether through
recursion or direction invocation, always follow the following rules.
* When there are multiple concurrent removers, only one puts the base
ref.
* Regardless of which one puts the base ref, all removers are blocked
until the target node is fully deactivated and removed.
To achieve the above, removal path now first marks all descendants
including self REMOVED and then deactivates and unlinks leftmost
descendant one-by-one. kernfs_deactivate() is called directly from
__kernfs_removal() and drops and regrabs kernfs_mutex for each
descendant to drain active refs. As this means that multiple removers
can enter kernfs_deactivate() for the same node, the function is
updated so that it can handle multiple deactivators of the same node -
only one actually deactivates but all wait till drain completion.
The restructured removal path guarantees that a removed node gets
unlinked only after the node is deactivated and drained. Combined
with proper multiple deactivator handling, this guarantees that any
invocation of kernfs_remove() returns only after the node itself and
all its descendants are deactivated, drained and removed.
v2: Draining separated into a separate loop (used to be in the same
loop as unlink) and done from __kernfs_deactivate(). This is to
allow exposing deactivation as a separate interface later.
Root node removal was broken in v1 patch. Fixed.
v3: Revert most of v2 except for root node removal fix and
simplification of KERNFS_REMOVED setting loop.
v4: Refreshed on top of ("kernfs: make kernfs_deactivate() honor
KERNFS_LOCKDEP flag").
Signed-off-by: Tejun Heo <tj@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-02-04 03:02:56 +08:00
|
|
|
static bool kernfs_unlink_sibling(struct kernfs_node *kn)
|
2013-11-29 03:54:33 +08:00
|
|
|
{
|
kernfs: restructure removal path to fix possible premature return
The recursive nature of kernfs_remove() means that, even if
kernfs_remove() is not allowed to be called multiple times on the same
node, there may be race conditions between removal of parent and its
descendants. While we can claim that kernfs_remove() shouldn't be
called on one of the descendants while the removal of an ancestor is
in progress, such rule is unnecessarily restrictive and very difficult
to enforce. It's better to simply allow invoking kernfs_remove() as
the caller sees fit as long as the caller ensures that the node is
accessible.
The current behavior in such situations is broken. Whoever enters
removal path first takes the node off the hierarchy and then
deactivates. Following removers either return as soon as it notices
that it's not the first one or can't even find the target node as it
has already been removed from the hierarchy. In both cases, the
following removers may finish prematurely while the nodes which should
be removed and drained are still being processed by the first one.
This patch restructures so that multiple removers, whether through
recursion or direction invocation, always follow the following rules.
* When there are multiple concurrent removers, only one puts the base
ref.
* Regardless of which one puts the base ref, all removers are blocked
until the target node is fully deactivated and removed.
To achieve the above, removal path now first marks all descendants
including self REMOVED and then deactivates and unlinks leftmost
descendant one-by-one. kernfs_deactivate() is called directly from
__kernfs_removal() and drops and regrabs kernfs_mutex for each
descendant to drain active refs. As this means that multiple removers
can enter kernfs_deactivate() for the same node, the function is
updated so that it can handle multiple deactivators of the same node -
only one actually deactivates but all wait till drain completion.
The restructured removal path guarantees that a removed node gets
unlinked only after the node is deactivated and drained. Combined
with proper multiple deactivator handling, this guarantees that any
invocation of kernfs_remove() returns only after the node itself and
all its descendants are deactivated, drained and removed.
v2: Draining separated into a separate loop (used to be in the same
loop as unlink) and done from __kernfs_deactivate(). This is to
allow exposing deactivation as a separate interface later.
Root node removal was broken in v1 patch. Fixed.
v3: Revert most of v2 except for root node removal fix and
simplification of KERNFS_REMOVED setting loop.
v4: Refreshed on top of ("kernfs: make kernfs_deactivate() honor
KERNFS_LOCKDEP flag").
Signed-off-by: Tejun Heo <tj@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-02-04 03:02:56 +08:00
|
|
|
if (RB_EMPTY_NODE(&kn->rb))
|
|
|
|
return false;
|
|
|
|
|
2013-12-12 03:11:56 +08:00
|
|
|
if (kernfs_type(kn) == KERNFS_DIR)
|
2013-12-12 03:11:54 +08:00
|
|
|
kn->parent->dir.subdirs--;
|
2013-11-29 03:54:33 +08:00
|
|
|
|
2013-12-12 03:11:54 +08:00
|
|
|
rb_erase(&kn->rb, &kn->parent->dir.children);
|
kernfs: restructure removal path to fix possible premature return
The recursive nature of kernfs_remove() means that, even if
kernfs_remove() is not allowed to be called multiple times on the same
node, there may be race conditions between removal of parent and its
descendants. While we can claim that kernfs_remove() shouldn't be
called on one of the descendants while the removal of an ancestor is
in progress, such rule is unnecessarily restrictive and very difficult
to enforce. It's better to simply allow invoking kernfs_remove() as
the caller sees fit as long as the caller ensures that the node is
accessible.
The current behavior in such situations is broken. Whoever enters
removal path first takes the node off the hierarchy and then
deactivates. Following removers either return as soon as it notices
that it's not the first one or can't even find the target node as it
has already been removed from the hierarchy. In both cases, the
following removers may finish prematurely while the nodes which should
be removed and drained are still being processed by the first one.
This patch restructures so that multiple removers, whether through
recursion or direction invocation, always follow the following rules.
* When there are multiple concurrent removers, only one puts the base
ref.
* Regardless of which one puts the base ref, all removers are blocked
until the target node is fully deactivated and removed.
To achieve the above, removal path now first marks all descendants
including self REMOVED and then deactivates and unlinks leftmost
descendant one-by-one. kernfs_deactivate() is called directly from
__kernfs_removal() and drops and regrabs kernfs_mutex for each
descendant to drain active refs. As this means that multiple removers
can enter kernfs_deactivate() for the same node, the function is
updated so that it can handle multiple deactivators of the same node -
only one actually deactivates but all wait till drain completion.
The restructured removal path guarantees that a removed node gets
unlinked only after the node is deactivated and drained. Combined
with proper multiple deactivator handling, this guarantees that any
invocation of kernfs_remove() returns only after the node itself and
all its descendants are deactivated, drained and removed.
v2: Draining separated into a separate loop (used to be in the same
loop as unlink) and done from __kernfs_deactivate(). This is to
allow exposing deactivation as a separate interface later.
Root node removal was broken in v1 patch. Fixed.
v3: Revert most of v2 except for root node removal fix and
simplification of KERNFS_REMOVED setting loop.
v4: Refreshed on top of ("kernfs: make kernfs_deactivate() honor
KERNFS_LOCKDEP flag").
Signed-off-by: Tejun Heo <tj@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-02-04 03:02:56 +08:00
|
|
|
RB_CLEAR_NODE(&kn->rb);
|
|
|
|
return true;
|
2013-11-29 03:54:33 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
/**
|
2013-12-12 03:11:58 +08:00
|
|
|
* kernfs_get_active - get an active reference to kernfs_node
|
2013-12-12 03:11:53 +08:00
|
|
|
* @kn: kernfs_node to get an active reference to
|
2013-11-29 03:54:33 +08:00
|
|
|
*
|
2013-12-12 03:11:53 +08:00
|
|
|
* Get an active reference of @kn. This function is noop if @kn
|
2013-11-29 03:54:33 +08:00
|
|
|
* is NULL.
|
|
|
|
*
|
|
|
|
* RETURNS:
|
2013-12-12 03:11:53 +08:00
|
|
|
* Pointer to @kn on success, NULL on failure.
|
2013-11-29 03:54:33 +08:00
|
|
|
*/
|
2013-12-12 03:11:58 +08:00
|
|
|
struct kernfs_node *kernfs_get_active(struct kernfs_node *kn)
|
2013-11-29 03:54:33 +08:00
|
|
|
{
|
2013-12-12 03:11:53 +08:00
|
|
|
if (unlikely(!kn))
|
2013-11-29 03:54:33 +08:00
|
|
|
return NULL;
|
|
|
|
|
2014-01-14 06:13:39 +08:00
|
|
|
if (!atomic_inc_unless_negative(&kn->active))
|
|
|
|
return NULL;
|
2014-01-10 21:57:25 +08:00
|
|
|
|
2014-02-04 03:02:59 +08:00
|
|
|
if (kernfs_lockdep(kn))
|
2014-01-14 06:13:39 +08:00
|
|
|
rwsem_acquire_read(&kn->dep_map, 0, 1, _RET_IP_);
|
|
|
|
return kn;
|
2013-11-29 03:54:33 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
/**
|
2013-12-12 03:11:58 +08:00
|
|
|
* kernfs_put_active - put an active reference to kernfs_node
|
2013-12-12 03:11:53 +08:00
|
|
|
* @kn: kernfs_node to put an active reference to
|
2013-11-29 03:54:33 +08:00
|
|
|
*
|
2013-12-12 03:11:53 +08:00
|
|
|
* Put an active reference to @kn. This function is noop if @kn
|
2013-11-29 03:54:33 +08:00
|
|
|
* is NULL.
|
|
|
|
*/
|
2013-12-12 03:11:58 +08:00
|
|
|
void kernfs_put_active(struct kernfs_node *kn)
|
2013-11-29 03:54:33 +08:00
|
|
|
{
|
2014-02-04 03:02:55 +08:00
|
|
|
struct kernfs_root *root = kernfs_root(kn);
|
2013-11-29 03:54:33 +08:00
|
|
|
int v;
|
|
|
|
|
2013-12-12 03:11:53 +08:00
|
|
|
if (unlikely(!kn))
|
2013-11-29 03:54:33 +08:00
|
|
|
return;
|
|
|
|
|
2014-02-04 03:02:59 +08:00
|
|
|
if (kernfs_lockdep(kn))
|
2013-12-12 03:11:53 +08:00
|
|
|
rwsem_release(&kn->dep_map, 1, _RET_IP_);
|
2013-12-12 03:11:54 +08:00
|
|
|
v = atomic_dec_return(&kn->active);
|
2013-12-12 03:11:56 +08:00
|
|
|
if (likely(v != KN_DEACTIVATED_BIAS))
|
2013-11-29 03:54:33 +08:00
|
|
|
return;
|
|
|
|
|
2014-02-04 03:02:55 +08:00
|
|
|
wake_up_all(&root->deactivate_waitq);
|
2013-11-29 03:54:33 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
/**
|
2014-02-04 03:03:00 +08:00
|
|
|
* kernfs_drain - drain kernfs_node
|
|
|
|
* @kn: kernfs_node to drain
|
2013-11-29 03:54:33 +08:00
|
|
|
*
|
2014-02-04 03:03:00 +08:00
|
|
|
* Drain existing usages and nuke all existing mmaps of @kn. Mutiple
|
|
|
|
* removers may invoke this function concurrently on @kn and all will
|
|
|
|
* return after draining is complete.
|
2013-11-29 03:54:33 +08:00
|
|
|
*/
|
2014-02-04 03:03:00 +08:00
|
|
|
static void kernfs_drain(struct kernfs_node *kn)
|
kernfs: restructure removal path to fix possible premature return
The recursive nature of kernfs_remove() means that, even if
kernfs_remove() is not allowed to be called multiple times on the same
node, there may be race conditions between removal of parent and its
descendants. While we can claim that kernfs_remove() shouldn't be
called on one of the descendants while the removal of an ancestor is
in progress, such rule is unnecessarily restrictive and very difficult
to enforce. It's better to simply allow invoking kernfs_remove() as
the caller sees fit as long as the caller ensures that the node is
accessible.
The current behavior in such situations is broken. Whoever enters
removal path first takes the node off the hierarchy and then
deactivates. Following removers either return as soon as it notices
that it's not the first one or can't even find the target node as it
has already been removed from the hierarchy. In both cases, the
following removers may finish prematurely while the nodes which should
be removed and drained are still being processed by the first one.
This patch restructures so that multiple removers, whether through
recursion or direction invocation, always follow the following rules.
* When there are multiple concurrent removers, only one puts the base
ref.
* Regardless of which one puts the base ref, all removers are blocked
until the target node is fully deactivated and removed.
To achieve the above, removal path now first marks all descendants
including self REMOVED and then deactivates and unlinks leftmost
descendant one-by-one. kernfs_deactivate() is called directly from
__kernfs_removal() and drops and regrabs kernfs_mutex for each
descendant to drain active refs. As this means that multiple removers
can enter kernfs_deactivate() for the same node, the function is
updated so that it can handle multiple deactivators of the same node -
only one actually deactivates but all wait till drain completion.
The restructured removal path guarantees that a removed node gets
unlinked only after the node is deactivated and drained. Combined
with proper multiple deactivator handling, this guarantees that any
invocation of kernfs_remove() returns only after the node itself and
all its descendants are deactivated, drained and removed.
v2: Draining separated into a separate loop (used to be in the same
loop as unlink) and done from __kernfs_deactivate(). This is to
allow exposing deactivation as a separate interface later.
Root node removal was broken in v1 patch. Fixed.
v3: Revert most of v2 except for root node removal fix and
simplification of KERNFS_REMOVED setting loop.
v4: Refreshed on top of ("kernfs: make kernfs_deactivate() honor
KERNFS_LOCKDEP flag").
Signed-off-by: Tejun Heo <tj@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-02-04 03:02:56 +08:00
|
|
|
__releases(&kernfs_mutex) __acquires(&kernfs_mutex)
|
2013-11-29 03:54:33 +08:00
|
|
|
{
|
2014-02-04 03:02:55 +08:00
|
|
|
struct kernfs_root *root = kernfs_root(kn);
|
2013-11-29 03:54:33 +08:00
|
|
|
|
kernfs: restructure removal path to fix possible premature return
The recursive nature of kernfs_remove() means that, even if
kernfs_remove() is not allowed to be called multiple times on the same
node, there may be race conditions between removal of parent and its
descendants. While we can claim that kernfs_remove() shouldn't be
called on one of the descendants while the removal of an ancestor is
in progress, such rule is unnecessarily restrictive and very difficult
to enforce. It's better to simply allow invoking kernfs_remove() as
the caller sees fit as long as the caller ensures that the node is
accessible.
The current behavior in such situations is broken. Whoever enters
removal path first takes the node off the hierarchy and then
deactivates. Following removers either return as soon as it notices
that it's not the first one or can't even find the target node as it
has already been removed from the hierarchy. In both cases, the
following removers may finish prematurely while the nodes which should
be removed and drained are still being processed by the first one.
This patch restructures so that multiple removers, whether through
recursion or direction invocation, always follow the following rules.
* When there are multiple concurrent removers, only one puts the base
ref.
* Regardless of which one puts the base ref, all removers are blocked
until the target node is fully deactivated and removed.
To achieve the above, removal path now first marks all descendants
including self REMOVED and then deactivates and unlinks leftmost
descendant one-by-one. kernfs_deactivate() is called directly from
__kernfs_removal() and drops and regrabs kernfs_mutex for each
descendant to drain active refs. As this means that multiple removers
can enter kernfs_deactivate() for the same node, the function is
updated so that it can handle multiple deactivators of the same node -
only one actually deactivates but all wait till drain completion.
The restructured removal path guarantees that a removed node gets
unlinked only after the node is deactivated and drained. Combined
with proper multiple deactivator handling, this guarantees that any
invocation of kernfs_remove() returns only after the node itself and
all its descendants are deactivated, drained and removed.
v2: Draining separated into a separate loop (used to be in the same
loop as unlink) and done from __kernfs_deactivate(). This is to
allow exposing deactivation as a separate interface later.
Root node removal was broken in v1 patch. Fixed.
v3: Revert most of v2 except for root node removal fix and
simplification of KERNFS_REMOVED setting loop.
v4: Refreshed on top of ("kernfs: make kernfs_deactivate() honor
KERNFS_LOCKDEP flag").
Signed-off-by: Tejun Heo <tj@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-02-04 03:02:56 +08:00
|
|
|
lockdep_assert_held(&kernfs_mutex);
|
2014-02-04 03:03:00 +08:00
|
|
|
WARN_ON_ONCE(kernfs_active(kn));
|
2014-01-10 21:57:19 +08:00
|
|
|
|
kernfs: restructure removal path to fix possible premature return
The recursive nature of kernfs_remove() means that, even if
kernfs_remove() is not allowed to be called multiple times on the same
node, there may be race conditions between removal of parent and its
descendants. While we can claim that kernfs_remove() shouldn't be
called on one of the descendants while the removal of an ancestor is
in progress, such rule is unnecessarily restrictive and very difficult
to enforce. It's better to simply allow invoking kernfs_remove() as
the caller sees fit as long as the caller ensures that the node is
accessible.
The current behavior in such situations is broken. Whoever enters
removal path first takes the node off the hierarchy and then
deactivates. Following removers either return as soon as it notices
that it's not the first one or can't even find the target node as it
has already been removed from the hierarchy. In both cases, the
following removers may finish prematurely while the nodes which should
be removed and drained are still being processed by the first one.
This patch restructures so that multiple removers, whether through
recursion or direction invocation, always follow the following rules.
* When there are multiple concurrent removers, only one puts the base
ref.
* Regardless of which one puts the base ref, all removers are blocked
until the target node is fully deactivated and removed.
To achieve the above, removal path now first marks all descendants
including self REMOVED and then deactivates and unlinks leftmost
descendant one-by-one. kernfs_deactivate() is called directly from
__kernfs_removal() and drops and regrabs kernfs_mutex for each
descendant to drain active refs. As this means that multiple removers
can enter kernfs_deactivate() for the same node, the function is
updated so that it can handle multiple deactivators of the same node -
only one actually deactivates but all wait till drain completion.
The restructured removal path guarantees that a removed node gets
unlinked only after the node is deactivated and drained. Combined
with proper multiple deactivator handling, this guarantees that any
invocation of kernfs_remove() returns only after the node itself and
all its descendants are deactivated, drained and removed.
v2: Draining separated into a separate loop (used to be in the same
loop as unlink) and done from __kernfs_deactivate(). This is to
allow exposing deactivation as a separate interface later.
Root node removal was broken in v1 patch. Fixed.
v3: Revert most of v2 except for root node removal fix and
simplification of KERNFS_REMOVED setting loop.
v4: Refreshed on top of ("kernfs: make kernfs_deactivate() honor
KERNFS_LOCKDEP flag").
Signed-off-by: Tejun Heo <tj@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-02-04 03:02:56 +08:00
|
|
|
mutex_unlock(&kernfs_mutex);
|
2014-02-04 03:02:55 +08:00
|
|
|
|
2014-02-04 03:02:59 +08:00
|
|
|
if (kernfs_lockdep(kn)) {
|
kernfs: restructure removal path to fix possible premature return
The recursive nature of kernfs_remove() means that, even if
kernfs_remove() is not allowed to be called multiple times on the same
node, there may be race conditions between removal of parent and its
descendants. While we can claim that kernfs_remove() shouldn't be
called on one of the descendants while the removal of an ancestor is
in progress, such rule is unnecessarily restrictive and very difficult
to enforce. It's better to simply allow invoking kernfs_remove() as
the caller sees fit as long as the caller ensures that the node is
accessible.
The current behavior in such situations is broken. Whoever enters
removal path first takes the node off the hierarchy and then
deactivates. Following removers either return as soon as it notices
that it's not the first one or can't even find the target node as it
has already been removed from the hierarchy. In both cases, the
following removers may finish prematurely while the nodes which should
be removed and drained are still being processed by the first one.
This patch restructures so that multiple removers, whether through
recursion or direction invocation, always follow the following rules.
* When there are multiple concurrent removers, only one puts the base
ref.
* Regardless of which one puts the base ref, all removers are blocked
until the target node is fully deactivated and removed.
To achieve the above, removal path now first marks all descendants
including self REMOVED and then deactivates and unlinks leftmost
descendant one-by-one. kernfs_deactivate() is called directly from
__kernfs_removal() and drops and regrabs kernfs_mutex for each
descendant to drain active refs. As this means that multiple removers
can enter kernfs_deactivate() for the same node, the function is
updated so that it can handle multiple deactivators of the same node -
only one actually deactivates but all wait till drain completion.
The restructured removal path guarantees that a removed node gets
unlinked only after the node is deactivated and drained. Combined
with proper multiple deactivator handling, this guarantees that any
invocation of kernfs_remove() returns only after the node itself and
all its descendants are deactivated, drained and removed.
v2: Draining separated into a separate loop (used to be in the same
loop as unlink) and done from __kernfs_deactivate(). This is to
allow exposing deactivation as a separate interface later.
Root node removal was broken in v1 patch. Fixed.
v3: Revert most of v2 except for root node removal fix and
simplification of KERNFS_REMOVED setting loop.
v4: Refreshed on top of ("kernfs: make kernfs_deactivate() honor
KERNFS_LOCKDEP flag").
Signed-off-by: Tejun Heo <tj@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-02-04 03:02:56 +08:00
|
|
|
rwsem_acquire(&kn->dep_map, 0, 0, _RET_IP_);
|
|
|
|
if (atomic_read(&kn->active) != KN_DEACTIVATED_BIAS)
|
|
|
|
lock_contended(&kn->dep_map, _RET_IP_);
|
|
|
|
}
|
2014-02-04 03:02:55 +08:00
|
|
|
|
kernfs: restructure removal path to fix possible premature return
The recursive nature of kernfs_remove() means that, even if
kernfs_remove() is not allowed to be called multiple times on the same
node, there may be race conditions between removal of parent and its
descendants. While we can claim that kernfs_remove() shouldn't be
called on one of the descendants while the removal of an ancestor is
in progress, such rule is unnecessarily restrictive and very difficult
to enforce. It's better to simply allow invoking kernfs_remove() as
the caller sees fit as long as the caller ensures that the node is
accessible.
The current behavior in such situations is broken. Whoever enters
removal path first takes the node off the hierarchy and then
deactivates. Following removers either return as soon as it notices
that it's not the first one or can't even find the target node as it
has already been removed from the hierarchy. In both cases, the
following removers may finish prematurely while the nodes which should
be removed and drained are still being processed by the first one.
This patch restructures so that multiple removers, whether through
recursion or direction invocation, always follow the following rules.
* When there are multiple concurrent removers, only one puts the base
ref.
* Regardless of which one puts the base ref, all removers are blocked
until the target node is fully deactivated and removed.
To achieve the above, removal path now first marks all descendants
including self REMOVED and then deactivates and unlinks leftmost
descendant one-by-one. kernfs_deactivate() is called directly from
__kernfs_removal() and drops and regrabs kernfs_mutex for each
descendant to drain active refs. As this means that multiple removers
can enter kernfs_deactivate() for the same node, the function is
updated so that it can handle multiple deactivators of the same node -
only one actually deactivates but all wait till drain completion.
The restructured removal path guarantees that a removed node gets
unlinked only after the node is deactivated and drained. Combined
with proper multiple deactivator handling, this guarantees that any
invocation of kernfs_remove() returns only after the node itself and
all its descendants are deactivated, drained and removed.
v2: Draining separated into a separate loop (used to be in the same
loop as unlink) and done from __kernfs_deactivate(). This is to
allow exposing deactivation as a separate interface later.
Root node removal was broken in v1 patch. Fixed.
v3: Revert most of v2 except for root node removal fix and
simplification of KERNFS_REMOVED setting loop.
v4: Refreshed on top of ("kernfs: make kernfs_deactivate() honor
KERNFS_LOCKDEP flag").
Signed-off-by: Tejun Heo <tj@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-02-04 03:02:56 +08:00
|
|
|
/* but everyone should wait for draining */
|
2014-02-04 03:02:55 +08:00
|
|
|
wait_event(root->deactivate_waitq,
|
|
|
|
atomic_read(&kn->active) == KN_DEACTIVATED_BIAS);
|
2013-11-29 03:54:33 +08:00
|
|
|
|
2014-02-04 03:02:59 +08:00
|
|
|
if (kernfs_lockdep(kn)) {
|
2014-02-04 03:02:54 +08:00
|
|
|
lock_acquired(&kn->dep_map, _RET_IP_);
|
|
|
|
rwsem_release(&kn->dep_map, 1, _RET_IP_);
|
|
|
|
}
|
kernfs: restructure removal path to fix possible premature return
The recursive nature of kernfs_remove() means that, even if
kernfs_remove() is not allowed to be called multiple times on the same
node, there may be race conditions between removal of parent and its
descendants. While we can claim that kernfs_remove() shouldn't be
called on one of the descendants while the removal of an ancestor is
in progress, such rule is unnecessarily restrictive and very difficult
to enforce. It's better to simply allow invoking kernfs_remove() as
the caller sees fit as long as the caller ensures that the node is
accessible.
The current behavior in such situations is broken. Whoever enters
removal path first takes the node off the hierarchy and then
deactivates. Following removers either return as soon as it notices
that it's not the first one or can't even find the target node as it
has already been removed from the hierarchy. In both cases, the
following removers may finish prematurely while the nodes which should
be removed and drained are still being processed by the first one.
This patch restructures so that multiple removers, whether through
recursion or direction invocation, always follow the following rules.
* When there are multiple concurrent removers, only one puts the base
ref.
* Regardless of which one puts the base ref, all removers are blocked
until the target node is fully deactivated and removed.
To achieve the above, removal path now first marks all descendants
including self REMOVED and then deactivates and unlinks leftmost
descendant one-by-one. kernfs_deactivate() is called directly from
__kernfs_removal() and drops and regrabs kernfs_mutex for each
descendant to drain active refs. As this means that multiple removers
can enter kernfs_deactivate() for the same node, the function is
updated so that it can handle multiple deactivators of the same node -
only one actually deactivates but all wait till drain completion.
The restructured removal path guarantees that a removed node gets
unlinked only after the node is deactivated and drained. Combined
with proper multiple deactivator handling, this guarantees that any
invocation of kernfs_remove() returns only after the node itself and
all its descendants are deactivated, drained and removed.
v2: Draining separated into a separate loop (used to be in the same
loop as unlink) and done from __kernfs_deactivate(). This is to
allow exposing deactivation as a separate interface later.
Root node removal was broken in v1 patch. Fixed.
v3: Revert most of v2 except for root node removal fix and
simplification of KERNFS_REMOVED setting loop.
v4: Refreshed on top of ("kernfs: make kernfs_deactivate() honor
KERNFS_LOCKDEP flag").
Signed-off-by: Tejun Heo <tj@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-02-04 03:02:56 +08:00
|
|
|
|
2014-02-04 03:02:57 +08:00
|
|
|
kernfs_unmap_bin_file(kn);
|
|
|
|
|
kernfs: restructure removal path to fix possible premature return
The recursive nature of kernfs_remove() means that, even if
kernfs_remove() is not allowed to be called multiple times on the same
node, there may be race conditions between removal of parent and its
descendants. While we can claim that kernfs_remove() shouldn't be
called on one of the descendants while the removal of an ancestor is
in progress, such rule is unnecessarily restrictive and very difficult
to enforce. It's better to simply allow invoking kernfs_remove() as
the caller sees fit as long as the caller ensures that the node is
accessible.
The current behavior in such situations is broken. Whoever enters
removal path first takes the node off the hierarchy and then
deactivates. Following removers either return as soon as it notices
that it's not the first one or can't even find the target node as it
has already been removed from the hierarchy. In both cases, the
following removers may finish prematurely while the nodes which should
be removed and drained are still being processed by the first one.
This patch restructures so that multiple removers, whether through
recursion or direction invocation, always follow the following rules.
* When there are multiple concurrent removers, only one puts the base
ref.
* Regardless of which one puts the base ref, all removers are blocked
until the target node is fully deactivated and removed.
To achieve the above, removal path now first marks all descendants
including self REMOVED and then deactivates and unlinks leftmost
descendant one-by-one. kernfs_deactivate() is called directly from
__kernfs_removal() and drops and regrabs kernfs_mutex for each
descendant to drain active refs. As this means that multiple removers
can enter kernfs_deactivate() for the same node, the function is
updated so that it can handle multiple deactivators of the same node -
only one actually deactivates but all wait till drain completion.
The restructured removal path guarantees that a removed node gets
unlinked only after the node is deactivated and drained. Combined
with proper multiple deactivator handling, this guarantees that any
invocation of kernfs_remove() returns only after the node itself and
all its descendants are deactivated, drained and removed.
v2: Draining separated into a separate loop (used to be in the same
loop as unlink) and done from __kernfs_deactivate(). This is to
allow exposing deactivation as a separate interface later.
Root node removal was broken in v1 patch. Fixed.
v3: Revert most of v2 except for root node removal fix and
simplification of KERNFS_REMOVED setting loop.
v4: Refreshed on top of ("kernfs: make kernfs_deactivate() honor
KERNFS_LOCKDEP flag").
Signed-off-by: Tejun Heo <tj@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-02-04 03:02:56 +08:00
|
|
|
mutex_lock(&kernfs_mutex);
|
2013-11-29 03:54:33 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
/**
|
2013-12-12 03:11:53 +08:00
|
|
|
* kernfs_get - get a reference count on a kernfs_node
|
|
|
|
* @kn: the target kernfs_node
|
2013-11-29 03:54:33 +08:00
|
|
|
*/
|
2013-12-12 03:11:53 +08:00
|
|
|
void kernfs_get(struct kernfs_node *kn)
|
2013-11-29 03:54:33 +08:00
|
|
|
{
|
2013-12-12 03:11:53 +08:00
|
|
|
if (kn) {
|
2013-12-12 03:11:54 +08:00
|
|
|
WARN_ON(!atomic_read(&kn->count));
|
|
|
|
atomic_inc(&kn->count);
|
2013-11-29 03:54:33 +08:00
|
|
|
}
|
|
|
|
}
|
|
|
|
EXPORT_SYMBOL_GPL(kernfs_get);
|
|
|
|
|
|
|
|
/**
|
2013-12-12 03:11:53 +08:00
|
|
|
* kernfs_put - put a reference count on a kernfs_node
|
|
|
|
* @kn: the target kernfs_node
|
2013-11-29 03:54:33 +08:00
|
|
|
*
|
2013-12-12 03:11:53 +08:00
|
|
|
* Put a reference count of @kn and destroy it if it reached zero.
|
2013-11-29 03:54:33 +08:00
|
|
|
*/
|
2013-12-12 03:11:53 +08:00
|
|
|
void kernfs_put(struct kernfs_node *kn)
|
2013-11-29 03:54:33 +08:00
|
|
|
{
|
2013-12-12 03:11:53 +08:00
|
|
|
struct kernfs_node *parent;
|
2013-11-29 03:54:40 +08:00
|
|
|
struct kernfs_root *root;
|
2013-11-29 03:54:33 +08:00
|
|
|
|
2013-12-12 03:11:54 +08:00
|
|
|
if (!kn || !atomic_dec_and_test(&kn->count))
|
2013-11-29 03:54:33 +08:00
|
|
|
return;
|
2013-12-12 03:11:53 +08:00
|
|
|
root = kernfs_root(kn);
|
2013-11-29 03:54:33 +08:00
|
|
|
repeat:
|
2014-02-04 03:03:00 +08:00
|
|
|
/*
|
|
|
|
* Moving/renaming is always done while holding reference.
|
2013-12-12 03:11:54 +08:00
|
|
|
* kn->parent won't change beneath us.
|
2013-11-29 03:54:33 +08:00
|
|
|
*/
|
2013-12-12 03:11:54 +08:00
|
|
|
parent = kn->parent;
|
2013-11-29 03:54:33 +08:00
|
|
|
|
2014-02-04 03:03:00 +08:00
|
|
|
WARN_ONCE(atomic_read(&kn->active) != KN_DEACTIVATED_BIAS,
|
|
|
|
"kernfs_put: %s/%s: released with incorrect active_ref %d\n",
|
|
|
|
parent ? parent->name : "", kn->name, atomic_read(&kn->active));
|
2013-12-12 03:11:53 +08:00
|
|
|
|
2013-12-12 03:11:56 +08:00
|
|
|
if (kernfs_type(kn) == KERNFS_LINK)
|
2013-12-12 03:11:54 +08:00
|
|
|
kernfs_put(kn->symlink.target_kn);
|
2015-02-14 06:36:31 +08:00
|
|
|
|
|
|
|
kfree_const(kn->name);
|
|
|
|
|
2013-12-12 03:11:54 +08:00
|
|
|
if (kn->iattr) {
|
|
|
|
if (kn->iattr->ia_secdata)
|
|
|
|
security_release_secctx(kn->iattr->ia_secdata,
|
|
|
|
kn->iattr->ia_secdata_len);
|
|
|
|
simple_xattrs_free(&kn->iattr->xattrs);
|
2013-11-24 06:40:02 +08:00
|
|
|
}
|
2013-12-12 03:11:54 +08:00
|
|
|
kfree(kn->iattr);
|
|
|
|
ida_simple_remove(&root->ino_ida, kn->ino);
|
2013-12-12 03:11:57 +08:00
|
|
|
kmem_cache_free(kernfs_node_cache, kn);
|
2013-11-29 03:54:33 +08:00
|
|
|
|
2013-12-12 03:11:53 +08:00
|
|
|
kn = parent;
|
|
|
|
if (kn) {
|
2013-12-12 03:11:54 +08:00
|
|
|
if (atomic_dec_and_test(&kn->count))
|
2013-11-29 03:54:40 +08:00
|
|
|
goto repeat;
|
|
|
|
} else {
|
2013-12-12 03:11:53 +08:00
|
|
|
/* just released the root kn, free @root too */
|
2013-11-29 03:54:41 +08:00
|
|
|
ida_destroy(&root->ino_ida);
|
2013-11-29 03:54:40 +08:00
|
|
|
kfree(root);
|
|
|
|
}
|
2013-11-29 03:54:33 +08:00
|
|
|
}
|
|
|
|
EXPORT_SYMBOL_GPL(kernfs_put);
|
|
|
|
|
2013-12-12 03:11:58 +08:00
|
|
|
static int kernfs_dop_revalidate(struct dentry *dentry, unsigned int flags)
|
2013-11-29 03:54:33 +08:00
|
|
|
{
|
2013-12-12 03:11:53 +08:00
|
|
|
struct kernfs_node *kn;
|
2013-11-29 03:54:33 +08:00
|
|
|
|
|
|
|
if (flags & LOOKUP_RCU)
|
|
|
|
return -ECHILD;
|
|
|
|
|
2013-12-12 05:02:59 +08:00
|
|
|
/* Always perform fresh lookup for negatives */
|
2015-03-18 06:25:59 +08:00
|
|
|
if (d_really_is_negative(dentry))
|
2013-12-12 05:02:59 +08:00
|
|
|
goto out_bad_unlocked;
|
|
|
|
|
2013-12-12 03:11:53 +08:00
|
|
|
kn = dentry->d_fsdata;
|
2013-12-12 03:11:57 +08:00
|
|
|
mutex_lock(&kernfs_mutex);
|
2013-11-29 03:54:33 +08:00
|
|
|
|
2014-02-04 03:03:00 +08:00
|
|
|
/* The kernfs node has been deactivated */
|
|
|
|
if (!kernfs_active(kn))
|
2013-11-29 03:54:33 +08:00
|
|
|
goto out_bad;
|
|
|
|
|
2013-12-12 03:11:58 +08:00
|
|
|
/* The kernfs node has been moved? */
|
2013-12-12 03:11:54 +08:00
|
|
|
if (dentry->d_parent->d_fsdata != kn->parent)
|
2013-11-29 03:54:33 +08:00
|
|
|
goto out_bad;
|
|
|
|
|
2013-12-12 03:11:58 +08:00
|
|
|
/* The kernfs node has been renamed */
|
2013-12-12 03:11:54 +08:00
|
|
|
if (strcmp(dentry->d_name.name, kn->name) != 0)
|
2013-11-29 03:54:33 +08:00
|
|
|
goto out_bad;
|
|
|
|
|
2013-12-12 03:11:58 +08:00
|
|
|
/* The kernfs node has been moved to a different namespace */
|
2013-12-12 03:11:54 +08:00
|
|
|
if (kn->parent && kernfs_ns_enabled(kn->parent) &&
|
2013-12-12 03:11:55 +08:00
|
|
|
kernfs_info(dentry->d_sb)->ns != kn->ns)
|
2013-11-29 03:54:33 +08:00
|
|
|
goto out_bad;
|
|
|
|
|
2013-12-12 03:11:57 +08:00
|
|
|
mutex_unlock(&kernfs_mutex);
|
2013-11-29 03:54:33 +08:00
|
|
|
return 1;
|
|
|
|
out_bad:
|
2013-12-12 03:11:57 +08:00
|
|
|
mutex_unlock(&kernfs_mutex);
|
2013-12-12 05:02:59 +08:00
|
|
|
out_bad_unlocked:
|
2013-11-29 03:54:33 +08:00
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
2013-12-12 03:11:58 +08:00
|
|
|
static void kernfs_dop_release(struct dentry *dentry)
|
2013-11-29 03:54:33 +08:00
|
|
|
{
|
|
|
|
kernfs_put(dentry->d_fsdata);
|
|
|
|
}
|
|
|
|
|
2013-12-12 03:11:57 +08:00
|
|
|
const struct dentry_operations kernfs_dops = {
|
2013-12-12 03:11:58 +08:00
|
|
|
.d_revalidate = kernfs_dop_revalidate,
|
|
|
|
.d_release = kernfs_dop_release,
|
2013-11-29 03:54:33 +08:00
|
|
|
};
|
|
|
|
|
2014-02-04 03:09:15 +08:00
|
|
|
/**
|
|
|
|
* kernfs_node_from_dentry - determine kernfs_node associated with a dentry
|
|
|
|
* @dentry: the dentry in question
|
|
|
|
*
|
|
|
|
* Return the kernfs_node associated with @dentry. If @dentry is not a
|
|
|
|
* kernfs one, %NULL is returned.
|
|
|
|
*
|
|
|
|
* While the returned kernfs_node will stay accessible as long as @dentry
|
|
|
|
* is accessible, the returned node can be in any state and the caller is
|
|
|
|
* fully responsible for determining what's accessible.
|
|
|
|
*/
|
|
|
|
struct kernfs_node *kernfs_node_from_dentry(struct dentry *dentry)
|
|
|
|
{
|
2014-02-14 16:57:27 +08:00
|
|
|
if (dentry->d_sb->s_op == &kernfs_sops)
|
2014-02-04 03:09:15 +08:00
|
|
|
return dentry->d_fsdata;
|
|
|
|
return NULL;
|
|
|
|
}
|
|
|
|
|
2014-01-17 22:58:25 +08:00
|
|
|
static struct kernfs_node *__kernfs_new_node(struct kernfs_root *root,
|
|
|
|
const char *name, umode_t mode,
|
|
|
|
unsigned flags)
|
2013-11-29 03:54:33 +08:00
|
|
|
{
|
2013-12-12 03:11:53 +08:00
|
|
|
struct kernfs_node *kn;
|
2013-11-29 03:54:41 +08:00
|
|
|
int ret;
|
2013-11-29 03:54:33 +08:00
|
|
|
|
2015-02-14 06:36:31 +08:00
|
|
|
name = kstrdup_const(name, GFP_KERNEL);
|
|
|
|
if (!name)
|
|
|
|
return NULL;
|
2013-11-29 03:54:33 +08:00
|
|
|
|
2013-12-12 03:11:57 +08:00
|
|
|
kn = kmem_cache_zalloc(kernfs_node_cache, GFP_KERNEL);
|
2013-12-12 03:11:53 +08:00
|
|
|
if (!kn)
|
2013-11-29 03:54:33 +08:00
|
|
|
goto err_out1;
|
|
|
|
|
2015-05-15 06:16:58 +08:00
|
|
|
/*
|
|
|
|
* If the ino of the sysfs entry created for a kmem cache gets
|
|
|
|
* allocated from an ida layer, which is accounted to the memcg that
|
|
|
|
* owns the cache, the memcg will get pinned forever. So do not account
|
|
|
|
* ino ida allocations.
|
|
|
|
*/
|
|
|
|
ret = ida_simple_get(&root->ino_ida, 1, 0,
|
|
|
|
GFP_KERNEL | __GFP_NOACCOUNT);
|
2013-11-29 03:54:41 +08:00
|
|
|
if (ret < 0)
|
2013-11-29 03:54:33 +08:00
|
|
|
goto err_out2;
|
2013-12-12 03:11:54 +08:00
|
|
|
kn->ino = ret;
|
2013-11-29 03:54:33 +08:00
|
|
|
|
2013-12-12 03:11:54 +08:00
|
|
|
atomic_set(&kn->count, 1);
|
2014-02-04 03:03:00 +08:00
|
|
|
atomic_set(&kn->active, KN_DEACTIVATED_BIAS);
|
kernfs: restructure removal path to fix possible premature return
The recursive nature of kernfs_remove() means that, even if
kernfs_remove() is not allowed to be called multiple times on the same
node, there may be race conditions between removal of parent and its
descendants. While we can claim that kernfs_remove() shouldn't be
called on one of the descendants while the removal of an ancestor is
in progress, such rule is unnecessarily restrictive and very difficult
to enforce. It's better to simply allow invoking kernfs_remove() as
the caller sees fit as long as the caller ensures that the node is
accessible.
The current behavior in such situations is broken. Whoever enters
removal path first takes the node off the hierarchy and then
deactivates. Following removers either return as soon as it notices
that it's not the first one or can't even find the target node as it
has already been removed from the hierarchy. In both cases, the
following removers may finish prematurely while the nodes which should
be removed and drained are still being processed by the first one.
This patch restructures so that multiple removers, whether through
recursion or direction invocation, always follow the following rules.
* When there are multiple concurrent removers, only one puts the base
ref.
* Regardless of which one puts the base ref, all removers are blocked
until the target node is fully deactivated and removed.
To achieve the above, removal path now first marks all descendants
including self REMOVED and then deactivates and unlinks leftmost
descendant one-by-one. kernfs_deactivate() is called directly from
__kernfs_removal() and drops and regrabs kernfs_mutex for each
descendant to drain active refs. As this means that multiple removers
can enter kernfs_deactivate() for the same node, the function is
updated so that it can handle multiple deactivators of the same node -
only one actually deactivates but all wait till drain completion.
The restructured removal path guarantees that a removed node gets
unlinked only after the node is deactivated and drained. Combined
with proper multiple deactivator handling, this guarantees that any
invocation of kernfs_remove() returns only after the node itself and
all its descendants are deactivated, drained and removed.
v2: Draining separated into a separate loop (used to be in the same
loop as unlink) and done from __kernfs_deactivate(). This is to
allow exposing deactivation as a separate interface later.
Root node removal was broken in v1 patch. Fixed.
v3: Revert most of v2 except for root node removal fix and
simplification of KERNFS_REMOVED setting loop.
v4: Refreshed on top of ("kernfs: make kernfs_deactivate() honor
KERNFS_LOCKDEP flag").
Signed-off-by: Tejun Heo <tj@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-02-04 03:02:56 +08:00
|
|
|
RB_CLEAR_NODE(&kn->rb);
|
2013-11-29 03:54:33 +08:00
|
|
|
|
2013-12-12 03:11:54 +08:00
|
|
|
kn->name = name;
|
|
|
|
kn->mode = mode;
|
2014-02-04 03:03:00 +08:00
|
|
|
kn->flags = flags;
|
2013-11-29 03:54:33 +08:00
|
|
|
|
2013-12-12 03:11:53 +08:00
|
|
|
return kn;
|
2013-11-29 03:54:33 +08:00
|
|
|
|
|
|
|
err_out2:
|
2013-12-12 03:11:57 +08:00
|
|
|
kmem_cache_free(kernfs_node_cache, kn);
|
2013-11-29 03:54:33 +08:00
|
|
|
err_out1:
|
2015-02-14 06:36:31 +08:00
|
|
|
kfree_const(name);
|
2013-11-29 03:54:33 +08:00
|
|
|
return NULL;
|
|
|
|
}
|
|
|
|
|
2014-01-17 22:58:25 +08:00
|
|
|
struct kernfs_node *kernfs_new_node(struct kernfs_node *parent,
|
|
|
|
const char *name, umode_t mode,
|
|
|
|
unsigned flags)
|
|
|
|
{
|
|
|
|
struct kernfs_node *kn;
|
|
|
|
|
|
|
|
kn = __kernfs_new_node(kernfs_root(parent), name, mode, flags);
|
|
|
|
if (kn) {
|
|
|
|
kernfs_get(parent);
|
|
|
|
kn->parent = parent;
|
|
|
|
}
|
|
|
|
return kn;
|
|
|
|
}
|
|
|
|
|
2013-11-29 03:54:33 +08:00
|
|
|
/**
|
2013-12-12 03:11:58 +08:00
|
|
|
* kernfs_add_one - add kernfs_node to parent without warning
|
2013-12-12 03:11:53 +08:00
|
|
|
* @kn: kernfs_node to be added
|
2013-11-29 03:54:33 +08:00
|
|
|
*
|
2014-01-17 22:58:25 +08:00
|
|
|
* The caller must already have initialized @kn->parent. This
|
|
|
|
* function increments nlink of the parent's inode if @kn is a
|
|
|
|
* directory and link into the children list of the parent.
|
2013-11-29 03:54:33 +08:00
|
|
|
*
|
|
|
|
* RETURNS:
|
|
|
|
* 0 on success, -EEXIST if entry with the given name already
|
|
|
|
* exists.
|
|
|
|
*/
|
2014-02-04 03:02:58 +08:00
|
|
|
int kernfs_add_one(struct kernfs_node *kn)
|
2013-11-29 03:54:33 +08:00
|
|
|
{
|
2014-01-17 22:58:25 +08:00
|
|
|
struct kernfs_node *parent = kn->parent;
|
2013-12-12 03:11:55 +08:00
|
|
|
struct kernfs_iattrs *ps_iattr;
|
2014-02-04 03:02:58 +08:00
|
|
|
bool has_ns;
|
2013-11-29 03:54:33 +08:00
|
|
|
int ret;
|
|
|
|
|
2014-02-04 03:02:58 +08:00
|
|
|
mutex_lock(&kernfs_mutex);
|
|
|
|
|
|
|
|
ret = -EINVAL;
|
|
|
|
has_ns = kernfs_ns_enabled(parent);
|
|
|
|
if (WARN(has_ns != (bool)kn->ns, KERN_WARNING "kernfs: ns %s in '%s' for '%s'\n",
|
|
|
|
has_ns ? "required" : "invalid", parent->name, kn->name))
|
|
|
|
goto out_unlock;
|
2013-11-29 03:54:33 +08:00
|
|
|
|
2013-12-12 03:11:56 +08:00
|
|
|
if (kernfs_type(parent) != KERNFS_DIR)
|
2014-02-04 03:02:58 +08:00
|
|
|
goto out_unlock;
|
2013-11-29 03:54:33 +08:00
|
|
|
|
2014-02-04 03:02:58 +08:00
|
|
|
ret = -ENOENT;
|
2015-05-14 05:09:29 +08:00
|
|
|
if (parent->flags & KERNFS_EMPTY_DIR)
|
|
|
|
goto out_unlock;
|
|
|
|
|
2014-02-04 03:09:12 +08:00
|
|
|
if ((parent->flags & KERNFS_ACTIVATED) && !kernfs_active(parent))
|
2014-02-04 03:02:58 +08:00
|
|
|
goto out_unlock;
|
2014-01-14 06:36:03 +08:00
|
|
|
|
2013-12-12 03:11:58 +08:00
|
|
|
kn->hash = kernfs_name_hash(kn->name, kn->ns);
|
2013-11-29 03:54:33 +08:00
|
|
|
|
2013-12-12 03:11:58 +08:00
|
|
|
ret = kernfs_link_sibling(kn);
|
2013-11-29 03:54:33 +08:00
|
|
|
if (ret)
|
2014-02-04 03:02:58 +08:00
|
|
|
goto out_unlock;
|
2013-11-29 03:54:33 +08:00
|
|
|
|
|
|
|
/* Update timestamps on the parent */
|
2013-12-12 03:11:54 +08:00
|
|
|
ps_iattr = parent->iattr;
|
2013-11-29 03:54:33 +08:00
|
|
|
if (ps_iattr) {
|
|
|
|
struct iattr *ps_iattrs = &ps_iattr->ia_iattr;
|
|
|
|
ps_iattrs->ia_ctime = ps_iattrs->ia_mtime = CURRENT_TIME;
|
|
|
|
}
|
|
|
|
|
2014-02-04 03:09:12 +08:00
|
|
|
mutex_unlock(&kernfs_mutex);
|
|
|
|
|
|
|
|
/*
|
|
|
|
* Activate the new node unless CREATE_DEACTIVATED is requested.
|
|
|
|
* If not activated here, the kernfs user is responsible for
|
|
|
|
* activating the node with kernfs_activate(). A node which hasn't
|
|
|
|
* been activated is not visible to userland and its removal won't
|
|
|
|
* trigger deactivation.
|
|
|
|
*/
|
|
|
|
if (!(kernfs_root(kn)->flags & KERNFS_ROOT_CREATE_DEACTIVATED))
|
|
|
|
kernfs_activate(kn);
|
|
|
|
return 0;
|
|
|
|
|
2014-02-04 03:02:58 +08:00
|
|
|
out_unlock:
|
2013-12-12 03:11:57 +08:00
|
|
|
mutex_unlock(&kernfs_mutex);
|
2014-02-04 03:02:58 +08:00
|
|
|
return ret;
|
2013-11-29 03:54:33 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
/**
|
2013-12-12 03:11:53 +08:00
|
|
|
* kernfs_find_ns - find kernfs_node with the given name
|
|
|
|
* @parent: kernfs_node to search under
|
2013-11-29 03:54:33 +08:00
|
|
|
* @name: name to look for
|
|
|
|
* @ns: the namespace tag to use
|
|
|
|
*
|
2013-12-12 03:11:53 +08:00
|
|
|
* Look for kernfs_node with name @name under @parent. Returns pointer to
|
|
|
|
* the found kernfs_node on success, %NULL on failure.
|
2013-11-29 03:54:33 +08:00
|
|
|
*/
|
2013-12-12 03:11:53 +08:00
|
|
|
static struct kernfs_node *kernfs_find_ns(struct kernfs_node *parent,
|
|
|
|
const unsigned char *name,
|
|
|
|
const void *ns)
|
2013-11-29 03:54:33 +08:00
|
|
|
{
|
2013-12-12 03:11:54 +08:00
|
|
|
struct rb_node *node = parent->dir.children.rb_node;
|
2013-11-30 06:19:09 +08:00
|
|
|
bool has_ns = kernfs_ns_enabled(parent);
|
2013-11-29 03:54:33 +08:00
|
|
|
unsigned int hash;
|
|
|
|
|
2013-12-12 03:11:57 +08:00
|
|
|
lockdep_assert_held(&kernfs_mutex);
|
2013-11-29 03:54:33 +08:00
|
|
|
|
|
|
|
if (has_ns != (bool)ns) {
|
2013-12-12 03:11:58 +08:00
|
|
|
WARN(1, KERN_WARNING "kernfs: ns %s in '%s' for '%s'\n",
|
2013-12-12 03:11:54 +08:00
|
|
|
has_ns ? "required" : "invalid", parent->name, name);
|
2013-11-29 03:54:33 +08:00
|
|
|
return NULL;
|
|
|
|
}
|
|
|
|
|
2013-12-12 03:11:58 +08:00
|
|
|
hash = kernfs_name_hash(name, ns);
|
2013-11-29 03:54:33 +08:00
|
|
|
while (node) {
|
2013-12-12 03:11:53 +08:00
|
|
|
struct kernfs_node *kn;
|
2013-11-29 03:54:33 +08:00
|
|
|
int result;
|
|
|
|
|
2013-12-12 03:11:53 +08:00
|
|
|
kn = rb_to_kn(node);
|
2013-12-12 03:11:58 +08:00
|
|
|
result = kernfs_name_compare(hash, name, ns, kn);
|
2013-11-29 03:54:33 +08:00
|
|
|
if (result < 0)
|
|
|
|
node = node->rb_left;
|
|
|
|
else if (result > 0)
|
|
|
|
node = node->rb_right;
|
|
|
|
else
|
2013-12-12 03:11:53 +08:00
|
|
|
return kn;
|
2013-11-29 03:54:33 +08:00
|
|
|
}
|
|
|
|
return NULL;
|
|
|
|
}
|
|
|
|
|
|
|
|
/**
|
2013-12-12 03:11:53 +08:00
|
|
|
* kernfs_find_and_get_ns - find and get kernfs_node with the given name
|
|
|
|
* @parent: kernfs_node to search under
|
2013-11-29 03:54:33 +08:00
|
|
|
* @name: name to look for
|
|
|
|
* @ns: the namespace tag to use
|
|
|
|
*
|
2013-12-12 03:11:53 +08:00
|
|
|
* Look for kernfs_node with name @name under @parent and get a reference
|
2013-11-29 03:54:33 +08:00
|
|
|
* if found. This function may sleep and returns pointer to the found
|
2013-12-12 03:11:53 +08:00
|
|
|
* kernfs_node on success, %NULL on failure.
|
2013-11-29 03:54:33 +08:00
|
|
|
*/
|
2013-12-12 03:11:53 +08:00
|
|
|
struct kernfs_node *kernfs_find_and_get_ns(struct kernfs_node *parent,
|
|
|
|
const char *name, const void *ns)
|
2013-11-29 03:54:33 +08:00
|
|
|
{
|
2013-12-12 03:11:53 +08:00
|
|
|
struct kernfs_node *kn;
|
2013-11-29 03:54:33 +08:00
|
|
|
|
2013-12-12 03:11:57 +08:00
|
|
|
mutex_lock(&kernfs_mutex);
|
2013-12-12 03:11:53 +08:00
|
|
|
kn = kernfs_find_ns(parent, name, ns);
|
|
|
|
kernfs_get(kn);
|
2013-12-12 03:11:57 +08:00
|
|
|
mutex_unlock(&kernfs_mutex);
|
2013-11-29 03:54:33 +08:00
|
|
|
|
2013-12-12 03:11:53 +08:00
|
|
|
return kn;
|
2013-11-29 03:54:33 +08:00
|
|
|
}
|
|
|
|
EXPORT_SYMBOL_GPL(kernfs_find_and_get_ns);
|
|
|
|
|
2013-11-29 03:54:40 +08:00
|
|
|
/**
|
|
|
|
* kernfs_create_root - create a new kernfs hierarchy
|
2014-02-04 03:09:09 +08:00
|
|
|
* @scops: optional syscall operations for the hierarchy
|
2014-02-04 03:09:12 +08:00
|
|
|
* @flags: KERNFS_ROOT_* flags
|
2013-11-29 03:54:40 +08:00
|
|
|
* @priv: opaque data associated with the new directory
|
|
|
|
*
|
|
|
|
* Returns the root of the new hierarchy on success, ERR_PTR() value on
|
|
|
|
* failure.
|
|
|
|
*/
|
2014-02-04 03:09:09 +08:00
|
|
|
struct kernfs_root *kernfs_create_root(struct kernfs_syscall_ops *scops,
|
2014-02-04 03:09:12 +08:00
|
|
|
unsigned int flags, void *priv)
|
2013-11-29 03:54:40 +08:00
|
|
|
{
|
|
|
|
struct kernfs_root *root;
|
2013-12-12 03:11:53 +08:00
|
|
|
struct kernfs_node *kn;
|
2013-11-29 03:54:40 +08:00
|
|
|
|
|
|
|
root = kzalloc(sizeof(*root), GFP_KERNEL);
|
|
|
|
if (!root)
|
|
|
|
return ERR_PTR(-ENOMEM);
|
|
|
|
|
2013-11-29 03:54:41 +08:00
|
|
|
ida_init(&root->ino_ida);
|
2014-04-09 23:07:30 +08:00
|
|
|
INIT_LIST_HEAD(&root->supers);
|
2013-11-29 03:54:41 +08:00
|
|
|
|
2014-01-17 22:58:25 +08:00
|
|
|
kn = __kernfs_new_node(root, "", S_IFDIR | S_IRUGO | S_IXUGO,
|
|
|
|
KERNFS_DIR);
|
2013-12-12 03:11:53 +08:00
|
|
|
if (!kn) {
|
2013-11-29 03:54:41 +08:00
|
|
|
ida_destroy(&root->ino_ida);
|
2013-11-29 03:54:40 +08:00
|
|
|
kfree(root);
|
|
|
|
return ERR_PTR(-ENOMEM);
|
|
|
|
}
|
|
|
|
|
2013-12-12 03:11:53 +08:00
|
|
|
kn->priv = priv;
|
2013-12-12 03:11:54 +08:00
|
|
|
kn->dir.root = root;
|
2013-11-29 03:54:40 +08:00
|
|
|
|
2014-02-04 03:09:09 +08:00
|
|
|
root->syscall_ops = scops;
|
2014-02-04 03:09:12 +08:00
|
|
|
root->flags = flags;
|
2013-12-12 03:11:53 +08:00
|
|
|
root->kn = kn;
|
2014-02-04 03:02:55 +08:00
|
|
|
init_waitqueue_head(&root->deactivate_waitq);
|
2013-11-29 03:54:40 +08:00
|
|
|
|
2014-02-04 03:09:12 +08:00
|
|
|
if (!(root->flags & KERNFS_ROOT_CREATE_DEACTIVATED))
|
|
|
|
kernfs_activate(kn);
|
|
|
|
|
2013-11-29 03:54:40 +08:00
|
|
|
return root;
|
|
|
|
}
|
|
|
|
|
|
|
|
/**
|
|
|
|
* kernfs_destroy_root - destroy a kernfs hierarchy
|
|
|
|
* @root: root of the hierarchy to destroy
|
|
|
|
*
|
|
|
|
* Destroy the hierarchy anchored at @root by removing all existing
|
|
|
|
* directories and destroying @root.
|
|
|
|
*/
|
|
|
|
void kernfs_destroy_root(struct kernfs_root *root)
|
|
|
|
{
|
2013-12-12 03:11:53 +08:00
|
|
|
kernfs_remove(root->kn); /* will also free @root */
|
2013-11-29 03:54:40 +08:00
|
|
|
}
|
|
|
|
|
2013-11-29 03:54:33 +08:00
|
|
|
/**
|
|
|
|
* kernfs_create_dir_ns - create a directory
|
|
|
|
* @parent: parent in which to create a new directory
|
|
|
|
* @name: name of the new directory
|
2013-12-12 05:02:55 +08:00
|
|
|
* @mode: mode of the new directory
|
2013-11-29 03:54:33 +08:00
|
|
|
* @priv: opaque data associated with the new directory
|
|
|
|
* @ns: optional namespace tag of the directory
|
|
|
|
*
|
|
|
|
* Returns the created node on success, ERR_PTR() value on failure.
|
|
|
|
*/
|
2013-12-12 03:11:53 +08:00
|
|
|
struct kernfs_node *kernfs_create_dir_ns(struct kernfs_node *parent,
|
2013-12-12 05:02:55 +08:00
|
|
|
const char *name, umode_t mode,
|
|
|
|
void *priv, const void *ns)
|
2013-11-29 03:54:33 +08:00
|
|
|
{
|
2013-12-12 03:11:53 +08:00
|
|
|
struct kernfs_node *kn;
|
2013-11-29 03:54:33 +08:00
|
|
|
int rc;
|
|
|
|
|
|
|
|
/* allocate */
|
2014-01-17 22:58:25 +08:00
|
|
|
kn = kernfs_new_node(parent, name, mode | S_IFDIR, KERNFS_DIR);
|
2013-12-12 03:11:53 +08:00
|
|
|
if (!kn)
|
2013-11-29 03:54:33 +08:00
|
|
|
return ERR_PTR(-ENOMEM);
|
|
|
|
|
2013-12-12 03:11:54 +08:00
|
|
|
kn->dir.root = parent->dir.root;
|
|
|
|
kn->ns = ns;
|
2013-12-12 03:11:53 +08:00
|
|
|
kn->priv = priv;
|
2013-11-29 03:54:33 +08:00
|
|
|
|
|
|
|
/* link in */
|
2014-02-04 03:02:58 +08:00
|
|
|
rc = kernfs_add_one(kn);
|
2013-11-29 03:54:33 +08:00
|
|
|
if (!rc)
|
2013-12-12 03:11:53 +08:00
|
|
|
return kn;
|
2013-11-29 03:54:33 +08:00
|
|
|
|
2013-12-12 03:11:53 +08:00
|
|
|
kernfs_put(kn);
|
2013-11-29 03:54:33 +08:00
|
|
|
return ERR_PTR(rc);
|
|
|
|
}
|
|
|
|
|
2015-05-14 05:09:29 +08:00
|
|
|
/**
|
|
|
|
* kernfs_create_empty_dir - create an always empty directory
|
|
|
|
* @parent: parent in which to create a new directory
|
|
|
|
* @name: name of the new directory
|
|
|
|
*
|
|
|
|
* Returns the created node on success, ERR_PTR() value on failure.
|
|
|
|
*/
|
|
|
|
struct kernfs_node *kernfs_create_empty_dir(struct kernfs_node *parent,
|
|
|
|
const char *name)
|
|
|
|
{
|
|
|
|
struct kernfs_node *kn;
|
|
|
|
int rc;
|
|
|
|
|
|
|
|
/* allocate */
|
|
|
|
kn = kernfs_new_node(parent, name, S_IRUGO|S_IXUGO|S_IFDIR, KERNFS_DIR);
|
|
|
|
if (!kn)
|
|
|
|
return ERR_PTR(-ENOMEM);
|
|
|
|
|
|
|
|
kn->flags |= KERNFS_EMPTY_DIR;
|
|
|
|
kn->dir.root = parent->dir.root;
|
|
|
|
kn->ns = NULL;
|
|
|
|
kn->priv = NULL;
|
|
|
|
|
|
|
|
/* link in */
|
|
|
|
rc = kernfs_add_one(kn);
|
|
|
|
if (!rc)
|
|
|
|
return kn;
|
|
|
|
|
|
|
|
kernfs_put(kn);
|
|
|
|
return ERR_PTR(rc);
|
|
|
|
}
|
|
|
|
|
2013-12-12 03:11:58 +08:00
|
|
|
static struct dentry *kernfs_iop_lookup(struct inode *dir,
|
|
|
|
struct dentry *dentry,
|
|
|
|
unsigned int flags)
|
2013-11-29 03:54:33 +08:00
|
|
|
{
|
2013-12-12 05:02:59 +08:00
|
|
|
struct dentry *ret;
|
2013-12-12 03:11:53 +08:00
|
|
|
struct kernfs_node *parent = dentry->d_parent->d_fsdata;
|
|
|
|
struct kernfs_node *kn;
|
2013-11-29 03:54:33 +08:00
|
|
|
struct inode *inode;
|
|
|
|
const void *ns = NULL;
|
|
|
|
|
2013-12-12 03:11:57 +08:00
|
|
|
mutex_lock(&kernfs_mutex);
|
2013-11-29 03:54:33 +08:00
|
|
|
|
2013-12-12 03:11:53 +08:00
|
|
|
if (kernfs_ns_enabled(parent))
|
2013-12-12 03:11:55 +08:00
|
|
|
ns = kernfs_info(dir->i_sb)->ns;
|
2013-11-29 03:54:33 +08:00
|
|
|
|
2013-12-12 03:11:53 +08:00
|
|
|
kn = kernfs_find_ns(parent, dentry->d_name.name, ns);
|
2013-11-29 03:54:33 +08:00
|
|
|
|
|
|
|
/* no such entry */
|
2014-02-04 03:09:11 +08:00
|
|
|
if (!kn || !kernfs_active(kn)) {
|
2013-12-12 05:02:59 +08:00
|
|
|
ret = NULL;
|
2013-11-29 03:54:33 +08:00
|
|
|
goto out_unlock;
|
|
|
|
}
|
2013-12-12 03:11:53 +08:00
|
|
|
kernfs_get(kn);
|
|
|
|
dentry->d_fsdata = kn;
|
2013-11-29 03:54:33 +08:00
|
|
|
|
|
|
|
/* attach dentry and inode */
|
2013-12-12 03:11:58 +08:00
|
|
|
inode = kernfs_get_inode(dir->i_sb, kn);
|
2013-11-29 03:54:33 +08:00
|
|
|
if (!inode) {
|
|
|
|
ret = ERR_PTR(-ENOMEM);
|
|
|
|
goto out_unlock;
|
|
|
|
}
|
|
|
|
|
|
|
|
/* instantiate and hash dentry */
|
2014-10-13 10:24:21 +08:00
|
|
|
ret = d_splice_alias(inode, dentry);
|
2013-11-29 03:54:33 +08:00
|
|
|
out_unlock:
|
2013-12-12 03:11:57 +08:00
|
|
|
mutex_unlock(&kernfs_mutex);
|
2013-11-29 03:54:33 +08:00
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
|
2013-12-12 05:03:00 +08:00
|
|
|
static int kernfs_iop_mkdir(struct inode *dir, struct dentry *dentry,
|
|
|
|
umode_t mode)
|
|
|
|
{
|
|
|
|
struct kernfs_node *parent = dir->i_private;
|
2014-02-04 03:09:09 +08:00
|
|
|
struct kernfs_syscall_ops *scops = kernfs_root(parent)->syscall_ops;
|
2014-02-04 03:09:08 +08:00
|
|
|
int ret;
|
2013-12-12 05:03:00 +08:00
|
|
|
|
2014-02-04 03:09:09 +08:00
|
|
|
if (!scops || !scops->mkdir)
|
2013-12-12 05:03:00 +08:00
|
|
|
return -EPERM;
|
|
|
|
|
2014-02-04 03:09:08 +08:00
|
|
|
if (!kernfs_get_active(parent))
|
|
|
|
return -ENODEV;
|
|
|
|
|
2014-02-04 03:09:09 +08:00
|
|
|
ret = scops->mkdir(parent, dentry->d_name.name, mode);
|
2014-02-04 03:09:08 +08:00
|
|
|
|
|
|
|
kernfs_put_active(parent);
|
|
|
|
return ret;
|
2013-12-12 05:03:00 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
static int kernfs_iop_rmdir(struct inode *dir, struct dentry *dentry)
|
|
|
|
{
|
|
|
|
struct kernfs_node *kn = dentry->d_fsdata;
|
2014-02-04 03:09:09 +08:00
|
|
|
struct kernfs_syscall_ops *scops = kernfs_root(kn)->syscall_ops;
|
2014-02-04 03:09:08 +08:00
|
|
|
int ret;
|
2013-12-12 05:03:00 +08:00
|
|
|
|
2014-02-04 03:09:09 +08:00
|
|
|
if (!scops || !scops->rmdir)
|
2013-12-12 05:03:00 +08:00
|
|
|
return -EPERM;
|
|
|
|
|
2014-02-04 03:09:08 +08:00
|
|
|
if (!kernfs_get_active(kn))
|
|
|
|
return -ENODEV;
|
|
|
|
|
2014-02-04 03:09:09 +08:00
|
|
|
ret = scops->rmdir(kn);
|
2014-02-04 03:09:08 +08:00
|
|
|
|
|
|
|
kernfs_put_active(kn);
|
|
|
|
return ret;
|
2013-12-12 05:03:00 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
static int kernfs_iop_rename(struct inode *old_dir, struct dentry *old_dentry,
|
|
|
|
struct inode *new_dir, struct dentry *new_dentry)
|
|
|
|
{
|
|
|
|
struct kernfs_node *kn = old_dentry->d_fsdata;
|
|
|
|
struct kernfs_node *new_parent = new_dir->i_private;
|
2014-02-04 03:09:09 +08:00
|
|
|
struct kernfs_syscall_ops *scops = kernfs_root(kn)->syscall_ops;
|
2014-02-04 03:09:08 +08:00
|
|
|
int ret;
|
2013-12-12 05:03:00 +08:00
|
|
|
|
2014-02-04 03:09:09 +08:00
|
|
|
if (!scops || !scops->rename)
|
2013-12-12 05:03:00 +08:00
|
|
|
return -EPERM;
|
|
|
|
|
2014-02-04 03:09:08 +08:00
|
|
|
if (!kernfs_get_active(kn))
|
|
|
|
return -ENODEV;
|
|
|
|
|
|
|
|
if (!kernfs_get_active(new_parent)) {
|
|
|
|
kernfs_put_active(kn);
|
|
|
|
return -ENODEV;
|
|
|
|
}
|
|
|
|
|
2014-02-04 03:09:09 +08:00
|
|
|
ret = scops->rename(kn, new_parent, new_dentry->d_name.name);
|
2014-02-04 03:09:08 +08:00
|
|
|
|
|
|
|
kernfs_put_active(new_parent);
|
|
|
|
kernfs_put_active(kn);
|
|
|
|
return ret;
|
2013-12-12 05:03:00 +08:00
|
|
|
}
|
|
|
|
|
2013-12-12 03:11:57 +08:00
|
|
|
const struct inode_operations kernfs_dir_iops = {
|
2013-12-12 03:11:58 +08:00
|
|
|
.lookup = kernfs_iop_lookup,
|
|
|
|
.permission = kernfs_iop_permission,
|
|
|
|
.setattr = kernfs_iop_setattr,
|
|
|
|
.getattr = kernfs_iop_getattr,
|
|
|
|
.setxattr = kernfs_iop_setxattr,
|
|
|
|
.removexattr = kernfs_iop_removexattr,
|
|
|
|
.getxattr = kernfs_iop_getxattr,
|
|
|
|
.listxattr = kernfs_iop_listxattr,
|
2013-12-12 05:03:00 +08:00
|
|
|
|
|
|
|
.mkdir = kernfs_iop_mkdir,
|
|
|
|
.rmdir = kernfs_iop_rmdir,
|
|
|
|
.rename = kernfs_iop_rename,
|
2013-11-29 03:54:33 +08:00
|
|
|
};
|
|
|
|
|
2013-12-12 03:11:58 +08:00
|
|
|
static struct kernfs_node *kernfs_leftmost_descendant(struct kernfs_node *pos)
|
2013-11-29 03:54:33 +08:00
|
|
|
{
|
2013-12-12 03:11:53 +08:00
|
|
|
struct kernfs_node *last;
|
2013-11-29 03:54:33 +08:00
|
|
|
|
|
|
|
while (true) {
|
|
|
|
struct rb_node *rbn;
|
|
|
|
|
|
|
|
last = pos;
|
|
|
|
|
2013-12-12 03:11:56 +08:00
|
|
|
if (kernfs_type(pos) != KERNFS_DIR)
|
2013-11-29 03:54:33 +08:00
|
|
|
break;
|
|
|
|
|
2013-12-12 03:11:54 +08:00
|
|
|
rbn = rb_first(&pos->dir.children);
|
2013-11-29 03:54:33 +08:00
|
|
|
if (!rbn)
|
|
|
|
break;
|
|
|
|
|
2013-12-12 03:11:53 +08:00
|
|
|
pos = rb_to_kn(rbn);
|
2013-11-29 03:54:33 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
return last;
|
|
|
|
}
|
|
|
|
|
|
|
|
/**
|
2013-12-12 03:11:58 +08:00
|
|
|
* kernfs_next_descendant_post - find the next descendant for post-order walk
|
2013-11-29 03:54:33 +08:00
|
|
|
* @pos: the current position (%NULL to initiate traversal)
|
2013-12-12 03:11:53 +08:00
|
|
|
* @root: kernfs_node whose descendants to walk
|
2013-11-29 03:54:33 +08:00
|
|
|
*
|
|
|
|
* Find the next descendant to visit for post-order traversal of @root's
|
|
|
|
* descendants. @root is included in the iteration and the last node to be
|
|
|
|
* visited.
|
|
|
|
*/
|
2013-12-12 03:11:58 +08:00
|
|
|
static struct kernfs_node *kernfs_next_descendant_post(struct kernfs_node *pos,
|
|
|
|
struct kernfs_node *root)
|
2013-11-29 03:54:33 +08:00
|
|
|
{
|
|
|
|
struct rb_node *rbn;
|
|
|
|
|
2013-12-12 03:11:57 +08:00
|
|
|
lockdep_assert_held(&kernfs_mutex);
|
2013-11-29 03:54:33 +08:00
|
|
|
|
|
|
|
/* if first iteration, visit leftmost descendant which may be root */
|
|
|
|
if (!pos)
|
2013-12-12 03:11:58 +08:00
|
|
|
return kernfs_leftmost_descendant(root);
|
2013-11-29 03:54:33 +08:00
|
|
|
|
|
|
|
/* if we visited @root, we're done */
|
|
|
|
if (pos == root)
|
|
|
|
return NULL;
|
|
|
|
|
|
|
|
/* if there's an unvisited sibling, visit its leftmost descendant */
|
2013-12-12 03:11:54 +08:00
|
|
|
rbn = rb_next(&pos->rb);
|
2013-11-29 03:54:33 +08:00
|
|
|
if (rbn)
|
2013-12-12 03:11:58 +08:00
|
|
|
return kernfs_leftmost_descendant(rb_to_kn(rbn));
|
2013-11-29 03:54:33 +08:00
|
|
|
|
|
|
|
/* no sibling left, visit parent */
|
2013-12-12 03:11:54 +08:00
|
|
|
return pos->parent;
|
2013-11-29 03:54:33 +08:00
|
|
|
}
|
|
|
|
|
2014-02-04 03:09:12 +08:00
|
|
|
/**
|
|
|
|
* kernfs_activate - activate a node which started deactivated
|
|
|
|
* @kn: kernfs_node whose subtree is to be activated
|
|
|
|
*
|
|
|
|
* If the root has KERNFS_ROOT_CREATE_DEACTIVATED set, a newly created node
|
|
|
|
* needs to be explicitly activated. A node which hasn't been activated
|
|
|
|
* isn't visible to userland and deactivation is skipped during its
|
|
|
|
* removal. This is useful to construct atomic init sequences where
|
|
|
|
* creation of multiple nodes should either succeed or fail atomically.
|
|
|
|
*
|
|
|
|
* The caller is responsible for ensuring that this function is not called
|
|
|
|
* after kernfs_remove*() is invoked on @kn.
|
|
|
|
*/
|
|
|
|
void kernfs_activate(struct kernfs_node *kn)
|
|
|
|
{
|
|
|
|
struct kernfs_node *pos;
|
|
|
|
|
|
|
|
mutex_lock(&kernfs_mutex);
|
|
|
|
|
|
|
|
pos = NULL;
|
|
|
|
while ((pos = kernfs_next_descendant_post(pos, kn))) {
|
|
|
|
if (!pos || (pos->flags & KERNFS_ACTIVATED))
|
|
|
|
continue;
|
|
|
|
|
|
|
|
WARN_ON_ONCE(pos->parent && RB_EMPTY_NODE(&pos->rb));
|
|
|
|
WARN_ON_ONCE(atomic_read(&pos->active) != KN_DEACTIVATED_BIAS);
|
|
|
|
|
|
|
|
atomic_sub(KN_DEACTIVATED_BIAS, &pos->active);
|
|
|
|
pos->flags |= KERNFS_ACTIVATED;
|
|
|
|
}
|
|
|
|
|
|
|
|
mutex_unlock(&kernfs_mutex);
|
|
|
|
}
|
|
|
|
|
2014-02-04 03:02:58 +08:00
|
|
|
static void __kernfs_remove(struct kernfs_node *kn)
|
2013-11-29 03:54:33 +08:00
|
|
|
{
|
kernfs: restructure removal path to fix possible premature return
The recursive nature of kernfs_remove() means that, even if
kernfs_remove() is not allowed to be called multiple times on the same
node, there may be race conditions between removal of parent and its
descendants. While we can claim that kernfs_remove() shouldn't be
called on one of the descendants while the removal of an ancestor is
in progress, such rule is unnecessarily restrictive and very difficult
to enforce. It's better to simply allow invoking kernfs_remove() as
the caller sees fit as long as the caller ensures that the node is
accessible.
The current behavior in such situations is broken. Whoever enters
removal path first takes the node off the hierarchy and then
deactivates. Following removers either return as soon as it notices
that it's not the first one or can't even find the target node as it
has already been removed from the hierarchy. In both cases, the
following removers may finish prematurely while the nodes which should
be removed and drained are still being processed by the first one.
This patch restructures so that multiple removers, whether through
recursion or direction invocation, always follow the following rules.
* When there are multiple concurrent removers, only one puts the base
ref.
* Regardless of which one puts the base ref, all removers are blocked
until the target node is fully deactivated and removed.
To achieve the above, removal path now first marks all descendants
including self REMOVED and then deactivates and unlinks leftmost
descendant one-by-one. kernfs_deactivate() is called directly from
__kernfs_removal() and drops and regrabs kernfs_mutex for each
descendant to drain active refs. As this means that multiple removers
can enter kernfs_deactivate() for the same node, the function is
updated so that it can handle multiple deactivators of the same node -
only one actually deactivates but all wait till drain completion.
The restructured removal path guarantees that a removed node gets
unlinked only after the node is deactivated and drained. Combined
with proper multiple deactivator handling, this guarantees that any
invocation of kernfs_remove() returns only after the node itself and
all its descendants are deactivated, drained and removed.
v2: Draining separated into a separate loop (used to be in the same
loop as unlink) and done from __kernfs_deactivate(). This is to
allow exposing deactivation as a separate interface later.
Root node removal was broken in v1 patch. Fixed.
v3: Revert most of v2 except for root node removal fix and
simplification of KERNFS_REMOVED setting loop.
v4: Refreshed on top of ("kernfs: make kernfs_deactivate() honor
KERNFS_LOCKDEP flag").
Signed-off-by: Tejun Heo <tj@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-02-04 03:02:56 +08:00
|
|
|
struct kernfs_node *pos;
|
|
|
|
|
|
|
|
lockdep_assert_held(&kernfs_mutex);
|
2013-11-29 03:54:33 +08:00
|
|
|
|
kernfs, sysfs, driver-core: implement kernfs_remove_self() and its wrappers
Sometimes it's necessary to implement a node which wants to delete
nodes including itself. This isn't straightforward because of kernfs
active reference. While a file operation is in progress, an active
reference is held and kernfs_remove() waits for all such references to
drain before completing. For a self-deleting node, this is a deadlock
as kernfs_remove() ends up waiting for an active reference that itself
is sitting on top of.
This currently is worked around in the sysfs layer using
sysfs_schedule_callback() which makes such removals asynchronous.
While it works, it's rather cumbersome and inherently breaks
synchronicity of the operation - the file operation which triggered
the operation may complete before the removal is finished (or even
started) and the removal may fail asynchronously. If a removal
operation is immmediately followed by another operation which expects
the specific name to be available (e.g. removal followed by rename
onto the same name), there's no way to make the latter operation
reliable.
The thing is there's no inherent reason for this to be asynchrnous.
All that's necessary to do this synchronous is a dedicated operation
which drops its own active ref and deactivates self. This patch
implements kernfs_remove_self() and its wrappers in sysfs and driver
core. kernfs_remove_self() is to be called from one of the file
operations, drops the active ref the task is holding, removes the self
node, and restores active ref to the dead node so that the ref is
balanced afterwards. __kernfs_remove() is updated so that it takes an
early exit if the target node is already fully removed so that the
active ref restored by kernfs_remove_self() after removal doesn't
confuse the deactivation path.
This makes implementing self-deleting nodes very easy. The normal
removal path doesn't even need to be changed to use
kernfs_remove_self() for the self-deleting node. The method can
invoke kernfs_remove_self() on itself before proceeding the normal
removal path. kernfs_remove() invoked on the node by the normal
deletion path will simply be ignored.
This will replace sysfs_schedule_callback(). A subtle feature of
sysfs_schedule_callback() is that it collapses multiple invocations -
even if multiple removals are triggered, the removal callback is run
only once. An equivalent effect can be achieved by testing the return
value of kernfs_remove_self() - only the one which gets %true return
value should proceed with actual deletion. All other instances of
kernfs_remove_self() will wait till the enclosing kernfs operation
which invoked the winning instance of kernfs_remove_self() finishes
and then return %false. This trivially makes all users of
kernfs_remove_self() automatically show correct synchronous behavior
even when there are multiple concurrent operations - all "echo 1 >
delete" instances will finish only after the whole operation is
completed by one of the instances.
Note that manipulation of active ref is implemented in separate public
functions - kernfs_[un]break_active_protection().
kernfs_remove_self() is the only user at the moment but this will be
used to cater to more complex cases.
v2: For !CONFIG_SYSFS, dummy version kernfs_remove_self() was missing
and sysfs_remove_file_self() had incorrect return type. Fix it.
Reported by kbuild test bot.
v3: kernfs_[un]break_active_protection() separated out from
kernfs_remove_self() and exposed as public API.
Signed-off-by: Tejun Heo <tj@kernel.org>
Cc: Alan Stern <stern@rowland.harvard.edu>
Cc: kbuild test robot <fengguang.wu@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-02-04 03:03:01 +08:00
|
|
|
/*
|
|
|
|
* Short-circuit if non-root @kn has already finished removal.
|
|
|
|
* This is for kernfs_remove_self() which plays with active ref
|
|
|
|
* after removal.
|
|
|
|
*/
|
|
|
|
if (!kn || (kn->parent && RB_EMPTY_NODE(&kn->rb)))
|
2014-01-14 05:50:31 +08:00
|
|
|
return;
|
|
|
|
|
2013-12-12 03:11:58 +08:00
|
|
|
pr_debug("kernfs %s: removing\n", kn->name);
|
2013-11-29 03:54:33 +08:00
|
|
|
|
2014-02-04 03:03:00 +08:00
|
|
|
/* prevent any new usage under @kn by deactivating all nodes */
|
kernfs: restructure removal path to fix possible premature return
The recursive nature of kernfs_remove() means that, even if
kernfs_remove() is not allowed to be called multiple times on the same
node, there may be race conditions between removal of parent and its
descendants. While we can claim that kernfs_remove() shouldn't be
called on one of the descendants while the removal of an ancestor is
in progress, such rule is unnecessarily restrictive and very difficult
to enforce. It's better to simply allow invoking kernfs_remove() as
the caller sees fit as long as the caller ensures that the node is
accessible.
The current behavior in such situations is broken. Whoever enters
removal path first takes the node off the hierarchy and then
deactivates. Following removers either return as soon as it notices
that it's not the first one or can't even find the target node as it
has already been removed from the hierarchy. In both cases, the
following removers may finish prematurely while the nodes which should
be removed and drained are still being processed by the first one.
This patch restructures so that multiple removers, whether through
recursion or direction invocation, always follow the following rules.
* When there are multiple concurrent removers, only one puts the base
ref.
* Regardless of which one puts the base ref, all removers are blocked
until the target node is fully deactivated and removed.
To achieve the above, removal path now first marks all descendants
including self REMOVED and then deactivates and unlinks leftmost
descendant one-by-one. kernfs_deactivate() is called directly from
__kernfs_removal() and drops and regrabs kernfs_mutex for each
descendant to drain active refs. As this means that multiple removers
can enter kernfs_deactivate() for the same node, the function is
updated so that it can handle multiple deactivators of the same node -
only one actually deactivates but all wait till drain completion.
The restructured removal path guarantees that a removed node gets
unlinked only after the node is deactivated and drained. Combined
with proper multiple deactivator handling, this guarantees that any
invocation of kernfs_remove() returns only after the node itself and
all its descendants are deactivated, drained and removed.
v2: Draining separated into a separate loop (used to be in the same
loop as unlink) and done from __kernfs_deactivate(). This is to
allow exposing deactivation as a separate interface later.
Root node removal was broken in v1 patch. Fixed.
v3: Revert most of v2 except for root node removal fix and
simplification of KERNFS_REMOVED setting loop.
v4: Refreshed on top of ("kernfs: make kernfs_deactivate() honor
KERNFS_LOCKDEP flag").
Signed-off-by: Tejun Heo <tj@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-02-04 03:02:56 +08:00
|
|
|
pos = NULL;
|
|
|
|
while ((pos = kernfs_next_descendant_post(pos, kn)))
|
2014-02-04 03:03:00 +08:00
|
|
|
if (kernfs_active(pos))
|
|
|
|
atomic_add(KN_DEACTIVATED_BIAS, &pos->active);
|
kernfs: restructure removal path to fix possible premature return
The recursive nature of kernfs_remove() means that, even if
kernfs_remove() is not allowed to be called multiple times on the same
node, there may be race conditions between removal of parent and its
descendants. While we can claim that kernfs_remove() shouldn't be
called on one of the descendants while the removal of an ancestor is
in progress, such rule is unnecessarily restrictive and very difficult
to enforce. It's better to simply allow invoking kernfs_remove() as
the caller sees fit as long as the caller ensures that the node is
accessible.
The current behavior in such situations is broken. Whoever enters
removal path first takes the node off the hierarchy and then
deactivates. Following removers either return as soon as it notices
that it's not the first one or can't even find the target node as it
has already been removed from the hierarchy. In both cases, the
following removers may finish prematurely while the nodes which should
be removed and drained are still being processed by the first one.
This patch restructures so that multiple removers, whether through
recursion or direction invocation, always follow the following rules.
* When there are multiple concurrent removers, only one puts the base
ref.
* Regardless of which one puts the base ref, all removers are blocked
until the target node is fully deactivated and removed.
To achieve the above, removal path now first marks all descendants
including self REMOVED and then deactivates and unlinks leftmost
descendant one-by-one. kernfs_deactivate() is called directly from
__kernfs_removal() and drops and regrabs kernfs_mutex for each
descendant to drain active refs. As this means that multiple removers
can enter kernfs_deactivate() for the same node, the function is
updated so that it can handle multiple deactivators of the same node -
only one actually deactivates but all wait till drain completion.
The restructured removal path guarantees that a removed node gets
unlinked only after the node is deactivated and drained. Combined
with proper multiple deactivator handling, this guarantees that any
invocation of kernfs_remove() returns only after the node itself and
all its descendants are deactivated, drained and removed.
v2: Draining separated into a separate loop (used to be in the same
loop as unlink) and done from __kernfs_deactivate(). This is to
allow exposing deactivation as a separate interface later.
Root node removal was broken in v1 patch. Fixed.
v3: Revert most of v2 except for root node removal fix and
simplification of KERNFS_REMOVED setting loop.
v4: Refreshed on top of ("kernfs: make kernfs_deactivate() honor
KERNFS_LOCKDEP flag").
Signed-off-by: Tejun Heo <tj@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-02-04 03:02:56 +08:00
|
|
|
|
|
|
|
/* deactivate and unlink the subtree node-by-node */
|
2013-11-29 03:54:33 +08:00
|
|
|
do {
|
kernfs: restructure removal path to fix possible premature return
The recursive nature of kernfs_remove() means that, even if
kernfs_remove() is not allowed to be called multiple times on the same
node, there may be race conditions between removal of parent and its
descendants. While we can claim that kernfs_remove() shouldn't be
called on one of the descendants while the removal of an ancestor is
in progress, such rule is unnecessarily restrictive and very difficult
to enforce. It's better to simply allow invoking kernfs_remove() as
the caller sees fit as long as the caller ensures that the node is
accessible.
The current behavior in such situations is broken. Whoever enters
removal path first takes the node off the hierarchy and then
deactivates. Following removers either return as soon as it notices
that it's not the first one or can't even find the target node as it
has already been removed from the hierarchy. In both cases, the
following removers may finish prematurely while the nodes which should
be removed and drained are still being processed by the first one.
This patch restructures so that multiple removers, whether through
recursion or direction invocation, always follow the following rules.
* When there are multiple concurrent removers, only one puts the base
ref.
* Regardless of which one puts the base ref, all removers are blocked
until the target node is fully deactivated and removed.
To achieve the above, removal path now first marks all descendants
including self REMOVED and then deactivates and unlinks leftmost
descendant one-by-one. kernfs_deactivate() is called directly from
__kernfs_removal() and drops and regrabs kernfs_mutex for each
descendant to drain active refs. As this means that multiple removers
can enter kernfs_deactivate() for the same node, the function is
updated so that it can handle multiple deactivators of the same node -
only one actually deactivates but all wait till drain completion.
The restructured removal path guarantees that a removed node gets
unlinked only after the node is deactivated and drained. Combined
with proper multiple deactivator handling, this guarantees that any
invocation of kernfs_remove() returns only after the node itself and
all its descendants are deactivated, drained and removed.
v2: Draining separated into a separate loop (used to be in the same
loop as unlink) and done from __kernfs_deactivate(). This is to
allow exposing deactivation as a separate interface later.
Root node removal was broken in v1 patch. Fixed.
v3: Revert most of v2 except for root node removal fix and
simplification of KERNFS_REMOVED setting loop.
v4: Refreshed on top of ("kernfs: make kernfs_deactivate() honor
KERNFS_LOCKDEP flag").
Signed-off-by: Tejun Heo <tj@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-02-04 03:02:56 +08:00
|
|
|
pos = kernfs_leftmost_descendant(kn);
|
|
|
|
|
|
|
|
/*
|
2014-02-04 03:03:00 +08:00
|
|
|
* kernfs_drain() drops kernfs_mutex temporarily and @pos's
|
|
|
|
* base ref could have been put by someone else by the time
|
|
|
|
* the function returns. Make sure it doesn't go away
|
|
|
|
* underneath us.
|
kernfs: restructure removal path to fix possible premature return
The recursive nature of kernfs_remove() means that, even if
kernfs_remove() is not allowed to be called multiple times on the same
node, there may be race conditions between removal of parent and its
descendants. While we can claim that kernfs_remove() shouldn't be
called on one of the descendants while the removal of an ancestor is
in progress, such rule is unnecessarily restrictive and very difficult
to enforce. It's better to simply allow invoking kernfs_remove() as
the caller sees fit as long as the caller ensures that the node is
accessible.
The current behavior in such situations is broken. Whoever enters
removal path first takes the node off the hierarchy and then
deactivates. Following removers either return as soon as it notices
that it's not the first one or can't even find the target node as it
has already been removed from the hierarchy. In both cases, the
following removers may finish prematurely while the nodes which should
be removed and drained are still being processed by the first one.
This patch restructures so that multiple removers, whether through
recursion or direction invocation, always follow the following rules.
* When there are multiple concurrent removers, only one puts the base
ref.
* Regardless of which one puts the base ref, all removers are blocked
until the target node is fully deactivated and removed.
To achieve the above, removal path now first marks all descendants
including self REMOVED and then deactivates and unlinks leftmost
descendant one-by-one. kernfs_deactivate() is called directly from
__kernfs_removal() and drops and regrabs kernfs_mutex for each
descendant to drain active refs. As this means that multiple removers
can enter kernfs_deactivate() for the same node, the function is
updated so that it can handle multiple deactivators of the same node -
only one actually deactivates but all wait till drain completion.
The restructured removal path guarantees that a removed node gets
unlinked only after the node is deactivated and drained. Combined
with proper multiple deactivator handling, this guarantees that any
invocation of kernfs_remove() returns only after the node itself and
all its descendants are deactivated, drained and removed.
v2: Draining separated into a separate loop (used to be in the same
loop as unlink) and done from __kernfs_deactivate(). This is to
allow exposing deactivation as a separate interface later.
Root node removal was broken in v1 patch. Fixed.
v3: Revert most of v2 except for root node removal fix and
simplification of KERNFS_REMOVED setting loop.
v4: Refreshed on top of ("kernfs: make kernfs_deactivate() honor
KERNFS_LOCKDEP flag").
Signed-off-by: Tejun Heo <tj@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-02-04 03:02:56 +08:00
|
|
|
*/
|
|
|
|
kernfs_get(pos);
|
|
|
|
|
2014-02-04 03:09:12 +08:00
|
|
|
/*
|
|
|
|
* Drain iff @kn was activated. This avoids draining and
|
|
|
|
* its lockdep annotations for nodes which have never been
|
|
|
|
* activated and allows embedding kernfs_remove() in create
|
|
|
|
* error paths without worrying about draining.
|
|
|
|
*/
|
|
|
|
if (kn->flags & KERNFS_ACTIVATED)
|
|
|
|
kernfs_drain(pos);
|
|
|
|
else
|
|
|
|
WARN_ON_ONCE(atomic_read(&kn->active) != KN_DEACTIVATED_BIAS);
|
kernfs: restructure removal path to fix possible premature return
The recursive nature of kernfs_remove() means that, even if
kernfs_remove() is not allowed to be called multiple times on the same
node, there may be race conditions between removal of parent and its
descendants. While we can claim that kernfs_remove() shouldn't be
called on one of the descendants while the removal of an ancestor is
in progress, such rule is unnecessarily restrictive and very difficult
to enforce. It's better to simply allow invoking kernfs_remove() as
the caller sees fit as long as the caller ensures that the node is
accessible.
The current behavior in such situations is broken. Whoever enters
removal path first takes the node off the hierarchy and then
deactivates. Following removers either return as soon as it notices
that it's not the first one or can't even find the target node as it
has already been removed from the hierarchy. In both cases, the
following removers may finish prematurely while the nodes which should
be removed and drained are still being processed by the first one.
This patch restructures so that multiple removers, whether through
recursion or direction invocation, always follow the following rules.
* When there are multiple concurrent removers, only one puts the base
ref.
* Regardless of which one puts the base ref, all removers are blocked
until the target node is fully deactivated and removed.
To achieve the above, removal path now first marks all descendants
including self REMOVED and then deactivates and unlinks leftmost
descendant one-by-one. kernfs_deactivate() is called directly from
__kernfs_removal() and drops and regrabs kernfs_mutex for each
descendant to drain active refs. As this means that multiple removers
can enter kernfs_deactivate() for the same node, the function is
updated so that it can handle multiple deactivators of the same node -
only one actually deactivates but all wait till drain completion.
The restructured removal path guarantees that a removed node gets
unlinked only after the node is deactivated and drained. Combined
with proper multiple deactivator handling, this guarantees that any
invocation of kernfs_remove() returns only after the node itself and
all its descendants are deactivated, drained and removed.
v2: Draining separated into a separate loop (used to be in the same
loop as unlink) and done from __kernfs_deactivate(). This is to
allow exposing deactivation as a separate interface later.
Root node removal was broken in v1 patch. Fixed.
v3: Revert most of v2 except for root node removal fix and
simplification of KERNFS_REMOVED setting loop.
v4: Refreshed on top of ("kernfs: make kernfs_deactivate() honor
KERNFS_LOCKDEP flag").
Signed-off-by: Tejun Heo <tj@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-02-04 03:02:56 +08:00
|
|
|
|
|
|
|
/*
|
|
|
|
* kernfs_unlink_sibling() succeeds once per node. Use it
|
|
|
|
* to decide who's responsible for cleanups.
|
|
|
|
*/
|
|
|
|
if (!pos->parent || kernfs_unlink_sibling(pos)) {
|
|
|
|
struct kernfs_iattrs *ps_iattr =
|
|
|
|
pos->parent ? pos->parent->iattr : NULL;
|
|
|
|
|
|
|
|
/* update timestamps on the parent */
|
|
|
|
if (ps_iattr) {
|
|
|
|
ps_iattr->ia_iattr.ia_ctime = CURRENT_TIME;
|
|
|
|
ps_iattr->ia_iattr.ia_mtime = CURRENT_TIME;
|
|
|
|
}
|
|
|
|
|
2014-02-04 03:02:58 +08:00
|
|
|
kernfs_put(pos);
|
kernfs: restructure removal path to fix possible premature return
The recursive nature of kernfs_remove() means that, even if
kernfs_remove() is not allowed to be called multiple times on the same
node, there may be race conditions between removal of parent and its
descendants. While we can claim that kernfs_remove() shouldn't be
called on one of the descendants while the removal of an ancestor is
in progress, such rule is unnecessarily restrictive and very difficult
to enforce. It's better to simply allow invoking kernfs_remove() as
the caller sees fit as long as the caller ensures that the node is
accessible.
The current behavior in such situations is broken. Whoever enters
removal path first takes the node off the hierarchy and then
deactivates. Following removers either return as soon as it notices
that it's not the first one or can't even find the target node as it
has already been removed from the hierarchy. In both cases, the
following removers may finish prematurely while the nodes which should
be removed and drained are still being processed by the first one.
This patch restructures so that multiple removers, whether through
recursion or direction invocation, always follow the following rules.
* When there are multiple concurrent removers, only one puts the base
ref.
* Regardless of which one puts the base ref, all removers are blocked
until the target node is fully deactivated and removed.
To achieve the above, removal path now first marks all descendants
including self REMOVED and then deactivates and unlinks leftmost
descendant one-by-one. kernfs_deactivate() is called directly from
__kernfs_removal() and drops and regrabs kernfs_mutex for each
descendant to drain active refs. As this means that multiple removers
can enter kernfs_deactivate() for the same node, the function is
updated so that it can handle multiple deactivators of the same node -
only one actually deactivates but all wait till drain completion.
The restructured removal path guarantees that a removed node gets
unlinked only after the node is deactivated and drained. Combined
with proper multiple deactivator handling, this guarantees that any
invocation of kernfs_remove() returns only after the node itself and
all its descendants are deactivated, drained and removed.
v2: Draining separated into a separate loop (used to be in the same
loop as unlink) and done from __kernfs_deactivate(). This is to
allow exposing deactivation as a separate interface later.
Root node removal was broken in v1 patch. Fixed.
v3: Revert most of v2 except for root node removal fix and
simplification of KERNFS_REMOVED setting loop.
v4: Refreshed on top of ("kernfs: make kernfs_deactivate() honor
KERNFS_LOCKDEP flag").
Signed-off-by: Tejun Heo <tj@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-02-04 03:02:56 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
kernfs_put(pos);
|
|
|
|
} while (pos != kn);
|
2013-11-29 03:54:33 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
/**
|
2013-12-12 03:11:53 +08:00
|
|
|
* kernfs_remove - remove a kernfs_node recursively
|
|
|
|
* @kn: the kernfs_node to remove
|
2013-11-29 03:54:33 +08:00
|
|
|
*
|
2013-12-12 03:11:53 +08:00
|
|
|
* Remove @kn along with all its subdirectories and files.
|
2013-11-29 03:54:33 +08:00
|
|
|
*/
|
2013-12-12 03:11:53 +08:00
|
|
|
void kernfs_remove(struct kernfs_node *kn)
|
2013-11-29 03:54:33 +08:00
|
|
|
{
|
2014-02-04 03:02:58 +08:00
|
|
|
mutex_lock(&kernfs_mutex);
|
|
|
|
__kernfs_remove(kn);
|
|
|
|
mutex_unlock(&kernfs_mutex);
|
2013-11-29 03:54:33 +08:00
|
|
|
}
|
|
|
|
|
kernfs, sysfs, driver-core: implement kernfs_remove_self() and its wrappers
Sometimes it's necessary to implement a node which wants to delete
nodes including itself. This isn't straightforward because of kernfs
active reference. While a file operation is in progress, an active
reference is held and kernfs_remove() waits for all such references to
drain before completing. For a self-deleting node, this is a deadlock
as kernfs_remove() ends up waiting for an active reference that itself
is sitting on top of.
This currently is worked around in the sysfs layer using
sysfs_schedule_callback() which makes such removals asynchronous.
While it works, it's rather cumbersome and inherently breaks
synchronicity of the operation - the file operation which triggered
the operation may complete before the removal is finished (or even
started) and the removal may fail asynchronously. If a removal
operation is immmediately followed by another operation which expects
the specific name to be available (e.g. removal followed by rename
onto the same name), there's no way to make the latter operation
reliable.
The thing is there's no inherent reason for this to be asynchrnous.
All that's necessary to do this synchronous is a dedicated operation
which drops its own active ref and deactivates self. This patch
implements kernfs_remove_self() and its wrappers in sysfs and driver
core. kernfs_remove_self() is to be called from one of the file
operations, drops the active ref the task is holding, removes the self
node, and restores active ref to the dead node so that the ref is
balanced afterwards. __kernfs_remove() is updated so that it takes an
early exit if the target node is already fully removed so that the
active ref restored by kernfs_remove_self() after removal doesn't
confuse the deactivation path.
This makes implementing self-deleting nodes very easy. The normal
removal path doesn't even need to be changed to use
kernfs_remove_self() for the self-deleting node. The method can
invoke kernfs_remove_self() on itself before proceeding the normal
removal path. kernfs_remove() invoked on the node by the normal
deletion path will simply be ignored.
This will replace sysfs_schedule_callback(). A subtle feature of
sysfs_schedule_callback() is that it collapses multiple invocations -
even if multiple removals are triggered, the removal callback is run
only once. An equivalent effect can be achieved by testing the return
value of kernfs_remove_self() - only the one which gets %true return
value should proceed with actual deletion. All other instances of
kernfs_remove_self() will wait till the enclosing kernfs operation
which invoked the winning instance of kernfs_remove_self() finishes
and then return %false. This trivially makes all users of
kernfs_remove_self() automatically show correct synchronous behavior
even when there are multiple concurrent operations - all "echo 1 >
delete" instances will finish only after the whole operation is
completed by one of the instances.
Note that manipulation of active ref is implemented in separate public
functions - kernfs_[un]break_active_protection().
kernfs_remove_self() is the only user at the moment but this will be
used to cater to more complex cases.
v2: For !CONFIG_SYSFS, dummy version kernfs_remove_self() was missing
and sysfs_remove_file_self() had incorrect return type. Fix it.
Reported by kbuild test bot.
v3: kernfs_[un]break_active_protection() separated out from
kernfs_remove_self() and exposed as public API.
Signed-off-by: Tejun Heo <tj@kernel.org>
Cc: Alan Stern <stern@rowland.harvard.edu>
Cc: kbuild test robot <fengguang.wu@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-02-04 03:03:01 +08:00
|
|
|
/**
|
|
|
|
* kernfs_break_active_protection - break out of active protection
|
|
|
|
* @kn: the self kernfs_node
|
|
|
|
*
|
|
|
|
* The caller must be running off of a kernfs operation which is invoked
|
|
|
|
* with an active reference - e.g. one of kernfs_ops. Each invocation of
|
|
|
|
* this function must also be matched with an invocation of
|
|
|
|
* kernfs_unbreak_active_protection().
|
|
|
|
*
|
|
|
|
* This function releases the active reference of @kn the caller is
|
|
|
|
* holding. Once this function is called, @kn may be removed at any point
|
|
|
|
* and the caller is solely responsible for ensuring that the objects it
|
|
|
|
* dereferences are accessible.
|
|
|
|
*/
|
|
|
|
void kernfs_break_active_protection(struct kernfs_node *kn)
|
|
|
|
{
|
|
|
|
/*
|
|
|
|
* Take out ourself out of the active ref dependency chain. If
|
|
|
|
* we're called without an active ref, lockdep will complain.
|
|
|
|
*/
|
|
|
|
kernfs_put_active(kn);
|
|
|
|
}
|
|
|
|
|
|
|
|
/**
|
|
|
|
* kernfs_unbreak_active_protection - undo kernfs_break_active_protection()
|
|
|
|
* @kn: the self kernfs_node
|
|
|
|
*
|
|
|
|
* If kernfs_break_active_protection() was called, this function must be
|
|
|
|
* invoked before finishing the kernfs operation. Note that while this
|
|
|
|
* function restores the active reference, it doesn't and can't actually
|
|
|
|
* restore the active protection - @kn may already or be in the process of
|
|
|
|
* being removed. Once kernfs_break_active_protection() is invoked, that
|
|
|
|
* protection is irreversibly gone for the kernfs operation instance.
|
|
|
|
*
|
|
|
|
* While this function may be called at any point after
|
|
|
|
* kernfs_break_active_protection() is invoked, its most useful location
|
|
|
|
* would be right before the enclosing kernfs operation returns.
|
|
|
|
*/
|
|
|
|
void kernfs_unbreak_active_protection(struct kernfs_node *kn)
|
|
|
|
{
|
|
|
|
/*
|
|
|
|
* @kn->active could be in any state; however, the increment we do
|
|
|
|
* here will be undone as soon as the enclosing kernfs operation
|
|
|
|
* finishes and this temporary bump can't break anything. If @kn
|
|
|
|
* is alive, nothing changes. If @kn is being deactivated, the
|
|
|
|
* soon-to-follow put will either finish deactivation or restore
|
|
|
|
* deactivated state. If @kn is already removed, the temporary
|
|
|
|
* bump is guaranteed to be gone before @kn is released.
|
|
|
|
*/
|
|
|
|
atomic_inc(&kn->active);
|
|
|
|
if (kernfs_lockdep(kn))
|
|
|
|
rwsem_acquire(&kn->dep_map, 0, 1, _RET_IP_);
|
|
|
|
}
|
|
|
|
|
|
|
|
/**
|
|
|
|
* kernfs_remove_self - remove a kernfs_node from its own method
|
|
|
|
* @kn: the self kernfs_node to remove
|
|
|
|
*
|
|
|
|
* The caller must be running off of a kernfs operation which is invoked
|
|
|
|
* with an active reference - e.g. one of kernfs_ops. This can be used to
|
|
|
|
* implement a file operation which deletes itself.
|
|
|
|
*
|
|
|
|
* For example, the "delete" file for a sysfs device directory can be
|
|
|
|
* implemented by invoking kernfs_remove_self() on the "delete" file
|
|
|
|
* itself. This function breaks the circular dependency of trying to
|
|
|
|
* deactivate self while holding an active ref itself. It isn't necessary
|
|
|
|
* to modify the usual removal path to use kernfs_remove_self(). The
|
|
|
|
* "delete" implementation can simply invoke kernfs_remove_self() on self
|
|
|
|
* before proceeding with the usual removal path. kernfs will ignore later
|
|
|
|
* kernfs_remove() on self.
|
|
|
|
*
|
|
|
|
* kernfs_remove_self() can be called multiple times concurrently on the
|
|
|
|
* same kernfs_node. Only the first one actually performs removal and
|
|
|
|
* returns %true. All others will wait until the kernfs operation which
|
|
|
|
* won self-removal finishes and return %false. Note that the losers wait
|
|
|
|
* for the completion of not only the winning kernfs_remove_self() but also
|
|
|
|
* the whole kernfs_ops which won the arbitration. This can be used to
|
|
|
|
* guarantee, for example, all concurrent writes to a "delete" file to
|
|
|
|
* finish only after the whole operation is complete.
|
|
|
|
*/
|
|
|
|
bool kernfs_remove_self(struct kernfs_node *kn)
|
|
|
|
{
|
|
|
|
bool ret;
|
|
|
|
|
|
|
|
mutex_lock(&kernfs_mutex);
|
|
|
|
kernfs_break_active_protection(kn);
|
|
|
|
|
|
|
|
/*
|
|
|
|
* SUICIDAL is used to arbitrate among competing invocations. Only
|
|
|
|
* the first one will actually perform removal. When the removal
|
|
|
|
* is complete, SUICIDED is set and the active ref is restored
|
|
|
|
* while holding kernfs_mutex. The ones which lost arbitration
|
|
|
|
* waits for SUICDED && drained which can happen only after the
|
|
|
|
* enclosing kernfs operation which executed the winning instance
|
|
|
|
* of kernfs_remove_self() finished.
|
|
|
|
*/
|
|
|
|
if (!(kn->flags & KERNFS_SUICIDAL)) {
|
|
|
|
kn->flags |= KERNFS_SUICIDAL;
|
|
|
|
__kernfs_remove(kn);
|
|
|
|
kn->flags |= KERNFS_SUICIDED;
|
|
|
|
ret = true;
|
|
|
|
} else {
|
|
|
|
wait_queue_head_t *waitq = &kernfs_root(kn)->deactivate_waitq;
|
|
|
|
DEFINE_WAIT(wait);
|
|
|
|
|
|
|
|
while (true) {
|
|
|
|
prepare_to_wait(waitq, &wait, TASK_UNINTERRUPTIBLE);
|
|
|
|
|
|
|
|
if ((kn->flags & KERNFS_SUICIDED) &&
|
|
|
|
atomic_read(&kn->active) == KN_DEACTIVATED_BIAS)
|
|
|
|
break;
|
|
|
|
|
|
|
|
mutex_unlock(&kernfs_mutex);
|
|
|
|
schedule();
|
|
|
|
mutex_lock(&kernfs_mutex);
|
|
|
|
}
|
|
|
|
finish_wait(waitq, &wait);
|
|
|
|
WARN_ON_ONCE(!RB_EMPTY_NODE(&kn->rb));
|
|
|
|
ret = false;
|
|
|
|
}
|
|
|
|
|
|
|
|
/*
|
|
|
|
* This must be done while holding kernfs_mutex; otherwise, waiting
|
|
|
|
* for SUICIDED && deactivated could finish prematurely.
|
|
|
|
*/
|
|
|
|
kernfs_unbreak_active_protection(kn);
|
|
|
|
|
|
|
|
mutex_unlock(&kernfs_mutex);
|
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
|
2013-11-29 03:54:33 +08:00
|
|
|
/**
|
2013-12-12 03:11:53 +08:00
|
|
|
* kernfs_remove_by_name_ns - find a kernfs_node by name and remove it
|
|
|
|
* @parent: parent of the target
|
|
|
|
* @name: name of the kernfs_node to remove
|
|
|
|
* @ns: namespace tag of the kernfs_node to remove
|
2013-11-29 03:54:33 +08:00
|
|
|
*
|
2013-12-12 03:11:53 +08:00
|
|
|
* Look for the kernfs_node with @name and @ns under @parent and remove it.
|
|
|
|
* Returns 0 on success, -ENOENT if such entry doesn't exist.
|
2013-11-29 03:54:33 +08:00
|
|
|
*/
|
2013-12-12 03:11:53 +08:00
|
|
|
int kernfs_remove_by_name_ns(struct kernfs_node *parent, const char *name,
|
2013-11-29 03:54:33 +08:00
|
|
|
const void *ns)
|
|
|
|
{
|
2013-12-12 03:11:53 +08:00
|
|
|
struct kernfs_node *kn;
|
2013-11-29 03:54:33 +08:00
|
|
|
|
2013-12-12 03:11:53 +08:00
|
|
|
if (!parent) {
|
2013-12-12 03:11:58 +08:00
|
|
|
WARN(1, KERN_WARNING "kernfs: can not remove '%s', no directory\n",
|
2013-11-29 03:54:33 +08:00
|
|
|
name);
|
|
|
|
return -ENOENT;
|
|
|
|
}
|
|
|
|
|
2014-02-04 03:02:58 +08:00
|
|
|
mutex_lock(&kernfs_mutex);
|
2013-11-29 03:54:33 +08:00
|
|
|
|
2013-12-12 03:11:53 +08:00
|
|
|
kn = kernfs_find_ns(parent, name, ns);
|
|
|
|
if (kn)
|
2014-02-04 03:02:58 +08:00
|
|
|
__kernfs_remove(kn);
|
2013-11-29 03:54:33 +08:00
|
|
|
|
2014-02-04 03:02:58 +08:00
|
|
|
mutex_unlock(&kernfs_mutex);
|
2013-11-29 03:54:33 +08:00
|
|
|
|
2013-12-12 03:11:53 +08:00
|
|
|
if (kn)
|
2013-11-29 03:54:33 +08:00
|
|
|
return 0;
|
|
|
|
else
|
|
|
|
return -ENOENT;
|
|
|
|
}
|
|
|
|
|
|
|
|
/**
|
|
|
|
* kernfs_rename_ns - move and rename a kernfs_node
|
2013-12-12 03:11:53 +08:00
|
|
|
* @kn: target node
|
2013-11-29 03:54:33 +08:00
|
|
|
* @new_parent: new parent to put @sd under
|
|
|
|
* @new_name: new name
|
|
|
|
* @new_ns: new namespace tag
|
|
|
|
*/
|
2013-12-12 03:11:53 +08:00
|
|
|
int kernfs_rename_ns(struct kernfs_node *kn, struct kernfs_node *new_parent,
|
2013-11-29 03:54:33 +08:00
|
|
|
const char *new_name, const void *new_ns)
|
|
|
|
{
|
2014-02-08 02:32:07 +08:00
|
|
|
struct kernfs_node *old_parent;
|
|
|
|
const char *old_name = NULL;
|
2013-11-29 03:54:33 +08:00
|
|
|
int error;
|
|
|
|
|
2014-02-08 02:32:07 +08:00
|
|
|
/* can't move or rename root */
|
|
|
|
if (!kn->parent)
|
|
|
|
return -EINVAL;
|
|
|
|
|
2014-01-14 06:36:03 +08:00
|
|
|
mutex_lock(&kernfs_mutex);
|
|
|
|
|
2013-12-12 05:02:56 +08:00
|
|
|
error = -ENOENT;
|
2015-05-14 05:09:29 +08:00
|
|
|
if (!kernfs_active(kn) || !kernfs_active(new_parent) ||
|
|
|
|
(new_parent->flags & KERNFS_EMPTY_DIR))
|
2013-12-12 05:02:56 +08:00
|
|
|
goto out;
|
|
|
|
|
2013-11-29 03:54:33 +08:00
|
|
|
error = 0;
|
2013-12-12 03:11:54 +08:00
|
|
|
if ((kn->parent == new_parent) && (kn->ns == new_ns) &&
|
|
|
|
(strcmp(kn->name, new_name) == 0))
|
2014-01-14 06:36:03 +08:00
|
|
|
goto out; /* nothing to rename */
|
2013-11-29 03:54:33 +08:00
|
|
|
|
|
|
|
error = -EEXIST;
|
|
|
|
if (kernfs_find_ns(new_parent, new_name, new_ns))
|
2014-01-14 06:36:03 +08:00
|
|
|
goto out;
|
2013-11-29 03:54:33 +08:00
|
|
|
|
2013-12-12 03:11:53 +08:00
|
|
|
/* rename kernfs_node */
|
2013-12-12 03:11:54 +08:00
|
|
|
if (strcmp(kn->name, new_name) != 0) {
|
2013-11-29 03:54:33 +08:00
|
|
|
error = -ENOMEM;
|
2015-02-14 06:36:27 +08:00
|
|
|
new_name = kstrdup_const(new_name, GFP_KERNEL);
|
2013-11-29 03:54:33 +08:00
|
|
|
if (!new_name)
|
2014-01-14 06:36:03 +08:00
|
|
|
goto out;
|
2014-02-08 02:32:07 +08:00
|
|
|
} else {
|
|
|
|
new_name = NULL;
|
2013-11-29 03:54:33 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
/*
|
|
|
|
* Move to the appropriate place in the appropriate directories rbtree.
|
|
|
|
*/
|
2013-12-12 03:11:58 +08:00
|
|
|
kernfs_unlink_sibling(kn);
|
2013-11-29 03:54:33 +08:00
|
|
|
kernfs_get(new_parent);
|
2014-02-08 02:32:07 +08:00
|
|
|
|
|
|
|
/* rename_lock protects ->parent and ->name accessors */
|
|
|
|
spin_lock_irq(&kernfs_rename_lock);
|
|
|
|
|
|
|
|
old_parent = kn->parent;
|
2013-12-12 03:11:54 +08:00
|
|
|
kn->parent = new_parent;
|
2014-02-08 02:32:07 +08:00
|
|
|
|
|
|
|
kn->ns = new_ns;
|
|
|
|
if (new_name) {
|
2015-02-14 06:36:31 +08:00
|
|
|
old_name = kn->name;
|
2014-02-08 02:32:07 +08:00
|
|
|
kn->name = new_name;
|
|
|
|
}
|
|
|
|
|
|
|
|
spin_unlock_irq(&kernfs_rename_lock);
|
|
|
|
|
2014-02-11 06:57:09 +08:00
|
|
|
kn->hash = kernfs_name_hash(kn->name, kn->ns);
|
2013-12-12 03:11:58 +08:00
|
|
|
kernfs_link_sibling(kn);
|
2013-11-29 03:54:33 +08:00
|
|
|
|
2014-02-08 02:32:07 +08:00
|
|
|
kernfs_put(old_parent);
|
2015-02-14 06:36:27 +08:00
|
|
|
kfree_const(old_name);
|
2014-02-08 02:32:07 +08:00
|
|
|
|
2013-11-29 03:54:33 +08:00
|
|
|
error = 0;
|
2014-01-14 06:36:03 +08:00
|
|
|
out:
|
2013-12-12 03:11:57 +08:00
|
|
|
mutex_unlock(&kernfs_mutex);
|
2013-11-29 03:54:33 +08:00
|
|
|
return error;
|
|
|
|
}
|
|
|
|
|
|
|
|
/* Relationship between s_mode and the DT_xxx types */
|
2013-12-12 03:11:53 +08:00
|
|
|
static inline unsigned char dt_type(struct kernfs_node *kn)
|
2013-11-29 03:54:33 +08:00
|
|
|
{
|
2013-12-12 03:11:54 +08:00
|
|
|
return (kn->mode >> 12) & 15;
|
2013-11-29 03:54:33 +08:00
|
|
|
}
|
|
|
|
|
2013-12-12 03:11:58 +08:00
|
|
|
static int kernfs_dir_fop_release(struct inode *inode, struct file *filp)
|
2013-11-29 03:54:33 +08:00
|
|
|
{
|
|
|
|
kernfs_put(filp->private_data);
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
2013-12-12 03:11:58 +08:00
|
|
|
static struct kernfs_node *kernfs_dir_pos(const void *ns,
|
2013-12-12 03:11:53 +08:00
|
|
|
struct kernfs_node *parent, loff_t hash, struct kernfs_node *pos)
|
2013-11-29 03:54:33 +08:00
|
|
|
{
|
|
|
|
if (pos) {
|
2014-02-04 03:03:00 +08:00
|
|
|
int valid = kernfs_active(pos) &&
|
2014-01-14 06:36:03 +08:00
|
|
|
pos->parent == parent && hash == pos->hash;
|
2013-11-29 03:54:33 +08:00
|
|
|
kernfs_put(pos);
|
|
|
|
if (!valid)
|
|
|
|
pos = NULL;
|
|
|
|
}
|
|
|
|
if (!pos && (hash > 1) && (hash < INT_MAX)) {
|
2013-12-12 03:11:54 +08:00
|
|
|
struct rb_node *node = parent->dir.children.rb_node;
|
2013-11-29 03:54:33 +08:00
|
|
|
while (node) {
|
2013-12-12 03:11:53 +08:00
|
|
|
pos = rb_to_kn(node);
|
2013-11-29 03:54:33 +08:00
|
|
|
|
2013-12-12 03:11:54 +08:00
|
|
|
if (hash < pos->hash)
|
2013-11-29 03:54:33 +08:00
|
|
|
node = node->rb_left;
|
2013-12-12 03:11:54 +08:00
|
|
|
else if (hash > pos->hash)
|
2013-11-29 03:54:33 +08:00
|
|
|
node = node->rb_right;
|
|
|
|
else
|
|
|
|
break;
|
|
|
|
}
|
|
|
|
}
|
2014-02-04 03:09:11 +08:00
|
|
|
/* Skip over entries which are dying/dead or in the wrong namespace */
|
|
|
|
while (pos && (!kernfs_active(pos) || pos->ns != ns)) {
|
2013-12-12 03:11:54 +08:00
|
|
|
struct rb_node *node = rb_next(&pos->rb);
|
2013-11-29 03:54:33 +08:00
|
|
|
if (!node)
|
|
|
|
pos = NULL;
|
|
|
|
else
|
2013-12-12 03:11:53 +08:00
|
|
|
pos = rb_to_kn(node);
|
2013-11-29 03:54:33 +08:00
|
|
|
}
|
|
|
|
return pos;
|
|
|
|
}
|
|
|
|
|
2013-12-12 03:11:58 +08:00
|
|
|
static struct kernfs_node *kernfs_dir_next_pos(const void *ns,
|
2013-12-12 03:11:53 +08:00
|
|
|
struct kernfs_node *parent, ino_t ino, struct kernfs_node *pos)
|
2013-11-29 03:54:33 +08:00
|
|
|
{
|
2013-12-12 03:11:58 +08:00
|
|
|
pos = kernfs_dir_pos(ns, parent, ino, pos);
|
2014-02-04 03:09:11 +08:00
|
|
|
if (pos) {
|
2013-11-29 03:54:33 +08:00
|
|
|
do {
|
2013-12-12 03:11:54 +08:00
|
|
|
struct rb_node *node = rb_next(&pos->rb);
|
2013-11-29 03:54:33 +08:00
|
|
|
if (!node)
|
|
|
|
pos = NULL;
|
|
|
|
else
|
2013-12-12 03:11:53 +08:00
|
|
|
pos = rb_to_kn(node);
|
2014-02-04 03:09:11 +08:00
|
|
|
} while (pos && (!kernfs_active(pos) || pos->ns != ns));
|
|
|
|
}
|
2013-11-29 03:54:33 +08:00
|
|
|
return pos;
|
|
|
|
}
|
|
|
|
|
2013-12-12 03:11:58 +08:00
|
|
|
static int kernfs_fop_readdir(struct file *file, struct dir_context *ctx)
|
2013-11-29 03:54:33 +08:00
|
|
|
{
|
|
|
|
struct dentry *dentry = file->f_path.dentry;
|
2013-12-12 03:11:53 +08:00
|
|
|
struct kernfs_node *parent = dentry->d_fsdata;
|
|
|
|
struct kernfs_node *pos = file->private_data;
|
2013-11-29 03:54:33 +08:00
|
|
|
const void *ns = NULL;
|
|
|
|
|
|
|
|
if (!dir_emit_dots(file, ctx))
|
|
|
|
return 0;
|
2013-12-12 03:11:57 +08:00
|
|
|
mutex_lock(&kernfs_mutex);
|
2013-11-29 03:54:33 +08:00
|
|
|
|
2013-12-12 03:11:53 +08:00
|
|
|
if (kernfs_ns_enabled(parent))
|
2013-12-12 03:11:55 +08:00
|
|
|
ns = kernfs_info(dentry->d_sb)->ns;
|
2013-11-29 03:54:33 +08:00
|
|
|
|
2013-12-12 03:11:58 +08:00
|
|
|
for (pos = kernfs_dir_pos(ns, parent, ctx->pos, pos);
|
2013-11-29 03:54:33 +08:00
|
|
|
pos;
|
2013-12-12 03:11:58 +08:00
|
|
|
pos = kernfs_dir_next_pos(ns, parent, ctx->pos, pos)) {
|
2013-12-12 03:11:54 +08:00
|
|
|
const char *name = pos->name;
|
2013-11-29 03:54:33 +08:00
|
|
|
unsigned int type = dt_type(pos);
|
|
|
|
int len = strlen(name);
|
2013-12-12 03:11:54 +08:00
|
|
|
ino_t ino = pos->ino;
|
2013-11-29 03:54:33 +08:00
|
|
|
|
2013-12-12 03:11:54 +08:00
|
|
|
ctx->pos = pos->hash;
|
2013-11-29 03:54:33 +08:00
|
|
|
file->private_data = pos;
|
|
|
|
kernfs_get(pos);
|
|
|
|
|
2013-12-12 03:11:57 +08:00
|
|
|
mutex_unlock(&kernfs_mutex);
|
2013-11-29 03:54:33 +08:00
|
|
|
if (!dir_emit(ctx, name, len, ino, type))
|
|
|
|
return 0;
|
2013-12-12 03:11:57 +08:00
|
|
|
mutex_lock(&kernfs_mutex);
|
2013-11-29 03:54:33 +08:00
|
|
|
}
|
2013-12-12 03:11:57 +08:00
|
|
|
mutex_unlock(&kernfs_mutex);
|
2013-11-29 03:54:33 +08:00
|
|
|
file->private_data = NULL;
|
|
|
|
ctx->pos = INT_MAX;
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
2013-12-12 03:11:58 +08:00
|
|
|
static loff_t kernfs_dir_fop_llseek(struct file *file, loff_t offset,
|
|
|
|
int whence)
|
2013-11-29 03:54:33 +08:00
|
|
|
{
|
|
|
|
struct inode *inode = file_inode(file);
|
|
|
|
loff_t ret;
|
|
|
|
|
|
|
|
mutex_lock(&inode->i_mutex);
|
|
|
|
ret = generic_file_llseek(file, offset, whence);
|
|
|
|
mutex_unlock(&inode->i_mutex);
|
|
|
|
|
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
|
2013-12-12 03:11:57 +08:00
|
|
|
const struct file_operations kernfs_dir_fops = {
|
2013-11-29 03:54:33 +08:00
|
|
|
.read = generic_read_dir,
|
2013-12-12 03:11:58 +08:00
|
|
|
.iterate = kernfs_fop_readdir,
|
|
|
|
.release = kernfs_dir_fop_release,
|
|
|
|
.llseek = kernfs_dir_fop_llseek,
|
2013-11-29 03:54:33 +08:00
|
|
|
};
|