Skip to content

Commit

Permalink
dm: merge __flush_deferred_io into caller
Browse files Browse the repository at this point in the history
Merge __flush_deferred_io() into the only caller, dm_wq_work().

There's no need to have a function that has only one caller.

Signed-off-by: Mikulas Patocka <mpatocka@redhat.com>
Signed-off-by: Alasdair G Kergon <agk@redhat.com>
  • Loading branch information
Mikulas Patocka authored and Alasdair G Kergon committed Apr 2, 2009
1 parent f0b9a45 commit ef20858
Showing 1 changed file with 7 additions and 11 deletions.
18 changes: 7 additions & 11 deletions drivers/md/dm.c
Original file line number Diff line number Diff line change
Expand Up @@ -1401,14 +1401,20 @@ static int dm_wait_for_completion(struct mapped_device *md)
/*
* Process the deferred bios
*/
static void __flush_deferred_io(struct mapped_device *md)
static void dm_wq_work(struct work_struct *work)
{
struct mapped_device *md = container_of(work, struct mapped_device,
work);
struct bio *c;

down_write(&md->io_lock);

while ((c = bio_list_pop(&md->deferred)))
__split_and_process_bio(md, c);

clear_bit(DMF_BLOCK_IO, &md->flags);

up_write(&md->io_lock);
}

static void __merge_pushback_list(struct mapped_device *md)
Expand All @@ -1422,16 +1428,6 @@ static void __merge_pushback_list(struct mapped_device *md)
spin_unlock_irqrestore(&md->pushback_lock, flags);
}

static void dm_wq_work(struct work_struct *work)
{
struct mapped_device *md = container_of(work, struct mapped_device,
work);

down_write(&md->io_lock);
__flush_deferred_io(md);
up_write(&md->io_lock);
}

static void dm_queue_flush(struct mapped_device *md)
{
queue_work(md->wq, &md->work);
Expand Down

0 comments on commit ef20858

Please sign in to comment.