On Fri 25-01-13 16:17:55, Glauber Costa wrote:
> From: Glauber Costa <glommer@xxxxxxxxxxxxx>
>
> The macro for_each_memcg_cache_index contains a silly yet potentially
> deadly mistake. Although the macro parameter is _idx, the loop tests are
> done over i, not _idx.
>
> This hasn't generated any problems so far, because all users use i as a
> loop index. However, while playing with an extension of the code I
> ended using another loop index and the compiler was quick to complain.
>
> Unfortunately, this is not the kind of thing that testing reveals =(
>
> Signed-off-by: Glauber Costa <glommer@xxxxxxxxxxxxx>
> Cc: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>
> Cc: Michal Hocko <mhocko@xxxxxxx>
> Cc: Kamezawa Hiroyuki <kamezawa.hiroyu@xxxxxxxxxxxxxx>
> Cc: Johannes Weiner <hannes@xxxxxxxxxxx>
Ouch.
Acked-by: Michal Hocko <mhocko@xxxxxxx>
> ---
> include/linux/memcontrol.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/include/linux/memcontrol.h b/include/linux/memcontrol.h
> index 904084f..2a876a3 100644
> --- a/include/linux/memcontrol.h
> +++ b/include/linux/memcontrol.h
> @@ -424,7 +424,7 @@ extern int memcg_limited_groups_array_size;
> * the slab_mutex must be held when looping through those caches
> */
> #define for_each_memcg_cache_index(_idx) \
> - for ((_idx) = 0; i < memcg_limited_groups_array_size; (_idx)++)
> + for ((_idx) = 0; (_idx) < memcg_limited_groups_array_size; (_idx)++)
>
> static inline bool memcg_kmem_enabled(void)
> {
> --
> 1.8.1
>
--
Michal Hocko
SUSE Labs
--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@xxxxxxxxx. For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>