Ferruh, if it save times, can you please do that.
Thanks
Ajit
On Tue, Jun 6, 2017 at 7:47 AM, Ferruh Yigit <ferruh.yi...@intel.com> wrote:
> On 6/1/2017 6:07 PM, Ajit Khaparde wrote:
> > This patch adds support to modify MTU using the set_mtu dev_op.
> > To support frames > 2k, the PMD creates an aggregator ring.
> > When a frame greater than 2k is received, it is fragmented
> > and the resulting fragments are DMA'ed to the aggregator ring.
> > Now the driver can support jumbo frames upto 9500 bytes.
> >
> > Signed-off-by: Steeven Li <steeven...@broadcom.com>
> > Signed-off-by: Ajit Khaparde <ajit.khapa...@broadcom.com>
> >
> > --
> > v1->v2: regroup related patches and incorporate other review comments
> >
> > v2->v3:
> > - rebasing to next-net tree
> > - Use net/bnxt instead of just bnxt in patch subject
>
> <...>
>
> > +int bnxt_hwrm_vnic_plcmode_cfg(struct bnxt *bp,
> > + struct bnxt_vnic_info *vnic)
> > +{
> > + int rc = 0;
> > + struct hwrm_vnic_plcmodes_cfg_input req = {.req_type = 0 };
> > + struct hwrm_vnic_plcmodes_cfg_output *resp =
> bp->hwrm_cmd_resp_addr;
> > + uint16_t size;
> > +
> > + HWRM_PREP(req, VNIC_PLCMODES_CFG, -1, resp);
> > +
> > + req.flags = rte_cpu_to_le_32(
> > +// HWRM_VNIC_PLCMODES_CFG_INPUT_FLAGS_REGULAR_PLACEMENT
> |
> > + HWRM_VNIC_PLCMODES_CFG_INPUT_
> FLAGS_JUMBO_PLACEMENT);
> > +// HWRM_VNIC_PLCMODES_CFG_INPUT_FLAGS_HDS_IPV4 |
> //TODO
> > +// HWRM_VNIC_PLCMODES_CFG_INPUT_FLAGS_HDS_IPV6);
>
> Hi Ajit,
>
> Would you mind if I remove these commented code, in this patch and other
> patches, while applying?
>
> Of course it would be better if you send the new version of the patch to
> fix them, but I believe I can do this faster. Just let me know please.
>
> Thanks,
> ferruh
>
> > + req.enables = rte_cpu_to_le_32(
> > + HWRM_VNIC_PLCMODES_CFG_INPUT_ENABLES_JUMBO_THRESH_VALID);
> > +// HWRM_VNIC_PLCMODES_CFG_INPUT_ENABLES_HDS_THRESHOLD_VALID);
> > +
> > + size = rte_pktmbuf_data_room_size(bp->rx_queues[0]->mb_pool);
> > + size -= RTE_PKTMBUF_HEADROOM;
> > +
> > + req.jumbo_thresh = rte_cpu_to_le_16(size);
> > +// req.hds_threshold = rte_cpu_to_le_16(size);
> > + req.vnic_id = rte_cpu_to_le_32(vnic->fw_vnic_id);
> > +
> > + rc = bnxt_hwrm_send_message(bp, &req, sizeof(req));
> > +
> > + HWRM_CHECK_RESULT;
> > +
> > + return rc;
> > +}
>
> <...>
>
>
