Commit f6344757 authored by NeilBrown's avatar NeilBrown Committed by Linus Torvalds

[PATCH] md: Remove bi_end_io call out from under a spinlock

raid5 overloads bi_phys_segments to count the number of blocks that the
request was broken in to so that it knows when the bio is completely handled.

Accessing this must always be done under a spinlock.  In one case we also call
bi_end_io under that spinlock, which probably isn't ideal as bi_end_io could
be expensive (even though it isn't allowed to sleep).

So we reducde the range of the spinlock to just accessing bi_phys_segments.
Signed-off-by: default avatarNeil Brown <neilb@suse.de>
Signed-off-by: default avatarAndrew Morton <akpm@osdl.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@osdl.org>
parent b3b46be3
...@@ -1743,6 +1743,7 @@ static int make_request(request_queue_t *q, struct bio * bi) ...@@ -1743,6 +1743,7 @@ static int make_request(request_queue_t *q, struct bio * bi)
sector_t logical_sector, last_sector; sector_t logical_sector, last_sector;
struct stripe_head *sh; struct stripe_head *sh;
const int rw = bio_data_dir(bi); const int rw = bio_data_dir(bi);
int remaining;
if (unlikely(bio_barrier(bi))) { if (unlikely(bio_barrier(bi))) {
bio_endio(bi, bi->bi_size, -EOPNOTSUPP); bio_endio(bi, bi->bi_size, -EOPNOTSUPP);
...@@ -1852,7 +1853,9 @@ static int make_request(request_queue_t *q, struct bio * bi) ...@@ -1852,7 +1853,9 @@ static int make_request(request_queue_t *q, struct bio * bi)
} }
spin_lock_irq(&conf->device_lock); spin_lock_irq(&conf->device_lock);
if (--bi->bi_phys_segments == 0) { remaining = --bi->bi_phys_segments;
spin_unlock_irq(&conf->device_lock);
if (remaining == 0) {
int bytes = bi->bi_size; int bytes = bi->bi_size;
if ( bio_data_dir(bi) == WRITE ) if ( bio_data_dir(bi) == WRITE )
...@@ -1860,7 +1863,6 @@ static int make_request(request_queue_t *q, struct bio * bi) ...@@ -1860,7 +1863,6 @@ static int make_request(request_queue_t *q, struct bio * bi)
bi->bi_size = 0; bi->bi_size = 0;
bi->bi_end_io(bi, bytes, 0); bi->bi_end_io(bi, bytes, 0);
} }
spin_unlock_irq(&conf->device_lock);
return 0; return 0;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment