On Wed, 02 Dec 2015 21:43:48 +0100, Maeda wrote: > > /So the patch prevents the wrong max value reset but it's still > incomplete for the left channel volume?/ > > -> Yes, worst : it plays nothing on left channel -no loud at all ;) > > OK. > Just tried with the line 'usleep(range)' uncommented. It's better, no > loud bug (just a big POP sound on speakers on first time, nothing > after), but I have errors in output. I manage to get them with `dmesg`. > See attachment. Use mdelay(3) instead of usleep_range() call. I forgot that it's in a spinlock. It's not ideal, but good just for a test. Takashi > No error nor bug when playing two times the same sample frequency, as > expected... Playing alternatively r48000 and r96000 will output errors > each time, despite of no bug, no loud sound, just expected play. Better > then. > > That seems not very 'clean'. What do you think about it ? > > Le 02/12/2015 12:36, Takashi Iwai a écrit : > > On Wed, 02 Dec 2015 11:11:27 +0100, > > Maeda wrote: > >> Hi Takashi ! > >> > >> Sorry for the late answer. I do compile the 'test' kernel after editing > >> the file with the three lines as you said on your previous mail. > >> > >> The result is that the bug isn't occurring anymore, tested with 96KHz > >> sample : > >> > >> `aplay -fcd -d3 /dev/zero ; aplay -fS32_LE -r96000 -c2 -d3 /dev/zero ; > >> speaker-test -c2 -twav` > >> > >> _BUT_, I only have the right speaker that works, I need to change the > >> volume to restore sound on the left channel. > > So the patch prevents the wrong max value reset but it's still > > incomplete for the left channel volume? > > > > Did you try to enable the usleep_range() that was commented out in the > > patch? > > > > > > Takashi > > > >> Maeda. > >> > >> Le 21/11/2015 20:45, Takashi Iwai a écrit : > >>> On Sat, 21 Nov 2015 18:42:06 +0100, > >>> Maeda wrote: > >>>> Hi ! > >>>> > >>>> Thanks for answering me. > >>>> Yes, turning the output volume up or down as soon as it's playing at > >>>> full output : it plays at the % it should play. > >>>> > >>>> I'm OK to try the patch, but I don't find the /rme96.c/ file. Where is > >>>> it located ? I think a recompilation's needed then ? Never done that. > >>> Well, you need to learn how to compile the kernel. It's possible to > >>> compile only a module from the current running tree, too. > >>> > >>> Which distro are you using? You can ask distro guys for assistance. > >>> > >>> > >>> Takashi > >>> > >>>> Thanks for help. > >>>> > >>>> Kind regards. > >>>> > >>>> Le 20/11/2015 17:34, Takashi Iwai a écrit : > >>>>> On Tue, 10 Nov 2015 13:41:44 +0100, > >>>>> Maeda wrote: > >>>>>> Hi ! > >>>>>> > >>>>>> Someone's here ? > >>>>>> > >>>>>> Thanks. > >>>>>> > >>>>>> Le 23/10/2015 18:30, Maeda a écrit : > >>>>>>> Hi all ! > >>>>>>> > >>>>>>> I don't know if the bug is known from your team, but some people > >>>>>>> advice me to send an email to the Alsa devel list if nobody answer in > >>>>>>> the kernel's bugtracking. > >>>>>>> > >>>>>>> Here is the bug I have : > >>>>>>> https://bugzilla.kernel.org/show_bug.cgi?id=105771 > >>>>>>> > >>>>>>> What do you think about it ? > >>>>> Does the volume go down when you adjust DAC volume by mixer > >>>>> application when this happens? > >>>>> > >>>>> Through a quick glance at the driver code, the only smelling part is > >>>>> that it's calling snd_rme96_reset_dac() from > >>>>> snd_rme96_playback_setrate(). If the DAC volume adjustment really > >>>>> works, the patch below might work. Please give it a try. > >>>>> > >>>>> The patch has a code to give some delay that is commented out for > >>>>> now. The delay is found in the resume path, but I'm not sure whether > >>>>> it's mandatory. So let's try at first without the delay. If it's > >>>>> unstable, you can uncomment the line and retry. > >>>>> > >>>>> > >>>>> thanks, > >>>>> > >>>>> Takashi > >>>>> > >>>>> --- > >>>>> diff --git a/sound/pci/rme96.c b/sound/pci/rme96.c > >>>>> index 714df906249e..a9d8a66fc3da 100644 > >>>>> --- a/sound/pci/rme96.c > >>>>> +++ b/sound/pci/rme96.c > >>>>> @@ -741,6 +741,9 @@ snd_rme96_playback_setrate(struct rme96 *rme96, > >>>>> { > >>>>> /* change to/from double-speed: reset the DAC (if available) */ > >>>>> snd_rme96_reset_dac(rme96); > >>>>> + /* usleep_range(3000, 10000); */ > >>>>> + if (RME96_HAS_ANALOG_OUT(rme96)) > >>>>> + snd_rme96_apply_dac_volume(rme96); > >>>>> } else { > >>>>> writel(rme96->wcreg, rme96->iobase + RME96_IO_CONTROL_REGISTER); > >>>>> } > >>>>> > >>>>> > >>>>> > >>>>> > >> [2 <text/html; utf-8 (8bit)>] > >> > > [ 713.579803] BUG: scheduling while atomic: aplay/2752/0x00000002 > [ 713.579869] Modules linked in: joydev mousedev hidp ppdev parport_pc parport fuse vmw_vsock_vmci_transport vsock vmw_vmci cfg80211 ipt_MASQUERADE nf_nat_masquerade_ipv4 iptable_nat nf_nat_ipv4 nf_nat nf_conntrack_ipv4 nf_defrag_ipv4 xt_conntrack nf_conntrack iptable_filter nct6775 hwmon_vid bnep snd_hda_codec_hdmi intel_rapl iosf_mbi x86_pkg_temp_thermal intel_powerclamp coretemp snd_hda_codec_realtek snd_hda_codec_generic kvm_intel kvm eeepc_wmi snd_hda_intel asus_wmi iTCO_wdt crct10dif_pclmul btusb iTCO_vendor_support crc32_pclmul sparse_keymap snd_hda_codec crc32c_intel btrtl btbcm snd_hda_core btintel snd_rme96 snd_hwdep bluetooth snd_pcm aesni_intel input_leds aes_x86_64 r8169 snd_timer lrw led_class 8139too gf128mul snd mei_me glue_helper evdev ablk_helper 8139cp rfkill mac_hid cryptd mii mei > [ 713.583684] psmouse soundcore lpc_ich i2c_i801 shpchp serio_raw pcspkr wmi thermal fan battery video processor button sch_fq_codel nfsd auth_rpcgss nfs oid_registry nfs_acl lockd grace sunrpc fscache ip_tables x_tables ext4 crc16 mbcache jbd2 hid_generic usbhid hid sd_mod sr_mod cdrom atkbd libps2 ahci libahci libata xhci_pci ehci_pci scsi_mod xhci_hcd ehci_hcd usbcore usb_common i8042 serio > [ 713.586275] CPU: 6 PID: 2752 Comm: aplay Tainted: G W 4.2.6-ARCH-TESTRME96 #1 > [ 713.586353] Hardware name: ASUS All Series/H87-PRO, BIOS 2102 07/29/2014 > [ 713.586418] 0000000000000000 000000007e73a456 ffff880819b97b98 ffffffff81570f3a > [ 713.586694] 0000000000000000 ffff88083ed95200 ffff880819b97ba8 ffffffff81099e9b > [ 713.586968] ffff880819b97bf8 ffffffff815728cb 000000000000f800 ffff8807fcdfc4c0 > [ 713.587239] Call Trace: > [ 713.587307] [<ffffffff81570f3a>] dump_stack+0x4c/0x6e > [ 713.587374] [<ffffffff81099e9b>] __schedule_bug+0x4b/0x60 > [ 713.587441] [<ffffffff815728cb>] __schedule+0x89b/0xa00 > [ 713.587508] [<ffffffff81572a6e>] schedule+0x3e/0x90 > [ 713.587572] [<ffffffff81575737>] schedule_hrtimeout_range_clock.part.7+0x97/0x100 > [ 713.587651] [<ffffffff810ddde0>] ? hrtimer_init+0x110/0x110 > [ 713.587717] [<ffffffff8157572b>] ? schedule_hrtimeout_range_clock.part.7+0x8b/0x100 > [ 713.587794] [<ffffffff815757b9>] schedule_hrtimeout_range_clock+0x19/0x40 > [ 713.587861] [<ffffffff815757f3>] schedule_hrtimeout_range+0x13/0x20 > [ 713.587927] [<ffffffff8157525f>] usleep_range+0x4f/0x70 > [ 713.587994] [<ffffffffa056f0d9>] snd_rme96_playback_hw_params+0x1a9/0x330 [snd_rme96] > [ 713.588072] [<ffffffffa055bb0b>] snd_pcm_hw_params+0xbb/0x380 [snd_pcm] > [ 713.588139] [<ffffffff81175f46>] ? memdup_user+0x46/0x80 > [ 713.588207] [<ffffffffa055d1df>] snd_pcm_common_ioctl1+0x34f/0xb50 [snd_pcm] > [ 713.588275] [<ffffffffa055daf3>] snd_pcm_playback_ioctl1+0x113/0x270 [snd_pcm] > [ 713.588352] [<ffffffffa055dc78>] snd_pcm_playback_ioctl+0x28/0x40 [snd_pcm] > [ 713.588419] [<ffffffff811e2c35>] do_vfs_ioctl+0x295/0x480 > [ 713.588485] [<ffffffff811e2e99>] SyS_ioctl+0x79/0x90 > [ 713.588551] [<ffffffff815764ae>] entry_SYSCALL_64_fastpath+0x12/0x71 > _______________________________________________ > Alsa-devel mailing list > Alsa-devel@xxxxxxxxxxxxxxxx > http://mailman.alsa-project.org/mailman/listinfo/alsa-devel _______________________________________________ Alsa-devel mailing list Alsa-devel@xxxxxxxxxxxxxxxx http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
