Thread (19 messages) 19 messages, 5 authors, 2018-10-23

Re: [PATCH v2 4/7] arm64/modules: Add rlimit checking for arm64 modules

From: Dave Hansen <hidden>
Date: 2018-10-12 07:17:36
Also in: linux-arm-kernel, linux-fsdevel, linux-mips, linux-s390, lkml, sparclinux

On 10/11/2018 04:31 PM, Rick Edgecombe wrote:
quoted hunk ↗ jump to hunk
+	if (check_inc_mod_rlimit(size))
+		return NULL;
+
 	p = __vmalloc_node_range(size, MODULE_ALIGN, module_alloc_base,
 				module_alloc_base + MODULES_VSIZE,
 				gfp_mask, PAGE_KERNEL_EXEC, 0,
@@ -65,6 +68,8 @@ void *module_alloc(unsigned long size)
 		return NULL;
 	}
 
+	update_mod_rlimit(p, size);
Is there a reason we couldn't just rename all of the existing per-arch
module_alloc() calls to be called, say, "arch_module_alloc()", then put
this new rlimit code in a generic helper that does:


	if (check_inc_mod_rlimit(size))
		return NULL;

	p = arch_module_alloc(...);

	...

	update_mod_rlimit(p, size);
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help