On 1 Mar 2024, at 11:00, Simon Horman wrote:
> 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. Thanks, with that Acked-by: Eelco Chaudron <echau...@redhat.com> > ... > >>> @@ -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 d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev
