2005-12-16 03:31:16

by Ashutosh Naik

[permalink] [raw]
Subject: [PATCH] kernel/module.c Getting rid of the redundant spinlock in resolve_symbol()

On 12/15/05, Rusty Russell <[email protected]> wrote:
> On Wed, 2005-12-14 at 11:16 +0530, Ashutosh Naik wrote:
> > On 12/14/05, Rusty Russell <[email protected]> wrote:
> > Was just wondering, in that case, if we really need the spinlock in
> > resolve_symbol() function, where there exists a spinlock around the
> > __find_symbol() function
>
> Yes, I think that's redundant as well. We're not altering the module
> list itself, so either of the two locks is sufficient, and we have the
> semaphore.

Changelog -

This patch gets rid of the redundant spinlock in the function
resolve_symbol() as we are not altering the module list, and we
already hold the semaphore.

Signed-off-by: Ashutosh Naik <[email protected]>


Attachments:
(No filename) (735.00 B)
mod-patch2.txt (684.00 B)
Download all attachments