[Ocfs2-devel] [patch] ocfs2/dlm: use GFP_ATOMIC inside a spin_lock

[Joel Becker]

On Thu, Jul 26, 2012 at 04:05:05PM +0300, Dan Carpenter wrote:
> My static checker complains that this is called with a spin_lock held
> in dlm_master_requery_handler() from dlmrecovery.c.  Probably the reason
> we have not received any bug reports about this is that recovery is not
> a common operation.
> 
> Signed-off-by: Dan Carpenter <dan.carpenter at oracle.com>

This patch is now part of the fixes branch of ocfs2.git.

Joel

> 
> diff --git a/fs/ocfs2/dlm/dlmmaster.c b/fs/ocfs2/dlm/dlmmaster.c
> index 005261c..33ecbe0 100644
> --- a/fs/ocfs2/dlm/dlmmaster.c
> +++ b/fs/ocfs2/dlm/dlmmaster.c
> @@ -2020,7 +2020,7 @@ int dlm_dispatch_assert_master(struct dlm_ctxt *dlm,
>  			       int ignore_higher, u8 request_from, u32 flags)
>  {
>  	struct dlm_work_item *item;
> -	item = kzalloc(sizeof(*item), GFP_NOFS);
> +	item = kzalloc(sizeof(*item), GFP_ATOMIC);
>  	if (!item)
>  		return -ENOMEM;
>  

-- 

"We will have to repent in this generation not merely for the
 vitriolic words and actions of the bad people, but for the 
 appalling silence of the good people."
	- Rev. Dr. Martin Luther King, Jr.

			http://www.jlbec.org/
			jlbec at evilplan.org