On Fri, 2015-12-11 at 05:30 +0100, Kamil Rytarowski wrote:
> Patch from pkgsrc by Jonathan Perkin (Joyent).
> ---
The two patches should be separated, since they are unrelated.
> Â src/modules/module-solaris.c | 12 +++++++++---
>  src/utils/padsp.c            |  2 ++
> Â 2 files changed, 11 insertions(+), 3 deletions(-)
>
> diff --git a/src/modules/module-solaris.c b/src/modules/module-
> solaris.c
> index c79918a..2fa0bff 100644
> --- a/src/modules/module-solaris.c
> +++ b/src/modules/module-solaris.c
> @@ -412,10 +412,12 @@ static int sink_process_msg(pa_msgobject *o,
> int code, void *data, int64_t offse
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â pa_smoother_resume(u->smoother,
> pa_rtclock_now(), true);
> Â
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â if (!u->source || u->source_suspended) {
> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â bool mute;
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â if (unsuspend(u) < 0)
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â return -1;
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â u->sink->get_volume(u->sink);
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â u->sink->get_mute(u->sink);
> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â if (u->sink->get_mute(u->sink, &mute) >=
> 0)
> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â pa_sink_set_mute(u->sink, mute,
> false);
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â }
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â u->sink_suspended = false;
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â }
> @@ -1033,8 +1035,12 @@ int pa__init(pa_module *m) {
> Â
> Â Â Â Â Â Â Â Â Â if (sink_new_data.muted_is_set)
> Â Â Â Â Â Â Â Â Â Â Â Â Â u->sink->set_mute(u->sink);
> -Â Â Â Â Â Â Â Â else
> -Â Â Â Â Â Â Â Â Â Â Â Â u->sink->get_mute(u->sink);
> +Â Â Â Â Â Â Â Â else {
> +Â Â Â Â Â Â Â Â Â Â Â Â bool mute;
> +
> +Â Â Â Â Â Â Â Â Â Â Â Â if (u->sink->get_mute(u->sink, &mute) >= 0)
> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â pa_sink_set_mute(u->sink, mute, false);
> +Â Â Â Â Â Â Â Â }
> Â
> Â Â Â Â Â Â Â Â Â pa_sink_put(u->sink);
> Â Â Â Â Â }
> diff --git a/src/utils/padsp.c b/src/utils/padsp.c
> index 5e336bb..2eff0f0 100644
> --- a/src/utils/padsp.c
> +++ b/src/utils/padsp.c
> @@ -2278,6 +2278,7 @@ static int dsp_ioctl(fd_info *i, unsigned long
> request, void*argp, int *_errno)
> Â Â Â Â Â Â Â Â Â Â Â Â Â break;
> Â Â Â Â Â Â Â Â Â }
> Â
> +#ifndef __sun
> Â Â Â Â Â Â Â Â Â case SOUND_PCM_READ_RATE:
> Â Â Â Â Â Â Â Â Â Â Â Â Â debug(DEBUG_LEVEL_NORMAL, __FILE__":
> SOUND_PCM_READ_RATE\n");
> Â
> @@ -2301,6 +2302,7 @@ static int dsp_ioctl(fd_info *i, unsigned long
> request, void*argp, int *_errno)
> Â Â Â Â Â Â Â Â Â Â Â Â Â *(int*) argp = pa_sample_size(&i->sample_spec)*8;
> Â Â Â Â Â Â Â Â Â Â Â Â Â pa_threaded_mainloop_unlock(i->mainloop);
> Â Â Â Â Â Â Â Â Â Â Â Â Â break;
> +#endif
> Â
> Â Â Â Â Â Â Â Â Â case SNDCTL_DSP_GETOPTR: {
> Â Â Â Â Â Â Â Â Â Â Â Â Â count_info *info;
Is this really not supported on Solaris? Do you have a bug report or
some other reference for this (which would ideally go into the commit
message after splitting this off separately).
-- Arun
