diff options
| author | Jonathan Cameron | 2022-07-01 15:23:00 +0200 |
|---|---|---|
| committer | Michael S. Tsirkin | 2022-07-26 16:40:58 +0200 |
| commit | cb70b7e8712e17e5761a7447defdce5572cd4b80 (patch) | |
| tree | 8ce77afe08e8fa82a99643fce4943e8b0d6bb374 /include/hw/cxl | |
| parent | hw/i386/pc: Always place CXL Memory Regions after device_memory (diff) | |
| download | qemu-cb70b7e8712e17e5761a7447defdce5572cd4b80.tar.gz qemu-cb70b7e8712e17e5761a7447defdce5572cd4b80.tar.xz qemu-cb70b7e8712e17e5761a7447defdce5572cd4b80.zip | |
hw/cxl: Fix size of constant in interleave granularity function.
Whilst the interleave granularity is always small enough that this isn't
a real problem (much less than 4GiB) let's change the constant
to ULL to fix the coverity warning.
Reported-by: Peter Maydell <peter.maydell@linaro.org>
Fixes: 829de299d1 ("hw/cxl/component: Add utils for interleave parameter encoding/decoding")
Fixes: Coverity CID 1488868
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
Message-Id: <20220701132300.2264-4-Jonathan.Cameron@huawei.com>
Acked-by: Igor Mammedov <imammedo@redhat.com>
Reviewed-by: Michael S. Tsirkin <mst@redhat.com>
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
Diffstat (limited to 'include/hw/cxl')
| -rw-r--r-- | include/hw/cxl/cxl_component.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/hw/cxl/cxl_component.h b/include/hw/cxl/cxl_component.h index 70b5018156..94ec2f07d7 100644 --- a/include/hw/cxl/cxl_component.h +++ b/include/hw/cxl/cxl_component.h @@ -215,7 +215,7 @@ uint8_t cxl_interleave_granularity_enc(uint64_t gran, Error **errp); static inline hwaddr cxl_decode_ig(int ig) { - return 1 << (ig + 8); + return 1ULL << (ig + 8); } CXLComponentState *cxl_get_hb_cstate(PCIHostState *hb); |