Re: [ovs-dev] [PATCH 1/5] Add error codes for Open Flow v1.2

2012-03-26 Thread Ben Pfaff
On Tue, Mar 27, 2012 at 10:14:24AM +0900, Simon Horman wrote: > On Mon, Mar 26, 2012 at 01:57:09PM -0700, Ben Pfaff wrote: > > Anyway, I'll send this out a new series derived from this patch in a > > minute. Simon, please review it. > > Sure, will do. The changes you describe above sound entirely

Re: [ovs-dev] [PATCH 1/5] Add error codes for Open Flow v1.2

2012-03-26 Thread Simon Horman
On Mon, Mar 26, 2012 at 01:57:09PM -0700, Ben Pfaff wrote: > On Mon, Mar 26, 2012 at 02:30:23PM +0900, Simon Horman wrote: > > Signed-off-by: Simon Horman > > > > --- > > v2 > > As suggested by Ben Pfaff > > * Where Open Flow 1.2 breaks apart error codes defined > > in previous versions, provid

Re: [ovs-dev] [PATCH 1/5] Add error codes for Open Flow v1.2

2012-03-26 Thread Ben Pfaff
On Mon, Mar 26, 2012 at 02:30:23PM +0900, Simon Horman wrote: > Signed-off-by: Simon Horman > > --- > v2 > As suggested by Ben Pfaff > * Where Open Flow 1.2 breaks apart error codes defined > in previous versions, provide all new definitions to > previous versions and map the numeric error co

[ovs-dev] [PATCH 1/5] Add error codes for Open Flow v1.2

2012-03-25 Thread Simon Horman
Signed-off-by: Simon Horman --- v2 As suggested by Ben Pfaff * Where Open Flow 1.2 breaks apart error codes defined in previous versions, provide all new definitions to previous versions and map the numeric error code to the first first definition supplied in ofp-errors.h. The case handle