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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

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/