diff mbox series

[ovs-dev] controller: Fix the wrong 'struct' type for 'pflow_output_data'.

Message ID 20210622201030.1243621-1-numans@ovn.org
State Accepted
Headers show
Series [ovs-dev] controller: Fix the wrong 'struct' type for 'pflow_output_data'. | expand

Commit Message

Numan Siddique June 22, 2021, 8:10 p.m. UTC
From: Numan Siddique <numans@ovn.org>

'pflow_output_data' should be of type 'struct ed_type_pflow_output'
and not 'struct ed_type_lflow_output'.

Fixes: e07e397b7ae("ovn-controller: Split logical flow and physical flow processing.")
Signed-off-by: Numan Siddique <numans@ovn.org>
---
 controller/ovn-controller.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Mark Michelson June 23, 2021, 3:25 p.m. UTC | #1
Acked-by: Mark Michelson <mmichels@redhat.com>

On 6/22/21 4:10 PM, numans@ovn.org wrote:
> From: Numan Siddique <numans@ovn.org>
> 
> 'pflow_output_data' should be of type 'struct ed_type_pflow_output'
> and not 'struct ed_type_lflow_output'.
> 
> Fixes: e07e397b7ae("ovn-controller: Split logical flow and physical flow processing.")
> Signed-off-by: Numan Siddique <numans@ovn.org>
> ---
>   controller/ovn-controller.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/controller/ovn-controller.c b/controller/ovn-controller.c
> index 1cfe4b713..3968ef059 100644
> --- a/controller/ovn-controller.c
> +++ b/controller/ovn-controller.c
> @@ -2980,7 +2980,7 @@ main(int argc, char *argv[])
>   
>       struct ed_type_lflow_output *lflow_output_data =
>           engine_get_internal_data(&en_lflow_output);
> -    struct ed_type_lflow_output *pflow_output_data =
> +    struct ed_type_pflow_output *pflow_output_data =
>           engine_get_internal_data(&en_pflow_output);
>       struct ed_type_ct_zones *ct_zones_data =
>           engine_get_internal_data(&en_ct_zones);
>
Numan Siddique June 23, 2021, 4:53 p.m. UTC | #2
On Wed, Jun 23, 2021 at 11:26 AM Mark Michelson <mmichels@redhat.com> wrote:
>
> Acked-by: Mark Michelson <mmichels@redhat.com>

Thanks.  Applied to the main branch.

Numan

>
> On 6/22/21 4:10 PM, numans@ovn.org wrote:
> > From: Numan Siddique <numans@ovn.org>
> >
> > 'pflow_output_data' should be of type 'struct ed_type_pflow_output'
> > and not 'struct ed_type_lflow_output'.
> >
> > Fixes: e07e397b7ae("ovn-controller: Split logical flow and physical flow processing.")
> > Signed-off-by: Numan Siddique <numans@ovn.org>
> > ---
> >   controller/ovn-controller.c | 2 +-
> >   1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/controller/ovn-controller.c b/controller/ovn-controller.c
> > index 1cfe4b713..3968ef059 100644
> > --- a/controller/ovn-controller.c
> > +++ b/controller/ovn-controller.c
> > @@ -2980,7 +2980,7 @@ main(int argc, char *argv[])
> >
> >       struct ed_type_lflow_output *lflow_output_data =
> >           engine_get_internal_data(&en_lflow_output);
> > -    struct ed_type_lflow_output *pflow_output_data =
> > +    struct ed_type_pflow_output *pflow_output_data =
> >           engine_get_internal_data(&en_pflow_output);
> >       struct ed_type_ct_zones *ct_zones_data =
> >           engine_get_internal_data(&en_ct_zones);
> >
>
> _______________________________________________
> dev mailing list
> dev@openvswitch.org
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
>
diff mbox series

Patch

diff --git a/controller/ovn-controller.c b/controller/ovn-controller.c
index 1cfe4b713..3968ef059 100644
--- a/controller/ovn-controller.c
+++ b/controller/ovn-controller.c
@@ -2980,7 +2980,7 @@  main(int argc, char *argv[])
 
     struct ed_type_lflow_output *lflow_output_data =
         engine_get_internal_data(&en_lflow_output);
-    struct ed_type_lflow_output *pflow_output_data =
+    struct ed_type_pflow_output *pflow_output_data =
         engine_get_internal_data(&en_pflow_output);
     struct ed_type_ct_zones *ct_zones_data =
         engine_get_internal_data(&en_ct_zones);