FreeBSD Bugzilla – Attachment 171139 Details for
Bug 210098
Refine slightly search of highest priority thread on the queue(sleepq_signal).
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
Remember
[x]
|
Forgot Password
Login:
[x]
[patch]
patch loop by eliminating otherwise redundant comparisons besttd against NULL.
subr_sleepqueue.c.patch (text/plain), 716 bytes, created by
Bulat
on 2016-06-07 12:34:50 UTC
(
hide
)
Description:
patch loop by eliminating otherwise redundant comparisons besttd against NULL.
Filename:
MIME Type:
Creator:
Bulat
Created:
2016-06-07 12:34:50 UTC
Size:
716 bytes
patch
obsolete
>--- subr_sleepqueue.c.old 2016-06-07 15:07:22.906730260 +0300 >+++ subr_sleepqueue.c 2016-06-07 15:29:57.240748841 +0300 >@@ -846,12 +846,12 @@ > * been sleeping the longest since threads are always added to > * the tail of sleep queues. > */ >- besttd = NULL; >- TAILQ_FOREACH(td, &sq->sq_blocked[queue], td_slpq) { >- if (besttd == NULL || td->td_priority < besttd->td_priority) >+ besttd = td = TAILQ_FIRST(&sq->sq_blocked[queue]); >+ MPASS(besttd != NULL); >+ TAILQ_FOREACH_FROM(td, &sq->sq_blocked[queue], td_slpq) { >+ if (td->td_priority < besttd->td_priority) > besttd = td; > } >- MPASS(besttd != NULL); > thread_lock(besttd); > wakeup_swapper = sleepq_resume_thread(sq, besttd, pri); > thread_unlock(besttd);
<b>You cannot view the attachment while viewing its details because your browser does not support IFRAMEs. <a href="attachment.cgi?id=171139">View the attachment on a separate page</a>.</b>
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
Attachments on
bug 210098
: 171139