coding-style.rst: Generic alloc functions do not need OOM logging
authorJoe Perches <joe@perches.com>
Sat, 30 Mar 2019 17:25:03 +0000 (10:25 -0700)
committerJonathan Corbet <corbet@lwn.net>
Mon, 1 Apr 2019 20:58:46 +0000 (14:58 -0600)
Generic allocation functions already emit a dump_stack()
so additional error logging isn't useful.

Document it as such and add a reference to the allocation
API.

Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: Jonathan Corbet <corbet@lwn.net>
Documentation/process/coding-style.rst

index 8ea913e99fa17d563a970ded48079e0de0d29ab3..fa864a51e6ea0dcbcfb1d74a45c25f9e6b57a5f0 100644 (file)
@@ -843,7 +843,8 @@ used.
 The kernel provides the following general purpose memory allocators:
 kmalloc(), kzalloc(), kmalloc_array(), kcalloc(), vmalloc(), and
 vzalloc().  Please refer to the API documentation for further information
-about them.
+about them.  :ref:`Documentation/core-api/memory-allocation.rst
+<memory_allocation>`
 
 The preferred form for passing a size of a struct is the following:
 
@@ -874,6 +875,9 @@ The preferred form for allocating a zeroed array is the following:
 Both forms check for overflow on the allocation size n * sizeof(...),
 and return NULL if that occurred.
 
+These generic allocation functions all emit a stack dump on failure when used
+without __GFP_NOWARN so there is no use in emitting an additional failure
+message when NULL is returned.
 
 15) The inline disease
 ----------------------