Page MenuHomeFreeBSD

sched_unpin(): assert that curthread was pinned
ClosedPublic

Authored by kib on Jun 25 2024, 8:10 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Oct 1, 2:53 AM
Unknown Object (File)
Mon, Sep 29, 1:48 AM
Unknown Object (File)
Sun, Sep 21, 1:03 PM
Unknown Object (File)
Sep 3 2025, 3:26 PM
Unknown Object (File)
Aug 24 2025, 10:35 AM
Unknown Object (File)
Jul 8 2025, 8:46 PM
Unknown Object (File)
Jun 17 2025, 4:07 AM
Unknown Object (File)
Jun 15 2025, 1:36 AM
Subscribers

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

While here, it would be nice to include sys/kassert.h.

This revision is now accepted and ready to land.Jun 25 2024, 8:21 PM

While here, it would be nice to include sys/kassert.h.

The file does not include sys/proc.h, which is vital for inline functions. sys/proc.h itself includes sys/systm.h, which includes sys/kassert.h. I am not sure that including kassert.h is useful.

olce added a subscriber: olce.

As @markj said, while here, please include sys/assert.h explicitly. It is already due because of sched_userret() above.

In D45736#1043169, @kib wrote:

While here, it would be nice to include sys/kassert.h.

The file does not include sys/proc.h, which is vital for inline functions. sys/proc.h itself includes sys/systm.h, which includes sys/kassert.h. I am not sure that including kassert.h is useful.

IMO the header should include proc.h (when _KERNEL is defined) then, but yes, it's probably not too productive to include kassert.h by itself.

This revision was automatically updated to reflect the committed changes.