]> xenbits.xensource.com Git - people/julieng/freebsd.git/commit
- Sanity check that the parent ranges given in the "ranges" property
authormarius <marius@FreeBSD.org>
Sun, 13 Sep 2015 21:59:56 +0000 (21:59 +0000)
committermarius <marius@FreeBSD.org>
Sun, 13 Sep 2015 21:59:56 +0000 (21:59 +0000)
commit1aa7c27e98396330f1d8f783b0e99eab6bd04bf4
tree92f35df7b0cbd3dc6f5b6b0bc35ca16cfca211fe
parent7b4b3f9d39aca633245fcaab12bb3aca2c3788f2
- Sanity check that the parent ranges given in the "ranges" property
  of PCI-EBus-bridges actually match the BARs as specified in and
  required by [1, p. 113 f.]. Doing so earlier would have simplified
  diagnosing a bug in QEMU/OpenBIOS getting the mapping of child
  addresses wrong, which still needs to be fixed there.
  In theory, we could try to change the BARs accordingly if we hit
  this problem. However, at least with real machines changing the
  decoding likely won't work, especially if the PCI-EBus-bridge is
  beneath an APB one. So implementing such functionality generally
  is rather pointless.
- Actually change the allocation type of EBus resources if they
  change from SYS_RES_MEMORY to SYS_RES_IOPORT when mapping them
  to PCI ranges in ebus_alloc_resource() and passing them up to
  bus_activate_resource(9). This may happen with the QEMU/OpenBIOS
  PCI-EBus-bridge but not real ones. Still, this is only cleans up
  the code and the result of resource allocation and activation is
  unchanged.
- Change the remainder of printf(9) to device_printf(9) calls and
  canonicalize their wording.

MFC after: 1 week

Peripheral Component Interconnect Input Output Controller,
Part No.: 802-7837-01, Sun Microelectronics, March 1997 [1]
sys/sparc64/ebus/ebus.c