Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755762AbZCLB7u (ORCPT ); Wed, 11 Mar 2009 21:59:50 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1755134AbZCLB67 (ORCPT ); Wed, 11 Mar 2009 21:58:59 -0400 Received: from mx2.redhat.com ([66.187.237.31]:49316 "EHLO mx2.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754501AbZCLB65 (ORCPT ); Wed, 11 Mar 2009 21:58:57 -0400 From: Vivek Goyal To: nauman@google.com, dpshah@google.com, lizf@cn.fujitsu.com, mikew@google.com, fchecconi@gmail.com, paolo.valente@unimore.it, jens.axboe@oracle.com, ryov@valinux.co.jp, fernando@intellilink.co.jp, s-uchida@ap.jp.nec.com, taka@valinux.co.jp, guijianfeng@cn.fujitsu.com, arozansk@redhat.com, jmoyer@redhat.com, oz-kernel@redhat.com, dhaval@linux.vnet.ibm.com, balbir@linux.vnet.ibm.com, linux-kernel@vger.kernel.org, containers@lists.linux-foundation.org Cc: vgoyal@redhat.com, akpm@linux-foundation.org, menage@google.com, peterz@infradead.org Subject: [PATCH 07/10] Prepare elevator layer for single queue schedulers Date: Wed, 11 Mar 2009 21:56:52 -0400 Message-Id: <1236823015-4183-8-git-send-email-vgoyal@redhat.com> In-Reply-To: <1236823015-4183-1-git-send-email-vgoyal@redhat.com> References: <1236823015-4183-1-git-send-email-vgoyal@redhat.com> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 12036 Lines: 405 Elevator layer now has support for hierarchical fair queuing. cfq has been migrated to make use of it and now it is time to do groundwork for noop, deadline and AS. noop deadline and AS don't maintain separate queues for different processes. There is only one single queue. Effectively one can think that in hierarchical setup, there will be one queue per cgroup where requests from all the processes in the cgroup will be queued. Generally io scheduler takes care of creating queues. Because there is only one queue here, we have modified common layer to take care of queue creation and some other functionality. This special casing helps in keeping the changes to noop, deadline and AS to the minimum. Signed-off-by: Nauman Rafique Signed-off-by: Vivek Goyal --- block/elevator-fq.c | 153 ++++++++++++++++++++++++++++++++++++++++++++++ block/elevator-fq.h | 67 ++++++++++++++++++++ block/elevator.c | 35 ++++++++++- include/linux/elevator.h | 14 ++++ 4 files changed, 268 insertions(+), 1 deletions(-) diff --git a/block/elevator-fq.c b/block/elevator-fq.c index 389f68e..172f9e3 100644 --- a/block/elevator-fq.c +++ b/block/elevator-fq.c @@ -857,6 +857,12 @@ void io_put_io_group_queues(struct elevator_queue *e, struct io_group *iog) /* Free up async idle queue */ elv_release_ioq(e, &iog->async_idle_queue); + +#ifdef CONFIG_GROUP_IOSCHED + /* Optimization for io schedulers having single ioq */ + if (elv_iosched_single_ioq(e)) + elv_release_ioq(e, &iog->ioq); +#endif } @@ -1538,6 +1544,153 @@ void elv_fq_set_request_io_group(struct request_queue *q, rq->iog = iog; } +/* + * Find/Create the io queue the rq should go in. This is an optimization + * for the io schedulers (noop, deadline and AS) which maintain only single + * io queue per cgroup. In this case common layer can just maintain a + * pointer in group data structure and keeps track of it. + * + * For the io schdulers like cfq, which maintain multiple io queues per + * cgroup, and decide the io queue of request based on process, this + * function is not invoked. + */ +int elv_fq_set_request_ioq(struct request_queue *q, struct request *rq, + gfp_t gfp_mask) +{ + struct elevator_queue *e = q->elevator; + unsigned long flags; + struct io_queue *ioq = NULL, *new_ioq = NULL; + struct io_group *iog; + void *sched_q = NULL, *new_sched_q = NULL; + + if (!elv_iosched_fair_queuing_enabled(e)) + return 0; + + might_sleep_if(gfp_mask & __GFP_WAIT); + spin_lock_irqsave(q->queue_lock, flags); + + /* Determine the io group request belongs to */ + iog = rq->iog; + BUG_ON(!iog); + +retry: + /* Get the iosched queue */ + ioq = io_group_ioq(iog); + if (!ioq) { + /* io queue and sched_queue needs to be allocated */ + BUG_ON(!e->ops->elevator_alloc_sched_queue_fn); + + if (new_sched_q) { + goto alloc_ioq; + } else if (gfp_mask & __GFP_WAIT) { + /* + * Inform the allocator of the fact that we will + * just repeat this allocation if it fails, to allow + * the allocator to do whatever it needs to attempt to + * free memory. + */ + spin_unlock_irq(q->queue_lock); + /* Call io scheduer to create scheduler queue */ + new_sched_q = e->ops->elevator_alloc_sched_queue_fn(q, + e, gfp_mask | __GFP_NOFAIL + | __GFP_ZERO); + spin_lock_irq(q->queue_lock); + goto retry; + } else { + sched_q = e->ops->elevator_alloc_sched_queue_fn(q, e, + gfp_mask | __GFP_ZERO); + if (!sched_q) + goto queue_fail; + } + +alloc_ioq: + if (new_ioq) { + ioq = new_ioq; + new_ioq = NULL; + sched_q = new_sched_q; + new_sched_q = NULL; + } else if (gfp_mask & __GFP_WAIT) { + /* + * Inform the allocator of the fact that we will + * just repeat this allocation if it fails, to allow + * the allocator to do whatever it needs to attempt to + * free memory. + */ + spin_unlock_irq(q->queue_lock); + new_ioq = elv_alloc_ioq(q, gfp_mask | __GFP_NOFAIL + | __GFP_ZERO); + spin_lock_irq(q->queue_lock); + goto retry; + } else { + ioq = elv_alloc_ioq(q, gfp_mask | __GFP_ZERO); + if (!ioq) { + e->ops->elevator_free_sched_queue_fn(e, + sched_q); + sched_q = NULL; + goto queue_fail; + } + } + + elv_init_ioq(e, ioq, sched_q, IOPRIO_CLASS_BE, 4, 1); + io_group_set_ioq(iog, ioq); + elv_mark_ioq_sync(ioq); + } + + if (new_sched_q) + e->ops->elevator_free_sched_queue_fn(q->elevator, sched_q); + + if (new_ioq) + elv_free_ioq(new_ioq); + + /* Request reference */ + elv_get_ioq(ioq); + rq->ioq = ioq; + spin_unlock_irqrestore(q->queue_lock, flags); + return 0; + +queue_fail: + WARN_ON((gfp_mask & __GFP_WAIT) && !ioq); + elv_schedule_dispatch(q); + spin_unlock_irqrestore(q->queue_lock, flags); + return 1; +} + +/* + * Find out the io queue of current task. Optimization for single ioq + * per io group io schedulers. + */ +struct io_queue *elv_lookup_ioq_current(struct request_queue *q) +{ + struct io_group *iog; + + /* Determine the io group and io queue of the bio submitting task */ + iog = io_lookup_io_group_current(q); + if (!iog) { + /* May be task belongs to a cgroup for which io group has + * not been setup yet. */ + return NULL; + } + return io_group_ioq(iog); +} + +/* + * This request has been serviced. Clean up ioq info and drop the reference. + * Again this is called only for single queue per cgroup schedulers (noop, + * deadline, AS). + */ +void elv_fq_unset_request_ioq(struct request_queue *q, struct request *rq) +{ + struct io_queue *ioq = rq->ioq; + + if (!elv_iosched_fair_queuing_enabled(q->elevator)) + return; + + if (ioq) { + rq->ioq = NULL; + elv_put_ioq(ioq); + } +} + #else /* GROUP_IOSCHED */ void bfq_init_entity(struct io_entity *entity, struct io_group *iog) { diff --git a/block/elevator-fq.h b/block/elevator-fq.h index 3fab8f8..fc4110d 100644 --- a/block/elevator-fq.h +++ b/block/elevator-fq.h @@ -232,6 +232,9 @@ struct io_group { /* async_queue and idle_queue are used only for cfq */ struct io_queue *async_queue[2][IOPRIO_BE_NR]; struct io_queue *async_idle_queue; + + /* Single ioq per group, used for noop, deadline, anticipatory */ + struct io_queue *ioq; }; /** @@ -461,6 +464,28 @@ extern void io_ioq_move(struct elevator_queue *e, struct io_queue *ioq, struct io_group *iog); extern void elv_fq_set_request_io_group(struct request_queue *q, struct request *rq); +extern int elv_fq_set_request_ioq(struct request_queue *q, struct request *rq, + gfp_t gfp_mask); +extern void elv_fq_unset_request_ioq(struct request_queue *q, + struct request *rq); +extern struct io_queue *elv_lookup_ioq_current(struct request_queue *q); + +/* Returns single ioq associated with the io group. */ +static inline struct io_queue *io_group_ioq(struct io_group *iog) +{ + BUG_ON(!iog); + return iog->ioq; +} + +/* Sets the single ioq associated with the io group. (noop, deadline, AS) */ +static inline void io_group_set_ioq(struct io_group *iog, struct io_queue *ioq) +{ + BUG_ON(!iog); + /* io group reference. Will be dropped when group is destroyed. */ + elv_get_ioq(ioq); + iog->ioq = ioq; +} + #else /* !GROUP_IOSCHED */ /* * No ioq movement is needed in case of flat setup. root io group gets cleaned @@ -486,6 +511,32 @@ static inline void elv_fq_set_request_io_group(struct request_queue *q, { } +/* Returns single ioq associated with the io group. */ +static inline struct io_queue *io_group_ioq(struct io_group *iog) +{ + return NULL; +} + +static inline void io_group_set_ioq(struct io_group *iog, struct io_queue *ioq) +{ +} + +static inline int elv_fq_set_request_ioq(struct request_queue *q, + struct request *rq, gfp_t gfp_mask) +{ + return 0; +} + +static inline void elv_fq_unset_request_ioq(struct request_queue *q, + struct request *rq) +{ +} + +static inline struct io_queue *elv_lookup_ioq_current(struct request_queue *q) +{ + return NULL; +} + #endif /* GROUP_IOSCHED */ /* Functions used by blksysfs.c */ @@ -588,5 +639,21 @@ static inline void elv_fq_set_request_io_group(struct request_queue *q, struct request *rq) { } +static inline int elv_fq_set_request_ioq(struct request_queue *q, + struct request *rq, gfp_t gfp_mask) +{ + return 0; +} + +static inline void elv_fq_unset_request_ioq(struct request_queue *q, + struct request *rq) +{ +} + +static inline struct io_queue *elv_lookup_ioq_current(struct request_queue *q) +{ + return NULL; +} + #endif /* CONFIG_ELV_FAIR_QUEUING */ #endif /* _BFQ_SCHED_H */ diff --git a/block/elevator.c b/block/elevator.c index 5df13c4..bce4421 100644 --- a/block/elevator.c +++ b/block/elevator.c @@ -207,6 +207,14 @@ static void *elevator_alloc_sched_queue(struct request_queue *q, { void *sched_queue = NULL; + /* + * If fair queuing is enabled, then queue allocation takes place + * during set_request() functions when request actually comes + * in. + */ + if (elv_iosched_fair_queuing_enabled(eq)) + return NULL; + if (eq->ops->elevator_alloc_sched_queue_fn) { sched_queue = eq->ops->elevator_alloc_sched_queue_fn(q, eq, GFP_KERNEL); @@ -936,6 +944,13 @@ int elv_set_request(struct request_queue *q, struct request *rq, gfp_t gfp_mask) elv_fq_set_request_io_group(q, rq); + /* + * Optimization for noop, deadline and AS which maintain only single + * ioq per io group + */ + if (elv_iosched_single_ioq(e)) + return elv_fq_set_request_ioq(q, rq, gfp_mask); + if (e->ops->elevator_set_req_fn) return e->ops->elevator_set_req_fn(q, rq, gfp_mask); @@ -947,6 +962,15 @@ void elv_put_request(struct request_queue *q, struct request *rq) { struct elevator_queue *e = q->elevator; + /* + * Optimization for noop, deadline and AS which maintain only single + * ioq per io group + */ + if (elv_iosched_single_ioq(e)) { + elv_fq_unset_request_ioq(q, rq); + return; + } + if (e->ops->elevator_put_req_fn) e->ops->elevator_put_req_fn(rq); } @@ -1329,9 +1353,18 @@ EXPORT_SYMBOL(elv_select_sched_queue); /* * Get the io scheduler queue pointer for current task. + * + * If fair queuing is enabled, determine the io group of task and retrieve + * the ioq pointer from that. This is used by only single queue ioschedulers + * for retrieving the queue associated with the group to decide whether the + * new bio can do a front merge or not. */ void *elv_get_sched_queue_current(struct request_queue *q) { - return q->elevator->sched_queue; + /* Fair queuing is not enabled. There is only one queue. */ + if (!elv_iosched_fair_queuing_enabled(q->elevator)) + return q->elevator->sched_queue; + + return ioq_sched_queue(elv_lookup_ioq_current(q)); } EXPORT_SYMBOL(elv_get_sched_queue_current); diff --git a/include/linux/elevator.h b/include/linux/elevator.h index bb5ae3a..8cee877 100644 --- a/include/linux/elevator.h +++ b/include/linux/elevator.h @@ -245,17 +245,31 @@ enum { /* iosched wants to use fq logic of elevator layer */ #define ELV_IOSCHED_NEED_FQ 1 +/* iosched maintains only single ioq per group.*/ +#define ELV_IOSCHED_SINGLE_IOQ 2 + static inline int elv_iosched_fair_queuing_enabled(struct elevator_queue *e) { return (e->elevator_type->elevator_features) & ELV_IOSCHED_NEED_FQ; } +static inline int elv_iosched_single_ioq(struct elevator_queue *e) +{ + return (e->elevator_type->elevator_features) & ELV_IOSCHED_SINGLE_IOQ; +} + #else /* ELV_IOSCHED_FAIR_QUEUING */ static inline int elv_iosched_fair_queuing_enabled(struct elevator_queue *e) { return 0; } + +static inline int elv_iosched_single_ioq(struct elevator_queue *e) +{ + return 0; +} + #endif /* ELV_IOSCHED_FAIR_QUEUING */ extern void *elv_get_sched_queue(struct request_queue *q, struct request *rq); extern void *elv_select_sched_queue(struct request_queue *q, int force); -- 1.6.0.1 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/