I've submitted PR http://www.freebsd.org/cgi/query-pr.cgi?pr=157657.

Dmitry

On Thu, Jun 2, 2011 at 6:34 PM, John Baldwin <[email protected]> wrote:
> On Wednesday, June 01, 2011 12:42:42 pm Dmitry Krivenok wrote:
>> Hello Hackers,
>> I think I found a bug in ksched_setscheduler() function.
>>
>> 209                         rtp.prio = 
>> p4prio_to_rtpprio(param->sched_priority);
>>
>> Shouldn't we use p4prio_to_tsprio instead of p4prio_to_rtpprio at the line 
>> 209?
>> This macro is defined but never used in kernel code:
>>
>> $ grep -r 'p4prio_to_tsprio' /usr/src/sys/
>> /usr/src/sys/kern/ksched.c:#define p4prio_to_tsprio(P)
>> ((PRI_MAX_TIMESHARE - PRI_MIN_TIMESHARE) - (P))
>> $
>>
>> Is it a real bug or just my misunderstanding of something?
>
> I think it is a real bug.  Can you come up with a test case to show it?
>
> --
> John Baldwin
>



-- 
Sincerely yours, Dmitry V. Krivenok
e-mail: [email protected]
skype: krivenok_dmitry
jabber: [email protected]
icq: 242-526-443
_______________________________________________
[email protected] mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-hackers
To unsubscribe, send any mail to "[email protected]"

Reply via email to