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

Home » Mailing lists » Devel » [PATCH] ext3: ext3_symlink should use GFP_NOFS allocations inside

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

Re: [PATCH] ext3: ext3_symlink should use GFP_NOFS allocations inside [message #1960 is a reply to message #1957]

Fri, 10 March 2006 08:31

Al Viro
Messages: 5
Registered: March 2006

Junior Member

On Fri, Mar 10, 2006 at 11:08:29AM +0300, Kirill Korotaev wrote:
> + page = find_or_create_page(mapping, 0,
> + mapping_gfp_mask(mapping) | gfp_mask);

> +int page_symlink(struct inode *inode, const char *symname, int len)
> +{
> + return __page_symlink(inode, symname, len, GFP_KERNEL);

s/GFP_KERNEL/0/; if somebody has e.g. GFP_NOFS in their mapping flags,
you end up breaking their code. We really pass extra flags to be added
to default ones; page_symlink() should pass 0.

Report message to a moderator

[Message index]

		[PATCH] ext3: ext3_symlink should use GFP_NOFS allocations inside By: Kirill Korotaev on Fri, 10 March 2006 08:03
		Re: [PATCH] ext3: ext3_symlink should use GFP_NOFS allocations inside By: dev on Fri, 10 March 2006 08:41
		Re: [PATCH] ext3: ext3_symlink should use GFP_NOFS allocations inside By: Al Viro on Fri, 10 March 2006 08:31

Previous Topic:	[Fwd: + i2o-memory-leak-in-i2o_exec_lct_modified.patch added to -mm tree]
Next Topic:	[PATCH] ext3: ext3_symlink should use GFP_NOFS allocations inside (ver. 3)

Goto Forum:

-=] Back to Top [=-

[ Syndicate this forum (XML) ] [

] [

]

Current Time: Sun Sep 21 09:09:12 GMT 2025

Total time taken to generate the page: 0.06505 seconds