[alsa-devel] alsa-utils say -c is sched_idle, but actually set sched_rr (BUG?)

[alsa-devel] alsa-utils say -c is sched_idle, but actually set sched_rr (BUG?)

[tommy]

Hi,

I'm not subscribed to the list.  I was looking for uses of SCHED_IDLE in
software in general when I stubled upon this.  Although I have no personal
interest in asla development, I thought it would be nice to report it here.

alsa-project/alsa-utils commit 611249ae26f435429dbe1894e210704a723ee277 says:

    alsactl: add --nice and --sched-idle options

    The state management can run at low priority, add --nice and --sched-idle
    options to set the scheduler.


Everywhere in the code it says SCHED_IDLE *except* for the actual call to
sched_setscheduler() which instead sets the policy to SCHED_RR.  I have
looked twice to see that I don't miss anything obvious, and suspect this is
an error, or that there should be a comment that explains this oddity.

Kind regards,
Tommy
_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel

Re: [alsa-devel] alsa-utils say -c is sched_idle, but actually set sched_rr (BUG?)

[Jaroslav Kysela]

Dne 02. 12. 19 v 16:17 tommy napsal(a):
> Hi,
> 
> I'm not subscribed to the list.  I was looking for uses of SCHED_IDLE in
> software in general when I stubled upon this.  Although I have no personal
> interest in asla development, I thought it would be nice to report it here.
> 
> alsa-project/alsa-utils commit 611249ae26f435429dbe1894e210704a723ee277 says:
> 
>      alsactl: add --nice and --sched-idle options
> 
>      The state management can run at low priority, add --nice and --sched-idle
>      options to set the scheduler.
> 
> 
> Everywhere in the code it says SCHED_IDLE *except* for the actual call to
> sched_setscheduler() which instead sets the policy to SCHED_RR.  I have
> looked twice to see that I don't miss anything obvious, and suspect this is
> an error, or that there should be a comment that explains this oddity.

Thanks, fixed in 34e373d0b12d96353c49c3dcd73ec52eee11484a .

					Jaroslav

-- 
Jaroslav Kysela <perex@perex.cz>
Linux Sound Maintainer; ALSA Project; Red Hat, Inc.
_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel