70c63e98f9
- mark more subpackages as !bootstrap for systemd-mini usage. - spec : remove --with-firmware-path, firmware loader was removed in v217 - spec: remove --disable-multi-seat-x, gone.(fixed in xorg) - spec: Do not enable systemd-readahead-collect.service and systemd-readahead-replay.service as these do not exist anymore. - spec: drop timedate-add-support-for-openSUSE-version-of-etc-sysconfig.patch Yast was fixed to write all timezone changes exactly how timedated expects things to be done. - spec: remove handle-etc-HOSTNAME.patch, since late 2014 the netcfg package handles the migration from /etc/HOSTNAME to /etc/hostname and owns both files. -spec: remove boot.udev and systemd-journald.init as they currently serve no purpose. - suse-sysv-bootd-support.diff: Remove HAVE_SYSVINIT conditions, we are in sysvcompat-only codepath, also remove the code targetting other distributions, never compiled as the TARGET_$DISTRO macros are never defined. - systemd-powerd-initctl-support.patch guard with HAVE_SYSV_COMPAT - set-and-use-default-logconsole.patch: fix HAVE_SYSV_COMPAT guards - insserv-generator.patch: Only build when sysvcompat is enabled - vhangup-on-all-consoles.patch add a comment indicating this is a workaround for a kernel bug. - spec: Add option to allow disabling sysvinit compat at build time. - spec: Add option to enable resolved at build time. - spec: Remove all %ifs for !factory products, current systemd releases can neither be built nor installed in older products without upgrading several components of the base system. (removed: 1008-add-msft-compability-rules.patch was only for =< 13.1) - spec: remove all dummy "aliases" to /etc/init.d, that made sense only when those init scripts still existed. (dummy localfs.service source: gone) OBS-URL: https://build.opensuse.org/request/show/286653 OBS-URL: https://build.opensuse.org/package/show/openSUSE:Factory/systemd?expand=0&rev=215
139 lines
5.2 KiB
Diff
139 lines
5.2 KiB
Diff
From: werner@suse.de
|
|
Upstream: marked as openSUSE-specific fix originally, but may well
|
|
worth be to be sent upstream.
|
|
|
|
Original description:
|
|
(none)
|
|
|
|
Notes [jengelh@inai.de]:
|
|
MAP_NONBLOCK is documented (mmap(2) in man-pages-3.69+) to render
|
|
MAP_POPULATE ineffective. Why do we have it?
|
|
What is the rationale for MAP_STACK?
|
|
---
|
|
src/journal/catalog.c | 12 +++++++++++-
|
|
src/journal/journal-authenticate.c | 7 +++++--
|
|
src/journal/journald-kmsg.c | 3 ++-
|
|
src/journal/mmap-cache.c | 16 +++++++++++++++-
|
|
4 files changed, 33 insertions(+), 5 deletions(-)
|
|
|
|
Index: systemd/src/journal/catalog.c
|
|
===================================================================
|
|
--- systemd.orig/src/journal/catalog.c
|
|
+++ systemd/src/journal/catalog.c
|
|
@@ -470,6 +470,10 @@ finish:
|
|
|
|
static int open_mmap(const char *database, int *_fd, struct stat *_st, void **_p) {
|
|
const CatalogHeader *h;
|
|
+ static const unsigned int advice[] =
|
|
+ {MADV_WILLNEED, MADV_SEQUENTIAL, MADV_DONTDUMP, MADV_DONTFORK};
|
|
+ unsigned int n;
|
|
+ size_t psize;
|
|
int fd;
|
|
void *p;
|
|
struct stat st;
|
|
@@ -492,12 +496,18 @@ static int open_mmap(const char *databas
|
|
return -EINVAL;
|
|
}
|
|
|
|
- p = mmap(NULL, PAGE_ALIGN(st.st_size), PROT_READ, MAP_SHARED, fd, 0);
|
|
+ psize = PAGE_ALIGN(st.st_size);
|
|
+ p = mmap(NULL, psize, PROT_READ, MAP_SHARED | MAP_POPULATE |
|
|
+ MAP_NONBLOCK, fd, 0);
|
|
if (p == MAP_FAILED) {
|
|
safe_close(fd);
|
|
return -errno;
|
|
}
|
|
|
|
+ for (n = 0; n < sizeof(advice) / sizeof(*advice); ++n)
|
|
+ if (madvise(p, psize, advice[n]) < 0)
|
|
+ log_warning("Failed to give advice about use of memory: %m");
|
|
+
|
|
h = p;
|
|
if (memcmp(h->signature, CATALOG_SIGNATURE, sizeof(h->signature)) != 0 ||
|
|
le64toh(h->header_size) < sizeof(CatalogHeader) ||
|
|
Index: systemd/src/journal/journal-authenticate.c
|
|
===================================================================
|
|
--- systemd.orig/src/journal/journal-authenticate.c
|
|
+++ systemd/src/journal/journal-authenticate.c
|
|
@@ -355,7 +355,8 @@ int journal_file_fss_load(JournalFile *f
|
|
goto finish;
|
|
}
|
|
|
|
- m = mmap(NULL, PAGE_ALIGN(sizeof(FSSHeader)), PROT_READ, MAP_SHARED, fd, 0);
|
|
+ m = mmap(NULL, PAGE_ALIGN(sizeof(FSSHeader)), PROT_READ,
|
|
+ MAP_SHARED | MAP_STACK, fd, 0);
|
|
if (m == MAP_FAILED) {
|
|
m = NULL;
|
|
r = -errno;
|
|
@@ -399,7 +400,9 @@ int journal_file_fss_load(JournalFile *f
|
|
goto finish;
|
|
}
|
|
|
|
- f->fss_file = mmap(NULL, PAGE_ALIGN(f->fss_file_size), PROT_READ|PROT_WRITE, MAP_SHARED, fd, 0);
|
|
+ f->fss_file = mmap(NULL, PAGE_ALIGN(f->fss_file_size),
|
|
+ PROT_READ | PROT_WRITE, MAP_SHARED | MAP_STACK,
|
|
+ fd, 0);
|
|
if (f->fss_file == MAP_FAILED) {
|
|
f->fss_file = NULL;
|
|
r = -errno;
|
|
Index: systemd/src/journal/journald-kmsg.c
|
|
===================================================================
|
|
--- systemd.orig/src/journal/journald-kmsg.c
|
|
+++ systemd/src/journal/journald-kmsg.c
|
|
@@ -454,7 +454,8 @@ int server_open_kernel_seqnum(Server *s)
|
|
return 0;
|
|
}
|
|
|
|
- p = mmap(NULL, sizeof(uint64_t), PROT_READ|PROT_WRITE, MAP_SHARED, fd, 0);
|
|
+ p = mmap(NULL, sizeof(uint64_t), PROT_READ | PROT_WRITE,
|
|
+ MAP_SHARED | MAP_STACK, fd, 0);
|
|
if (p == MAP_FAILED) {
|
|
log_error_errno(errno, "Failed to map sequential number file, ignoring: %m");
|
|
return 0;
|
|
Index: systemd/src/journal/mmap-cache.c
|
|
===================================================================
|
|
--- systemd.orig/src/journal/mmap-cache.c
|
|
+++ systemd/src/journal/mmap-cache.c
|
|
@@ -467,11 +467,14 @@ static int add_mmap(
|
|
struct stat *st,
|
|
void **ret) {
|
|
|
|
+ static const unsigned int advice[] =
|
|
+ {MADV_WILLNEED, MADV_SEQUENTIAL, MADV_DONTDUMP, MADV_DONTFORK};
|
|
uint64_t woffset, wsize;
|
|
Context *c;
|
|
FileDescriptor *f;
|
|
Window *w;
|
|
void *d;
|
|
+ unsigned int n;
|
|
int r;
|
|
|
|
assert(m);
|
|
@@ -510,7 +513,8 @@ static int add_mmap(
|
|
}
|
|
|
|
for (;;) {
|
|
- d = mmap(NULL, wsize, prot, MAP_SHARED, fd, woffset);
|
|
+ d = mmap(NULL, wsize, prot, MAP_SHARED | MAP_POPULATE |
|
|
+ MAP_NONBLOCK, fd, woffset);
|
|
if (d != MAP_FAILED)
|
|
break;
|
|
if (errno != ENOMEM)
|
|
@@ -523,6 +527,16 @@ static int add_mmap(
|
|
return -ENOMEM;
|
|
}
|
|
|
|
+ for (n = 0; n < sizeof(advice) / sizeof(*advice); ++n) {
|
|
+ if (advice[n] == MADV_DONTFORK) {
|
|
+ int flags = fcntl(fd, F_GETFD);
|
|
+ if (flags < 0 || !(flags & FD_CLOEXEC))
|
|
+ continue;
|
|
+ }
|
|
+ if (madvise(d, wsize, advice[n]) < 0)
|
|
+ log_warning("Failed to give advice about use of memory: %m");
|
|
+ }
|
|
+
|
|
c = context_add(m, context);
|
|
if (!c)
|
|
goto outofmem;
|