[ovs-dev] [PATCH 1/3] lib/flow: Update FLOW_WC_SEQ to skip assertions on miniflow_extract()

[Daniele Di Proietto]

Thanks,

Daniele

On 8/8/14, 10:20 AM, "Jarno Rajahalme" <jrajahalme at nicira.com> wrote:

>Pushed to master. No backports needed.
>
>As discussed offline, I¹ll wait for Ben to chime in on the other two
>patches before pushing them.
>
>  Jarno
>
>On Aug 8, 2014, at 9:09 AM, Jarno Rajahalme <jrajahalme at nicira.com> wrote:
>
>> Acked-by: Jarno Rajahalme <jrajahalme at nicira.com>
>> 
>>> On Aug 7, 2014, at 6:15 PM, Daniele Di Proietto
>>><ddiproietto at vmware.com> wrote:
>>> 
>>> Signed-off-by: Daniele Di Proietto <ddiproietto at vmware.com>
>>> ---
>>> lib/flow.c | 4 ++--
>>> 1 file changed, 2 insertions(+), 2 deletions(-)
>>> 
>>> diff --git a/lib/flow.c b/lib/flow.c
>>> index 5e04015..2e5ca0a 100644
>>> --- a/lib/flow.c
>>> +++ b/lib/flow.c
>>> @@ -118,10 +118,10 @@ struct mf_ctx {
>>> /* miniflow_push_* macros allow filling in a miniflow data values in
>>>order.
>>> * Assertions are needed only when the layout of the struct flow is
>>>modified.
>>> * 'ofs' is a compile-time constant, which allows most of the code be
>>>optimized
>>> - * away.  Some GCC versions gave warnigns on ALWAYS_INLINE, so these
>>>are
>>> + * away.  Some GCC versions gave warnings on ALWAYS_INLINE, so these
>>>are
>>> * defined as macros. */
>>> 
>>> -#if (FLOW_WC_SEQ != 26)
>>> +#if (FLOW_WC_SEQ != 27)
>>> #define MINIFLOW_ASSERT(X) ovs_assert(X)
>>> #else
>>> #define MINIFLOW_ASSERT(X)
>>> -- 
>>> 2.0.1
>>> 
>>> _______________________________________________
>>> dev mailing list
>>> dev at openvswitch.org
>>> 
>>>https://urldefense.proofpoint.com/v1/url?u=http://openvswitch.org/mailma
>>>n/listinfo/dev&k=oIvRg1%2BdGAgOoM1BIlLLqw%3D%3D%0A&r=MV9BdLjtFIdhBDBaw5z
>>>%2BU6SSA2gAfY4L%2F1HCy3VjlKU%3D%0A&m=NIMPJfBS5ghugDaXANfO8E2IeNbjWYZ9jCa
>>>uW%2Badsp0%3D%0A&s=9415ff5b8cc2683f30a955a13eb44963f7a4d0723d6a8ad3e1f7b
>>>c2acd8a1e59
>