diff options
author | Takashi Iwai <tiwai@suse.de> | 2008-06-17 10:30:27 -0400 |
---|---|---|
committer | Takashi Iwai <tiwai@suse.de> | 2008-06-17 10:30:27 -0400 |
commit | 284373059f9605442cac6453780f6aaecf9abac1 (patch) | |
tree | a4a449c3d19695dce2cff64deb2211c69c83eca8 /sound | |
parent | 9f515b6898d0f2c1b9eb34ae9986794045f34e2b (diff) |
ALSA: emu10k1 - simplify the last fix
Clean up the previous commit for fixing memory leaks.
Signed-off-by: Takashi Iwai <tiwai@suse.de>
Diffstat (limited to 'sound')
-rw-r--r-- | sound/pci/emu10k1/memory.c | 9 |
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); |