Re: [ovs-dev] [PATCH] Fix packet-in reason for OpenFlow1.3 table-miss flow entries

2013-10-23 Thread YAMAMOTO Takashi
> On Wed, Oct 23, 2013 at 03:44:50PM +0900, YAMAMOTO Takashi wrote: >> > On Fri, Oct 18, 2013 at 01:28:44PM +0900, YAMAMOTO Takashi wrote: >> >> As per spec, make packet-in reason for OpenFlow1.3 table-miss flow >> >> entries no_match rather than action. >> >> >> >> Signed-off-by: YAMAMOTO Takashi

Re: [ovs-dev] [PATCH] Fix packet-in reason for OpenFlow1.3 table-miss flow entries

2013-10-23 Thread Ben Pfaff
On Wed, Oct 23, 2013 at 03:44:50PM +0900, YAMAMOTO Takashi wrote: > > On Fri, Oct 18, 2013 at 01:28:44PM +0900, YAMAMOTO Takashi wrote: > >> As per spec, make packet-in reason for OpenFlow1.3 table-miss flow > >> entries no_match rather than action. > >> > >> Signed-off-by: YAMAMOTO Takashi > >

Re: [ovs-dev] [PATCH] Fix packet-in reason for OpenFlow1.3 table-miss flow entries

2013-10-22 Thread YAMAMOTO Takashi
> On Fri, Oct 18, 2013 at 01:28:44PM +0900, YAMAMOTO Takashi wrote: >> As per spec, make packet-in reason for OpenFlow1.3 table-miss flow >> entries no_match rather than action. >> >> Signed-off-by: YAMAMOTO Takashi > > Thanks a lot for sending a second version. > > The more I thought about thi

Re: [ovs-dev] [PATCH] Fix packet-in reason for OpenFlow1.3 table-miss flow entries

2013-10-22 Thread Ben Pfaff
On Fri, Oct 18, 2013 at 01:28:44PM +0900, YAMAMOTO Takashi wrote: > As per spec, make packet-in reason for OpenFlow1.3 table-miss flow > entries no_match rather than action. > > Signed-off-by: YAMAMOTO Takashi Thanks a lot for sending a second version. The more I thought about this, the more ug

Re: [ovs-dev] [PATCH] Fix packet-in reason for OpenFlow1.3 table-miss flow entries

2013-10-17 Thread YAMAMOTO Takashi
> On Fri, Oct 18, 2013 at 01:41:20PM +0900, YAMAMOTO Takashi wrote: >> >> On Thu, Oct 17, 2013 at 02:11:00PM +0900, YAMAMOTO Takashi wrote: >> >>> > On Wed, Oct 16, 2013 at 05:24:36PM +0900, YAMAMOTO Takashi wrote: >> >>> >> As per spec, make packet-in reason for OpenFlow1.3 table-miss flow >> >>>

Re: [ovs-dev] [PATCH] Fix packet-in reason for OpenFlow1.3 table-miss flow entries

2013-10-17 Thread Ben Pfaff
On Fri, Oct 18, 2013 at 01:41:20PM +0900, YAMAMOTO Takashi wrote: > >> On Thu, Oct 17, 2013 at 02:11:00PM +0900, YAMAMOTO Takashi wrote: > >>> > On Wed, Oct 16, 2013 at 05:24:36PM +0900, YAMAMOTO Takashi wrote: > >>> >> As per spec, make packet-in reason for OpenFlow1.3 table-miss flow > >>> >> ent

Re: [ovs-dev] [PATCH] Fix packet-in reason for OpenFlow1.3 table-miss flow entries

2013-10-17 Thread YAMAMOTO Takashi
>> On Thu, Oct 17, 2013 at 02:11:00PM +0900, YAMAMOTO Takashi wrote: >>> > On Wed, Oct 16, 2013 at 05:24:36PM +0900, YAMAMOTO Takashi wrote: >>> >> As per spec, make packet-in reason for OpenFlow1.3 table-miss flow >>> >> entries no_match rather than action. >>> >> >>> >> Signed-off-by: YAMAMOTO T

[ovs-dev] [PATCH] Fix packet-in reason for OpenFlow1.3 table-miss flow entries

2013-10-17 Thread YAMAMOTO Takashi
As per spec, make packet-in reason for OpenFlow1.3 table-miss flow entries no_match rather than action. Signed-off-by: YAMAMOTO Takashi --- OPENFLOW-1.1+ | 3 -- include/openflow/openflow-common.h | 5 +- lib/learn.c| 6 ++- lib/learning-switch.

Re: [ovs-dev] [PATCH] Fix packet-in reason for OpenFlow1.3 table-miss flow entries

2013-10-16 Thread YAMAMOTO Takashi
> On Thu, Oct 17, 2013 at 02:11:00PM +0900, YAMAMOTO Takashi wrote: >> > On Wed, Oct 16, 2013 at 05:24:36PM +0900, YAMAMOTO Takashi wrote: >> >> As per spec, make packet-in reason for OpenFlow1.3 table-miss flow >> >> entries no_match rather than action. >> >> >> >> Signed-off-by: YAMAMOTO Takashi

Re: [ovs-dev] [PATCH] Fix packet-in reason for OpenFlow1.3 table-miss flow entries

2013-10-16 Thread Ben Pfaff
On Thu, Oct 17, 2013 at 02:11:00PM +0900, YAMAMOTO Takashi wrote: > > On Wed, Oct 16, 2013 at 05:24:36PM +0900, YAMAMOTO Takashi wrote: > >> As per spec, make packet-in reason for OpenFlow1.3 table-miss flow > >> entries no_match rather than action. > >> > >> Signed-off-by: YAMAMOTO Takashi > >

Re: [ovs-dev] [PATCH] Fix packet-in reason for OpenFlow1.3 table-miss flow entries

2013-10-16 Thread YAMAMOTO Takashi
> On Wed, Oct 16, 2013 at 05:24:36PM +0900, YAMAMOTO Takashi wrote: >> As per spec, make packet-in reason for OpenFlow1.3 table-miss flow >> entries no_match rather than action. >> >> Signed-off-by: YAMAMOTO Takashi > > Thanks! I really appreciate that you are working on conformance to > later

Re: [ovs-dev] [PATCH] Fix packet-in reason for OpenFlow1.3 table-miss flow entries

2013-10-16 Thread Ben Pfaff
On Wed, Oct 16, 2013 at 05:24:36PM +0900, YAMAMOTO Takashi wrote: > As per spec, make packet-in reason for OpenFlow1.3 table-miss flow > entries no_match rather than action. > > Signed-off-by: YAMAMOTO Takashi Thanks! I really appreciate that you are working on conformance to later OpenFlow spe

[ovs-dev] [PATCH] Fix packet-in reason for OpenFlow1.3 table-miss flow entries

2013-10-16 Thread YAMAMOTO Takashi
As per spec, make packet-in reason for OpenFlow1.3 table-miss flow entries no_match rather than action. Signed-off-by: YAMAMOTO Takashi --- OPENFLOW-1.1+ | 3 --- include/openflow/openflow-common.h | 5 - lib/ofp-util.c | 3 +++ ofproto/connmgr.c