On Wed, Oct 17, 2012 at 1:14 PM, Kamezawa Hiroyuki <kamezawa.hiroyu@xxxxxxxxxxxxxx> wrote: > > I think the idea is reasonable. I have a request. Thanks for your comment. > > In current implemententation of vmscan.c, it seems sc.may_writepage, sc.may_swap > are handled independent from gfp_mask. > > So, could you drop changes from this patch and handle these flags in another patch > if these flags should be unset if ~GFP_IOFS ? OK, I agree. In theory, mm should make sure no I/O is involved if memory allocation users passes ~GFP_IOFS. > > I think try_to_free_page() path's sc.may_xxxx should be handled in the same way. Yes, alloc_page_buffers() and dma_alloc_from_contiguous may drop into the path, so gfp flag should be changed in try_to_free_page() too. Thanks, -- Ming Lei -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxx. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>