memset() in spu_init_csa() is superfluous because that memory region is allready zeroed by kzalloc() on allocation (and I haven't seen any other reference to the function beside alloc_spu_context()). Signed-off-by: Sebastian Siewior --- a/arch/powerpc/platforms/cell/spufs/switch.c +++ b/arch/powerpc/platforms/cell/spufs/switch.c @@ -2201,7 +2201,6 @@ int spu_init_csa(struct spu_state *csa) if (!csa) return -EINVAL; - memset(csa, 0, sizeof(struct spu_state)); rc = spu_alloc_lscsa(csa); if (rc) --