NetBSD Problem Report #52685

From martin@aprisoft.de  Wed Nov  1 09:19:21 2017
Return-Path: <martin@aprisoft.de>
Received: from mail.netbsd.org (mail.netbsd.org [199.233.217.200])
	(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits))
	(Client CN "mail.NetBSD.org", Issuer "mail.NetBSD.org CA" (not verified))
	by mollari.NetBSD.org (Postfix) with ESMTPS id 94C2A7A0F3
	for <gnats-bugs@gnats.NetBSD.org>; Wed,  1 Nov 2017 09:19:21 +0000 (UTC)
Message-Id: <20171101091910.6A1565CC761@emmas.aprisoft.de>
Date: Wed,  1 Nov 2017 10:19:10 +0100 (CET)
From: martin@NetBSD.org
Reply-To: martin@NetBSD.org
To: gnats-bugs@NetBSD.org
Subject: audio stutters
X-Send-Pr-Version: 3.95

>Number:         52685
>Category:       kern
>Synopsis:       audio stutters
>Confidential:   no
>Severity:       critical
>Priority:       high
>Responsible:    kern-bug-people
>State:          closed
>Class:          sw-bug
>Submitter-Id:   net
>Arrival-Date:   Wed Nov 01 09:20:00 +0000 2017
>Closed-Date:    Sat Oct 05 03:55:33 +0000 2019
>Last-Modified:  Sat Oct 05 03:55:33 +0000 2019
>Originator:     Martin Husemann
>Release:        NetBSD 8.99.5
>Organization:
The NetBSD Foundation, Inc.
>Environment:
System: NetBSD whoever-brings-the-night.aprisoft.de 8.99.5 NetBSD 8.99.5 (WHOEVER) #184: Wed Nov 1 09:46:49 CET 2017 martin@seven-days-to-the-wolves.aprisoft.de:/work/src/sys/arch/sparc64/compile/WHOEVER sparc64
Architecture: sparc64
Machine: sparc64
>Description:

With very recent audio changes, there is a noticable repeating/stuttering
echo in audio rendering.

>How-To-Repeat:

Use mpg123 to play something on 

audio0 at autri0: full duplex, playback, capture, mmap, independent
autri0: Virtual format configured - Format SLINEAR, precision 16, channels 2, frequency 48000
midi0 at autri0: 4DWAVE MIDI UART

>Fix:
n/a

>Release-Note:

>Audit-Trail:
From: Martin Husemann <martin@duskware.de>
To: gnats-bugs@NetBSD.org
Cc: kern-bug-people@netbsd.org, gnats-admin@netbsd.org,
	netbsd-bugs@netbsd.org
Subject: Re: kern/52685: audio stutters
Date: Wed, 1 Nov 2017 10:24:46 +0100

 On Wed, Nov 01, 2017 at 09:20:00AM +0000, martin@NetBSD.org wrote:
 > Use mpg123 to play something on 
 > 
 > audio0 at autri0: full duplex, playback, capture, mmap, independent
 > autri0: Virtual format configured - Format SLINEAR, precision 16, channels 2, frequency 48000
 > midi0 at autri0: 4DWAVE MIDI UART

 I am not sure it happens always, maybe some songs just make it unnoticable.
 Certainly was very clear to hear on first playback after boot - or maybe
 it is kinda random.

 Martin

From: "Nathanial Sloss" <nat@netbsd.org>
To: gnats-bugs@gnats.NetBSD.org
Cc: 
Subject: PR/52685 CVS commit: src/sys/dev
Date: Thu, 2 Nov 2017 02:08:02 +0000

 Module Name:	src
 Committed By:	nat
 Date:		Thu Nov  2 02:08:02 UTC 2017

 Modified Files:
 	src/sys/dev: audio.c

 Log Message:
 Revert commit rev 1.419 to audio.c.  This should address PR kern/52685.

 This also means that drivers that have a static block size will have more
 latency at lower sample rates/precision/channels.  Also audio drivers that
 do more than supply rounded blocksizes in their round_blocksize functions
 will have to be changed.


 To generate a diff of this commit:
 cvs rdiff -u -r1.421 -r1.422 src/sys/dev/audio.c

 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.

Responsible-Changed-From-To: kern-bug-people->nat
Responsible-Changed-By: nat@NetBSD.org
Responsible-Changed-When: Sat, 04 Nov 2017 07:23:15 +0000
Responsible-Changed-Why:
Take.


State-Changed-From-To: open->feedback
State-Changed-By: nat@NetBSD.org
State-Changed-When: Sat, 04 Nov 2017 07:23:15 +0000
State-Changed-Why:
Should be fixed.  Please try a kernel after the 4th of November.


From: "Nathanial Sloss" <nat@netbsd.org>
To: gnats-bugs@gnats.NetBSD.org
Cc: 
Subject: PR/52685 CVS commit: src/sys/dev
Date: Thu, 16 Nov 2017 23:43:48 +0000

 Module Name:	src
 Committed By:	nat
 Date:		Thu Nov 16 23:43:48 UTC 2017

 Modified Files:
 	src/sys/dev: audio.c

 Log Message:
 Use correct combination of mix ring block size and vc playring used low
 for signalling the writer or fetching data from the vc play ring filters.

 When dealing with the ring buffer sc_mpr.s it is necessary to use the hwvc
 or mixring block sizes as they represent the final size of the data to be
 played back from the stream vc.

 When dealing with sc_pustream when there is play back filters or not one
 should use the vc->sc_mpr.blocksize, as this represents the amount of data
 before going through play back filters.

 This should address PR kern/52685.


 To generate a diff of this commit:
 cvs rdiff -u -r1.439 -r1.440 src/sys/dev/audio.c

 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.

Responsible-Changed-From-To: nat->kern-bug-people
Responsible-Changed-By: nat@NetBSD.org
Responsible-Changed-When: Wed, 08 May 2019 21:55:38 +0000
Responsible-Changed-Why:


State-Changed-From-To: feedback->closed
State-Changed-By: isaki@NetBSD.org
State-Changed-When: Sat, 05 Oct 2019 03:55:33 +0000
State-Changed-Why:
Obsoleted.
The audio driver has been completely replaced.
Please send new PR if you find any problems.


>Unformatted:

NetBSD Home
NetBSD PR Database Search

(Contact us) $NetBSD: query-full-pr,v 1.43 2018/01/16 07:36:43 maya Exp $
$NetBSD: gnats_config.sh,v 1.9 2014/08/02 14:16:04 spz Exp $
Copyright © 1994-2017 The NetBSD Foundation, Inc. ALL RIGHTS RESERVED.