Page MenuHomeFreeBSD

LinuxKPI rcu list additions
ClosedPublic

Authored by johalun on Jun 21 2019, 4:31 PM.
Tags
None
Referenced Files
F170731699: D20719.id58869.diff
Sun, Sep 6, 6:53 AM
F170703104: D20719.id58868.diff
Sun, Sep 6, 2:52 AM
F170646893: D20719.id58865.diff
Sat, Sep 5, 7:24 PM
F170646845: D20719.id58865.diff
Sat, Sep 5, 7:23 PM
F170616344: D20719.id58868.diff
Sat, Sep 5, 4:05 PM
F170578133: D20719.id58871.diff
Sat, Sep 5, 12:01 PM
Unknown Object (File)
Sat, Sep 5, 10:40 AM
Unknown Object (File)
Sat, Sep 5, 3:17 AM

Details

Summary

Add rcu list functions and change hlist_for_each_entry_rcu to use rcu and not the non-rcu version hlist_for_each_entry.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

sys/compat/linuxkpi/common/include/linux/rculist.h
36 ↗(On Diff #58865)

<tab> after #define

39 ↗(On Diff #58865)

wrap this line like the other ones

43 ↗(On Diff #58865)

Missing ()'s

&(pos)->member

*(pos)

72 ↗(On Diff #58865)

Missing ()'s
(pos) != NULL

Add () where needed. Make sure define is followed by a tab.

johalun added inline comments.
sys/compat/linuxkpi/common/include/linux/rculist.h
39 ↗(On Diff #58865)

I follow the style of the hlist_* equivalents below... Still want me to wrap this one?

Also bump freebsd version so we can have a checkpoint from where the virtualbox drm driver builds.

This revision is now accepted and ready to land.Jun 21 2019, 6:35 PM
This revision was automatically updated to reflect the committed changes.