On Thu, May 10, 2018 at 01:55:31PM -0400, Aaron Conole wrote:
> Ben Pfaff <[email protected]> writes:
> 
> > On Mon, Apr 16, 2018 at 12:16:24PM -0700, Ben Pfaff wrote:
> >> Signed-off-by: Ben Pfaff <[email protected]>
> >
> > This still needs a review.
> 
> Looks good, but I found one part slightly confusing.
> 
> >> +    These compromises wouldn't suit every site, since some might want to
> >> +    allocate more bits to the datapath, some to the egress port.
> 
> I'm having trouble with the clarity in the final clause of this
> paragraph.  Maybe reprase as:
> 
> +    These compromises wouldn't suit every site, since some deployments
> +    may need to allocate more bits to the datapath or egress port
> +    identifiers.
> 
> Otherwise,
> Acked-by: Aaron Conole <[email protected]>

Thanks!  I applied your suggestion and then pushed this to master.
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to