[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PULL 39/60] hw/pci-bridge/cxl_upstream: Drop g_malloc() failure handlin
From: |
Michael S. Tsirkin |
Subject: |
[PULL 39/60] hw/pci-bridge/cxl_upstream: Drop g_malloc() failure handling |
Date: |
Wed, 14 Feb 2024 06:15:16 -0500 |
From: Jonathan Cameron <Jonathan.Cameron@huawei.com>
As a failure of g_malloc() will result in QEMU exiting, it
won't return a NULL to check. As such, drop the incorrect handling
of such NULL returns in the cdat table building code.
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
Message-Id: <20240126120132.24248-4-Jonathan.Cameron@huawei.com>
Reviewed-by: Michael S. Tsirkin <mst@redhat.com>
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
---
hw/pci-bridge/cxl_upstream.c | 6 ------
1 file changed, 6 deletions(-)
diff --git a/hw/pci-bridge/cxl_upstream.c b/hw/pci-bridge/cxl_upstream.c
index 36737189c6..d5341b530f 100644
--- a/hw/pci-bridge/cxl_upstream.c
+++ b/hw/pci-bridge/cxl_upstream.c
@@ -228,9 +228,6 @@ static int build_cdat_table(CDATSubHeader ***cdat_table,
void *priv)
sslbis_size = sizeof(CDATSslbis) + sizeof(*sslbis_latency->sslbe) * count;
sslbis_latency = g_malloc(sslbis_size);
- if (!sslbis_latency) {
- return -ENOMEM;
- }
*sslbis_latency = (CDATSslbis) {
.sslbis_header = {
.header = {
@@ -251,9 +248,6 @@ static int build_cdat_table(CDATSubHeader ***cdat_table,
void *priv)
}
sslbis_bandwidth = g_malloc(sslbis_size);
- if (!sslbis_bandwidth) {
- return 0;
- }
*sslbis_bandwidth = (CDATSslbis) {
.sslbis_header = {
.header = {
--
MST
- [PULL 28/60] hw/block/fdc-isa: Implement relocation and enabling/disabling for TYPE_ISA_FDC, (continued)
- [PULL 28/60] hw/block/fdc-isa: Implement relocation and enabling/disabling for TYPE_ISA_FDC, Michael S. Tsirkin, 2024/02/14
- [PULL 29/60] hw/char/serial-isa: Implement relocation and enabling/disabling for TYPE_ISA_SERIAL, Michael S. Tsirkin, 2024/02/14
- [PULL 30/60] hw/char/parallel-isa: Implement relocation and enabling/disabling for TYPE_ISA_PARALLEL, Michael S. Tsirkin, 2024/02/14
- [PULL 31/60] hw/ppc/pegasos2: Let pegasos2 machine configure SuperI/O functions, Michael S. Tsirkin, 2024/02/14
- [PULL 32/60] hw/isa/vt82c686: Implement relocation and toggling of SuperI/O functions, Michael S. Tsirkin, 2024/02/14
- [PULL 33/60] vhost-user.rst: Fix vring address description, Michael S. Tsirkin, 2024/02/14
- [PULL 35/60] virtio_iommu: Clear IOMMUPciBus pointer cache when system reset, Michael S. Tsirkin, 2024/02/14
- [PULL 36/60] smmu: Clear SMMUPciBus pointer cache when system reset, Michael S. Tsirkin, 2024/02/14
- [PULL 34/60] MAINTAINERS: Drop myself as VT-d maintainers, Michael S. Tsirkin, 2024/02/14
- [PULL 40/60] cxl/cdat: Fix header sum value in CDAT checksum, Michael S. Tsirkin, 2024/02/14
- [PULL 39/60] hw/pci-bridge/cxl_upstream: Drop g_malloc() failure handling,
Michael S. Tsirkin <=
- [PULL 41/60] hw/cxl/mbox: Remove dead code, Michael S. Tsirkin, 2024/02/14
- [PULL 42/60] hw/cxl/device: read from register values in mdev_reg_read(), Michael S. Tsirkin, 2024/02/14
- [PULL 46/60] tests/acpi: Allow update of DSDT.cxl, Michael S. Tsirkin, 2024/02/14
- [PULL 47/60] hw/i386: Fix _STA return value for ACPI0017, Michael S. Tsirkin, 2024/02/14
- [PULL 37/60] cxl/cdat: Handle cdat table build errors, Michael S. Tsirkin, 2024/02/14
- [PULL 38/60] hw/mem/cxl_type3: Drop handling of failure of g_malloc0() and g_malloc(), Michael S. Tsirkin, 2024/02/14
- [PULL 43/60] hw/cxl: Pass CXLComponentState to cache_mem_ops, Michael S. Tsirkin, 2024/02/14
- [PULL 44/60] hw/cxl: Pass NULL for a NULL MemoryRegionOps, Michael S. Tsirkin, 2024/02/14
- [PULL 45/60] hw/mem/cxl_type3: Fix potential divide by zero reported by coverity, Michael S. Tsirkin, 2024/02/14
- [PULL 48/60] tests/acpi: Update DSDT.cxl to reflect change _STA return value., Michael S. Tsirkin, 2024/02/14