Page MenuHomeFreeBSD

LinuxKPI: tasklet add (*callback) support
ClosedPublic

Authored by bz on Oct 30 2022, 5:32 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, May 18, 6:07 AM
Unknown Object (File)
Jan 28 2024, 6:52 AM
Unknown Object (File)
Dec 20 2023, 8:35 AM
Unknown Object (File)
Dec 12 2023, 8:54 AM
Unknown Object (File)
Dec 7 2023, 9:09 PM
Unknown Object (File)
Sep 19 2023, 3:07 AM
Unknown Object (File)
Sep 18 2023, 3:59 PM
Unknown Object (File)
Aug 30 2023, 8:10 AM
Subscribers

Details

Summary

In addition to (*func) tasklet also seems to have a (*callback) with
a different argument. Add support for this and add tasklet_setup()
as well for support in more drivers.

The from_tasklet() definition is duplicated in the tree; hide it there
under #ifndef to avoid a re-definition. People should generally add
LinuxKPI bits to linuxkpi rather than private files if they also rely
on other LinuxKPI bits.

X-MFC: DO NOT MFC, space allocated by drivers not us.

Diff Detail

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

Event Timeline

bz requested review of this revision.Oct 30 2022, 5:32 PM

It's been months since I had done this change and I wonder if we could do it without the bool and just check if func is != NULL then use func and otherwise use callback; that would also make it MFCable I think. Someone might have to take a peek at Linux and check if there is the invariant of only func or callback that can be set at one time and whether one has to be set?

if there is the invariant of only func or callback that can be set at one time

This does not appear to be the case.

if there is the invariant of only func or callback that can be set at one time

This does not appear to be the case.

Thanks Ed! So we need the bool and we cannot MFC (correct me if I misunderstand).

Anyone up for the review?

This revision is now accepted and ready to land.Nov 7 2022, 3:33 PM
This revision was automatically updated to reflect the committed changes.