lkml.org 
[lkml]   [2018]   [Oct]   [16]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH AUTOSEL 4.18 085/100] bnxt_en: Fix VNIC reservations on the PF.
    Date
    From: Michael Chan <michael.chan@broadcom.com>

    [ Upstream commit dbe80d446c859873820eedfff4abc61c71f1927b ]

    The enables bit for VNIC was set wrong when calling the HWRM_FUNC_CFG
    firmware call to reserve VNICs. This has the effect that the firmware
    will keep a large number of VNICs for the PF, and having very few for
    VFs. DPDK driver running on the VFs, which requires more VNICs, may not
    work properly as a result.

    Fixes: 674f50a5b026 ("bnxt_en: Implement new method to reserve rings.")
    Signed-off-by: Michael Chan <michael.chan@broadcom.com>
    Signed-off-by: David S. Miller <davem@davemloft.net>
    Signed-off-by: Sasha Levin <sashal@kernel.org>
    ---
    drivers/net/ethernet/broadcom/bnxt/bnxt.c | 2 +-
    1 file changed, 1 insertion(+), 1 deletion(-)

    diff --git a/drivers/net/ethernet/broadcom/bnxt/bnxt.c b/drivers/net/ethernet/broadcom/bnxt/bnxt.c
    index bcb4a6731d76..84a96c277994 100644
    --- a/drivers/net/ethernet/broadcom/bnxt/bnxt.c
    +++ b/drivers/net/ethernet/broadcom/bnxt/bnxt.c
    @@ -4650,7 +4650,7 @@ __bnxt_hwrm_reserve_pf_rings(struct bnxt *bp, struct hwrm_func_cfg_input *req,
    FUNC_CFG_REQ_ENABLES_NUM_STAT_CTXS : 0;
    enables |= ring_grps ?
    FUNC_CFG_REQ_ENABLES_NUM_HW_RING_GRPS : 0;
    - enables |= vnics ? FUNC_VF_CFG_REQ_ENABLES_NUM_VNICS : 0;
    + enables |= vnics ? FUNC_CFG_REQ_ENABLES_NUM_VNICS : 0;

    req->num_rx_rings = cpu_to_le16(rx_rings);
    req->num_hw_ring_grps = cpu_to_le16(ring_grps);
    --
    2.17.1
    \
     
     \ /
      Last update: 2018-10-16 06:31    [W:4.028 / U:0.092 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site