Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 31 Dec 2007 12:19:07 -0500
From:      John Baldwin <jhb@FreeBSD.org>
To:        Randall Stewart <rrs@FreeBSD.org>
Cc:        cvs-src@FreeBSD.org, src-committers@FreeBSD.org, cvs-all@FreeBSD.org
Subject:   Re: cvs commit: src/sys/netinet sctp_bsd_addr.c
Message-ID:  <200712311219.08286.jhb@freebsd.org>
In-Reply-To: <200712291507.lBTF7sTA069010@repoman.freebsd.org>
References:  <200712291507.lBTF7sTA069010@repoman.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Saturday 29 December 2007 10:07:54 am Randall Stewart wrote:
> rrs         2007-12-29 15:07:54 UTC
> 
>   FreeBSD src repository
> 
>   Modified files:        (Branch: RELENG_7_0)
>     sys/netinet          sctp_bsd_addr.c 
>   Log:
>   - sctp-iterator should run at PI_NET priority ...not 0.
>   Approved by:    re@freebsd.org(gnn)
>   
>   Revision  Changes    Path
>   1.14.4.1  +1 -1      src/sys/netinet/sctp_bsd_addr.c

The more correct fix though is to do a 'sched_prio()' at the start of the 
thread's main loop to set the priority and then not adjust it via msleep().  
Kernel threads really should never pass a priority to msleep() but always '0' 
(which means "don't change my priority").

-- 
John Baldwin



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200712311219.08286.jhb>