sg_init_table() should use unsigned loop index variable
authorChuck Lever <[email protected]>
Fri, 26 Oct 2007 17:29:48 +0000 (19:29 +0200)
committerJens Axboe <[email protected]>
Mon, 29 Oct 2007 08:18:04 +0000 (09:18 +0100)
Clean up: fix a mixed sign comparison in sg_init_table() accidentally
introduced by commit d6ec0842.  The sign of the loop index variable
should match the sign of the "nents" argument.

Signed-off-by: Chuck Lever <[email protected]>
Cc: Jens Axboe <[email protected]>
Signed-off-by: Jens Axboe <[email protected]>
include/linux/scatterlist.h

index 19b751aabd1684b9250a9cb2e4a5e489ac2f8ca4..32326c293d7bfbd0c9fe318ce4f5143258bed65a 100644 (file)
@@ -243,7 +243,7 @@ static inline void sg_init_table(struct scatterlist *sgl, unsigned int nents)
        sg_mark_end(sgl, nents);
 #ifdef CONFIG_DEBUG_SG
        {
-               int i;
+               unsigned int i;
                for (i = 0; i < nents; i++)
                        sgl[i].sg_magic = SG_MAGIC;
        }