From mboxrd@z Thu Jan 1 00:00:00 1970 From: Markus Trippelsdorf Subject: Re: snd-usb: "delay: estimated 0, actual 352" Date: Thu, 6 Sep 2012 15:17:58 +0200 Message-ID: <20120906131758.GA256@x4> References: <20120906060220.GA252@x4> <504843BA.2040808@gmail.com> <20120906065340.GA257@x4> <50484BEB.4020103@gmail.com> <20120906071757.GB257@x4> <20120906094348.GA257@x4> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org To: Takashi Iwai Cc: Daniel Mack , Linus Torvalds , linux-kernel@vger.kernel.org, alsa-devel@alsa-project.org, Pierre-Louis Bossart List-Id: alsa-devel@alsa-project.org On 2012.09.06 at 15:08 +0200, Takashi Iwai wrote: > At Thu, 6 Sep 2012 11:43:48 +0200, > Markus Trippelsdorf wrote: > > > > On 2012.09.06 at 10:21 +0200, Takashi Iwai wrote: > > > At Thu, 06 Sep 2012 09:35:26 +0200, > > > Takashi Iwai wrote: > > > > > > In short, a patch like below may fix the issue (note: completely > > > untested!) > > > > No it doesn't, unfortunately... > > OK, I start tracking down the problem a bit more deeply now. > > The issue happens when the first two URBs are passed to > retire_playback_urb(). These are URBs filled before start_endpoints() > are set, so they contain actually zero size. Even though these are > a sort of dummy packets, the driver still tries to check with the > queued delay account, and gives bogus errors. > > So, essentially the messages are harmless and nothing to worry too > much, but surely it doesn't look sexy. > > The patch below should fix the problem. Please give it a try. Yes, your patch finally fixes the problem. Thank you Takashi-san. -- Markus