Page MenuHomeFreeBSD

linux(4): Deduplicate unimpl/dummy syscall handlers
ClosedPublic

Authored by cem on Nov 5 2020, 6:44 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Nov 2, 12:09 AM
Unknown Object (File)
Oct 5 2024, 9:02 PM
Unknown Object (File)
Sep 17 2024, 8:08 PM
Unknown Object (File)
Sep 12 2024, 2:34 AM
Unknown Object (File)
Sep 12 2024, 1:45 AM
Unknown Object (File)
Sep 8 2024, 5:25 PM
Unknown Object (File)
Sep 6 2024, 8:12 AM
Unknown Object (File)
Sep 2 2024, 2:56 AM
Subscribers

Details

Summary

No functional change.

Test Plan

Tinderbox amd64 arm64 i386 passes.

Diff Detail

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

Event Timeline

cem requested review of this revision.Nov 5 2020, 6:44 AM

I like it. Some of the per-architecture DUMMY() should probably get moved to the MI file too - statx for example - but that's better done later on.

This revision is now accepted and ready to land.Nov 5 2020, 11:10 AM

I like it. Some of the per-architecture DUMMY() should probably get moved to the MI file too - statx for example - but that's better done later on.

Thanks! Yeah, I wasn't sure to what extent each of the syscalls.master files was accurate, but that can be updated relatively easily after this big change.

Thanks. I didn't check each individual entry but am very happy to see this.

All of the linux 4.X+ entries likely belong in the common file but agreed that can be a subsequent change

sys/arm64/linux/linux_dummy_machdep.c
4 ↗(On Diff #79198)

This is probably not correct for arm64 now

Thanks!

sys/arm64/linux/linux_dummy_machdep.c
4 ↗(On Diff #79198)

Just keeping the existing copyright header (this is an svn mv from linux_dummy.c) for now: https://github.com/freebsd/freebsd/blob/master/sys/arm64/linux/linux_dummy.c#L4

sys/arm64/linux/linux_dummy_machdep.c
4 ↗(On Diff #79198)

Right - I copied that statement when I copied amd64's dummy file to arm64, but everything that was copied has now been moved to the common file, so all that's left here now is what I added and your comment, which is probably the most copyrightable piece :)

Anyway doesn't really matter.

This revision was automatically updated to reflect the committed changes.