Re: dsp mmap change

2011-05-09 Thread John Baldwin
On Saturday, May 07, 2011 3:16:25 pm Kostik Belousov wrote: On Fri, May 06, 2011 at 04:16:40PM -0400, John Baldwin wrote: On Friday, May 06, 2011 10:04:28 am Kostik Belousov wrote: On Fri, May 06, 2011 at 04:38:00PM +0300, Andriy Gapon wrote: on 06/05/2011 16:32 Kostik Belousov said the

Re: dsp mmap change

2011-05-09 Thread Alan Cox
On Mon, May 9, 2011 at 8:45 AM, John Baldwin j...@freebsd.org wrote: On Saturday, May 07, 2011 3:16:25 pm Kostik Belousov wrote: On Fri, May 06, 2011 at 04:16:40PM -0400, John Baldwin wrote: On Friday, May 06, 2011 10:04:28 am Kostik Belousov wrote: On Fri, May 06, 2011 at 04:38:00PM

Re: dsp mmap change

2011-05-09 Thread John Baldwin
On Monday, May 09, 2011 11:35:07 am Alan Cox wrote: On Mon, May 9, 2011 at 8:45 AM, John Baldwin j...@freebsd.org wrote: On Saturday, May 07, 2011 3:16:25 pm Kostik Belousov wrote: On Fri, May 06, 2011 at 04:16:40PM -0400, John Baldwin wrote: On Friday, May 06, 2011 10:04:28 am Kostik

Re: dsp mmap change

2011-05-09 Thread Kostik Belousov
On Mon, May 09, 2011 at 01:38:24PM -0400, John Baldwin wrote: On Monday, May 09, 2011 11:35:07 am Alan Cox wrote: On Mon, May 9, 2011 at 8:45 AM, John Baldwin j...@freebsd.org wrote: On Saturday, May 07, 2011 3:16:25 pm Kostik Belousov wrote: On Fri, May 06, 2011 at 04:16:40PM -0400,

Re: dsp mmap change

2011-05-07 Thread Kostik Belousov
On Fri, May 06, 2011 at 04:16:40PM -0400, John Baldwin wrote: On Friday, May 06, 2011 10:04:28 am Kostik Belousov wrote: On Fri, May 06, 2011 at 04:38:00PM +0300, Andriy Gapon wrote: on 06/05/2011 16:32 Kostik Belousov said the following: On Fri, May 06, 2011 at 11:55:00AM +0300, Andriy

dsp mmap change

2011-05-06 Thread Andriy Gapon
I would like to ask for a review and/or testing of the following patch: http://people.freebsd.org/~avg/dev_dsp_mmap.diff It's supposed to fix an issue described here: http://lists.freebsd.org/pipermail/freebsd-multimedia/2011-February/011691.html In short, the following pseudo-code should do

Re: dsp mmap change

2011-05-06 Thread Kostik Belousov
On Fri, May 06, 2011 at 11:55:00AM +0300, Andriy Gapon wrote: I would like to ask for a review and/or testing of the following patch: http://people.freebsd.org/~avg/dev_dsp_mmap.diff It's supposed to fix an issue described here:

Re: dsp mmap change

2011-05-06 Thread Andriy Gapon
on 06/05/2011 16:32 Kostik Belousov said the following: On Fri, May 06, 2011 at 11:55:00AM +0300, Andriy Gapon wrote: I would like to ask for a review and/or testing of the following patch: http://people.freebsd.org/~avg/dev_dsp_mmap.diff It's supposed to fix an issue described here:

Re: dsp mmap change

2011-05-06 Thread Andriy Gapon
on 06/05/2011 16:38 Andriy Gapon said the following: on 06/05/2011 16:32 Kostik Belousov said the following: Your patch hardcodes an assumption that sndbufs are always contiguous. I was unable to convince myself that this is true. I think that this should be true for the case when DMA is

Re: dsp mmap change

2011-05-06 Thread Kostik Belousov
On Fri, May 06, 2011 at 04:38:00PM +0300, Andriy Gapon wrote: on 06/05/2011 16:32 Kostik Belousov said the following: On Fri, May 06, 2011 at 11:55:00AM +0300, Andriy Gapon wrote: I would like to ask for a review and/or testing of the following patch:

Re: dsp mmap change

2011-05-06 Thread John Baldwin
On Friday, May 06, 2011 4:55:00 am Andriy Gapon wrote: I would like to ask for a review and/or testing of the following patch: http://people.freebsd.org/~avg/dev_dsp_mmap.diff It's supposed to fix an issue described here: http://lists.freebsd.org/pipermail/freebsd-multimedia/2011-

Re: dsp mmap change

2011-05-06 Thread Andriy Gapon
on 06/05/2011 16:00 John Baldwin said the following: On Friday, May 06, 2011 4:55:00 am Andriy Gapon wrote: I would like to ask for a review and/or testing of the following patch: http://people.freebsd.org/~avg/dev_dsp_mmap.diff It's supposed to fix an issue described here:

Re: dsp mmap change

2011-05-06 Thread Juergen Lock
On Fri, May 06, 2011 at 11:55:00AM +0300, Andriy Gapon wrote: I would like to ask for a review and/or testing of the following patch: http://people.freebsd.org/~avg/dev_dsp_mmap.diff It's supposed to fix an issue described here: