[ovs-dev] [PATCH] ovs-rcu: Improve comments on ovsrcu_postpone().
Russell Bryant
russell at ovn.org
Mon Jan 25 16:41:33 UTC 2016
On 01/23/2016 08:17 PM, Ben Pfaff wrote:
> Suggested-by: William Tu <u9012063 at gmail.com>
> Signed-off-by: Ben Pfaff <blp at 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 <russell at ovn.org>
--
Russell Bryant
More information about the dev
mailing list