[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <226E1C65E4F6164E8EA5FD3CC913AE8C0229390C@G3W0639.americas.hpqcorp.net>
Date: Tue, 16 Oct 2007 14:55:11 -0000
From: "Miller, Mike (OS Dev)" <Mike.Miller@...com>
To: "Jeff Garzik" <jeff@...zik.org>,
"Andrew Morton" <akpm@...ux-foundation.org>,
"Linus Torvalds" <torvalds@...ux-foundation.org>
Cc: "LKML" <linux-kernel@...r.kernel.org>, <jens.axboe@...cle.com>,
"ISS StorageDev" <iss_storagedev@...com>
Subject: RE: [PATCH] drivers/block/cpqarray,cciss: kill unused var
> -----Original Message-----
> From: Jeff Garzik [mailto:jeff@...zik.org]
> Sent: Saturday, October 13, 2007 2:04 PM
> To: Andrew Morton; Linus Torvalds
> Cc: LKML; jens.axboe@...cle.com; ISS StorageDev
> Subject: [PATCH] drivers/block/cpqarray,cciss: kill unused var
>
> The recent bio work and subsequent fixups created unused variables.
>
> Signed-off-by: Jeff Garzik <jgarzik@...hat.com>
Acked-by: Mike Miller <mike.miller@...com>
> ---
> drivers/block/cciss.c | 1 -
> drivers/block/cpqarray.c | 3 +--
> 2 files changed, 1 insertion(+), 3 deletions(-)
>
> diff --git a/drivers/block/cciss.c b/drivers/block/cciss.c
> index 28d1457..27401d6 100644
> --- a/drivers/block/cciss.c
> +++ b/drivers/block/cciss.c
> @@ -1191,7 +1191,6 @@ static inline void
> complete_buffers(struct bio *bio, int status) {
> while (bio) {
> struct bio *xbh = bio->bi_next;
> - int nr_sectors = bio_sectors(bio);
>
> bio->bi_next = NULL;
> bio_endio(bio, status ? 0 : -EIO);
> diff --git a/drivers/block/cpqarray.c
> b/drivers/block/cpqarray.c index 3853c9a..568603d 100644
> --- a/drivers/block/cpqarray.c
> +++ b/drivers/block/cpqarray.c
> @@ -981,9 +981,8 @@ static void start_io(ctlr_info_t *h)
> static inline void complete_buffers(struct bio *bio, int ok) {
> struct bio *xbh;
> - while(bio) {
> - int nr_sectors = bio_sectors(bio);
>
> + while (bio) {
> xbh = bio->bi_next;
> bio->bi_next = NULL;
>
>
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists