On 5/16/17 12:18 PM, David Miller wrote:

Please correct the address of the netdev list (it is just plain
'netdev' not 'linux-netdev').

Thanks. Shortly after my first email, I sent a corrected submit as well.
Sorry for the spam.


Secondly, __always_inline should not be defined by types.h

That has to come from linux/compiler.h which we have no reason
to define a private version of for eBPF clang compilation.

The problem is that via several layers of indirection, linux/types.h
eventually includes linux/compiler.h and that is probably the more
appropriate thing for you to do.

Right. I found out simply including string.h will eventually include
linux/compiler.h so that I do not need explicitly defining
__always_inline.

Will send a revised patch soon.

Thanks,

Yonghong


Reply via email to