Re: [PATCH] percpu: update comments to reflect that percpuallocations are always zero-filled

From: Stephane Eranian
Date: Fri Sep 10 2010 - 05:07:31 EST


Thanks

On Fri, Sep 10, 2010 at 11:03 AM, Tejun Heo <tj@xxxxxxxxxx> wrote:
> Signed-off-by: Tejun Heo <tj@xxxxxxxxxx>
> Reported-by: Stephane Eranian <eranian@xxxxxxxxxx>
> ---
> Âmm/percpu.c | Â 11 ++++++-----
> Â1 files changed, 6 insertions(+), 5 deletions(-)
>
> diff --git a/mm/percpu.c b/mm/percpu.c
> index 0cd4bf6..12dea33 100644
> --- a/mm/percpu.c
> +++ b/mm/percpu.c
> @@ -826,8 +826,8 @@ fail_unlock_mutex:
> Â* @size: size of area to allocate in bytes
> Â* @align: alignment of area (max PAGE_SIZE)
> Â*
> - * Allocate percpu area of @size bytes aligned at @align. ÂMight
> - * sleep. ÂMight trigger writeouts.
> + * Allocate zero-filled percpu area of @size bytes aligned at @align.
> + * Might sleep. ÂMight trigger writeouts.
> Â*
> Â* CONTEXT:
> Â* Does GFP_KERNEL allocation.
> @@ -846,9 +846,10 @@ EXPORT_SYMBOL_GPL(__alloc_percpu);
> Â* @size: size of area to allocate in bytes
> Â* @align: alignment of area (max PAGE_SIZE)
> Â*
> - * Allocate percpu area of @size bytes aligned at @align from reserved
> - * percpu area if arch has set it up; otherwise, allocation is served
> - * from the same dynamic area. ÂMight sleep. ÂMight trigger writeouts.
> + * Allocate zero-filled percpu area of @size bytes aligned at @align
> + * from reserved percpu area if arch has set it up; otherwise,
> + * allocation is served from the same dynamic area. ÂMight sleep.
> + * Might trigger writeouts.
> Â*
> Â* CONTEXT:
> Â* Does GFP_KERNEL allocation.
> --
> 1.7.1
>
>
--
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/