Page MenuHomeFreeBSD

config_intrhook: Move from TAILQ to STAILQ and padding
ClosedPublic

Authored by imp on Mar 2 2021, 12:00 AM.
Tags
None
Referenced Files
Unknown Object (File)
Feb 23 2024, 1:58 PM
Unknown Object (File)
Jan 15 2024, 11:58 PM
Unknown Object (File)
Dec 20 2023, 8:05 AM
Unknown Object (File)
Dec 14 2023, 10:19 PM
Unknown Object (File)
Dec 14 2023, 10:49 AM
Unknown Object (File)
Dec 11 2023, 7:30 PM
Unknown Object (File)
Sep 22 2023, 5:39 PM
Unknown Object (File)
Aug 31 2023, 10:41 PM
Subscribers

Details

Summary

config_intrhook doesn't need to be a two-pointer TAILQ. We rarely add/delete
from this and so those need not be optimized. Instaed, use the one-pointer
STAILQ plus a uintptr_t to be used as a flags word. This will allow these
changes to be MFC'd to 12 and 13 to fix a race in removable devices.

Diff Detail

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

Event Timeline

imp requested review of this revision.Mar 2 2021, 12:00 AM
imp created this revision.
mav added a subscriber: mav.

Should probably be merged with D29005.

This revision is now accepted and ready to land.Mar 4 2021, 2:37 PM

Maybe 'and add flags' instead of 'and padding' for the commit subject.

s/Instaed/Instead/ in the log as well.

sys/sys/kernel.h
471

Since you don't use these flags yet, I would not add these constants until D29005 and just add the ich_flags member in this commit.

sys/sys/kernel.h
471

Maybe call it ich_pad in this commit since you end up renaming it in D29005 anyway. I guess then you might say "and keep padding" or some such instead in your commit log instead of "and add flags" and similarly adjust the log to say you are leaving the second pointer as a padding placeholder for a subsequent commit.

sys/sys/kernel.h
471

I'll just call it ich_state, just like in D29005 and tweak the commit message.

In D29004#650789, @mav wrote:

Should probably be merged with D29005.

I'd prefer keeping them separate. I'd like people to have a bisection point that's just the STAILQ in case there's a bug with it...

move to generic ich_padding

This revision now requires review to proceed.Mar 8 2021, 5:17 PM
This revision is now accepted and ready to land.Mar 8 2021, 6:07 PM