Re: [ovs-dev] [PATCH 1/2] Fix handling of OFPP_ANY in OpenFlow 1.1 and later.

2013-01-08 Thread Justin Pettit
On Jan 4, 2013, at 8:58 AM, Ben Pfaff wrote: > On Thu, Jan 03, 2013 at 01:47:32PM -0800, Gurucharan Shetty wrote: >> On Thu, Jan 3, 2013 at 11:20 AM, Ben Pfaff wrote: >> >>> Ouch. This is nasty. OFPP_NONE is definitely not the same as a >>> wildcard. >>> >>> I guess we have a few choices: >>

Re: [ovs-dev] [PATCH 1/2] Fix handling of OFPP_ANY in OpenFlow 1.1 and later.

2013-01-04 Thread Ben Pfaff
On Thu, Jan 03, 2013 at 01:47:32PM -0800, Gurucharan Shetty wrote: > On Thu, Jan 3, 2013 at 11:20 AM, Ben Pfaff wrote: > > > On Thu, Jan 03, 2013 at 10:42:59AM -0800, Gurucharan Shetty wrote: > > > On Mon, Nov 26, 2012 at 10:11 AM, Ben Pfaff wrote: > > > > > > > From: Jarno Rajahalme > > > > >

Re: [ovs-dev] [PATCH 1/2] Fix handling of OFPP_ANY in OpenFlow 1.1 and later.

2013-01-03 Thread Gurucharan Shetty
On Thu, Jan 3, 2013 at 11:20 AM, Ben Pfaff wrote: > On Thu, Jan 03, 2013 at 10:42:59AM -0800, Gurucharan Shetty wrote: > > On Mon, Nov 26, 2012 at 10:11 AM, Ben Pfaff wrote: > > > > > From: Jarno Rajahalme > > > > > > Add OFPP_ANY to include/openflow/openflow-1.1.h, and allow it as a > port in

Re: [ovs-dev] [PATCH 1/2] Fix handling of OFPP_ANY in OpenFlow 1.1 and later.

2013-01-03 Thread Ben Pfaff
On Thu, Jan 03, 2013 at 10:42:59AM -0800, Gurucharan Shetty wrote: > On Mon, Nov 26, 2012 at 10:11 AM, Ben Pfaff wrote: > > > From: Jarno Rajahalme > > > > Add OFPP_ANY to include/openflow/openflow-1.1.h, and allow it as a port in > > queue stats request. Make ovs_ofctl use OFPP_ANY instead of O

Re: [ovs-dev] [PATCH 1/2] Fix handling of OFPP_ANY in OpenFlow 1.1 and later.

2013-01-03 Thread Gurucharan Shetty
On Mon, Nov 26, 2012 at 10:11 AM, Ben Pfaff wrote: > From: Jarno Rajahalme > > Add OFPP_ANY to include/openflow/openflow-1.1.h, and allow it as a port in > queue stats request. Make ovs_ofctl use OFPP_ANY instead of OFPP_ALL for > queue > stats requests on OF 1.1+. > > This patch changes "none"

Re: [ovs-dev] [PATCH 1/2] Fix handling of OFPP_ANY in OpenFlow 1.1 and later.

2012-11-26 Thread Jarno Rajahalme
On Nov 26, 2012, at 20:11 , ext Ben Pfaff wrote: > From: Jarno Rajahalme > > Add OFPP_ANY to include/openflow/openflow-1.1.h, and allow it as a port in > queue stats request. Make ovs_ofctl use OFPP_ANY instead of OFPP_ALL for queue > stats requests on OF 1.1+. > > This patch changes "none" por

[ovs-dev] [PATCH 1/2] Fix handling of OFPP_ANY in OpenFlow 1.1 and later.

2012-11-26 Thread Ben Pfaff
From: Jarno Rajahalme Add OFPP_ANY to include/openflow/openflow-1.1.h, and allow it as a port in queue stats request. Make ovs_ofctl use OFPP_ANY instead of OFPP_ALL for queue stats requests on OF 1.1+. This patch changes "none" ports print out. "none" is still accepted on input for backwards co