Blame view

drivers/lightnvm/pblk-read.c 11.7 KB
02a1520d5   Javier González   lightnvm: pblk: a...
1
  // SPDX-License-Identifier: GPL-2.0
a4bd217b4   Javier González   lightnvm: physica...
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
  /*
   * Copyright (C) 2016 CNEX Labs
   * Initial release: Javier Gonzalez <javier@cnexlabs.com>
   *                  Matias Bjorling <matias@cnexlabs.com>
   *
   * This program is free software; you can redistribute it and/or
   * modify it under the terms of the GNU General Public License version
   * 2 as published by the Free Software Foundation.
   *
   * This program is distributed in the hope that it will be useful, but
   * WITHOUT ANY WARRANTY; without even the implied warranty of
   * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
   * General Public License for more details.
   *
   * pblk-read.c - pblk's read path
   */
  
  #include "pblk.h"
  
  /*
   * There is no guarantee that the value read from cache has not been updated and
   * resides at another location in the cache. We guarantee though that if the
   * value is read from the cache, it belongs to the mapped lba. In order to
   * guarantee and order between writes and reads are ordered, a flush must be
   * issued.
   */
  static int pblk_read_from_cache(struct pblk *pblk, struct bio *bio,
a96de64a2   Igor Konopko   lightnvm: pblk: s...
29
  				sector_t lba, struct ppa_addr ppa)
a4bd217b4   Javier González   lightnvm: physica...
30
  {
880eda544   Matias Bjørling   lightnvm: move NV...
31
  #ifdef CONFIG_NVM_PBLK_DEBUG
a4bd217b4   Javier González   lightnvm: physica...
32
33
34
35
  	/* Callers must ensure that the ppa points to a cache address */
  	BUG_ON(pblk_ppa_empty(ppa));
  	BUG_ON(!pblk_addr_in_cache(ppa));
  #endif
a96de64a2   Igor Konopko   lightnvm: pblk: s...
36
  	return pblk_rb_copy_to_bio(&pblk->rwb, bio, lba, ppa);
a4bd217b4   Javier González   lightnvm: physica...
37
  }
a96de64a2   Igor Konopko   lightnvm: pblk: s...
38
  static int pblk_read_ppalist_rq(struct pblk *pblk, struct nvm_rq *rqd,
87cc40bbe   Javier González   lightnvm: pblk: r...
39
  				 struct bio *bio, sector_t blba,
a96de64a2   Igor Konopko   lightnvm: pblk: s...
40
  				 bool *from_cache)
a4bd217b4   Javier González   lightnvm: physica...
41
  {
faa79f27f   Igor Konopko   lightnvm: pblk: a...
42
  	void *meta_list = rqd->meta_list;
a96de64a2   Igor Konopko   lightnvm: pblk: s...
43
  	int nr_secs, i;
a4bd217b4   Javier González   lightnvm: physica...
44

a96de64a2   Igor Konopko   lightnvm: pblk: s...
45
46
47
48
49
50
  retry:
  	nr_secs = pblk_lookup_l2p_seq(pblk, rqd->ppa_list, blba, rqd->nr_ppas,
  					from_cache);
  
  	if (!*from_cache)
  		goto end;
a4bd217b4   Javier González   lightnvm: physica...
51
52
  
  	for (i = 0; i < nr_secs; i++) {
faa79f27f   Igor Konopko   lightnvm: pblk: a...
53
  		struct pblk_sec_meta *meta = pblk_get_meta(pblk, meta_list, i);
a4bd217b4   Javier González   lightnvm: physica...
54
  		sector_t lba = blba + i;
a96de64a2   Igor Konopko   lightnvm: pblk: s...
55
  		if (pblk_ppa_empty(rqd->ppa_list[i])) {
faa79f27f   Igor Konopko   lightnvm: pblk: a...
56
  			__le64 addr_empty = cpu_to_le64(ADDR_EMPTY);
faa79f27f   Igor Konopko   lightnvm: pblk: a...
57
  			meta->lba = addr_empty;
a96de64a2   Igor Konopko   lightnvm: pblk: s...
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
  		} else if (pblk_addr_in_cache(rqd->ppa_list[i])) {
  			/*
  			 * Try to read from write buffer. The address is later
  			 * checked on the write buffer to prevent retrieving
  			 * overwritten data.
  			 */
  			if (!pblk_read_from_cache(pblk, bio, lba,
  							rqd->ppa_list[i])) {
  				if (i == 0) {
  					/*
  					 * We didn't call with bio_advance()
  					 * yet, so we can just retry.
  					 */
  					goto retry;
  				} else {
  					/*
  					 * We already call bio_advance()
  					 * so we cannot retry and we need
  					 * to quit that function in order
  					 * to allow caller to handle the bio
  					 * splitting in the current sector
  					 * position.
  					 */
  					nr_secs = i;
  					goto end;
  				}
75cb8e939   Javier González   lightnvm: pblk: a...
84
  			}
faa79f27f   Igor Konopko   lightnvm: pblk: a...
85
  			meta->lba = cpu_to_le64(lba);
880eda544   Matias Bjørling   lightnvm: move NV...
86
  #ifdef CONFIG_NVM_PBLK_DEBUG
db7ada33c   Javier González   lightnvm: pblk: a...
87
88
  			atomic_long_inc(&pblk->cache_reads);
  #endif
a4bd217b4   Javier González   lightnvm: physica...
89
  		}
a96de64a2   Igor Konopko   lightnvm: pblk: s...
90
  		bio_advance(bio, PBLK_EXPOSED_PAGE_SIZE);
a4bd217b4   Javier González   lightnvm: physica...
91
  	}
a96de64a2   Igor Konopko   lightnvm: pblk: s...
92
  end:
f9c101523   Javier González   lightnvm: pblk: i...
93
  	if (pblk_io_aligned(pblk, nr_secs))
d7b680167   Matias Bjørling   lightnvm: combine...
94
  		rqd->is_seq = 1;
f9c101523   Javier González   lightnvm: pblk: i...
95

880eda544   Matias Bjørling   lightnvm: move NV...
96
  #ifdef CONFIG_NVM_PBLK_DEBUG
a4bd217b4   Javier González   lightnvm: physica...
97
98
  	atomic_long_add(nr_secs, &pblk->inflight_reads);
  #endif
a96de64a2   Igor Konopko   lightnvm: pblk: s...
99
100
  
  	return nr_secs;
a4bd217b4   Javier González   lightnvm: physica...
101
  }
a4bd217b4   Javier González   lightnvm: physica...
102

03a34b2d2   Javier González   lightnvm: pblk: i...
103
104
  static void pblk_read_check_seq(struct pblk *pblk, struct nvm_rq *rqd,
  				sector_t blba)
a4809fee4   Javier González   lightnvm: pblk: c...
105
  {
faa79f27f   Igor Konopko   lightnvm: pblk: a...
106
  	void *meta_list = rqd->meta_list;
03a34b2d2   Javier González   lightnvm: pblk: i...
107
  	int nr_lbas = rqd->nr_ppas;
a4809fee4   Javier González   lightnvm: pblk: c...
108
  	int i;
55d8ec353   Igor Konopko   lightnvm: pblk: s...
109
110
  	if (!pblk_is_oob_meta_supported(pblk))
  		return;
a4809fee4   Javier González   lightnvm: pblk: c...
111
  	for (i = 0; i < nr_lbas; i++) {
faa79f27f   Igor Konopko   lightnvm: pblk: a...
112
113
  		struct pblk_sec_meta *meta = pblk_get_meta(pblk, meta_list, i);
  		u64 lba = le64_to_cpu(meta->lba);
a4809fee4   Javier González   lightnvm: pblk: c...
114
115
116
  
  		if (lba == ADDR_EMPTY)
  			continue;
03a34b2d2   Javier González   lightnvm: pblk: i...
117
  		if (lba != blba + i) {
880eda544   Matias Bjørling   lightnvm: move NV...
118
  #ifdef CONFIG_NVM_PBLK_DEBUG
d68a93440   Hans Holmberg   lightnvm: introdu...
119
  			struct ppa_addr *ppa_list = nvm_rq_to_ppa_list(rqd);
03a34b2d2   Javier González   lightnvm: pblk: i...
120

d68a93440   Hans Holmberg   lightnvm: introdu...
121
  			print_ppa(pblk, &ppa_list[i], "seq", i);
03a34b2d2   Javier González   lightnvm: pblk: i...
122
  #endif
4e495a46b   Matias Bjørling   lightnvm: pblk: e...
123
124
  			pblk_err(pblk, "corrupted read LBA (%llu/%llu)
  ",
03a34b2d2   Javier González   lightnvm: pblk: i...
125
126
127
  							lba, (u64)blba + i);
  			WARN_ON(1);
  		}
a4809fee4   Javier González   lightnvm: pblk: c...
128
129
  	}
  }
