From: "Srinivas Ganji" <srinivasganji.linux@gmail.com>
To: "Jens Axboe" <axboe@suse.de>
Cc: linux-kernel@vger.kernel.org
Subject: Re: Spinlock Related Debug Messages in Block Driver
Date: Tue, 11 Jul 2006 13:12:42 +0530 [thread overview]
Message-ID: <4dccc9070607110042r78aeb843gb7b23d70505dfcec@mail.gmail.com> (raw)
In-Reply-To: <20060711071753.GS5210@suse.de>
On 7/11/06, Jens Axboe <axboe@suse.de> wrote:
> On Tue, Jul 11 2006, Srinivas Ganji wrote:
> > Dear All,
> >
> > I implemented the sample block driver for the removable devices and
> > the code contains no statements related to the spin lock except a lock
> > to the blk_init_queue API as shown below.
> >
> > spinlock_t qlock;
> > gDisk->blkqueue = blk_init_queue(do_my_request, &gDisk->qlock);
> >
> > The kernel version is 2.6.10.
> >
> > Everything works fine but when I try to copy a file of huge size
> > (about 100MB), the following debug messages are getting displayed on
> > the console:
> >
> > Fc3: drivers/block/ll_rw_blk.c: 2351: spin_lock already locked by the
> > drivers.
> > Fc3: drivers/block/ll_rw_blk.c: 2468: spin_unlock not locked by the drivers.
> >
> > In spite of these debug messages; the file is getting copied successfully.
> > I studied the Block driver 16th chapter in LDD third edition.
> >
> > Can any one provide a pointer to these debug messages. Do I need to
> > implement any patches for the kernel.
>
> You need to initialize the lock passed to blk_init_queue() first. See eg
> spin_lock_init(), or one of the static initializers
> (SPIN_LOCK_UNLOCKED).
>
> --
> Jens Axboe
Dear Jen,
Thanks for your reply.
I used the spin_lock_init API before blk_init_queue API. Even though I
got the error.
Is there any other reason for gertting the error messages?
Thanks and Regards,
Srinivas G
next prev parent reply other threads:[~2006-07-11 7:42 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-07-11 6:03 Srinivas Ganji
2006-07-11 7:17 ` Jens Axboe
2006-07-11 7:42 ` Srinivas Ganji [this message]
2006-07-11 7:46 ` Jens Axboe
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4dccc9070607110042r78aeb843gb7b23d70505dfcec@mail.gmail.com \
--to=srinivasganji.linux@gmail.com \
--cc=axboe@suse.de \
--cc=linux-kernel@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®