[dpdk-dev] [PATCH v1] ixgbe: remove vector pmd burst size restriction

2015-08-03 Thread Liang, Cunming
Hi, [...] > > Another thing, that I just thought about: > > Right now we invoke ixgbe_rxq_rearm() only at the start of > > _recv_raw_pkts_vec(). > > Before it was ok, as _recv_raw_pkts_vec() would never try to read more then > 32 > > RXDs. > > But what would happen if nb_pkts > rxq->nb_desc and

[dpdk-dev] [PATCH v1] ixgbe: remove vector pmd burst size restriction

2015-08-03 Thread Liang, Cunming
Hi, [...] > > > uint16_t > > > ixgbe_recv_scattered_pkts_vec(void *rx_queue, struct rte_mbuf **rx_pkts, > > > uint16_t nb_pkts) > > > { > > > struct ixgbe_rx_queue *rxq = rx_queue; > > > - uint8_t split_flags[RTE_IXGBE_VPMD_RX_BURST] = {0}; > > > + uint8_t

[dpdk-dev] [PATCH v1] ixgbe: remove vector pmd burst size restriction

2015-08-03 Thread Liang, Cunming
Hi, [...] > > diff --git a/drivers/net/ixgbe/ixgbe_rxtx.c b/drivers/net/ixgbe/ixgbe_rxtx.c > > index 3f808b3..dbdb761 100644 > > --- a/drivers/net/ixgbe/ixgbe_rxtx.c > > +++ b/drivers/net/ixgbe/ixgbe_rxtx.c > > @@ -4008,7 +4008,8 @@ ixgbe_set_rx_function(struct rte_eth_dev *dev) > > */ > >

[dpdk-dev] [PATCH v1] ixgbe: remove vector pmd burst size restriction

2015-07-31 Thread Cunming Liang
The patch removes the restriction of burst size on a constant 32. On receive side, the burst size floor now aligns to RTE_IXGBE_DESCS_PER_LOOP power of 2. According to this rule, the burst size less than 4 still won't receive anything. (Before this change, the burst size less than 32 can't

[dpdk-dev] [PATCH v1] ixgbe: remove vector pmd burst size restriction

2015-07-31 Thread Zoltan Kiss
On 31/07/15 12:57, Zoltan Kiss wrote: >> >> Another thing, that I just thought about: >> Right now we invoke ixgbe_rxq_rearm() only at the start of >> _recv_raw_pkts_vec(). >> Before it was ok, as _recv_raw_pkts_vec() would never try to read more >> then 32 RXDs. >> But what would happen if

[dpdk-dev] [PATCH v1] ixgbe: remove vector pmd burst size restriction

2015-07-31 Thread Zoltan Kiss
On 31/07/15 11:21, Ananyev, Konstantin wrote: > > >> -Original Message- >> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Zoltan Kiss >> Sent: Friday, July 31, 2015 11:04 AM >> To: Liang, Cunming; dev at dpdk.org >> Subject: Re: [dpdk-dev]

[dpdk-dev] [PATCH v1] ixgbe: remove vector pmd burst size restriction

2015-07-31 Thread Zoltan Kiss
On 31/07/15 09:17, Cunming Liang wrote: > The patch removes the restriction of burst size on a constant 32. > > On receive side, the burst size floor now aligns to RTE_IXGBE_DESCS_PER_LOOP > power of 2. > According to this rule, the burst size less than 4 still won't receive > anything. >

[dpdk-dev] [PATCH v1] ixgbe: remove vector pmd burst size restriction

2015-07-31 Thread Ananyev, Konstantin
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Zoltan Kiss > Sent: Friday, July 31, 2015 11:04 AM > To: Liang, Cunming; dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH v1] ixgbe: remove vector pmd burst size > restriction > >

[dpdk-dev] [PATCH v1] ixgbe: remove vector pmd burst size restriction

2015-07-31 Thread Ananyev, Konstantin
> -Original Message- > From: Liang, Cunming > Sent: Friday, July 31, 2015 9:18 AM > To: dev at dpdk.org > Cc: Ananyev, Konstantin; Liang, Cunming > Subject: [PATCH v1] ixgbe: remove vector pmd burst size restriction > > The patch removes the restriction of burst size on a constant 32. >