* [for-dora] systemd: journald fix ignored disk space restrictions
@ 2014-01-31 20:04 Diego Sueiro
2014-02-01 7:42 ` Martin Jansa
0 siblings, 1 reply; 8+ messages in thread
From: Diego Sueiro @ 2014-01-31 20:04 UTC (permalink / raw)
To: openembedded-devel
The upstream bug report can be seen at:
[Systemd #68161] -- https://bugs.freedesktop.org/show_bug.cgi?id=68161
This backports the patches need to address this in the 206 version.
Signed-off-by: Diego Sueiro <diego.sueiro@gmail.com>
---
...ournald-fix-fd-leak-in-journal_file_empty.patch | 27 ++++++++++++++
...urnald-fix-vacuuming-of-archived-journals.patch | 37 ++++++++++++++++++++
meta/recipes-core/systemd/systemd_206.bb | 2 ++
3 files changed, 66 insertions(+)
create mode 100644 meta/recipes-core/systemd/systemd/journald-fix-fd-leak-in-journal_file_empty.patch
create mode 100644 meta/recipes-core/systemd/systemd/journald-fix-vacuuming-of-archived-journals.patch
diff --git a/meta/recipes-core/systemd/systemd/journald-fix-fd-leak-in-journal_file_empty.patch b/meta/recipes-core/systemd/systemd/journald-fix-fd-leak-in-journal_file_empty.patch
new file mode 100644
index 0000000..a83c858
--- /dev/null
+++ b/meta/recipes-core/systemd/systemd/journald-fix-fd-leak-in-journal_file_empty.patch
@@ -0,0 +1,27 @@
+journald: fix fd leak in journal_file_empty
+Before my previous patch, journal_file_empty wasn't be called with the
+correct filename. Now that it's being called with the correct filename
+it leaks file descriptors. This patch closes the file descriptors before
+returning.
+
+Signed-off-by: George McCollister <george.mccollister@gmail.com>
+
+
+Index: systemd-206/src/journal/journal-vacuum.c
+===================================================================
+--- systemd-206.orig/src/journal/journal-vacuum.c 2013-07-21 19:43:28.146183128 -0300
++++ systemd-206/src/journal/journal-vacuum.c 2014-01-29 20:43:27.492343295 -0200
+@@ -129,9 +129,9 @@
+ }
+
+ static int journal_file_empty(int dir_fd, const char *name) {
+- int fd, r;
++ int r;
+ le64_t n_entries;
+-
++ _cleanup_close_ int fd;
+ fd = openat(dir_fd, name, O_RDONLY|O_CLOEXEC|O_NOFOLLOW|O_NONBLOCK);
+ if (fd < 0)
+ return -errno;
+
+Upstream-Status: Backport [207]
diff --git a/meta/recipes-core/systemd/systemd/journald-fix-vacuuming-of-archived-journals.patch b/meta/recipes-core/systemd/systemd/journald-fix-vacuuming-of-archived-journals.patch
new file mode 100644
index 0000000..7c59200
--- /dev/null
+++ b/meta/recipes-core/systemd/systemd/journald-fix-vacuuming-of-archived-journals.patch
@@ -0,0 +1,37 @@
+journald: fix vacuuming of archived journals
+d_name is modified on line 227 so if the entire journal name is needed
+again p must be used. Before this change when journal_file_empty was called
+on archived journals it would always return with -2.
+
+Signed-off-by: George McCollister <george.mccollister@gmail.com>
+
+Index: systemd-206/src/journal/journal-vacuum.c
+===================================================================
+--- systemd-206.orig/src/journal/journal-vacuum.c 2014-01-29 20:43:27.492343295 -0200
++++ systemd-206/src/journal/journal-vacuum.c 2014-01-29 20:48:55.076356876 -0200
+@@ -265,18 +265,18 @@
+ /* We do not vacuum active files or unknown files! */
+ continue;
+
+- if (journal_file_empty(dirfd(d), de->d_name)) {
++ if (journal_file_empty(dirfd(d), p)) {
+
+ /* Always vacuum empty non-online files. */
+
+- if (unlinkat(dirfd(d), de->d_name, 0) >= 0)
+- log_debug("Deleted empty journal %s/%s.", directory, de->d_name);
++ if (unlinkat(dirfd(d), p, 0) >= 0)
++ log_debug("Deleted empty journal %s/%s.", directory, p);
+ else if (errno != ENOENT)
+- log_warning("Failed to delete %s/%s: %m", directory, de->d_name);
++ log_warning("Failed to delete %s/%s: %m", directory, p);
+ continue;
+ }
+
+- patch_realtime(directory, de->d_name, &st, &realtime);
++ patch_realtime(directory, p, &st, &realtime);
+
+ GREEDY_REALLOC(list, n_allocated, n_list + 1);
+
+
+Upstream-Status: Backport [207]
diff --git a/meta/recipes-core/systemd/systemd_206.bb b/meta/recipes-core/systemd/systemd_206.bb
index a1a8bd7..e7d0f1f 100644
--- a/meta/recipes-core/systemd/systemd_206.bb
+++ b/meta/recipes-core/systemd/systemd_206.bb
@@ -21,6 +21,8 @@ SRC_URI = "http://www.freedesktop.org/software/systemd/systemd-${PV}.tar.xz \
file://0001-use-CAP_MKNOD-ConditionCapability.patch \
file://0001-Use-bin-mkdir-instead-of-host-mkdir-path.patch \
file://binfmt-install.patch \
+ file://journald-fix-fd-leak-in-journal_file_empty.patch \
+ file://journald-fix-vacuuming-of-archived-journals.patch \
file://touchscreen.rules \
${UCLIBCPATCHES} \
file://00-create-volatile.conf \
--
1.7.10.4
^ permalink raw reply related [flat|nested] 8+ messages in thread* Re: [for-dora] systemd: journald fix ignored disk space restrictions
2014-01-31 20:04 [for-dora] systemd: journald fix ignored disk space restrictions Diego Sueiro
@ 2014-02-01 7:42 ` Martin Jansa
2014-02-01 9:02 ` Diego Sueiro
0 siblings, 1 reply; 8+ messages in thread
From: Martin Jansa @ 2014-02-01 7:42 UTC (permalink / raw)
To: openembedded-devel
[-- Attachment #1: Type: text/plain, Size: 5869 bytes --]
On Fri, Jan 31, 2014 at 06:04:04PM -0200, Diego Sueiro wrote:
> The upstream bug report can be seen at:
> [Systemd #68161] -- https://bugs.freedesktop.org/show_bug.cgi?id=68161
>
> This backports the patches need to address this in the 206 version.
Hello Diego, this belongs to openembedded-core ML.
It's not clear from the bug report, but these patches are included in
208 release right? Please check and mention it in commit message to
explain that this isn't needed in oe-core/master.
Thanks
> Signed-off-by: Diego Sueiro <diego.sueiro@gmail.com>
> ---
> ...ournald-fix-fd-leak-in-journal_file_empty.patch | 27 ++++++++++++++
> ...urnald-fix-vacuuming-of-archived-journals.patch | 37 ++++++++++++++++++++
> meta/recipes-core/systemd/systemd_206.bb | 2 ++
> 3 files changed, 66 insertions(+)
> create mode 100644 meta/recipes-core/systemd/systemd/journald-fix-fd-leak-in-journal_file_empty.patch
> create mode 100644 meta/recipes-core/systemd/systemd/journald-fix-vacuuming-of-archived-journals.patch
>
> diff --git a/meta/recipes-core/systemd/systemd/journald-fix-fd-leak-in-journal_file_empty.patch b/meta/recipes-core/systemd/systemd/journald-fix-fd-leak-in-journal_file_empty.patch
> new file mode 100644
> index 0000000..a83c858
> --- /dev/null
> +++ b/meta/recipes-core/systemd/systemd/journald-fix-fd-leak-in-journal_file_empty.patch
> @@ -0,0 +1,27 @@
> +journald: fix fd leak in journal_file_empty
> +Before my previous patch, journal_file_empty wasn't be called with the
> +correct filename. Now that it's being called with the correct filename
> +it leaks file descriptors. This patch closes the file descriptors before
> +returning.
> +
> +Signed-off-by: George McCollister <george.mccollister@gmail.com>
> +
> +
> +Index: systemd-206/src/journal/journal-vacuum.c
> +===================================================================
> +--- systemd-206.orig/src/journal/journal-vacuum.c 2013-07-21 19:43:28.146183128 -0300
> ++++ systemd-206/src/journal/journal-vacuum.c 2014-01-29 20:43:27.492343295 -0200
> +@@ -129,9 +129,9 @@
> + }
> +
> + static int journal_file_empty(int dir_fd, const char *name) {
> +- int fd, r;
> ++ int r;
> + le64_t n_entries;
> +-
> ++ _cleanup_close_ int fd;
> + fd = openat(dir_fd, name, O_RDONLY|O_CLOEXEC|O_NOFOLLOW|O_NONBLOCK);
> + if (fd < 0)
> + return -errno;
> +
> +Upstream-Status: Backport [207]
> diff --git a/meta/recipes-core/systemd/systemd/journald-fix-vacuuming-of-archived-journals.patch b/meta/recipes-core/systemd/systemd/journald-fix-vacuuming-of-archived-journals.patch
> new file mode 100644
> index 0000000..7c59200
> --- /dev/null
> +++ b/meta/recipes-core/systemd/systemd/journald-fix-vacuuming-of-archived-journals.patch
> @@ -0,0 +1,37 @@
> +journald: fix vacuuming of archived journals
> +d_name is modified on line 227 so if the entire journal name is needed
> +again p must be used. Before this change when journal_file_empty was called
> +on archived journals it would always return with -2.
> +
> +Signed-off-by: George McCollister <george.mccollister@gmail.com>
> +
> +Index: systemd-206/src/journal/journal-vacuum.c
> +===================================================================
> +--- systemd-206.orig/src/journal/journal-vacuum.c 2014-01-29 20:43:27.492343295 -0200
> ++++ systemd-206/src/journal/journal-vacuum.c 2014-01-29 20:48:55.076356876 -0200
> +@@ -265,18 +265,18 @@
> + /* We do not vacuum active files or unknown files! */
> + continue;
> +
> +- if (journal_file_empty(dirfd(d), de->d_name)) {
> ++ if (journal_file_empty(dirfd(d), p)) {
> +
> + /* Always vacuum empty non-online files. */
> +
> +- if (unlinkat(dirfd(d), de->d_name, 0) >= 0)
> +- log_debug("Deleted empty journal %s/%s.", directory, de->d_name);
> ++ if (unlinkat(dirfd(d), p, 0) >= 0)
> ++ log_debug("Deleted empty journal %s/%s.", directory, p);
> + else if (errno != ENOENT)
> +- log_warning("Failed to delete %s/%s: %m", directory, de->d_name);
> ++ log_warning("Failed to delete %s/%s: %m", directory, p);
> + continue;
> + }
> +
> +- patch_realtime(directory, de->d_name, &st, &realtime);
> ++ patch_realtime(directory, p, &st, &realtime);
> +
> + GREEDY_REALLOC(list, n_allocated, n_list + 1);
> +
> +
> +Upstream-Status: Backport [207]
> diff --git a/meta/recipes-core/systemd/systemd_206.bb b/meta/recipes-core/systemd/systemd_206.bb
> index a1a8bd7..e7d0f1f 100644
> --- a/meta/recipes-core/systemd/systemd_206.bb
> +++ b/meta/recipes-core/systemd/systemd_206.bb
> @@ -21,6 +21,8 @@ SRC_URI = "http://www.freedesktop.org/software/systemd/systemd-${PV}.tar.xz \
> file://0001-use-CAP_MKNOD-ConditionCapability.patch \
> file://0001-Use-bin-mkdir-instead-of-host-mkdir-path.patch \
> file://binfmt-install.patch \
> + file://journald-fix-fd-leak-in-journal_file_empty.patch \
> + file://journald-fix-vacuuming-of-archived-journals.patch \
> file://touchscreen.rules \
> ${UCLIBCPATCHES} \
> file://00-create-volatile.conf \
> --
> 1.7.10.4
>
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-devel
--
Martin 'JaMa' Jansa jabber: Martin.Jansa@gmail.com
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 205 bytes --]
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: [for-dora] systemd: journald fix ignored disk space restrictions
2014-02-01 7:42 ` Martin Jansa
@ 2014-02-01 9:02 ` Diego Sueiro
0 siblings, 0 replies; 8+ messages in thread
From: Diego Sueiro @ 2014-02-01 9:02 UTC (permalink / raw)
To: openembedded-devel
On Sat, Feb 1, 2014 at 5:42 AM, Martin Jansa <martin.jansa@gmail.com> wrote:
> Hello Diego, this belongs to openembedded-core ML.
>
Sorry about this.
>
> It's not clear from the bug report, but these patches are included in
> 208 release right?
>
I believe that these patches are included in 207 release.
> Please check and mention it in commit message to
> explain that this isn't needed in oe-core/master.
>
Ok.
Thanks for reviewing it.
Regards,
--
*dS
Diego Sueiro
Administrador do Embarcados
www.embarcados.com.br
/*long live rock 'n roll*/
^ permalink raw reply [flat|nested] 8+ messages in thread
* [for-dora] systemd: journald fix ignored disk space restrictions
@ 2014-02-01 9:07 Diego Sueiro
2014-02-06 10:34 ` Diego Sueiro
0 siblings, 1 reply; 8+ messages in thread
From: Diego Sueiro @ 2014-02-01 9:07 UTC (permalink / raw)
To: openembedded-core
The upstream bug report can be seen at:
[Systemd #68161] -- https://bugs.freedesktop.org/show_bug.cgi?id=68161
This backports patches come from 207 and need to address this in the 206 version for dora branch.
Signed-off-by: Diego Sueiro <diego.sueiro@gmail.com>
---
...ournald-fix-fd-leak-in-journal_file_empty.patch | 27 ++++++++++++++
...urnald-fix-vacuuming-of-archived-journals.patch | 37 ++++++++++++++++++++
meta/recipes-core/systemd/systemd_206.bb | 2 ++
3 files changed, 66 insertions(+)
create mode 100644 meta/recipes-core/systemd/systemd/journald-fix-fd-leak-in-journal_file_empty.patch
create mode 100644 meta/recipes-core/systemd/systemd/journald-fix-vacuuming-of-archived-journals.patch
diff --git a/meta/recipes-core/systemd/systemd/journald-fix-fd-leak-in-journal_file_empty.patch b/meta/recipes-core/systemd/systemd/journald-fix-fd-leak-in-journal_file_empty.patch
new file mode 100644
index 0000000..a83c858
--- /dev/null
+++ b/meta/recipes-core/systemd/systemd/journald-fix-fd-leak-in-journal_file_empty.patch
@@ -0,0 +1,27 @@
+journald: fix fd leak in journal_file_empty
+Before my previous patch, journal_file_empty wasn't be called with the
+correct filename. Now that it's being called with the correct filename
+it leaks file descriptors. This patch closes the file descriptors before
+returning.
+
+Signed-off-by: George McCollister <george.mccollister@gmail.com>
+
+
+Index: systemd-206/src/journal/journal-vacuum.c
+===================================================================
+--- systemd-206.orig/src/journal/journal-vacuum.c 2013-07-21 19:43:28.146183128 -0300
++++ systemd-206/src/journal/journal-vacuum.c 2014-01-29 20:43:27.492343295 -0200
+@@ -129,9 +129,9 @@
+ }
+
+ static int journal_file_empty(int dir_fd, const char *name) {
+- int fd, r;
++ int r;
+ le64_t n_entries;
+-
++ _cleanup_close_ int fd;
+ fd = openat(dir_fd, name, O_RDONLY|O_CLOEXEC|O_NOFOLLOW|O_NONBLOCK);
+ if (fd < 0)
+ return -errno;
+
+Upstream-Status: Backport [207]
diff --git a/meta/recipes-core/systemd/systemd/journald-fix-vacuuming-of-archived-journals.patch b/meta/recipes-core/systemd/systemd/journald-fix-vacuuming-of-archived-journals.patch
new file mode 100644
index 0000000..7c59200
--- /dev/null
+++ b/meta/recipes-core/systemd/systemd/journald-fix-vacuuming-of-archived-journals.patch
@@ -0,0 +1,37 @@
+journald: fix vacuuming of archived journals
+d_name is modified on line 227 so if the entire journal name is needed
+again p must be used. Before this change when journal_file_empty was called
+on archived journals it would always return with -2.
+
+Signed-off-by: George McCollister <george.mccollister@gmail.com>
+
+Index: systemd-206/src/journal/journal-vacuum.c
+===================================================================
+--- systemd-206.orig/src/journal/journal-vacuum.c 2014-01-29 20:43:27.492343295 -0200
++++ systemd-206/src/journal/journal-vacuum.c 2014-01-29 20:48:55.076356876 -0200
+@@ -265,18 +265,18 @@
+ /* We do not vacuum active files or unknown files! */
+ continue;
+
+- if (journal_file_empty(dirfd(d), de->d_name)) {
++ if (journal_file_empty(dirfd(d), p)) {
+
+ /* Always vacuum empty non-online files. */
+
+- if (unlinkat(dirfd(d), de->d_name, 0) >= 0)
+- log_debug("Deleted empty journal %s/%s.", directory, de->d_name);
++ if (unlinkat(dirfd(d), p, 0) >= 0)
++ log_debug("Deleted empty journal %s/%s.", directory, p);
+ else if (errno != ENOENT)
+- log_warning("Failed to delete %s/%s: %m", directory, de->d_name);
++ log_warning("Failed to delete %s/%s: %m", directory, p);
+ continue;
+ }
+
+- patch_realtime(directory, de->d_name, &st, &realtime);
++ patch_realtime(directory, p, &st, &realtime);
+
+ GREEDY_REALLOC(list, n_allocated, n_list + 1);
+
+
+Upstream-Status: Backport [207]
diff --git a/meta/recipes-core/systemd/systemd_206.bb b/meta/recipes-core/systemd/systemd_206.bb
index a1a8bd7..e7d0f1f 100644
--- a/meta/recipes-core/systemd/systemd_206.bb
+++ b/meta/recipes-core/systemd/systemd_206.bb
@@ -21,6 +21,8 @@ SRC_URI = "http://www.freedesktop.org/software/systemd/systemd-${PV}.tar.xz \
file://0001-use-CAP_MKNOD-ConditionCapability.patch \
file://0001-Use-bin-mkdir-instead-of-host-mkdir-path.patch \
file://binfmt-install.patch \
+ file://journald-fix-fd-leak-in-journal_file_empty.patch \
+ file://journald-fix-vacuuming-of-archived-journals.patch \
file://touchscreen.rules \
${UCLIBCPATCHES} \
file://00-create-volatile.conf \
--
1.7.10.4
^ permalink raw reply related [flat|nested] 8+ messages in thread* Re: [for-dora] systemd: journald fix ignored disk space restrictions
2014-02-01 9:07 Diego Sueiro
@ 2014-02-06 10:34 ` Diego Sueiro
2014-02-06 18:49 ` Randy MacLeod
0 siblings, 1 reply; 8+ messages in thread
From: Diego Sueiro @ 2014-02-06 10:34 UTC (permalink / raw)
To: openembedded-core
[-- Attachment #1: Type: text/plain, Size: 5645 bytes --]
Folks,
Any chance to get this up-streamed?
Regards,
--
*dS
Diego Sueiro
Administrador do Embarcados
www.embarcados.com.br
/*long live rock 'n roll*/
On Sat, Feb 1, 2014 at 7:07 AM, Diego Sueiro <diego.sueiro@gmail.com> wrote:
> The upstream bug report can be seen at:
> [Systemd #68161] -- https://bugs.freedesktop.org/show_bug.cgi?id=68161
>
> This backports patches come from 207 and need to address this in the 206
> version for dora branch.
>
> Signed-off-by: Diego Sueiro <diego.sueiro@gmail.com>
> ---
> ...ournald-fix-fd-leak-in-journal_file_empty.patch | 27 ++++++++++++++
> ...urnald-fix-vacuuming-of-archived-journals.patch | 37
> ++++++++++++++++++++
> meta/recipes-core/systemd/systemd_206.bb | 2 ++
> 3 files changed, 66 insertions(+)
> create mode 100644
> meta/recipes-core/systemd/systemd/journald-fix-fd-leak-in-journal_file_empty.patch
> create mode 100644
> meta/recipes-core/systemd/systemd/journald-fix-vacuuming-of-archived-journals.patch
>
> diff --git
> a/meta/recipes-core/systemd/systemd/journald-fix-fd-leak-in-journal_file_empty.patch
> b/meta/recipes-core/systemd/systemd/journald-fix-fd-leak-in-journal_file_empty.patch
> new file mode 100644
> index 0000000..a83c858
> --- /dev/null
> +++
> b/meta/recipes-core/systemd/systemd/journald-fix-fd-leak-in-journal_file_empty.patch
> @@ -0,0 +1,27 @@
> +journald: fix fd leak in journal_file_empty
> +Before my previous patch, journal_file_empty wasn't be called with the
> +correct filename. Now that it's being called with the correct filename
> +it leaks file descriptors. This patch closes the file descriptors before
> +returning.
> +
> +Signed-off-by: George McCollister <george.mccollister@gmail.com>
> +
> +
> +Index: systemd-206/src/journal/journal-vacuum.c
> +===================================================================
> +--- systemd-206.orig/src/journal/journal-vacuum.c 2013-07-21
> 19:43:28.146183128 -0300
> ++++ systemd-206/src/journal/journal-vacuum.c 2014-01-29
> 20:43:27.492343295 -0200
> +@@ -129,9 +129,9 @@
> + }
> +
> + static int journal_file_empty(int dir_fd, const char *name) {
> +- int fd, r;
> ++ int r;
> + le64_t n_entries;
> +-
> ++ _cleanup_close_ int fd;
> + fd = openat(dir_fd, name,
> O_RDONLY|O_CLOEXEC|O_NOFOLLOW|O_NONBLOCK);
> + if (fd < 0)
> + return -errno;
> +
> +Upstream-Status: Backport [207]
> diff --git
> a/meta/recipes-core/systemd/systemd/journald-fix-vacuuming-of-archived-journals.patch
> b/meta/recipes-core/systemd/systemd/journald-fix-vacuuming-of-archived-journals.patch
> new file mode 100644
> index 0000000..7c59200
> --- /dev/null
> +++
> b/meta/recipes-core/systemd/systemd/journald-fix-vacuuming-of-archived-journals.patch
> @@ -0,0 +1,37 @@
> +journald: fix vacuuming of archived journals
> +d_name is modified on line 227 so if the entire journal name is needed
> +again p must be used. Before this change when journal_file_empty was
> called
> +on archived journals it would always return with -2.
> +
> +Signed-off-by: George McCollister <george.mccollister@gmail.com>
> +
> +Index: systemd-206/src/journal/journal-vacuum.c
> +===================================================================
> +--- systemd-206.orig/src/journal/journal-vacuum.c 2014-01-29
> 20:43:27.492343295 -0200
> ++++ systemd-206/src/journal/journal-vacuum.c 2014-01-29
> 20:48:55.076356876 -0200
> +@@ -265,18 +265,18 @@
> + /* We do not vacuum active files or unknown
> files! */
> + continue;
> +
> +- if (journal_file_empty(dirfd(d), de->d_name)) {
> ++ if (journal_file_empty(dirfd(d), p)) {
> +
> + /* Always vacuum empty non-online files. */
> +
> +- if (unlinkat(dirfd(d), de->d_name, 0) >= 0)
> +- log_debug("Deleted empty journal
> %s/%s.", directory, de->d_name);
> ++ if (unlinkat(dirfd(d), p, 0) >= 0)
> ++ log_debug("Deleted empty journal
> %s/%s.", directory, p);
> + else if (errno != ENOENT)
> +- log_warning("Failed to delete %s/%s:
> %m", directory, de->d_name);
> ++ log_warning("Failed to delete %s/%s:
> %m", directory, p);
> + continue;
> + }
> +
> +- patch_realtime(directory, de->d_name, &st, &realtime);
> ++ patch_realtime(directory, p, &st, &realtime);
> +
> + GREEDY_REALLOC(list, n_allocated, n_list + 1);
> +
> +
> +Upstream-Status: Backport [207]
> diff --git a/meta/recipes-core/systemd/systemd_206.bbb/meta/recipes-core/systemd/
> systemd_206.bb
> index a1a8bd7..e7d0f1f 100644
> --- a/meta/recipes-core/systemd/systemd_206.bb
> +++ b/meta/recipes-core/systemd/systemd_206.bb
> @@ -21,6 +21,8 @@ SRC_URI = "
> http://www.freedesktop.org/software/systemd/systemd-${PV}.tar.xz \
> file://0001-use-CAP_MKNOD-ConditionCapability.patch \
> file://0001-Use-bin-mkdir-instead-of-host-mkdir-path.patch \
> file://binfmt-install.patch \
> + file://journald-fix-fd-leak-in-journal_file_empty.patch \
> + file://journald-fix-vacuuming-of-archived-journals.patch \
> file://touchscreen.rules \
> ${UCLIBCPATCHES} \
> file://00-create-volatile.conf \
> --
> 1.7.10.4
>
>
[-- Attachment #2: Type: text/html, Size: 7016 bytes --]
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: [for-dora] systemd: journald fix ignored disk space restrictions
2014-02-06 10:34 ` Diego Sueiro
@ 2014-02-06 18:49 ` Randy MacLeod
2014-02-06 23:58 ` Diego Sueiro
0 siblings, 1 reply; 8+ messages in thread
From: Randy MacLeod @ 2014-02-06 18:49 UTC (permalink / raw)
To: openembedded-core, diego.sueiro, Robert Yang
On 14-02-06 05:34 AM, Diego Sueiro wrote:
> Folks,
>
> Any chance to get this up-streamed?
Hi *dS, :)
Robert, who maintains dora, lives in China where they are
currently enjoying the Spring Festival holiday week.
I think Robert is back early next week. Please be patient.
../Randy
>
> Regards,
>
> --
> *dS
> Diego Sueiro
>
> Administrador do Embarcados
> www.embarcados.com.br <http://www.embarcados.com.br>
>
> /*long live rock 'n roll*/
>
>
> On Sat, Feb 1, 2014 at 7:07 AM, Diego Sueiro <diego.sueiro@gmail.com
> <mailto:diego.sueiro@gmail.com>> wrote:
>
> The upstream bug report can be seen at:
> [Systemd #68161] -- https://bugs.freedesktop.org/show_bug.cgi?id=68161
>
> This backports patches come from 207 and need to address this in the
> 206 version for dora branch.
>
> Signed-off-by: Diego Sueiro <diego.sueiro@gmail.com
> <mailto:diego.sueiro@gmail.com>>
> ---
> ...ournald-fix-fd-leak-in-journal_file_empty.patch | 27
> ++++++++++++++
> ...urnald-fix-vacuuming-of-archived-journals.patch | 37
> ++++++++++++++++++++
> meta/recipes-core/systemd/systemd_206.bb <http://systemd_206.bb>
> | 2 ++
> 3 files changed, 66 insertions(+)
> create mode 100644
> meta/recipes-core/systemd/systemd/journald-fix-fd-leak-in-journal_file_empty.patch
> create mode 100644
> meta/recipes-core/systemd/systemd/journald-fix-vacuuming-of-archived-journals.patch
>
> diff --git
> a/meta/recipes-core/systemd/systemd/journald-fix-fd-leak-in-journal_file_empty.patch
> b/meta/recipes-core/systemd/systemd/journald-fix-fd-leak-in-journal_file_empty.patch
> new file mode 100644
> index 0000000..a83c858
> --- /dev/null
> +++
> b/meta/recipes-core/systemd/systemd/journald-fix-fd-leak-in-journal_file_empty.patch
> @@ -0,0 +1,27 @@
> +journald: fix fd leak in journal_file_empty
> +Before my previous patch, journal_file_empty wasn't be called with the
> +correct filename. Now that it's being called with the correct filename
> +it leaks file descriptors. This patch closes the file descriptors
> before
> +returning.
> +
> +Signed-off-by: George McCollister <george.mccollister@gmail.com
> <mailto:george.mccollister@gmail.com>>
> +
> +
> +Index: systemd-206/src/journal/journal-vacuum.c
> +===================================================================
> +--- systemd-206.orig/src/journal/journal-vacuum.c 2013-07-21
> 19:43:28.146183128 -0300
> ++++ systemd-206/src/journal/journal-vacuum.c 2014-01-29
> 20:43:27.492343295 -0200
> +@@ -129,9 +129,9 @@
> + }
> +
> + static int journal_file_empty(int dir_fd, const char *name) {
> +- int fd, r;
> ++ int r;
> + le64_t n_entries;
> +-
> ++ _cleanup_close_ int fd;
> + fd = openat(dir_fd, name,
> O_RDONLY|O_CLOEXEC|O_NOFOLLOW|O_NONBLOCK);
> + if (fd < 0)
> + return -errno;
> +
> +Upstream-Status: Backport [207]
> diff --git
> a/meta/recipes-core/systemd/systemd/journald-fix-vacuuming-of-archived-journals.patch
> b/meta/recipes-core/systemd/systemd/journald-fix-vacuuming-of-archived-journals.patch
> new file mode 100644
> index 0000000..7c59200
> --- /dev/null
> +++
> b/meta/recipes-core/systemd/systemd/journald-fix-vacuuming-of-archived-journals.patch
> @@ -0,0 +1,37 @@
> +journald: fix vacuuming of archived journals
> +d_name is modified on line 227 so if the entire journal name is needed
> +again p must be used. Before this change when journal_file_empty
> was called
> +on archived journals it would always return with -2.
> +
> +Signed-off-by: George McCollister <george.mccollister@gmail.com
> <mailto:george.mccollister@gmail.com>>
> +
> +Index: systemd-206/src/journal/journal-vacuum.c
> +===================================================================
> +--- systemd-206.orig/src/journal/journal-vacuum.c 2014-01-29
> 20:43:27.492343295 -0200
> ++++ systemd-206/src/journal/journal-vacuum.c 2014-01-29
> 20:48:55.076356876 -0200
> +@@ -265,18 +265,18 @@
> + /* We do not vacuum active files or
> unknown files! */
> + continue;
> +
> +- if (journal_file_empty(dirfd(d), de->d_name)) {
> ++ if (journal_file_empty(dirfd(d), p)) {
> +
> + /* Always vacuum empty non-online files. */
> +
> +- if (unlinkat(dirfd(d), de->d_name, 0) >= 0)
> +- log_debug("Deleted empty journal
> %s/%s.", directory, de->d_name);
> ++ if (unlinkat(dirfd(d), p, 0) >= 0)
> ++ log_debug("Deleted empty journal
> %s/%s.", directory, p);
> + else if (errno != ENOENT)
> +- log_warning("Failed to delete
> %s/%s: %m", directory, de->d_name);
> ++ log_warning("Failed to delete
> %s/%s: %m", directory, p);
> + continue;
> + }
> +
> +- patch_realtime(directory, de->d_name, &st, &realtime);
> ++ patch_realtime(directory, p, &st, &realtime);
> +
> + GREEDY_REALLOC(list, n_allocated, n_list + 1);
> +
> +
> +Upstream-Status: Backport [207]
> diff --git a/meta/recipes-core/systemd/systemd_206.bb
> <http://systemd_206.bb> b/meta/recipes-core/systemd/systemd_206.bb
> <http://systemd_206.bb>
> index a1a8bd7..e7d0f1f 100644
> --- a/meta/recipes-core/systemd/systemd_206.bb <http://systemd_206.bb>
> +++ b/meta/recipes-core/systemd/systemd_206.bb <http://systemd_206.bb>
> @@ -21,6 +21,8 @@ SRC_URI =
> "http://www.freedesktop.org/software/systemd/systemd-${PV}.tar.xz \
> file://0001-use-CAP_MKNOD-ConditionCapability.patch \
>
> file://0001-Use-bin-mkdir-instead-of-host-mkdir-path.patch \
> file://binfmt-install.patch \
> + file://journald-fix-fd-leak-in-journal_file_empty.patch \
> + file://journald-fix-vacuuming-of-archived-journals.patch \
> file://touchscreen.rules \
> ${UCLIBCPATCHES} \
> file://00-create-volatile.conf \
> --
> 1.7.10.4
>
>
>
>
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
>
--
# Randy MacLeod. SMTS, Linux, Wind River
Direct: 613.963.1350
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: [for-dora] systemd: journald fix ignored disk space restrictions
2014-02-06 18:49 ` Randy MacLeod
@ 2014-02-06 23:58 ` Diego Sueiro
2014-02-07 5:36 ` Robert Yang
0 siblings, 1 reply; 8+ messages in thread
From: Diego Sueiro @ 2014-02-06 23:58 UTC (permalink / raw)
To: Randy MacLeod; +Cc: openembedded-core
[-- Attachment #1: Type: text/plain, Size: 448 bytes --]
On Thu, Feb 6, 2014 at 4:49 PM, Randy MacLeod
<randy.macleod@windriver.com>wrote:
> Hi *dS, :)
>
> Robert, who maintains dora, lives in China where they are
> currently enjoying the Spring Festival holiday week.
> I think Robert is back early next week. Please be patient.
>
:-D
Thanks for the feedback Randy.
Regards,
--
*dS
Diego Sueiro
Administrador do Embarcados
www.embarcados.com.br
/*long live rock 'n roll*/
[-- Attachment #2: Type: text/html, Size: 1020 bytes --]
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [for-dora] systemd: journald fix ignored disk space restrictions
2014-02-06 23:58 ` Diego Sueiro
@ 2014-02-07 5:36 ` Robert Yang
0 siblings, 0 replies; 8+ messages in thread
From: Robert Yang @ 2014-02-07 5:36 UTC (permalink / raw)
To: Diego Sueiro, Randy MacLeod; +Cc: openembedded-core
Yes, I'm OOO, thanks for the patch, I will take it when I come back
on next Tue.
// Robert
On 2014年02月07日 07:58, Diego Sueiro wrote:
> On Thu, Feb 6, 2014 at 4:49 PM, Randy MacLeod
> <randy.macleod@windriver.com>wrote:
>
>> Hi *dS, :)
>>
>> Robert, who maintains dora, lives in China where they are
>> currently enjoying the Spring Festival holiday week.
>> I think Robert is back early next week. Please be patient.
>>
>
> :-D
> Thanks for the feedback Randy.
>
>
> Regards,
>
> --
> *dS
> Diego Sueiro
>
> Administrador do Embarcados
> www.embarcados.com.br
>
> /*long live rock 'n roll*/
>
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2014-02-07 5:36 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-01-31 20:04 [for-dora] systemd: journald fix ignored disk space restrictions Diego Sueiro
2014-02-01 7:42 ` Martin Jansa
2014-02-01 9:02 ` Diego Sueiro
-- strict thread matches above, loose matches on Subject: below --
2014-02-01 9:07 Diego Sueiro
2014-02-06 10:34 ` Diego Sueiro
2014-02-06 18:49 ` Randy MacLeod
2014-02-06 23:58 ` Diego Sueiro
2014-02-07 5:36 ` Robert Yang
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.