diff options
| author | Dave Jiang <dave.jiang@intel.com> | 2023-09-04 14:28:04 +0100 |
|---|---|---|
| committer | Michael Tokarev <mjt@tls.msk.ru> | 2023-09-21 11:31:18 +0300 |
| commit | bc63c99ef8184aaf2f6ea488e5fc9cfa391b871e (patch) | |
| tree | ee678d21ddcc219a56da6722d22eb49ebfaf12f5 /hw | |
| parent | 7b165fa164022b756c2b001d0a1525f98199d3ac (diff) | |
| download | qemu-bc63c99ef8184aaf2f6ea488e5fc9cfa391b871e.zip qemu-bc63c99ef8184aaf2f6ea488e5fc9cfa391b871e.tar.gz qemu-bc63c99ef8184aaf2f6ea488e5fc9cfa391b871e.tar.bz2 | |
hw/pci-bridge/cxl_upstream: Fix bandwidth entry base unit for SSLBIS
According to ACPI spec 6.5 5.2.28.4 System Locality Latency and Bandwidth
Information Structure, if the "Entry Base Unit" is 1024 for BW and the
matrix entry has the value of 100, the BW is 100 GB/s. So the
entry_base_unit should be changed from 1000 to 1024 given the comment notes
it's 16GB/s for .latency_bandwidth.
Fixes: 882877fc359d ("hw/pci-bridge/cxl-upstream: Add a CDAT table access DOE")
Signed-off-by: Dave Jiang <dave.jiang@intel.com>
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
Reviewed-by: Fan Ni <fan.ni@samsung.com>
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Signed-off-by: Michael Tokarev <mjt@tls.msk.ru>
Diffstat (limited to 'hw')
| -rw-r--r-- | hw/pci-bridge/cxl_upstream.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/hw/pci-bridge/cxl_upstream.c b/hw/pci-bridge/cxl_upstream.c index 9159f48..2b9cf0c 100644 --- a/hw/pci-bridge/cxl_upstream.c +++ b/hw/pci-bridge/cxl_upstream.c @@ -262,7 +262,7 @@ static int build_cdat_table(CDATSubHeader ***cdat_table, void *priv) .length = sslbis_size, }, .data_type = HMATLB_DATA_TYPE_ACCESS_BANDWIDTH, - .entry_base_unit = 1000, + .entry_base_unit = 1024, }, }; |