diff options
author | Andreas Gruenbacher <agruen@linbit.com> | 2011-03-16 10:31:39 -0400 |
---|---|---|
committer | Philipp Reisner <philipp.reisner@linbit.com> | 2012-11-08 10:44:43 -0500 |
commit | 99920dc5c5fe52182fe922aa70330861e2b6418b (patch) | |
tree | 51890fb25f704bf1cbea56695d794be8166deb81 /drivers/block/drbd/drbd_state.c | |
parent | b2f0ab62ecfe8711fefb82223b40430f8141a949 (diff) |
drbd: Make all worker callbacks return 0 upon success and an error code otherwise
Signed-off-by: Philipp Reisner <philipp.reisner@linbit.com>
Signed-off-by: Lars Ellenberg <lars.ellenberg@linbit.com>
Diffstat (limited to 'drivers/block/drbd/drbd_state.c')
-rw-r--r-- | drivers/block/drbd/drbd_state.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/drivers/block/drbd/drbd_state.c b/drivers/block/drbd/drbd_state.c index c6d972009707..79e4a90a6c7e 100644 --- a/drivers/block/drbd/drbd_state.c +++ b/drivers/block/drbd/drbd_state.c | |||
@@ -37,7 +37,7 @@ struct after_state_chg_work { | |||
37 | struct completion *done; | 37 | struct completion *done; |
38 | }; | 38 | }; |
39 | 39 | ||
40 | static long w_after_state_ch(struct drbd_work *w, int unused); | 40 | static int w_after_state_ch(struct drbd_work *w, int unused); |
41 | static void after_state_ch(struct drbd_conf *mdev, union drbd_state os, | 41 | static void after_state_ch(struct drbd_conf *mdev, union drbd_state os, |
42 | union drbd_state ns, enum chg_state_flags flags); | 42 | union drbd_state ns, enum chg_state_flags flags); |
43 | static void after_all_state_ch(struct drbd_tconn *tconn); | 43 | static void after_all_state_ch(struct drbd_tconn *tconn); |
@@ -918,7 +918,7 @@ __drbd_set_state(struct drbd_conf *mdev, union drbd_state ns, | |||
918 | return rv; | 918 | return rv; |
919 | } | 919 | } |
920 | 920 | ||
921 | static long w_after_state_ch(struct drbd_work *w, int unused) | 921 | static int w_after_state_ch(struct drbd_work *w, int unused) |
922 | { | 922 | { |
923 | struct after_state_chg_work *ascw = | 923 | struct after_state_chg_work *ascw = |
924 | container_of(w, struct after_state_chg_work, w); | 924 | container_of(w, struct after_state_chg_work, w); |
@@ -931,7 +931,7 @@ static long w_after_state_ch(struct drbd_work *w, int unused) | |||
931 | } | 931 | } |
932 | kfree(ascw); | 932 | kfree(ascw); |
933 | 933 | ||
934 | return 1; | 934 | return 0; |
935 | } | 935 | } |
936 | 936 | ||
937 | static void abw_start_sync(struct drbd_conf *mdev, int rv) | 937 | static void abw_start_sync(struct drbd_conf *mdev, int rv) |
@@ -1289,7 +1289,7 @@ static void after_all_state_ch(struct drbd_tconn *tconn) | |||
1289 | } | 1289 | } |
1290 | } | 1290 | } |
1291 | 1291 | ||
1292 | static long w_after_conn_state_ch(struct drbd_work *w, int unused) | 1292 | static int w_after_conn_state_ch(struct drbd_work *w, int unused) |
1293 | { | 1293 | { |
1294 | struct after_conn_state_chg_work *acscw = | 1294 | struct after_conn_state_chg_work *acscw = |
1295 | container_of(w, struct after_conn_state_chg_work, w); | 1295 | container_of(w, struct after_conn_state_chg_work, w); |
@@ -1306,7 +1306,7 @@ static long w_after_conn_state_ch(struct drbd_work *w, int unused) | |||
1306 | //conn_err(tconn, STATE_FMT, STATE_ARGS("nms", nms)); | 1306 | //conn_err(tconn, STATE_FMT, STATE_ARGS("nms", nms)); |
1307 | after_all_state_ch(tconn); | 1307 | after_all_state_ch(tconn); |
1308 | 1308 | ||
1309 | return 1; | 1309 | return 0; |
1310 | } | 1310 | } |
1311 | 1311 | ||
1312 | static void print_conn_state_change(struct drbd_tconn *tconn, enum drbd_conns oc, enum drbd_conns nc) | 1312 | static void print_conn_state_change(struct drbd_tconn *tconn, enum drbd_conns oc, enum drbd_conns nc) |