Re: [2.6 patch] noop-iosched.c: make some functions static

From: Jens Axboe
Date: Mon Nov 29 2004 - 14:48:28 EST


On Mon, Nov 29 2004, Adrian Bunk wrote:
> The patch below makes some needlessly global functions static.
>
>
> diffstat output:
> drivers/block/noop-iosched.c | 12 ++++++------
> 1 files changed, 6 insertions(+), 6 deletions(-)
>
>
> Signed-off-by: Adrian Bunk <bunk@xxxxxxxxx>

Acked-by: Jens Axboe <axboe@xxxxxxx>

> --- linux-2.6.10-rc1-mm3-full/drivers/block/noop-iosched.c.old 2004-11-06 20:10:24.000000000 +0100
> +++ linux-2.6.10-rc1-mm3-full/drivers/block/noop-iosched.c 2004-11-06 20:11:23.000000000 +0100
> @@ -17,7 +17,7 @@
> /*
> * See if we can find a request that this buffer can be coalesced with.
> */
> -int elevator_noop_merge(request_queue_t *q, struct request **req,
> +static int elevator_noop_merge(request_queue_t *q, struct request **req,
> struct bio *bio)
> {
> struct list_head *entry = &q->queue_head;
> @@ -50,13 +50,13 @@
> return ELEVATOR_NO_MERGE;
> }
>
> -void elevator_noop_merge_requests(request_queue_t *q, struct request *req,
> +static void elevator_noop_merge_requests(request_queue_t *q, struct request *req,
> struct request *next)
> {
> list_del_init(&next->queuelist);
> }
>
> -void elevator_noop_add_request(request_queue_t *q, struct request *rq,
> +static void elevator_noop_add_request(request_queue_t *q, struct request *rq,
> int where)
> {
> struct list_head *insert = q->queue_head.prev;
> @@ -75,7 +75,7 @@
> q->last_merge = rq;
> }
>
> -struct request *elevator_noop_next_request(request_queue_t *q)
> +static struct request *elevator_noop_next_request(request_queue_t *q)
> {
> if (!list_empty(&q->queue_head))
> return list_entry_rq(q->queue_head.next);
> @@ -94,12 +94,12 @@
> .elevator_owner = THIS_MODULE,
> };
>
> -int noop_init(void)
> +static int noop_init(void)
> {
> return elv_register(&elevator_noop);
> }
>
> -void noop_exit(void)
> +static void noop_exit(void)
> {
> elv_unregister(&elevator_noop);
> }
>
>

--
Jens Axboe

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/