On Wed, May 09, 2012 at 11:09:00PM -0400, Kent Overstreet wrote:
>
> Signed-off-by: Kent Overstreet <koverstreet@google.com>
> ---
> include/linux/ratelimit.h | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/include/linux/ratelimit.h b/include/linux/ratelimit.h
> index e11ccb4..9ad57d3 100644
> --- a/include/linux/ratelimit.h
> +++ b/include/linux/ratelimit.h
> @@ -20,7 +20,7 @@ struct ratelimit_state {
> #define DEFINE_RATELIMIT_STATE(name, interval_init, burst_init)
>
> struct ratelimit_state name = {
> - .lock = __RAW_SPIN_LOCK_UNLOCKED(name.lock),
> + .lock = __RAW_SPIN_LOCK_INITIALIZER(name.lock),

NACK, kernel is not built in c99 mode and using anonymous struct in
struct initializers is an accepted convention.

Thanks.

--
tejun

--
dm-devel mailing list
dm-devel@redhat.com
https://www.redhat.com/mailman/listinfo/dm-devel