RE: [PATCH] drivers/block/cpqarray,cciss: kill unused var

From: Miller, Mike (OS Dev)
Date: Tue Oct 16 2007 - 10:55:58 EST




> -----Original Message-----
> From: Jeff Garzik [mailto:jeff@xxxxxxxxxx]
> Sent: Saturday, October 13, 2007 2:04 PM
> To: Andrew Morton; Linus Torvalds
> Cc: LKML; jens.axboe@xxxxxxxxxx; 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@xxxxxxxxxx>

Acked-by: Mike Miller <mike.miller@xxxxxx>

> ---
> 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@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/