310df5822   Javier González   lightnvm: pblk: c...
130
131
132
  /*
   * There can be holes in the lba list.
   */
03a34b2d2   Javier González   lightnvm: pblk: i...
133
134
  static void pblk_read_check_rand(struct pblk *pblk, struct nvm_rq *rqd,
  				 u64 *lba_list, int nr_lbas)
310df5822   Javier González   lightnvm: pblk: c...
135
  {
faa79f27f   Igor Konopko   lightnvm: pblk: a...
136
  	void *meta_lba_list = rqd->meta_list;
310df5822   Javier González   lightnvm: pblk: c...
137
  	int i, j;
55d8ec353   Igor Konopko   lightnvm: pblk: s...
138
139
  	if (!pblk_is_oob_meta_supported(pblk))
  		return;
310df5822   Javier González   lightnvm: pblk: c...
140
  	for (i = 0, j = 0; i < nr_lbas; i++) {
faa79f27f   Igor Konopko   lightnvm: pblk: a...
141
142
  		struct pblk_sec_meta *meta = pblk_get_meta(pblk,
  							   meta_lba_list, j);
310df5822   Javier González   lightnvm: pblk: c...
143
144
145
146
147
  		u64 lba = lba_list[i];
  		u64 meta_lba;
  
  		if (lba == ADDR_EMPTY)
  			continue;
faa79f27f   Igor Konopko   lightnvm: pblk: a...
148
  		meta_lba = le64_to_cpu(meta->lba);
310df5822   Javier González   lightnvm: pblk: c...
149
150
  
  		if (lba != meta_lba) {
880eda544   Matias Bjørling   lightnvm: move NV...
151
  #ifdef CONFIG_NVM_PBLK_DEBUG
d68a93440   Hans Holmberg   lightnvm: introdu...
152
  			struct ppa_addr *ppa_list = nvm_rq_to_ppa_list(rqd);
03a34b2d2   Javier González   lightnvm: pblk: i...
153

ea1d24bc3   Hans Holmberg   lightnvm: pblk: f...
154
  			print_ppa(pblk, &ppa_list[j], "rnd", j);
03a34b2d2   Javier González   lightnvm: pblk: i...
155
  #endif
4e495a46b   Matias Bjørling   lightnvm: pblk: e...
156
157
  			pblk_err(pblk, "corrupted read LBA (%llu/%llu)
  ",
ea1d24bc3   Hans Holmberg   lightnvm: pblk: f...
158
  							meta_lba, lba);
310df5822   Javier González   lightnvm: pblk: c...
159
160
  			WARN_ON(1);
  		}
03a34b2d2   Javier González   lightnvm: pblk: i...
161
162
  
  		j++;
310df5822   Javier González   lightnvm: pblk: c...
163
  	}
03a34b2d2   Javier González   lightnvm: pblk: i...
164
165
166
  
  	WARN_ONCE(j != rqd->nr_ppas, "pblk: corrupted random request
  ");
310df5822   Javier González   lightnvm: pblk: c...
167
  }
d38954ed1   Igor Konopko   lightnvm: pblk: s...
168
  static void pblk_end_user_read(struct bio *bio, int error)
7bd4d370d   Javier González   lightnvm: pblk: g...
169
  {
d38954ed1   Igor Konopko   lightnvm: pblk: s...
170
171
  	if (error && error != NVM_RSP_WARN_HIGHECC)
  		bio_io_error(bio);
3e03f6322   Igor Konopko   lightnvm: pblk: I...
172
173
  	else
  		bio_endio(bio);
7bd4d370d   Javier González   lightnvm: pblk: g...
174
175
176
177
  }
  
  static void __pblk_end_io_read(struct pblk *pblk, struct nvm_rq *rqd,
  			       bool put_line)
a4bd217b4   Javier González   lightnvm: physica...
178
  {
998ba6297   Javier González   lightnvm: pblk: a...
179
  	struct nvm_tgt_dev *dev = pblk->dev;
084ec9ba0   Javier González   lightnvm: pblk: r...
180
  	struct pblk_g_ctx *r_ctx = nvm_rq_to_pdu(rqd);
87cc40bbe   Javier González   lightnvm: pblk: r...
181
  	struct bio *int_bio = rqd->bio;
998ba6297   Javier González   lightnvm: pblk: a...
182
  	unsigned long start_time = r_ctx->start_time;
ddcf35d39   Michael Callahan   block: Add and us...
183
  	generic_end_io_acct(dev->q, REQ_OP_READ, &pblk->disk->part0, start_time);
a4bd217b4   Javier González   lightnvm: physica...
184
185
186
  
  	if (rqd->error)
  		pblk_log_read_err(pblk, rqd);
a4bd217b4   Javier González   lightnvm: physica...
187

03a34b2d2   Javier González   lightnvm: pblk: i...
188
  	pblk_read_check_seq(pblk, rqd, r_ctx->lba);
a96de64a2   Igor Konopko   lightnvm: pblk: s...
189
  	bio_put(int_bio);
084ec9ba0   Javier González   lightnvm: pblk: r...
190

7bd4d370d   Javier González   lightnvm: pblk: g...
191
  	if (put_line)
ae14cc044   Matias Bjørling   lightnvm: pblk: r...
192
  		pblk_rq_to_line_put(pblk, rqd);
a4bd217b4   Javier González   lightnvm: physica...
193

880eda544   Matias Bjørling   lightnvm: move NV...
194
  #ifdef CONFIG_NVM_PBLK_DEBUG
a4bd217b4   Javier González   lightnvm: physica...
195
196
197
  	atomic_long_add(rqd->nr_ppas, &pblk->sync_reads);
  	atomic_long_sub(rqd->nr_ppas, &pblk->inflight_reads);
  #endif
e2cddf208   Javier González   lightnvm: pblk: i...
198
  	pblk_free_rqd(pblk, rqd, PBLK_READ);
588726d3e   Javier González   lightnvm: pblk: f...
199
  	atomic_dec(&pblk->inflight_io);
a4bd217b4   Javier González   lightnvm: physica...
200
  }
7bd4d370d   Javier González   lightnvm: pblk: g...
201
202
203
  static void pblk_end_io_read(struct nvm_rq *rqd)
  {
  	struct pblk *pblk = rqd->private;
87cc40bbe   Javier González   lightnvm: pblk: r...
204
205
  	struct pblk_g_ctx *r_ctx = nvm_rq_to_pdu(rqd);
  	struct bio *bio = (struct bio *)r_ctx->private;
7bd4d370d   Javier González   lightnvm: pblk: g...
206

d38954ed1   Igor Konopko   lightnvm: pblk: s...
207
  	pblk_end_user_read(bio, rqd->error);
7bd4d370d   Javier González   lightnvm: pblk: g...
208
209
  	__pblk_end_io_read(pblk, rqd, true);
  }
87cc40bbe   Javier González   lightnvm: pblk: r...
210
  static void pblk_read_rq(struct pblk *pblk, struct nvm_rq *rqd, struct bio *bio,
a96de64a2   Igor Konopko   lightnvm: pblk: s...
211
  			 sector_t lba, bool *from_cache)
a4bd217b4   Javier González   lightnvm: physica...
212
  {
faa79f27f   Igor Konopko   lightnvm: pblk: a...
213
  	struct pblk_sec_meta *meta = pblk_get_meta(pblk, rqd->meta_list, 0);
a4bd217b4   Javier González   lightnvm: physica...
214
  	struct ppa_addr ppa;
a4bd217b4   Javier González   lightnvm: physica...
215

a96de64a2   Igor Konopko   lightnvm: pblk: s...
216
  	pblk_lookup_l2p_seq(pblk, &ppa, lba, 1, from_cache);
a4bd217b4   Javier González   lightnvm: physica...
217

880eda544   Matias Bjørling   lightnvm: move NV...
218
  #ifdef CONFIG_NVM_PBLK_DEBUG
a4bd217b4   Javier González   lightnvm: physica...
219
220
221
222
223
  	atomic_long_inc(&pblk->inflight_reads);
  #endif
  
  retry:
  	if (pblk_ppa_empty(ppa)) {
faa79f27f   Igor Konopko   lightnvm: pblk: a...
224
  		__le64 addr_empty = cpu_to_le64(ADDR_EMPTY);
faa79f27f   Igor Konopko   lightnvm: pblk: a...
225
  		meta->lba = addr_empty;
a4bd217b4   Javier González   lightnvm: physica...
226
227
228
229
230
231
232
  		return;
  	}
  
  	/* Try to read from write buffer. The address is later checked on the
  	 * write buffer to prevent retrieving overwritten data.
  	 */
  	if (pblk_addr_in_cache(ppa)) {
a96de64a2   Igor Konopko   lightnvm: pblk: s...
233
234
  		if (!pblk_read_from_cache(pblk, bio, lba, ppa)) {
  			pblk_lookup_l2p_seq(pblk, &ppa, lba, 1, from_cache);
a4bd217b4   Javier González   lightnvm: physica...
235
236
  			goto retry;
  		}
a4809fee4   Javier González   lightnvm: pblk: c...
237

faa79f27f   Igor Konopko   lightnvm: pblk: a...
238
  		meta->lba = cpu_to_le64(lba);
a4809fee4   Javier González   lightnvm: pblk: c...
239

880eda544   Matias Bjørling   lightnvm: move NV...
240
  #ifdef CONFIG_NVM_PBLK_DEBUG
7bd4d370d   Javier González   lightnvm: pblk: g...
241
  		atomic_long_inc(&pblk->cache_reads);
db7ada33c   Javier González   lightnvm: pblk: a...
242
  #endif
a4bd217b4   Javier González   lightnvm: physica...
243
244
245
246
  	} else {
  		rqd->ppa_addr = ppa;
  	}
  }
3e03f6322   Igor Konopko   lightnvm: pblk: I...
247
  void pblk_submit_read(struct pblk *pblk, struct bio *bio)
a4bd217b4   Javier González   lightnvm: physica...
248
249
  {
  	struct nvm_tgt_dev *dev = pblk->dev;
998ba6297   Javier González   lightnvm: pblk: a...
250
  	struct request_queue *q = dev->q;
84454e6de   Javier González   lightnvm: pblk: r...
251
  	sector_t blba = pblk_get_lba(bio);
5bf1e1ee6   Dan Carpenter   lightnvm: potenti...
252
  	unsigned int nr_secs = pblk_get_secs(bio);
a96de64a2   Igor Konopko   lightnvm: pblk: s...
253
  	bool from_cache;
a4809fee4   Javier González   lightnvm: pblk: c...
254
  	struct pblk_g_ctx *r_ctx;
a4bd217b4   Javier González   lightnvm: physica...
255
  	struct nvm_rq *rqd;
a96de64a2   Igor Konopko   lightnvm: pblk: s...
256
  	struct bio *int_bio, *split_bio;
a4bd217b4   Javier González   lightnvm: physica...
257

ddcf35d39   Michael Callahan   block: Add and us...
258
259
  	generic_start_io_acct(q, REQ_OP_READ, bio_sectors(bio),
  			      &pblk->disk->part0);
998ba6297   Javier González   lightnvm: pblk: a...
260

e2cddf208   Javier González   lightnvm: pblk: i...
261
  	rqd = pblk_alloc_rqd(pblk, PBLK_READ);
a4bd217b4   Javier González   lightnvm: physica...
262
263
  
  	rqd->opcode = NVM_OP_PREAD;
a4bd217b4   Javier González   lightnvm: physica...
264
265
266
  	rqd->nr_ppas = nr_secs;
  	rqd->private = pblk;
  	rqd->end_io = pblk_end_io_read;
a4809fee4   Javier González   lightnvm: pblk: c...
267
  	r_ctx = nvm_rq_to_pdu(rqd);
998ba6297   Javier González   lightnvm: pblk: a...
268
  	r_ctx->start_time = jiffies;
a4809fee4   Javier González   lightnvm: pblk: c...
269
  	r_ctx->lba = blba;
3e03f6322   Igor Konopko   lightnvm: pblk: I...
270
271
272
273
274
275
276
277
  	if (pblk_alloc_rqd_meta(pblk, rqd)) {
  		bio_io_error(bio);
  		pblk_free_rqd(pblk, rqd, PBLK_READ);
  		return;
  	}
  
  	/* Clone read bio to deal internally with:
  	 * -read errors when reading from drive
a96de64a2   Igor Konopko   lightnvm: pblk: s...
278
  	 * -bio_advance() calls during cache reads
3e03f6322   Igor Konopko   lightnvm: pblk: I...
279
280
  	 */
  	int_bio = bio_clone_fast(bio, GFP_KERNEL, &pblk_bio_set);
a4bd217b4   Javier González   lightnvm: physica...
281

45dcf29b9   Javier González   lightnvm: pblk: e...
282
  	if (nr_secs > 1)
a96de64a2   Igor Konopko   lightnvm: pblk: s...
283
284
  		nr_secs = pblk_read_ppalist_rq(pblk, rqd, int_bio, blba,
  						&from_cache);
45dcf29b9   Javier González   lightnvm: pblk: e...
285
  	else
a96de64a2   Igor Konopko   lightnvm: pblk: s...
286
  		pblk_read_rq(pblk, rqd, int_bio, blba, &from_cache);
a4bd217b4   Javier González   lightnvm: physica...
287

a96de64a2   Igor Konopko   lightnvm: pblk: s...
288
  split_retry:
3e03f6322   Igor Konopko   lightnvm: pblk: I...
289
290
  	r_ctx->private = bio; /* original bio */
  	rqd->bio = int_bio; /* internal bio */
a96de64a2   Igor Konopko   lightnvm: pblk: s...
291
292
  	if (from_cache && nr_secs == rqd->nr_ppas) {
  		/* All data was read from cache, we can complete the IO. */
3e03f6322   Igor Konopko   lightnvm: pblk: I...
293
  		pblk_end_user_read(bio, 0);
588726d3e   Javier González   lightnvm: pblk: f...
294
  		atomic_inc(&pblk->inflight_io);
7bd4d370d   Javier González   lightnvm: pblk: g...
295
  		__pblk_end_io_read(pblk, rqd, false);
a96de64a2   Igor Konopko   lightnvm: pblk: s...
296
  	} else if (nr_secs != rqd->nr_ppas) {
3e03f6322   Igor Konopko   lightnvm: pblk: I...
297
298
  		/* The read bio request could be partially filled by the write
  		 * buffer, but there are some holes that need to be read from
a96de64a2   Igor Konopko   lightnvm: pblk: s...
299
300
301
  		 * the drive. In order to handle this, we will use block layer
  		 * mechanism to split this request in to smaller ones and make
  		 * a chain of it.
3e03f6322   Igor Konopko   lightnvm: pblk: I...
302
  		 */
a96de64a2   Igor Konopko   lightnvm: pblk: s...
303
304
305
306
307
308
309
310
311
312
313
314
315
316
317
  		split_bio = bio_split(bio, nr_secs * NR_PHY_IN_LOG, GFP_KERNEL,
  					&pblk_bio_set);
  		bio_chain(split_bio, bio);
  		generic_make_request(bio);
  
  		/* New bio contains first N sectors of the previous one, so
  		 * we can continue to use existing rqd, but we need to shrink
  		 * the number of PPAs in it. New bio is also guaranteed that
  		 * it contains only either data from cache or from drive, newer
  		 * mix of them.
  		 */
  		bio = split_bio;
  		rqd->nr_ppas = nr_secs;
  		if (rqd->nr_ppas == 1)
  			rqd->ppa_addr = rqd->ppa_list[0];
a4bd217b4   Javier González   lightnvm: physica...
318

a96de64a2   Igor Konopko   lightnvm: pblk: s...
319
320
321
322
323
324
  		/* Recreate int_bio - existing might have some needed internal
  		 * fields modified already.
  		 */
  		bio_put(int_bio);
  		int_bio = bio_clone_fast(bio, GFP_KERNEL, &pblk_bio_set);
  		goto split_retry;
48e5da725   Hans Holmberg   lightnvm: move me...
325
  	} else if (pblk_submit_io(pblk, rqd, NULL)) {
a96de64a2   Igor Konopko   lightnvm: pblk: s...
326
327
328
  		/* Submitting IO to drive failed, let's report an error */
  		rqd->error = -ENODEV;
  		pblk_end_io_read(rqd);
3e03f6322   Igor Konopko   lightnvm: pblk: I...
329
  	}
a4bd217b4   Javier González   lightnvm: physica...
330
331
332
333
  }
  
  static int read_ppalist_rq_gc(struct pblk *pblk, struct nvm_rq *rqd,
  			      struct pblk_line *line, u64 *lba_list,
d340121eb   Javier González   lightnvm: pblk: s...
334
  			      u64 *paddr_list_gc, unsigned int nr_secs)
a4bd217b4   Javier González   lightnvm: physica...
335
  {
afdc23c91   Matias Bjørling   lightnvm: pblk: u...
336
  	struct ppa_addr ppa_list_l2p[NVM_MAX_VLBA];
d340121eb   Javier González   lightnvm: pblk: s...
337
  	struct ppa_addr ppa_gc;
a4bd217b4   Javier González   lightnvm: physica...
338
339
  	int valid_secs = 0;
  	int i;
d340121eb   Javier González   lightnvm: pblk: s...
340
  	pblk_lookup_l2p_rand(pblk, ppa_list_l2p, lba_list, nr_secs);
a4bd217b4   Javier González   lightnvm: physica...
341
342
  
  	for (i = 0; i < nr_secs; i++) {
d340121eb   Javier González   lightnvm: pblk: s...
343
344
345
346
347
348
  		if (lba_list[i] == ADDR_EMPTY)
  			continue;
  
  		ppa_gc = addr_to_gen_ppa(pblk, paddr_list_gc[i], line->id);
  		if (!pblk_ppa_comp(ppa_list_l2p[i], ppa_gc)) {
  			paddr_list_gc[i] = lba_list[i] = ADDR_EMPTY;
a4bd217b4   Javier González   lightnvm: physica...
349
350
  			continue;
  		}
d340121eb   Javier González   lightnvm: pblk: s...
351
  		rqd->ppa_list[valid_secs++] = ppa_list_l2p[i];
a4bd217b4   Javier González   lightnvm: physica...
352
  	}
880eda544   Matias Bjørling   lightnvm: move NV...
353
  #ifdef CONFIG_NVM_PBLK_DEBUG
a4bd217b4   Javier González   lightnvm: physica...
354
355
  	atomic_long_add(valid_secs, &pblk->inflight_reads);
  #endif
d340121eb   Javier González   lightnvm: pblk: s...
356

a4bd217b4   Javier González   lightnvm: physica...
357
358
359
360
  	return valid_secs;
  }
  
  static int read_rq_gc(struct pblk *pblk, struct nvm_rq *rqd,
d340121eb   Javier González   lightnvm: pblk: s...
361
362
  		      struct pblk_line *line, sector_t lba,
  		      u64 paddr_gc)
a4bd217b4   Javier González   lightnvm: physica...
363
  {
d340121eb   Javier González   lightnvm: pblk: s...
364
  	struct ppa_addr ppa_l2p, ppa_gc;
a4bd217b4   Javier González   lightnvm: physica...
365
  	int valid_secs = 0;
659226eb6   Dan Carpenter   lightnvm: don't p...
366
367
  	if (lba == ADDR_EMPTY)
  		goto out;
a4bd217b4   Javier González   lightnvm: physica...
368
  	/* logic error: lba out-of-bounds */
847a3a278   Igor Konopko   lightnvm: pblk: r...
369
  	if (lba >= pblk->capacity) {
2a79efd83   Dan Carpenter   lightnvm: fix som...
370
371
  		WARN(1, "pblk: read lba out of bounds
  ");
a4bd217b4   Javier González   lightnvm: physica...
372
373
  		goto out;
  	}
a4bd217b4   Javier González   lightnvm: physica...
374
  	spin_lock(&pblk->trans_lock);
d340121eb   Javier González   lightnvm: pblk: s...
375
  	ppa_l2p = pblk_trans_map_get(pblk, lba);
a4bd217b4   Javier González   lightnvm: physica...
376
  	spin_unlock(&pblk->trans_lock);
d340121eb   Javier González   lightnvm: pblk: s...
377
378
  	ppa_gc = addr_to_gen_ppa(pblk, paddr_gc, line->id);
  	if (!pblk_ppa_comp(ppa_l2p, ppa_gc))
a4bd217b4   Javier González   lightnvm: physica...
379
  		goto out;
d340121eb   Javier González   lightnvm: pblk: s...
380
  	rqd->ppa_addr = ppa_l2p;
a4bd217b4   Javier González   lightnvm: physica...
381
  	valid_secs = 1;
880eda544   Matias Bjørling   lightnvm: move NV...
382
  #ifdef CONFIG_NVM_PBLK_DEBUG
a4bd217b4   Javier González   lightnvm: physica...
383
384
385
386
387
388
  	atomic_long_inc(&pblk->inflight_reads);
  #endif
  
  out:
  	return valid_secs;
  }
d340121eb   Javier González   lightnvm: pblk: s...
389
  int pblk_submit_read_gc(struct pblk *pblk, struct pblk_gc_rq *gc_rq)
a4bd217b4   Javier González   lightnvm: physica...
390
  {
a4bd217b4   Javier González   lightnvm: physica...
391
  	struct nvm_rq rqd;
d340121eb   Javier González   lightnvm: pblk: s...
392
  	int ret = NVM_IO_OK;
a4bd217b4   Javier González   lightnvm: physica...
393
394
  
  	memset(&rqd, 0, sizeof(struct nvm_rq));
45dcf29b9   Javier González   lightnvm: pblk: e...
395
396
397
  	ret = pblk_alloc_rqd_meta(pblk, &rqd);
  	if (ret)
  		return ret;
63e3809cf   Javier González   lightnvm: pblk: s...
398

d340121eb   Javier González   lightnvm: pblk: s...
399
  	if (gc_rq->nr_secs > 1) {
d340121eb   Javier González   lightnvm: pblk: s...
400
401
402
403
404
  		gc_rq->secs_to_gc = read_ppalist_rq_gc(pblk, &rqd, gc_rq->line,
  							gc_rq->lba_list,
  							gc_rq->paddr_list,
  							gc_rq->nr_secs);
  		if (gc_rq->secs_to_gc == 1)
63e3809cf   Javier González   lightnvm: pblk: s...
405
  			rqd.ppa_addr = rqd.ppa_list[0];
a4bd217b4   Javier González   lightnvm: physica...
406
  	} else {
d340121eb   Javier González   lightnvm: pblk: s...
407
408
409
  		gc_rq->secs_to_gc = read_rq_gc(pblk, &rqd, gc_rq->line,
  							gc_rq->lba_list[0],
  							gc_rq->paddr_list[0]);
a4bd217b4   Javier González   lightnvm: physica...
410
  	}
d340121eb   Javier González   lightnvm: pblk: s...
411
  	if (!(gc_rq->secs_to_gc))
a4bd217b4   Javier González   lightnvm: physica...
412
  		goto out;
a4bd217b4   Javier González   lightnvm: physica...
413
  	rqd.opcode = NVM_OP_PREAD;
d340121eb   Javier González   lightnvm: pblk: s...
414
  	rqd.nr_ppas = gc_rq->secs_to_gc;
a4bd217b4   Javier González   lightnvm: physica...
415

48e5da725   Hans Holmberg   lightnvm: move me...
416
  	if (pblk_submit_io_sync(pblk, &rqd, gc_rq->data)) {
d340121eb   Javier González   lightnvm: pblk: s...
417
  		ret = -EIO;
48e5da725   Hans Holmberg   lightnvm: move me...
418
  		goto err_free_dma;
a4bd217b4   Javier González   lightnvm: physica...
419
  	}
03a34b2d2   Javier González   lightnvm: pblk: i...
420
  	pblk_read_check_rand(pblk, &rqd, gc_rq->lba_list, gc_rq->nr_secs);
310df5822   Javier González   lightnvm: pblk: c...
421

588726d3e   Javier González   lightnvm: pblk: f...
422
  	atomic_dec(&pblk->inflight_io);
a4bd217b4   Javier González   lightnvm: physica...
423
424
425
  
  	if (rqd.error) {
  		atomic_long_inc(&pblk->read_failed_gc);
880eda544   Matias Bjørling   lightnvm: move NV...
426
  #ifdef CONFIG_NVM_PBLK_DEBUG
a4bd217b4   Javier González   lightnvm: physica...
427
428
429
  		pblk_print_failed_rqd(pblk, &rqd, rqd.error);
  #endif
  	}
880eda544   Matias Bjørling   lightnvm: move NV...
430
  #ifdef CONFIG_NVM_PBLK_DEBUG
d340121eb   Javier González   lightnvm: pblk: s...
431
432
433
  	atomic_long_add(gc_rq->secs_to_gc, &pblk->sync_reads);
  	atomic_long_add(gc_rq->secs_to_gc, &pblk->recov_gc_reads);
  	atomic_long_sub(gc_rq->secs_to_gc, &pblk->inflight_reads);
a4bd217b4   Javier González   lightnvm: physica...
434
435
436
  #endif
  
  out:
45dcf29b9   Javier González   lightnvm: pblk: e...
437
  	pblk_free_rqd_meta(pblk, &rqd);
d340121eb   Javier González   lightnvm: pblk: s...
438
  	return ret;
a4bd217b4   Javier González   lightnvm: physica...
439
440
  
  err_free_dma:
45dcf29b9   Javier González   lightnvm: pblk: e...
441
  	pblk_free_rqd_meta(pblk, &rqd);
d340121eb   Javier González   lightnvm: pblk: s...
442
  	return ret;
a4bd217b4   Javier González   lightnvm: physica...
443
  }