Brad Bishop | 1a4b7ee | 2018-12-16 17:11:34 -0800 | [diff] [blame] | 1 | From 0066b862bb3a09f39295abd5d972a53ac8dc1555 Mon Sep 17 00:00:00 2001 |
| 2 | From: Peter Bergin <peter@berginkonsult.se> |
| 3 | Date: Wed, 19 Sep 2018 15:12:31 +0200 |
| 4 | Subject: [PATCH] rpm/rpmio.c: restrict virtual memory usage if limit set |
| 5 | |
| 6 | A solution to avoid OOM situation when the virtual memory is restricted |
| 7 | for a user (ulimit -v). As the lzopen_internal function is run in parallel |
| 8 | one instance per CPU thread the available virtual memory is limited per |
| 9 | CPU thread. |
| 10 | |
| 11 | Upstream-Status: Pending [merge of multithreading patches to upstream] |
| 12 | |
| 13 | Signed-off-by: Peter Bergin <peter@berginkonsult.se> |
Andrew Geissler | 5199d83 | 2021-09-24 16:47:35 -0500 | [diff] [blame] | 14 | Signed-off-by: Ranjitsinh Rathod <ranjitsinh.rathod@kpit.com> |
Brad Bishop | 1a4b7ee | 2018-12-16 17:11:34 -0800 | [diff] [blame] | 15 | --- |
Andrew Geissler | 5199d83 | 2021-09-24 16:47:35 -0500 | [diff] [blame] | 16 | rpmio/rpmio.c | 36 ++++++++++++++++++++++++++++++++++++ |
| 17 | 1 file changed, 36 insertions(+) |
Brad Bishop | 1a4b7ee | 2018-12-16 17:11:34 -0800 | [diff] [blame] | 18 | |
| 19 | diff --git a/rpmio/rpmio.c b/rpmio/rpmio.c |
| 20 | index e051c98..b3c56b6 100644 |
| 21 | --- a/rpmio/rpmio.c |
| 22 | +++ b/rpmio/rpmio.c |
Andrew Geissler | 5199d83 | 2021-09-24 16:47:35 -0500 | [diff] [blame] | 23 | @@ -845,6 +845,42 @@ static LZFILE *lzopen_internal(const char *mode, int fd, int xz) |
Brad Bishop | 1a4b7ee | 2018-12-16 17:11:34 -0800 | [diff] [blame] | 24 | } |
| 25 | #endif |
| 26 | |
Andrew Geissler | 5199d83 | 2021-09-24 16:47:35 -0500 | [diff] [blame] | 27 | + struct rlimit virtual_memory = {RLIM_INFINITY , RLIM_INFINITY}; |
| 28 | + int status = getrlimit(RLIMIT_AS, &virtual_memory); |
| 29 | + if ((status != -1) && (virtual_memory.rlim_cur != RLIM_INFINITY)) { |
Brad Bishop | 1a4b7ee | 2018-12-16 17:11:34 -0800 | [diff] [blame] | 30 | + const uint64_t virtual_memlimit = virtual_memory.rlim_cur; |
Andrew Geissler | 5199d83 | 2021-09-24 16:47:35 -0500 | [diff] [blame] | 31 | + uint32_t threads_max = lzma_cputhreads(); |
Brad Bishop | 1a4b7ee | 2018-12-16 17:11:34 -0800 | [diff] [blame] | 32 | + const uint64_t virtual_memlimit_per_cpu_thread = |
Andrew Geissler | 5199d83 | 2021-09-24 16:47:35 -0500 | [diff] [blame] | 33 | + virtual_memlimit / ((threads_max == 0) ? 1 : threads_max); |
Brad Bishop | 1a4b7ee | 2018-12-16 17:11:34 -0800 | [diff] [blame] | 34 | + rpmlog(RPMLOG_NOTICE, "XZ: virtual memory restricted to %lu and " |
| 35 | + "per CPU thread %lu\n", virtual_memlimit, virtual_memlimit_per_cpu_thread); |
Andrew Geissler | 5199d83 | 2021-09-24 16:47:35 -0500 | [diff] [blame] | 36 | + uint64_t memory_usage_virt; |
Brad Bishop | 1a4b7ee | 2018-12-16 17:11:34 -0800 | [diff] [blame] | 37 | + /* keep reducing the number of compression threads until memory |
| 38 | + usage falls below the limit per CPU thread*/ |
| 39 | + while ((memory_usage_virt = lzma_stream_encoder_mt_memusage(&mt_options)) > |
| 40 | + virtual_memlimit_per_cpu_thread) { |
Andrew Geissler | 5199d83 | 2021-09-24 16:47:35 -0500 | [diff] [blame] | 41 | + /* If number of threads goes down to zero or in case of any other error |
| 42 | + * lzma_stream_encoder_mt_memusage will return UINT64_MAX. We must check |
| 43 | + * for both the cases here to avoid an infinite loop. |
Brad Bishop | 1a4b7ee | 2018-12-16 17:11:34 -0800 | [diff] [blame] | 44 | + * If we get into situation that one thread requires more virtual memory |
| 45 | + * than available we set one thread, print error message and try anyway. */ |
Andrew Geissler | 5199d83 | 2021-09-24 16:47:35 -0500 | [diff] [blame] | 46 | + if ((--mt_options.threads == 0) || (memory_usage_virt == UINT64_MAX)) { |
Brad Bishop | 1a4b7ee | 2018-12-16 17:11:34 -0800 | [diff] [blame] | 47 | + mt_options.threads = 1; |
| 48 | + rpmlog(RPMLOG_WARNING, |
| 49 | + "XZ: Could not adjust number of threads to get below " |
| 50 | + "virtual memory limit %lu. usage %lu\n", |
| 51 | + virtual_memlimit_per_cpu_thread, memory_usage_virt); |
| 52 | + break; |
| 53 | + } |
| 54 | + } |
| 55 | + if (threads != (int)mt_options.threads) |
| 56 | + rpmlog(RPMLOG_NOTICE, |
| 57 | + "XZ: Adjusted the number of threads from %d to %d to not " |
| 58 | + "exceed the memory usage limit of %lu bytes\n", |
| 59 | + threads, mt_options.threads, virtual_memlimit); |
| 60 | + |
| 61 | + } |
| 62 | + |
| 63 | ret = lzma_stream_encoder_mt(&lzfile->strm, &mt_options); |
| 64 | } |
| 65 | #endif |
| 66 | -- |
| 67 | 2.7.4 |
| 68 | |