All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] cpuset: remove cpuset_unlock()
@ 2011-01-04  8:23 Lai Jiangshan
  2011-01-04 19:52 ` Paul Menage
  0 siblings, 1 reply; 2+ messages in thread
From: Lai Jiangshan @ 2011-01-04  8:23 UTC (permalink / raw)
  To: Paul Menage, Li Zefan, MiaoX, LKML

It is not used.
It is not declared.
There is no corresponding cpuset_lock().

signed-off-by: Lai Jiangshan <laijs@cn.fujitsu.com>
---
diff --git a/kernel/cpuset.c b/kernel/cpuset.c
index 4349935..6ef90dd 100644
--- a/kernel/cpuset.c
+++ b/kernel/cpuset.c
@@ -2440,17 +2440,6 @@ int __cpuset_node_allowed_hardwall(int node, gfp_t gfp_mask)
 }
 
 /**
- * cpuset_unlock - release lock on cpuset changes
- *
- * Undo the lock taken in a previous cpuset_lock() call.
- */
-
-void cpuset_unlock(void)
-{
-	mutex_unlock(&callback_mutex);
-}
-
-/**
  * cpuset_mem_spread_node() - On which node to begin search for a file page
  * cpuset_slab_spread_node() - On which node to begin search for a slab page
  *

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] cpuset: remove cpuset_unlock()
  2011-01-04  8:23 [PATCH] cpuset: remove cpuset_unlock() Lai Jiangshan
@ 2011-01-04 19:52 ` Paul Menage
  0 siblings, 0 replies; 2+ messages in thread
From: Paul Menage @ 2011-01-04 19:52 UTC (permalink / raw)
  To: Lai Jiangshan; +Cc: Li Zefan, MiaoX, LKML

On Tue, Jan 4, 2011 at 12:23 AM, Lai Jiangshan <laijs@cn.fujitsu.com> wrote:
> It is not used.
> It is not declared.
> There is no corresponding cpuset_lock().
>
> signed-off-by: Lai Jiangshan <laijs@cn.fujitsu.com>

Acked-by: Paul Menage <menage@google.com>

Thanks,
Paul

> ---
> diff --git a/kernel/cpuset.c b/kernel/cpuset.c
> index 4349935..6ef90dd 100644
> --- a/kernel/cpuset.c
> +++ b/kernel/cpuset.c
> @@ -2440,17 +2440,6 @@ int __cpuset_node_allowed_hardwall(int node, gfp_t gfp_mask)
>  }
>
>  /**
> - * cpuset_unlock - release lock on cpuset changes
> - *
> - * Undo the lock taken in a previous cpuset_lock() call.
> - */
> -
> -void cpuset_unlock(void)
> -{
> -       mutex_unlock(&callback_mutex);
> -}
> -
> -/**
>  * cpuset_mem_spread_node() - On which node to begin search for a file page
>  * cpuset_slab_spread_node() - On which node to begin search for a slab page
>  *
>

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2011-01-04 19:52 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-01-04  8:23 [PATCH] cpuset: remove cpuset_unlock() Lai Jiangshan
2011-01-04 19:52 ` Paul Menage

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.