[ovs-dev,1/3] ofproto-dpif: Update handle_action() comment.
diff mbox

Message ID 1490250377-9519-1-git-send-email-jpettit@ovn.org
State Accepted
Headers show

Commit Message

Justin Pettit March 23, 2017, 6:26 a.m. UTC
The comment didn't mention the SLOW_LLDP and SLOW_ACTION reasons.

Signed-off-by: Justin Pettit <jpettit@ovn.org>
---
 ofproto/ofproto-dpif-upcall.c | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

Comments

Jarno Rajahalme March 23, 2017, 4:58 p.m. UTC | #1
Acked-by: Jarno Rajahalme <jarno@ovn.org>

> On Mar 22, 2017, at 11:26 PM, Justin Pettit <jpettit@ovn.org> wrote:
> 
> The comment didn't mention the SLOW_LLDP and SLOW_ACTION reasons.
> 
> Signed-off-by: Justin Pettit <jpettit@ovn.org>
> ---
> ofproto/ofproto-dpif-upcall.c | 7 +++++--
> 1 file changed, 5 insertions(+), 2 deletions(-)
> 
> diff --git a/ofproto/ofproto-dpif-upcall.c b/ofproto/ofproto-dpif-upcall.c
> index 07086ee385cc..158d65eecefa 100644
> --- a/ofproto/ofproto-dpif-upcall.c
> +++ b/ofproto/ofproto-dpif-upcall.c
> @@ -1352,12 +1352,15 @@ handle_upcalls(struct udpif *udpif, struct upcall *upcalls,
> 
>     /* Handle the packets individually in order of arrival.
>      *
> -     *   - For SLOW_CFM, SLOW_LACP, SLOW_STP, and SLOW_BFD, translation is what
> -     *     processes received packets for these protocols.
> +     *   - For SLOW_CFM, SLOW_LACP, SLOW_STP, SLOW_BFD, and SLOW_LLDP,
> +     *     translation is what processes received packets for these
> +     *     protocols.
>      *
>      *   - For SLOW_CONTROLLER, translation sends the packet to the OpenFlow
>      *     controller.
>      *
> +     *   - For SLOW_ACTION, translation executes the actions directly.
> +     *
>      * The loop fills 'ops' with an array of operations to execute in the
>      * datapath. */
>     n_ops = 0;
> -- 
> 2.7.4
> 
> _______________________________________________
> dev mailing list
> dev@openvswitch.org
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Patch
diff mbox

diff --git a/ofproto/ofproto-dpif-upcall.c b/ofproto/ofproto-dpif-upcall.c
index 07086ee385cc..158d65eecefa 100644
--- a/ofproto/ofproto-dpif-upcall.c
+++ b/ofproto/ofproto-dpif-upcall.c
@@ -1352,12 +1352,15 @@  handle_upcalls(struct udpif *udpif, struct upcall *upcalls,
 
     /* Handle the packets individually in order of arrival.
      *
-     *   - For SLOW_CFM, SLOW_LACP, SLOW_STP, and SLOW_BFD, translation is what
-     *     processes received packets for these protocols.
+     *   - For SLOW_CFM, SLOW_LACP, SLOW_STP, SLOW_BFD, and SLOW_LLDP,
+     *     translation is what processes received packets for these
+     *     protocols.
      *
      *   - For SLOW_CONTROLLER, translation sends the packet to the OpenFlow
      *     controller.
      *
+     *   - For SLOW_ACTION, translation executes the actions directly.
+     *
      * The loop fills 'ops' with an array of operations to execute in the
      * datapath. */
     n_ops = 0;