Re: [PATCH 6/7] cifs: Don't use PF_MEMALLOC

On Tue, 17 Nov 2009 16:22:32 +0900 (JST)
KOSAKI Motohiro <[email protected]> wrote:

> 
> Non MM subsystem must not use PF_MEMALLOC. Memory reclaim need few
> memory, anyone must not prevent it. Otherwise the system cause
> mysterious hang-up and/or OOM Killer invokation.
> 
> Cc: Steve French <[email protected]>
> Cc: [email protected]
> Cc: [email protected]
> Signed-off-by: KOSAKI Motohiro <[email protected]>
> ---
>  fs/cifs/connect.c |    1 -
>  1 files changed, 0 insertions(+), 1 deletions(-)
> 
> diff --git a/fs/cifs/connect.c b/fs/cifs/connect.c
> index 63ea83f..f9b1553 100644
> --- a/fs/cifs/connect.c
> +++ b/fs/cifs/connect.c
> @@ -337,7 +337,6 @@ cifs_demultiplex_thread(struct TCP_Server_Info *server)
>  	bool isMultiRsp;
>  	int reconnect;
>  
> -	current->flags |= PF_MEMALLOC;
>  	cFYI(1, ("Demultiplex PID: %d", task_pid_nr(current)));
>  
>  	length = atomic_inc_return(&tcpSesAllocCount);

This patch appears to be safe for CIFS. I believe that the demultiplex
thread only does mempool allocations currently. The only other case
where it did an allocation was recently changed with the conversion of
the oplock break code to use slow_work.

Barring anything I've missed...

Acked-by: Jeff Layton <[email protected]>

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to [email protected].  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"[email protected]"> [email protected] </a>