Re: [ovs-dev] [PATCH v1] ofproto: Fix OVS crash when packets hit Openflow rules with certain combinations of nested actions

2019-11-06 Thread Anil Kumar Koli via dev
s=TABLE' Thanks & Regards, Anil Kumar. -Original Message- From: Anil Kumar Koli Sent: Monday, 4 November, 2019 09:46 AM To: 'Ben Pfaff' ; 'Ilya Maximets' Cc: 'Ilya Maximets' ; 'ovs-dev' Subject: RE: [ovs-dev] [PATCH v1] ofproto: Fix OVS crash when packets hit Openflow rules with certa

Re: [ovs-dev] [PATCH v1] ofproto: Fix OVS crash when packets hit Openflow rules with certain combinations of nested actions

2019-11-03 Thread Anil Kumar Koli via dev
Anil Kumar Koli ; ovs-dev Subject: Re: [ovs-dev] [PATCH v1] ofproto: Fix OVS crash when packets hit Openflow rules with certain combinations of nested actions On Fri, Nov 01, 2019 at 02:35:55PM +0100, Ilya Maximets wrote: > CC: ovs-dev > Sorry. > > On 01.11.2019 14:34, Ily

Re: [ovs-dev] [PATCH v1] ofproto: Fix OVS crash when packets hit Openflow rules with certain combinations of nested actions

2019-11-01 Thread Ben Pfaff
On Fri, Nov 01, 2019 at 02:35:55PM +0100, Ilya Maximets wrote: > CC: ovs-dev > Sorry. > > On 01.11.2019 14:34, Ilya Maximets wrote: > > On 31.10.2019 18:12, Ben Pfaff wrote: > > > On Thu, Sep 05, 2019 at 11:12:26PM +0530, Anil Kumar Koli wrote: > > > > > * We can't get rid of ofproto_mutex in

Re: [ovs-dev] [PATCH v1] ofproto: Fix OVS crash when packets hit Openflow rules with certain combinations of nested actions

2019-11-01 Thread Ben Pfaff
On Fri, Nov 01, 2019 at 02:39:30PM +0100, Ilya Maximets wrote: > On 31.10.2019 18:24, Ben Pfaff wrote: > > For Ilya and others: the email to Anil bounced, so he probably isn't at > > Calsoft Labs any longer. > > Are you sure that it wasn't mine old email address? Oh, you're right. Apparently I

Re: [ovs-dev] [PATCH v1] ofproto: Fix OVS crash when packets hit Openflow rules with certain combinations of nested actions

2019-11-01 Thread Ilya Maximets
On 31.10.2019 18:24, Ben Pfaff wrote: For Ilya and others: the email to Anil bounced, so he probably isn't at Calsoft Labs any longer. Are you sure that it wasn't mine old email address? ___ dev mailing list d...@openvswitch.org

Re: [ovs-dev] [PATCH v1] ofproto: Fix OVS crash when packets hit Openflow rules with certain combinations of nested actions

2019-11-01 Thread Ilya Maximets
CC: ovs-dev Sorry. On 01.11.2019 14:34, Ilya Maximets wrote: On 31.10.2019 18:12, Ben Pfaff wrote: On Thu, Sep 05, 2019 at 11:12:26PM +0530, Anil Kumar Koli wrote: * We can't get rid of ofproto_mutex in do_bundle_commit(), or drop it    temporarily around flow translation (i.e. the call to   

Re: [ovs-dev] [PATCH v1] ofproto: Fix OVS crash when packets hit Openflow rules with certain combinations of nested actions

2019-10-31 Thread Ben Pfaff
For Ilya and others: the email to Anil bounced, so he probably isn't at Calsoft Labs any longer. On Thu, Oct 31, 2019 at 10:12:04AM -0700, Ben Pfaff wrote: > On Thu, Sep 05, 2019 at 11:12:26PM +0530, Anil Kumar Koli wrote: > > > * We can't get rid of ofproto_mutex in do_bundle_commit(), or drop

Re: [ovs-dev] [PATCH v1] ofproto: Fix OVS crash when packets hit Openflow rules with certain combinations of nested actions

2019-10-31 Thread Ben Pfaff
On Thu, Sep 05, 2019 at 11:12:26PM +0530, Anil Kumar Koli wrote: > > * We can't get rid of ofproto_mutex in do_bundle_commit(), or drop it > > temporarily around flow translation (i.e. the call to > > ofproto_packet_out_start()), because it might need to revert all the > > flow table changes

Re: [ovs-dev] [PATCH v1] ofproto: Fix OVS crash when packets hit Openflow rules with certain combinations of nested actions

