From mboxrd@z Thu Jan  1 00:00:00 1970
From: Luca Boccassi <bluca@debian.org>
Subject: Re: [dpdk-stable] [PATCH] net/bonding: ensure fairness
	among slaves
Date: Wed, 19 Sep 2018 17:06:59 +0100
Message-ID: <1537373219.10481.26.camel@debian.org>
References: <20180919154825.5183-1-3chas3@gmail.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Cc: declan.doherty@intel.com, matan@mellanox.com, ehkinzie@gmail.com, Chas
 Williams <chas3@att.com>, stable@dpdk.org
To: Chas Williams <3chas3@gmail.com>, dev@dpdk.org
Return-path: <dev-bounces@dpdk.org>
In-Reply-To: <20180919154825.5183-1-3chas3@gmail.com>
List-Id: DPDK patches and discussions <dev.dpdk.org>
List-Unsubscribe: <https://mails.dpdk.org/options/dev>,
 <mailto:dev-request@dpdk.org?subject=unsubscribe>
List-Archive: <http://mails.dpdk.org/archives/dev/>
List-Post: <mailto:dev@dpdk.org>
List-Help: <mailto:dev-request@dpdk.org?subject=help>
List-Subscribe: <https://mails.dpdk.org/listinfo/dev>,
 <mailto:dev-request@dpdk.org?subject=subscribe>
Errors-To: dev-bounces@dpdk.org
Sender: "dev" <dev-bounces@dpdk.org>

On Wed, 2018-09-19 at 11:48 -0400, Chas Williams wrote:
> From: Chas Williams <chas3@att.com>
>=20
> Some PMDs, especially ones with vector receives, require a minimum
> number
> of receive buffers in order to receive any packets.=C2=A0=C2=A0If the fir=
st
> slave
> read leaves less than this number available, a read from the next
> slave
> may return 0 implying that the slave doesn't have any packets which
> results in skipping over that slave as the next active slave.
>=20
> To fix this, implement round robin for the slaves during receive that
> is only advanced to the next slave at the end of each receive burst.
> This should also provide some additional fairness in processing in
> bond_ethdev_rx_burst as well.
>=20
> Fixes: 2efb58cbab6e ("bond: new link bonding library")
> Cc: stable@dpdk.org
>=20
> Signed-off-by: Chas Williams <chas3@att.com>
> ---
> =C2=A0drivers/net/bonding/rte_eth_bond_pmd.c | 50 ++++++++++++++++++++++
> ------------
> =C2=A01 file changed, 32 insertions(+), 18 deletions(-)

Acked-by: Luca Boccassi <bluca@debian.org>

--=20
Kind regards,
Luca Boccassi