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, Jul 15, 4:27 PM

	Unknown Object (File)
	Tue, Jul 15, 3:06 AM

	Unknown Object (File)
	Sat, Jul 12, 3:23 PM

	Unknown Object (File)
	Fri, Jul 11, 12:52 AM

	Unknown Object (File)
	Thu, Jul 10, 2:36 PM

	Unknown Object (File)
	Sun, Jul 6, 4:25 PM

	Unknown Object (File)
	Fri, Jul 4, 2:36 AM

	Unknown Object (File)
	Fri, Jul 4, 12:54 AM

View All 59 Files

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 61773
Build 58657: 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/

2 lines

2 lines

Diff 149361

sys/net/dummymbuf.c

Loading...

sys/net/if_ethersubr.c

Loading...