[ovs-dev] [PATCH v2] ofp-flow: skip flow reply if it exceeds the maximum message size

2022-01-05 Thread Eelco Chaudron
Currently, if a flow reply results in a message which exceeds the maximum reply size, it will assert OVS. This would happen when OVN uses OpenFlow15 to add large flows, and they get read using OpenFlow10 with ovs-ofctl. This patch prevents this and adds a test case to make sure the code behaves as

Re: [ovs-dev] [PATCH v2] ofp-flow: skip flow reply if it exceeds the maximum message size

2022-01-07 Thread Maxime Coquelin
Hi Eelco, On 1/5/22 10:48, Eelco Chaudron wrote: Currently, if a flow reply results in a message which exceeds the maximum reply size, it will assert OVS. This would happen when OVN uses OpenFlow15 to add large flows, and they get read using OpenFlow10 with ovs-ofctl. This patch prevents this a

Re: [ovs-dev] [PATCH v2] ofp-flow: skip flow reply if it exceeds the maximum message size

2022-01-07 Thread Eelco Chaudron
On 7 Jan 2022, at 13:34, Maxime Coquelin wrote: > Hi Eelco, > > On 1/5/22 10:48, Eelco Chaudron wrote: >> Currently, if a flow reply results in a message which exceeds >> the maximum reply size, it will assert OVS. This would happen >> when OVN uses OpenFlow15 to add large flows, and they get r

Re: [ovs-dev] [PATCH v2] ofp-flow: skip flow reply if it exceeds the maximum message size

2022-01-07 Thread Ilya Maximets
On 1/7/22 13:43, Eelco Chaudron wrote: > > > On 7 Jan 2022, at 13:34, Maxime Coquelin wrote: > >> Hi Eelco, >> >> On 1/5/22 10:48, Eelco Chaudron wrote: >>> Currently, if a flow reply results in a message which exceeds >>> the maximum reply size, it will assert OVS. This would happen >>> when OV

Re: [ovs-dev] [PATCH v2] ofp-flow: skip flow reply if it exceeds the maximum message size

2022-01-07 Thread Eelco Chaudron
On 7 Jan 2022, at 14:15, Ilya Maximets wrote: > On 1/7/22 13:43, Eelco Chaudron wrote: >> >> >> On 7 Jan 2022, at 13:34, Maxime Coquelin wrote: >> >>> Hi Eelco, >>> >>> On 1/5/22 10:48, Eelco Chaudron wrote: Currently, if a flow reply results in a message which exceeds the maximum rep