On Sat, Aug 04, 2018 at 02:43:24AM +0200, Stefano Brivio wrote:

On Fri, 3 Aug 2018 16:01:08 -0700
Ben Pfaff [off-list ref] wrote:

quoted

I would be very pleased if we could integrate a simple mechanism for
fairness, based for now on some simple criteria like the source port,
but thinking ahead to how we could later make it gracefully extensible
to consider more general and possibly customizable criteria.

We could change the patch so that instead of just using the vport for
round-robin queue insertion, we generalise that and use "buckets"
instead of vports, and have a set of possible functions that are called
instead of using port_no directly in ovs_dp_upcall_queue_roundrobin(),
making this configurable via netlink, per datapath.

We could implement selection based on source port or a hash on the
source 5-tuple, and the relevant bits of
ovs_dp_upcall_queue_roundrobin() would look like this:

[...]

What do you think?

I'd support that.  Thanks.