Re: CVS commit: src/sys

[yamt%mwd.biglobe.ne.jp@localhost (YAMAMOTO Takashi)]

> YAMAMOTO Takashi wrote:
> >>  int
> >>  linux_sys_sched_setparam(struct lwp *l, const struct 
> >> linux_sys_sched_setparam_args *uap, register_t *retval)
> >>  {
> > 
> >> +  error = sched_linux2native(LINUX_SCHED_OTHER, &lp, &policy, &sp);
> >> +  if (error)
> >> +          goto out;
> >>  
> >> -  return 0;
> >> +  error = do_sched_setparam(SCARG(uap, pid), 0, policy, &sp);
> >> +  if (error)
> >> +          goto out;
> >> +
> >> + out:
> >> +  return error;
> > 
> > you should use the current policy instead of assuming SCHED_OTHER.
> 
> Heh I thought I changed that... anyway, can't we just pass SCHED_NONE
> for the FreeBSD/Linux setparam cases?
> 
> -e.

i think policy is necessary to convert priority.

YAMAMOTO Takashi