Date: Sun, 25 Dec 2011 09:48:32 -0600 (CST) From: Larry Rosenman <ler@lerctr.org> To: John Baldwin <jhb@freebsd.org> Cc: Alexander Motin <mav@freebsd.org>, freebsd-current@freebsd.org Subject: Re: scheduler panic Message-ID: <alpine.BSF.2.00.1112250946550.8467@borg> In-Reply-To: <4EF4AF5C.1060000@lerctr.org> References: <4EF48065.8000802@lerctr.org> <201112230954.57591.jhb@freebsd.org> <4EF4AF5C.1060000@lerctr.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On Fri, 23 Dec 2011, Larry Rosenman wrote:
> On 12/23/2011 8:54 AM, John Baldwin wrote:
>> The sloppiest fix might be to do this:
>>
>> Index: sched_ule.c
>> ===================================================================
>>
>>
> - --- sched_ule.c (revision 228777)
>> +++ sched_ule.c (working copy) @@ -1434,7 +1434,8 @@
>> sched_priority(struct thread *td) } else { pri = SCHED_PRI_MIN; if
>> (td->td_sched->ts_ticks) - pri += SCHED_PRI_TICKS(td->td_sched);
>> + pri += min(SCHED_PRI_TICKS(td->td_sched), +
>> SCHED_PRI_RANGE); pri += SCHED_PRI_NICE(td->td_proc->p_nice);
>> KASSERT(pri >= PRI_MIN_BATCH && pri <= PRI_MAX_BATCH,
>> ("sched_priority: invalid priority %d: nice %d, "
>>
>
> I've applied this to both the host and the guest, and am recompiling
> the guest kernel (hopefully it'll stay up long enough...).
>
> I'll report back.
>
> Do y'all (FreeBSD Devs) want a PR?
>
>
I've run 2 complete buildworld/buildkernel cycles with the patch applied
in the guest, and it's made it all the way through. It wouldn't
do that without it.
Can we get this (or something else like it) applied?
Do I need to file a PR?
--
Larry Rosenman http://www.lerctr.org/~ler
Phone: +1 512-248-2683 E-Mail: ler@lerctr.org
US Mail: 430 Valona Loop, Round Rock, TX 78681-3893
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?alpine.BSF.2.00.1112250946550.8467>