Re: [ovs-dev] [PATCH 1/2] test-classifier: Ensure priority is not negative.

2014-10-31 Thread Jarno Rajahalme
On Oct 31, 2014, at 4:06 PM, Ben Pfaff wrote: > On Fri, Oct 31, 2014 at 04:02:23PM -0700, Jarno Rajahalme wrote: >> >> On Oct 31, 2014, at 2:24 PM, Ben Pfaff wrote: >> >>> On Fri, Oct 31, 2014 at 02:17:56PM -0700, Jarno Rajahalme wrote: Classifier users should not use negative priorities

Re: [ovs-dev] [PATCH 1/2] test-classifier: Ensure priority is not negative.

2014-10-31 Thread Ben Pfaff
On Fri, Oct 31, 2014 at 04:02:23PM -0700, Jarno Rajahalme wrote: > > On Oct 31, 2014, at 2:24 PM, Ben Pfaff wrote: > > > On Fri, Oct 31, 2014 at 02:17:56PM -0700, Jarno Rajahalme wrote: > >> Classifier users should not use negative priorities. > >> > >> Signed-off-by: Jarno Rajahalme > > > >

Re: [ovs-dev] [PATCH 1/2] test-classifier: Ensure priority is not negative.

2014-10-31 Thread Jarno Rajahalme
On Oct 31, 2014, at 2:24 PM, Ben Pfaff wrote: > On Fri, Oct 31, 2014 at 02:17:56PM -0700, Jarno Rajahalme wrote: >> Classifier users should not use negative priorities. >> >> Signed-off-by: Jarno Rajahalme > > The only priority that the classifier can't handle properly is > INT_MIN. Unfortun

Re: [ovs-dev] [PATCH 1/2] test-classifier: Ensure priority is not negative.

2014-10-31 Thread Ben Pfaff
On Fri, Oct 31, 2014 at 02:17:56PM -0700, Jarno Rajahalme wrote: > Classifier users should not use negative priorities. > > Signed-off-by: Jarno Rajahalme The only priority that the classifier can't handle properly is INT_MIN. Unfortunately, that's also the one priority that this patch doesn't