diff options
| author | Chuck Lever <chuck.lever@oracle.com> | 2007-10-26 19:29:47 +0200 |
|---|---|---|
| committer | Jens Axboe <jens.axboe@oracle.com> | 2007-10-29 09:18:04 +0100 |
| commit | 74eb94f7b84f4e631a0e020991fb16f17ce85ab7 (patch) | |
| tree | 9d08217290654cefa5505d085a032ca60d021e5f /include/linux | |
| parent | 30fa0d0f0c0ab2aa0d4c2f88eda49eaa19ea6f8d (diff) | |
sg_last() should use unsigned loop index variable
Clean up: fix a mixed sign comparison in sg_last() accidentally
introduced by commit 70eb8040. The sign of the loop index variable
should match the sign of the "nents" argument.
Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
Signed-off-by: Jens Axboe <axboe@carl.home.kernel.dk>
Diffstat (limited to 'include/linux')
| -rw-r--r-- | include/linux/scatterlist.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/linux/scatterlist.h b/include/linux/scatterlist.h index b2ec8421b89f..19b751aabd16 100644 --- a/include/linux/scatterlist.h +++ b/include/linux/scatterlist.h @@ -150,7 +150,7 @@ static inline struct scatterlist *sg_last(struct scatterlist *sgl, struct scatterlist *ret = &sgl[nents - 1]; #else struct scatterlist *sg, *ret = NULL; - int i; + unsigned int i; for_each_sg(sgl, sg, nents, i) ret = sg; |