On Mon, Jan 25, 2016 at 11:41:33AM -0500, Russell Bryant wrote: > On 01/23/2016 08:17 PM, Ben Pfaff wrote: > > Suggested-by: William Tu <u9012...@gmail.com> > > Signed-off-by: Ben Pfaff <b...@ovn.org> > > --- > > lib/ovs-rcu.h | 6 ++++-- > > 1 file changed, 4 insertions(+), 2 deletions(-) > > > > diff --git a/lib/ovs-rcu.h b/lib/ovs-rcu.h > > index 5cd4152..7fe7d47 100644 > > --- a/lib/ovs-rcu.h > > +++ b/lib/ovs-rcu.h > > @@ -1,5 +1,5 @@ > > /* > > - * Copyright (c) 2014, 2015 Nicira, Inc. > > + * Copyright (c) 2014, 2015, 2016 Nicira, Inc. > > * > > * Licensed under the Apache License, Version 2.0 (the "License"); > > * you may not use this file except in compliance with the License. > > @@ -207,7 +207,9 @@ static inline void ovsrcu_set__(struct ovsrcu_pointer > > *pointer, > > * grace period. See "Usage" above for an example. */ > > void ovsrcu_postpone__(void (*function)(void *aux), void *aux); > > #define ovsrcu_postpone(FUNCTION, ARG) \ > > - ((void) sizeof((FUNCTION)(ARG), 1), \ > > + (/* Verify that ARG is appropriate for FUNCTION. */ \ > > + (void) sizeof((FUNCTION)(ARG), 1), \ > > + /* Verify that ARG is a pointer type. */ \ > > (void) sizeof(*(ARG)), \ > > ovsrcu_postpone__((void (*)(void *))(FUNCTION), ARG)) > > > > > > Pretty clever checks. :-) > > Acked-by: Russell Bryant <russ...@ovn.org>
Thanks Russell, I applied this to master. _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
