Re: [PATCH] hrtimers: conditionally lock/unlock spinlock in hrtimer_get_next_event

From: Stanislav Fomichev
Date: Mon Jun 09 2014 - 07:11:34 EST


> What's wrong with simply doing:
>
> if (!hrtimer_hres_active())
> return mindelta;
Nothing wrong, just makes diff bigger :-)

> That saves and indentation level and makes the code more
> readable. Also the lockless check wants a comment why it is correct.
Ok, I'll follow with updated patch.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/