OpenVZ Forum: Devel » [PATCH] ext3: ext3_symlink should use GFP

Home » Mailing lists » Devel » [PATCH] ext3: ext3_symlink should use GFP_NOFS allocations inside (ver. 3)

Show: Today's Messages :: Show Polls :: Message Navigator
E-mail to friend

Re: [PATCH] ext3: ext3_symlink should use GFP_NOFS allocations inside (ver. 3) [message #1962 is a reply to message #1958]

Fri, 10 March 2006 08:46

Arjan van de Ven
Messages: 14
Registered: March 2006

Junior Member

On Fri, 2006-03-10 at 11:46 +0300, Kirill Korotaev wrote:
> Andrew,
>
> Fixed both comments from Al Viro (thanks, Al):
> - should have a separate helper
> - should pass 0 instead of GFP_KERNEL in page_symlink()

>
> + page = find_or_create_page(mapping, 0,
> + mapping_gfp_mask(mapping) | gfp_mask);

this does not work; GFP_NOFS has a bit *LESS* than GFP_KERNEL, not a bit
more. As such a | operation isn't going to be useful....

(So I think that while Al's intention was good, the implication of it
isn't ;)

Report message to a moderator

[Message index]

		[PATCH] ext3: ext3_symlink should use GFP_NOFS allocations inside (ver. 3) By: Kirill Korotaev on Fri, 10 March 2006 08:41
		Re: [PATCH] ext3: ext3_symlink should use GFP_NOFS allocations inside (ver. 3) By: Arjan van de Ven on Fri, 10 March 2006 08:46
		Re: [PATCH] ext3: ext3_symlink should use GFP_NOFS allocations inside (ver. 3) By: dev on Fri, 10 March 2006 08:52
		Re: [PATCH] ext3: ext3_symlink should use GFP_NOFS allocations inside (ver. 3) By: Andrew Morton on Fri, 10 March 2006 08:56
		Re: [PATCH] ext3: ext3_symlink should use GFP_NOFS allocations inside (ver. 3) By: dev on Fri, 10 March 2006 09:06
		Re: [PATCH] ext3: ext3_symlink should use GFP_NOFS allocations inside (ver. 3) By: Al Viro on Fri, 10 March 2006 08:59
		Re: [PATCH] ext3: ext3_symlink should use GFP_NOFS allocations inside (ver. 3) By: Arjan van de Ven on Fri, 10 March 2006 09:40

Previous Topic:	[PATCH] ext3: ext3_symlink should use GFP_NOFS allocations inside
Next Topic:	[PATCH 1/2] virtualized ipt_REDIRECT

Goto Forum:

-=] Back to Top [=-

[ Syndicate this forum (XML) ] [

] [

]

Current Time: Tue Dec 02 03:08:22 GMT 2025

Total time taken to generate the page: 0.55925 seconds