Commit 569e5590 authored by Shaohua Li's avatar Shaohua Li Committed by Linus Torvalds

thp: improve the error code path

Improve the error code path.  Delete unnecessary sysfs file for example.
Also remove the #ifdef xxx to make code better.
Signed-off-by: default avatarShaohua Li <shaohua.li@intel.com>
Reviewed-by: default avatarAndrea Arcangeli <aarcange@redhat.com>
Cc: David Rientjes <rientjes@google.com>
Cc: Johannes Weiner <jweiner@redhat.com>
Cc: Minchan Kim <minchan.kim@gmail.com>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent 0efc8eb9
...@@ -487,41 +487,68 @@ static struct attribute_group khugepaged_attr_group = { ...@@ -487,41 +487,68 @@ static struct attribute_group khugepaged_attr_group = {
.attrs = khugepaged_attr, .attrs = khugepaged_attr,
.name = "khugepaged", .name = "khugepaged",
}; };
#endif /* CONFIG_SYSFS */
static int __init hugepage_init(void) static int __init hugepage_init_sysfs(struct kobject **hugepage_kobj)
{ {
int err; int err;
#ifdef CONFIG_SYSFS
static struct kobject *hugepage_kobj;
#endif
err = -EINVAL; *hugepage_kobj = kobject_create_and_add("transparent_hugepage", mm_kobj);
if (!has_transparent_hugepage()) { if (unlikely(!*hugepage_kobj)) {
transparent_hugepage_flags = 0;
goto out;
}
#ifdef CONFIG_SYSFS
err = -ENOMEM;
hugepage_kobj = kobject_create_and_add("transparent_hugepage", mm_kobj);
if (unlikely(!hugepage_kobj)) {
printk(KERN_ERR "hugepage: failed kobject create\n"); printk(KERN_ERR "hugepage: failed kobject create\n");
goto out; return -ENOMEM;
} }
err = sysfs_create_group(hugepage_kobj, &hugepage_attr_group); err = sysfs_create_group(*hugepage_kobj, &hugepage_attr_group);
if (err) { if (err) {
printk(KERN_ERR "hugepage: failed register hugeage group\n"); printk(KERN_ERR "hugepage: failed register hugeage group\n");
goto out; goto delete_obj;
} }
err = sysfs_create_group(hugepage_kobj, &khugepaged_attr_group); err = sysfs_create_group(*hugepage_kobj, &khugepaged_attr_group);
if (err) { if (err) {
printk(KERN_ERR "hugepage: failed register hugeage group\n"); printk(KERN_ERR "hugepage: failed register hugeage group\n");
goto out; goto remove_hp_group;
} }
#endif
return 0;
remove_hp_group:
sysfs_remove_group(*hugepage_kobj, &hugepage_attr_group);
delete_obj:
kobject_put(*hugepage_kobj);
return err;
}
static void __init hugepage_exit_sysfs(struct kobject *hugepage_kobj)
{
sysfs_remove_group(hugepage_kobj, &khugepaged_attr_group);
sysfs_remove_group(hugepage_kobj, &hugepage_attr_group);
kobject_put(hugepage_kobj);
}
#else
static inline int hugepage_init_sysfs(struct kobject **hugepage_kobj)
{
return 0;
}
static inline void hugepage_exit_sysfs(struct kobject *hugepage_kobj)
{
}
#endif /* CONFIG_SYSFS */
static int __init hugepage_init(void)
{
int err;
struct kobject *hugepage_kobj;
if (!has_transparent_hugepage()) {
transparent_hugepage_flags = 0;
return -EINVAL;
}
err = hugepage_init_sysfs(&hugepage_kobj);
if (err)
return err;
err = khugepaged_slab_init(); err = khugepaged_slab_init();
if (err) if (err)
...@@ -545,7 +572,9 @@ static int __init hugepage_init(void) ...@@ -545,7 +572,9 @@ static int __init hugepage_init(void)
set_recommended_min_free_kbytes(); set_recommended_min_free_kbytes();
return 0;
out: out:
hugepage_exit_sysfs(hugepage_kobj);
return err; return err;
} }
module_init(hugepage_init) module_init(hugepage_init)
......
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