From owner-cvs-src@FreeBSD.ORG  Mon Nov 14 18:20:48 2005
Return-Path: <owner-cvs-src@FreeBSD.ORG>
X-Original-To: cvs-src@FreeBSD.org
Delivered-To: cvs-src@FreeBSD.org
Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125])
	by hub.freebsd.org (Postfix) with ESMTP id 940F416A41F;
	Mon, 14 Nov 2005 18:20:48 +0000 (GMT)
	(envelope-from ariff@FreeBSD.org)
Received: from repoman.freebsd.org (repoman.freebsd.org [216.136.204.115])
	by mx1.FreeBSD.org (Postfix) with ESMTP id D868643D5D;
	Mon, 14 Nov 2005 18:20:47 +0000 (GMT)
	(envelope-from ariff@FreeBSD.org)
Received: from repoman.freebsd.org (localhost [127.0.0.1])
	by repoman.freebsd.org (8.13.1/8.13.1) with ESMTP id jAEIKlKD040439;
	Mon, 14 Nov 2005 18:20:47 GMT
	(envelope-from ariff@repoman.freebsd.org)
Received: (from ariff@localhost)
	by repoman.freebsd.org (8.13.1/8.13.1/Submit) id jAEIKlXq040438;
	Mon, 14 Nov 2005 18:20:47 GMT (envelope-from ariff)
Message-Id: <200511141820.jAEIKlXq040438@repoman.freebsd.org>
From: Ariff Abdullah <ariff@FreeBSD.org>
Date: Mon, 14 Nov 2005 18:20:47 +0000 (UTC)
To: src-committers@FreeBSD.org, cvs-src@FreeBSD.org, cvs-all@FreeBSD.org
X-FreeBSD-CVS-Branch: HEAD
Cc: 
Subject: cvs commit: src/sys/dev/sound/pcm dsp.c
X-BeenThere: cvs-src@freebsd.org
X-Mailman-Version: 2.1.5
Precedence: list
List-Id: CVS commit messages for the src tree <cvs-src.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-src>,
	<mailto:cvs-src-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/cvs-src>
List-Post: <mailto:cvs-src@freebsd.org>
List-Help: <mailto:cvs-src-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-src>,
	<mailto:cvs-src-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Mon, 14 Nov 2005 18:20:48 -0000

ariff       2005-11-14 18:20:47 UTC

  FreeBSD src repository

  Modified files:
    sys/dev/sound/pcm    dsp.c 
  Log:
  From luigi:
          In SNDCTL_DSP_SETFRAGMENT, if you specify both read and
          write channels, the existing code first acts on the
          read channel, but as a side effect it updates the
          arguments (maxfrags, fragsz) passed by the caller according
          to acceptable values for the read channel, and then uses the
          modified values to act on the write channel.
          The problem with this approach is that, given a
          (maxfrags, fragsz) user-specified value, the actual
          values computed by the read and write channels may differ:
          e.g. the read channel might want to allocate more fragments
          than what the user specified because it has no side-effects
          on the delay and it helps in case of slow readers,
          whereas the write channel needs to use as few fragments
          as possible to keep the audio latency low (very important
          with telephony apps).
  
          This patch stores the values computed by the read channel
          into temproary variables so the write channel will use
          the actual arguments of the ioctl.
  
          This patch is very helpful with telephony apps such as asterisk.
  
  Submitted by:   luigi
  Approved by:    netchild (mentor)
  
  Revision  Changes    Path
  1.88      +9 -2      src/sys/dev/sound/pcm/dsp.c