On Wed, Jul 22, 2020 at 1:27 AM Aaron Conole <[email protected]> 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
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
