diff options
| author | Andreas Gruenbacher <agruen@linbit.com> | 2014-02-27 14:49:54 -0500 |
|---|---|---|
| committer | Philipp Reisner <philipp.reisner@linbit.com> | 2014-07-10 12:34:58 -0400 |
| commit | caa3db0e14cc301f07e758f4cadc36d4dead145a (patch) | |
| tree | 5e0fafd49d110cb48525cac0b0739b76cbf320bf /drivers/block | |
| parent | 8ce953aa39e2bfd66036a27abdf761c2cb93f02c (diff) | |
drbd: Remove unnecessary/unused code
Get rid of dump_stack() debug statements.
There is no point whatsoever in registering and unregistering a reboot
notifier that doesn't do anything.
The intention was to switch to an "emergency read-only" mode,
so we won't have to resync the full activity log just because
we had been Primary before the reboot.
Once we have that implemented, we may re-introduce the reboot notifier.
Signed-off-by: Philipp Reisner <philipp.reisner@linbit.com>
Signed-off-by: Lars Ellenberg <lars.ellenberg@linbit.com>
Diffstat (limited to 'drivers/block')
| -rw-r--r-- | drivers/block/drbd/drbd_int.h | 1 | ||||
| -rw-r--r-- | drivers/block/drbd/drbd_main.c | 18 | ||||
| -rw-r--r-- | drivers/block/drbd/drbd_proc.c | 2 |
3 files changed, 1 insertions, 20 deletions
diff --git a/drivers/block/drbd/drbd_int.h b/drivers/block/drbd/drbd_int.h index 72b500176083..e306a22a60f1 100644 --- a/drivers/block/drbd/drbd_int.h +++ b/drivers/block/drbd/drbd_int.h | |||
| @@ -1468,7 +1468,6 @@ static inline void drbd_generic_make_request(struct drbd_device *device, | |||
| 1468 | printk(KERN_ERR "drbd%d: drbd_generic_make_request: " | 1468 | printk(KERN_ERR "drbd%d: drbd_generic_make_request: " |
| 1469 | "bio->bi_bdev == NULL\n", | 1469 | "bio->bi_bdev == NULL\n", |
| 1470 | device_to_minor(device)); | 1470 | device_to_minor(device)); |
| 1471 | dump_stack(); | ||
| 1472 | bio_endio(bio, -ENODEV); | 1471 | bio_endio(bio, -ENODEV); |
| 1473 | return; | 1472 | return; |
| 1474 | } | 1473 | } |
diff --git a/drivers/block/drbd/drbd_main.c b/drivers/block/drbd/drbd_main.c index 68ec774d52ea..7c060243ae46 100644 --- a/drivers/block/drbd/drbd_main.c +++ b/drivers/block/drbd/drbd_main.c | |||
| @@ -2125,20 +2125,6 @@ Enomem: | |||
| 2125 | return -ENOMEM; | 2125 | return -ENOMEM; |
| 2126 | } | 2126 | } |
| 2127 | 2127 | ||
| 2128 | static int drbd_notify_sys(struct notifier_block *this, unsigned long code, | ||
| 2129 | void *unused) | ||
| 2130 | { | ||
| 2131 | /* just so we have it. you never know what interesting things we | ||
| 2132 | * might want to do here some day... | ||
| 2133 | */ | ||
| 2134 | |||
| 2135 | return NOTIFY_DONE; | ||
| 2136 | } | ||
| 2137 | |||
| 2138 | static struct notifier_block drbd_notifier = { | ||
| 2139 | .notifier_call = drbd_notify_sys, | ||
| 2140 | }; | ||
| 2141 | |||
| 2142 | static void drbd_release_all_peer_reqs(struct drbd_device *device) | 2128 | static void drbd_release_all_peer_reqs(struct drbd_device *device) |
| 2143 | { | 2129 | { |
| 2144 | int rr; | 2130 | int rr; |
| @@ -2314,8 +2300,6 @@ static void drbd_cleanup(void) | |||
| 2314 | struct drbd_device *device; | 2300 | struct drbd_device *device; |
| 2315 | struct drbd_resource *resource, *tmp; | 2301 | struct drbd_resource *resource, *tmp; |
| 2316 | 2302 | ||
| 2317 | unregister_reboot_notifier(&drbd_notifier); | ||
| 2318 | |||
| 2319 | /* first remove proc, | 2303 | /* first remove proc, |
| 2320 | * drbdsetup uses it's presence to detect | 2304 | * drbdsetup uses it's presence to detect |
| 2321 | * whether DRBD is loaded. | 2305 | * whether DRBD is loaded. |
| @@ -2899,8 +2883,6 @@ static int __init drbd_init(void) | |||
| 2899 | return err; | 2883 | return err; |
| 2900 | } | 2884 | } |
| 2901 | 2885 | ||
| 2902 | register_reboot_notifier(&drbd_notifier); | ||
| 2903 | |||
| 2904 | /* | 2886 | /* |
| 2905 | * allocate all necessary structs | 2887 | * allocate all necessary structs |
| 2906 | */ | 2888 | */ |
diff --git a/drivers/block/drbd/drbd_proc.c b/drivers/block/drbd/drbd_proc.c index 46bb8dd7890d..886f6bef70dc 100644 --- a/drivers/block/drbd/drbd_proc.c +++ b/drivers/block/drbd/drbd_proc.c | |||
| @@ -299,7 +299,7 @@ static int drbd_proc_open(struct inode *inode, struct file *file) | |||
| 299 | int err; | 299 | int err; |
| 300 | 300 | ||
| 301 | if (try_module_get(THIS_MODULE)) { | 301 | if (try_module_get(THIS_MODULE)) { |
| 302 | err = single_open(file, drbd_seq_show, PDE_DATA(inode)); | 302 | err = single_open(file, drbd_seq_show, NULL); |
| 303 | if (err) | 303 | if (err) |
| 304 | module_put(THIS_MODULE); | 304 | module_put(THIS_MODULE); |
| 305 | return err; | 305 | return err; |
