From mboxrd@z Thu Jan 1 00:00:00 1970 From: Clemens Ladisch Subject: Re: PulseAudio and SNDRV_PCM_INFO_BATCH Date: Sat, 27 Jun 2015 19:15:59 +0200 Message-ID: <558EDA4F.5030309@ladisch.de> References: <557E86DD.8020709@metafoo.de> <558EC138.8090906@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from dehamd003.servertools24.de (dehamd003.servertools24.de [31.47.254.18]) by alsa0.perex.cz (Postfix) with ESMTP id 0BCC12619BE for ; Sat, 27 Jun 2015 19:17:11 +0200 (CEST) In-Reply-To: <558EC138.8090906@gmail.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: alsa-devel-bounces@alsa-project.org Sender: alsa-devel-bounces@alsa-project.org To: "Alexander E. Patrakov" , Lars-Peter Clausen , Arun Raghavan , "alsa-devel@alsa-project.org" Cc: Takashi Iwai , Tanu Kaskinen , David Henningsson List-Id: alsa-devel@alsa-project.org Alexander E. Patrakov wrote: > 15.06.2015 13:03, Lars-Peter Clausen wrote: >> So SNDRV_PCM_INFO_BLOCK_TRANSFER is mainly important for rewind handling >> and devices with that flag set might need additional headroom since the >> data up to one period after the pointer position has already been copied >> to the dedicated memory and hence can no longer be overwritten. > > In kernel sources, sound/pci/hda/hda_controller.c mentions > SNDRV_PCM_INFO_BLOCK_TRANSFER. However, sub-period rewinds work fine > on this driver, and the avail granularity is something like 64 bytes. HDA is a very typical PCI controller; if this flag were correct here, pretty much _every_ driver would need it. Some (older) HDA controllers have problems with position reporting (with workarounds in the drivers), but those problems are with the timing, not with the granularity. As far as I can see, snd-hda-intel should just drop this flag. Regards, Clemens