Page MenuHomeFreeBSD

linuxkpi: Small tweaks to _ONCE macros
AbandonedPublic

Authored by imp on Jul 25 2024, 11:57 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Oct 19, 9:31 AM
Unknown Object (File)
Sep 12 2024, 8:12 PM
Unknown Object (File)
Sep 9 2024, 1:00 AM
Unknown Object (File)
Sep 8 2024, 12:20 AM
Unknown Object (File)
Aug 17 2024, 12:51 PM
Unknown Object (File)
Aug 10 2024, 9:38 PM
Unknown Object (File)
Aug 9 2024, 10:05 AM
Unknown Object (File)
Jul 28 2024, 4:10 AM
Subscribers

Details

Reviewers
None
Group Reviewers
linuxkpi
Summary

Provide ACCESS_ONCE and use it where it was inlined in READ_ONCE and
WRITE_ONCE. This will allow us to use the Linux KPI version of
compiler.h rather than the ancient copy of it for OpenZFS.

Sponsored by: Netflix

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 58838
Build 55725: arc lint + arc unit

Event Timeline

imp requested review of this revision.Jul 25 2024, 11:57 PM
bz added a subscriber: bz.

Please see 046d8d89ed1956944b12ec4f0580c30bc202bbfb . Seems you are reverting that.

It does undo the ACCESS_ONCE part of it, but doesn't undo the proper uintptr_t casting part. Oh, but I see the const cast is there for READ...
I can omit that bit too. OpenZFS really just reads READ_ONCE and WRITE_ONCE anyway. It
suffers from defining this stuff in too many different places :(.

So once I do all that, I can just drop this commit....