0x104d9069 needs LPIB quirk

[Lauri Kainulainen]

Listening to music (in this case via rhythmbox) results in stuttering sound. This happens once in a while (every 1min or so) on low load and increases when there is a lot of disk activity.

Using the driver version 2.6.32-22.201005210601 from audio-dev-ppa. Problem does not appear with the official lucid drivers in linux-backports-modules-alsa-lucid-generic (2.6.32.22.23). I'm using the dev version for nvidia hdmi-support.

my alsa-info.sh output: http://www.alsa-project.org/db/?f=b8502df177229c979aa475c0aaad0a02c2825b23

dmesg get's filled with these (not in accordance with stuttering):

[ 1761.476830] ALSA hda_intel.c:1666: azx_pcm_prepare: bufsize=0x10000, format=0x4011
[ 1761.476844] ALSA hda_codec.c:1158: hda_codec_setup_stream: NID=0x2, stream=0x5, channel=0, format=0x4011
[ 1761.493735] ALSA hda_codec.c:1158: hda_codec_setup_stream: NID=0x3, stream=0x5, channel=0, format=0x4011
[ 1761.513715] ALSA hda_intel.c:1666: azx_pcm_prepare: bufsize=0x10000, format=0x4011
[ 1761.513726] ALSA hda_codec.c:1158: hda_codec_setup_stream: NID=0x2, stream=0x5, channel=0, format=0x4011
[ 1761.533673] ALSA hda_codec.c:1158: hda_codec_setup_stream: NID=0x3, stream=0x5, channel=0, format=0x4011

Happy to debug further!

[Daniel T Chen (crimsun)]

You really shouldn't file a bug against this version from the ppa, because it changes daily (well, used to, but the compile-failure is being fixed) and thus isn't useful. Also, it's an unofficial package unsupported by the platform team.

That said, did you add the position_fix=1 quirk?

[Lauri Kainulainen (luopio)]

ok. i was just wondering where the place to file bugs against the dev-version was but if there's none, then i'll refrain myself from posting in the future :)

my line in /etc/modprobe.d/alsa-base.conf looks like this:
options snd-hda-intel model=auto position_fix=1

and it seems that the stutter is gone.

thanks!

[Daniel T Chen (crimsun)]

If you keep position_fix=1, do you still need model=auto?

[Lauri Kainulainen (luopio)]

seems to work fine without model=auto

[Daniel T Chen (crimsun)]

http://git.kernel.org/?p=linux/kernel/git/tiwai/sound-2.6.git;a=commitdiff;h=e96d3127760a2fc509bca6bf7e61e8bc61497aeb

[madbiologist (me-again)]

The abovementioned fix is now upstream in kernel 2.6.32.16.

[Steve Conklin (sconklin)]

SRU Justification

Impact: The upstream process for stable tree updates is quite similar in scope to the Ubuntu SRU process, e.g., each patch has to demonstrably fix a bug, and each patch is vetted by upstream by originating either directly from Linus' tree or in a minimally backported form of that patch.

The 2.6.32.21 upstream stable patch set is now available and contains fixes for this problem. It should be included in the Ubuntu kernel as well.

Related commits in the Lucid git repo are:

4d6ddae ALSA: hda: Use LPIB for Sony VPCS11V9E

[Martin Pitt (pitti)]

Accepted linux into lucid-proposed, the package will build now and be available in a few hours. Please test and give feedback here. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you in advance!

[Stefan Bader (smb)]

Can anybody affected by the bug please give feedback whether the current proposed kernel does resolve the issue? Thanks.

[Lauri Kainulainen (luopio)]

Did some testing:
0) checked that position_fix is disabled in /etc/modprobe.d/alsa-base
1) uninstalled linux-alsa-driver-modules-2.6.32-22-generic (from audio-dev/ppa)
2) added lucid-proposed repo & updated
3) installed the new kernel provided
(... some messing about w/ custom nvidia drivers & rebooted ...)

  $ uname -a
  Linux harmaakarhu 2.6.32-25-generic #43-Ubuntu SMP Wed Sep 1 09:46:13 UTC 2010 x86_64 GNU/Linux

  $ dmesg | grep hda
  [ 23.134431] hda_intel: position_fix set to 1 for device 104d:9069
  [ 23.353925] hda_intel: Disable MSI for Nvidia chipset
  [ 23.353927] hda_intel: position_fix set to 1 for device 104d:9069
  [ 34.884710] hda-intel: IRQ timing workaround is activated for card #0. Suggest a bigger bdl_pos_adj.

