2011-05-25 20:45:16

by Luca Tettamanti

[permalink] [raw]
Subject: [PATCH] bio_attempt_front_merge: remove unused variable.

sector is never read inside the function.

Signed-off-by: Luca Tettamanti <[email protected]>
---
block/blk-core.c | 3 ---
1 files changed, 0 insertions(+), 3 deletions(-)

diff --git a/block/blk-core.c b/block/blk-core.c
index 3fe00a1..1439bac 100644
--- a/block/blk-core.c
+++ b/block/blk-core.c
@@ -1139,7 +1139,6 @@ static bool bio_attempt_front_merge(struct request_queue *q,
struct request *req, struct bio *bio)
{
const int ff = bio->bi_rw & REQ_FAILFAST_MASK;
- sector_t sector;

/*
* Debug stuff, kill later
@@ -1157,8 +1156,6 @@ static bool bio_attempt_front_merge(struct request_queue *q,
if ((req->cmd_flags & REQ_FAILFAST_MASK) != ff)
blk_rq_set_mixed_merge(req);

- sector = bio->bi_sector;
-
bio->bi_next = req->bio;
req->bio = bio;

--
1.7.5.1


2011-05-26 19:08:24

by Jens Axboe

[permalink] [raw]
Subject: Re: [PATCH] bio_attempt_front_merge: remove unused variable.

On 2011-05-25 22:45, Luca Tettamanti wrote:
> sector is never read inside the function.

Thanks, applied.

--
Jens Axboe