From owner-cvs-src@FreeBSD.ORG  Mon Nov 14 18:18:12 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 C991216A41F;
	Mon, 14 Nov 2005 18:18:12 +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 4B25843D4C;
	Mon, 14 Nov 2005 18:18:12 +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 jAEIIC2N040186;
	Mon, 14 Nov 2005 18:18:12 GMT
	(envelope-from ariff@repoman.freebsd.org)
Received: (from ariff@localhost)
	by repoman.freebsd.org (8.13.1/8.13.1/Submit) id jAEIICUW040185;
	Mon, 14 Nov 2005 18:18:12 GMT (envelope-from ariff)
Message-Id: <200511141818.jAEIICUW040185@repoman.freebsd.org>
From: Ariff Abdullah <ariff@FreeBSD.org>
Date: Mon, 14 Nov 2005 18:18:12 +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/pci ich.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:18:12 -0000

ariff       2005-11-14 18:18:12 UTC

  FreeBSD src repository

  Modified files:
    sys/dev/sound/pci    ich.c 
  Log:
  - Added few more Intel HDA ids (ICH 6/7) which does have backward
    compatible AC97 codec.
  - As the driver supports so many variants, create a table ids for
    ease of probing and maintenance.
    Submitted by:         yongari
    Reviewed/Tested by:   multimedia@
  - From luigi:
          The code to compute fragment sizes in the ich driver almost
          invariably ends up using the full buffer available, no matter
          how the user specifies fragment size and number.
          With audio telephony (8khz, 16bit-stereo) and the 16k buffer
          size this results in an unbearable 500ms delay.
          This patch makes sure that we never use more than 4 fragments,
          (i don't think we need more unless there are huge interrupt
          servicing latencies), and obey to the requested fragment size,
          so that latency is acceptable.
    Based on this (and after much regression tests), I can conclude
    that this driver works best with 2 fragments, thus solving various
    long standing issues of ICH driver not capable to flush or play
    short files perfectly.
    Suggested by:         luigi (the idea of smaller fragments)
  - MPSAFE conversion.
  
  Approved by:    netchild (mentor)
  
  Revision  Changes    Path
  1.55      +153 -103  src/sys/dev/sound/pci/ich.c