From: Francis Deslauriers Date: Tue, 16 Oct 2018 19:23:22 +0000 (-0400) Subject: Fix: implicit declarations caused by buffer size checks. X-Git-Tag: v2.9.11~4 X-Git-Url: https://git.lttng.org./?a=commitdiff_plain;h=b1ddb9416fae4adb1e74d5218cecc4d152e36468;p=lttng-modules.git Fix: implicit declarations caused by buffer size checks. Issue ===== Three kernel functions used in the following commit are unavailable on some supported kernels: commit 1f0ab1eb0409d23de5f67cc588c3ea4cee4d10e0 Prevent allocation of buffers if exceeding available memory * si_mem_available() was added in kernel 4.6 with commit d02bd27. * {set, clear}_current_oom_origin() were added in kernel 3.8 with commit: e1e12d2f Solution ======== Add wrappers around these functions such that older kernels will build with these functions defined as NOP or trivial return value. wrapper_check_enough_free_pages() uses the si_mem_available() kernel function to compute if the number pages requested passed as parameter is smaller than the number of pages available on the machine. If the si_mem_available() kernel function is unavailable, we always return true. wrapper_set_current_oom_origin() function wraps the set_current_oom_origin() kernel function when it is available. If set_current_oom_origin() is unavailable the wrapper is empty. wrapper_clear_current_oom_origin() function wraps the clear_current_oom_origin() kernel function when it is available. If clear_current_oom_origin() is unavailable the wrapper is empty. Drawbacks ========= None. Signed-off-by: Francis Deslauriers Signed-off-by: Mathieu Desnoyers --- diff --git a/lib/ringbuffer/ring_buffer_backend.c b/lib/ringbuffer/ring_buffer_backend.c index d300980d..d430f71c 100644 --- a/lib/ringbuffer/ring_buffer_backend.c +++ b/lib/ringbuffer/ring_buffer_backend.c @@ -25,11 +25,11 @@ #include #include #include -#include #include #include #include +#include #include /* for wrapper_vmalloc_sync_all() */ #include #include @@ -59,11 +59,13 @@ int lib_ring_buffer_backend_allocate(const struct lib_ring_buffer_config *config num_pages = size >> PAGE_SHIFT; /* - * Verify that the number of pages requested for that buffer is smaller - * than the number of available pages on the system. si_mem_available() - * returns an _estimate_ of the number of available pages. + * Verify that there is enough free pages available on the system for + * the current allocation request. + * wrapper_check_enough_free_pages uses si_mem_available() if available + * and returns if there should be enough free pages based on the + * current estimate. */ - if (num_pages > si_mem_available()) + if (!wrapper_check_enough_free_pages(num_pages)) goto not_enough_pages; /* @@ -72,7 +74,7 @@ int lib_ring_buffer_backend_allocate(const struct lib_ring_buffer_config *config * end up running out of memory because of this buffer allocation, we * want to kill the offending app first. */ - set_current_oom_origin(); + wrapper_set_current_oom_origin(); num_pages_per_subbuf = num_pages >> get_count_order(num_subbuf); subbuf_size = chanb->subbuf_size; @@ -168,7 +170,7 @@ int lib_ring_buffer_backend_allocate(const struct lib_ring_buffer_config *config * will not fault. */ wrapper_vmalloc_sync_all(); - clear_current_oom_origin(); + wrapper_clear_current_oom_origin(); vfree(pages); return 0; @@ -185,7 +187,7 @@ depopulate: array_error: vfree(pages); pages_error: - clear_current_oom_origin(); + wrapper_clear_current_oom_origin(); not_enough_pages: return -ENOMEM; } diff --git a/wrapper/mm.h b/wrapper/mm.h new file mode 100644 index 00000000..672855b1 --- /dev/null +++ b/wrapper/mm.h @@ -0,0 +1,70 @@ +/* SPDX-License-Identifier: (GPL-2.0 or LGPL-2.1) + * + * wrapper/mm.h + * + * Copyright (C) 2018 Francis Deslauriers + */ + +#ifndef _LTTNG_WRAPPER_MM_H +#define _LTTNG_WRAPPER_MM_H + +#include +#include + +#include + +#if (LINUX_VERSION_CODE >= KERNEL_VERSION(4,6,0) \ + || LTTNG_UBUNTU_KERNEL_RANGE(4,4,25,44, 4,5,0,0)) + +/* + * Returns true if the current estimation of the number of page available is + * larger than the number of pages passed as parameter. + */ +static inline +bool wrapper_check_enough_free_pages(unsigned long num_pages) +{ + return num_pages < si_mem_available(); +} + +#else + +static inline +bool wrapper_check_enough_free_pages(unsigned long num_pages) +{ + /* + * The si_mem_available function is not available on this kernel. Since + * we can't reliably know if there is enough memory available, so we + * return true. + */ + return true; +} +#endif + +#if (LINUX_VERSION_CODE >= KERNEL_VERSION(3,8,0)) +static inline +void wrapper_set_current_oom_origin(void) +{ + return set_current_oom_origin(); +} + +static inline +void wrapper_clear_current_oom_origin(void) +{ + return clear_current_oom_origin(); +} + +#else /* #if (LINUX_VERSION_CODE >= KERNEL_VERSION(3,8,0)) */ + +static inline +void wrapper_set_current_oom_origin(void) +{ + return; +} + +static inline +void wrapper_clear_current_oom_origin() +{ + return; +} +#endif /* #else #if (LINUX_VERSION_CODE >= KERNEL_VERSION(3,8,0)) */ +#endif /* _LTTNG_WRAPPER_MM_H */