derek/gem5
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

dev: Adding support for large BAR

Browse Source 
During PCI setup, this patch checks if a Base Address Register (BAR) is
used as a large BAR (64 bits rather than 32), and return proper address
range. The order which updates are done is decided by kernel, so this
patch implements both cases (writing lower or upper bits first).

Bit 2 in a BAR indicates a 64-bit decoder (10X to be more exact, 11X is
reserved).

The addresses in BARAddrs are full addresses and are set to zero for BAR
providing upper 32 bits to avoid conflicts in addr ranges reported.

Change-Id: I93303d36ac83dab9ed6837c81e77c9dfb778f409
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/22082
Reviewed-by: Jason Lowe-Power <jason@lowepower.com>
Maintainer: Jason Lowe-Power <jason@lowepower.com>
Tested-by: kokoro <noreply+kokoro@google.com>
This commit is contained in:
Pouya Fotouhi
2019-10-22 17:29:57 -05:00
committed by Pouya Fotouhi
parent fca416d154
commit e0040fe8fe
2 changed files with 33 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

27
src/dev/pci/device.cc
View File

@@ -380,9 +380,30 @@ PciDevice::writeConfig(PacketPtr pkt)
// does it mean something special to write 0 to a BAR?
he_new_bar &= ~bar_mask;
if (he_new_bar) {
BARAddrs[barnum] = BAR_IO_SPACE(he_old_bar) ?
hostInterface.pioAddr(he_new_bar) :
hostInterface.memAddr(he_new_bar);
if (isLargeBAR(barnum)) {
if (BAR_IO_SPACE(he_old_bar))
warn("IO BARs can't be set as large BAR");
uint64_t he_large_bar =
letoh(config.baseAddr[barnum + 1]);
he_large_bar = he_large_bar << 32;
he_large_bar += he_new_bar;
BARAddrs[barnum] =
hostInterface.memAddr(he_large_bar);
} else if (isLargeBAR(barnum - 1)) {
BARAddrs[barnum] = 0;
uint64_t he_large_bar = he_new_bar;
he_large_bar = he_large_bar << 32;
// We need to apply mask to lower bits
he_large_bar +=
letoh(config.baseAddr[barnum - 1]
& ~bar_mask);
BARAddrs[barnum - 1] =
hostInterface.memAddr(he_large_bar);
} else {
BARAddrs[barnum] = BAR_IO_SPACE(he_old_bar) ?
hostInterface.pioAddr(he_new_bar) :
hostInterface.memAddr(he_new_bar);
}
pioPort.sendRangeChange();
}
}

9
src/dev/pci/device.hh
View File

@@ -115,6 +115,15 @@ class PciDevice : public DmaDevice
/** Whether the BARs are really hardwired legacy IO locations. */
bool legacyIO[6];
/**
* Does the given BAR represent 32 lower bits of a 64-bit address?
*/
bool
isLargeBAR(int bar) const
{
return bits(config.baseAddr[bar], 2, 1) == 0x2;
}
/**
* Does the given address lie within the space mapped by the given
* base address register?