Page MenuHomeFreeBSD

LinuxKPI: Add compat_ptr_ioctl
Needs ReviewPublic

Authored by siri_racha.ca on Jul 7 2026, 7:38 AM.
Referenced Files
F171059255: D58076.diff
Tue, Sep 8, 11:18 AM
Unknown Object (File)
Tue, Sep 8, 4:39 AM
Unknown Object (File)
Mon, Sep 7, 9:14 PM
Unknown Object (File)
Sun, Sep 6, 4:55 AM
Unknown Object (File)
Sun, Sep 6, 12:06 AM
Unknown Object (File)
Sat, Sep 5, 8:44 PM
Unknown Object (File)
Sat, Sep 5, 8:44 PM
Unknown Object (File)
Sat, Sep 5, 3:31 PM

Details

Reviewers
dumbbell
Group Reviewers
linuxkpi
Summary

Adding the compat_ptr_ioctl function for amdkfd support it's called by
kfd_chardev.c

Test Plan

Passes test build.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

dumbbell requested changes to this revision.Jul 9 2026, 9:58 PM
dumbbell added a subscriber: dumbbell.
dumbbell added inline comments.
sys/compat/linuxkpi/common/include/linux/fs.h
438โ€“441

I would swap the logic like this:

if (unlocked_ioctl == NULL)

return (-ENOIOCTLCMD);

return (file->...);

439

arg is already an unsigned long, the cast is not needed.

This revision now requires changes to proceed.Jul 9 2026, 9:58 PM
siri_racha.ca added inline comments.
sys/compat/linuxkpi/common/include/linux/fs.h
438โ€“441

Done/Amended

This revision is now accepted and ready to land.Jul 11 2026, 8:22 AM

Same trouble applying as D58075 (presumed whitespace)

siri_racha.ca edited the summary of this revision. (Show Details)
siri_racha.ca edited the test plan for this revision. (Show Details)

Fixed Whitespace issues

This revision now requires review to proceed.Sun, Sep 6, 5:04 AM

@emaste white space issues should be fixed now
cc @dumbbell