Regression in winealsa (message only?)

[Jeremy White]

Hi Paul,

I see the same behavior after Robert's change to use plug:hw:0
(and removing my settings in ~/.wine/config).

The attached patch 'fixes' the problem for me; can you try it?

However, to be honest, the real issue is that I don't really
understand what's going on.  It appears as though the
plug: devices are legitimate PCM device names, but that you
can't use them for the control interface.  If that's true,
then this patch is correct.  I just can't find any sensible
Alsa document to give me comfort that this is right.

I'd appreciate affirmation of this or a pointer to better
Alsa doco <grin>.

Cheers,

Jeremy

Paul Vriens wrote:
> On Thu, 2005-03-24 at 21:26, Jeremy White wrote:
> 
>>Hmm.  That's my patch, but it bumps into one from Robert Reif.
>>
>>Robert, is plug:hw:N the correct device naming convention for
>>PCM opens?  I've always done plughw:N.  It appears as though
>>the snd_hctl_open of plug:hw:0 is failing.
>>
>>I can't tell because of the lack of Alsa documentation which
>>is supposed to be right; if we're supposed to do plughw:0
>>instead of plug:hw:0, I'll have to fudge that code for the
>>volume control stuff.
>>
>>Paul, you can test this for yourself by adding a
>>
>>[ALSA]
>>"PlaybackDevice"="plughw"
> 
> 
> This doesn't make a difference.
> 
>>to your config  (or just try "hw" if that fails too).
> 
> 
> The error message disappears but a new is there:
> 
> fixme:wave:ALSA_WaveInit -
> 
> 
>>and seeing if everything still works (if the error below
>>goes away but a new one pops up, then I have work to do :-/).
>>
>>Cheers,
>>
>>Jeremy
>>
> 
> It doesn't seem to affect the sound though. 
> 
> Running current CVS with a +wave gives:
> 
> trace:wave:ALSA_WaveInit using waveout device "plug:hw:0"
> trace:wave:ALSA_WaveInit dev=0 id=Intel ICH name=Intel 82801DB-ICH4 subdev=0 subdev_name=subdevice #0 subdev_avail=0 subdev_num=1 stream=PLAYBACK subclass=GENERIC MIX
> trace:wave:ALSA_TraceParameters FLAGS: sampleres=false overrng=true pause=true resume=true syncstart=true batch=true block=true double=true halfd=true joint=true
> trace:wave:ALSA_TraceParameters access=(null)
> trace:wave:ALSA_TraceParameters format=(null)
> trace:wave:ALSA_TraceParameters channels_min=1, channels_min_max=10000
> trace:wave:ALSA_TraceParameters buffer_size_min=1, buffer_size_min_max=4294967294
> trace:wave:ALSA_TraceParameters rate_min=4000 rate_max=4294967295
> trace:wave:ALSA_TraceParameters buffer_time_min=1 buffer_time_max=4294967295
> trace:wave:ALSA_TraceParameters periods_min=0 periods_max=4294967295
> trace:wave:ALSA_TraceParameters period_size_min=0, period_size_min_max=4294967295
> trace:wave:ALSA_TraceParameters period_time_min=83 period_time_max=2048000
> trace:wave:ALSA_TraceParameters tick_time=1000
> trace:wave:ALSA_WaveInit Configured with dwFmts=000fffff dwSupport=00000060
> ALSA lib control.c:654:(snd_ctl_open_noupdate) Invalid CTL plug:hw:0
> trace:wave:ALSA_WaveInit using waveout device "plug:hw:1"
> trace:wave:ALSA_WaveInit using wavein device "plug:hw:0"
> trace:wave:ALSA_WaveInit dev=0 id=Intel ICH name=Intel 82801DB-ICH4 subdev=0 subdev_name=subdevice #0 subdev_avail=0 subdev_num=1 stream=CAPTURE subclass=GENERIC MIX
> trace:wave:ALSA_TraceParameters FLAGS: sampleres=false overrng=true pause=true resume=true syncstart=true batch=true block=true double=true halfd=true joint=true
> trace:wave:ALSA_TraceParameters access=(null)
> trace:wave:ALSA_TraceParameters format=(null)
> trace:wave:ALSA_TraceParameters channels_min=1, channels_min_max=10000
> trace:wave:ALSA_TraceParameters buffer_size_min=1, buffer_size_min_max=4294967294
> trace:wave:ALSA_TraceParameters rate_min=4000 rate_max=4294967295
> trace:wave:ALSA_TraceParameters buffer_time_min=1 buffer_time_max=4294967295
> trace:wave:ALSA_TraceParameters periods_min=0 periods_max=4294967295
> trace:wave:ALSA_TraceParameters period_size_min=0, period_size_min_max=4294967295
> trace:wave:ALSA_TraceParameters period_time_min=166 period_time_max=2048000
> trace:wave:ALSA_TraceParameters tick_time=1000
> trace:wave:ALSA_WaveInit Configured with dwFmts=000fffff
> trace:wave:ALSA_WaveInit using wavein device "plug:hw:1"
> trace:wave:ALSA_widMessage (0, DRVM_INIT, 00000000, 00000000, 00000000);
> trace:wave:ALSA_widMessage (0, WIDM_GETNUMDEVS, 00000000, 00000000, 00000000);
> trace:wave:ALSA_wodMessage (0, DRVM_INIT, 00000000, 00000000, 00000000);
> trace:wave:ALSA_wodMessage (0, WODM_GETNUMDEVS, 00000000, 00000000, 00000000);
> 
> The message is already present after a clean tools/wineinstall and:
> 
> regsvr32 ir50_32.dll (needed for AVI).
> 
> Cheers,
> 
> Paul.
> 
> 

-------------- next part --------------
A non-text attachment was scrubbed...
Name: stripplug.diff
Type: text/x-patch
Size: 1152 bytes
Desc: not available
Url : http://www.winehq.org/pipermail/wine-devel/attachments/20050329/2cb62347/stripplug.bin