Use LIST_FOREACH_SAFE in sleepq_chains_remove_matching()
ClosedPublic
Actions

Authored by markj on Mar 25 2018, 6:44 PM.

Tags

None

Referenced Files

	F160435010: D14847.id40732.diff
	Wed, Jun 24, 9:47 AM

	Unknown Object (File)
	May 20 2026, 11:56 AM

	Unknown Object (File)
	May 20 2026, 11:56 AM

	Unknown Object (File)
	Apr 27 2026, 1:52 AM

	Unknown Object (File)
	Apr 18 2026, 8:16 PM

	Unknown Object (File)
	Apr 16 2026, 12:10 PM

	Unknown Object (File)
	Apr 10 2026, 7:58 PM

	Unknown Object (File)
	Apr 9 2026, 12:57 AM

View All Files

Subscribers

imp

Details

Reviewers

vangyzen
kib
jhb

Commits

rS331536: Use LIST_FOREACH_SAFE in sleepq_chains_remove_matching().

Summary

We may remove a sleepqueue from the hashtable in sleepq_resume_thread().

Test Plan

I hit panics in Isilon OneFS after merging r315280, and we have QUEUE_MACRO_DEBUG configured.

Diff Detail

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 15797
Build 15808: arc lint + arc unit

Event Timeline

markj created this revision.Mar 25 2018, 6:44 PM

Harbormaster completed remote builds in B15797: Diff 40732.Mar 25 2018, 6:44 PM

markj edited the test plan for this revision. (Show Details)Mar 25 2018, 6:46 PM

markj added reviewers: vangyzen, kib, jhb.

kib accepted this revision.Mar 25 2018, 7:43 PM

This revision is now accepted and ready to land.Mar 25 2018, 7:43 PM

Closed by commit rS331536: Use LIST_FOREACH_SAFE in sleepq_chains_remove_matching(). (authored by markj). · Explain WhyMar 25 2018, 8:12 PM

This revision was automatically updated to reflect the committed changes.

Herald added a subscriber: imp. · View Herald TranscriptMar 25 2018, 8:12 PM

Revision Contents
Changeset List

Path

Size

sys/

kern/

subr_sleepqueue.c

4 lines

Diff 40732

View Options

sys/kern/subr_sleepqueue.c

Use LIST_FOREACH_SAFE in sleepq_chains_remove_matching()ClosedPublicActions