2019-10-30 Thread Ilya Maximets
On 29.08.2019 13:39, Ilya Maximets wrote: On 29.08.2019 12:52, Anil Kumar Koli wrote: Hi Ilya, Thanks for the review. Please find below the stack trace of the crash (gdb) bt #0 0x7f0a3da46c37 in __GI_raise (sig=sig@entry=6) at ../nptl/sysdeps/unix/sysv/linux/raise.c:56 #1

Re: [ovs-dev] [PATCH v1] ofproto: Fix OVS crash when packets hit Openflow rules with certain combinations of nested actions

2019-09-24 Thread Anil Kumar Koli via dev
faff' Cc: 'Ilya Maximets' ; 'ovs-dev@openvswitch.org' Subject: RE: [ovs-dev] [PATCH v1] ofproto: Fix OVS crash when packets hit Openflow rules with certain combinations of nested actions Hi Ben, Please consider this as a gentle remainder and provide your inputs on the below issue. Thanks &

Re: [ovs-dev] [PATCH v1] ofproto: Fix OVS crash when packets hit Openflow rules with certain combinations of nested actions

2019-09-12 Thread Anil Kumar Koli via dev
ject: RE: [ovs-dev] [PATCH v1] ofproto: Fix OVS crash when packets hit Openflow rules with certain combinations of nested actions Hi Ben, Please consider this as a gentle remainder and provide your inputs on the below issue. This issue has been discussed earlier in the following thread. h

Re: [ovs-dev] [PATCH v1] ofproto: Fix OVS crash when packets hit Openflow rules with certain combinations of nested actions

2019-09-05 Thread Anil Kumar Koli via dev
Kumar Koli Sent: Tuesday, 3 September, 2019 11:35 AM To: 'Ilya Maximets' ; 'ovs-dev@openvswitch.org' Cc: 'Ben Pfaff' Subject: RE: [ovs-dev] [PATCH v1] ofproto: Fix OVS crash when packets hit Openflow rules with certain combinations of nested actions Hi Ilya, Sorry for late reply and thanks

Re: [ovs-dev] [PATCH v1] ofproto: Fix OVS crash when packets hit Openflow rules with certain combinations of nested actions

2019-09-03 Thread Anil Kumar Koli via dev
] ofproto: Fix OVS crash when packets hit Openflow rules with certain combinations of nested actions On 29.08.2019 12:52, Anil Kumar Koli wrote: > Hi Ilya, > > Thanks for the review. > Please find below the stack trace of the crash > > (gdb) bt > #0 0x7f0a3da46c37 in __GI_r

Re: [ovs-dev] [PATCH v1] ofproto: Fix OVS crash when packets hit Openflow rules with certain combinations of nested actions

2019-08-29 Thread Ilya Maximets
On 29.08.2019 12:52, Anil Kumar Koli wrote: > Hi Ilya, > > Thanks for the review. > Please find below the stack trace of the crash > > (gdb) bt > #0 0x7f0a3da46c37 in __GI_raise (sig=sig@entry=6) at > ../nptl/sysdeps/unix/sysv/linux/raise.c:56 > #1 0x7f0a3da4a028 in __GI_abort () at

Re: [ovs-dev] [PATCH v1] ofproto: Fix OVS crash when packets hit Openflow rules with certain combinations of nested actions

2019-08-29 Thread Anil Kumar Koli via dev
Kumar ; ovs-dev@openvswitch.org Subject: Re: [ovs-dev] [PATCH v1] ofproto: Fix OVS crash when packets hit Openflow rules with certain combinations of nested actions On 28.08.2019 13:24, Anil Kumar wrote: > OVS crashes when a packet sent with action set to OFPP_TABLE hits > Openflow

Re: [ovs-dev] [PATCH v1] ofproto: Fix OVS crash when packets hit Openflow rules with certain combinations of nested actions

2019-08-28 Thread Ilya Maximets
On 28.08.2019 13:24, Anil Kumar wrote: > OVS crashes when a packet sent with action set to OFPP_TABLE hits Openflow > rules with conntrack and learn actions. > > For example: > The crash can be triggered by installing the following Open flow rules and > sending packet with action set to

[ovs-dev] [PATCH v1] ofproto: Fix OVS crash when packets hit Openflow rules with certain combinations of nested actions

2019-08-28 Thread Anil Kumar via dev
OVS crashes when a packet sent with action set to OFPP_TABLE hits Openflow rules with conntrack and learn actions. For example: The crash can be triggered by installing the following Open flow rules and sending packet with action set to OFPP_TABLE 1. ovs-ofctl -OOpenflow13 add-flow br-int