Re: [RFC][PATCH 2/8] Create emergency trigger

From: Paul Jackson
Date: Fri Nov 18 2005 - 19:28:20 EST


> @@ -876,6 +879,16 @@ __alloc_pages(gfp_t gfp_mask, unsigned i
> int can_try_harder;
> int did_some_progress;
>
> + if (is_emergency_alloc(gfp_mask)) {

Can this check for is_emergency_alloc be moved lower in __alloc_pages?

I don't see any reason why most __alloc_pages() calls, that succeed
easily in the first loop over the zonelist, have to make this check.
This would save one conditional test and jump on the most heavily
used code path in __alloc_pages().

--
I won't rest till it's the best ...
Programmer, Linux Scalability
Paul Jackson <pj@xxxxxxx> 1.925.600.0401
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/