Alan Cox wrote:
> o Next incarnation of the i810 audio driver (Doug Ledford)
Is this the i810 that's in Red Hat's CVS or the last copy of the big file that
I sent you? If it's the last copy of the big file I sent you, then it has a
memory leak that needs fixed. I committed the fix for the memory leak to the
CVS archive something like two days ago. The patch is attached.
--Doug Ledford <dledford@redhat.com> http://people.redhat.com/dledford Please check my web site for aic7xxx updates/answers before e-mailing me about problems
--- linux/drivers/sound/i810_audio.c.save Wed Mar 21 20:44:29 2001 +++ linux/drivers/sound/i810_audio.c Wed Mar 21 20:44:34 2001 @@ -1820,12 +1820,11 @@ return -EBUSY; } stop_dac(state); - dealloc_dmabuf(state); } if(dmabuf->enable & ADC_RUNNING) { stop_adc(state); - dealloc_dmabuf(state); } + dealloc_dmabuf(state); if (file->f_mode & FMODE_WRITE) { state->card->free_pcm_channel(state->card, dmabuf->write_channel->num); }
- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
This archive was generated by hypermail 2b29 : Fri Mar 23 2001 - 21:00:19 EST