Commit fa307474 authored Oct 17, 2020 by Matthew Wilcox (Oracle) Committed by Linus Torvalds Oct 18, 2020

mm: update the documentation for vfree



Patch series "remove alloc_vm_area", v4.

This series removes alloc_vm_area, which was left over from the big
vmalloc interface rework.  It is a rather arkane interface, basicaly the
equivalent of get_vm_area + actually faulting in all PTEs in the allocated
area.  It was originally addeds for Xen (which isn't modular to start
with), and then grew users in zsmalloc and i915 which seems to mostly
qualify as abuses of the interface, especially for i915 as a random driver
should not set up PTE bits directly.

This patch (of 11):

 * Document that you can call vfree() on an address returned from vmap()
 * Remove the note about the minimum size -- the minimum size of a vmalloc
   allocation is one page
 * Add a Context: section
 * Fix capitalisation
 * Reword the prohibition on calling from NMI context to avoid a double
   negative

Signed-off-by: Matthew Wilcox (Oracle) <willy@infradead.org>
Signed-off-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Boris Ostrovsky <boris.ostrovsky@oracle.com>
Cc: Juergen Gross <jgross@suse.com>
Cc: Stefano Stabellini <sstabellini@kernel.org>
Cc: Jani Nikula <jani.nikula@linux.intel.com>
Cc: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
Cc: Tvrtko Ursulin <tvrtko.ursulin@linux.intel.com>
Cc: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Matthew Auld <matthew.auld@intel.com>
Cc: Rodrigo Vivi <rodrigo.vivi@intel.com>
Cc: Minchan Kim <minchan@kernel.org>
Cc: Matthew Wilcox <willy@infradead.org>
Cc: Nitin Gupta <ngupta@vflare.org>
Cc: Uladzislau Rezki (Sony) <urezki@gmail.com>
Link: https://lkml.kernel.org/r/20201002122204.1534411-1-hch@lst.de
Link: https://lkml.kernel.org/r/20201002122204.1534411-2-hch@lst.de


Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>

parent ecb8ac8b

mm/vmalloc.c

+11 −10

Original line number	Diff line number	Diff line
		@@ -2321,20 +2321,21 @@ static void __vfree(const void *addr)
		}

		/**
		* vfree - release memory allocated by vmalloc()
		* @addr: memory base address
		* vfree - Release memory allocated by vmalloc()
		* @addr: Memory base address
		*
		* Free the virtually continuous memory area starting at @addr, as
		* obtained from vmalloc(), vmalloc_32() or __vmalloc(). If @addr is
		* NULL, no operation is performed.
		* Free the virtually continuous memory area starting at @addr, as obtained
		* from one of the vmalloc() family of APIs. This will usually also free the
		* physical memory underlying the virtual allocation, but that memory is
		* reference counted, so it will not be freed until the last user goes away.
		*
		* Must not be called in NMI context (strictly speaking, only if we don't
		* have CONFIG_ARCH_HAVE_NMI_SAFE_CMPXCHG, but making the calling
		* conventions for vfree() arch-depenedent would be a really bad idea)
		* If @addr is NULL, no operation is performed.
		*
		* Context:
		* May sleep if called not from interrupt context.
		*
		* NOTE: assumes that the object at @addr has a size >= sizeof(llist_node)
		* Must not be called in NMI context (strictly speaking, it could be
		* if we have CONFIG_ARCH_HAVE_NMI_SAFE_CMPXCHG, but making the calling
		* conventions for vfree() arch-depenedent would be a really bad idea).
		*/
		void vfree(const void *addr)
		{

Admin message