Differential D48490

sys/net: fix various signature mismatch needs for KCFI
ClosedPublic
Actions

Authored by aokblast on Jan 16 2025, 5:51 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Tue, Apr 7, 4:37 AM

	Unknown Object (File)
	Tue, Apr 7, 1:28 AM

	Unknown Object (File)
	Sat, Apr 4, 2:02 AM

	Unknown Object (File)
	Fri, Apr 3, 9:47 PM

	Unknown Object (File)
	Thu, Apr 2, 10:45 PM

	Unknown Object (File)
	Thu, Apr 2, 1:19 PM

	Unknown Object (File)
	Sun, Mar 29, 12:41 AM

	Unknown Object (File)
	Sat, Mar 28, 11:41 AM

Subscribers

Details

Reviewers

zlei
lwhsu
glebius

Commits

rG8be18c2d5a63: sys/net: fix several sysinit_cfunc_t signature mismatches

Summary

KCFI needs signature exactly match (also const). It is needed to make the function signature be comply with net_sysinit interface

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 61774
Build 58658: arc lint + arc unit

Event Timeline

aokblast created this revision.Jan 16 2025, 5:51 PM

Herald added subscribers: glebius, melifaro, ae, imp. · View Herald TranscriptJan 16 2025, 5:51 PM

aokblast requested review of this revision.Jan 16 2025, 5:51 PM

Harbormaster completed remote builds in B61773: Diff 149361.Jan 16 2025, 5:51 PM

Also for uninit

Harbormaster completed remote builds in B61774: Diff 149363.Jan 16 2025, 5:54 PM

aokblast edited the summary of this revision. (Show Details)Jan 16 2025, 5:56 PM

aokblast added reviewers: zlei, lwhsu.

glebius accepted this revision.Jan 16 2025, 6:00 PM

This revision is now accepted and ready to land.Jan 16 2025, 6:00 PM

If you want me to commit it, please tell what name+email should I use for the commit author. Or point me to your github repo and I will pull from there.

Great! Thank you.

Here is the patch: https://github.com/aokblast/freebsd-src/commit/bcad8ea91b19ee21d23140bd52d6114ad2d56029

Closed by commit rG8be18c2d5a63: sys/net: fix several sysinit_cfunc_t signature mismatches (authored by aokblast, committed by glebius). · Explain WhyJan 16 2025, 6:10 PM

This revision was automatically updated to reflect the committed changes.

glebius added a commit: rG8be18c2d5a63: sys/net: fix several sysinit_cfunc_t signature mismatches.

Revision Contents
Changeset List

Path

Size

sys/

net/

4 lines

4 lines

Diff 149363

sys/net/dummymbuf.c

Loading...

sys/net/if_ethersubr.c

Loading...