On Fri, May 29, 2015 at 10:20 AM, Joe Stringer <joestrin...@nicira.com> wrote:
> On 28 May 2015 at 18:39, Jesse Gross <je...@nicira.com> wrote:
>> On Wed, May 27, 2015 at 3:18 PM, Andy Zhou <az...@nicira.com> wrote:
>>> On Wed, May 27, 2015 at 10:48 AM, Jesse Gross <je...@nicira.com> wrote:
>>>> Even though userspace does not yet support Geneve options,
>>>> the kernel does and there is some basic support for decoding
>>>> those attributes. This adds the ability to print Geneve
>>>> attributes that might potentially come from the kernel.
>>>>
>>>> Signed-off-by: Jesse Gross <je...@nicira.com>
>>>> ---
>>>
>>> git am says:
>>>
>>> Applying: odp-util: Geneve netlink decoding.
>>> /home/azhou/projs/ovs-review/ovs/.git/rebase-apply/patch:50: trailing
>>> whitespace.
>>> if (mask && nl_attr_get_size(attr) != nl_attr_get_size(mask_attr)) {
>>> warning: 1 line adds whitespace errors.
>>>
>>> Assume this will be fixed: Acked-by: Andy Zhou <az...@nicira.com>
>>
>> Thanks for the reviews. I pushed this series to master with the
>> changes mentioned.
>
> Clang complains about cast alignment:
> https://travis-ci.org/openvswitch/ovs/jobs/64510105
Thanks for pointing that out. I just send out a patch to fix it.
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
