[ovs-dev] [PATCH] datapath: Simplify ODPAT_SET_DL_TCI action.

Jesse Gross jesse at nicira.com
Fri Oct 15 23:00:05 UTC 2010


On Tue, Oct 12, 2010 at 11:40 AM, Ben Pfaff <blp at nicira.com> wrote:
> There's no need to have a mask in this action, because both parts of the
> TCI are part of the flow structure.
>
> Signed-off-by: Ben Pfaff <blp at nicira.com>
> ---
> So far, this is oompile-tested only, so there could be some really
> stupid things in it.

The only problem that I see is the above sentence.  You don't seem to
have run it through your oompiler.

Otherwise I like the simplification.
Acked-by: Jesse Gross <jesse at nicira.com>




More information about the dev mailing list