Re: [ovs-dev] [PATCH] ip_tunnels: fix grammer error.

2018-02-23 Thread Ben Pfaff
On Wed, Feb 21, 2018 at 03:10:54PM -0800, Gregory Rose wrote:
> On 2/21/2018 2:54 PM, William Tu wrote:
> >Trivial grammer fix.
> >
> >Signed-off-by: William Tu 
> >---
> >  datapath/linux/compat/include/net/ip_tunnels.h | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> >diff --git a/datapath/linux/compat/include/net/ip_tunnels.h 
> >b/datapath/linux/compat/include/net/ip_tunnels.h
> >index abb8ca63e5e4..ae60f09e2a23 100644
> >--- a/datapath/linux/compat/include/net/ip_tunnels.h
> >+++ b/datapath/linux/compat/include/net/ip_tunnels.h
> >@@ -34,7 +34,7 @@ int ovs_iptunnel_handle_offloads(struct sk_buff *skb,
> >  int gso_type_mask,
> >  void (*fix_segment)(struct sk_buff *));
> >-/* This is is required to compile upstream gre.h. gre_handle_offloads()
> >+/* This is required to compile upstream gre.h. gre_handle_offloads()
> >   * is defined in gre.h and needs iptunnel_handle_offloads(). This provides
> >   * default signature for this function.
> >   * rpl prefix is to make OVS build happy.
> 
> LGTM
> Reviewed-by: Greg Rose 

Thanks William and Greg, I applied this to master.
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev


Re: [ovs-dev] [PATCH] ip_tunnels: fix grammer error.

2018-02-21 Thread Gregory Rose

On 2/21/2018 2:54 PM, William Tu wrote:

Trivial grammer fix.

Signed-off-by: William Tu 
---
  datapath/linux/compat/include/net/ip_tunnels.h | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/datapath/linux/compat/include/net/ip_tunnels.h 
b/datapath/linux/compat/include/net/ip_tunnels.h
index abb8ca63e5e4..ae60f09e2a23 100644
--- a/datapath/linux/compat/include/net/ip_tunnels.h
+++ b/datapath/linux/compat/include/net/ip_tunnels.h
@@ -34,7 +34,7 @@ int ovs_iptunnel_handle_offloads(struct sk_buff *skb,
 int gso_type_mask,
 void (*fix_segment)(struct sk_buff *));
  
-/* This is is required to compile upstream gre.h. gre_handle_offloads()

+/* This is required to compile upstream gre.h. gre_handle_offloads()
   * is defined in gre.h and needs iptunnel_handle_offloads(). This provides
   * default signature for this function.
   * rpl prefix is to make OVS build happy.


LGTM
Reviewed-by: Greg Rose 

___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev