diff options
Diffstat (limited to 'block')
-rw-r--r-- | block/blk-core.c | 24 | ||||
-rw-r--r-- | block/blk-merge.c | 2 | ||||
-rw-r--r-- | block/blk.h | 4 | ||||
-rw-r--r-- | block/genhd.c | 14 |
4 files changed, 31 insertions, 13 deletions
diff --git a/block/blk-core.c b/block/blk-core.c index 797d5095eb83..ddc68332d655 100644 --- a/block/blk-core.c +++ b/block/blk-core.c | |||
@@ -64,13 +64,15 @@ static void drive_stat_acct(struct request *rq, int new_io) | |||
64 | return; | 64 | return; |
65 | 65 | ||
66 | cpu = part_stat_lock(); | 66 | cpu = part_stat_lock(); |
67 | part = disk_map_sector_rcu(rq->rq_disk, blk_rq_pos(rq)); | ||
68 | 67 | ||
69 | if (!new_io) | 68 | if (!new_io) { |
69 | part = rq->part; | ||
70 | part_stat_inc(cpu, part, merges[rw]); | 70 | part_stat_inc(cpu, part, merges[rw]); |
71 | else { | 71 | } else { |
72 | part = disk_map_sector_rcu(rq->rq_disk, blk_rq_pos(rq)); | ||
72 | part_round_stats(cpu, part); | 73 | part_round_stats(cpu, part); |
73 | part_inc_in_flight(part, rw); | 74 | part_inc_in_flight(part, rw); |
75 | rq->part = part; | ||
74 | } | 76 | } |
75 | 77 | ||
76 | part_stat_unlock(); | 78 | part_stat_unlock(); |
@@ -128,6 +130,7 @@ void blk_rq_init(struct request_queue *q, struct request *rq) | |||
128 | rq->ref_count = 1; | 130 | rq->ref_count = 1; |
129 | rq->start_time = jiffies; | 131 | rq->start_time = jiffies; |
130 | set_start_time_ns(rq); | 132 | set_start_time_ns(rq); |
133 | rq->part = NULL; | ||
131 | } | 134 | } |
132 | EXPORT_SYMBOL(blk_rq_init); | 135 | EXPORT_SYMBOL(blk_rq_init); |
133 | 136 | ||
@@ -804,11 +807,16 @@ static struct request *get_request(struct request_queue *q, int rw_flags, | |||
804 | rl->starved[is_sync] = 0; | 807 | rl->starved[is_sync] = 0; |
805 | 808 | ||
806 | priv = !test_bit(QUEUE_FLAG_ELVSWITCH, &q->queue_flags); | 809 | priv = !test_bit(QUEUE_FLAG_ELVSWITCH, &q->queue_flags); |
807 | if (priv) | 810 | if (priv) { |
808 | rl->elvpriv++; | 811 | rl->elvpriv++; |
809 | 812 | ||
810 | if (blk_queue_io_stat(q)) | 813 | /* |
811 | rw_flags |= REQ_IO_STAT; | 814 | * Don't do stats for non-priv requests |
815 | */ | ||
816 | if (blk_queue_io_stat(q)) | ||
817 | rw_flags |= REQ_IO_STAT; | ||
818 | } | ||
819 | |||
812 | spin_unlock_irq(q->queue_lock); | 820 | spin_unlock_irq(q->queue_lock); |
813 | 821 | ||
814 | rq = blk_alloc_request(q, rw_flags, priv, gfp_mask); | 822 | rq = blk_alloc_request(q, rw_flags, priv, gfp_mask); |
@@ -1777,7 +1785,7 @@ static void blk_account_io_completion(struct request *req, unsigned int bytes) | |||
1777 | int cpu; | 1785 | int cpu; |
1778 | 1786 | ||
1779 | cpu = part_stat_lock(); | 1787 | cpu = part_stat_lock(); |
1780 | part = disk_map_sector_rcu(req->rq_disk, blk_rq_pos(req)); | 1788 | part = req->part; |
1781 | part_stat_add(cpu, part, sectors[rw], bytes >> 9); | 1789 | part_stat_add(cpu, part, sectors[rw], bytes >> 9); |
1782 | part_stat_unlock(); | 1790 | part_stat_unlock(); |
1783 | } | 1791 | } |
@@ -1797,7 +1805,7 @@ static void blk_account_io_done(struct request *req) | |||
1797 | int cpu; | 1805 | int cpu; |
1798 | 1806 | ||
1799 | cpu = part_stat_lock(); | 1807 | cpu = part_stat_lock(); |
1800 | part = disk_map_sector_rcu(req->rq_disk, blk_rq_pos(req)); | 1808 | part = req->part; |
1801 | 1809 | ||
1802 | part_stat_inc(cpu, part, ios[rw]); | 1810 | part_stat_inc(cpu, part, ios[rw]); |
1803 | part_stat_add(cpu, part, ticks[rw], duration); | 1811 | part_stat_add(cpu, part, ticks[rw], duration); |
diff --git a/block/blk-merge.c b/block/blk-merge.c index 6a725461654d..38ff234012a4 100644 --- a/block/blk-merge.c +++ b/block/blk-merge.c | |||
@@ -351,7 +351,7 @@ static void blk_account_io_merge(struct request *req) | |||
351 | int cpu; | 351 | int cpu; |
352 | 352 | ||
353 | cpu = part_stat_lock(); | 353 | cpu = part_stat_lock(); |
354 | part = disk_map_sector_rcu(req->rq_disk, blk_rq_pos(req)); | 354 | part = req->part; |
355 | 355 | ||
356 | part_round_stats(cpu, part); | 356 | part_round_stats(cpu, part); |
357 | part_dec_in_flight(part, rq_data_dir(req)); | 357 | part_dec_in_flight(part, rq_data_dir(req)); |
diff --git a/block/blk.h b/block/blk.h index 6738831ba447..1340cce5721a 100644 --- a/block/blk.h +++ b/block/blk.h | |||
@@ -110,10 +110,6 @@ void blk_queue_congestion_threshold(struct request_queue *q); | |||
110 | 110 | ||
111 | int blk_dev_init(void); | 111 | int blk_dev_init(void); |
112 | 112 | ||
113 | void elv_quiesce_start(struct request_queue *q); | ||
114 | void elv_quiesce_end(struct request_queue *q); | ||
115 | |||
116 | |||
117 | /* | 113 | /* |
118 | * Return the threshold (number of used requests) at which the queue is | 114 | * Return the threshold (number of used requests) at which the queue is |
119 | * considered to be congested. It include a little hysteresis to keep the | 115 | * considered to be congested. It include a little hysteresis to keep the |
diff --git a/block/genhd.c b/block/genhd.c index 7923e720ddf5..8313834596db 100644 --- a/block/genhd.c +++ b/block/genhd.c | |||
@@ -932,8 +932,15 @@ static void disk_free_ptbl_rcu_cb(struct rcu_head *head) | |||
932 | { | 932 | { |
933 | struct disk_part_tbl *ptbl = | 933 | struct disk_part_tbl *ptbl = |
934 | container_of(head, struct disk_part_tbl, rcu_head); | 934 | container_of(head, struct disk_part_tbl, rcu_head); |
935 | struct gendisk *disk = ptbl->disk; | ||
936 | struct request_queue *q = disk->queue; | ||
937 | unsigned long flags; | ||
935 | 938 | ||
936 | kfree(ptbl); | 939 | kfree(ptbl); |
940 | |||
941 | spin_lock_irqsave(q->queue_lock, flags); | ||
942 | elv_quiesce_end(q); | ||
943 | spin_unlock_irqrestore(q->queue_lock, flags); | ||
937 | } | 944 | } |
938 | 945 | ||
939 | /** | 946 | /** |
@@ -951,11 +958,17 @@ static void disk_replace_part_tbl(struct gendisk *disk, | |||
951 | struct disk_part_tbl *new_ptbl) | 958 | struct disk_part_tbl *new_ptbl) |
952 | { | 959 | { |
953 | struct disk_part_tbl *old_ptbl = disk->part_tbl; | 960 | struct disk_part_tbl *old_ptbl = disk->part_tbl; |
961 | struct request_queue *q = disk->queue; | ||
954 | 962 | ||
955 | rcu_assign_pointer(disk->part_tbl, new_ptbl); | 963 | rcu_assign_pointer(disk->part_tbl, new_ptbl); |
956 | 964 | ||
957 | if (old_ptbl) { | 965 | if (old_ptbl) { |
958 | rcu_assign_pointer(old_ptbl->last_lookup, NULL); | 966 | rcu_assign_pointer(old_ptbl->last_lookup, NULL); |
967 | |||
968 | spin_lock_irq(q->queue_lock); | ||
969 | elv_quiesce_start(q); | ||
970 | spin_unlock_irq(q->queue_lock); | ||
971 | |||
959 | call_rcu(&old_ptbl->rcu_head, disk_free_ptbl_rcu_cb); | 972 | call_rcu(&old_ptbl->rcu_head, disk_free_ptbl_rcu_cb); |
960 | } | 973 | } |
961 | } | 974 | } |
@@ -996,6 +1009,7 @@ int disk_expand_part_tbl(struct gendisk *disk, int partno) | |||
996 | return -ENOMEM; | 1009 | return -ENOMEM; |
997 | 1010 | ||
998 | new_ptbl->len = target; | 1011 | new_ptbl->len = target; |
1012 | new_ptbl->disk = disk; | ||
999 | 1013 | ||
1000 | for (i = 0; i < len; i++) | 1014 | for (i = 0; i < len; i++) |
1001 | rcu_assign_pointer(new_ptbl->part[i], old_ptbl->part[i]); | 1015 | rcu_assign_pointer(new_ptbl->part[i], old_ptbl->part[i]); |