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 wrote: > > > > > On Saturday, May 07, 2011 3:16:25 pm Kostik Belousov wrote: > > > > On Fri, May 06, 2011 at 04:16:40PM -0400, John

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 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 Belous

Re: dsp mmap change

2011-05-09 Thread Alan Cox
On Mon, May 9, 2011 at 8:45 AM, John Baldwin 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 +0300, An

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

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

Re: dsp mmap change

2011-05-06 Thread John Baldwin
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 Gapon wrote: > > >> > > >> I would like to ask for a review a

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: > http://lists.freebsd.org/pipermail/freebsd-multimedia/20

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: >> http

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- Feb

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: > >> http://people.freebsd.org/

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

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 her

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: > http://lists.freebsd.org/pipermail/freebsd-multimedia/20