diff options
author | Philipp Reisner <philipp.reisner@linbit.com> | 2010-08-25 04:47:17 -0400 |
---|---|---|
committer | Philipp Reisner <philipp.reisner@linbit.com> | 2010-10-23 07:02:34 -0400 |
commit | 650789c87f16dcdf1dd0a67ac7461b7537534855 (patch) | |
tree | 47e8ffc3b45542fd4d447a8c902b977c108fc00f /drivers | |
parent | a8a4e51e6965db84d2af041370ea2ab6232aa4f1 (diff) |
drbd: Removed checks for REQ_HARDBARRIER on incomming BIOs
Signed-off-by: Philipp Reisner <philipp.reisner@linbit.com>
Signed-off-by: Lars Ellenberg <lars.ellenberg@linbit.com>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/block/drbd/drbd_int.h | 1 | ||||
-rw-r--r-- | drivers/block/drbd/drbd_main.c | 5 | ||||
-rw-r--r-- | drivers/block/drbd/drbd_req.c | 14 |
3 files changed, 0 insertions, 20 deletions
diff --git a/drivers/block/drbd/drbd_int.h b/drivers/block/drbd/drbd_int.h index 575bfba1b0da..0c527d281622 100644 --- a/drivers/block/drbd/drbd_int.h +++ b/drivers/block/drbd/drbd_int.h | |||
@@ -832,7 +832,6 @@ enum { | |||
832 | * Gets cleared when the state.conn | 832 | * Gets cleared when the state.conn |
833 | * goes into C_CONNECTED state. */ | 833 | * goes into C_CONNECTED state. */ |
834 | WRITE_BM_AFTER_RESYNC, /* A kmalloc() during resync failed */ | 834 | WRITE_BM_AFTER_RESYNC, /* A kmalloc() during resync failed */ |
835 | NO_BARRIER_SUPP, /* underlying block device doesn't implement barriers */ | ||
836 | CONSIDER_RESYNC, | 835 | CONSIDER_RESYNC, |
837 | 836 | ||
838 | MD_NO_FUA, /* Users wants us to not use FUA/FLUSH on meta data dev */ | 837 | MD_NO_FUA, /* Users wants us to not use FUA/FLUSH on meta data dev */ |
diff --git a/drivers/block/drbd/drbd_main.c b/drivers/block/drbd/drbd_main.c index 8e0d707df23d..d7072bf0630d 100644 --- a/drivers/block/drbd/drbd_main.c +++ b/drivers/block/drbd/drbd_main.c | |||
@@ -2788,11 +2788,6 @@ void drbd_init_set_defaults(struct drbd_conf *mdev) | |||
2788 | 2788 | ||
2789 | drbd_set_defaults(mdev); | 2789 | drbd_set_defaults(mdev); |
2790 | 2790 | ||
2791 | /* for now, we do NOT yet support it, | ||
2792 | * even though we start some framework | ||
2793 | * to eventually support barriers */ | ||
2794 | set_bit(NO_BARRIER_SUPP, &mdev->flags); | ||
2795 | |||
2796 | atomic_set(&mdev->ap_bio_cnt, 0); | 2791 | atomic_set(&mdev->ap_bio_cnt, 0); |
2797 | atomic_set(&mdev->ap_pending_cnt, 0); | 2792 | atomic_set(&mdev->ap_pending_cnt, 0); |
2798 | atomic_set(&mdev->rs_pending_cnt, 0); | 2793 | atomic_set(&mdev->rs_pending_cnt, 0); |
diff --git a/drivers/block/drbd/drbd_req.c b/drivers/block/drbd/drbd_req.c index 5c2254853559..11a75d32a2e2 100644 --- a/drivers/block/drbd/drbd_req.c +++ b/drivers/block/drbd/drbd_req.c | |||
@@ -1032,20 +1032,6 @@ int drbd_make_request_26(struct request_queue *q, struct bio *bio) | |||
1032 | return 0; | 1032 | return 0; |
1033 | } | 1033 | } |
1034 | 1034 | ||
1035 | /* Reject barrier requests if we know the underlying device does | ||
1036 | * not support them. | ||
1037 | * XXX: Need to get this info from peer as well some how so we | ||
1038 | * XXX: reject if EITHER side/data/metadata area does not support them. | ||
1039 | * | ||
1040 | * because of those XXX, this is not yet enabled, | ||
1041 | * i.e. in drbd_init_set_defaults we set the NO_BARRIER_SUPP bit. | ||
1042 | */ | ||
1043 | if (unlikely(bio->bi_rw & REQ_HARDBARRIER) && test_bit(NO_BARRIER_SUPP, &mdev->flags)) { | ||
1044 | /* dev_warn(DEV, "Rejecting barrier request as underlying device does not support\n"); */ | ||
1045 | bio_endio(bio, -EOPNOTSUPP); | ||
1046 | return 0; | ||
1047 | } | ||
1048 | |||
1049 | /* | 1035 | /* |
1050 | * what we "blindly" assume: | 1036 | * what we "blindly" assume: |
1051 | */ | 1037 | */ |