Commit a1212d27 authored by Linus Torvalds's avatar Linus Torvalds

Revert "sysfs: drop kobj_ns_type handling"

This reverts commit cb26a311.

It mysteriously causes NetworkManager to not find the wireless device
for me.  As far as I can tell, Tejun *meant* for this commit to not make
any semantic changes, but there clearly are some.  So revert it, taking
into account some of the calling convention changes that happened in
this area in subsequent commits.

Cc: Tejun Heo <tj@kernel.org>
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent 0b1e73ed
...@@ -111,11 +111,6 @@ static int sysfs_link_sibling(struct sysfs_dirent *sd) ...@@ -111,11 +111,6 @@ static int sysfs_link_sibling(struct sysfs_dirent *sd)
/* add new node and rebalance the tree */ /* add new node and rebalance the tree */
rb_link_node(&sd->s_rb, parent, node); rb_link_node(&sd->s_rb, parent, node);
rb_insert_color(&sd->s_rb, &sd->s_parent->s_dir.children); rb_insert_color(&sd->s_rb, &sd->s_parent->s_dir.children);
/* if @sd has ns tag, mark the parent to enable ns filtering */
if (sd->s_ns)
sd->s_parent->s_flags |= SYSFS_FLAG_HAS_NS;
return 0; return 0;
} }
...@@ -135,13 +130,6 @@ static void sysfs_unlink_sibling(struct sysfs_dirent *sd) ...@@ -135,13 +130,6 @@ static void sysfs_unlink_sibling(struct sysfs_dirent *sd)
sd->s_parent->s_dir.subdirs--; sd->s_parent->s_dir.subdirs--;
rb_erase(&sd->s_rb, &sd->s_parent->s_dir.children); rb_erase(&sd->s_rb, &sd->s_parent->s_dir.children);
/*
* Either all or none of the children have tags. Clearing HAS_NS
* when there's no child left is enough to keep the flag synced.
*/
if (RB_EMPTY_ROOT(&sd->s_parent->s_dir.children))
sd->s_parent->s_flags &= ~SYSFS_FLAG_HAS_NS;
} }
/** /**
...@@ -291,6 +279,7 @@ static int sysfs_dentry_delete(const struct dentry *dentry) ...@@ -291,6 +279,7 @@ static int sysfs_dentry_delete(const struct dentry *dentry)
static int sysfs_dentry_revalidate(struct dentry *dentry, unsigned int flags) static int sysfs_dentry_revalidate(struct dentry *dentry, unsigned int flags)
{ {
struct sysfs_dirent *sd; struct sysfs_dirent *sd;
int type;
if (flags & LOOKUP_RCU) if (flags & LOOKUP_RCU)
return -ECHILD; return -ECHILD;
...@@ -311,8 +300,13 @@ static int sysfs_dentry_revalidate(struct dentry *dentry, unsigned int flags) ...@@ -311,8 +300,13 @@ static int sysfs_dentry_revalidate(struct dentry *dentry, unsigned int flags)
goto out_bad; goto out_bad;
/* The sysfs dirent has been moved to a different namespace */ /* The sysfs dirent has been moved to a different namespace */
if (sd->s_ns && sd->s_ns != sysfs_info(dentry->d_sb)->ns) type = KOBJ_NS_TYPE_NONE;
goto out_bad; if (sd->s_parent) {
type = sysfs_ns_type(sd->s_parent);
if (type != KOBJ_NS_TYPE_NONE &&
sysfs_info(dentry->d_sb)->ns[type] != sd->s_ns)
goto out_bad;
}
mutex_unlock(&sysfs_mutex); mutex_unlock(&sysfs_mutex);
out_valid: out_valid:
...@@ -432,6 +426,13 @@ int __sysfs_add_one(struct sysfs_addrm_cxt *acxt, struct sysfs_dirent *sd, ...@@ -432,6 +426,13 @@ int __sysfs_add_one(struct sysfs_addrm_cxt *acxt, struct sysfs_dirent *sd,
struct sysfs_inode_attrs *ps_iattr; struct sysfs_inode_attrs *ps_iattr;
int ret; int ret;
if (!!sysfs_ns_type(parent_sd) != !!sd->s_ns) {
WARN(1, KERN_WARNING "sysfs: ns %s in '%s' for '%s'\n",
sysfs_ns_type(parent_sd) ? "required" : "invalid",
parent_sd->s_name, sd->s_name);
return -EINVAL;
}
sd->s_hash = sysfs_name_hash(sd->s_name, sd->s_ns); sd->s_hash = sysfs_name_hash(sd->s_name, sd->s_ns);
sd->s_parent = sysfs_get(parent_sd); sd->s_parent = sysfs_get(parent_sd);
...@@ -611,6 +612,13 @@ struct sysfs_dirent *sysfs_find_dirent(struct sysfs_dirent *parent_sd, ...@@ -611,6 +612,13 @@ struct sysfs_dirent *sysfs_find_dirent(struct sysfs_dirent *parent_sd,
struct rb_node *node = parent_sd->s_dir.children.rb_node; struct rb_node *node = parent_sd->s_dir.children.rb_node;
unsigned int hash; unsigned int hash;
if (!!sysfs_ns_type(parent_sd) != !!ns) {
WARN(1, KERN_WARNING "sysfs: ns %s in '%s' for '%s'\n",
sysfs_ns_type(parent_sd) ? "required" : "invalid",
parent_sd->s_name, name);
return NULL;
}
hash = sysfs_name_hash(name, ns); hash = sysfs_name_hash(name, ns);
while (node) { while (node) {
struct sysfs_dirent *sd; struct sysfs_dirent *sd;
...@@ -659,6 +667,7 @@ struct sysfs_dirent *sysfs_get_dirent_ns(struct sysfs_dirent *parent_sd, ...@@ -659,6 +667,7 @@ struct sysfs_dirent *sysfs_get_dirent_ns(struct sysfs_dirent *parent_sd,
EXPORT_SYMBOL_GPL(sysfs_get_dirent_ns); EXPORT_SYMBOL_GPL(sysfs_get_dirent_ns);
static int create_dir(struct kobject *kobj, struct sysfs_dirent *parent_sd, static int create_dir(struct kobject *kobj, struct sysfs_dirent *parent_sd,
enum kobj_ns_type type,
const char *name, const void *ns, const char *name, const void *ns,
struct sysfs_dirent **p_sd) struct sysfs_dirent **p_sd)
{ {
...@@ -672,6 +681,7 @@ static int create_dir(struct kobject *kobj, struct sysfs_dirent *parent_sd, ...@@ -672,6 +681,7 @@ static int create_dir(struct kobject *kobj, struct sysfs_dirent *parent_sd,
if (!sd) if (!sd)
return -ENOMEM; return -ENOMEM;
sd->s_flags |= (type << SYSFS_NS_TYPE_SHIFT);
sd->s_ns = ns; sd->s_ns = ns;
sd->s_dir.kobj = kobj; sd->s_dir.kobj = kobj;
...@@ -691,7 +701,33 @@ static int create_dir(struct kobject *kobj, struct sysfs_dirent *parent_sd, ...@@ -691,7 +701,33 @@ static int create_dir(struct kobject *kobj, struct sysfs_dirent *parent_sd,
int sysfs_create_subdir(struct kobject *kobj, const char *name, int sysfs_create_subdir(struct kobject *kobj, const char *name,
struct sysfs_dirent **p_sd) struct sysfs_dirent **p_sd)
{ {
return create_dir(kobj, kobj->sd, name, NULL, p_sd); return create_dir(kobj, kobj->sd,
KOBJ_NS_TYPE_NONE, name, NULL, p_sd);
}
/**
* sysfs_read_ns_type: return associated ns_type
* @kobj: the kobject being queried
*
* Each kobject can be tagged with exactly one namespace type
* (i.e. network or user). Return the ns_type associated with
* this object if any
*/
static enum kobj_ns_type sysfs_read_ns_type(struct kobject *kobj)
{
const struct kobj_ns_type_operations *ops;
enum kobj_ns_type type;
ops = kobj_child_ns_ops(kobj);
if (!ops)
return KOBJ_NS_TYPE_NONE;
type = ops->type;
BUG_ON(type <= KOBJ_NS_TYPE_NONE);
BUG_ON(type >= KOBJ_NS_TYPES);
BUG_ON(!kobj_ns_type_registered(type));
return type;
} }
/** /**
...@@ -701,6 +737,7 @@ int sysfs_create_subdir(struct kobject *kobj, const char *name, ...@@ -701,6 +737,7 @@ int sysfs_create_subdir(struct kobject *kobj, const char *name,
*/ */
int sysfs_create_dir_ns(struct kobject *kobj, const void *ns) int sysfs_create_dir_ns(struct kobject *kobj, const void *ns)
{ {
enum kobj_ns_type type;
struct sysfs_dirent *parent_sd, *sd; struct sysfs_dirent *parent_sd, *sd;
int error = 0; int error = 0;
...@@ -714,7 +751,9 @@ int sysfs_create_dir_ns(struct kobject *kobj, const void *ns) ...@@ -714,7 +751,9 @@ int sysfs_create_dir_ns(struct kobject *kobj, const void *ns)
if (!parent_sd) if (!parent_sd)
return -ENOENT; return -ENOENT;
error = create_dir(kobj, parent_sd, kobject_name(kobj), ns, &sd); type = sysfs_read_ns_type(kobj);
error = create_dir(kobj, parent_sd, type, kobject_name(kobj), ns, &sd);
if (!error) if (!error)
kobj->sd = sd; kobj->sd = sd;
return error; return error;
...@@ -728,12 +767,13 @@ static struct dentry *sysfs_lookup(struct inode *dir, struct dentry *dentry, ...@@ -728,12 +767,13 @@ static struct dentry *sysfs_lookup(struct inode *dir, struct dentry *dentry,
struct sysfs_dirent *parent_sd = parent->d_fsdata; struct sysfs_dirent *parent_sd = parent->d_fsdata;
struct sysfs_dirent *sd; struct sysfs_dirent *sd;
struct inode *inode; struct inode *inode;
const void *ns = NULL; enum kobj_ns_type type;
const void *ns;
mutex_lock(&sysfs_mutex); mutex_lock(&sysfs_mutex);
if (parent_sd->s_flags & SYSFS_FLAG_HAS_NS) type = sysfs_ns_type(parent_sd);
ns = sysfs_info(dir->i_sb)->ns; ns = sysfs_info(dir->i_sb)->ns[type];
sd = sysfs_find_dirent(parent_sd, dentry->d_name.name, ns); sd = sysfs_find_dirent(parent_sd, dentry->d_name.name, ns);
...@@ -1056,15 +1096,15 @@ static int sysfs_readdir(struct file *file, struct dir_context *ctx) ...@@ -1056,15 +1096,15 @@ static int sysfs_readdir(struct file *file, struct dir_context *ctx)
struct dentry *dentry = file->f_path.dentry; struct dentry *dentry = file->f_path.dentry;
struct sysfs_dirent *parent_sd = dentry->d_fsdata; struct sysfs_dirent *parent_sd = dentry->d_fsdata;
struct sysfs_dirent *pos = file->private_data; struct sysfs_dirent *pos = file->private_data;
const void *ns = NULL; enum kobj_ns_type type;
const void *ns;
type = sysfs_ns_type(parent_sd);
ns = sysfs_info(dentry->d_sb)->ns[type];
if (!dir_emit_dots(file, ctx)) if (!dir_emit_dots(file, ctx))
return 0; return 0;
mutex_lock(&sysfs_mutex); mutex_lock(&sysfs_mutex);
if (parent_sd->s_flags & SYSFS_FLAG_HAS_NS)
ns = sysfs_info(dentry->d_sb)->ns;
for (pos = sysfs_dir_pos(ns, parent_sd, ctx->pos, pos); for (pos = sysfs_dir_pos(ns, parent_sd, ctx->pos, pos);
pos; pos;
pos = sysfs_dir_next_pos(ns, parent_sd, ctx->pos, pos)) { pos = sysfs_dir_next_pos(ns, parent_sd, ctx->pos, pos)) {
......
...@@ -36,7 +36,7 @@ static const struct super_operations sysfs_ops = { ...@@ -36,7 +36,7 @@ static const struct super_operations sysfs_ops = {
struct sysfs_dirent sysfs_root = { struct sysfs_dirent sysfs_root = {
.s_name = "", .s_name = "",
.s_count = ATOMIC_INIT(1), .s_count = ATOMIC_INIT(1),
.s_flags = SYSFS_DIR, .s_flags = SYSFS_DIR | (KOBJ_NS_TYPE_NONE << SYSFS_NS_TYPE_SHIFT),
.s_mode = S_IFDIR | S_IRUGO | S_IXUGO, .s_mode = S_IFDIR | S_IRUGO | S_IXUGO,
.s_ino = 1, .s_ino = 1,
}; };
...@@ -77,8 +77,14 @@ static int sysfs_test_super(struct super_block *sb, void *data) ...@@ -77,8 +77,14 @@ static int sysfs_test_super(struct super_block *sb, void *data)
{ {
struct sysfs_super_info *sb_info = sysfs_info(sb); struct sysfs_super_info *sb_info = sysfs_info(sb);
struct sysfs_super_info *info = data; struct sysfs_super_info *info = data;
enum kobj_ns_type type;
int found = 1;
return sb_info->ns == info->ns; for (type = KOBJ_NS_TYPE_NONE; type < KOBJ_NS_TYPES; type++) {
if (sb_info->ns[type] != info->ns[type])
found = 0;
}
return found;
} }
static int sysfs_set_super(struct super_block *sb, void *data) static int sysfs_set_super(struct super_block *sb, void *data)
...@@ -92,7 +98,9 @@ static int sysfs_set_super(struct super_block *sb, void *data) ...@@ -92,7 +98,9 @@ static int sysfs_set_super(struct super_block *sb, void *data)
static void free_sysfs_super_info(struct sysfs_super_info *info) static void free_sysfs_super_info(struct sysfs_super_info *info)
{ {
kobj_ns_drop(KOBJ_NS_TYPE_NET, info->ns); int type;
for (type = KOBJ_NS_TYPE_NONE; type < KOBJ_NS_TYPES; type++)
kobj_ns_drop(type, info->ns[type]);
kfree(info); kfree(info);
} }
...@@ -100,6 +108,7 @@ static struct dentry *sysfs_mount(struct file_system_type *fs_type, ...@@ -100,6 +108,7 @@ static struct dentry *sysfs_mount(struct file_system_type *fs_type,
int flags, const char *dev_name, void *data) int flags, const char *dev_name, void *data)
{ {
struct sysfs_super_info *info; struct sysfs_super_info *info;
enum kobj_ns_type type;
struct super_block *sb; struct super_block *sb;
int error; int error;
...@@ -107,15 +116,18 @@ static struct dentry *sysfs_mount(struct file_system_type *fs_type, ...@@ -107,15 +116,18 @@ static struct dentry *sysfs_mount(struct file_system_type *fs_type,
if (!capable(CAP_SYS_ADMIN) && !fs_fully_visible(fs_type)) if (!capable(CAP_SYS_ADMIN) && !fs_fully_visible(fs_type))
return ERR_PTR(-EPERM); return ERR_PTR(-EPERM);
if (!kobj_ns_current_may_mount(KOBJ_NS_TYPE_NET)) for (type = KOBJ_NS_TYPE_NONE; type < KOBJ_NS_TYPES; type++) {
return ERR_PTR(-EPERM); if (!kobj_ns_current_may_mount(type))
return ERR_PTR(-EPERM);
}
} }
info = kzalloc(sizeof(*info), GFP_KERNEL); info = kzalloc(sizeof(*info), GFP_KERNEL);
if (!info) if (!info)
return ERR_PTR(-ENOMEM); return ERR_PTR(-ENOMEM);
info->ns = kobj_ns_grab_current(KOBJ_NS_TYPE_NET); for (type = KOBJ_NS_TYPE_NONE; type < KOBJ_NS_TYPES; type++)
info->ns[type] = kobj_ns_grab_current(type);
sb = sget(fs_type, sysfs_test_super, sysfs_set_super, flags, info); sb = sget(fs_type, sysfs_test_super, sysfs_set_super, flags, info);
if (IS_ERR(sb) || sb->s_fs_info != info) if (IS_ERR(sb) || sb->s_fs_info != info)
......
...@@ -28,6 +28,7 @@ static int sysfs_do_create_link_sd(struct sysfs_dirent *parent_sd, ...@@ -28,6 +28,7 @@ static int sysfs_do_create_link_sd(struct sysfs_dirent *parent_sd,
struct sysfs_dirent *target_sd = NULL; struct sysfs_dirent *target_sd = NULL;
struct sysfs_dirent *sd = NULL; struct sysfs_dirent *sd = NULL;
struct sysfs_addrm_cxt acxt; struct sysfs_addrm_cxt acxt;
enum kobj_ns_type ns_type;
int error; int error;
BUG_ON(!name || !parent_sd); BUG_ON(!name || !parent_sd);
...@@ -51,15 +52,29 @@ static int sysfs_do_create_link_sd(struct sysfs_dirent *parent_sd, ...@@ -51,15 +52,29 @@ static int sysfs_do_create_link_sd(struct sysfs_dirent *parent_sd,
if (!sd) if (!sd)
goto out_put; goto out_put;
sd->s_ns = target_sd->s_ns; ns_type = sysfs_ns_type(parent_sd);
if (ns_type)
sd->s_ns = target_sd->s_ns;
sd->s_symlink.target_sd = target_sd; sd->s_symlink.target_sd = target_sd;
target_sd = NULL; /* reference is now owned by the symlink */ target_sd = NULL; /* reference is now owned by the symlink */
sysfs_addrm_start(&acxt); sysfs_addrm_start(&acxt);
if (warn) /* Symlinks must be between directories with the same ns_type */
error = sysfs_add_one(&acxt, sd, parent_sd); if (!ns_type ||
else (ns_type == sysfs_ns_type(sd->s_symlink.target_sd->s_parent))) {
error = __sysfs_add_one(&acxt, sd, parent_sd); if (warn)
error = sysfs_add_one(&acxt, sd, parent_sd);
else
error = __sysfs_add_one(&acxt, sd, parent_sd);
} else {
error = -EINVAL;
WARN(1, KERN_WARNING
"sysfs: symlink across ns_types %s/%s -> %s/%s\n",
parent_sd->s_name,
sd->s_name,
sd->s_symlink.target_sd->s_parent->s_name,
sd->s_symlink.target_sd->s_name);
}
sysfs_addrm_finish(&acxt); sysfs_addrm_finish(&acxt);
if (error) if (error)
...@@ -149,7 +164,7 @@ void sysfs_delete_link(struct kobject *kobj, struct kobject *targ, ...@@ -149,7 +164,7 @@ void sysfs_delete_link(struct kobject *kobj, struct kobject *targ,
* sysfs_remove_dir() for details. * sysfs_remove_dir() for details.
*/ */
spin_lock(&sysfs_symlink_target_lock); spin_lock(&sysfs_symlink_target_lock);
if (targ->sd) if (targ->sd && sysfs_ns_type(kobj->sd))
ns = targ->sd->s_ns; ns = targ->sd->s_ns;
spin_unlock(&sysfs_symlink_target_lock); spin_unlock(&sysfs_symlink_target_lock);
sysfs_hash_and_remove(kobj->sd, name, ns); sysfs_hash_and_remove(kobj->sd, name, ns);
......
...@@ -90,8 +90,11 @@ struct sysfs_dirent { ...@@ -90,8 +90,11 @@ struct sysfs_dirent {
#define SYSFS_COPY_NAME (SYSFS_DIR | SYSFS_KOBJ_LINK) #define SYSFS_COPY_NAME (SYSFS_DIR | SYSFS_KOBJ_LINK)
#define SYSFS_ACTIVE_REF (SYSFS_KOBJ_ATTR | SYSFS_KOBJ_BIN_ATTR) #define SYSFS_ACTIVE_REF (SYSFS_KOBJ_ATTR | SYSFS_KOBJ_BIN_ATTR)
#define SYSFS_FLAG_MASK ~SYSFS_TYPE_MASK /* identify any namespace tag on sysfs_dirents */
#define SYSFS_FLAG_HAS_NS 0x01000 #define SYSFS_NS_TYPE_MASK 0xf00
#define SYSFS_NS_TYPE_SHIFT 8
#define SYSFS_FLAG_MASK ~(SYSFS_NS_TYPE_MASK|SYSFS_TYPE_MASK)
#define SYSFS_FLAG_REMOVED 0x02000 #define SYSFS_FLAG_REMOVED 0x02000
static inline unsigned int sysfs_type(struct sysfs_dirent *sd) static inline unsigned int sysfs_type(struct sysfs_dirent *sd)
...@@ -99,6 +102,15 @@ static inline unsigned int sysfs_type(struct sysfs_dirent *sd) ...@@ -99,6 +102,15 @@ static inline unsigned int sysfs_type(struct sysfs_dirent *sd)
return sd->s_flags & SYSFS_TYPE_MASK; return sd->s_flags & SYSFS_TYPE_MASK;
} }
/*
* Return any namespace tags on this dirent.
* enum kobj_ns_type is defined in linux/kobject.h
*/
static inline enum kobj_ns_type sysfs_ns_type(struct sysfs_dirent *sd)
{
return (sd->s_flags & SYSFS_NS_TYPE_MASK) >> SYSFS_NS_TYPE_SHIFT;
}
#ifdef CONFIG_DEBUG_LOCK_ALLOC #ifdef CONFIG_DEBUG_LOCK_ALLOC
#define sysfs_dirent_init_lockdep(sd) \ #define sysfs_dirent_init_lockdep(sd) \
...@@ -143,13 +155,12 @@ struct sysfs_addrm_cxt { ...@@ -143,13 +155,12 @@ struct sysfs_addrm_cxt {
*/ */
/* /*
* Each sb is associated with one namespace tag, currently the network * Each sb is associated with a set of namespace tags (i.e.
* namespace of the task which mounted this sysfs instance. If multiple * the network namespace of the task which mounted this sysfs
* tags become necessary, make the following an array and compare * instance).
* sysfs_dirent tag against every entry.
*/ */
struct sysfs_super_info { struct sysfs_super_info {
void *ns; void *ns[KOBJ_NS_TYPES];
}; };
#define sysfs_info(SB) ((struct sysfs_super_info *)(SB->s_fs_info)) #define sysfs_info(SB) ((struct sysfs_super_info *)(SB->s_fs_info))
extern struct sysfs_dirent sysfs_root; extern struct sysfs_dirent sysfs_root;
......
...@@ -30,14 +30,11 @@ ...@@ -30,14 +30,11 @@
const void *kobject_namespace(struct kobject *kobj) const void *kobject_namespace(struct kobject *kobj)
{ {
const struct kobj_ns_type_operations *ns_ops = kobj_ns_ops(kobj); const struct kobj_ns_type_operations *ns_ops = kobj_ns_ops(kobj);
const void *ns;
if (!ns_ops || ns_ops->type == KOBJ_NS_TYPE_NONE) if (!ns_ops || ns_ops->type == KOBJ_NS_TYPE_NONE)
return NULL; return NULL;
ns = kobj->ktype->namespace(kobj); return kobj->ktype->namespace(kobj);
WARN_ON(!ns); /* @kobj in a namespace is required to have !NULL tag */
return ns;
} }
/* /*
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment