[ovs-dev] [PATCH] print the IP TOS for flows with any IP network types
Romain Lenglet
romain.lenglet at berabera.info
Fri Nov 5 01:52:16 UTC 2010
On 11/5/10 10:23, Jesse Gross wrote:
> On Thu, Nov 4, 2010 at 5:30 PM,<romain.lenglet at berabera.info> wrote:
>> From: Romain Lenglet<romain.lenglet at berabera.info>
>>
>> Print the NW_TOS field for all flows matching the IP Ethernet packet
>> type, for all NW_PROTO types.
>
> One minor thing: normally the subject line of the commit starts with
> the general area that it affects, followed by a colon, followed by the
> description starting with a capital letter.
Done. I just didn't know what area was covering this file, so I had left
it out...
>> + if (!skip_tos) {
>> + print_wild(&f, "nw_tos=", w& OFPFW_NW_TOS, verbosity,
>> + "%u", om->nw_tos);
>> + }
>
> I think you should be able to accomplish this without using the
> skip_tos flag. If the ToS isn't applicable (because the protocol
> isn't IP) then it will be wildcarded. print_wild() doesn't print
> wildcarded fields, unless the verbosity is high.
Right. I wasn't sure if flows were normalized at this level or not.
> The skip_XXX fields are just used if the protocol combination has a
> short form such as "tcp", in which case the fields which represent TCP
> are left out. However, in this case the ToS is never part of a short
> form and should always be printed if applicable.
Done.
Thanks for your quick review and your comments.
I sent the updated patch separately.
--
Romain Lenglet
More information about the dev
mailing list