On Wed, May 29, 2019 at 2:48 AM Maxime Coquelin <maxime.coque...@redhat.com> wrote: > > > > On 5/28/19 9:23 PM, Lance Richardson wrote: > > diff --git a/drivers/net/bnxt/bnxt_ethdev.c b/drivers/net/bnxt/bnxt_ethdev.c > > index 5b3932b7b..bccba445a 100644 > > --- a/drivers/net/bnxt/bnxt_ethdev.c > > +++ b/drivers/net/bnxt/bnxt_ethdev.c > > @@ -638,13 +638,73 @@ static int bnxt_scattered_rx(struct rte_eth_dev > > *eth_dev) > > > > buf_size = > > (uint16_t)(rte_pktmbuf_data_room_size(rxq->mb_pool) - > > RTE_PKTMBUF_HEADROOM); > > - if (eth_dev->data->dev_conf.rxmode.max_rx_pkt_len + > > - VLAN_TAG_SIZE * BNXT_NUM_VLANS > buf_size) > > + if (eth_dev->data->dev_conf.rxmode.max_rx_pkt_len > buf_size) > > return 1; > > } > > return 0; > > } > > It seems you forgot to squash this in patch 3.
Thanks, good catch... it appears that I did squash patch 4 from the v1 set, but I mistakenly squashed it into patch 3 instead of patch 2. I'll fix in the next spin. Lance
