diff options
author | Hugh Dickins <hugh@veritas.com> | 2005-11-22 00:32:22 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2005-11-22 12:13:43 -0500 |
commit | f3d48f0373c14a6203202f7b1dfc7b0d8aaf6ed2 (patch) | |
tree | e09198efae7c68b9b4799b0f85d67951cc33bb54 /arch/sparc/kernel | |
parent | 0b0db14c536debd92328819fe6c51a49717e8440 (diff) |
[PATCH] unpaged: fix sound Bad page states
Earlier I unifdefed PageCompound, so that snd_pcm_mmap_control_nopage and
others can give out a 0-order component of a higher-order page, which won't
be mistakenly freed when zap_pte_range unmaps it. But many Bad page states
reported a PG_reserved was freed after all: I had missed that we need to
say __GFP_COMP to get compound page behaviour.
Some of these higher-order pages are allocated by snd_malloc_pages, some by
snd_malloc_dev_pages; or if SBUS, by sbus_alloc_consistent - but that has
no gfp arg, so add __GFP_COMP into its sparc32/64 implementations.
I'm still rather puzzled that DRM seems not to need a similar change.
Signed-off-by: Hugh Dickins <hugh@veritas.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'arch/sparc/kernel')
-rw-r--r-- | arch/sparc/kernel/ioport.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/arch/sparc/kernel/ioport.c b/arch/sparc/kernel/ioport.c index d0f2bd227c4c..d39c9f206271 100644 --- a/arch/sparc/kernel/ioport.c +++ b/arch/sparc/kernel/ioport.c | |||
@@ -252,7 +252,7 @@ void *sbus_alloc_consistent(struct sbus_dev *sdev, long len, u32 *dma_addrp) | |||
252 | } | 252 | } |
253 | 253 | ||
254 | order = get_order(len_total); | 254 | order = get_order(len_total); |
255 | if ((va = __get_free_pages(GFP_KERNEL, order)) == 0) | 255 | if ((va = __get_free_pages(GFP_KERNEL|__GFP_COMP, order)) == 0) |
256 | goto err_nopages; | 256 | goto err_nopages; |
257 | 257 | ||
258 | if ((res = kmalloc(sizeof(struct resource), GFP_KERNEL)) == NULL) | 258 | if ((res = kmalloc(sizeof(struct resource), GFP_KERNEL)) == NULL) |