netdev - Re: [v7, net-next 09/10] bng_en: Register default VNIC

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <aa898848-810e-4a6f-9fcf-0289e620229d@oracle.com>
Date: Thu, 18 Sep 2025 01:48:29 +0530
From: ALOK TIWARI <alok.a.tiwari@...cle.com>
To: Bhargava Marreddy <bhargava.marreddy@...adcom.com>, davem@...emloft.net,
        edumazet@...gle.com, kuba@...nel.org, pabeni@...hat.com,
        andrew+netdev@...n.ch, horms@...nel.org
Cc: netdev@...r.kernel.org, linux-kernel@...r.kernel.org,
        michael.chan@...adcom.com, pavan.chebbi@...adcom.com,
        vsrama-krishna.nemani@...adcom.com, vikas.gupta@...adcom.com,
        Rajashekar Hudumula <rajashekar.hudumula@...adcom.com>
Subject: Re: [v7, net-next 09/10] bng_en: Register default VNIC



On 9/12/2025 1:05 AM, Bhargava Marreddy wrote:
> +int bnge_hwrm_vnic_cfg(struct bnge_net *bn, struct bnge_vnic_info *vnic)
> +{
> +	struct bnge_rx_ring_info *rxr = &bn->rx_ring[0];
> +	struct hwrm_vnic_cfg_input *req;
> +	struct bnge_dev *bd = bn->bd;
> +	int rc;
> +
> +	rc = bnge_hwrm_req_init(bd, req, HWRM_VNIC_CFG);
> +	if (rc)
> +		return rc;
> +
> +	req->default_rx_ring_id =
> +		cpu_to_le16(rxr->rx_ring_struct.fw_ring_id);
> +	req->default_cmpl_ring_id =
> +		cpu_to_le16(bnge_cp_ring_for_rx(rxr));
> +	req->enables =
> +		cpu_to_le32(VNIC_CFG_REQ_ENABLES_DEFAULT_RX_RING_ID |
> +			    VNIC_CFG_REQ_ENABLES_DEFAULT_CMPL_RING_ID);
> +	vnic->mru = bd->netdev->mtu + ETH_HLEN + VLAN_HLEN;

nit: does "struct bnge_dev" hold a netdev ?
if not should be bn->netdev->mtu.

> +	req->mru = cpu_to_le16(vnic->mru);
> +
> +	req->vnic_id = cpu_to_le16(vnic->fw_vnic_id);
> +
> +	if (bd->flags & BNGE_EN_STRIP_VLAN)
> +		req->flags |= cpu_to_le32(VNIC_CFG_REQ_FLAGS_VLAN_STRIP_MODE);
> +	if (vnic->vnic_id == BNGE_VNIC_DEFAULT && bnge_aux_registered(bd))
> +		req->flags |= cpu_to_le32(BNGE_VNIC_CFG_ROCE_DUAL_MODE);
> +
> +	return bnge_hwrm_req_send(bd, req);
> +}


Thanks,
Alok