Page MenuHomeFreeBSD

LinuxKPI: Add compat_ptr_ioctl
ClosedPublic

Authored by siri_racha.ca on Jul 7 2026, 7:38 AM.
Referenced Files
Unknown Object (File)
Fri, Sep 11, 1:38 PM
Unknown Object (File)
Fri, Sep 11, 9:49 AM
Unknown Object (File)
Fri, Sep 11, 7:53 AM
Unknown Object (File)
Thu, Sep 10, 7:19 PM
Unknown Object (File)
Thu, Sep 10, 4:54 AM
Unknown Object (File)
Wed, Sep 9, 10:15 AM
Unknown Object (File)
Tue, Sep 8, 11:18 AM
Unknown Object (File)
Tue, Sep 8, 4:39 AM

Details

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 Not Applicable
Unit
Tests Not Applicable

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

This revision was not accepted when it landed; it landed in state Needs Review.Fri, Sep 11, 9:51 AM
Closed by commit rGeb9e38e37c82: linuxkpi: Add compat_ptr_ioctl (authored by siri_racha.ca, committed by emaste). · Explain Why
This revision was automatically updated to reflect the committed changes.