staging: fsl-dpaa2/eth: Fix SGT allocation
authorIoana Radulescu <ruxandra.radulescu@nxp.com>
Fri, 23 Mar 2018 13:44:12 +0000 (08:44 -0500)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 23 Mar 2018 15:01:57 +0000 (16:01 +0100)
We mistakenly allocate space for too many entries in the
scatter-gather table of multi buffer egress frames.

While it doesn't have a negative impact from a functional
point of view, it wastes resources so fix it.

Signed-off-by: Ioana Radulescu <ruxandra.radulescu@nxp.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/fsl-dpaa2/ethernet/dpaa2-eth.c

index 9994922..14bd8fe 100644 (file)
@@ -373,7 +373,7 @@ static int build_sg_fd(struct dpaa2_eth_priv *priv,
 
        /* Prepare the HW SGT structure */
        sgt_buf_size = priv->tx_data_offset +
-                      sizeof(struct dpaa2_sg_entry) * (1 + num_dma_bufs);
+                      sizeof(struct dpaa2_sg_entry) *  num_dma_bufs;
        sgt_buf = netdev_alloc_frag(sgt_buf_size + DPAA2_ETH_TX_BUF_ALIGN);
        if (unlikely(!sgt_buf)) {
                err = -ENOMEM;