Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 1 | /* |
| 2 | drbd_req.c |
| 3 | |
| 4 | This file is part of DRBD by Philipp Reisner and Lars Ellenberg. |
| 5 | |
| 6 | Copyright (C) 2001-2008, LINBIT Information Technologies GmbH. |
| 7 | Copyright (C) 1999-2008, Philipp Reisner <philipp.reisner@linbit.com>. |
| 8 | Copyright (C) 2002-2008, Lars Ellenberg <lars.ellenberg@linbit.com>. |
| 9 | |
| 10 | drbd is free software; you can redistribute it and/or modify |
| 11 | it under the terms of the GNU General Public License as published by |
| 12 | the Free Software Foundation; either version 2, or (at your option) |
| 13 | any later version. |
| 14 | |
| 15 | drbd is distributed in the hope that it will be useful, |
| 16 | but WITHOUT ANY WARRANTY; without even the implied warranty of |
| 17 | MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the |
| 18 | GNU General Public License for more details. |
| 19 | |
| 20 | You should have received a copy of the GNU General Public License |
| 21 | along with drbd; see the file COPYING. If not, write to |
| 22 | the Free Software Foundation, 675 Mass Ave, Cambridge, MA 02139, USA. |
| 23 | |
| 24 | */ |
| 25 | |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 26 | #include <linux/module.h> |
| 27 | |
| 28 | #include <linux/slab.h> |
| 29 | #include <linux/drbd.h> |
| 30 | #include "drbd_int.h" |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 31 | #include "drbd_req.h" |
| 32 | |
| 33 | |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 34 | static bool drbd_may_do_local_read(struct drbd_device *device, sector_t sector, int size); |
Philipp Reisner | 57bcb6c | 2011-12-03 11:18:56 +0100 | [diff] [blame] | 35 | |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 36 | /* Update disk stats at start of I/O request */ |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 37 | static void _drbd_start_io_acct(struct drbd_device *device, struct drbd_request *req) |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 38 | { |
Lars Ellenberg | 6d9febe | 2013-03-19 18:16:50 +0100 | [diff] [blame] | 39 | const int rw = bio_data_dir(req->master_bio); |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 40 | int cpu; |
| 41 | cpu = part_stat_lock(); |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 42 | part_round_stats(cpu, &device->vdisk->part0); |
| 43 | part_stat_inc(cpu, &device->vdisk->part0, ios[rw]); |
| 44 | part_stat_add(cpu, &device->vdisk->part0, sectors[rw], req->i.size >> 9); |
Philipp Reisner | 376694a | 2011-11-07 10:54:28 +0100 | [diff] [blame] | 45 | (void) cpu; /* The macro invocations above want the cpu argument, I do not like |
| 46 | the compiler warning about cpu only assigned but never used... */ |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 47 | part_inc_in_flight(&device->vdisk->part0, rw); |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 48 | part_stat_unlock(); |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 49 | } |
| 50 | |
| 51 | /* Update disk stats when completing request upwards */ |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 52 | static void _drbd_end_io_acct(struct drbd_device *device, struct drbd_request *req) |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 53 | { |
| 54 | int rw = bio_data_dir(req->master_bio); |
| 55 | unsigned long duration = jiffies - req->start_time; |
| 56 | int cpu; |
| 57 | cpu = part_stat_lock(); |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 58 | part_stat_add(cpu, &device->vdisk->part0, ticks[rw], duration); |
| 59 | part_round_stats(cpu, &device->vdisk->part0); |
| 60 | part_dec_in_flight(&device->vdisk->part0, rw); |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 61 | part_stat_unlock(); |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 62 | } |
| 63 | |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 64 | static struct drbd_request *drbd_req_new(struct drbd_device *device, |
Andreas Gruenbacher | 9e204cd | 2011-01-26 18:45:11 +0100 | [diff] [blame] | 65 | struct bio *bio_src) |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 66 | { |
Andreas Gruenbacher | 9e204cd | 2011-01-26 18:45:11 +0100 | [diff] [blame] | 67 | struct drbd_request *req; |
| 68 | |
| 69 | req = mempool_alloc(drbd_request_mempool, GFP_NOIO); |
| 70 | if (!req) |
| 71 | return NULL; |
| 72 | |
| 73 | drbd_req_make_private_bio(req, bio_src); |
| 74 | req->rq_state = bio_data_dir(bio_src) == WRITE ? RQ_WRITE : 0; |
Andreas Gruenbacher | 84b8c06 | 2011-07-28 15:27:51 +0200 | [diff] [blame] | 75 | req->device = device; |
Andreas Gruenbacher | 9e204cd | 2011-01-26 18:45:11 +0100 | [diff] [blame] | 76 | req->master_bio = bio_src; |
| 77 | req->epoch = 0; |
Andreas Gruenbacher | 5384064 | 2011-01-28 10:31:04 +0100 | [diff] [blame] | 78 | |
Andreas Gruenbacher | 9e204cd | 2011-01-26 18:45:11 +0100 | [diff] [blame] | 79 | drbd_clear_interval(&req->i); |
Kent Overstreet | 4f024f3 | 2013-10-11 15:44:27 -0700 | [diff] [blame] | 80 | req->i.sector = bio_src->bi_iter.bi_sector; |
| 81 | req->i.size = bio_src->bi_iter.bi_size; |
Andreas Gruenbacher | 5e47226 | 2011-01-27 14:42:51 +0100 | [diff] [blame] | 82 | req->i.local = true; |
Andreas Gruenbacher | 5384064 | 2011-01-28 10:31:04 +0100 | [diff] [blame] | 83 | req->i.waiting = false; |
| 84 | |
Andreas Gruenbacher | 9e204cd | 2011-01-26 18:45:11 +0100 | [diff] [blame] | 85 | INIT_LIST_HEAD(&req->tl_requests); |
| 86 | INIT_LIST_HEAD(&req->w.list); |
| 87 | |
Lars Ellenberg | a0d856d | 2012-01-24 17:19:42 +0100 | [diff] [blame] | 88 | /* one reference to be put by __drbd_make_request */ |
Lars Ellenberg | b406777 | 2012-01-24 16:58:11 +0100 | [diff] [blame] | 89 | atomic_set(&req->completion_ref, 1); |
Lars Ellenberg | a0d856d | 2012-01-24 17:19:42 +0100 | [diff] [blame] | 90 | /* one kref as long as completion_ref > 0 */ |
Lars Ellenberg | b406777 | 2012-01-24 16:58:11 +0100 | [diff] [blame] | 91 | kref_init(&req->kref); |
Andreas Gruenbacher | 9e204cd | 2011-01-26 18:45:11 +0100 | [diff] [blame] | 92 | return req; |
| 93 | } |
| 94 | |
Lars Ellenberg | 08d0dab | 2014-03-20 11:19:22 +0100 | [diff] [blame^] | 95 | static void drbd_remove_request_interval(struct rb_root *root, |
| 96 | struct drbd_request *req) |
| 97 | { |
| 98 | struct drbd_device *device = req->device; |
| 99 | struct drbd_interval *i = &req->i; |
| 100 | |
| 101 | drbd_remove_interval(root, i); |
| 102 | |
| 103 | /* Wake up any processes waiting for this request to complete. */ |
| 104 | if (i->waiting) |
| 105 | wake_up(&device->misc_wait); |
| 106 | } |
| 107 | |
Lars Ellenberg | 9a278a7 | 2012-07-24 10:12:36 +0200 | [diff] [blame] | 108 | void drbd_req_destroy(struct kref *kref) |
Andreas Gruenbacher | 9e204cd | 2011-01-26 18:45:11 +0100 | [diff] [blame] | 109 | { |
Lars Ellenberg | b406777 | 2012-01-24 16:58:11 +0100 | [diff] [blame] | 110 | struct drbd_request *req = container_of(kref, struct drbd_request, kref); |
Andreas Gruenbacher | 84b8c06 | 2011-07-28 15:27:51 +0200 | [diff] [blame] | 111 | struct drbd_device *device = req->device; |
Lars Ellenberg | a0d856d | 2012-01-24 17:19:42 +0100 | [diff] [blame] | 112 | const unsigned s = req->rq_state; |
| 113 | |
| 114 | if ((req->master_bio && !(s & RQ_POSTPONED)) || |
| 115 | atomic_read(&req->completion_ref) || |
| 116 | (s & RQ_LOCAL_PENDING) || |
| 117 | ((s & RQ_NET_MASK) && !(s & RQ_NET_DONE))) { |
Andreas Gruenbacher | d018017 | 2011-07-03 17:53:52 +0200 | [diff] [blame] | 118 | drbd_err(device, "drbd_req_destroy: Logic BUG rq_state = 0x%x, completion_ref = %d\n", |
Lars Ellenberg | a0d856d | 2012-01-24 17:19:42 +0100 | [diff] [blame] | 119 | s, atomic_read(&req->completion_ref)); |
| 120 | return; |
| 121 | } |
Philipp Reisner | 288f422 | 2010-05-27 15:07:43 +0200 | [diff] [blame] | 122 | |
| 123 | /* remove it from the transfer log. |
| 124 | * well, only if it had been there in the first |
| 125 | * place... if it had not (local only or conflicting |
| 126 | * and never sent), it should still be "empty" as |
| 127 | * initialized in drbd_req_new(), so we can list_del() it |
| 128 | * here unconditionally */ |
Lars Ellenberg | 2312f0b3 | 2011-11-24 10:36:25 +0100 | [diff] [blame] | 129 | list_del_init(&req->tl_requests); |
Philipp Reisner | 288f422 | 2010-05-27 15:07:43 +0200 | [diff] [blame] | 130 | |
Lars Ellenberg | 08d0dab | 2014-03-20 11:19:22 +0100 | [diff] [blame^] | 131 | /* finally remove the request from the conflict detection |
| 132 | * respective block_id verification interval tree. */ |
| 133 | if (!drbd_interval_empty(&req->i)) { |
| 134 | struct rb_root *root; |
| 135 | |
| 136 | if (s & RQ_WRITE) |
| 137 | root = &device->write_requests; |
| 138 | else |
| 139 | root = &device->read_requests; |
| 140 | drbd_remove_request_interval(root, req); |
| 141 | } else if (s & (RQ_NET_MASK & ~RQ_NET_DONE) && req->i.size != 0) |
| 142 | drbd_err(device, "drbd_req_destroy: Logic BUG: interval empty, but: rq_state=0x%x, sect=%llu, size=%u\n", |
| 143 | s, (unsigned long long)req->i.sector, req->i.size); |
| 144 | |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 145 | /* if it was a write, we may have to set the corresponding |
| 146 | * bit(s) out-of-sync first. If it had a local part, we need to |
| 147 | * release the reference to the activity log. */ |
Lars Ellenberg | b406777 | 2012-01-24 16:58:11 +0100 | [diff] [blame] | 148 | if (s & RQ_WRITE) { |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 149 | /* Set out-of-sync unless both OK flags are set |
| 150 | * (local only or remote failed). |
| 151 | * Other places where we set out-of-sync: |
| 152 | * READ with local io-error */ |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 153 | |
Lars Ellenberg | 70f17b6 | 2012-09-03 14:08:35 +0200 | [diff] [blame] | 154 | /* There is a special case: |
| 155 | * we may notice late that IO was suspended, |
| 156 | * and postpone, or schedule for retry, a write, |
| 157 | * before it even was submitted or sent. |
| 158 | * In that case we do not want to touch the bitmap at all. |
| 159 | */ |
| 160 | if ((s & (RQ_POSTPONED|RQ_LOCAL_MASK|RQ_NET_MASK)) != RQ_POSTPONED) { |
Philipp Reisner | d764401 | 2012-08-28 14:39:44 +0200 | [diff] [blame] | 161 | if (!(s & RQ_NET_OK) || !(s & RQ_LOCAL_OK)) |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 162 | drbd_set_out_of_sync(device, req->i.sector, req->i.size); |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 163 | |
Philipp Reisner | d764401 | 2012-08-28 14:39:44 +0200 | [diff] [blame] | 164 | if ((s & RQ_NET_OK) && (s & RQ_LOCAL_OK) && (s & RQ_NET_SIS)) |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 165 | drbd_set_in_sync(device, req->i.sector, req->i.size); |
Philipp Reisner | d764401 | 2012-08-28 14:39:44 +0200 | [diff] [blame] | 166 | } |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 167 | |
| 168 | /* one might be tempted to move the drbd_al_complete_io |
Andreas Gruenbacher | fcefa62 | 2011-02-17 16:46:59 +0100 | [diff] [blame] | 169 | * to the local io completion callback drbd_request_endio. |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 170 | * but, if this was a mirror write, we may only |
| 171 | * drbd_al_complete_io after this is RQ_NET_DONE, |
| 172 | * otherwise the extent could be dropped from the al |
| 173 | * before it has actually been written on the peer. |
| 174 | * if we crash before our peer knows about the request, |
| 175 | * but after the extent has been dropped from the al, |
| 176 | * we would forget to resync the corresponding extent. |
| 177 | */ |
Philipp Reisner | 76590cd | 2012-08-29 15:23:14 +0200 | [diff] [blame] | 178 | if (s & RQ_IN_ACT_LOG) { |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 179 | if (get_ldev_if_state(device, D_FAILED)) { |
| 180 | drbd_al_complete_io(device, &req->i); |
| 181 | put_ldev(device); |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 182 | } else if (__ratelimit(&drbd_ratelimit_state)) { |
Andreas Gruenbacher | d018017 | 2011-07-03 17:53:52 +0200 | [diff] [blame] | 183 | drbd_warn(device, "Should have called drbd_al_complete_io(, %llu, %u), " |
Lars Ellenberg | 181286a | 2011-03-31 15:18:56 +0200 | [diff] [blame] | 184 | "but my Disk seems to have failed :(\n", |
| 185 | (unsigned long long) req->i.sector, req->i.size); |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 186 | } |
| 187 | } |
| 188 | } |
| 189 | |
Lars Ellenberg | 9a278a7 | 2012-07-24 10:12:36 +0200 | [diff] [blame] | 190 | mempool_free(req, drbd_request_mempool); |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 191 | } |
| 192 | |
Andreas Gruenbacher | bde89a9 | 2011-05-30 16:32:41 +0200 | [diff] [blame] | 193 | static void wake_all_senders(struct drbd_connection *connection) |
| 194 | { |
| 195 | wake_up(&connection->sender_work.q_wait); |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 196 | } |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 197 | |
Lars Ellenberg | b6dd1a8 | 2011-11-28 15:04:49 +0100 | [diff] [blame] | 198 | /* must hold resource->req_lock */ |
Andreas Gruenbacher | bde89a9 | 2011-05-30 16:32:41 +0200 | [diff] [blame] | 199 | void start_new_tl_epoch(struct drbd_connection *connection) |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 200 | { |
Lars Ellenberg | 99b4d8f | 2012-08-07 06:42:09 +0200 | [diff] [blame] | 201 | /* no point closing an epoch, if it is empty, anyways. */ |
Andreas Gruenbacher | bde89a9 | 2011-05-30 16:32:41 +0200 | [diff] [blame] | 202 | if (connection->current_tle_writes == 0) |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 203 | return; |
| 204 | |
Andreas Gruenbacher | bde89a9 | 2011-05-30 16:32:41 +0200 | [diff] [blame] | 205 | connection->current_tle_writes = 0; |
| 206 | atomic_inc(&connection->current_tle_nr); |
| 207 | wake_all_senders(connection); |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 208 | } |
| 209 | |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 210 | void complete_master_bio(struct drbd_device *device, |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 211 | struct bio_and_error *m) |
| 212 | { |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 213 | bio_endio(m->bio, m->error); |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 214 | dec_ap_bio(device); |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 215 | } |
| 216 | |
Andreas Gruenbacher | 5384064 | 2011-01-28 10:31:04 +0100 | [diff] [blame] | 217 | |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 218 | /* Helper for __req_mod(). |
| 219 | * Set m->bio to the master bio, if it is fit to be completed, |
| 220 | * or leave it alone (it is initialized to NULL in __req_mod), |
| 221 | * if it has already been completed, or cannot be completed yet. |
| 222 | * If m->bio is set, the error status to be returned is placed in m->error. |
| 223 | */ |
Lars Ellenberg | 6870ca6 | 2012-03-26 17:02:45 +0200 | [diff] [blame] | 224 | static |
Lars Ellenberg | a0d856d | 2012-01-24 17:19:42 +0100 | [diff] [blame] | 225 | void drbd_req_complete(struct drbd_request *req, struct bio_and_error *m) |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 226 | { |
Lars Ellenberg | a0d856d | 2012-01-24 17:19:42 +0100 | [diff] [blame] | 227 | const unsigned s = req->rq_state; |
Andreas Gruenbacher | 84b8c06 | 2011-07-28 15:27:51 +0200 | [diff] [blame] | 228 | struct drbd_device *device = req->device; |
Lars Ellenberg | a0d856d | 2012-01-24 17:19:42 +0100 | [diff] [blame] | 229 | int rw; |
| 230 | int error, ok; |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 231 | |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 232 | /* we must not complete the master bio, while it is |
| 233 | * still being processed by _drbd_send_zc_bio (drbd_send_dblock) |
| 234 | * not yet acknowledged by the peer |
| 235 | * not yet completed by the local io subsystem |
| 236 | * these flags may get cleared in any order by |
| 237 | * the worker, |
| 238 | * the receiver, |
| 239 | * the bio_endio completion callbacks. |
| 240 | */ |
Lars Ellenberg | a0d856d | 2012-01-24 17:19:42 +0100 | [diff] [blame] | 241 | if ((s & RQ_LOCAL_PENDING && !(s & RQ_LOCAL_ABORTED)) || |
| 242 | (s & RQ_NET_QUEUED) || (s & RQ_NET_PENDING) || |
| 243 | (s & RQ_COMPLETION_SUSP)) { |
Andreas Gruenbacher | d018017 | 2011-07-03 17:53:52 +0200 | [diff] [blame] | 244 | drbd_err(device, "drbd_req_complete: Logic BUG rq_state = 0x%x\n", s); |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 245 | return; |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 246 | } |
Lars Ellenberg | a0d856d | 2012-01-24 17:19:42 +0100 | [diff] [blame] | 247 | |
| 248 | if (!req->master_bio) { |
Andreas Gruenbacher | d018017 | 2011-07-03 17:53:52 +0200 | [diff] [blame] | 249 | drbd_err(device, "drbd_req_complete: Logic BUG, master_bio == NULL!\n"); |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 250 | return; |
Lars Ellenberg | a0d856d | 2012-01-24 17:19:42 +0100 | [diff] [blame] | 251 | } |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 252 | |
Lars Ellenberg | a0d856d | 2012-01-24 17:19:42 +0100 | [diff] [blame] | 253 | rw = bio_rw(req->master_bio); |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 254 | |
Lars Ellenberg | a0d856d | 2012-01-24 17:19:42 +0100 | [diff] [blame] | 255 | /* |
| 256 | * figure out whether to report success or failure. |
| 257 | * |
| 258 | * report success when at least one of the operations succeeded. |
| 259 | * or, to put the other way, |
| 260 | * only report failure, when both operations failed. |
| 261 | * |
| 262 | * what to do about the failures is handled elsewhere. |
| 263 | * what we need to do here is just: complete the master_bio. |
| 264 | * |
| 265 | * local completion error, if any, has been stored as ERR_PTR |
| 266 | * in private_bio within drbd_request_endio. |
| 267 | */ |
| 268 | ok = (s & RQ_LOCAL_OK) || (s & RQ_NET_OK); |
| 269 | error = PTR_ERR(req->private_bio); |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 270 | |
Lars Ellenberg | a0d856d | 2012-01-24 17:19:42 +0100 | [diff] [blame] | 271 | /* Before we can signal completion to the upper layers, |
| 272 | * we may need to close the current transfer log epoch. |
| 273 | * We are within the request lock, so we can simply compare |
| 274 | * the request epoch number with the current transfer log |
| 275 | * epoch number. If they match, increase the current_tle_nr, |
| 276 | * and reset the transfer log epoch write_cnt. |
| 277 | */ |
| 278 | if (rw == WRITE && |
Andreas Gruenbacher | a6b32bc | 2011-05-31 14:33:49 +0200 | [diff] [blame] | 279 | req->epoch == atomic_read(&first_peer_device(device)->connection->current_tle_nr)) |
| 280 | start_new_tl_epoch(first_peer_device(device)->connection); |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 281 | |
Lars Ellenberg | a0d856d | 2012-01-24 17:19:42 +0100 | [diff] [blame] | 282 | /* Update disk stats */ |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 283 | _drbd_end_io_acct(device, req); |
Lars Ellenberg | a0d856d | 2012-01-24 17:19:42 +0100 | [diff] [blame] | 284 | |
| 285 | /* If READ failed, |
| 286 | * have it be pushed back to the retry work queue, |
| 287 | * so it will re-enter __drbd_make_request(), |
| 288 | * and be re-assigned to a suitable local or remote path, |
| 289 | * or failed if we do not have access to good data anymore. |
| 290 | * |
| 291 | * Unless it was failed early by __drbd_make_request(), |
| 292 | * because no path was available, in which case |
| 293 | * it was not even added to the transfer_log. |
| 294 | * |
| 295 | * READA may fail, and will not be retried. |
| 296 | * |
| 297 | * WRITE should have used all available paths already. |
| 298 | */ |
| 299 | if (!ok && rw == READ && !list_empty(&req->tl_requests)) |
| 300 | req->rq_state |= RQ_POSTPONED; |
| 301 | |
| 302 | if (!(req->rq_state & RQ_POSTPONED)) { |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 303 | m->error = ok ? 0 : (error ?: -EIO); |
| 304 | m->bio = req->master_bio; |
| 305 | req->master_bio = NULL; |
Lars Ellenberg | 08d0dab | 2014-03-20 11:19:22 +0100 | [diff] [blame^] | 306 | /* We leave it in the tree, to be able to verify later |
| 307 | * write-acks in protocol != C during resync. |
| 308 | * But we mark it as "complete", so it won't be counted as |
| 309 | * conflict in a multi-primary setup. */ |
| 310 | req->i.completed = true; |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 311 | } |
Lars Ellenberg | 08d0dab | 2014-03-20 11:19:22 +0100 | [diff] [blame^] | 312 | |
| 313 | if (req->i.waiting) |
| 314 | wake_up(&device->misc_wait); |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 315 | } |
| 316 | |
Lars Ellenberg | a0d856d | 2012-01-24 17:19:42 +0100 | [diff] [blame] | 317 | static int drbd_req_put_completion_ref(struct drbd_request *req, struct bio_and_error *m, int put) |
Philipp Reisner | cfa0341 | 2010-06-23 17:18:51 +0200 | [diff] [blame] | 318 | { |
Andreas Gruenbacher | 84b8c06 | 2011-07-28 15:27:51 +0200 | [diff] [blame] | 319 | struct drbd_device *device = req->device; |
Andreas Gruenbacher | 0b0ba1e | 2011-06-27 16:23:33 +0200 | [diff] [blame] | 320 | D_ASSERT(device, m || (req->rq_state & RQ_POSTPONED)); |
Philipp Reisner | cfa0341 | 2010-06-23 17:18:51 +0200 | [diff] [blame] | 321 | |
Lars Ellenberg | a0d856d | 2012-01-24 17:19:42 +0100 | [diff] [blame] | 322 | if (!atomic_sub_and_test(put, &req->completion_ref)) |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 323 | return 0; |
| 324 | |
Lars Ellenberg | a0d856d | 2012-01-24 17:19:42 +0100 | [diff] [blame] | 325 | drbd_req_complete(req, m); |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 326 | |
Lars Ellenberg | 9a278a7 | 2012-07-24 10:12:36 +0200 | [diff] [blame] | 327 | if (req->rq_state & RQ_POSTPONED) { |
| 328 | /* don't destroy the req object just yet, |
| 329 | * but queue it for retry */ |
| 330 | drbd_restart_request(req); |
| 331 | return 0; |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 332 | } |
| 333 | |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 334 | return 1; |
| 335 | } |
| 336 | |
Lars Ellenberg | a0d856d | 2012-01-24 17:19:42 +0100 | [diff] [blame] | 337 | /* I'd like this to be the only place that manipulates |
| 338 | * req->completion_ref and req->kref. */ |
| 339 | static void mod_rq_state(struct drbd_request *req, struct bio_and_error *m, |
| 340 | int clear, int set) |
| 341 | { |
Andreas Gruenbacher | 84b8c06 | 2011-07-28 15:27:51 +0200 | [diff] [blame] | 342 | struct drbd_device *device = req->device; |
Lars Ellenberg | a0d856d | 2012-01-24 17:19:42 +0100 | [diff] [blame] | 343 | unsigned s = req->rq_state; |
| 344 | int c_put = 0; |
| 345 | int k_put = 0; |
| 346 | |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 347 | if (drbd_suspended(device) && !((s | clear) & RQ_COMPLETION_SUSP)) |
Philipp Reisner | 5af2e8c | 2012-08-14 11:28:52 +0200 | [diff] [blame] | 348 | set |= RQ_COMPLETION_SUSP; |
| 349 | |
Lars Ellenberg | a0d856d | 2012-01-24 17:19:42 +0100 | [diff] [blame] | 350 | /* apply */ |
| 351 | |
| 352 | req->rq_state &= ~clear; |
| 353 | req->rq_state |= set; |
| 354 | |
| 355 | /* no change? */ |
| 356 | if (req->rq_state == s) |
| 357 | return; |
| 358 | |
| 359 | /* intent: get references */ |
| 360 | |
| 361 | if (!(s & RQ_LOCAL_PENDING) && (set & RQ_LOCAL_PENDING)) |
| 362 | atomic_inc(&req->completion_ref); |
| 363 | |
| 364 | if (!(s & RQ_NET_PENDING) && (set & RQ_NET_PENDING)) { |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 365 | inc_ap_pending(device); |
Lars Ellenberg | a0d856d | 2012-01-24 17:19:42 +0100 | [diff] [blame] | 366 | atomic_inc(&req->completion_ref); |
| 367 | } |
| 368 | |
| 369 | if (!(s & RQ_NET_QUEUED) && (set & RQ_NET_QUEUED)) |
| 370 | atomic_inc(&req->completion_ref); |
| 371 | |
| 372 | if (!(s & RQ_EXP_BARR_ACK) && (set & RQ_EXP_BARR_ACK)) |
| 373 | kref_get(&req->kref); /* wait for the DONE */ |
| 374 | |
| 375 | if (!(s & RQ_NET_SENT) && (set & RQ_NET_SENT)) |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 376 | atomic_add(req->i.size >> 9, &device->ap_in_flight); |
Lars Ellenberg | a0d856d | 2012-01-24 17:19:42 +0100 | [diff] [blame] | 377 | |
Philipp Reisner | 5af2e8c | 2012-08-14 11:28:52 +0200 | [diff] [blame] | 378 | if (!(s & RQ_COMPLETION_SUSP) && (set & RQ_COMPLETION_SUSP)) |
| 379 | atomic_inc(&req->completion_ref); |
| 380 | |
Lars Ellenberg | a0d856d | 2012-01-24 17:19:42 +0100 | [diff] [blame] | 381 | /* progress: put references */ |
| 382 | |
| 383 | if ((s & RQ_COMPLETION_SUSP) && (clear & RQ_COMPLETION_SUSP)) |
| 384 | ++c_put; |
| 385 | |
| 386 | if (!(s & RQ_LOCAL_ABORTED) && (set & RQ_LOCAL_ABORTED)) { |
Andreas Gruenbacher | 0b0ba1e | 2011-06-27 16:23:33 +0200 | [diff] [blame] | 387 | D_ASSERT(device, req->rq_state & RQ_LOCAL_PENDING); |
Lars Ellenberg | a0d856d | 2012-01-24 17:19:42 +0100 | [diff] [blame] | 388 | /* local completion may still come in later, |
| 389 | * we need to keep the req object around. */ |
| 390 | kref_get(&req->kref); |
| 391 | ++c_put; |
| 392 | } |
| 393 | |
| 394 | if ((s & RQ_LOCAL_PENDING) && (clear & RQ_LOCAL_PENDING)) { |
| 395 | if (req->rq_state & RQ_LOCAL_ABORTED) |
| 396 | ++k_put; |
| 397 | else |
| 398 | ++c_put; |
| 399 | } |
| 400 | |
| 401 | if ((s & RQ_NET_PENDING) && (clear & RQ_NET_PENDING)) { |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 402 | dec_ap_pending(device); |
Lars Ellenberg | a0d856d | 2012-01-24 17:19:42 +0100 | [diff] [blame] | 403 | ++c_put; |
| 404 | } |
| 405 | |
| 406 | if ((s & RQ_NET_QUEUED) && (clear & RQ_NET_QUEUED)) |
| 407 | ++c_put; |
| 408 | |
| 409 | if ((s & RQ_EXP_BARR_ACK) && !(s & RQ_NET_DONE) && (set & RQ_NET_DONE)) { |
| 410 | if (req->rq_state & RQ_NET_SENT) |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 411 | atomic_sub(req->i.size >> 9, &device->ap_in_flight); |
Lars Ellenberg | a0d856d | 2012-01-24 17:19:42 +0100 | [diff] [blame] | 412 | ++k_put; |
| 413 | } |
| 414 | |
| 415 | /* potentially complete and destroy */ |
| 416 | |
| 417 | if (k_put || c_put) { |
| 418 | /* Completion does it's own kref_put. If we are going to |
| 419 | * kref_sub below, we need req to be still around then. */ |
| 420 | int at_least = k_put + !!c_put; |
| 421 | int refcount = atomic_read(&req->kref.refcount); |
| 422 | if (refcount < at_least) |
Andreas Gruenbacher | d018017 | 2011-07-03 17:53:52 +0200 | [diff] [blame] | 423 | drbd_err(device, |
Lars Ellenberg | a0d856d | 2012-01-24 17:19:42 +0100 | [diff] [blame] | 424 | "mod_rq_state: Logic BUG: %x -> %x: refcount = %d, should be >= %d\n", |
| 425 | s, req->rq_state, refcount, at_least); |
| 426 | } |
| 427 | |
| 428 | /* If we made progress, retry conflicting peer requests, if any. */ |
| 429 | if (req->i.waiting) |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 430 | wake_up(&device->misc_wait); |
Lars Ellenberg | a0d856d | 2012-01-24 17:19:42 +0100 | [diff] [blame] | 431 | |
| 432 | if (c_put) |
| 433 | k_put += drbd_req_put_completion_ref(req, m, c_put); |
| 434 | if (k_put) |
| 435 | kref_sub(&req->kref, k_put, drbd_req_destroy); |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 436 | } |
| 437 | |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 438 | static void drbd_report_io_error(struct drbd_device *device, struct drbd_request *req) |
Lars Ellenberg | ccae786 | 2012-09-26 14:07:04 +0200 | [diff] [blame] | 439 | { |
| 440 | char b[BDEVNAME_SIZE]; |
| 441 | |
Lars Ellenberg | 42839f6 | 2012-09-27 15:19:38 +0200 | [diff] [blame] | 442 | if (!__ratelimit(&drbd_ratelimit_state)) |
Lars Ellenberg | ccae786 | 2012-09-26 14:07:04 +0200 | [diff] [blame] | 443 | return; |
| 444 | |
Andreas Gruenbacher | d018017 | 2011-07-03 17:53:52 +0200 | [diff] [blame] | 445 | drbd_warn(device, "local %s IO error sector %llu+%u on %s\n", |
Lars Ellenberg | ccae786 | 2012-09-26 14:07:04 +0200 | [diff] [blame] | 446 | (req->rq_state & RQ_WRITE) ? "WRITE" : "READ", |
Lars Ellenberg | 42839f6 | 2012-09-27 15:19:38 +0200 | [diff] [blame] | 447 | (unsigned long long)req->i.sector, |
| 448 | req->i.size >> 9, |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 449 | bdevname(device->ldev->backing_bdev, b)); |
Lars Ellenberg | ccae786 | 2012-09-26 14:07:04 +0200 | [diff] [blame] | 450 | } |
| 451 | |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 452 | /* obviously this could be coded as many single functions |
| 453 | * instead of one huge switch, |
| 454 | * or by putting the code directly in the respective locations |
| 455 | * (as it has been before). |
| 456 | * |
| 457 | * but having it this way |
| 458 | * enforces that it is all in this one place, where it is easier to audit, |
| 459 | * it makes it obvious that whatever "event" "happens" to a request should |
| 460 | * happen "atomically" within the req_lock, |
| 461 | * and it enforces that we have to think in a very structured manner |
| 462 | * about the "events" that may happen to a request during its life time ... |
| 463 | */ |
Philipp Reisner | 2a80699 | 2010-06-09 14:07:43 +0200 | [diff] [blame] | 464 | int __req_mod(struct drbd_request *req, enum drbd_req_event what, |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 465 | struct bio_and_error *m) |
| 466 | { |
Lars Ellenberg | 44a4d55 | 2013-11-22 12:40:58 +0100 | [diff] [blame] | 467 | struct drbd_device *const device = req->device; |
| 468 | struct drbd_peer_device *const peer_device = first_peer_device(device); |
| 469 | struct drbd_connection *const connection = peer_device ? peer_device->connection : NULL; |
Philipp Reisner | 44ed167 | 2011-04-19 17:10:19 +0200 | [diff] [blame] | 470 | struct net_conf *nc; |
Philipp Reisner | 303d144 | 2011-04-13 16:24:47 -0700 | [diff] [blame] | 471 | int p, rv = 0; |
Andreas Gruenbacher | 7be8da0 | 2011-02-22 02:15:32 +0100 | [diff] [blame] | 472 | |
| 473 | if (m) |
| 474 | m->bio = NULL; |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 475 | |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 476 | switch (what) { |
| 477 | default: |
Andreas Gruenbacher | d018017 | 2011-07-03 17:53:52 +0200 | [diff] [blame] | 478 | drbd_err(device, "LOGIC BUG in %s:%u\n", __FILE__ , __LINE__); |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 479 | break; |
| 480 | |
| 481 | /* does not happen... |
| 482 | * initialization done in drbd_req_new |
Andreas Gruenbacher | 8554df1 | 2011-01-25 15:37:43 +0100 | [diff] [blame] | 483 | case CREATED: |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 484 | break; |
| 485 | */ |
| 486 | |
Andreas Gruenbacher | 8554df1 | 2011-01-25 15:37:43 +0100 | [diff] [blame] | 487 | case TO_BE_SENT: /* via network */ |
Andreas Gruenbacher | 7be8da0 | 2011-02-22 02:15:32 +0100 | [diff] [blame] | 488 | /* reached via __drbd_make_request |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 489 | * and from w_read_retry_remote */ |
Andreas Gruenbacher | 0b0ba1e | 2011-06-27 16:23:33 +0200 | [diff] [blame] | 490 | D_ASSERT(device, !(req->rq_state & RQ_NET_MASK)); |
Philipp Reisner | 44ed167 | 2011-04-19 17:10:19 +0200 | [diff] [blame] | 491 | rcu_read_lock(); |
Lars Ellenberg | 44a4d55 | 2013-11-22 12:40:58 +0100 | [diff] [blame] | 492 | nc = rcu_dereference(connection->net_conf); |
Philipp Reisner | 44ed167 | 2011-04-19 17:10:19 +0200 | [diff] [blame] | 493 | p = nc->wire_protocol; |
| 494 | rcu_read_unlock(); |
Philipp Reisner | 303d144 | 2011-04-13 16:24:47 -0700 | [diff] [blame] | 495 | req->rq_state |= |
| 496 | p == DRBD_PROT_C ? RQ_EXP_WRITE_ACK : |
| 497 | p == DRBD_PROT_B ? RQ_EXP_RECEIVE_ACK : 0; |
Lars Ellenberg | a0d856d | 2012-01-24 17:19:42 +0100 | [diff] [blame] | 498 | mod_rq_state(req, m, 0, RQ_NET_PENDING); |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 499 | break; |
| 500 | |
Andreas Gruenbacher | 8554df1 | 2011-01-25 15:37:43 +0100 | [diff] [blame] | 501 | case TO_BE_SUBMITTED: /* locally */ |
Andreas Gruenbacher | 7be8da0 | 2011-02-22 02:15:32 +0100 | [diff] [blame] | 502 | /* reached via __drbd_make_request */ |
Andreas Gruenbacher | 0b0ba1e | 2011-06-27 16:23:33 +0200 | [diff] [blame] | 503 | D_ASSERT(device, !(req->rq_state & RQ_LOCAL_MASK)); |
Lars Ellenberg | a0d856d | 2012-01-24 17:19:42 +0100 | [diff] [blame] | 504 | mod_rq_state(req, m, 0, RQ_LOCAL_PENDING); |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 505 | break; |
| 506 | |
Andreas Gruenbacher | 8554df1 | 2011-01-25 15:37:43 +0100 | [diff] [blame] | 507 | case COMPLETED_OK: |
Philipp Reisner | 2b4dd36 | 2011-03-14 13:01:50 +0100 | [diff] [blame] | 508 | if (req->rq_state & RQ_WRITE) |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 509 | device->writ_cnt += req->i.size >> 9; |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 510 | else |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 511 | device->read_cnt += req->i.size >> 9; |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 512 | |
Lars Ellenberg | a0d856d | 2012-01-24 17:19:42 +0100 | [diff] [blame] | 513 | mod_rq_state(req, m, RQ_LOCAL_PENDING, |
| 514 | RQ_LOCAL_COMPLETED|RQ_LOCAL_OK); |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 515 | break; |
| 516 | |
Philipp Reisner | cdfda63 | 2011-07-05 15:38:59 +0200 | [diff] [blame] | 517 | case ABORT_DISK_IO: |
Lars Ellenberg | a0d856d | 2012-01-24 17:19:42 +0100 | [diff] [blame] | 518 | mod_rq_state(req, m, 0, RQ_LOCAL_ABORTED); |
Philipp Reisner | 2b4dd36 | 2011-03-14 13:01:50 +0100 | [diff] [blame] | 519 | break; |
| 520 | |
Lars Ellenberg | edc9f5e | 2012-09-27 15:18:21 +0200 | [diff] [blame] | 521 | case WRITE_COMPLETED_WITH_ERROR: |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 522 | drbd_report_io_error(device, req); |
| 523 | __drbd_chk_io_error(device, DRBD_WRITE_ERROR); |
Lars Ellenberg | edc9f5e | 2012-09-27 15:18:21 +0200 | [diff] [blame] | 524 | mod_rq_state(req, m, RQ_LOCAL_PENDING, RQ_LOCAL_COMPLETED); |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 525 | break; |
| 526 | |
Andreas Gruenbacher | 8554df1 | 2011-01-25 15:37:43 +0100 | [diff] [blame] | 527 | case READ_COMPLETED_WITH_ERROR: |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 528 | drbd_set_out_of_sync(device, req->i.sector, req->i.size); |
| 529 | drbd_report_io_error(device, req); |
| 530 | __drbd_chk_io_error(device, DRBD_READ_ERROR); |
Lars Ellenberg | a0d856d | 2012-01-24 17:19:42 +0100 | [diff] [blame] | 531 | /* fall through. */ |
| 532 | case READ_AHEAD_COMPLETED_WITH_ERROR: |
| 533 | /* it is legal to fail READA, no __drbd_chk_io_error in that case. */ |
| 534 | mod_rq_state(req, m, RQ_LOCAL_PENDING, RQ_LOCAL_COMPLETED); |
Lars Ellenberg | 4439c40 | 2012-03-26 17:29:30 +0200 | [diff] [blame] | 535 | break; |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 536 | |
Lars Ellenberg | 2f632ae | 2014-04-28 18:43:24 +0200 | [diff] [blame] | 537 | case DISCARD_COMPLETED_NOTSUPP: |
| 538 | case DISCARD_COMPLETED_WITH_ERROR: |
| 539 | /* I'd rather not detach from local disk just because it |
| 540 | * failed a REQ_DISCARD. */ |
| 541 | mod_rq_state(req, m, RQ_LOCAL_PENDING, RQ_LOCAL_COMPLETED); |
| 542 | break; |
| 543 | |
Andreas Gruenbacher | 8554df1 | 2011-01-25 15:37:43 +0100 | [diff] [blame] | 544 | case QUEUE_FOR_NET_READ: |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 545 | /* READ or READA, and |
| 546 | * no local disk, |
| 547 | * or target area marked as invalid, |
| 548 | * or just got an io-error. */ |
Andreas Gruenbacher | 7be8da0 | 2011-02-22 02:15:32 +0100 | [diff] [blame] | 549 | /* from __drbd_make_request |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 550 | * or from bio_endio during read io-error recovery */ |
| 551 | |
Lars Ellenberg | 6870ca6 | 2012-03-26 17:02:45 +0200 | [diff] [blame] | 552 | /* So we can verify the handle in the answer packet. |
| 553 | * Corresponding drbd_remove_request_interval is in |
Lars Ellenberg | a0d856d | 2012-01-24 17:19:42 +0100 | [diff] [blame] | 554 | * drbd_req_complete() */ |
Andreas Gruenbacher | 0b0ba1e | 2011-06-27 16:23:33 +0200 | [diff] [blame] | 555 | D_ASSERT(device, drbd_interval_empty(&req->i)); |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 556 | drbd_insert_interval(&device->read_requests, &req->i); |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 557 | |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 558 | set_bit(UNPLUG_REMOTE, &device->flags); |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 559 | |
Andreas Gruenbacher | 0b0ba1e | 2011-06-27 16:23:33 +0200 | [diff] [blame] | 560 | D_ASSERT(device, req->rq_state & RQ_NET_PENDING); |
| 561 | D_ASSERT(device, (req->rq_state & RQ_LOCAL_MASK) == 0); |
Lars Ellenberg | a0d856d | 2012-01-24 17:19:42 +0100 | [diff] [blame] | 562 | mod_rq_state(req, m, 0, RQ_NET_QUEUED); |
Lars Ellenberg | 4439c40 | 2012-03-26 17:29:30 +0200 | [diff] [blame] | 563 | req->w.cb = w_send_read_req; |
Lars Ellenberg | 44a4d55 | 2013-11-22 12:40:58 +0100 | [diff] [blame] | 564 | drbd_queue_work(&connection->sender_work, |
Andreas Gruenbacher | 84b8c06 | 2011-07-28 15:27:51 +0200 | [diff] [blame] | 565 | &req->w); |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 566 | break; |
| 567 | |
Andreas Gruenbacher | 8554df1 | 2011-01-25 15:37:43 +0100 | [diff] [blame] | 568 | case QUEUE_FOR_NET_WRITE: |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 569 | /* assert something? */ |
Andreas Gruenbacher | 7be8da0 | 2011-02-22 02:15:32 +0100 | [diff] [blame] | 570 | /* from __drbd_make_request only */ |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 571 | |
Lars Ellenberg | 6870ca6 | 2012-03-26 17:02:45 +0200 | [diff] [blame] | 572 | /* Corresponding drbd_remove_request_interval is in |
Lars Ellenberg | a0d856d | 2012-01-24 17:19:42 +0100 | [diff] [blame] | 573 | * drbd_req_complete() */ |
Andreas Gruenbacher | 0b0ba1e | 2011-06-27 16:23:33 +0200 | [diff] [blame] | 574 | D_ASSERT(device, drbd_interval_empty(&req->i)); |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 575 | drbd_insert_interval(&device->write_requests, &req->i); |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 576 | |
| 577 | /* NOTE |
| 578 | * In case the req ended up on the transfer log before being |
| 579 | * queued on the worker, it could lead to this request being |
| 580 | * missed during cleanup after connection loss. |
| 581 | * So we have to do both operations here, |
| 582 | * within the same lock that protects the transfer log. |
| 583 | * |
| 584 | * _req_add_to_epoch(req); this has to be after the |
| 585 | * _maybe_start_new_epoch(req); which happened in |
Andreas Gruenbacher | 7be8da0 | 2011-02-22 02:15:32 +0100 | [diff] [blame] | 586 | * __drbd_make_request, because we now may set the bit |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 587 | * again ourselves to close the current epoch. |
| 588 | * |
| 589 | * Add req to the (now) current epoch (barrier). */ |
| 590 | |
Lars Ellenberg | 83c3883 | 2009-11-03 02:22:06 +0100 | [diff] [blame] | 591 | /* otherwise we may lose an unplug, which may cause some remote |
| 592 | * io-scheduler timeout to expire, increasing maximum latency, |
| 593 | * hurting performance. */ |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 594 | set_bit(UNPLUG_REMOTE, &device->flags); |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 595 | |
| 596 | /* queue work item to send data */ |
Andreas Gruenbacher | 0b0ba1e | 2011-06-27 16:23:33 +0200 | [diff] [blame] | 597 | D_ASSERT(device, req->rq_state & RQ_NET_PENDING); |
Lars Ellenberg | a0d856d | 2012-01-24 17:19:42 +0100 | [diff] [blame] | 598 | mod_rq_state(req, m, 0, RQ_NET_QUEUED|RQ_EXP_BARR_ACK); |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 599 | req->w.cb = w_send_dblock; |
Lars Ellenberg | 44a4d55 | 2013-11-22 12:40:58 +0100 | [diff] [blame] | 600 | drbd_queue_work(&connection->sender_work, |
Andreas Gruenbacher | 84b8c06 | 2011-07-28 15:27:51 +0200 | [diff] [blame] | 601 | &req->w); |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 602 | |
| 603 | /* close the epoch, in case it outgrew the limit */ |
Philipp Reisner | 44ed167 | 2011-04-19 17:10:19 +0200 | [diff] [blame] | 604 | rcu_read_lock(); |
Lars Ellenberg | 44a4d55 | 2013-11-22 12:40:58 +0100 | [diff] [blame] | 605 | nc = rcu_dereference(connection->net_conf); |
Philipp Reisner | 44ed167 | 2011-04-19 17:10:19 +0200 | [diff] [blame] | 606 | p = nc->max_epoch_size; |
| 607 | rcu_read_unlock(); |
Lars Ellenberg | 44a4d55 | 2013-11-22 12:40:58 +0100 | [diff] [blame] | 608 | if (connection->current_tle_writes >= p) |
| 609 | start_new_tl_epoch(connection); |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 610 | |
| 611 | break; |
| 612 | |
Andreas Gruenbacher | 8554df1 | 2011-01-25 15:37:43 +0100 | [diff] [blame] | 613 | case QUEUE_FOR_SEND_OOS: |
Lars Ellenberg | a0d856d | 2012-01-24 17:19:42 +0100 | [diff] [blame] | 614 | mod_rq_state(req, m, 0, RQ_NET_QUEUED); |
Andreas Gruenbacher | 8f7bed7 | 2010-12-19 23:53:14 +0100 | [diff] [blame] | 615 | req->w.cb = w_send_out_of_sync; |
Lars Ellenberg | 44a4d55 | 2013-11-22 12:40:58 +0100 | [diff] [blame] | 616 | drbd_queue_work(&connection->sender_work, |
Andreas Gruenbacher | 84b8c06 | 2011-07-28 15:27:51 +0200 | [diff] [blame] | 617 | &req->w); |
Philipp Reisner | 73a01a1 | 2010-10-27 14:33:00 +0200 | [diff] [blame] | 618 | break; |
| 619 | |
Lars Ellenberg | ea9d672 | 2012-03-26 16:46:39 +0200 | [diff] [blame] | 620 | case READ_RETRY_REMOTE_CANCELED: |
Andreas Gruenbacher | 8554df1 | 2011-01-25 15:37:43 +0100 | [diff] [blame] | 621 | case SEND_CANCELED: |
Andreas Gruenbacher | 8554df1 | 2011-01-25 15:37:43 +0100 | [diff] [blame] | 622 | case SEND_FAILED: |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 623 | /* real cleanup will be done from tl_clear. just update flags |
| 624 | * so it is no longer marked as on the worker queue */ |
Lars Ellenberg | a0d856d | 2012-01-24 17:19:42 +0100 | [diff] [blame] | 625 | mod_rq_state(req, m, RQ_NET_QUEUED, 0); |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 626 | break; |
| 627 | |
Andreas Gruenbacher | 8554df1 | 2011-01-25 15:37:43 +0100 | [diff] [blame] | 628 | case HANDED_OVER_TO_NETWORK: |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 629 | /* assert something? */ |
| 630 | if (bio_data_dir(req->master_bio) == WRITE && |
Philipp Reisner | 303d144 | 2011-04-13 16:24:47 -0700 | [diff] [blame] | 631 | !(req->rq_state & (RQ_EXP_RECEIVE_ACK | RQ_EXP_WRITE_ACK))) { |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 632 | /* this is what is dangerous about protocol A: |
| 633 | * pretend it was successfully written on the peer. */ |
Lars Ellenberg | a0d856d | 2012-01-24 17:19:42 +0100 | [diff] [blame] | 634 | if (req->rq_state & RQ_NET_PENDING) |
| 635 | mod_rq_state(req, m, RQ_NET_PENDING, RQ_NET_OK); |
| 636 | /* else: neg-ack was faster... */ |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 637 | /* it is still not yet RQ_NET_DONE until the |
| 638 | * corresponding epoch barrier got acked as well, |
| 639 | * so we know what to dirty on connection loss */ |
| 640 | } |
Lars Ellenberg | a0d856d | 2012-01-24 17:19:42 +0100 | [diff] [blame] | 641 | mod_rq_state(req, m, RQ_NET_QUEUED, RQ_NET_SENT); |
Lars Ellenberg | 6d49e10 | 2012-01-11 09:43:25 +0100 | [diff] [blame] | 642 | break; |
| 643 | |
Lars Ellenberg | 27a434f | 2012-03-26 16:44:59 +0200 | [diff] [blame] | 644 | case OOS_HANDED_TO_NETWORK: |
Lars Ellenberg | 6d49e10 | 2012-01-11 09:43:25 +0100 | [diff] [blame] | 645 | /* Was not set PENDING, no longer QUEUED, so is now DONE |
| 646 | * as far as this connection is concerned. */ |
Lars Ellenberg | a0d856d | 2012-01-24 17:19:42 +0100 | [diff] [blame] | 647 | mod_rq_state(req, m, RQ_NET_QUEUED, RQ_NET_DONE); |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 648 | break; |
| 649 | |
Andreas Gruenbacher | 8554df1 | 2011-01-25 15:37:43 +0100 | [diff] [blame] | 650 | case CONNECTION_LOST_WHILE_PENDING: |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 651 | /* transfer log cleanup after connection loss */ |
Lars Ellenberg | a0d856d | 2012-01-24 17:19:42 +0100 | [diff] [blame] | 652 | mod_rq_state(req, m, |
| 653 | RQ_NET_OK|RQ_NET_PENDING|RQ_COMPLETION_SUSP, |
| 654 | RQ_NET_DONE); |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 655 | break; |
| 656 | |
Lars Ellenberg | d4dabbe | 2012-08-01 12:33:51 +0200 | [diff] [blame] | 657 | case CONFLICT_RESOLVED: |
| 658 | /* for superseded conflicting writes of multiple primaries, |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 659 | * there is no need to keep anything in the tl, potential |
Lars Ellenberg | 934722a | 2012-07-24 09:31:18 +0200 | [diff] [blame] | 660 | * node crashes are covered by the activity log. |
| 661 | * |
| 662 | * If this request had been marked as RQ_POSTPONED before, |
Lars Ellenberg | d4dabbe | 2012-08-01 12:33:51 +0200 | [diff] [blame] | 663 | * it will actually not be completed, but "restarted", |
Lars Ellenberg | 934722a | 2012-07-24 09:31:18 +0200 | [diff] [blame] | 664 | * resubmitted from the retry worker context. */ |
Andreas Gruenbacher | 0b0ba1e | 2011-06-27 16:23:33 +0200 | [diff] [blame] | 665 | D_ASSERT(device, req->rq_state & RQ_NET_PENDING); |
| 666 | D_ASSERT(device, req->rq_state & RQ_EXP_WRITE_ACK); |
Lars Ellenberg | 934722a | 2012-07-24 09:31:18 +0200 | [diff] [blame] | 667 | mod_rq_state(req, m, RQ_NET_PENDING, RQ_NET_DONE|RQ_NET_OK); |
| 668 | break; |
| 669 | |
Lars Ellenberg | 0afd569 | 2012-03-26 16:51:11 +0200 | [diff] [blame] | 670 | case WRITE_ACKED_BY_PEER_AND_SIS: |
Lars Ellenberg | 934722a | 2012-07-24 09:31:18 +0200 | [diff] [blame] | 671 | req->rq_state |= RQ_NET_SIS; |
Andreas Gruenbacher | 8554df1 | 2011-01-25 15:37:43 +0100 | [diff] [blame] | 672 | case WRITE_ACKED_BY_PEER: |
Lars Ellenberg | 08d0dab | 2014-03-20 11:19:22 +0100 | [diff] [blame^] | 673 | /* Normal operation protocol C: successfully written on peer. |
| 674 | * During resync, even in protocol != C, |
| 675 | * we requested an explicit write ack anyways. |
| 676 | * Which means we cannot even assert anything here. |
Lars Ellenberg | d64957c | 2012-03-23 14:42:19 +0100 | [diff] [blame] | 677 | * Nothing more to do here. |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 678 | * We want to keep the tl in place for all protocols, to cater |
Lars Ellenberg | d64957c | 2012-03-23 14:42:19 +0100 | [diff] [blame] | 679 | * for volatile write-back caches on lower level devices. */ |
Philipp Reisner | 303d144 | 2011-04-13 16:24:47 -0700 | [diff] [blame] | 680 | goto ack_common; |
Andreas Gruenbacher | 8554df1 | 2011-01-25 15:37:43 +0100 | [diff] [blame] | 681 | case RECV_ACKED_BY_PEER: |
Andreas Gruenbacher | 0b0ba1e | 2011-06-27 16:23:33 +0200 | [diff] [blame] | 682 | D_ASSERT(device, req->rq_state & RQ_EXP_RECEIVE_ACK); |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 683 | /* protocol B; pretends to be successfully written on peer. |
Andreas Gruenbacher | 8554df1 | 2011-01-25 15:37:43 +0100 | [diff] [blame] | 684 | * see also notes above in HANDED_OVER_TO_NETWORK about |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 685 | * protocol != C */ |
Philipp Reisner | 303d144 | 2011-04-13 16:24:47 -0700 | [diff] [blame] | 686 | ack_common: |
Lars Ellenberg | a0d856d | 2012-01-24 17:19:42 +0100 | [diff] [blame] | 687 | mod_rq_state(req, m, RQ_NET_PENDING, RQ_NET_OK); |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 688 | break; |
| 689 | |
Andreas Gruenbacher | 7be8da0 | 2011-02-22 02:15:32 +0100 | [diff] [blame] | 690 | case POSTPONE_WRITE: |
Andreas Gruenbacher | 0b0ba1e | 2011-06-27 16:23:33 +0200 | [diff] [blame] | 691 | D_ASSERT(device, req->rq_state & RQ_EXP_WRITE_ACK); |
Philipp Reisner | 303d144 | 2011-04-13 16:24:47 -0700 | [diff] [blame] | 692 | /* If this node has already detected the write conflict, the |
Andreas Gruenbacher | 7be8da0 | 2011-02-22 02:15:32 +0100 | [diff] [blame] | 693 | * worker will be waiting on misc_wait. Wake it up once this |
| 694 | * request has completed locally. |
| 695 | */ |
Andreas Gruenbacher | 0b0ba1e | 2011-06-27 16:23:33 +0200 | [diff] [blame] | 696 | D_ASSERT(device, req->rq_state & RQ_NET_PENDING); |
Andreas Gruenbacher | 7be8da0 | 2011-02-22 02:15:32 +0100 | [diff] [blame] | 697 | req->rq_state |= RQ_POSTPONED; |
Lars Ellenberg | a0d856d | 2012-01-24 17:19:42 +0100 | [diff] [blame] | 698 | if (req->i.waiting) |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 699 | wake_up(&device->misc_wait); |
Lars Ellenberg | a0d856d | 2012-01-24 17:19:42 +0100 | [diff] [blame] | 700 | /* Do not clear RQ_NET_PENDING. This request will make further |
| 701 | * progress via restart_conflicting_writes() or |
| 702 | * fail_postponed_requests(). Hopefully. */ |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 703 | break; |
| 704 | |
Andreas Gruenbacher | 8554df1 | 2011-01-25 15:37:43 +0100 | [diff] [blame] | 705 | case NEG_ACKED: |
Lars Ellenberg | 46e21bb | 2012-08-07 06:47:14 +0200 | [diff] [blame] | 706 | mod_rq_state(req, m, RQ_NET_OK|RQ_NET_PENDING, 0); |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 707 | break; |
| 708 | |
Andreas Gruenbacher | 8554df1 | 2011-01-25 15:37:43 +0100 | [diff] [blame] | 709 | case FAIL_FROZEN_DISK_IO: |
Philipp Reisner | 265be2d | 2010-05-31 10:14:17 +0200 | [diff] [blame] | 710 | if (!(req->rq_state & RQ_LOCAL_COMPLETED)) |
| 711 | break; |
Lars Ellenberg | a0d856d | 2012-01-24 17:19:42 +0100 | [diff] [blame] | 712 | mod_rq_state(req, m, RQ_COMPLETION_SUSP, 0); |
Philipp Reisner | 265be2d | 2010-05-31 10:14:17 +0200 | [diff] [blame] | 713 | break; |
| 714 | |
Andreas Gruenbacher | 8554df1 | 2011-01-25 15:37:43 +0100 | [diff] [blame] | 715 | case RESTART_FROZEN_DISK_IO: |
Philipp Reisner | 265be2d | 2010-05-31 10:14:17 +0200 | [diff] [blame] | 716 | if (!(req->rq_state & RQ_LOCAL_COMPLETED)) |
| 717 | break; |
| 718 | |
Lars Ellenberg | a0d856d | 2012-01-24 17:19:42 +0100 | [diff] [blame] | 719 | mod_rq_state(req, m, |
| 720 | RQ_COMPLETION_SUSP|RQ_LOCAL_COMPLETED, |
| 721 | RQ_LOCAL_PENDING); |
Philipp Reisner | 265be2d | 2010-05-31 10:14:17 +0200 | [diff] [blame] | 722 | |
| 723 | rv = MR_READ; |
| 724 | if (bio_data_dir(req->master_bio) == WRITE) |
| 725 | rv = MR_WRITE; |
| 726 | |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 727 | get_ldev(device); /* always succeeds in this call path */ |
Philipp Reisner | 265be2d | 2010-05-31 10:14:17 +0200 | [diff] [blame] | 728 | req->w.cb = w_restart_disk_io; |
Lars Ellenberg | 44a4d55 | 2013-11-22 12:40:58 +0100 | [diff] [blame] | 729 | drbd_queue_work(&connection->sender_work, |
Andreas Gruenbacher | 84b8c06 | 2011-07-28 15:27:51 +0200 | [diff] [blame] | 730 | &req->w); |
Philipp Reisner | 265be2d | 2010-05-31 10:14:17 +0200 | [diff] [blame] | 731 | break; |
| 732 | |
Andreas Gruenbacher | 8554df1 | 2011-01-25 15:37:43 +0100 | [diff] [blame] | 733 | case RESEND: |
Philipp Reisner | 509fc01 | 2012-07-31 11:22:58 +0200 | [diff] [blame] | 734 | /* Simply complete (local only) READs. */ |
| 735 | if (!(req->rq_state & RQ_WRITE) && !req->w.cb) { |
Philipp Reisner | 8a0bab2 | 2012-08-07 13:28:00 +0200 | [diff] [blame] | 736 | mod_rq_state(req, m, RQ_COMPLETION_SUSP, 0); |
Philipp Reisner | 509fc01 | 2012-07-31 11:22:58 +0200 | [diff] [blame] | 737 | break; |
| 738 | } |
| 739 | |
Philipp Reisner | 11b58e7 | 2010-05-12 17:08:26 +0200 | [diff] [blame] | 740 | /* If RQ_NET_OK is already set, we got a P_WRITE_ACK or P_RECV_ACK |
Lars Ellenberg | a0d856d | 2012-01-24 17:19:42 +0100 | [diff] [blame] | 741 | before the connection loss (B&C only); only P_BARRIER_ACK |
| 742 | (or the local completion?) was missing when we suspended. |
Lars Ellenberg | 6870ca6 | 2012-03-26 17:02:45 +0200 | [diff] [blame] | 743 | Throwing them out of the TL here by pretending we got a BARRIER_ACK. |
| 744 | During connection handshake, we ensure that the peer was not rebooted. */ |
Philipp Reisner | 11b58e7 | 2010-05-12 17:08:26 +0200 | [diff] [blame] | 745 | if (!(req->rq_state & RQ_NET_OK)) { |
Andreas Gruenbacher | 84b8c06 | 2011-07-28 15:27:51 +0200 | [diff] [blame] | 746 | /* FIXME could this possibly be a req->dw.cb == w_send_out_of_sync? |
Lars Ellenberg | a0d856d | 2012-01-24 17:19:42 +0100 | [diff] [blame] | 747 | * in that case we must not set RQ_NET_PENDING. */ |
| 748 | |
| 749 | mod_rq_state(req, m, RQ_COMPLETION_SUSP, RQ_NET_QUEUED|RQ_NET_PENDING); |
Philipp Reisner | 11b58e7 | 2010-05-12 17:08:26 +0200 | [diff] [blame] | 750 | if (req->w.cb) { |
Lars Ellenberg | 44a4d55 | 2013-11-22 12:40:58 +0100 | [diff] [blame] | 751 | /* w.cb expected to be w_send_dblock, or w_send_read_req */ |
| 752 | drbd_queue_work(&connection->sender_work, |
Andreas Gruenbacher | 84b8c06 | 2011-07-28 15:27:51 +0200 | [diff] [blame] | 753 | &req->w); |
Philipp Reisner | 11b58e7 | 2010-05-12 17:08:26 +0200 | [diff] [blame] | 754 | rv = req->rq_state & RQ_WRITE ? MR_WRITE : MR_READ; |
Lars Ellenberg | a0d856d | 2012-01-24 17:19:42 +0100 | [diff] [blame] | 755 | } /* else: FIXME can this happen? */ |
Philipp Reisner | 11b58e7 | 2010-05-12 17:08:26 +0200 | [diff] [blame] | 756 | break; |
| 757 | } |
Andreas Gruenbacher | 8554df1 | 2011-01-25 15:37:43 +0100 | [diff] [blame] | 758 | /* else, fall through to BARRIER_ACKED */ |
Philipp Reisner | 11b58e7 | 2010-05-12 17:08:26 +0200 | [diff] [blame] | 759 | |
Andreas Gruenbacher | 8554df1 | 2011-01-25 15:37:43 +0100 | [diff] [blame] | 760 | case BARRIER_ACKED: |
Lars Ellenberg | a0d856d | 2012-01-24 17:19:42 +0100 | [diff] [blame] | 761 | /* barrier ack for READ requests does not make sense */ |
Philipp Reisner | 288f422 | 2010-05-27 15:07:43 +0200 | [diff] [blame] | 762 | if (!(req->rq_state & RQ_WRITE)) |
| 763 | break; |
| 764 | |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 765 | if (req->rq_state & RQ_NET_PENDING) { |
Andreas Gruenbacher | a209b4a | 2011-08-17 12:43:25 +0200 | [diff] [blame] | 766 | /* barrier came in before all requests were acked. |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 767 | * this is bad, because if the connection is lost now, |
| 768 | * we won't be able to clean them up... */ |
Andreas Gruenbacher | d018017 | 2011-07-03 17:53:52 +0200 | [diff] [blame] | 769 | drbd_err(device, "FIXME (BARRIER_ACKED but pending)\n"); |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 770 | } |
Lars Ellenberg | a0d856d | 2012-01-24 17:19:42 +0100 | [diff] [blame] | 771 | /* Allowed to complete requests, even while suspended. |
| 772 | * As this is called for all requests within a matching epoch, |
| 773 | * we need to filter, and only set RQ_NET_DONE for those that |
| 774 | * have actually been on the wire. */ |
| 775 | mod_rq_state(req, m, RQ_COMPLETION_SUSP, |
| 776 | (req->rq_state & RQ_NET_MASK) ? RQ_NET_DONE : 0); |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 777 | break; |
| 778 | |
Andreas Gruenbacher | 8554df1 | 2011-01-25 15:37:43 +0100 | [diff] [blame] | 779 | case DATA_RECEIVED: |
Andreas Gruenbacher | 0b0ba1e | 2011-06-27 16:23:33 +0200 | [diff] [blame] | 780 | D_ASSERT(device, req->rq_state & RQ_NET_PENDING); |
Lars Ellenberg | a0d856d | 2012-01-24 17:19:42 +0100 | [diff] [blame] | 781 | mod_rq_state(req, m, RQ_NET_PENDING, RQ_NET_OK|RQ_NET_DONE); |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 782 | break; |
Lars Ellenberg | 7074e4a | 2013-03-27 14:08:41 +0100 | [diff] [blame] | 783 | |
| 784 | case QUEUE_AS_DRBD_BARRIER: |
Lars Ellenberg | 44a4d55 | 2013-11-22 12:40:58 +0100 | [diff] [blame] | 785 | start_new_tl_epoch(connection); |
Lars Ellenberg | 7074e4a | 2013-03-27 14:08:41 +0100 | [diff] [blame] | 786 | mod_rq_state(req, m, 0, RQ_NET_OK|RQ_NET_DONE); |
| 787 | break; |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 788 | }; |
Philipp Reisner | 2a80699 | 2010-06-09 14:07:43 +0200 | [diff] [blame] | 789 | |
| 790 | return rv; |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 791 | } |
| 792 | |
| 793 | /* we may do a local read if: |
| 794 | * - we are consistent (of course), |
| 795 | * - or we are generally inconsistent, |
| 796 | * BUT we are still/already IN SYNC for this area. |
| 797 | * since size may be bigger than BM_BLOCK_SIZE, |
| 798 | * we may need to check several bits. |
| 799 | */ |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 800 | static bool drbd_may_do_local_read(struct drbd_device *device, sector_t sector, int size) |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 801 | { |
| 802 | unsigned long sbnr, ebnr; |
| 803 | sector_t esector, nr_sectors; |
| 804 | |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 805 | if (device->state.disk == D_UP_TO_DATE) |
Andreas Gruenbacher | 0da34df | 2010-12-19 20:48:29 +0100 | [diff] [blame] | 806 | return true; |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 807 | if (device->state.disk != D_INCONSISTENT) |
Andreas Gruenbacher | 0da34df | 2010-12-19 20:48:29 +0100 | [diff] [blame] | 808 | return false; |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 809 | esector = sector + (size >> 9) - 1; |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 810 | nr_sectors = drbd_get_capacity(device->this_bdev); |
Andreas Gruenbacher | 0b0ba1e | 2011-06-27 16:23:33 +0200 | [diff] [blame] | 811 | D_ASSERT(device, sector < nr_sectors); |
| 812 | D_ASSERT(device, esector < nr_sectors); |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 813 | |
| 814 | sbnr = BM_SECT_TO_BIT(sector); |
| 815 | ebnr = BM_SECT_TO_BIT(esector); |
| 816 | |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 817 | return drbd_bm_count_bits(device, sbnr, ebnr) == 0; |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 818 | } |
| 819 | |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 820 | static bool remote_due_to_read_balancing(struct drbd_device *device, sector_t sector, |
Lars Ellenberg | 5da9c83 | 2012-03-29 17:04:14 +0200 | [diff] [blame] | 821 | enum drbd_read_balancing rbm) |
Philipp Reisner | 380207d | 2011-11-11 12:31:20 +0100 | [diff] [blame] | 822 | { |
Philipp Reisner | 380207d | 2011-11-11 12:31:20 +0100 | [diff] [blame] | 823 | struct backing_dev_info *bdi; |
Philipp Reisner | d60de03 | 2011-11-17 10:12:31 +0100 | [diff] [blame] | 824 | int stripe_shift; |
Philipp Reisner | 380207d | 2011-11-11 12:31:20 +0100 | [diff] [blame] | 825 | |
Philipp Reisner | 380207d | 2011-11-11 12:31:20 +0100 | [diff] [blame] | 826 | switch (rbm) { |
| 827 | case RB_CONGESTED_REMOTE: |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 828 | bdi = &device->ldev->backing_bdev->bd_disk->queue->backing_dev_info; |
Philipp Reisner | 380207d | 2011-11-11 12:31:20 +0100 | [diff] [blame] | 829 | return bdi_read_congested(bdi); |
| 830 | case RB_LEAST_PENDING: |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 831 | return atomic_read(&device->local_cnt) > |
| 832 | atomic_read(&device->ap_pending_cnt) + atomic_read(&device->rs_pending_cnt); |
Philipp Reisner | d60de03 | 2011-11-17 10:12:31 +0100 | [diff] [blame] | 833 | case RB_32K_STRIPING: /* stripe_shift = 15 */ |
| 834 | case RB_64K_STRIPING: |
| 835 | case RB_128K_STRIPING: |
| 836 | case RB_256K_STRIPING: |
| 837 | case RB_512K_STRIPING: |
| 838 | case RB_1M_STRIPING: /* stripe_shift = 20 */ |
| 839 | stripe_shift = (rbm - RB_32K_STRIPING + 15); |
| 840 | return (sector >> (stripe_shift - 9)) & 1; |
Philipp Reisner | 380207d | 2011-11-11 12:31:20 +0100 | [diff] [blame] | 841 | case RB_ROUND_ROBIN: |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 842 | return test_and_change_bit(READ_BALANCE_RR, &device->flags); |
Philipp Reisner | 380207d | 2011-11-11 12:31:20 +0100 | [diff] [blame] | 843 | case RB_PREFER_REMOTE: |
| 844 | return true; |
| 845 | case RB_PREFER_LOCAL: |
| 846 | default: |
| 847 | return false; |
| 848 | } |
| 849 | } |
| 850 | |
Andreas Gruenbacher | 6024fec | 2011-01-28 15:53:51 +0100 | [diff] [blame] | 851 | /* |
| 852 | * complete_conflicting_writes - wait for any conflicting write requests |
| 853 | * |
| 854 | * The write_requests tree contains all active write requests which we |
| 855 | * currently know about. Wait for any requests to complete which conflict with |
| 856 | * the new one. |
Lars Ellenberg | 648e46b | 2012-03-26 20:12:24 +0200 | [diff] [blame] | 857 | * |
| 858 | * Only way out: remove the conflicting intervals from the tree. |
Andreas Gruenbacher | 6024fec | 2011-01-28 15:53:51 +0100 | [diff] [blame] | 859 | */ |
Lars Ellenberg | 648e46b | 2012-03-26 20:12:24 +0200 | [diff] [blame] | 860 | static void complete_conflicting_writes(struct drbd_request *req) |
Andreas Gruenbacher | 6024fec | 2011-01-28 15:53:51 +0100 | [diff] [blame] | 861 | { |
Lars Ellenberg | 648e46b | 2012-03-26 20:12:24 +0200 | [diff] [blame] | 862 | DEFINE_WAIT(wait); |
Andreas Gruenbacher | 84b8c06 | 2011-07-28 15:27:51 +0200 | [diff] [blame] | 863 | struct drbd_device *device = req->device; |
Lars Ellenberg | 648e46b | 2012-03-26 20:12:24 +0200 | [diff] [blame] | 864 | struct drbd_interval *i; |
| 865 | sector_t sector = req->i.sector; |
| 866 | int size = req->i.size; |
Andreas Gruenbacher | 6024fec | 2011-01-28 15:53:51 +0100 | [diff] [blame] | 867 | |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 868 | i = drbd_find_overlap(&device->write_requests, sector, size); |
Lars Ellenberg | 648e46b | 2012-03-26 20:12:24 +0200 | [diff] [blame] | 869 | if (!i) |
| 870 | return; |
| 871 | |
| 872 | for (;;) { |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 873 | prepare_to_wait(&device->misc_wait, &wait, TASK_UNINTERRUPTIBLE); |
| 874 | i = drbd_find_overlap(&device->write_requests, sector, size); |
Andreas Gruenbacher | 6024fec | 2011-01-28 15:53:51 +0100 | [diff] [blame] | 875 | if (!i) |
Lars Ellenberg | 648e46b | 2012-03-26 20:12:24 +0200 | [diff] [blame] | 876 | break; |
| 877 | /* Indicate to wake up device->misc_wait on progress. */ |
| 878 | i->waiting = true; |
Andreas Gruenbacher | 0500813 | 2011-07-07 14:19:42 +0200 | [diff] [blame] | 879 | spin_unlock_irq(&device->resource->req_lock); |
Lars Ellenberg | 648e46b | 2012-03-26 20:12:24 +0200 | [diff] [blame] | 880 | schedule(); |
Andreas Gruenbacher | 0500813 | 2011-07-07 14:19:42 +0200 | [diff] [blame] | 881 | spin_lock_irq(&device->resource->req_lock); |
Andreas Gruenbacher | 6024fec | 2011-01-28 15:53:51 +0100 | [diff] [blame] | 882 | } |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 883 | finish_wait(&device->misc_wait, &wait); |
Andreas Gruenbacher | 6024fec | 2011-01-28 15:53:51 +0100 | [diff] [blame] | 884 | } |
| 885 | |
Lars Ellenberg | 5da9c83 | 2012-03-29 17:04:14 +0200 | [diff] [blame] | 886 | /* called within req_lock and rcu_read_lock() */ |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 887 | static void maybe_pull_ahead(struct drbd_device *device) |
Lars Ellenberg | 0d5934e | 2012-06-08 14:17:36 +0200 | [diff] [blame] | 888 | { |
Andreas Gruenbacher | a6b32bc | 2011-05-31 14:33:49 +0200 | [diff] [blame] | 889 | struct drbd_connection *connection = first_peer_device(device)->connection; |
Lars Ellenberg | 5da9c83 | 2012-03-29 17:04:14 +0200 | [diff] [blame] | 890 | struct net_conf *nc; |
| 891 | bool congested = false; |
| 892 | enum drbd_on_congestion on_congestion; |
| 893 | |
Lars Ellenberg | 607f25e | 2013-03-27 14:08:45 +0100 | [diff] [blame] | 894 | rcu_read_lock(); |
Andreas Gruenbacher | bde89a9 | 2011-05-30 16:32:41 +0200 | [diff] [blame] | 895 | nc = rcu_dereference(connection->net_conf); |
Lars Ellenberg | 5da9c83 | 2012-03-29 17:04:14 +0200 | [diff] [blame] | 896 | on_congestion = nc ? nc->on_congestion : OC_BLOCK; |
Lars Ellenberg | 607f25e | 2013-03-27 14:08:45 +0100 | [diff] [blame] | 897 | rcu_read_unlock(); |
Lars Ellenberg | 5da9c83 | 2012-03-29 17:04:14 +0200 | [diff] [blame] | 898 | if (on_congestion == OC_BLOCK || |
Andreas Gruenbacher | bde89a9 | 2011-05-30 16:32:41 +0200 | [diff] [blame] | 899 | connection->agreed_pro_version < 96) |
Lars Ellenberg | 3b9ef85 | 2012-07-30 09:06:26 +0200 | [diff] [blame] | 900 | return; |
Lars Ellenberg | 0d5934e | 2012-06-08 14:17:36 +0200 | [diff] [blame] | 901 | |
| 902 | /* If I don't even have good local storage, we can not reasonably try |
| 903 | * to pull ahead of the peer. We also need the local reference to make |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 904 | * sure device->act_log is there. |
Lars Ellenberg | 0d5934e | 2012-06-08 14:17:36 +0200 | [diff] [blame] | 905 | */ |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 906 | if (!get_ldev_if_state(device, D_UP_TO_DATE)) |
Lars Ellenberg | 0d5934e | 2012-06-08 14:17:36 +0200 | [diff] [blame] | 907 | return; |
| 908 | |
Lars Ellenberg | 5da9c83 | 2012-03-29 17:04:14 +0200 | [diff] [blame] | 909 | if (nc->cong_fill && |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 910 | atomic_read(&device->ap_in_flight) >= nc->cong_fill) { |
Andreas Gruenbacher | d018017 | 2011-07-03 17:53:52 +0200 | [diff] [blame] | 911 | drbd_info(device, "Congestion-fill threshold reached\n"); |
Lars Ellenberg | 5da9c83 | 2012-03-29 17:04:14 +0200 | [diff] [blame] | 912 | congested = true; |
Lars Ellenberg | 0d5934e | 2012-06-08 14:17:36 +0200 | [diff] [blame] | 913 | } |
| 914 | |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 915 | if (device->act_log->used >= nc->cong_extents) { |
Andreas Gruenbacher | d018017 | 2011-07-03 17:53:52 +0200 | [diff] [blame] | 916 | drbd_info(device, "Congestion-extents threshold reached\n"); |
Lars Ellenberg | 5da9c83 | 2012-03-29 17:04:14 +0200 | [diff] [blame] | 917 | congested = true; |
Lars Ellenberg | 0d5934e | 2012-06-08 14:17:36 +0200 | [diff] [blame] | 918 | } |
| 919 | |
| 920 | if (congested) { |
Lars Ellenberg | 99b4d8f | 2012-08-07 06:42:09 +0200 | [diff] [blame] | 921 | /* start a new epoch for non-mirrored writes */ |
Andreas Gruenbacher | a6b32bc | 2011-05-31 14:33:49 +0200 | [diff] [blame] | 922 | start_new_tl_epoch(first_peer_device(device)->connection); |
Lars Ellenberg | 0d5934e | 2012-06-08 14:17:36 +0200 | [diff] [blame] | 923 | |
Lars Ellenberg | 5da9c83 | 2012-03-29 17:04:14 +0200 | [diff] [blame] | 924 | if (on_congestion == OC_PULL_AHEAD) |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 925 | _drbd_set_state(_NS(device, conn, C_AHEAD), 0, NULL); |
Lars Ellenberg | 5da9c83 | 2012-03-29 17:04:14 +0200 | [diff] [blame] | 926 | else /*nc->on_congestion == OC_DISCONNECT */ |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 927 | _drbd_set_state(_NS(device, conn, C_DISCONNECTING), 0, NULL); |
Lars Ellenberg | 0d5934e | 2012-06-08 14:17:36 +0200 | [diff] [blame] | 928 | } |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 929 | put_ldev(device); |
Lars Ellenberg | 0d5934e | 2012-06-08 14:17:36 +0200 | [diff] [blame] | 930 | } |
| 931 | |
Lars Ellenberg | 5da9c83 | 2012-03-29 17:04:14 +0200 | [diff] [blame] | 932 | /* If this returns false, and req->private_bio is still set, |
| 933 | * this should be submitted locally. |
| 934 | * |
| 935 | * If it returns false, but req->private_bio is not set, |
| 936 | * we do not have access to good data :( |
| 937 | * |
| 938 | * Otherwise, this destroys req->private_bio, if any, |
| 939 | * and returns true. |
| 940 | */ |
| 941 | static bool do_remote_read(struct drbd_request *req) |
| 942 | { |
Andreas Gruenbacher | 84b8c06 | 2011-07-28 15:27:51 +0200 | [diff] [blame] | 943 | struct drbd_device *device = req->device; |
Lars Ellenberg | 5da9c83 | 2012-03-29 17:04:14 +0200 | [diff] [blame] | 944 | enum drbd_read_balancing rbm; |
| 945 | |
| 946 | if (req->private_bio) { |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 947 | if (!drbd_may_do_local_read(device, |
Lars Ellenberg | 5da9c83 | 2012-03-29 17:04:14 +0200 | [diff] [blame] | 948 | req->i.sector, req->i.size)) { |
| 949 | bio_put(req->private_bio); |
| 950 | req->private_bio = NULL; |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 951 | put_ldev(device); |
Lars Ellenberg | 5da9c83 | 2012-03-29 17:04:14 +0200 | [diff] [blame] | 952 | } |
| 953 | } |
| 954 | |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 955 | if (device->state.pdsk != D_UP_TO_DATE) |
Lars Ellenberg | 5da9c83 | 2012-03-29 17:04:14 +0200 | [diff] [blame] | 956 | return false; |
| 957 | |
Lars Ellenberg | a0d856d | 2012-01-24 17:19:42 +0100 | [diff] [blame] | 958 | if (req->private_bio == NULL) |
| 959 | return true; |
| 960 | |
Lars Ellenberg | 5da9c83 | 2012-03-29 17:04:14 +0200 | [diff] [blame] | 961 | /* TODO: improve read balancing decisions, take into account drbd |
| 962 | * protocol, pending requests etc. */ |
| 963 | |
| 964 | rcu_read_lock(); |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 965 | rbm = rcu_dereference(device->ldev->disk_conf)->read_balancing; |
Lars Ellenberg | 5da9c83 | 2012-03-29 17:04:14 +0200 | [diff] [blame] | 966 | rcu_read_unlock(); |
| 967 | |
| 968 | if (rbm == RB_PREFER_LOCAL && req->private_bio) |
| 969 | return false; /* submit locally */ |
| 970 | |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 971 | if (remote_due_to_read_balancing(device, req->i.sector, rbm)) { |
Lars Ellenberg | 5da9c83 | 2012-03-29 17:04:14 +0200 | [diff] [blame] | 972 | if (req->private_bio) { |
| 973 | bio_put(req->private_bio); |
| 974 | req->private_bio = NULL; |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 975 | put_ldev(device); |
Lars Ellenberg | 5da9c83 | 2012-03-29 17:04:14 +0200 | [diff] [blame] | 976 | } |
| 977 | return true; |
| 978 | } |
| 979 | |
| 980 | return false; |
| 981 | } |
| 982 | |
| 983 | /* returns number of connections (== 1, for drbd 8.4) |
| 984 | * expected to actually write this data, |
| 985 | * which does NOT include those that we are L_AHEAD for. */ |
| 986 | static int drbd_process_write_request(struct drbd_request *req) |
| 987 | { |
Andreas Gruenbacher | 84b8c06 | 2011-07-28 15:27:51 +0200 | [diff] [blame] | 988 | struct drbd_device *device = req->device; |
Lars Ellenberg | 5da9c83 | 2012-03-29 17:04:14 +0200 | [diff] [blame] | 989 | int remote, send_oos; |
| 990 | |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 991 | remote = drbd_should_do_remote(device->state); |
| 992 | send_oos = drbd_should_send_out_of_sync(device->state); |
Lars Ellenberg | 5da9c83 | 2012-03-29 17:04:14 +0200 | [diff] [blame] | 993 | |
Lars Ellenberg | 519b6d3e | 2012-08-03 02:19:09 +0200 | [diff] [blame] | 994 | /* Need to replicate writes. Unless it is an empty flush, |
| 995 | * which is better mapped to a DRBD P_BARRIER packet, |
| 996 | * also for drbd wire protocol compatibility reasons. |
| 997 | * If this was a flush, just start a new epoch. |
| 998 | * Unless the current epoch was empty anyways, or we are not currently |
| 999 | * replicating, in which case there is no point. */ |
| 1000 | if (unlikely(req->i.size == 0)) { |
| 1001 | /* The only size==0 bios we expect are empty flushes. */ |
Andreas Gruenbacher | 0b0ba1e | 2011-06-27 16:23:33 +0200 | [diff] [blame] | 1002 | D_ASSERT(device, req->master_bio->bi_rw & REQ_FLUSH); |
Lars Ellenberg | 99b4d8f | 2012-08-07 06:42:09 +0200 | [diff] [blame] | 1003 | if (remote) |
Lars Ellenberg | 7074e4a | 2013-03-27 14:08:41 +0100 | [diff] [blame] | 1004 | _req_mod(req, QUEUE_AS_DRBD_BARRIER); |
| 1005 | return remote; |
Lars Ellenberg | 519b6d3e | 2012-08-03 02:19:09 +0200 | [diff] [blame] | 1006 | } |
| 1007 | |
Lars Ellenberg | 5da9c83 | 2012-03-29 17:04:14 +0200 | [diff] [blame] | 1008 | if (!remote && !send_oos) |
| 1009 | return 0; |
| 1010 | |
Andreas Gruenbacher | 0b0ba1e | 2011-06-27 16:23:33 +0200 | [diff] [blame] | 1011 | D_ASSERT(device, !(remote && send_oos)); |
Lars Ellenberg | 5da9c83 | 2012-03-29 17:04:14 +0200 | [diff] [blame] | 1012 | |
| 1013 | if (remote) { |
| 1014 | _req_mod(req, TO_BE_SENT); |
| 1015 | _req_mod(req, QUEUE_FOR_NET_WRITE); |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 1016 | } else if (drbd_set_out_of_sync(device, req->i.sector, req->i.size)) |
Lars Ellenberg | 5da9c83 | 2012-03-29 17:04:14 +0200 | [diff] [blame] | 1017 | _req_mod(req, QUEUE_FOR_SEND_OOS); |
| 1018 | |
| 1019 | return remote; |
| 1020 | } |
| 1021 | |
| 1022 | static void |
| 1023 | drbd_submit_req_private_bio(struct drbd_request *req) |
| 1024 | { |
Andreas Gruenbacher | 84b8c06 | 2011-07-28 15:27:51 +0200 | [diff] [blame] | 1025 | struct drbd_device *device = req->device; |
Lars Ellenberg | 5da9c83 | 2012-03-29 17:04:14 +0200 | [diff] [blame] | 1026 | struct bio *bio = req->private_bio; |
| 1027 | const int rw = bio_rw(bio); |
| 1028 | |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 1029 | bio->bi_bdev = device->ldev->backing_bdev; |
Lars Ellenberg | 5da9c83 | 2012-03-29 17:04:14 +0200 | [diff] [blame] | 1030 | |
| 1031 | /* State may have changed since we grabbed our reference on the |
| 1032 | * ->ldev member. Double check, and short-circuit to endio. |
| 1033 | * In case the last activity log transaction failed to get on |
| 1034 | * stable storage, and this is a WRITE, we may not even submit |
| 1035 | * this bio. */ |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 1036 | if (get_ldev(device)) { |
| 1037 | if (drbd_insert_fault(device, |
Lars Ellenberg | 5da9c83 | 2012-03-29 17:04:14 +0200 | [diff] [blame] | 1038 | rw == WRITE ? DRBD_FAULT_DT_WR |
| 1039 | : rw == READ ? DRBD_FAULT_DT_RD |
| 1040 | : DRBD_FAULT_DT_RA)) |
| 1041 | bio_endio(bio, -EIO); |
| 1042 | else |
| 1043 | generic_make_request(bio); |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 1044 | put_ldev(device); |
Lars Ellenberg | 5da9c83 | 2012-03-29 17:04:14 +0200 | [diff] [blame] | 1045 | } else |
| 1046 | bio_endio(bio, -EIO); |
| 1047 | } |
| 1048 | |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 1049 | static void drbd_queue_write(struct drbd_device *device, struct drbd_request *req) |
Lars Ellenberg | 779b3fe | 2013-03-19 18:16:54 +0100 | [diff] [blame] | 1050 | { |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 1051 | spin_lock(&device->submit.lock); |
| 1052 | list_add_tail(&req->tl_requests, &device->submit.writes); |
| 1053 | spin_unlock(&device->submit.lock); |
| 1054 | queue_work(device->submit.wq, &device->submit.worker); |
Lars Ellenberg | 779b3fe | 2013-03-19 18:16:54 +0100 | [diff] [blame] | 1055 | } |
| 1056 | |
Lars Ellenberg | 6d9febe | 2013-03-19 18:16:50 +0100 | [diff] [blame] | 1057 | /* returns the new drbd_request pointer, if the caller is expected to |
| 1058 | * drbd_send_and_submit() it (to save latency), or NULL if we queued the |
| 1059 | * request on the submitter thread. |
| 1060 | * Returns ERR_PTR(-ENOMEM) if we cannot allocate a drbd_request. |
| 1061 | */ |
Rashika Kheria | 01cd263 | 2013-12-19 15:12:27 +0530 | [diff] [blame] | 1062 | static struct drbd_request * |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 1063 | drbd_request_prepare(struct drbd_device *device, struct bio *bio, unsigned long start_time) |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 1064 | { |
Lars Ellenberg | 6d9febe | 2013-03-19 18:16:50 +0100 | [diff] [blame] | 1065 | const int rw = bio_data_dir(bio); |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 1066 | struct drbd_request *req; |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 1067 | |
| 1068 | /* allocate outside of all locks; */ |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 1069 | req = drbd_req_new(device, bio); |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 1070 | if (!req) { |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 1071 | dec_ap_bio(device); |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 1072 | /* only pass the error to the upper layers. |
| 1073 | * if user cannot handle io errors, that's not our business. */ |
Andreas Gruenbacher | d018017 | 2011-07-03 17:53:52 +0200 | [diff] [blame] | 1074 | drbd_err(device, "could not kmalloc() req\n"); |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 1075 | bio_endio(bio, -ENOMEM); |
Lars Ellenberg | 6d9febe | 2013-03-19 18:16:50 +0100 | [diff] [blame] | 1076 | return ERR_PTR(-ENOMEM); |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 1077 | } |
Philipp Reisner | aeda1cd6 | 2010-11-09 17:45:06 +0100 | [diff] [blame] | 1078 | req->start_time = start_time; |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 1079 | |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 1080 | if (!get_ldev(device)) { |
Lars Ellenberg | 5da9c83 | 2012-03-29 17:04:14 +0200 | [diff] [blame] | 1081 | bio_put(req->private_bio); |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 1082 | req->private_bio = NULL; |
| 1083 | } |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 1084 | |
Lars Ellenberg | 7e8c288 | 2013-03-19 18:16:57 +0100 | [diff] [blame] | 1085 | /* Update disk stats */ |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 1086 | _drbd_start_io_acct(device, req); |
Lars Ellenberg | 7e8c288 | 2013-03-19 18:16:57 +0100 | [diff] [blame] | 1087 | |
Lars Ellenberg | 519b6d3e | 2012-08-03 02:19:09 +0200 | [diff] [blame] | 1088 | if (rw == WRITE && req->private_bio && req->i.size |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 1089 | && !test_bit(AL_SUSPENDED, &device->flags)) { |
| 1090 | if (!drbd_al_begin_io_fastpath(device, &req->i)) { |
| 1091 | drbd_queue_write(device, req); |
Lars Ellenberg | 779b3fe | 2013-03-19 18:16:54 +0100 | [diff] [blame] | 1092 | return NULL; |
| 1093 | } |
Philipp Reisner | 0778286 | 2010-08-31 12:00:50 +0200 | [diff] [blame] | 1094 | req->rq_state |= RQ_IN_ACT_LOG; |
Philipp Reisner | 0778286 | 2010-08-31 12:00:50 +0200 | [diff] [blame] | 1095 | } |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 1096 | |
Lars Ellenberg | 6d9febe | 2013-03-19 18:16:50 +0100 | [diff] [blame] | 1097 | return req; |
| 1098 | } |
| 1099 | |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 1100 | static void drbd_send_and_submit(struct drbd_device *device, struct drbd_request *req) |
Lars Ellenberg | 6d9febe | 2013-03-19 18:16:50 +0100 | [diff] [blame] | 1101 | { |
Lars Ellenberg | 35b5ed5 | 2013-12-04 12:07:09 +0100 | [diff] [blame] | 1102 | struct drbd_resource *resource = device->resource; |
Lars Ellenberg | 6d9febe | 2013-03-19 18:16:50 +0100 | [diff] [blame] | 1103 | const int rw = bio_rw(req->master_bio); |
| 1104 | struct bio_and_error m = { NULL, }; |
| 1105 | bool no_remote = false; |
Lars Ellenberg | 35b5ed5 | 2013-12-04 12:07:09 +0100 | [diff] [blame] | 1106 | bool submit_private_bio = false; |
Lars Ellenberg | 6d9febe | 2013-03-19 18:16:50 +0100 | [diff] [blame] | 1107 | |
Lars Ellenberg | 35b5ed5 | 2013-12-04 12:07:09 +0100 | [diff] [blame] | 1108 | spin_lock_irq(&resource->req_lock); |
Andreas Gruenbacher | 6024fec | 2011-01-28 15:53:51 +0100 | [diff] [blame] | 1109 | if (rw == WRITE) { |
Lars Ellenberg | 648e46b | 2012-03-26 20:12:24 +0200 | [diff] [blame] | 1110 | /* This may temporarily give up the req_lock, |
| 1111 | * but will re-aquire it before it returns here. |
| 1112 | * Needs to be before the check on drbd_suspended() */ |
| 1113 | complete_conflicting_writes(req); |
Lars Ellenberg | 607f25e | 2013-03-27 14:08:45 +0100 | [diff] [blame] | 1114 | /* no more giving up req_lock from now on! */ |
| 1115 | |
| 1116 | /* check for congestion, and potentially stop sending |
| 1117 | * full data updates, but start sending "dirty bits" only. */ |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 1118 | maybe_pull_ahead(device); |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 1119 | } |
| 1120 | |
Lars Ellenberg | 5da9c83 | 2012-03-29 17:04:14 +0200 | [diff] [blame] | 1121 | |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 1122 | if (drbd_suspended(device)) { |
Lars Ellenberg | 5da9c83 | 2012-03-29 17:04:14 +0200 | [diff] [blame] | 1123 | /* push back and retry: */ |
| 1124 | req->rq_state |= RQ_POSTPONED; |
| 1125 | if (req->private_bio) { |
| 1126 | bio_put(req->private_bio); |
| 1127 | req->private_bio = NULL; |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 1128 | put_ldev(device); |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 1129 | } |
Lars Ellenberg | 5da9c83 | 2012-03-29 17:04:14 +0200 | [diff] [blame] | 1130 | goto out; |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 1131 | } |
| 1132 | |
Lars Ellenberg | 5da9c83 | 2012-03-29 17:04:14 +0200 | [diff] [blame] | 1133 | /* We fail READ/READA early, if we can not serve it. |
| 1134 | * We must do this before req is registered on any lists. |
Lars Ellenberg | a0d856d | 2012-01-24 17:19:42 +0100 | [diff] [blame] | 1135 | * Otherwise, drbd_req_complete() will queue failed READ for retry. */ |
Lars Ellenberg | 5da9c83 | 2012-03-29 17:04:14 +0200 | [diff] [blame] | 1136 | if (rw != WRITE) { |
| 1137 | if (!do_remote_read(req) && !req->private_bio) |
| 1138 | goto nodata; |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 1139 | } |
| 1140 | |
Lars Ellenberg | b6dd1a8 | 2011-11-28 15:04:49 +0100 | [diff] [blame] | 1141 | /* which transfer log epoch does this belong to? */ |
Andreas Gruenbacher | a6b32bc | 2011-05-31 14:33:49 +0200 | [diff] [blame] | 1142 | req->epoch = atomic_read(&first_peer_device(device)->connection->current_tle_nr); |
Philipp Reisner | 288f422 | 2010-05-27 15:07:43 +0200 | [diff] [blame] | 1143 | |
Lars Ellenberg | 227f052 | 2012-07-31 09:31:11 +0200 | [diff] [blame] | 1144 | /* no point in adding empty flushes to the transfer log, |
| 1145 | * they are mapped to drbd barriers already. */ |
Lars Ellenberg | 99b4d8f | 2012-08-07 06:42:09 +0200 | [diff] [blame] | 1146 | if (likely(req->i.size!=0)) { |
| 1147 | if (rw == WRITE) |
Andreas Gruenbacher | a6b32bc | 2011-05-31 14:33:49 +0200 | [diff] [blame] | 1148 | first_peer_device(device)->connection->current_tle_writes++; |
Philipp Reisner | 288f422 | 2010-05-27 15:07:43 +0200 | [diff] [blame] | 1149 | |
Andreas Gruenbacher | a6b32bc | 2011-05-31 14:33:49 +0200 | [diff] [blame] | 1150 | list_add_tail(&req->tl_requests, &first_peer_device(device)->connection->transfer_log); |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 1151 | } |
Philipp Reisner | 6753171 | 2010-10-27 12:21:30 +0200 | [diff] [blame] | 1152 | |
Lars Ellenberg | 5da9c83 | 2012-03-29 17:04:14 +0200 | [diff] [blame] | 1153 | if (rw == WRITE) { |
| 1154 | if (!drbd_process_write_request(req)) |
| 1155 | no_remote = true; |
| 1156 | } else { |
| 1157 | /* We either have a private_bio, or we can read from remote. |
| 1158 | * Otherwise we had done the goto nodata above. */ |
| 1159 | if (req->private_bio == NULL) { |
| 1160 | _req_mod(req, TO_BE_SENT); |
| 1161 | _req_mod(req, QUEUE_FOR_NET_READ); |
Lars Ellenberg | 6719fb0 | 2010-10-18 23:04:07 +0200 | [diff] [blame] | 1162 | } else |
Lars Ellenberg | 5da9c83 | 2012-03-29 17:04:14 +0200 | [diff] [blame] | 1163 | no_remote = true; |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 1164 | } |
| 1165 | |
Lars Ellenberg | 5da9c83 | 2012-03-29 17:04:14 +0200 | [diff] [blame] | 1166 | if (req->private_bio) { |
| 1167 | /* needs to be marked within the same spinlock */ |
| 1168 | _req_mod(req, TO_BE_SUBMITTED); |
| 1169 | /* but we need to give up the spinlock to submit */ |
Lars Ellenberg | 35b5ed5 | 2013-12-04 12:07:09 +0100 | [diff] [blame] | 1170 | submit_private_bio = true; |
Lars Ellenberg | 5da9c83 | 2012-03-29 17:04:14 +0200 | [diff] [blame] | 1171 | } else if (no_remote) { |
| 1172 | nodata: |
| 1173 | if (__ratelimit(&drbd_ratelimit_state)) |
Andreas Gruenbacher | d018017 | 2011-07-03 17:53:52 +0200 | [diff] [blame] | 1174 | drbd_err(device, "IO ERROR: neither local nor remote data, sector %llu+%u\n", |
Lars Ellenberg | 42839f6 | 2012-09-27 15:19:38 +0200 | [diff] [blame] | 1175 | (unsigned long long)req->i.sector, req->i.size >> 9); |
Lars Ellenberg | 5da9c83 | 2012-03-29 17:04:14 +0200 | [diff] [blame] | 1176 | /* A write may have been queued for send_oos, however. |
Lars Ellenberg | a0d856d | 2012-01-24 17:19:42 +0100 | [diff] [blame] | 1177 | * So we can not simply free it, we must go through drbd_req_put_completion_ref() */ |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 1178 | } |
| 1179 | |
Lars Ellenberg | 5da9c83 | 2012-03-29 17:04:14 +0200 | [diff] [blame] | 1180 | out: |
Lars Ellenberg | a0d856d | 2012-01-24 17:19:42 +0100 | [diff] [blame] | 1181 | if (drbd_req_put_completion_ref(req, &m, 1)) |
| 1182 | kref_put(&req->kref, drbd_req_destroy); |
Lars Ellenberg | 35b5ed5 | 2013-12-04 12:07:09 +0100 | [diff] [blame] | 1183 | spin_unlock_irq(&resource->req_lock); |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 1184 | |
Lars Ellenberg | 35b5ed5 | 2013-12-04 12:07:09 +0100 | [diff] [blame] | 1185 | /* Even though above is a kref_put(), this is safe. |
| 1186 | * As long as we still need to submit our private bio, |
| 1187 | * we hold a completion ref, and the request cannot disappear. |
| 1188 | * If however this request did not even have a private bio to submit |
| 1189 | * (e.g. remote read), req may already be invalid now. |
| 1190 | * That's why we cannot check on req->private_bio. */ |
| 1191 | if (submit_private_bio) |
| 1192 | drbd_submit_req_private_bio(req); |
Lars Ellenberg | 5da9c83 | 2012-03-29 17:04:14 +0200 | [diff] [blame] | 1193 | if (m.bio) |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 1194 | complete_master_bio(device, &m); |
Lars Ellenberg | 6d9febe | 2013-03-19 18:16:50 +0100 | [diff] [blame] | 1195 | } |
| 1196 | |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 1197 | void __drbd_make_request(struct drbd_device *device, struct bio *bio, unsigned long start_time) |
Lars Ellenberg | 6d9febe | 2013-03-19 18:16:50 +0100 | [diff] [blame] | 1198 | { |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 1199 | struct drbd_request *req = drbd_request_prepare(device, bio, start_time); |
Lars Ellenberg | 6d9febe | 2013-03-19 18:16:50 +0100 | [diff] [blame] | 1200 | if (IS_ERR_OR_NULL(req)) |
| 1201 | return; |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 1202 | drbd_send_and_submit(device, req); |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 1203 | } |
| 1204 | |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 1205 | static void submit_fast_path(struct drbd_device *device, struct list_head *incoming) |
Lars Ellenberg | 113fef9 | 2013-03-22 18:14:40 -0600 | [diff] [blame] | 1206 | { |
Lars Ellenberg | 08a1dda | 2013-03-19 18:16:56 +0100 | [diff] [blame] | 1207 | struct drbd_request *req, *tmp; |
| 1208 | list_for_each_entry_safe(req, tmp, incoming, tl_requests) { |
| 1209 | const int rw = bio_data_dir(req->master_bio); |
Lars Ellenberg | 113fef9 | 2013-03-22 18:14:40 -0600 | [diff] [blame] | 1210 | |
Lars Ellenberg | 08a1dda | 2013-03-19 18:16:56 +0100 | [diff] [blame] | 1211 | if (rw == WRITE /* rw != WRITE should not even end up here! */ |
| 1212 | && req->private_bio && req->i.size |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 1213 | && !test_bit(AL_SUSPENDED, &device->flags)) { |
| 1214 | if (!drbd_al_begin_io_fastpath(device, &req->i)) |
Lars Ellenberg | 08a1dda | 2013-03-19 18:16:56 +0100 | [diff] [blame] | 1215 | continue; |
| 1216 | |
| 1217 | req->rq_state |= RQ_IN_ACT_LOG; |
| 1218 | } |
| 1219 | |
| 1220 | list_del_init(&req->tl_requests); |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 1221 | drbd_send_and_submit(device, req); |
Lars Ellenberg | 113fef9 | 2013-03-22 18:14:40 -0600 | [diff] [blame] | 1222 | } |
Lars Ellenberg | 113fef9 | 2013-03-22 18:14:40 -0600 | [diff] [blame] | 1223 | } |
| 1224 | |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 1225 | static bool prepare_al_transaction_nonblock(struct drbd_device *device, |
Lars Ellenberg | 08a1dda | 2013-03-19 18:16:56 +0100 | [diff] [blame] | 1226 | struct list_head *incoming, |
| 1227 | struct list_head *pending) |
| 1228 | { |
| 1229 | struct drbd_request *req, *tmp; |
| 1230 | int wake = 0; |
| 1231 | int err; |
| 1232 | |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 1233 | spin_lock_irq(&device->al_lock); |
Lars Ellenberg | 08a1dda | 2013-03-19 18:16:56 +0100 | [diff] [blame] | 1234 | list_for_each_entry_safe(req, tmp, incoming, tl_requests) { |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 1235 | err = drbd_al_begin_io_nonblock(device, &req->i); |
Lars Ellenberg | 08a1dda | 2013-03-19 18:16:56 +0100 | [diff] [blame] | 1236 | if (err == -EBUSY) |
| 1237 | wake = 1; |
| 1238 | if (err) |
| 1239 | continue; |
| 1240 | req->rq_state |= RQ_IN_ACT_LOG; |
| 1241 | list_move_tail(&req->tl_requests, pending); |
| 1242 | } |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 1243 | spin_unlock_irq(&device->al_lock); |
Lars Ellenberg | 08a1dda | 2013-03-19 18:16:56 +0100 | [diff] [blame] | 1244 | if (wake) |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 1245 | wake_up(&device->al_wait); |
Lars Ellenberg | 08a1dda | 2013-03-19 18:16:56 +0100 | [diff] [blame] | 1246 | |
| 1247 | return !list_empty(pending); |
| 1248 | } |
Lars Ellenberg | 113fef9 | 2013-03-22 18:14:40 -0600 | [diff] [blame] | 1249 | |
| 1250 | void do_submit(struct work_struct *ws) |
| 1251 | { |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 1252 | struct drbd_device *device = container_of(ws, struct drbd_device, submit.worker); |
Lars Ellenberg | 08a1dda | 2013-03-19 18:16:56 +0100 | [diff] [blame] | 1253 | LIST_HEAD(incoming); |
| 1254 | LIST_HEAD(pending); |
Lars Ellenberg | 113fef9 | 2013-03-22 18:14:40 -0600 | [diff] [blame] | 1255 | struct drbd_request *req, *tmp; |
| 1256 | |
Lars Ellenberg | 08a1dda | 2013-03-19 18:16:56 +0100 | [diff] [blame] | 1257 | for (;;) { |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 1258 | spin_lock(&device->submit.lock); |
| 1259 | list_splice_tail_init(&device->submit.writes, &incoming); |
| 1260 | spin_unlock(&device->submit.lock); |
Lars Ellenberg | 113fef9 | 2013-03-22 18:14:40 -0600 | [diff] [blame] | 1261 | |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 1262 | submit_fast_path(device, &incoming); |
Lars Ellenberg | 08a1dda | 2013-03-19 18:16:56 +0100 | [diff] [blame] | 1263 | if (list_empty(&incoming)) |
| 1264 | break; |
| 1265 | |
Lars Ellenberg | e4d7d6f | 2014-04-28 18:43:28 +0200 | [diff] [blame] | 1266 | skip_fast_path: |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 1267 | wait_event(device->al_wait, prepare_al_transaction_nonblock(device, &incoming, &pending)); |
Lars Ellenberg | 45ad07b | 2013-03-19 18:16:58 +0100 | [diff] [blame] | 1268 | /* Maybe more was queued, while we prepared the transaction? |
| 1269 | * Try to stuff them into this transaction as well. |
| 1270 | * Be strictly non-blocking here, no wait_event, we already |
| 1271 | * have something to commit. |
| 1272 | * Stop if we don't make any more progres. |
| 1273 | */ |
| 1274 | for (;;) { |
| 1275 | LIST_HEAD(more_pending); |
| 1276 | LIST_HEAD(more_incoming); |
| 1277 | bool made_progress; |
| 1278 | |
| 1279 | /* It is ok to look outside the lock, |
| 1280 | * it's only an optimization anyways */ |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 1281 | if (list_empty(&device->submit.writes)) |
Lars Ellenberg | 45ad07b | 2013-03-19 18:16:58 +0100 | [diff] [blame] | 1282 | break; |
| 1283 | |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 1284 | spin_lock(&device->submit.lock); |
| 1285 | list_splice_tail_init(&device->submit.writes, &more_incoming); |
| 1286 | spin_unlock(&device->submit.lock); |
Lars Ellenberg | 45ad07b | 2013-03-19 18:16:58 +0100 | [diff] [blame] | 1287 | |
| 1288 | if (list_empty(&more_incoming)) |
| 1289 | break; |
| 1290 | |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 1291 | made_progress = prepare_al_transaction_nonblock(device, &more_incoming, &more_pending); |
Lars Ellenberg | 45ad07b | 2013-03-19 18:16:58 +0100 | [diff] [blame] | 1292 | |
| 1293 | list_splice_tail_init(&more_pending, &pending); |
| 1294 | list_splice_tail_init(&more_incoming, &incoming); |
| 1295 | |
| 1296 | if (!made_progress) |
| 1297 | break; |
| 1298 | } |
Lars Ellenberg | 4dd726f | 2014-02-11 11:15:36 +0100 | [diff] [blame] | 1299 | drbd_al_begin_io_commit(device); |
Lars Ellenberg | 08a1dda | 2013-03-19 18:16:56 +0100 | [diff] [blame] | 1300 | |
| 1301 | list_for_each_entry_safe(req, tmp, &pending, tl_requests) { |
| 1302 | list_del_init(&req->tl_requests); |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 1303 | drbd_send_and_submit(device, req); |
Lars Ellenberg | 08a1dda | 2013-03-19 18:16:56 +0100 | [diff] [blame] | 1304 | } |
Lars Ellenberg | e4d7d6f | 2014-04-28 18:43:28 +0200 | [diff] [blame] | 1305 | |
| 1306 | /* If all currently hot activity log extents are kept busy by |
| 1307 | * incoming requests, we still must not totally starve new |
| 1308 | * requests to cold extents. In that case, prepare one request |
| 1309 | * in blocking mode. */ |
| 1310 | list_for_each_entry_safe(req, tmp, &incoming, tl_requests) { |
| 1311 | list_del_init(&req->tl_requests); |
| 1312 | req->rq_state |= RQ_IN_ACT_LOG; |
| 1313 | if (!drbd_al_begin_io_prepare(device, &req->i)) { |
| 1314 | /* Corresponding extent was hot after all? */ |
| 1315 | drbd_send_and_submit(device, req); |
| 1316 | } else { |
| 1317 | /* Found a request to a cold extent. |
| 1318 | * Put on "pending" list, |
| 1319 | * and try to cumulate with more. */ |
| 1320 | list_add(&req->tl_requests, &pending); |
| 1321 | goto skip_fast_path; |
| 1322 | } |
| 1323 | } |
Lars Ellenberg | 113fef9 | 2013-03-22 18:14:40 -0600 | [diff] [blame] | 1324 | } |
| 1325 | } |
| 1326 | |
Christoph Hellwig | 5a7bbad | 2011-09-12 12:12:01 +0200 | [diff] [blame] | 1327 | void drbd_make_request(struct request_queue *q, struct bio *bio) |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 1328 | { |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 1329 | struct drbd_device *device = (struct drbd_device *) q->queuedata; |
Philipp Reisner | aeda1cd6 | 2010-11-09 17:45:06 +0100 | [diff] [blame] | 1330 | unsigned long start_time; |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 1331 | |
Philipp Reisner | aeda1cd6 | 2010-11-09 17:45:06 +0100 | [diff] [blame] | 1332 | start_time = jiffies; |
| 1333 | |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 1334 | /* |
| 1335 | * what we "blindly" assume: |
| 1336 | */ |
Andreas Gruenbacher | 0b0ba1e | 2011-06-27 16:23:33 +0200 | [diff] [blame] | 1337 | D_ASSERT(device, IS_ALIGNED(bio->bi_iter.bi_size, 512)); |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 1338 | |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 1339 | inc_ap_bio(device); |
| 1340 | __drbd_make_request(device, bio, start_time); |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 1341 | } |
| 1342 | |
Lars Ellenberg | 23361cf | 2011-03-31 16:36:43 +0200 | [diff] [blame] | 1343 | /* This is called by bio_add_page(). |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 1344 | * |
Lars Ellenberg | 23361cf | 2011-03-31 16:36:43 +0200 | [diff] [blame] | 1345 | * q->max_hw_sectors and other global limits are already enforced there. |
| 1346 | * |
| 1347 | * We need to call down to our lower level device, |
| 1348 | * in case it has special restrictions. |
| 1349 | * |
| 1350 | * We also may need to enforce configured max-bio-bvecs limits. |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 1351 | * |
| 1352 | * As long as the BIO is empty we have to allow at least one bvec, |
Lars Ellenberg | 23361cf | 2011-03-31 16:36:43 +0200 | [diff] [blame] | 1353 | * regardless of size and offset, so no need to ask lower levels. |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 1354 | */ |
| 1355 | int drbd_merge_bvec(struct request_queue *q, struct bvec_merge_data *bvm, struct bio_vec *bvec) |
| 1356 | { |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 1357 | struct drbd_device *device = (struct drbd_device *) q->queuedata; |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 1358 | unsigned int bio_size = bvm->bi_size; |
Lars Ellenberg | 23361cf | 2011-03-31 16:36:43 +0200 | [diff] [blame] | 1359 | int limit = DRBD_MAX_BIO_SIZE; |
| 1360 | int backing_limit; |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 1361 | |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 1362 | if (bio_size && get_ldev(device)) { |
Lars Ellenberg | 35f47ef | 2013-10-23 10:59:19 +0200 | [diff] [blame] | 1363 | unsigned int max_hw_sectors = queue_max_hw_sectors(q); |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 1364 | struct request_queue * const b = |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 1365 | device->ldev->backing_bdev->bd_disk->queue; |
Lars Ellenberg | a1c88d0 | 2010-05-14 19:16:41 +0200 | [diff] [blame] | 1366 | if (b->merge_bvec_fn) { |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 1367 | backing_limit = b->merge_bvec_fn(b, bvm, bvec); |
| 1368 | limit = min(limit, backing_limit); |
| 1369 | } |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 1370 | put_ldev(device); |
Lars Ellenberg | 35f47ef | 2013-10-23 10:59:19 +0200 | [diff] [blame] | 1371 | if ((limit >> 9) > max_hw_sectors) |
| 1372 | limit = max_hw_sectors << 9; |
Philipp Reisner | b411b36 | 2009-09-25 16:07:19 -0700 | [diff] [blame] | 1373 | } |
| 1374 | return limit; |
| 1375 | } |
Philipp Reisner | 7fde2be | 2011-03-01 11:08:28 +0100 | [diff] [blame] | 1376 | |
Lars Ellenberg | 0853546 | 2014-04-28 18:43:31 +0200 | [diff] [blame] | 1377 | static void find_oldest_requests( |
| 1378 | struct drbd_connection *connection, |
| 1379 | struct drbd_device *device, |
| 1380 | struct drbd_request **oldest_req_waiting_for_peer, |
| 1381 | struct drbd_request **oldest_req_waiting_for_disk) |
Lars Ellenberg | b6dd1a8 | 2011-11-28 15:04:49 +0100 | [diff] [blame] | 1382 | { |
Lars Ellenberg | b6dd1a8 | 2011-11-28 15:04:49 +0100 | [diff] [blame] | 1383 | struct drbd_request *r; |
Lars Ellenberg | 0853546 | 2014-04-28 18:43:31 +0200 | [diff] [blame] | 1384 | *oldest_req_waiting_for_peer = NULL; |
| 1385 | *oldest_req_waiting_for_disk = NULL; |
Andreas Gruenbacher | bde89a9 | 2011-05-30 16:32:41 +0200 | [diff] [blame] | 1386 | list_for_each_entry(r, &connection->transfer_log, tl_requests) { |
Lars Ellenberg | 0853546 | 2014-04-28 18:43:31 +0200 | [diff] [blame] | 1387 | const unsigned s = r->rq_state; |
| 1388 | if (!*oldest_req_waiting_for_peer |
| 1389 | && ((s & RQ_NET_MASK) && !(s & RQ_NET_DONE))) |
| 1390 | *oldest_req_waiting_for_peer = r; |
| 1391 | |
| 1392 | if (!*oldest_req_waiting_for_disk |
| 1393 | && (s & RQ_LOCAL_PENDING) && r->device == device) |
| 1394 | *oldest_req_waiting_for_disk = r; |
| 1395 | |
| 1396 | if (*oldest_req_waiting_for_peer && *oldest_req_waiting_for_disk) |
| 1397 | break; |
Lars Ellenberg | b6dd1a8 | 2011-11-28 15:04:49 +0100 | [diff] [blame] | 1398 | } |
Lars Ellenberg | b6dd1a8 | 2011-11-28 15:04:49 +0100 | [diff] [blame] | 1399 | } |
| 1400 | |
Philipp Reisner | 7fde2be | 2011-03-01 11:08:28 +0100 | [diff] [blame] | 1401 | void request_timer_fn(unsigned long data) |
| 1402 | { |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 1403 | struct drbd_device *device = (struct drbd_device *) data; |
Andreas Gruenbacher | a6b32bc | 2011-05-31 14:33:49 +0200 | [diff] [blame] | 1404 | struct drbd_connection *connection = first_peer_device(device)->connection; |
Lars Ellenberg | 0853546 | 2014-04-28 18:43:31 +0200 | [diff] [blame] | 1405 | struct drbd_request *req_disk, *req_peer; /* oldest request */ |
Philipp Reisner | 44ed167 | 2011-04-19 17:10:19 +0200 | [diff] [blame] | 1406 | struct net_conf *nc; |
Philipp Reisner | dfa8bed | 2011-06-29 14:06:08 +0200 | [diff] [blame] | 1407 | unsigned long ent = 0, dt = 0, et, nt; /* effective timeout = ko_count * timeout */ |
Lars Ellenberg | ba280c0 | 2012-04-25 11:46:14 +0200 | [diff] [blame] | 1408 | unsigned long now; |
Philipp Reisner | 7fde2be | 2011-03-01 11:08:28 +0100 | [diff] [blame] | 1409 | |
Philipp Reisner | 44ed167 | 2011-04-19 17:10:19 +0200 | [diff] [blame] | 1410 | rcu_read_lock(); |
Andreas Gruenbacher | bde89a9 | 2011-05-30 16:32:41 +0200 | [diff] [blame] | 1411 | nc = rcu_dereference(connection->net_conf); |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 1412 | if (nc && device->state.conn >= C_WF_REPORT_PARAMS) |
Lars Ellenberg | 07be15b | 2012-05-07 11:53:08 +0200 | [diff] [blame] | 1413 | ent = nc->timeout * HZ/10 * nc->ko_count; |
Philipp Reisner | cdfda63 | 2011-07-05 15:38:59 +0200 | [diff] [blame] | 1414 | |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 1415 | if (get_ldev(device)) { /* implicit state.disk >= D_INCONSISTENT */ |
| 1416 | dt = rcu_dereference(device->ldev->disk_conf)->disk_timeout * HZ / 10; |
| 1417 | put_ldev(device); |
Philipp Reisner | dfa8bed | 2011-06-29 14:06:08 +0200 | [diff] [blame] | 1418 | } |
Philipp Reisner | 44ed167 | 2011-04-19 17:10:19 +0200 | [diff] [blame] | 1419 | rcu_read_unlock(); |
| 1420 | |
Philipp Reisner | dfa8bed | 2011-06-29 14:06:08 +0200 | [diff] [blame] | 1421 | et = min_not_zero(dt, ent); |
| 1422 | |
Lars Ellenberg | ba280c0 | 2012-04-25 11:46:14 +0200 | [diff] [blame] | 1423 | if (!et) |
Philipp Reisner | 7fde2be | 2011-03-01 11:08:28 +0100 | [diff] [blame] | 1424 | return; /* Recurring timer stopped */ |
| 1425 | |
Lars Ellenberg | ba280c0 | 2012-04-25 11:46:14 +0200 | [diff] [blame] | 1426 | now = jiffies; |
| 1427 | |
Andreas Gruenbacher | 0500813 | 2011-07-07 14:19:42 +0200 | [diff] [blame] | 1428 | spin_lock_irq(&device->resource->req_lock); |
Lars Ellenberg | 0853546 | 2014-04-28 18:43:31 +0200 | [diff] [blame] | 1429 | find_oldest_requests(connection, device, &req_peer, &req_disk); |
| 1430 | if (req_peer == NULL && req_disk == NULL) { |
Andreas Gruenbacher | 0500813 | 2011-07-07 14:19:42 +0200 | [diff] [blame] | 1431 | spin_unlock_irq(&device->resource->req_lock); |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 1432 | mod_timer(&device->request_timer, now + et); |
Philipp Reisner | 7fde2be | 2011-03-01 11:08:28 +0100 | [diff] [blame] | 1433 | return; |
| 1434 | } |
| 1435 | |
Lars Ellenberg | ba280c0 | 2012-04-25 11:46:14 +0200 | [diff] [blame] | 1436 | /* The request is considered timed out, if |
| 1437 | * - we have some effective timeout from the configuration, |
| 1438 | * with above state restrictions applied, |
| 1439 | * - the oldest request is waiting for a response from the network |
| 1440 | * resp. the local disk, |
| 1441 | * - the oldest request is in fact older than the effective timeout, |
| 1442 | * - the connection was established (resp. disk was attached) |
| 1443 | * for longer than the timeout already. |
| 1444 | * Note that for 32bit jiffies and very stable connections/disks, |
| 1445 | * we may have a wrap around, which is catched by |
| 1446 | * !time_in_range(now, last_..._jif, last_..._jif + timeout). |
| 1447 | * |
| 1448 | * Side effect: once per 32bit wrap-around interval, which means every |
| 1449 | * ~198 days with 250 HZ, we have a window where the timeout would need |
| 1450 | * to expire twice (worst case) to become effective. Good enough. |
| 1451 | */ |
Lars Ellenberg | 0853546 | 2014-04-28 18:43:31 +0200 | [diff] [blame] | 1452 | if (ent && req_peer && |
| 1453 | time_after(now, req_peer->start_time + ent) && |
Andreas Gruenbacher | bde89a9 | 2011-05-30 16:32:41 +0200 | [diff] [blame] | 1454 | !time_in_range(now, connection->last_reconnect_jif, connection->last_reconnect_jif + ent)) { |
Andreas Gruenbacher | d018017 | 2011-07-03 17:53:52 +0200 | [diff] [blame] | 1455 | drbd_warn(device, "Remote failed to finish a request within ko-count * timeout\n"); |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 1456 | _drbd_set_state(_NS(device, conn, C_TIMEOUT), CS_VERBOSE | CS_HARD, NULL); |
Philipp Reisner | 7fde2be | 2011-03-01 11:08:28 +0100 | [diff] [blame] | 1457 | } |
Lars Ellenberg | 0853546 | 2014-04-28 18:43:31 +0200 | [diff] [blame] | 1458 | if (dt && req_disk && |
| 1459 | time_after(now, req_disk->start_time + dt) && |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 1460 | !time_in_range(now, device->last_reattach_jif, device->last_reattach_jif + dt)) { |
Andreas Gruenbacher | d018017 | 2011-07-03 17:53:52 +0200 | [diff] [blame] | 1461 | drbd_warn(device, "Local backing device failed to meet the disk-timeout\n"); |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 1462 | __drbd_chk_io_error(device, DRBD_FORCE_DETACH); |
Philipp Reisner | dfa8bed | 2011-06-29 14:06:08 +0200 | [diff] [blame] | 1463 | } |
Lars Ellenberg | 0853546 | 2014-04-28 18:43:31 +0200 | [diff] [blame] | 1464 | |
| 1465 | /* Reschedule timer for the nearest not already expired timeout. |
| 1466 | * Fallback to now + min(effective network timeout, disk timeout). */ |
| 1467 | ent = (ent && req_peer && time_before(now, req_peer->start_time + ent)) |
| 1468 | ? req_peer->start_time + ent : now + et; |
| 1469 | dt = (dt && req_disk && time_before(now, req_disk->start_time + dt)) |
| 1470 | ? req_disk->start_time + dt : now + et; |
| 1471 | nt = time_before(ent, dt) ? ent : dt; |
Andreas Gruenbacher | 0500813 | 2011-07-07 14:19:42 +0200 | [diff] [blame] | 1472 | spin_unlock_irq(&connection->resource->req_lock); |
Andreas Gruenbacher | b30ab79 | 2011-07-03 13:26:43 +0200 | [diff] [blame] | 1473 | mod_timer(&device->request_timer, nt); |
Philipp Reisner | 7fde2be | 2011-03-01 11:08:28 +0100 | [diff] [blame] | 1474 | } |