]> git.ipfire.org Git - thirdparty/kernel/stable-queue.git/commitdiff
6.11-stable patches
authorGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 8 Oct 2024 10:56:57 +0000 (12:56 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 8 Oct 2024 10:56:57 +0000 (12:56 +0200)
added patches:
alsa-control-fix-leftover-snd_power_unref.patch
crypto-octeontx-select-crypto_authenc.patch
drm-amd-display-revert-avoid-overflow-assignment.patch
perf-report-fix-segfault-when-sym-sort-key-is-not-used.patch
pmdomain-core-reduce-debug-summary-table-width.patch

queue-6.11/alsa-control-fix-leftover-snd_power_unref.patch [new file with mode: 0644]
queue-6.11/crypto-octeontx-select-crypto_authenc.patch [new file with mode: 0644]
queue-6.11/drm-amd-display-revert-avoid-overflow-assignment.patch [new file with mode: 0644]
queue-6.11/perf-report-fix-segfault-when-sym-sort-key-is-not-used.patch [new file with mode: 0644]
queue-6.11/pmdomain-core-reduce-debug-summary-table-width.patch [new file with mode: 0644]
queue-6.11/series

diff --git a/queue-6.11/alsa-control-fix-leftover-snd_power_unref.patch b/queue-6.11/alsa-control-fix-leftover-snd_power_unref.patch
new file mode 100644 (file)
index 0000000..14d83d3
--- /dev/null
@@ -0,0 +1,31 @@
+From fef1ac950c600ba50ef4d65ca03c8dae9be7f9ea Mon Sep 17 00:00:00 2001
+From: Takashi Iwai <tiwai@suse.de>
+Date: Thu, 1 Aug 2024 08:42:01 +0200
+Subject: ALSA: control: Fix leftover snd_power_unref()
+
+From: Takashi Iwai <tiwai@suse.de>
+
+commit fef1ac950c600ba50ef4d65ca03c8dae9be7f9ea upstream.
+
+One snd_power_unref() was forgotten and left at __snd_ctl_elem_info()
+in the previous change for reorganizing the locking order.
+
+Fixes: fcc62b19104a ("ALSA: control: Take power_ref lock primarily")
+Link: https://github.com/thesofproject/linux/pull/5127
+Link: https://patch.msgid.link/20240801064203.30284-1-tiwai@suse.de
+Signed-off-by: Takashi Iwai <tiwai@suse.de>
+Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
+---
+ sound/core/control.c |    1 -
+ 1 file changed, 1 deletion(-)
+
+--- a/sound/core/control.c
++++ b/sound/core/control.c
+@@ -1168,7 +1168,6 @@ static int __snd_ctl_elem_info(struct sn
+       info->access = 0;
+ #endif
+       result = kctl->info(kctl, info);
+-      snd_power_unref(card);
+       if (result >= 0) {
+               snd_BUG_ON(info->access);
+               index_offset = snd_ctl_get_ioff(kctl, &info->id);
diff --git a/queue-6.11/crypto-octeontx-select-crypto_authenc.patch b/queue-6.11/crypto-octeontx-select-crypto_authenc.patch
new file mode 100644 (file)
index 0000000..34a80f7
--- /dev/null
@@ -0,0 +1,40 @@
+From c398cb8eb0a263a1b7a18892d9f244751689675c Mon Sep 17 00:00:00 2001
+From: Herbert Xu <herbert@gondor.apana.org.au>
+Date: Thu, 5 Sep 2024 10:21:49 +0800
+Subject: crypto: octeontx* - Select CRYPTO_AUTHENC
+
+From: Herbert Xu <herbert@gondor.apana.org.au>
+
+commit c398cb8eb0a263a1b7a18892d9f244751689675c upstream.
+
+Select CRYPTO_AUTHENC as the function crypto_authenec_extractkeys
+may not be available without it.
+
+Fixes: 311eea7e37c4 ("crypto: octeontx - Fix authenc setkey")
+Fixes: 7ccb750dcac8 ("crypto: octeontx2 - Fix authenc setkey")
+Reported-by: kernel test robot <lkp@intel.com>
+Closes: https://lore.kernel.org/oe-kbuild-all/202409042013.gT2ZI4wR-lkp@intel.com/
+Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
+Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
+---
+ drivers/crypto/marvell/Kconfig |    2 ++
+ 1 file changed, 2 insertions(+)
+
+--- a/drivers/crypto/marvell/Kconfig
++++ b/drivers/crypto/marvell/Kconfig
+@@ -28,6 +28,7 @@ config CRYPTO_DEV_OCTEONTX_CPT
+       select CRYPTO_SKCIPHER
+       select CRYPTO_HASH
+       select CRYPTO_AEAD
++      select CRYPTO_AUTHENC
+       select CRYPTO_DEV_MARVELL
+       help
+               This driver allows you to utilize the Marvell Cryptographic
+@@ -47,6 +48,7 @@ config CRYPTO_DEV_OCTEONTX2_CPT
+       select CRYPTO_SKCIPHER
+       select CRYPTO_HASH
+       select CRYPTO_AEAD
++      select CRYPTO_AUTHENC
+       select NET_DEVLINK
+       help
+               This driver allows you to utilize the Marvell Cryptographic
diff --git a/queue-6.11/drm-amd-display-revert-avoid-overflow-assignment.patch b/queue-6.11/drm-amd-display-revert-avoid-overflow-assignment.patch
new file mode 100644 (file)
index 0000000..7c488aa
--- /dev/null
@@ -0,0 +1,57 @@
+From e80f8f491df873ea2e07c941c747831234814612 Mon Sep 17 00:00:00 2001
+From: Gabe Teeger <Gabe.Teeger@amd.com>
+Date: Thu, 25 Jul 2024 18:42:21 -0400
+Subject: drm/amd/display: Revert Avoid overflow assignment
+
+From: Gabe Teeger <Gabe.Teeger@amd.com>
+
+commit e80f8f491df873ea2e07c941c747831234814612 upstream.
+
+This reverts commit a15268787b79 ("drm/amd/display: Avoid overflow assignment in link_dp_cts")
+Due to regression causing DPMS hang.
+
+Reviewed-by: Alex Hung <alex.hung@amd.com>
+Signed-off-by: Gabe Teeger <Gabe.Teeger@amd.com>
+Signed-off-by: Wayne Lin <wayne.lin@amd.com>
+Tested-by: Daniel Wheeler <daniel.wheeler@amd.com>
+Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
+Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
+---
+ drivers/gpu/drm/amd/display/dc/dc_dp_types.h                  |    2 +-
+ drivers/gpu/drm/amd/display/dc/link/accessories/link_dp_cts.c |    3 +--
+ drivers/gpu/drm/amd/display/include/dpcd_defs.h               |    1 -
+ 3 files changed, 2 insertions(+), 4 deletions(-)
+
+--- a/drivers/gpu/drm/amd/display/dc/dc_dp_types.h
++++ b/drivers/gpu/drm/amd/display/dc/dc_dp_types.h
+@@ -727,7 +727,7 @@ struct dp_audio_test_data_flags {
+ struct dp_audio_test_data {
+       struct dp_audio_test_data_flags flags;
+-      uint32_t sampling_rate;
++      uint8_t sampling_rate;
+       uint8_t channel_count;
+       uint8_t pattern_type;
+       uint8_t pattern_period[8];
+--- a/drivers/gpu/drm/amd/display/dc/link/accessories/link_dp_cts.c
++++ b/drivers/gpu/drm/amd/display/dc/link/accessories/link_dp_cts.c
+@@ -775,8 +775,7 @@ bool dp_set_test_pattern(
+                       core_link_read_dpcd(link, DP_TRAINING_PATTERN_SET,
+                                           &training_pattern.raw,
+                                           sizeof(training_pattern));
+-                      if (pattern <= PHY_TEST_PATTERN_END_DP11)
+-                              training_pattern.v1_3.LINK_QUAL_PATTERN_SET = pattern;
++                      training_pattern.v1_3.LINK_QUAL_PATTERN_SET = pattern;
+                       core_link_write_dpcd(link, DP_TRAINING_PATTERN_SET,
+                                            &training_pattern.raw,
+                                            sizeof(training_pattern));
+--- a/drivers/gpu/drm/amd/display/include/dpcd_defs.h
++++ b/drivers/gpu/drm/amd/display/include/dpcd_defs.h
+@@ -76,7 +76,6 @@ enum dpcd_phy_test_patterns {
+       PHY_TEST_PATTERN_D10_2,
+       PHY_TEST_PATTERN_SYMBOL_ERROR,
+       PHY_TEST_PATTERN_PRBS7,
+-      PHY_TEST_PATTERN_END_DP11 = PHY_TEST_PATTERN_PRBS7,
+       PHY_TEST_PATTERN_80BIT_CUSTOM,/* For DP1.2 only */
+       PHY_TEST_PATTERN_CP2520_1,
+       PHY_TEST_PATTERN_CP2520_2,
diff --git a/queue-6.11/perf-report-fix-segfault-when-sym-sort-key-is-not-used.patch b/queue-6.11/perf-report-fix-segfault-when-sym-sort-key-is-not-used.patch
new file mode 100644 (file)
index 0000000..d12b913
--- /dev/null
@@ -0,0 +1,88 @@
+From 9af2efee41b27a0f386fb5aa95d8d0b4b5d9fede Mon Sep 17 00:00:00 2001
+From: Namhyung Kim <namhyung@kernel.org>
+Date: Mon, 26 Aug 2024 15:10:42 -0700
+Subject: perf report: Fix segfault when 'sym' sort key is not used
+
+From: Namhyung Kim <namhyung@kernel.org>
+
+commit 9af2efee41b27a0f386fb5aa95d8d0b4b5d9fede upstream.
+
+The fields in the hist_entry are filled on-demand which means they only
+have meaningful values when relevant sort keys are used.
+
+So if neither of 'dso' nor 'sym' sort keys are used, the map/symbols in
+the hist entry can be garbage.  So it shouldn't access it
+unconditionally.
+
+I got a segfault, when I wanted to see cgroup profiles.
+
+  $ sudo perf record -a --all-cgroups --synth=cgroup true
+
+  $ sudo perf report -s cgroup
+
+  Program received signal SIGSEGV, Segmentation fault.
+  0x00005555557a8d90 in map__dso (map=0x0) at util/map.h:48
+  48           return RC_CHK_ACCESS(map)->dso;
+  (gdb) bt
+  #0  0x00005555557a8d90 in map__dso (map=0x0) at util/map.h:48
+  #1  0x00005555557aa39b in map__load (map=0x0) at util/map.c:344
+  #2  0x00005555557aa592 in map__find_symbol (map=0x0, addr=140736115941088) at util/map.c:385
+  #3  0x00005555557ef000 in hists__findnew_entry (hists=0x555556039d60, entry=0x7fffffffa4c0, al=0x7fffffffa8c0, sample_self=true)
+      at util/hist.c:644
+  #4  0x00005555557ef61c in __hists__add_entry (hists=0x555556039d60, al=0x7fffffffa8c0, sym_parent=0x0, bi=0x0, mi=0x0, ki=0x0,
+      block_info=0x0, sample=0x7fffffffaa90, sample_self=true, ops=0x0) at util/hist.c:761
+  #5  0x00005555557ef71f in hists__add_entry (hists=0x555556039d60, al=0x7fffffffa8c0, sym_parent=0x0, bi=0x0, mi=0x0, ki=0x0,
+      sample=0x7fffffffaa90, sample_self=true) at util/hist.c:779
+  #6  0x00005555557f00fb in iter_add_single_normal_entry (iter=0x7fffffffa900, al=0x7fffffffa8c0) at util/hist.c:1015
+  #7  0x00005555557f09a7 in hist_entry_iter__add (iter=0x7fffffffa900, al=0x7fffffffa8c0, max_stack_depth=127, arg=0x7fffffffbce0)
+      at util/hist.c:1260
+  #8  0x00005555555ba7ce in process_sample_event (tool=0x7fffffffbce0, event=0x7ffff7c14128, sample=0x7fffffffaa90, evsel=0x555556039ad0,
+      machine=0x5555560388e8) at builtin-report.c:334
+  #9  0x00005555557b30c8 in evlist__deliver_sample (evlist=0x555556039010, tool=0x7fffffffbce0, event=0x7ffff7c14128,
+      sample=0x7fffffffaa90, evsel=0x555556039ad0, machine=0x5555560388e8) at util/session.c:1232
+  #10 0x00005555557b32bc in machines__deliver_event (machines=0x5555560388e8, evlist=0x555556039010, event=0x7ffff7c14128,
+      sample=0x7fffffffaa90, tool=0x7fffffffbce0, file_offset=110888, file_path=0x555556038ff0 "perf.data") at util/session.c:1271
+  #11 0x00005555557b3848 in perf_session__deliver_event (session=0x5555560386d0, event=0x7ffff7c14128, tool=0x7fffffffbce0,
+      file_offset=110888, file_path=0x555556038ff0 "perf.data") at util/session.c:1354
+  #12 0x00005555557affaf in ordered_events__deliver_event (oe=0x555556038e60, event=0x555556135aa0) at util/session.c:132
+  #13 0x00005555557bb605 in do_flush (oe=0x555556038e60, show_progress=false) at util/ordered-events.c:245
+  #14 0x00005555557bb95c in __ordered_events__flush (oe=0x555556038e60, how=OE_FLUSH__ROUND, timestamp=0) at util/ordered-events.c:324
+  #15 0x00005555557bba46 in ordered_events__flush (oe=0x555556038e60, how=OE_FLUSH__ROUND) at util/ordered-events.c:342
+  #16 0x00005555557b1b3b in perf_event__process_finished_round (tool=0x7fffffffbce0, event=0x7ffff7c15bb8, oe=0x555556038e60)
+      at util/session.c:780
+  #17 0x00005555557b3b27 in perf_session__process_user_event (session=0x5555560386d0, event=0x7ffff7c15bb8, file_offset=117688,
+      file_path=0x555556038ff0 "perf.data") at util/session.c:1406
+
+As you can see the entry->ms.map was NULL even if he->ms.map has a
+value.  This is because 'sym' sort key is not given, so it cannot assume
+whether he->ms.sym and entry->ms.sym is the same.  I only checked the
+'sym' sort key here as it implies 'dso' behavior (so maps are the same).
+
+Fixes: ac01c8c4246546fd ("perf hist: Update hist symbol when updating maps")
+Signed-off-by: Namhyung Kim <namhyung@kernel.org>
+Cc: Adrian Hunter <adrian.hunter@intel.com>
+Cc: Ian Rogers <irogers@google.com>
+Cc: Ingo Molnar <mingo@kernel.org>
+Cc: Jiri Olsa <jolsa@kernel.org>
+Cc: Kan Liang <kan.liang@linux.intel.com>
+Cc: Matt Fleming <matt@readmodwrite.com>
+Cc: Peter Zijlstra <peterz@infradead.org>
+Cc: Stephane Eranian <eranian@google.com>
+Link: https://lore.kernel.org/r/20240826221045.1202305-2-namhyung@kernel.org
+Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
+Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
+---
+ tools/perf/util/hist.c |    2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+--- a/tools/perf/util/hist.c
++++ b/tools/perf/util/hist.c
+@@ -636,7 +636,7 @@ static struct hist_entry *hists__findnew
+                        * mis-adjust symbol addresses when computing
+                        * the history counter to increment.
+                        */
+-                      if (he->ms.map != entry->ms.map) {
++                      if (hists__has(hists, sym) && he->ms.map != entry->ms.map) {
+                               if (he->ms.sym) {
+                                       u64 addr = he->ms.sym->start;
+                                       he->ms.sym = map__find_symbol(entry->ms.map, addr);
diff --git a/queue-6.11/pmdomain-core-reduce-debug-summary-table-width.patch b/queue-6.11/pmdomain-core-reduce-debug-summary-table-width.patch
new file mode 100644 (file)
index 0000000..4d7f160
--- /dev/null
@@ -0,0 +1,94 @@
+From c6ccb691d484544636bc4a097574c5c135ccccda Mon Sep 17 00:00:00 2001
+From: Geert Uytterhoeven <geert+renesas@glider.be>
+Date: Wed, 4 Sep 2024 16:30:48 +0200
+Subject: pmdomain: core: Reduce debug summary table width
+
+From: Geert Uytterhoeven <geert+renesas@glider.be>
+
+commit c6ccb691d484544636bc4a097574c5c135ccccda upstream.
+
+Commit 9094e53ff5c86ebe ("pmdomain: core: Use dev_name() instead of
+kobject_get_path() in debugfs") severely shortened the names of devices
+in a PM Domain.  Now the most common format[1] consists of a 32-bit
+unit-address (8 characters), followed by a dot and a node name (20
+characters for "air-pollution-sensor" and "interrupt-controller", which
+are the longest generic node names documented in the Devicetree
+Specification), for a typical maximum of 29 characters.
+
+This offers a good opportunity to reduce the table width of the debug
+summary:
+  - Reduce the device name field width from 50 to 30 characters, which
+    matches the PM Domain name width,
+  - Reduce the large inter-column space between the "performance" and
+    "managed by" columns.
+
+Visual impact:
+  - The "performance" column now starts at a position that is a
+    multiple of 16, just like the "status" and "children" columns,
+  - All of the "/device", "runtime status", and "managed by" columns are
+    now indented 4 characters more than the columns right above them,
+  - Everything fits in (one less than) 80 characters again ;-)
+
+[1] Note that some device names (e.g. TI AM335x interconnect target
+    modules) do not follow this convention, and may be much longer, but
+    these didn't fit in the old 50-character column width either.
+
+Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
+Link: https://lore.kernel.org/r/f8e1821364b6d5d11350447c128f6d2b470f33fe.1725459707.git.geert+renesas@glider.be
+Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org>
+Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
+---
+ drivers/pmdomain/core.c |   14 +++++++-------
+ 1 file changed, 7 insertions(+), 7 deletions(-)
+
+--- a/drivers/pmdomain/core.c
++++ b/drivers/pmdomain/core.c
+@@ -3182,7 +3182,7 @@ static void rtpm_status_str(struct seq_f
+       else
+               WARN_ON(1);
+-      seq_printf(s, "%-25s  ", p);
++      seq_printf(s, "%-26s  ", p);
+ }
+ static void mode_status_str(struct seq_file *s, struct device *dev)
+@@ -3191,7 +3191,7 @@ static void mode_status_str(struct seq_f
+       gpd_data = to_gpd_data(dev->power.subsys_data->domain_data);
+-      seq_printf(s, "%9s", gpd_data->hw_mode ? "HW" : "SW");
++      seq_printf(s, "%2s", gpd_data->hw_mode ? "HW" : "SW");
+ }
+ static void perf_status_str(struct seq_file *s, struct device *dev)
+@@ -3226,7 +3226,7 @@ static int genpd_summary_one(struct seq_
+       else
+               snprintf(state, sizeof(state), "%s",
+                        status_lookup[genpd->status]);
+-      seq_printf(s, "%-30s  %-49s  %u", genpd->name, state, genpd->performance_state);
++      seq_printf(s, "%-30s  %-30s  %u", genpd->name, state, genpd->performance_state);
+       /*
+        * Modifications on the list require holding locks on both
+@@ -3242,7 +3242,7 @@ static int genpd_summary_one(struct seq_
+       }
+       list_for_each_entry(pm_data, &genpd->dev_list, list_node) {
+-              seq_printf(s, "\n    %-50s  ", dev_name(pm_data->dev));
++              seq_printf(s, "\n    %-30s  ", dev_name(pm_data->dev));
+               rtpm_status_str(s, pm_data->dev);
+               perf_status_str(s, pm_data->dev);
+               mode_status_str(s, pm_data->dev);
+@@ -3260,9 +3260,9 @@ static int summary_show(struct seq_file
+       struct generic_pm_domain *genpd;
+       int ret = 0;
+-      seq_puts(s, "domain                          status          children                           performance\n");
+-      seq_puts(s, "    /device                                             runtime status                           managed by\n");
+-      seq_puts(s, "------------------------------------------------------------------------------------------------------------\n");
++      seq_puts(s, "domain                          status          children        performance\n");
++      seq_puts(s, "    /device                         runtime status                  managed by\n");
++      seq_puts(s, "------------------------------------------------------------------------------\n");
+       ret = mutex_lock_interruptible(&gpd_list_lock);
+       if (ret)
index 4937514c810c2b24a87fc3ae855e0842ee55535d..54fdbc2a6c1e1191ab57445dd318a5e44030504c 100644 (file)
@@ -551,3 +551,8 @@ revert-drm-amd-display-skip-recompute-dsc-params-if-no-stream-on-link.patch
 drm-sched-revert-always-increment-correct-scheduler-score.patch
 rxrpc-fix-a-race-between-socket-set-up-and-i-o-thread-creation.patch
 vhost-scsi-null-ptr-dereference-in-vhost_scsi_get_req.patch
+alsa-control-fix-leftover-snd_power_unref.patch
+crypto-octeontx-select-crypto_authenc.patch
+drm-amd-display-revert-avoid-overflow-assignment.patch
+perf-report-fix-segfault-when-sym-sort-key-is-not-used.patch
+pmdomain-core-reduce-debug-summary-table-width.patch