Commit c74c120a authored by Alexey Dobriyan's avatar Alexey Dobriyan Committed by Linus Torvalds

proc: remove proc_root from drivers

Remove proc_root export.  Creation and removal works well if parent PDE is
supplied as NULL -- it worked always that way.

So, one useless export removed and consistency added, some drivers created
PDEs with &proc_root as parent but removed them as NULL and so on.
Signed-off-by: default avatarAlexey Dobriyan <adobriyan@gmail.com>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent 928b4d8c
...@@ -35,7 +35,7 @@ create_proc_entries(void) ...@@ -35,7 +35,7 @@ create_proc_entries(void)
{ {
struct proc_dir_entry* tags_entry; struct proc_dir_entry* tags_entry;
tags_entry = create_proc_read_entry("atags", 0400, &proc_root, read_buffer, &tags_buffer); tags_entry = create_proc_read_entry("atags", 0400, NULL, read_buffer, &tags_buffer);
if (!tags_entry) if (!tags_entry)
return -ENOMEM; return -ENOMEM;
......
...@@ -302,7 +302,7 @@ void __init iop_init(void) ...@@ -302,7 +302,7 @@ void __init iop_init(void)
#if 0 /* Crashing in 2.4 now, not yet sure why. --jmt */ #if 0 /* Crashing in 2.4 now, not yet sure why. --jmt */
#ifdef CONFIG_PROC_FS #ifdef CONFIG_PROC_FS
create_proc_info_entry("mac_iop", 0, &proc_root, iop_get_proc_info); create_proc_info_entry("mac_iop", 0, NULL, iop_get_proc_info);
#endif #endif
#endif #endif
} }
......
...@@ -65,7 +65,7 @@ excite_bootrom_read(char *page, char **start, off_t off, int count, ...@@ -65,7 +65,7 @@ excite_bootrom_read(char *page, char **start, off_t off, int count,
void excite_procfs_init(void) void excite_procfs_init(void)
{ {
/* Create & populate /proc/excite */ /* Create & populate /proc/excite */
struct proc_dir_entry * const pdir = proc_mkdir("excite", &proc_root); struct proc_dir_entry * const pdir = proc_mkdir("excite", NULL);
if (pdir) { if (pdir) {
struct proc_dir_entry * e; struct proc_dir_entry * e;
......
...@@ -59,7 +59,7 @@ static int make_proc_exitcode(void) ...@@ -59,7 +59,7 @@ static int make_proc_exitcode(void)
{ {
struct proc_dir_entry *ent; struct proc_dir_entry *ent;
ent = create_proc_entry("exitcode", 0600, &proc_root); ent = create_proc_entry("exitcode", 0600, NULL);
if (ent == NULL) { if (ent == NULL) {
printk(KERN_WARNING "make_proc_exitcode : Failed to register " printk(KERN_WARNING "make_proc_exitcode : Failed to register "
"/proc/exitcode\n"); "/proc/exitcode\n");
......
...@@ -364,7 +364,7 @@ int __init make_proc_sysemu(void) ...@@ -364,7 +364,7 @@ int __init make_proc_sysemu(void)
if (!sysemu_supported) if (!sysemu_supported)
return 0; return 0;
ent = create_proc_entry("sysemu", 0600, &proc_root); ent = create_proc_entry("sysemu", 0600, NULL);
if (ent == NULL) if (ent == NULL)
{ {
......
...@@ -424,7 +424,7 @@ static int __init mtrr_if_init(void) ...@@ -424,7 +424,7 @@ static int __init mtrr_if_init(void)
return -ENODEV; return -ENODEV;
proc_root_mtrr = proc_root_mtrr =
proc_create("mtrr", S_IWUSR | S_IRUGO, &proc_root, &mtrr_fops); proc_create("mtrr", S_IWUSR | S_IRUGO, NULL, &mtrr_fops);
if (proc_root_mtrr) if (proc_root_mtrr)
proc_root_mtrr->owner = THIS_MODULE; proc_root_mtrr->owner = THIS_MODULE;
......
...@@ -423,7 +423,7 @@ cleanup_module(void) ...@@ -423,7 +423,7 @@ cleanup_module(void)
} }
put_tty_driver(ip2_tty_driver); put_tty_driver(ip2_tty_driver);
unregister_chrdev(IP2_IPL_MAJOR, pcIpl); unregister_chrdev(IP2_IPL_MAJOR, pcIpl);
remove_proc_entry("ip2mem", &proc_root); remove_proc_entry("ip2mem", NULL);
// free memory // free memory
for (i = 0; i < IP2_MAX_BOARDS; i++) { for (i = 0; i < IP2_MAX_BOARDS; i++) {
...@@ -695,7 +695,7 @@ ip2_loadmain(int *iop, int *irqp, unsigned char *firmware, int firmsize) ...@@ -695,7 +695,7 @@ ip2_loadmain(int *iop, int *irqp, unsigned char *firmware, int firmsize)
} }
} }
/* Register the read_procmem thing */ /* Register the read_procmem thing */
if (!create_proc_info_entry("ip2mem",0,&proc_root,ip2_read_procmem)) { if (!create_proc_info_entry("ip2mem",0,NULL,ip2_read_procmem)) {
printk(KERN_ERR "IP2: failed to register read_procmem\n"); printk(KERN_ERR "IP2: failed to register read_procmem\n");
} else { } else {
......
...@@ -183,7 +183,7 @@ void __init mca_do_proc_init(void) ...@@ -183,7 +183,7 @@ void __init mca_do_proc_init(void)
struct proc_dir_entry* node = NULL; struct proc_dir_entry* node = NULL;
struct mca_device *mca_dev; struct mca_device *mca_dev;
proc_mca = proc_mkdir("mca", &proc_root); proc_mca = proc_mkdir("mca", NULL);
create_proc_read_entry("pos",0,proc_mca,get_mca_info,NULL); create_proc_read_entry("pos",0,proc_mca,get_mca_info,NULL);
create_proc_read_entry("machine",0,proc_mca,get_mca_machine_info,NULL); create_proc_read_entry("machine",0,proc_mca,get_mca_machine_info,NULL);
......
...@@ -210,7 +210,7 @@ static int hdpu_cpustate_probe(struct platform_device *pdev) ...@@ -210,7 +210,7 @@ static int hdpu_cpustate_probe(struct platform_device *pdev)
return ret; return ret;
} }
proc_de = create_proc_entry("sky_cpustate", 0666, &proc_root); proc_de = create_proc_entry("sky_cpustate", 0666, NULL);
if (!proc_de) { if (!proc_de) {
printk(KERN_WARNING "sky_cpustate: " printk(KERN_WARNING "sky_cpustate: "
"Unable to create proc entry\n"); "Unable to create proc entry\n");
......
...@@ -102,8 +102,8 @@ static int hdpu_nexus_probe(struct platform_device *pdev) ...@@ -102,8 +102,8 @@ static int hdpu_nexus_probe(struct platform_device *pdev)
printk(KERN_ERR "sky_nexus: Could not map slot id\n"); printk(KERN_ERR "sky_nexus: Could not map slot id\n");
} }
hdpu_slot_id = create_proc_entry("sky_slot_id", 0666, &proc_root); hdpu_slot_id = create_proc_entry("sky_slot_id", 0666, NULL);
if (!hdpu_slot_id) { if (!hdpu_slot_id)
printk(KERN_WARNING "sky_nexus: " printk(KERN_WARNING "sky_nexus: "
"Unable to create proc dir entry: sky_slot_id\n"); "Unable to create proc dir entry: sky_slot_id\n");
} else { } else {
...@@ -111,8 +111,8 @@ static int hdpu_nexus_probe(struct platform_device *pdev) ...@@ -111,8 +111,8 @@ static int hdpu_nexus_probe(struct platform_device *pdev)
hdpu_slot_id->owner = THIS_MODULE; hdpu_slot_id->owner = THIS_MODULE;
} }
hdpu_chassis_id = create_proc_entry("sky_chassis_id", 0666, &proc_root); hdpu_chassis_id = create_proc_entry("sky_chassis_id", 0666, NULL);
if (!hdpu_chassis_id) { if (!hdpu_chassis_id)
printk(KERN_WARNING "sky_nexus: " printk(KERN_WARNING "sky_nexus: "
"Unable to create proc dir entry: sky_chassis_id\n"); "Unable to create proc dir entry: sky_chassis_id\n");
} else { } else {
...@@ -128,8 +128,8 @@ static int hdpu_nexus_remove(struct platform_device *pdev) ...@@ -128,8 +128,8 @@ static int hdpu_nexus_remove(struct platform_device *pdev)
slot_id = -1; slot_id = -1;
chassis_id = -1; chassis_id = -1;
remove_proc_entry("sky_slot_id", &proc_root); remove_proc_entry("sky_slot_id", NULL);
remove_proc_entry("sky_chassis_id", &proc_root); remove_proc_entry("sky_chassis_id", NULL);
hdpu_slot_id = 0; hdpu_slot_id = 0;
hdpu_chassis_id = 0; hdpu_chassis_id = 0;
......
...@@ -311,7 +311,7 @@ dasd_statistics_write(struct file *file, const char __user *user_buf, ...@@ -311,7 +311,7 @@ dasd_statistics_write(struct file *file, const char __user *user_buf,
int int
dasd_proc_init(void) dasd_proc_init(void)
{ {
dasd_proc_root_entry = proc_mkdir("dasd", &proc_root); dasd_proc_root_entry = proc_mkdir("dasd", NULL);
if (!dasd_proc_root_entry) if (!dasd_proc_root_entry)
goto out_nodasd; goto out_nodasd;
dasd_proc_root_entry->owner = THIS_MODULE; dasd_proc_root_entry->owner = THIS_MODULE;
...@@ -335,7 +335,7 @@ dasd_proc_init(void) ...@@ -335,7 +335,7 @@ dasd_proc_init(void)
out_nostatistics: out_nostatistics:
remove_proc_entry("devices", dasd_proc_root_entry); remove_proc_entry("devices", dasd_proc_root_entry);
out_nodevices: out_nodevices:
remove_proc_entry("dasd", &proc_root); remove_proc_entry("dasd", NULL);
out_nodasd: out_nodasd:
return -ENOENT; return -ENOENT;
} }
...@@ -345,5 +345,5 @@ dasd_proc_exit(void) ...@@ -345,5 +345,5 @@ dasd_proc_exit(void)
{ {
remove_proc_entry("devices", dasd_proc_root_entry); remove_proc_entry("devices", dasd_proc_root_entry);
remove_proc_entry("statistics", dasd_proc_root_entry); remove_proc_entry("statistics", dasd_proc_root_entry);
remove_proc_entry("dasd", &proc_root); remove_proc_entry("dasd", NULL);
} }
...@@ -125,7 +125,7 @@ tape_proc_init(void) ...@@ -125,7 +125,7 @@ tape_proc_init(void)
{ {
tape_proc_devices = tape_proc_devices =
create_proc_entry ("tapedevices", S_IFREG | S_IRUGO | S_IWUSR, create_proc_entry ("tapedevices", S_IFREG | S_IRUGO | S_IWUSR,
&proc_root); NULL);
if (tape_proc_devices == NULL) { if (tape_proc_devices == NULL) {
PRINT_WARN("tape: Cannot register procfs entry tapedevices\n"); PRINT_WARN("tape: Cannot register procfs entry tapedevices\n");
return; return;
...@@ -141,5 +141,5 @@ void ...@@ -141,5 +141,5 @@ void
tape_proc_cleanup(void) tape_proc_cleanup(void)
{ {
if (tape_proc_devices != NULL) if (tape_proc_devices != NULL)
remove_proc_entry ("tapedevices", &proc_root); remove_proc_entry ("tapedevices", NULL);
} }
...@@ -375,7 +375,7 @@ cio_ignore_proc_init (void) ...@@ -375,7 +375,7 @@ cio_ignore_proc_init (void)
struct proc_dir_entry *entry; struct proc_dir_entry *entry;
entry = create_proc_entry ("cio_ignore", S_IFREG | S_IRUGO | S_IWUSR, entry = create_proc_entry ("cio_ignore", S_IFREG | S_IRUGO | S_IWUSR,
&proc_root); NULL);
if (!entry) if (!entry)
return -ENOENT; return -ENOENT;
......
...@@ -3632,7 +3632,7 @@ qdio_add_procfs_entry(void) ...@@ -3632,7 +3632,7 @@ qdio_add_procfs_entry(void)
{ {
proc_perf_file_registration=0; proc_perf_file_registration=0;
qdio_perf_proc_file=create_proc_entry(QDIO_PERF, qdio_perf_proc_file=create_proc_entry(QDIO_PERF,
S_IFREG|0444,&proc_root); S_IFREG|0444,NULL);
if (qdio_perf_proc_file) { if (qdio_perf_proc_file) {
qdio_perf_proc_file->read_proc=&qdio_perf_procfile_read; qdio_perf_proc_file->read_proc=&qdio_perf_procfile_read;
} else proc_perf_file_registration=-1; } else proc_perf_file_registration=-1;
...@@ -3647,7 +3647,7 @@ static void ...@@ -3647,7 +3647,7 @@ static void
qdio_remove_procfs_entry(void) qdio_remove_procfs_entry(void)
{ {
if (!proc_perf_file_registration) /* means if it went ok earlier */ if (!proc_perf_file_registration) /* means if it went ok earlier */
remove_proc_entry(QDIO_PERF,&proc_root); remove_proc_entry(QDIO_PERF,NULL);
} }
/** /**
......
...@@ -4996,7 +4996,7 @@ static int __init megaraid_init(void) ...@@ -4996,7 +4996,7 @@ static int __init megaraid_init(void)
max_mbox_busy_wait = MBOX_BUSY_WAIT; max_mbox_busy_wait = MBOX_BUSY_WAIT;
#ifdef CONFIG_PROC_FS #ifdef CONFIG_PROC_FS
mega_proc_dir_entry = proc_mkdir("megaraid", &proc_root); mega_proc_dir_entry = proc_mkdir("megaraid", NULL);
if (!mega_proc_dir_entry) { if (!mega_proc_dir_entry) {
printk(KERN_WARNING printk(KERN_WARNING
"megaraid: failed to create megaraid root\n"); "megaraid: failed to create megaraid root\n");
...@@ -5005,7 +5005,7 @@ static int __init megaraid_init(void) ...@@ -5005,7 +5005,7 @@ static int __init megaraid_init(void)
error = pci_register_driver(&megaraid_pci_driver); error = pci_register_driver(&megaraid_pci_driver);
if (error) { if (error) {
#ifdef CONFIG_PROC_FS #ifdef CONFIG_PROC_FS
remove_proc_entry("megaraid", &proc_root); remove_proc_entry("megaraid", NULL);
#endif #endif
return error; return error;
} }
...@@ -5035,7 +5035,7 @@ static void __exit megaraid_exit(void) ...@@ -5035,7 +5035,7 @@ static void __exit megaraid_exit(void)
pci_unregister_driver(&megaraid_pci_driver); pci_unregister_driver(&megaraid_pci_driver);
#ifdef CONFIG_PROC_FS #ifdef CONFIG_PROC_FS
remove_proc_entry("megaraid", &proc_root); remove_proc_entry("megaraid", NULL);
#endif #endif
} }
......
...@@ -381,7 +381,7 @@ int __init clps711xfb_init(void) ...@@ -381,7 +381,7 @@ int __init clps711xfb_init(void)
/* Register the /proc entries. */ /* Register the /proc entries. */
clps7111fb_backlight_proc_entry = create_proc_entry("backlight", 0444, clps7111fb_backlight_proc_entry = create_proc_entry("backlight", 0444,
&proc_root); NULL);
if (clps7111fb_backlight_proc_entry == NULL) { if (clps7111fb_backlight_proc_entry == NULL) {
printk("Couldn't create the /proc entry for the backlight.\n"); printk("Couldn't create the /proc entry for the backlight.\n");
return -EINVAL; return -EINVAL;
......
...@@ -11,6 +11,7 @@ ...@@ -11,6 +11,7 @@
#include <linux/proc_fs.h> #include <linux/proc_fs.h>
extern struct proc_dir_entry proc_root;
#ifdef CONFIG_PROC_SYSCTL #ifdef CONFIG_PROC_SYSCTL
extern int proc_sys_init(void); extern int proc_sys_init(void);
#else #else
......
...@@ -854,7 +854,7 @@ void __init proc_misc_init(void) ...@@ -854,7 +854,7 @@ void __init proc_misc_init(void)
/* And now for trickier ones */ /* And now for trickier ones */
#ifdef CONFIG_PRINTK #ifdef CONFIG_PRINTK
proc_create("kmsg", S_IRUSR, &proc_root, &proc_kmsg_operations); proc_create("kmsg", S_IRUSR, NULL, &proc_kmsg_operations);
#endif #endif
proc_create("locks", 0, NULL, &proc_locks_operations); proc_create("locks", 0, NULL, &proc_locks_operations);
proc_create("devices", 0, NULL, &proc_devinfo_operations); proc_create("devices", 0, NULL, &proc_devinfo_operations);
......
...@@ -232,4 +232,3 @@ EXPORT_SYMBOL(proc_mkdir); ...@@ -232,4 +232,3 @@ EXPORT_SYMBOL(proc_mkdir);
EXPORT_SYMBOL(create_proc_entry); EXPORT_SYMBOL(create_proc_entry);
EXPORT_SYMBOL(proc_create); EXPORT_SYMBOL(proc_create);
EXPORT_SYMBOL(remove_proc_entry); EXPORT_SYMBOL(remove_proc_entry);
EXPORT_SYMBOL(proc_root);
...@@ -96,7 +96,6 @@ struct vmcore { ...@@ -96,7 +96,6 @@ struct vmcore {
#ifdef CONFIG_PROC_FS #ifdef CONFIG_PROC_FS
extern struct proc_dir_entry proc_root;
extern struct proc_dir_entry *proc_root_kcore; extern struct proc_dir_entry *proc_root_kcore;
extern spinlock_t proc_subdir_lock; extern spinlock_t proc_subdir_lock;
...@@ -243,8 +242,6 @@ struct tty_driver; ...@@ -243,8 +242,6 @@ struct tty_driver;
static inline void proc_tty_register_driver(struct tty_driver *driver) {}; static inline void proc_tty_register_driver(struct tty_driver *driver) {};
static inline void proc_tty_unregister_driver(struct tty_driver *driver) {}; static inline void proc_tty_unregister_driver(struct tty_driver *driver) {};
extern struct proc_dir_entry proc_root;
static inline int pid_ns_prepare_proc(struct pid_namespace *ns) static inline int pid_ns_prepare_proc(struct pid_namespace *ns)
{ {
return 0; return 0;
......
...@@ -79,8 +79,7 @@ static int __init ikconfig_init(void) ...@@ -79,8 +79,7 @@ static int __init ikconfig_init(void)
struct proc_dir_entry *entry; struct proc_dir_entry *entry;
/* create the current config file */ /* create the current config file */
entry = create_proc_entry("config.gz", S_IFREG | S_IRUGO, entry = create_proc_entry("config.gz", S_IFREG | S_IRUGO, NULL);
&proc_root);
if (!entry) if (!entry)
return -ENOMEM; return -ENOMEM;
...@@ -95,7 +94,7 @@ static int __init ikconfig_init(void) ...@@ -95,7 +94,7 @@ static int __init ikconfig_init(void)
static void __exit ikconfig_cleanup(void) static void __exit ikconfig_cleanup(void)
{ {
remove_proc_entry("config.gz", &proc_root); remove_proc_entry("config.gz", NULL);
} }
module_init(ikconfig_init); module_init(ikconfig_init);
......
...@@ -544,7 +544,7 @@ int __init snd_info_init(void) ...@@ -544,7 +544,7 @@ int __init snd_info_init(void)
{ {
struct proc_dir_entry *p; struct proc_dir_entry *p;
p = snd_create_proc_entry("asound", S_IFDIR | S_IRUGO | S_IXUGO, &proc_root); p = snd_create_proc_entry("asound", S_IFDIR | S_IRUGO | S_IXUGO, NULL);
if (p == NULL) if (p == NULL)
return -ENOMEM; return -ENOMEM;
snd_proc_root = p; snd_proc_root = p;
...@@ -594,7 +594,7 @@ int __exit snd_info_done(void) ...@@ -594,7 +594,7 @@ int __exit snd_info_done(void)
#ifdef CONFIG_SND_OSSEMUL #ifdef CONFIG_SND_OSSEMUL
snd_info_free_entry(snd_oss_root); snd_info_free_entry(snd_oss_root);
#endif #endif
snd_remove_proc_entry(&proc_root, snd_proc_root); snd_remove_proc_entry(NULL, snd_proc_root);
} }
return 0; return 0;
} }
......
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