diff options
author | Maxim Levitsky <maximlevitsky@gmail.com> | 2012-09-28 04:38:15 -0400 |
---|---|---|
committer | Jens Axboe <axboe@kernel.dk> | 2012-09-28 04:38:15 -0400 |
commit | 232f1b51062553b7cf49f99719fbd1b8a8d80f29 (patch) | |
tree | bf0ad31232df77562ad42a48b1bf21a0e2948c3f /lib/scatterlist.c | |
parent | 2e484610296b25f0a04b516bc144a00731d1d845 (diff) |
scatterlist: refactor the sg_nents
Replace 'while' with 'for' as suggested by Tejun Heo
Signed-off-by: Maxim Levitsky <maximlevitsky@gmail.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
Diffstat (limited to 'lib/scatterlist.c')
-rw-r--r-- | lib/scatterlist.c | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/lib/scatterlist.c b/lib/scatterlist.c index 1bf60efb5e02..8cd2ced68f36 100644 --- a/lib/scatterlist.c +++ b/lib/scatterlist.c | |||
@@ -49,12 +49,9 @@ EXPORT_SYMBOL(sg_next); | |||
49 | **/ | 49 | **/ |
50 | int sg_nents(struct scatterlist *sg) | 50 | int sg_nents(struct scatterlist *sg) |
51 | { | 51 | { |
52 | int nents = 0; | 52 | int nents; |
53 | while (sg) { | 53 | for (nents = 0; sg; sg = sg_next(sg)) |
54 | nents++; | 54 | nents++; |
55 | sg = sg_next(sg); | ||
56 | } | ||
57 | |||
58 | return nents; | 55 | return nents; |
59 | } | 56 | } |
60 | EXPORT_SYMBOL(sg_nents); | 57 | EXPORT_SYMBOL(sg_nents); |