On Wed, Feb 28, 2024 at 10:41:11AM +0100, Eelco Chaudron wrote:
> 
> 
> On 27 Feb 2024, at 16:36, Simon Horman wrote:
> 
> > Correct spelling errors in .rst files flagged by codespell.
> 
> Some small nits below.

Thanks Eelco,

I plan to apply this with your nit's addressed.

...

> > @@ -49,7 +49,7 @@ tc-police action, see ``man tc-police``.
> >  Configuration
> >  ~~~~~~~~~~~~~
> >
> > -There is no parameter change in ovs-ofctl command, to configue a meter and 
> > use
> > +There is no parameter change in ovs-ofctl command, to configure a meter 
> > and use
> >  it for a flow in the offload way. Usually the commands are like::
> >
> >      $ ovs-ofctl -O OpenFlow13 add-meter br0 "meter=1 pktps bands=type=drop 
> > rate=1"
> > @@ -59,7 +59,7 @@ For more details, see ``man ovs-ofctl``.
> >
> >  .. note::
> >    Each meter is mapped to one TC police action. To avovid the conflicton, 
> > the
> 
> This line also has an error; avovid, and maybe also change to ‘To avoid 
> conflicts, …”

Thanks for noticing that, and sorry for not having done so myself.

> > -  police action index of 0x10000000-0x1fffffff are resevered for the 
> > mapping.
> > +  police action index of 0x10000000-0x1fffffff are reserved for the 
> > mapping.
> >    You can check the police actions by the command ``tc action ls action 
> > police``
> >    in Linux system.
> >

...

> > @@ -506,17 +506,17 @@ options.
> >  Affinity
> >  ~~~~~~~~
> >
> > -For superior performance, DPDK pmd threads and Qemu vCPU threads needs to 
> > be
> > -affinitized accordingly.
> > +For superior performance, DPDK pmd threads and Qemu vCPU threads needs to
> 
> Not a native speaker, but does needs, here need to be need?

Yes, I think you are correct.
Sorry for messing that up.

> 
> > +have affinity set accordingly.
> >

...
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to