[ovs-dev] [PATCH v2 1/2] vtep: make vtep-ctl support L3 tables.

Guru Shetty guru at ovn.org
Thu Jan 28 22:04:54 UTC 2016


The code looks fine. But you have removed a page break (^L character).  I
will re-add it and push it after sometime.

On 26 January 2016 at 02:34, Wenyu Zhang <wenyuz at vmware.com> wrote:

> Add supporting for tables necessary for L3 usage:
> Logical_Router
> Arp_Sources_Local
> Arp_Sources_Remote
>
> Signed-off-by: Wenyu Zhang <wenyuz at vmware.com>
> Acked-by: Shuangmin Zhang <tsingzsm at gmail.com>
> ---
>  vtep/vtep-ctl.c | 13 ++++++++++++-
>  1 file changed, 12 insertions(+), 1 deletion(-)
>
> diff --git a/vtep/vtep-ctl.c b/vtep/vtep-ctl.c
> index 604d19d..048ee77 100644
> --- a/vtep/vtep-ctl.c
> +++ b/vtep/vtep-ctl.c
> @@ -2044,10 +2044,21 @@ static const struct ctl_table_class tables[] = {
>       {{NULL, NULL, NULL},
>        {NULL, NULL, NULL}}},
>
> +    {&vteprec_table_logical_router,
> +     {{&vteprec_table_logical_router, &vteprec_logical_router_col_name,
> NULL},
> +      {NULL, NULL, NULL}}},
> +
> +    {&vteprec_table_arp_sources_local,
> +     {{NULL, NULL, NULL},
> +      {NULL, NULL, NULL}}},
> +
> +    {&vteprec_table_arp_sources_remote,
> +     {{NULL, NULL, NULL},
> +      {NULL, NULL, NULL}}},
> +
>      {NULL, {{NULL, NULL, NULL}, {NULL, NULL, NULL}}}
>  };
>
> -
>  static void
>  vtep_ctl_context_init_command(struct vtep_ctl_context *vtepctl_ctx,
>                                struct ctl_command *command)
> --
> 2.1.4
>
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev
>



More information about the dev mailing list