Re: kernel BUG at include/linux/dcache.h:271!

From: Maneesh Soni (maneesh@in.ibm.com)
Date: Fri May 23 2003 - 07:55:35 EST


On Fri, May 23, 2003 at 06:13:24PM +0530, Maneesh Soni wrote:
> On Fri, May 23, 2003 at 08:25:08AM +0200, Jens Axboe wrote:
> > On Thu, May 22 2003, Andrew Morton wrote:
> > > Maneesh Soni <maneesh@in.ibm.com> wrote:
> > > >
> > > > ramdisk
> > > > - should have separate queues on for each ramdisk
> > > >
> > > > elevator
> > > > - should not re-register already registered queue in elv_register_queue
> > > >
> > > > sysfs
> > > > - should handle kobject with multiple parent kobjects
> > >
> > > I can't think of anywhere else where we are likely to want to support
> > > multiple devices from a single queue in this manner, so perhaps the best
> > > solution is to remove the exceptional case: allocate a separate queue for
> > > each ramdisk instance.
> > >
> > > Jens, do you agree?
> >
> > Completely and utterly agree :)
> >
> > --
> > Jens Axboe
>
> Hi,
>
> The following patch provides a separate queue for each ramdisk instance
> and the BUG is not seen now.
>
> Please check whether it is ok or not.
>
> Thanks,
> Maneesh

can't help.. I always have to send patch second time. Please see this one
instead.

- Provides a separate request queue for each ramdisk instance.

 drivers/block/rd.c | 19 +++++++++++++------
 1 files changed, 13 insertions(+), 6 deletions(-)

diff -puN drivers/block/rd.c~multiqueue_ramdisk drivers/block/rd.c
--- linux-2.5.69/drivers/block/rd.c~multiqueue_ramdisk 2003-05-23 16:04:38.000000000 +0530
+++ linux-2.5.69-maneesh/drivers/block/rd.c 2003-05-23 18:22:31.000000000 +0530
@@ -67,6 +67,7 @@
 
 static struct gendisk *rd_disks[NUM_RAMDISKS];
 static struct block_device *rd_bdev[NUM_RAMDISKS];/* Protected device data */
+static struct request_queue *rd_queue;
 
 /*
  * Parameters for the boot-loading of the RAM disk. These are set by
@@ -308,12 +309,11 @@ static void __exit rd_cleanup (void)
                 del_gendisk(rd_disks[i]);
                 put_disk(rd_disks[i]);
         }
-
+ kfree(rd_queue);
         devfs_remove("rd");
         unregister_blkdev(RAMDISK_MAJOR, "ramdisk" );
 }
 
-static struct request_queue rd_queue;
 /* This is the registration and initialization section of the RAM disk driver */
 static int __init rd_init (void)
 {
@@ -333,23 +333,28 @@ static int __init rd_init (void)
                         goto out;
         }
 
+ rd_queue = kmalloc(NUM_RAMDISKS * sizeof(struct request_queue),
+ GFP_KERNEL);
+ if (!rd_queue)
+ goto out;
+ memset(rd_queue, 0, NUM_RAMDISKS * sizeof(struct request_queue));
         if (register_blkdev(RAMDISK_MAJOR, "ramdisk")) {
                 err = -EIO;
- goto out;
+ goto out_queue;
         }
 
- blk_queue_make_request(&rd_queue, &rd_make_request);
-
         devfs_mk_dir("rd");
 
         for (i = 0; i < NUM_RAMDISKS; i++) {
                 struct gendisk *disk = rd_disks[i];
 
+ blk_queue_make_request(&rd_queue[i], &rd_make_request);
+
                 /* rd_size is given in kB */
                 disk->major = RAMDISK_MAJOR;
                 disk->first_minor = i;
                 disk->fops = &rd_bd_op;
- disk->queue = &rd_queue;
+ disk->queue = &rd_queue[i];
                 sprintf(disk->disk_name, "ram%d", i);
                 sprintf(disk->devfs_name, "rd/%d", i);
                 set_capacity(disk, rd_size * 2);
@@ -362,6 +367,8 @@ static int __init rd_init (void)
                NUM_RAMDISKS, rd_size, rd_blocksize);
 
         return 0;
+out_queue:
+ kfree(rd_queue);
 out:
         while (i--)
                 put_disk(rd_disks[i]);

_

 

-- 
Maneesh Soni
IBM Linux Technology Center, 
IBM India Software Lab, Bangalore.
Phone: +91-80-5044999 email: maneesh@in.ibm.com
http://lse.sourceforge.net/
-
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/



This archive was generated by hypermail 2b29 : Fri May 23 2003 - 22:00:55 EST