From: Greg Kroah-Hartman Date: Wed, 19 Aug 2020 11:52:04 +0000 (+0200) Subject: 4.9-stable patches X-Git-Tag: v4.14.194~40 X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=b4b6d59a0942162fdf864d8c4a84b848f307ed70;p=thirdparty%2Fkernel%2Fstable-queue.git 4.9-stable patches added patches: iio-dac-ad5592r-fix-unbalanced-mutex-unlocks-in-ad5592r_read_raw.patch powerpc-fix-circular-dependency-between-percpu.h-and-mmu.h.patch xtensa-fix-xtensa_pmu_setup-prototype.patch --- diff --git a/queue-4.9/iio-dac-ad5592r-fix-unbalanced-mutex-unlocks-in-ad5592r_read_raw.patch b/queue-4.9/iio-dac-ad5592r-fix-unbalanced-mutex-unlocks-in-ad5592r_read_raw.patch new file mode 100644 index 00000000000..7cacef77cec --- /dev/null +++ b/queue-4.9/iio-dac-ad5592r-fix-unbalanced-mutex-unlocks-in-ad5592r_read_raw.patch @@ -0,0 +1,47 @@ +From 65afb0932a81c1de719ceee0db0b276094b10ac8 Mon Sep 17 00:00:00 2001 +From: Alexandru Ardelean +Date: Mon, 6 Jul 2020 14:02:57 +0300 +Subject: iio: dac: ad5592r: fix unbalanced mutex unlocks in ad5592r_read_raw() + +From: Alexandru Ardelean + +commit 65afb0932a81c1de719ceee0db0b276094b10ac8 upstream. + +There are 2 exit paths where the lock isn't held, but try to unlock the +mutex when exiting. In these places we should just return from the +function. + +A neater approach would be to cleanup the ad5592r_read_raw(), but that +would make this patch more difficult to backport to stable versions. + +Fixes 56ca9db862bf3: ("iio: dac: Add support for the AD5592R/AD5593R ADCs/DACs") +Reported-by: Charles Stanhope +Signed-off-by: Alexandru Ardelean +Cc: +Signed-off-by: Jonathan Cameron +Signed-off-by: Greg Kroah-Hartman + +--- + drivers/iio/dac/ad5592r-base.c | 4 ++-- + 1 file changed, 2 insertions(+), 2 deletions(-) + +--- a/drivers/iio/dac/ad5592r-base.c ++++ b/drivers/iio/dac/ad5592r-base.c +@@ -417,7 +417,7 @@ static int ad5592r_read_raw(struct iio_d + s64 tmp = *val * (3767897513LL / 25LL); + *val = div_s64_rem(tmp, 1000000000LL, val2); + +- ret = IIO_VAL_INT_PLUS_MICRO; ++ return IIO_VAL_INT_PLUS_MICRO; + } else { + int mult; + +@@ -448,7 +448,7 @@ static int ad5592r_read_raw(struct iio_d + ret = IIO_VAL_INT; + break; + default: +- ret = -EINVAL; ++ return -EINVAL; + } + + unlock: diff --git a/queue-4.9/powerpc-fix-circular-dependency-between-percpu.h-and-mmu.h.patch b/queue-4.9/powerpc-fix-circular-dependency-between-percpu.h-and-mmu.h.patch new file mode 100644 index 00000000000..2c260bcdcb2 --- /dev/null +++ b/queue-4.9/powerpc-fix-circular-dependency-between-percpu.h-and-mmu.h.patch @@ -0,0 +1,66 @@ +From 0c83b277ada72b585e6a3e52b067669df15bcedb Mon Sep 17 00:00:00 2001 +From: Michael Ellerman +Date: Tue, 4 Aug 2020 22:44:06 +1000 +Subject: powerpc: Fix circular dependency between percpu.h and mmu.h + +From: Michael Ellerman + +commit 0c83b277ada72b585e6a3e52b067669df15bcedb upstream. + +Recently random.h started including percpu.h (see commit +f227e3ec3b5c ("random32: update the net random state on interrupt and +activity")), which broke corenet64_smp_defconfig: + + In file included from /linux/arch/powerpc/include/asm/paca.h:18, + from /linux/arch/powerpc/include/asm/percpu.h:13, + from /linux/include/linux/random.h:14, + from /linux/lib/uuid.c:14: + /linux/arch/powerpc/include/asm/mmu.h:139:22: error: unknown type name 'next_tlbcam_idx' + 139 | DECLARE_PER_CPU(int, next_tlbcam_idx); + +This is due to a circular header dependency: + asm/mmu.h includes asm/percpu.h, which includes asm/paca.h, which + includes asm/mmu.h + +Which means DECLARE_PER_CPU() isn't defined when mmu.h needs it. + +We can fix it by moving the include of paca.h below the include of +asm-generic/percpu.h. + +This moves the include of paca.h out of the #ifdef __powerpc64__, but +that is OK because paca.h is almost entirely inside #ifdef +CONFIG_PPC64 anyway. + +It also moves the include of paca.h out of the #ifdef CONFIG_SMP, +which could possibly break something, but seems to have no ill +effects. + +Fixes: f227e3ec3b5c ("random32: update the net random state on interrupt and activity") +Cc: stable@vger.kernel.org # v5.8 +Reported-by: Stephen Rothwell +Signed-off-by: Michael Ellerman +Link: https://lore.kernel.org/r/20200804130558.292328-1-mpe@ellerman.id.au +Signed-off-by: Greg Kroah-Hartman + +--- + arch/powerpc/include/asm/percpu.h | 4 ++-- + 1 file changed, 2 insertions(+), 2 deletions(-) + +--- a/arch/powerpc/include/asm/percpu.h ++++ b/arch/powerpc/include/asm/percpu.h +@@ -9,8 +9,6 @@ + + #ifdef CONFIG_SMP + +-#include +- + #define __my_cpu_offset local_paca->data_offset + + #endif /* CONFIG_SMP */ +@@ -18,4 +16,6 @@ + + #include + ++#include ++ + #endif /* _ASM_POWERPC_PERCPU_H_ */ diff --git a/queue-4.9/series b/queue-4.9/series index f1c33ab1933..c88b2963822 100644 --- a/queue-4.9/series +++ b/queue-4.9/series @@ -171,3 +171,6 @@ pci-hotplug-acpi-fix-context-refcounting-in-acpiphp_grab_context.patch btrfs-don-t-allocate-anonymous-block-device-for-user-invisible-roots.patch btrfs-only-search-for-left_info-if-there-is-no-right_info-in-try_merge_free_space.patch btrfs-fix-memory-leaks-after-failure-to-lookup-checksums-during-inode-logging.patch +iio-dac-ad5592r-fix-unbalanced-mutex-unlocks-in-ad5592r_read_raw.patch +xtensa-fix-xtensa_pmu_setup-prototype.patch +powerpc-fix-circular-dependency-between-percpu.h-and-mmu.h.patch diff --git a/queue-4.9/xtensa-fix-xtensa_pmu_setup-prototype.patch b/queue-4.9/xtensa-fix-xtensa_pmu_setup-prototype.patch new file mode 100644 index 00000000000..d56e9dce1d5 --- /dev/null +++ b/queue-4.9/xtensa-fix-xtensa_pmu_setup-prototype.patch @@ -0,0 +1,38 @@ +From 6d65d3769d1910379e1cfa61ebf387efc6bfb22c Mon Sep 17 00:00:00 2001 +From: Max Filippov +Date: Fri, 31 Jul 2020 12:37:32 -0700 +Subject: xtensa: fix xtensa_pmu_setup prototype +MIME-Version: 1.0 +Content-Type: text/plain; charset=UTF-8 +Content-Transfer-Encoding: 8bit + +From: Max Filippov + +commit 6d65d3769d1910379e1cfa61ebf387efc6bfb22c upstream. + +Fix the following build error in configurations with +CONFIG_XTENSA_VARIANT_HAVE_PERF_EVENTS=y: + + arch/xtensa/kernel/perf_event.c:420:29: error: passing argument 3 of + ‘cpuhp_setup_state’ from incompatible pointer type + +Cc: stable@vger.kernel.org +Fixes: 25a77b55e74c ("xtensa/perf: Convert the hotplug notifier to state machine callbacks") +Signed-off-by: Max Filippov +Signed-off-by: Greg Kroah-Hartman + +--- + arch/xtensa/kernel/perf_event.c | 2 +- + 1 file changed, 1 insertion(+), 1 deletion(-) + +--- a/arch/xtensa/kernel/perf_event.c ++++ b/arch/xtensa/kernel/perf_event.c +@@ -404,7 +404,7 @@ static struct pmu xtensa_pmu = { + .read = xtensa_pmu_read, + }; + +-static int xtensa_pmu_setup(int cpu) ++static int xtensa_pmu_setup(unsigned int cpu) + { + unsigned i; +