=> no sound (rhythmbox says it's playing, internal audio shows up in sound devices)

4) installed linux-backports-modules-alsa-lucid-generic & rebooted

  $ dmesg | grep hda
  [ 22.867468] hda_codec: ALC275: BIOS auto-probing.
  [ 22.868525] hda_codec: ALC275: no valid ADC found; using fallback 0x8
  [ 22.870616] hda_intel: Disable MSI for Nvidia chipset
  [ 34.256851] hda-intel: IRQ timing workaround is activated for card #1. Suggest a bigger bdl_pos_adj.
  [ 126.288578] hda-intel

=> sound works as it should

was this the correct way to test or should i do something else?

[Daniel T Chen (crimsun)]

Technically the bug report was never against Lucid's 2.6.32. However, can you verify that mixer controls are actually raised -- and that you're using the analog output -- with Lucid's proposed kernel?

[Lauri Kainulainen (luopio)]

yep. pcm and master both raised (only ones available), correct card selected (only one available) and on duplex (also tried just output).

[Launchpad Janitor (janitor)]

This bug was fixed in the package linux - 2.6.32-25.44

---------------
linux (2.6.32-25.44) lucid-proposed; urgency=low

  [ Brad Figg ]

  * SAUCE: (no-up) Modularize vesafb -- fix initialization
    - LP: #611471

  [ Stefan Bader ]

  * Revert "SAUCE: sync before umount to reduce time taken by ext4 umount"
    - LP: #543617, #585092

  [ Steve Conklin ]

  * Revert "SAUCE: tulip: Let dmfe handle davicom on non-sparc"
    - LP: #607824

  [ Tim Gardner ]

  * [Config] Added ums-cypress to udeb
    - LP: #576066

  [ Upstream Kernel Changes ]

  * Revert "PCI quirk: Disable MSI on VIA K8T890 systems"
    - LP: #607824
  * Revert "PCI quirks: disable msi on AMD rs4xx internal gfx bridges"
    - LP: #607824
  * Revert "(pre-stable) Input: psmouse - reset all types of mice before
    reconnecting"
    - LP: #607824
  * Revert "jbd: jbd-debug and jbd2-debug should be writable"
    - LP: #607824
  * Revert "ext4: Make fsync sync new parent directories in no-journal
    mode"
    - LP: #615548
  * Revert "ext4: Fix compat EXT4_IOC_ADD_GROUP"
    - LP: #615548
  * Revert "ext4: Conditionally define compat ioctl numbers"
    - LP: #615548
  * Revert "ext4: restart ext4_ext_remove_space() after transaction
    restart"
    - LP: #615548
  * Revert "ext4: Clear the EXT4_EOFBLOCKS_FL flag only when warranted"
    - LP: #615548
  * Revert "ext4: Avoid crashing on NULL ptr dereference on a filesystem
    error"
    - LP: #615548
  * Revert "ext4: Use bitops to read/modify i_flags in struct
    ext4_inode_info"
    - LP: #615548
  * Revert "ext4: Show journal_checksum option"
    - LP: #615548
  * Revert "ext4: check for a good block group before loading buddy pages"
    - LP: #615548
  * Revert "ext4: Prevent creation of files larger than RLIMIT_FSIZE using
    fallocate"
    - LP: #615548
  * Revert "ext4: Remove extraneous newlines in ext4_msg() calls"
    - LP: #615548
  * Revert "ext4: init statistics after journal recovery"
    - LP: #615548
  * Revert "ext4: clean up inode bitmaps manipulation in ext4_free_inode"
    - LP: #615548
  * Revert "ext4: Do not zero out uninitialized extents beyond i_size"
    - LP: #615548
  * Revert "ext4: don't scan/accumulate more pages than mballoc will
    allocate"
    - LP: #615548
  * Revert "ext4: stop issuing discards if not supported by device"
    - LP: #615548
  * Revert "ext4: check s_log_groups_per_flex in online resize code"
    - LP: #615548
  * Revert "ext4: fix quota accounting in case of fallocate"
    - LP: #615548
  * Revert "ext4: allow defrag (EXT4_IOC_MOVE_EXT) in 32bit compat mode"
    - LP: #615548
  * Revert "ext4: rename ext4_mb_release_desc() to ext4_mb_unload_buddy()"
    - LP: #615548
  * Revert "ext4: Remove unnecessary call to ext4_get_group_desc() in
    mballoc"
    - LP: #615548
  * Revert "ext4: fix memory leaks in error path handling of
    ext4_ext_zeroout()"
    - LP: #615548
  * Revert "ext4: check missed return value in ext4_sync_file()"
    - LP: #615548
  * Revert "ext4: Issue the discard operation *before* releasing the blocks
    to be reused"
    - LP: #615548
  * Revert "ext4: Fix buffer head leaks after calls to
    ext4_get_inode_loc()"
    - LP: #615548
  * Revert "ex...

[Mathew Hodson (mhodson)]

The linux package in lucid-proposed was released, so removing tag.

[Mathew Hodson (mhodson)]

It seems to me that the fix was released but in the linux package not the alsa-driver package.