padata: Handle empty padata cpumasks
authorSteffen Klassert <steffen.klassert@secunet.com>
Wed, 7 Jul 2010 13:31:26 +0000 (15:31 +0200)
committerHerbert Xu <herbert@gondor.apana.org.au>
Wed, 14 Jul 2010 12:29:29 +0000 (20:29 +0800)
commit33e54450683c5e970ac007489d7921ba792d093c
tree1e3d0a7fcc007bfa97c88c68d411b8e1691bb7e5
parentee836555120140f770005b8ce6673c913d1b9a98
padata: Handle empty padata cpumasks

This patch fixes a bug when the padata cpumask does not
intersect with the active cpumask. In this case we get a
division by zero in padata_alloc_pd and we end up with a
useless padata instance. Padata can end up with an empty
cpumask for two reasons:

1. A user removed the last cpu that belongs to the padata
cpumask and the active cpumask.

2. The last cpu that belongs to the padata cpumask and the
active cpumask goes offline.

We introduce a function padata_validate_cpumask to check if the padata
cpumask does intersect with the active cpumask. If the cpumasks do not
intersect we mark the instance as invalid, so it can't be used. We do not
allocate the cpumask dependend recources in this case. This fixes the
division by zero and keeps the padate instance in a consistent state.

It's not possible to trigger this bug by now because the only padata user,
pcrypt uses always the possible cpumask.

Reported-by: Dan Kruchinin <dkruchinin@acm.org>
Signed-off-by: Steffen Klassert <steffen.klassert@secunet.com>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
kernel/padata.c