From: Chuck Lever <chuck.lever@oracle.com>
Date: Fri, 26 Oct 2007 17:29:47 +0000 (+0200)
Subject: sg_last() should use unsigned loop index variable
X-Git-Tag: v2.6.24-rc2~104^2~2
X-Git-Url: https://err.no/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=74eb94f7b84f4e631a0e020991fb16f17ce85ab7;p=linux-2.6

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>
---

diff --git a/include/linux/scatterlist.h b/include/linux/scatterlist.h
index b2ec8421b8..19b751aabd 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;