Re: [PATCH] Document kfree and vfree NULL usage (resend)

[Kernel Stuff]

>   *	May not be called in interrupt context 
Does this need to change to 
      * Must not be called in interrupt context 
?
Is there a case where it is guaranteed that kfree will not sleep? "May" sounds 
to me like "You can call it in interrupt ctx, but it is not recommended", 
whereas calling it in interrupt ctx is definitely not recommended.

Parag

On Sunday 05 December 2004 10:33 am, Manfred Spraul wrote:
> Hi Andrew,
>
> I think it's worth to explicitely mention that kfree(NULL) is valid - too
> many users have/had their own (unnecessary) if(ptr) checks.
>
> Pekka wrote:
> > This patch adds comments for kfree() and vfree() stating that both
> > accept NULL pointers.
> >
> > Signed-off-by: Pekka Enberg <penberg@xxxxxxxxxxxxxx>
>
> Signed-Off-By: Manfred Spraul <manfred@xxxxxxxxxxxxxxxx>
>
> ---
>
>  slab.c    |    2 ++
>  vmalloc.c |    3 ++-
>  2 files changed, 4 insertions(+), 1 deletion(-)
>
> Index: 2.6.10-rc2/mm/slab.c
> ===================================================================
> --- 2.6.10-rc2.orig/mm/slab.c	2004-11-27 14:33:14.000000000 +0200
> +++ 2.6.10-rc2/mm/slab.c	2004-11-27 16:12:54.573387384 +0200
> @@ -2535,6 +2535,8 @@
>   * kfree - free previously allocated memory
>   * @objp: pointer returned by kmalloc.
>   *
> + * If @objp is NULL, no operation is performed.
> + *
>   * Don't free memory not originally allocated by kmalloc()
>   * or you will run into trouble.
>   */
> Index: 2.6.10-rc2/mm/vmalloc.c
> ===================================================================
> --- 2.6.10-rc2.orig/mm/vmalloc.c	2004-11-27 16:13:48.026261312 +0200
> +++ 2.6.10-rc2/mm/vmalloc.c	2004-11-27 16:14:04.875699808 +0200
> @@ -389,7 +389,8 @@
>   *	@addr:		memory base address
>   *
>   *	Free the virtually contiguous memory area starting at @addr, as
> - *	obtained from vmalloc(), vmalloc_32() or __vmalloc().
> + *	obtained from vmalloc(), vmalloc_32() or __vmalloc(). If @addr is
> + *	NULL, no operation is performed.
>   *
>   *	May not be called in interrupt context.
>   */
>
>
>
> -
> 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/
-
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/