aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorTakashi Iwai <tiwai@suse.de>2008-06-17 10:30:27 -0400
committerTakashi Iwai <tiwai@suse.de>2008-06-17 10:30:27 -0400
commit284373059f9605442cac6453780f6aaecf9abac1 (patch)
treea4a449c3d19695dce2cff64deb2211c69c83eca8
parent9f515b6898d0f2c1b9eb34ae9986794045f34e2b (diff)
ALSA: emu10k1 - simplify the last fix
Clean up the previous commit for fixing memory leaks. Signed-off-by: Takashi Iwai <tiwai@suse.de>
-rw-r--r--sound/pci/emu10k1/memory.c9
1 files changed, 3 insertions, 6 deletions
diff --git a/sound/pci/emu10k1/memory.c b/sound/pci/emu10k1/memory.c
index 759e29f89478..7d379f5131fb 100644
--- a/sound/pci/emu10k1/memory.c
+++ b/sound/pci/emu10k1/memory.c
@@ -465,15 +465,12 @@ static int synth_alloc_pages(struct snd_emu10k1 *emu, struct snd_emu10k1_memblk
465 struct page *p = alloc_page(GFP_KERNEL | GFP_DMA32 | 465 struct page *p = alloc_page(GFP_KERNEL | GFP_DMA32 |
466 __GFP_NOWARN); 466 __GFP_NOWARN);
467 if (!p || (page_to_pfn(p) & ~(emu->dma_mask >> PAGE_SHIFT))) { 467 if (!p || (page_to_pfn(p) & ~(emu->dma_mask >> PAGE_SHIFT))) {
468 if (p)
469 __free_page(p);
468 /* try to allocate from <16MB zone */ 470 /* try to allocate from <16MB zone */
469 struct page *p1 = 471 p = alloc_page(GFP_ATOMIC | GFP_DMA |
470 alloc_page(GFP_ATOMIC | GFP_DMA |
471 __GFP_NORETRY | /* no OOM-killer */ 472 __GFP_NORETRY | /* no OOM-killer */
472 __GFP_NOWARN); 473 __GFP_NOWARN);
473 /* free page outside dma_mask range */
474 if (p)
475 free_page((unsigned long)page_address(p));
476 p = p1;
477 } 474 }
478 if (!p) { 475 if (!p) {
479 __synth_free_pages(emu, first_page, page - 1); 476 __synth_free_pages(emu, first_page, page - 1);