Page MenuHomeFreeBSD

Centralize compat support for PCIOCGETCONF.
ClosedPublic

Authored by brooks on Sep 25 2018, 10:32 PM.
Tags
None
Referenced Files
F159974869: D17324.id48534.diff
Sat, Jun 20, 3:23 AM
Unknown Object (File)
Thu, Jun 18, 7:51 PM
Unknown Object (File)
Thu, Jun 18, 12:00 PM
Unknown Object (File)
Thu, Jun 18, 10:53 AM
Unknown Object (File)
May 15 2026, 5:45 PM
Unknown Object (File)
May 15 2026, 5:45 PM
Unknown Object (File)
May 15 2026, 5:19 PM
Unknown Object (File)
May 6 2026, 1:51 PM
Subscribers

Details

Summary

The pre-7 compat for both native and 32-bit code was already in
pci_user.c. Use this infrastructure to add implement 32-bit support.

Obtained from: CheriBSD

Diff Detail

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

Event Timeline

Why did you moved everything but pci bar mmap case ?

In D17324#369437, @kib wrote:

Why did you moved everything but pci bar mmap case ?

Because I already had this code (pre-dating the pci bar mmap case's existence). I'm happy to move it, but would prefer to do it in a separate change.

In D17324#369437, @kib wrote:

Why did you moved everything but pci bar mmap case ?

Because I already had this code (pre-dating the pci bar mmap case's existence). I'm happy to move it, but would prefer to do it in a separate change.

Ok. The move has more important consequence than just the code structure. Ioctl commands are specific to file descriptor, so the approach of freebsd32_ioctl() is flawed beyond repair. This is one of the reason of me asking about bar mmap.

This revision is now accepted and ready to land.Sep 26 2018, 10:22 PM

Agree with kib on the general point that all of freebsd32_ioctl is probably busted and should be moved to the actual ioctl handlers.

This revision was automatically updated to reflect the committed changes.