From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from lists.gentoo.org (pigeon.gentoo.org [208.92.234.80]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by finch.gentoo.org (Postfix) with ESMTPS id 953321580B3 for ; Sat, 4 Sep 2021 21:01:26 +0000 (UTC) Received: from pigeon.gentoo.org (localhost [127.0.0.1]) by pigeon.gentoo.org (Postfix) with SMTP id B0008E0821; Sat, 4 Sep 2021 21:01:25 +0000 (UTC) Received: from smtp.gentoo.org (woodpecker.gentoo.org [140.211.166.183]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by pigeon.gentoo.org (Postfix) with ESMTPS id 7EBDEE0821 for ; Sat, 4 Sep 2021 21:01:25 +0000 (UTC) Received: from oystercatcher.gentoo.org (oystercatcher.gentoo.org [148.251.78.52]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.gentoo.org (Postfix) with ESMTPS id 5966F33E5E4 for ; Sat, 4 Sep 2021 21:01:24 +0000 (UTC) Received: from localhost.localdomain (localhost [IPv6:::1]) by oystercatcher.gentoo.org (Postfix) with ESMTP id 6B7A9BA for ; Sat, 4 Sep 2021 21:01:21 +0000 (UTC) From: "Andreas Sturmlechner" To: gentoo-commits@lists.gentoo.org Content-Transfer-Encoding: 8bit Content-type: text/plain; charset=UTF-8 Reply-To: gentoo-dev@lists.gentoo.org, "Andreas Sturmlechner" Message-ID: <1630789123.9f7d20409337b1214e7afbb5ca8fa76d8e1c5c9f.asturm@gentoo> Subject: [gentoo-commits] repo/gentoo:master commit in: kde-frameworks/plasma/files/, kde-frameworks/plasma/ X-VCS-Repository: repo/gentoo X-VCS-Files: kde-frameworks/plasma/files/plasma-5.85.0-fix-svgicon-memleak.patch kde-frameworks/plasma/files/plasma-5.85.0-fix-theme-memleak.patch kde-frameworks/plasma/plasma-5.85.0-r3.ebuild X-VCS-Directories: kde-frameworks/plasma/ kde-frameworks/plasma/files/ X-VCS-Committer: asturm X-VCS-Committer-Name: Andreas Sturmlechner X-VCS-Revision: 9f7d20409337b1214e7afbb5ca8fa76d8e1c5c9f X-VCS-Branch: master Date: Sat, 4 Sep 2021 21:01:21 +0000 (UTC) Precedence: bulk List-Post: List-Help: List-Unsubscribe: List-Subscribe: List-Id: Gentoo Linux mail X-BeenThere: gentoo-commits@lists.gentoo.org X-Auto-Response-Suppress: DR, RN, NRN, OOF, AutoReply X-Archives-Salt: 355e27d9-bdbf-4f0b-9555-c6097eac4f17 X-Archives-Hash: 3158c639908c21c42b57c9daa48c2b59 commit: 9f7d20409337b1214e7afbb5ca8fa76d8e1c5c9f Author: Andreas Sturmlechner gentoo org> AuthorDate: Sat Sep 4 20:40:03 2021 +0000 Commit: Andreas Sturmlechner gentoo org> CommitDate: Sat Sep 4 20:58:43 2021 +0000 URL: https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=9f7d2040 kde-frameworks/plasma: Backport fixes for two longstanding memleaks See also: https://invent.kde.org/frameworks/plasma-framework/-/merge_requests/323 Package-Manager: Portage-3.0.22, Repoman-3.0.3 Signed-off-by: Andreas Sturmlechner gentoo.org> .../files/plasma-5.85.0-fix-svgicon-memleak.patch | 37 ++++++++++ .../files/plasma-5.85.0-fix-theme-memleak.patch | 86 ++++++++++++++++++++++ kde-frameworks/plasma/plasma-5.85.0-r3.ebuild | 2 + 3 files changed, 125 insertions(+) diff --git a/kde-frameworks/plasma/files/plasma-5.85.0-fix-svgicon-memleak.patch b/kde-frameworks/plasma/files/plasma-5.85.0-fix-svgicon-memleak.patch new file mode 100644 index 00000000000..89896911d0a --- /dev/null +++ b/kde-frameworks/plasma/files/plasma-5.85.0-fix-svgicon-memleak.patch @@ -0,0 +1,37 @@ +From 73782c8b39d1cc41fef003acca8df75ccdf384e4 Mon Sep 17 00:00:00 2001 +From: Matt Whitlock +Date: Mon, 16 Aug 2021 19:37:28 -0400 +Subject: [PATCH] avoid holding onto old Svg object when changing source of an + IconItem + +A long-lived IconItem instance can have its source changed many times +over its lifetime. Because SvgSource parents its internal Plasma::Svg +instance to the IconItem instance, this means that such Plasma::Svg +instance was not being destroyed when its responsible SvgSource +instance is destroyed and indeed would not be destroyed until the +IconItem instance is eventually destroyed, which could be arbitrarily +much later. This commit adds an explicit call in the SvgSource +destructor to delete the Plasma::Svg instance so it does not hang +around in memory until the IconItem instance is destroyed. This fixes +one of the major memory "leaks" in plasmashell. +--- + src/declarativeimports/core/iconitem.cpp | 3 +++ + 1 file changed, 3 insertions(+) + +diff --git a/src/declarativeimports/core/iconitem.cpp b/src/declarativeimports/core/iconitem.cpp +index 0db750acd..3f06b8b0e 100644 +--- a/src/declarativeimports/core/iconitem.cpp ++++ b/src/declarativeimports/core/iconitem.cpp +@@ -196,6 +196,9 @@ public: + { + if (m_svgIcon) { + QObject::disconnect(m_iconItem, nullptr, m_svgIcon, nullptr); ++ // the parent IconItem can outlive this IconItemSource, so delete our Plasma::Svg object ++ // explicitly to avoid leaving unreferenced Plasma::Svg objects parented to the IconItem ++ delete m_svgIcon; + } + } + +-- +GitLab + diff --git a/kde-frameworks/plasma/files/plasma-5.85.0-fix-theme-memleak.patch b/kde-frameworks/plasma/files/plasma-5.85.0-fix-theme-memleak.patch new file mode 100644 index 00000000000..1d7dc596c20 --- /dev/null +++ b/kde-frameworks/plasma/files/plasma-5.85.0-fix-theme-memleak.patch @@ -0,0 +1,86 @@ +From 14b495f933dadace7832fa6cbc809c3abdb7c682 Mon Sep 17 00:00:00 2001 +From: Matt Whitlock +Date: Mon, 28 Jun 2021 18:01:14 -0400 +Subject: [PATCH] don't make duplicate connections to + ThemePrivate::onAppExitCleanup +MIME-Version: 1.0 +Content-Type: text/plain; charset=UTF-8 +Content-Transfer-Encoding: 8bit + +Plasma::Theme::Theme(…) and Plasma::Theme::setThemeName(…) were +unconditionally connecting the QCoreApplication::aboutToQuit signal to +the ThemePrivate::onAppExitCleanup slot, even though the ThemePrivate +instances are cached and shared across multiple Theme instances. In +long-running applications that make heavy use of the Svg class (such as +plasmashell), a single ThemePrivate instance can be reused by huge +numbers of Theme instances. If the reference count of that ThemePrivate +instance never reaches zero, then the connections just keep piling up, +contributing to excessive memory usage. This commit moves the relevant +connect(…) call so that it only happens in the case that a new +ThemePrivate instance is constructed. Thus, there will only ever be one +connection from QCoreApplication::aboutToQuit to +ThemePrivate::onAppExitCleanup per instance of ThemePrivate. +--- + src/plasma/theme.cpp | 18 +++++++++--------- + 1 file changed, 9 insertions(+), 9 deletions(-) + +diff --git a/src/plasma/theme.cpp b/src/plasma/theme.cpp +index fabf98f4e..f403d393b 100644 +--- a/src/plasma/theme.cpp ++++ b/src/plasma/theme.cpp +@@ -39,13 +39,13 @@ Theme::Theme(QObject *parent) + if (!ThemePrivate::globalTheme) { + ThemePrivate::globalTheme = new ThemePrivate; + ThemePrivate::globalTheme->settingsChanged(false); ++ if (QCoreApplication::instance()) { ++ connect(QCoreApplication::instance(), &QCoreApplication::aboutToQuit, ThemePrivate::globalTheme, &ThemePrivate::onAppExitCleanup); ++ } + } + ThemePrivate::globalTheme->ref.ref(); + d = ThemePrivate::globalTheme; + +- if (QCoreApplication::instance()) { +- connect(QCoreApplication::instance(), &QCoreApplication::aboutToQuit, d, &ThemePrivate::onAppExitCleanup); +- } + connect(d, &ThemePrivate::themeChanged, this, &Theme::themeChanged); + connect(d, &ThemePrivate::defaultFontChanged, this, &Theme::defaultFontChanged); + connect(d, &ThemePrivate::smallestFontChanged, this, &Theme::smallestFontChanged); +@@ -57,6 +57,9 @@ Theme::Theme(const QString &themeName, QObject *parent) + auto &priv = ThemePrivate::themes[themeName]; + if (!priv) { + priv = new ThemePrivate; ++ if (QCoreApplication::instance()) { ++ connect(QCoreApplication::instance(), &QCoreApplication::aboutToQuit, priv, &ThemePrivate::onAppExitCleanup); ++ } + } + + priv->ref.ref(); +@@ -68,9 +71,6 @@ Theme::Theme(const QString &themeName, QObject *parent) + d->setThemeName(themeName, false, false); + d->cacheTheme = useCache; + d->fixedName = true; +- if (QCoreApplication::instance()) { +- connect(QCoreApplication::instance(), &QCoreApplication::aboutToQuit, d, &ThemePrivate::onAppExitCleanup); +- } + connect(d, &ThemePrivate::themeChanged, this, &Theme::themeChanged); + } + +@@ -105,12 +105,12 @@ void Theme::setThemeName(const QString &themeName) + auto &priv = ThemePrivate::themes[themeName]; + if (!priv) { + priv = new ThemePrivate; ++ if (QCoreApplication::instance()) { ++ connect(QCoreApplication::instance(), &QCoreApplication::aboutToQuit, priv, &ThemePrivate::onAppExitCleanup); ++ } + } + priv->ref.ref(); + d = priv; +- if (QCoreApplication::instance()) { +- connect(QCoreApplication::instance(), &QCoreApplication::aboutToQuit, d, &ThemePrivate::onAppExitCleanup); +- } + connect(d, &ThemePrivate::themeChanged, this, &Theme::themeChanged); + } + +-- +GitLab + diff --git a/kde-frameworks/plasma/plasma-5.85.0-r3.ebuild b/kde-frameworks/plasma/plasma-5.85.0-r3.ebuild index 433fe380d17..5fcce852061 100644 --- a/kde-frameworks/plasma/plasma-5.85.0-r3.ebuild +++ b/kde-frameworks/plasma/plasma-5.85.0-r3.ebuild @@ -64,6 +64,8 @@ PATCHES=( "${FILESDIR}"/${P}-fix-pinned-calendar-dots.patch # KDE-bug 440627 "${FILESDIR}"/${P}-fix-cmake.patch # bug 809815 "${FILESDIR}"/${P}-fix-ExpandableListItem-overlapping.patch # KDE-bug 428102 + "${FILESDIR}"/${P}-fix-theme-memleak.patch # in KF-5.86 + "${FILESDIR}"/${P}-fix-svgicon-memleak.patch # in KF-5.86 ) src_configure() {