summaryrefslogtreecommitdiff
path: root/arch/mips
diff options
context:
space:
mode:
authorMichal Hocko <mhocko@suse.cz>2015-04-14 22:47:11 (GMT)
committerLinus Torvalds <torvalds@linux-foundation.org>2015-04-14 23:49:04 (GMT)
commitf91e8d6d76866eafba255864ca617438cec268de (patch)
treee3e18550b6cd7d110c187c8e5947be11c99fcd55 /arch/mips
parent647757197cd34fae041e21af39ded00f5c346fc4 (diff)
downloadlinux-f91e8d6d76866eafba255864ca617438cec268de.tar.xz
sparc: clarify __GFP_NOFAIL allocation
Commit 920c3ed74134 ("[SPARC64]: Add basic infrastructure for MD add/remove notification") has added __GFP_NOFAIL for the allocation request but it hasn't mentioned why is this strict requirement really needed. The code was handling an allocation failure and propagated it properly up the callchain so it is not clear why it is needed. Dave has clarified the intention when I tried to remove the flag as not being necessary: : It is a serious failure. : : If we miss an MDESC update due to this allocation failure, the update : is not an event which gets retransmitted so we will lose the updated : machine description forever. : : We really need this allocation to succeed. So add a comment to clarify the nofail flag and get rid of the failure check because __GFP_NOFAIL allocation doesn't fail. Signed-off-by: Michal Hocko <mhocko@suse.cz> Cc: David Rientjes <rientjes@google.com> Cc: Johannes Weiner <hannes@cmpxchg.org> Cc: Dave Chinner <david@fromorbit.com> Cc: "Theodore Ts'o" <tytso@mit.edu> Cc: Mel Gorman <mgorman@suse.de> Cc: Tetsuo Handa <penguin-kernel@I-love.SAKURA.ne.jp> Cc: "David S. Miller" <davem@davemloft.net> Cc: Vipul Pandya <vipul@chelsio.com> Cc: Jan Kara <jack@suse.cz> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'arch/mips')
0 files changed, 0 insertions, 0 deletions