Page MenuHomeFreeBSD

Fix pci_host_generic IOPORT
ClosedPublic

Authored by bz on May 12 2016, 6:18 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Sep 9, 2:12 AM
Unknown Object (File)
Sun, Sep 8, 6:47 PM
Unknown Object (File)
Sat, Sep 7, 7:30 PM
Unknown Object (File)
Aug 12 2024, 3:20 PM
Unknown Object (File)
Jul 28 2024, 1:14 PM
Unknown Object (File)
Jul 5 2024, 7:43 PM
Unknown Object (File)
Apr 25 2024, 11:48 PM
Unknown Object (File)
Apr 25 2024, 11:48 PM
Subscribers

Details

Summary

When using IOPORT with pci_host_generic we are missing
setting the rman_end() which leads to end being before
start and thus a signed extended very large number of
size later on, which kva_alloc() will fail upon and we
will panic. Add the missing call.

Debugged with: andrew
Reviewed by:
Sponsored by: DARPA/AFRL
Found: while using virtio with gem5

Test Plan

Botting gem5 x86, armv7, arm64 with virtio/pci disks.

Diff Detail

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

Event Timeline

bz retitled this revision from to Fix pci_host_generic IOPORT.
bz updated this object.
bz edited the test plan for this revision. (Show Details)
bz added reviewers: andrew, br.
bz set the repository for this revision to rS FreeBSD src repository - subversion.
andrew edited edge metadata.

We might also need to update the virtual address, but that doesn't seem to have been an issue so far.

This revision is now accepted and ready to land.May 12 2016, 8:26 PM
br edited edge metadata.
This revision was automatically updated to reflect the committed changes.