Re: [PATCH] pgtables: fix GFP_KERNEL allocation with preemptdisabled

[David S. Miller]

On Tue, 19 Apr 2005 13:04:13 -0400
Martin Hicks <mort@xxxxxxx> wrote:

> This is a fix to the pgtable_quicklist code.  There is a GFP_KERNEL
> allocation in pgtable_quicklist_alloc(), which spews the usual warnings
> if the kernel is under heavy VM pressure and the reclaim code is
> invoked.
> 
> This patch is against 2.6.12-rc2-mm2

I think you should really drop the preempt disable during this allocation
instead, that's what we do in the sparc64 quicklist code.

It's very simple, change the code to:

		preempt_enable();
	} else {
		preempt_enable();
		ret = (unsigned long *) ...;
	}
	/* preempt_enable() no longer here */
-
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/