git: 74887e322101 - stable/14 - al_eth: Stop checking for failures from malloc/buf_ring_alloc(M_WAITOK)

[Zhenlei Huang]

The branch stable/14 has been updated by zlei:

URL: 
https://cgit.FreeBSD.org/src/commit/?id=74887e322101cbc9e531e3acd08a6eb10dd2169e

commit 74887e322101cbc9e531e3acd08a6eb10dd2169e
Author:     Zhenlei Huang <z...@freebsd.org>
AuthorDate: 2024-09-03 10:25:23 +0000
Commit:     Zhenlei Huang <z...@freebsd.org>
CommitDate: 2024-09-30 04:44:19 +0000

    al_eth: Stop checking for failures from malloc/buf_ring_alloc(M_WAITOK)
    
    MFC after:      1 week
    Differential Revision:  https://reviews.freebsd.org/D45852
    
    (cherry picked from commit 00ae9c1be02e2db6ede1abd5b787bb9a3fbd76b0)
---
 sys/dev/al_eth/al_eth.c | 18 ------------------
 1 file changed, 18 deletions(-)

diff --git a/sys/dev/al_eth/al_eth.c b/sys/dev/al_eth/al_eth.c
index baf47ac1d378..b4fbf041660b 100644
--- a/sys/dev/al_eth/al_eth.c
+++ b/sys/dev/al_eth/al_eth.c
@@ -2005,14 +2005,6 @@ al_eth_enable_msix(struct al_eth_adapter *adapter)
 
        adapter->msix_entries = malloc(msix_vecs*sizeof(*adapter->msix_entries),
            M_IFAL, M_ZERO | M_WAITOK);
-
-       if (adapter->msix_entries == NULL) {
-               device_printf_dbg(adapter->dev, "failed to allocate"
-                   " msix_entries %d\n", msix_vecs);
-               rc = ENOMEM;
-               goto exit;
-       }
-
        /* management vector (GROUP_A) @2*/
        adapter->msix_entries[AL_ETH_MGMT_IRQ_IDX].entry = 2;
        adapter->msix_entries[AL_ETH_MGMT_IRQ_IDX].vector = 0;
@@ -2300,9 +2292,6 @@ al_eth_setup_tx_resources(struct al_eth_adapter *adapter, 
int qid)
        size = sizeof(struct al_eth_tx_buffer) * tx_ring->sw_count;
 
        tx_ring->tx_buffer_info = malloc(size, M_IFAL, M_ZERO | M_WAITOK);
-       if (tx_ring->tx_buffer_info == NULL)
-               return (ENOMEM);
-
        tx_ring->descs_size = tx_ring->hw_count * sizeof(union al_udma_desc);
        q_params->size = tx_ring->hw_count;
 
@@ -2325,10 +2314,6 @@ al_eth_setup_tx_resources(struct al_eth_adapter 
*adapter, int qid)
        mtx_init(&tx_ring->br_mtx, "AlRingMtx", NULL, MTX_DEF);
        tx_ring->br = buf_ring_alloc(AL_BR_SIZE, M_DEVBUF, M_WAITOK,
            &tx_ring->br_mtx);
-       if (tx_ring->br == NULL) {
-               device_printf(dev, "Critical Failure setting up buf ring\n");
-               return (ENOMEM);
-       }
 
        /* Allocate taskqueues */
        TASK_INIT(&tx_ring->enqueue_task, 0, al_eth_start_xmit, tx_ring);
@@ -2477,9 +2462,6 @@ al_eth_setup_rx_resources(struct al_eth_adapter *adapter, 
unsigned int qid)
        size += 1;
 
        rx_ring->rx_buffer_info = malloc(size, M_IFAL, M_ZERO | M_WAITOK);
-       if (rx_ring->rx_buffer_info == NULL)
-               return (ENOMEM);
-
        rx_ring->descs_size = rx_ring->hw_count * sizeof(union al_udma_desc);
        q_params->size = rx_ring->hw_count;