On Wed, Jul 22, 2020 at 1:27 AM Aaron Conole <acon...@redhat.com> wrote: > To check: > > ip netns add left > ip netns add right > ip link add center-left type veth peer name left0 > ip link add center-right type veth peer name right0 > ip link set left0 netns left > ip link set right0 netns right
Nit: ip can set the peer netns upon veth creation: ip link add center-left type veth peer name left0 netns left ip link add center-right type veth peer name right0 netns right > +static uint32_t > +dpif_handler_port_idx_max(const struct dpif_handler *handler, > + struct dpif_channel **out_chn) > +{ > + uint32_t max = 0; > + size_t i; > + > + for (i = 0; i < handler->n_channels; ++i) { Any reason for using the prefix operator everywhere? They should be equal nowadays. > -- > 2.25.4 > Good job! Did you have the chance to run the same tests I did in 69c51582ff78? I'm curious to see how much it improves. Bye, -- Matteo Croce perl -e 'for($t=0;;$t++){print chr($t*($t>>8|$t>>13)&255)}' |aplay _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev