zram: remove request_queue from struct zram
authorSergey Senozhatsky <sergey.senozhatsky@gmail.com>
Thu, 12 Feb 2015 23:00:48 +0000 (15:00 -0800)
committerLinus Torvalds <torvalds@linux-foundation.org>
Fri, 13 Feb 2015 02:54:12 +0000 (18:54 -0800)
`struct zram' contains both `struct gendisk' and `struct request_queue'.
the latter can be deleted, because zram->disk carries ->queue pointer, and
->queue carries zram pointer:

create_device()
zram->queue->queuedata = zram
zram->disk->queue = zram->queue
zram->disk->private_data = zram

so zram->queue is not needed, we can access all necessary data anyway.

Signed-off-by: Sergey Senozhatsky <sergey.senozhatsky@gmail.com>
Cc: Minchan Kim <minchan@kernel.org>
Cc: Jerome Marchand <jmarchan@redhat.com>
Cc: Nitin Gupta <ngupta@vflare.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
drivers/block/zram/zram_drv.c
drivers/block/zram/zram_drv.h

index db94572b35c41e6226f47cd96b8375878aa0fdaa..eca4b67274c183815b3189b6bcba62ad96a8e3c7 100644 (file)
@@ -1061,19 +1061,19 @@ static struct attribute_group zram_disk_attr_group = {
 
 static int create_device(struct zram *zram, int device_id)
 {
+       struct request_queue *queue;
        int ret = -ENOMEM;
 
        init_rwsem(&zram->init_lock);
 
-       zram->queue = blk_alloc_queue(GFP_KERNEL);
-       if (!zram->queue) {
+       queue = blk_alloc_queue(GFP_KERNEL);
+       if (!queue) {
                pr_err("Error allocating disk queue for device %d\n",
                        device_id);
                goto out;
        }
 
-       blk_queue_make_request(zram->queue, zram_make_request);
-       zram->queue->queuedata = zram;
+       blk_queue_make_request(queue, zram_make_request);
 
         /* gendisk structure */
        zram->disk = alloc_disk(1);
@@ -1086,7 +1086,8 @@ static int create_device(struct zram *zram, int device_id)
        zram->disk->major = zram_major;
        zram->disk->first_minor = device_id;
        zram->disk->fops = &zram_devops;
-       zram->disk->queue = zram->queue;
+       zram->disk->queue = queue;
+       zram->disk->queue->queuedata = zram;
        zram->disk->private_data = zram;
        snprintf(zram->disk->disk_name, 16, "zram%d", device_id);
 
@@ -1137,7 +1138,7 @@ out_free_disk:
        del_gendisk(zram->disk);
        put_disk(zram->disk);
 out_free_queue:
-       blk_cleanup_queue(zram->queue);
+       blk_cleanup_queue(queue);
 out:
        return ret;
 }
@@ -1158,10 +1159,9 @@ static void destroy_devices(unsigned int nr)
 
                zram_reset_device(zram);
 
+               blk_cleanup_queue(zram->disk->queue);
                del_gendisk(zram->disk);
                put_disk(zram->disk);
-
-               blk_cleanup_queue(zram->queue);
        }
 
        kfree(zram_devices);
index 5249f51ccdb38513180c6150ca227e238417b16f..17056e5891467caafc3a238633f9b3eb6b094158 100644 (file)
@@ -101,7 +101,6 @@ struct zram_meta {
 struct zram {
        struct zram_meta *meta;
        struct zcomp *comp;
-       struct request_queue *queue;
        struct gendisk *disk;
        /* Prevent concurrent execution of device init */
        struct rw_semaphore init_lock;