6.1.113-rc1 review
Subsystems (5)
Greg Kroah-Hartman
maintainerPATCH 798/798 Revert "iommu/vt-d: Retrieve IOMMU perfmon capability information"
In reply to Greg Kroah-Hartman6.1-stable review patch. If anyone has any objections, please let me know.
------------------
From: Jack Wang <[email protected]>
This reverts commit 586e19c88a0cb58b6ff45ae085b3dd200d862153 which is commit a6a5006dad572a53b5df3f47e1471d207ae9ba49 upstream.
This commit is pulled in due to dependency for:8c91a4bfc7f8 ("iommu: Fix compilation without CONFIG_IOMMU_INTEL")
But the patch itself is part of a patchset, should not only include one, and it lead to boot hang on on Kernel 6.1.83+ with Dell PowerEdge R770 and Intel Xeon 6710E, so revert it for stable 6.1.112
--- drivers/iommu/intel/Kconfig | 11 -- drivers/iommu/intel/Makefile | 1 drivers/iommu/intel/dmar.c | 7 - drivers/iommu/intel/iommu.h | 43 ---------- drivers/iommu/intel/perfmon.c | 172 ------------------------------------------ drivers/iommu/intel/perfmon.h | 40 --------- 6 files changed, 1 insertion(+), 273 deletions(-) delete mode 100644 drivers/iommu/intel/perfmon.c delete mode 100644 drivers/iommu/intel/perfmon.h
drivers/iommu/intel/Kconfig +0 -11
@@ -96,15 +96,4 @@ config INTEL_IOMMU_SCALABLE_MODE_DEFAULT passing intel_iommu=sm_on to the kernel. If not sure, please use the default value. -config INTEL_IOMMU_PERF_EVENTS - def_bool y - bool "Intel IOMMU performance events" - depends on INTEL_IOMMU && PERF_EVENTS - help - Selecting this option will enable the performance monitoring - infrastructure in the Intel IOMMU. It collects information about - key events occurring during operation of the remapping hardware, - to aid performance tuning and debug. These are available on modern - processors which support Intel VT-d 4.0 and later. - endif # INTEL_IOMMU
drivers/iommu/intel/Makefile +0 -1
@@ -8,4 +8,3 @@ obj-$(CONFIG_INTEL_IOMMU_SVM) += svm.o ifdef CONFIG_INTEL_IOMMU obj-$(CONFIG_IRQ_REMAP) += irq_remapping.o endif -obj-$(CONFIG_INTEL_IOMMU_PERF_EVENTS) += perfmon.o
drivers/iommu/intel/dmar.c +0 -7
@@ -34,7 +34,6 @@#include "../irq_remapping.h" #include "perf.h" #include "trace.h" -#include "perfmon.h" typedef int (*dmar_res_handler_t )(struct acpi_dmar_header *,void *);struct dmar_res_callback {
@@ -1105,9 +1104,6 @@ static int alloc_iommu(struct dmar_drhd_if (sts &DMA_GSTS_QIES ) iommu->gcmd |=DMA_GCMD_QIE ; -if (alloc_iommu_pmu (iommu)) -pr_debug ("Cannot alloc PMU for iommu (seq_id = %d)\n" , iommu->seq_id); -raw_spin_lock_init (&iommu->register_lock);/*
@@ -1135,7 +1131,6 @@ static int alloc_iommu(struct dmar_drhd_ err_sysfs:iommu_device_sysfs_remove (&iommu->iommu); err_unmap: -free_iommu_pmu (iommu);unmap_iommu (iommu); error_free_seq_id:ida_free (&dmar_seq_ids, iommu->seq_id);
@@ -1151,8 +1146,6 @@ static void free_iommu(struct intel_iommiommu_device_sysfs_remove (&iommu->iommu); } -free_iommu_pmu (iommu); -if (iommu->irq) {if (iommu->pr_irq) {free_irq (iommu->pr_irq, iommu);
drivers/iommu/intel/iommu.h +1 -42
79 lines not on this page · raw message
drivers/iommu/intel/perfmon.c deleted +0 -172
172 lines not on this page · raw message
drivers/iommu/intel/perfmon.h deleted +0 -40
@@ -1,40 +0,0 @@ -/* SPDX-License-Identifier: GPL-2.0 */ - -/* -* PERFCFGOFF_REG, PERFFRZOFF_REG -* PERFOVFOFF_REG, PERFCNTROFF_REG -*/ -#define IOMMU_PMU_NUM_OFF_REGS 4 -#define IOMMU_PMU_OFF_REGS_STEP 4 - -#define IOMMU_PMU_CFG_OFFSET 0x100 -#define IOMMU_PMU_CFG_CNTRCAP_OFFSET 0x80 -#define IOMMU_PMU_CFG_CNTREVCAP_OFFSET 0x84 -#define IOMMU_PMU_CFG_SIZE 0x8 -#define IOMMU_PMU_CFG_FILTERS_OFFSET 0x4 - -#define IOMMU_PMU_CAP_REGS_STEP 8 - -#define iommu_cntrcap_pcc (p) ((p) & 0x1) -#define iommu_cntrcap_cw (p) (((p) >> 8) & 0xff) -#define iommu_cntrcap_ios (p) (((p) >> 16) & 0x1) -#define iommu_cntrcap_egcnt (p) (((p) >> 28) & 0xf) - -#define iommu_event_select (p) ((p) & 0xfffffff) -#define iommu_event_group (p) (((p) >> 28) & 0xf) - -#ifdef CONFIG_INTEL_IOMMU_PERF_EVENTS -int alloc_iommu_pmu (struct intel_iommu *iommu); -void free_iommu_pmu (struct intel_iommu *iommu); -#else -static inline int -alloc_iommu_pmu (struct intel_iommu *iommu) -{ -return 0 ; -} - -static inline void -free_iommu_pmu (struct intel_iommu *iommu) -{ -} -#endif /* CONFIG_INTEL_IOMMU_PERF_EVENTS */
Quoted 9 lines · Greg Kroah-Hartman
This is the start of the stable review cycle for the 6.1.113 release. There are 798 patches in this series, all will be posted as a response to this one. If anyone has any issues with these being applied, please let me know.
Responses should be made by Wed, 16 Oct 2024 14:09:57 +0000. Anything received after that time might be too late.
The whole patch series can be found in one patch at:
Quoted 8 lines
https://www.kernel.org/pub/linux/kernel/v6.x/stable-review/patch-6.1.113-rc1.gzor in the git tree and branch at:
git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.gitlinux-6.1.y
and the diffstat can be found below.thanks,
greg k-h
On ARCH_BRCMSTB using 32-bit and 64-bit ARM kernels, build tested on BMIPS_GENERIC:
Signature · 1 line
Florian
Am 14.10.2024 um 16:09 schrieb Greg Kroah-Hartman:
Quoted 4 lines
This is the start of the stable review cycle for the 6.1.113 release. There are 798 patches in this series, all will be posted as a response to this one. If anyone has any issues with these being applied, please let me know.
Builds, boots and works on my 2-socket Ivy Bridge Xeon E5-2697 v2 server. No dmesg oddities or regressions found.
Beste Grüße,
Peter Schneider
Signature · 8 lines
Climb the mountain not to plant your flag, but to embrace the challenge, enjoy the air and behold the view. Climb it so you can see the world, not so the world can see you. -- David McCullough Jr. OpenPGP: 0xA3828BD796CCE11A8CADE8866E3A92C92C3FF244 Download: https://www.peters-netzplatz.de/download/pschneider1968_pub.asc https://keys.mailvelope.com/pks/lookup?op=get&[email protected] https://keys.mailvelope.com/pks/lookup?op=get&[email protected]
Jon Hunter
maintainerHi Greg,
Quoted 20 lines · Greg Kroah-Hartman
This is the start of the stable review cycle for the 6.1.113 release. There are 798 patches in this series, all will be posted as a response to this one. If anyone has any issues with these being applied, please let me know.
Responses should be made by Wed, 16 Oct 2024 14:09:57 +0000. Anything received after that time might be too late.
The whole patch series can be found in one patch at:
https://www.kernel.org/pub/linux/kernel/v6.x/stable-review/patch-6.1.113-rc1.gzor in the git tree and branch at:
git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.gitlinux-6.1.y
and the diffstat can be found below.thanks,
greg k-h
-------------
Pseudo-Shortlog of commits:
...
Quoted 2 lines
Oleksij Rempel <[email protected]>
clk: imx6ul: add ethernet refclock mux support
I am seeing the following build issue for ARM multi_v7_defconfig and bisect is point to the commit ...
drivers/clk/imx/clk-imx6ul.c: In function ‘imx6ul_clocks_init’:drivers/clk/imx/clk-imx6ul.c:487:34: error: implicit declaration of function ‘imx_obtain_fixed_of_clock’; did you mean ‘imx_obtain_fixed_clock’? [-Werror=implicit-function-declaration]
hws[IMX6UL_CLK_ENET1_REF_PAD] = imx_obtain_fixed_of_clock(ccm_node, "enet1_ref_pad", 0);
^~~~~~~~~~~~~~~~~~~~~~~~~
imx_obtain_fixed_clock
drivers/clk/imx/clk-imx6ul.c:487:32: warning: assignment makes pointer from integer without a cast [-Wint-conversion]
hws[IMX6UL_CLK_ENET1_REF_PAD] = imx_obtain_fixed_of_clock(ccm_node, "enet1_ref_pad", 0);
^
drivers/clk/imx/clk-imx6ul.c:489:34: error: implicit declaration of function ‘imx_clk_gpr_mux’; did you mean ‘imx_clk_hw_mux’? [-Werror=implicit-function-declaration]
hws[IMX6UL_CLK_ENET1_REF_SEL] = imx_clk_gpr_mux("enet1_ref_sel", "fsl,imx6ul-iomuxc-gpr",
^~~~~~~~~~~~~~~
imx_clk_hw_mux
drivers/clk/imx/clk-imx6ul.c:489:32: warning: assignment makes pointer from integer without a cast [-Wint-conversion]
hws[IMX6UL_CLK_ENET1_REF_SEL] = imx_clk_gpr_mux("enet1_ref_sel", "fsl,imx6ul-iomuxc-gpr",
^
drivers/clk/imx/clk-imx6ul.c:492:32: warning: assignment makes pointer from integer without a cast [-Wint-conversion]
hws[IMX6UL_CLK_ENET2_REF_PAD] = imx_obtain_fixed_of_clock(ccm_node, "enet2_ref_pad", 0);
^drivers/clk/imx/clk-imx6ul.c:494:32: warning: assignment makes pointer from integer without a cast [-Wint-conversion]
hws[IMX6UL_CLK_ENET2_REF_SEL] = imx_clk_gpr_mux("enet2_ref_sel", "fsl,imx6ul-iomuxc-gpr",
Jon
Signature · 1 line
nvpublic
Jon Hunter
maintainerQuoted 24 lines · Jon Hunter
Hi Greg,
Quoted 14 lines · Greg Kroah-Hartman
This is the start of the stable review cycle for the 6.1.113 release. There are 798 patches in this series, all will be posted as a response to this one. If anyone has any issues with these being applied, please let me know.
Responses should be made by Wed, 16 Oct 2024 14:09:57 +0000. Anything received after that time might be too late.
The whole patch series can be found in one patch at:
https://www.kernel.org/pub/linux/kernel/v6.x/stable-review/patch-6.1.113-rc1.gz
or in the git tree and branch at:
git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git linux-6.1.y and the diffstat can be found below.Quoted 6 lines
thanks,
greg k-h
-------------
Pseudo-Shortlog of commits:
Quoted 8 lines
...
Quoted 2 lines
Oleksij Rempel <[email protected]>
clk: imx6ul: add ethernet refclock mux supportI am seeing the following build issue for ARM
multi_v7_defconfigand bisect is point to the commit ...
To be clear, I meant the 'above' commit.
Jon
Signature · 1 line
nvpublic
Greg Kroah-Hartman
maintainerQuoted 28 lines · Jon Hunter
Quoted 26 lines · Jon Hunter
Hi Greg,
Quoted 20 lines · Greg Kroah-Hartman
This is the start of the stable review cycle for the 6.1.113 release. There are 798 patches in this series, all will be posted as a response to this one. If anyone has any issues with these being applied, please let me know.
Responses should be made by Wed, 16 Oct 2024 14:09:57 +0000. Anything received after that time might be too late.
The whole patch series can be found in one patch at:
https://www.kernel.org/pub/linux/kernel/v6.x/stable-review/patch-6.1.113-rc1.gz
or in the git tree and branch at:
git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git linux-6.1.y and the diffstat can be found below.thanks,
greg k-h
-------------
Pseudo-Shortlog of commits:...
Quoted 8 lines
Quoted 6 lines
Quoted 2 lines
Oleksij Rempel <[email protected]>
clk: imx6ul: add ethernet refclock mux supportI am seeing the following build issue for ARM
multi_v7_defconfigand bisect is point to the commit ...To be clear, I meant the 'above' commit.
Thanks, let me go drop this (and the other ones around it) and push out some -rc2 releases.
greg k-h
Hi!
Quoted 8 lines
Quoted 2 lines
-------------
Pseudo-Shortlog of commits:...
Quoted 2 lines
Oleksij Rempel <[email protected]>
clk: imx6ul: add ethernet refclock mux support
Quoted 8 lines
I am seeing the following build issue for ARM
multi_v7_defconfigand bisect is point to the commit ...
drivers/clk/imx/clk-imx6ul.c: In function ‘imx6ul_clocks_init’:drivers/clk/imx/clk-imx6ul.c:487:34: error: implicit declaration of function ‘imx_obtain_fixed_of_clock’; did you mean ‘imx_obtain_fixed_clock’? [-Werror=implicit-function-declaration]hws[IMX6UL_CLK_ENET1_REF_PAD] = imx_obtain_fixed_of_clock(ccm_node, "enet1_ref_pad", 0); ^~~~~~~~~~~~~~~~~~~~~~~~~ imx_obtain_fixed_clock
CIP testing sees same problem:
https://gitlab.com/cip-project/cip-testing/linux-stable-rc-ci/-/jobs/8083991401
Best regards,
Pavel
Signature · 2 lines
DENX Software Engineering GmbH, Managing Director: Erika Unter HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Quoted 49 lines · Jon Hunter
Quoted 20 lines · Greg Kroah-Hartman
This is the start of the stable review cycle for the 6.1.113 release. There are 798 patches in this series, all will be posted as a response to this one. If anyone has any issues with these being applied, please let me know.
Responses should be made by Wed, 16 Oct 2024 14:09:57 +0000. Anything received after that time might be too late.
The whole patch series can be found in one patch at:
https://www.kernel.org/pub/linux/kernel/v6.x/stable-review/patch-6.1.113-rc1.gzor in the git tree and branch at:
git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.gitlinux-6.1.y
and the diffstat can be found below.thanks,
greg k-h
-------------
Pseudo-Shortlog of commits:...
Quoted 2 lines
Oleksij Rempel <[email protected]>
clk: imx6ul: add ethernet refclock mux supportI am seeing the following build issue for ARM
multi_v7_defconfigand bisect is point to the commit ...
drivers/clk/imx/clk-imx6ul.c: In function ‘imx6ul_clocks_init’:drivers/clk/imx/clk-imx6ul.c:487:34: error: implicit declaration of function ‘imx_obtain_fixed_of_clock’; did you mean ‘imx_obtain_fixed_clock’? [-Werror=implicit-function-declaration]hws[IMX6UL_CLK_ENET1_REF_PAD] = imx_obtain_fixed_of_clock(ccm_node, "enet1_ref_pad", 0); ^~~~~~~~~~~~~~~~~~~~~~~~~ imx_obtain_fixed_clock drivers/clk/imx/clk-imx6ul.c:487:32: warning: assignment makes pointer from integer without a cast [-Wint-conversion] hws[IMX6UL_CLK_ENET1_REF_PAD] = imx_obtain_fixed_of_clock(ccm_node, "enet1_ref_pad", 0); ^ drivers/clk/imx/clk-imx6ul.c:489:34: error: implicit declaration of function ‘imx_clk_gpr_mux’; did you mean ‘imx_clk_hw_mux’? [-Werror=implicit-function-declaration] hws[IMX6UL_CLK_ENET1_REF_SEL] = imx_clk_gpr_mux("enet1_ref_sel", "fsl,imx6ul-iomuxc-gpr", ^~~~~~~~~~~~~~~ imx_clk_hw_mux drivers/clk/imx/clk-imx6ul.c:489:32: warning: assignment makes pointer from integer without a cast [-Wint-conversion] hws[IMX6UL_CLK_ENET1_REF_SEL] = imx_clk_gpr_mux("enet1_ref_sel", "fsl,imx6ul-iomuxc-gpr", ^ drivers/clk/imx/clk-imx6ul.c:492:32: warning: assignment makes pointer from integer without a cast [-Wint-conversion] hws[IMX6UL_CLK_ENET2_REF_PAD] = imx_obtain_fixed_of_clock(ccm_node, "enet2_ref_pad", 0); ^
Quoted 2 lines
drivers/clk/imx/clk-imx6ul.c:494:32: warning: assignment makes pointer from integer without a cast [-Wint-conversion]
hws[IMX6UL_CLK_ENET2_REF_SEL] = imx_clk_gpr_mux("enet2_ref_sel", "fsl,imx6ul-iomuxc-gpr",
Missing backports of the other clock-related patches in the original series[1]?imx_obtain_fixed_clock() was introduced in commit 7757731053406dd0 ("clk: imx: add imx_obtain_fixed_of_clock()"), but some of the other patches from that series may be needed, too?
[1] https://lore.kernel.org/all/[email protected]/
Gr{oetje,eeting}s,
Geert
Signature · 5 lines
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- [email protected] In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds
CC Oleksij
Quoted 53 lines · Geert Uytterhoeven
Quoted 49 lines · Jon Hunter
Quoted 20 lines · Greg Kroah-Hartman
This is the start of the stable review cycle for the 6.1.113 release. There are 798 patches in this series, all will be posted as a response to this one. If anyone has any issues with these being applied, please let me know.
Responses should be made by Wed, 16 Oct 2024 14:09:57 +0000. Anything received after that time might be too late.
The whole patch series can be found in one patch at:
https://www.kernel.org/pub/linux/kernel/v6.x/stable-review/patch-6.1.113-rc1.gzor in the git tree and branch at:
git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.gitlinux-6.1.y
and the diffstat can be found below.thanks,
greg k-h
-------------
Pseudo-Shortlog of commits:...
Quoted 2 lines
Oleksij Rempel <[email protected]>
clk: imx6ul: add ethernet refclock mux supportI am seeing the following build issue for ARM
multi_v7_defconfigand bisect is point to the commit ...
drivers/clk/imx/clk-imx6ul.c: In function ‘imx6ul_clocks_init’:drivers/clk/imx/clk-imx6ul.c:487:34: error: implicit declaration of function ‘imx_obtain_fixed_of_clock’; did you mean ‘imx_obtain_fixed_clock’? [-Werror=implicit-function-declaration]hws[IMX6UL_CLK_ENET1_REF_PAD] = imx_obtain_fixed_of_clock(ccm_node, "enet1_ref_pad", 0); ^~~~~~~~~~~~~~~~~~~~~~~~~ imx_obtain_fixed_clock drivers/clk/imx/clk-imx6ul.c:487:32: warning: assignment makes pointer from integer without a cast [-Wint-conversion] hws[IMX6UL_CLK_ENET1_REF_PAD] = imx_obtain_fixed_of_clock(ccm_node, "enet1_ref_pad", 0); ^ drivers/clk/imx/clk-imx6ul.c:489:34: error: implicit declaration of function ‘imx_clk_gpr_mux’; did you mean ‘imx_clk_hw_mux’? [-Werror=implicit-function-declaration] hws[IMX6UL_CLK_ENET1_REF_SEL] = imx_clk_gpr_mux("enet1_ref_sel", "fsl,imx6ul-iomuxc-gpr", ^~~~~~~~~~~~~~~ imx_clk_hw_mux drivers/clk/imx/clk-imx6ul.c:489:32: warning: assignment makes pointer from integer without a cast [-Wint-conversion] hws[IMX6UL_CLK_ENET1_REF_SEL] = imx_clk_gpr_mux("enet1_ref_sel", "fsl,imx6ul-iomuxc-gpr", ^ drivers/clk/imx/clk-imx6ul.c:492:32: warning: assignment makes pointer from integer without a cast [-Wint-conversion] hws[IMX6UL_CLK_ENET2_REF_PAD] = imx_obtain_fixed_of_clock(ccm_node, "enet2_ref_pad", 0); ^Quoted 2 lines
drivers/clk/imx/clk-imx6ul.c:494:32: warning: assignment makes pointer from integer without a cast [-Wint-conversion]
hws[IMX6UL_CLK_ENET2_REF_SEL] = imx_clk_gpr_mux("enet2_ref_sel", "fsl,imx6ul-iomuxc-gpr",
Quoted 7 lines
Missing backports of the other clock-related patches in the original series[1]?
imx_obtain_fixed_clock()was introduced in commit7757731053406dd0("clk: imx: addimx_obtain_fixed_of_clock()"), but some of the other patches from that series may be needed, too?
Gr{oetje,eeting}s,
Geert
Signature · 5 lines
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- [email protected] In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds
Hi all,
Quoted 63 lines · Geert Uytterhoeven
CC Oleksij
Quoted 60 lines · Geert Uytterhoeven
Quoted 49 lines · Jon Hunter
Quoted 20 lines · Greg Kroah-Hartman
This is the start of the stable review cycle for the 6.1.113 release. There are 798 patches in this series, all will be posted as a response to this one. If anyone has any issues with these being applied, please let me know.
Responses should be made by Wed, 16 Oct 2024 14:09:57 +0000. Anything received after that time might be too late.
The whole patch series can be found in one patch at:
https://www.kernel.org/pub/linux/kernel/v6.x/stable-review/patch-6.1.113-rc1.gzor in the git tree and branch at:
git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.gitlinux-6.1.y
and the diffstat can be found below.thanks,
greg k-h
-------------
Pseudo-Shortlog of commits:...
Quoted 2 lines
Oleksij Rempel <[email protected]>
clk: imx6ul: add ethernet refclock mux supportI am seeing the following build issue for ARM
multi_v7_defconfigand bisect is point to the commit ...
drivers/clk/imx/clk-imx6ul.c: In function ‘imx6ul_clocks_init’:drivers/clk/imx/clk-imx6ul.c:487:34: error: implicit declaration of function ‘imx_obtain_fixed_of_clock’; did you mean ‘imx_obtain_fixed_clock’? [-Werror=implicit-function-declaration]hws[IMX6UL_CLK_ENET1_REF_PAD] = imx_obtain_fixed_of_clock(ccm_node, "enet1_ref_pad", 0); ^~~~~~~~~~~~~~~~~~~~~~~~~ imx_obtain_fixed_clock drivers/clk/imx/clk-imx6ul.c:487:32: warning: assignment makes pointer from integer without a cast [-Wint-conversion] hws[IMX6UL_CLK_ENET1_REF_PAD] = imx_obtain_fixed_of_clock(ccm_node, "enet1_ref_pad", 0); ^ drivers/clk/imx/clk-imx6ul.c:489:34: error: implicit declaration of function ‘imx_clk_gpr_mux’; did you mean ‘imx_clk_hw_mux’? [-Werror=implicit-function-declaration] hws[IMX6UL_CLK_ENET1_REF_SEL] = imx_clk_gpr_mux("enet1_ref_sel", "fsl,imx6ul-iomuxc-gpr", ^~~~~~~~~~~~~~~ imx_clk_hw_mux drivers/clk/imx/clk-imx6ul.c:489:32: warning: assignment makes pointer from integer without a cast [-Wint-conversion] hws[IMX6UL_CLK_ENET1_REF_SEL] = imx_clk_gpr_mux("enet1_ref_sel", "fsl,imx6ul-iomuxc-gpr", ^ drivers/clk/imx/clk-imx6ul.c:492:32: warning: assignment makes pointer from integer without a cast [-Wint-conversion] hws[IMX6UL_CLK_ENET2_REF_PAD] = imx_obtain_fixed_of_clock(ccm_node, "enet2_ref_pad", 0); ^Quoted 2 lines
drivers/clk/imx/clk-imx6ul.c:494:32: warning: assignment makes pointer from integer without a cast [-Wint-conversion]
hws[IMX6UL_CLK_ENET2_REF_SEL] = imx_clk_gpr_mux("enet2_ref_sel", "fsl,imx6ul-iomuxc-gpr",Missing backports of the other clock-related patches in the original series[1]?
imx_obtain_fixed_clock()was introduced in commit7757731053406dd0("clk: imx: addimx_obtain_fixed_of_clock()"), but some of the other patches from that series may be needed, too?
Yes, I agree, at least commit 7757731053406dd0 is missing. Other patches in [1] series are only interesting if corresponding device trees was upstream.
Regards,
Oleksij
Signature · 4 lines
Pengutronix e.K. | | Steuerwalder Str. 21 | http://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
Quoted 10 lines · Greg Kroah-Hartman
This is the start of the stable review cycle for the 6.1.113 release. There are 798 patches in this series, all will be posted as a response to this one. If anyone has any issues with these being applied, please let me know.
Responses should be made by Wed, 16 Oct 2024 14:09:57 +0000. Anything received after that time might be too late.
The whole patch series can be found in one patch at:
Quoted 8 lines
https://www.kernel.org/pub/linux/kernel/v6.x/stable-review/patch-6.1.113-rc1.gzor in the git tree and branch at:
git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.gitlinux-6.1.y
and the diffstat can be found below.thanks,
greg k-h
As others reported,
The arm build broke on the stable-rc linux-6.1.y branch due to following build warnings / errors.
First seen on v6.1.112-799-gc060104c065d
GOOD: v6.1.112
BAD: v6.1.112-799-gc060104c065d
The bisection pointed to,
clk: imx6ul: add ethernet refclock mux support
[ Upstream commit 4e197ee880c24ecb63f7fe17449b3653bc64b03c ]
List of regressions,
* arm, build
- clang-19-allmodconfig
- clang-19-defconfig
- clang-19-imx_v6_v7_defconfig
- clang-19-lkftconfig
- clang-19-lkftconfig-no-kselftest-frag
- clang-nightly-defconfig
- clang-nightly-imx_v6_v7_defconfig
- clang-nightly-lkftconfig
- clang-nightly-lkftconfig-kselftest
- gcc-13-allmodconfig
- gcc-13-defconfig
- gcc-13-imx_v6_v7_defconfig
- gcc-13-lkftconfig
- gcc-13-lkftconfig-debug
- gcc-13-lkftconfig-debug-kmemleak
- gcc-13-lkftconfig-kasan
- gcc-13-lkftconfig-kselftest
- gcc-13-lkftconfig-kunit
- gcc-13-lkftconfig-libgpiod
- gcc-13-lkftconfig-no-kselftest-frag
- gcc-13-lkftconfig-perf
- gcc-13-lkftconfig-rcutorture
- gcc-8-defconfig
- gcc-8-imx_v6_v7_defconfig
Build log:
---------drivers/clk/imx/clk-imx6ul.c:489:41: error: implicit declaration of function 'imx_obtain_fixed_of_clock'; did you mean 'imx_obtain_fixed_clock'? [-Werror=implicit-function-declaration]
489 | hws[IMX6UL_CLK_ENET1_REF_PAD] =
imx_obtain_fixed_of_clock(ccm_node, "enet1_ref_pad", 0);
| ^~~~~~~~~~~~~~~~~~~~~~~~~
| imx_obtain_fixed_clockdrivers/clk/imx/clk-imx6ul.c:489:39: warning: assignment to 'struct clk_hw *' from 'int' makes pointer from integer without a cast [-Wint-conversion]
489 | hws[IMX6UL_CLK_ENET1_REF_PAD] =imx_obtain_fixed_of_clock(ccm_node, "enet1_ref_pad", 0);
| ^drivers/clk/imx/clk-imx6ul.c:491:41: error: implicit declaration of function 'imx_clk_gpr_mux'; did you mean 'imx_clk_hw_mux'? [-Werror=implicit-function-declaration]
491 | hws[IMX6UL_CLK_ENET1_REF_SEL] =
imx_clk_gpr_mux("enet1_ref_sel", "fsl,imx6ul-iomuxc-gpr",
| ^~~~~~~~~~~~~~~
| imx_clk_hw_muxdrivers/clk/imx/clk-imx6ul.c:491:39: warning: assignment to 'struct clk_hw *' from 'int' makes pointer from integer without a cast [-Wint-conversion]
491 | hws[IMX6UL_CLK_ENET1_REF_SEL] =
imx_clk_gpr_mux("enet1_ref_sel", "fsl,imx6ul-iomuxc-gpr",
| ^drivers/clk/imx/clk-imx6ul.c:494:39: warning: assignment to 'struct clk_hw *' from 'int' makes pointer from integer without a cast [-Wint-conversion]
494 | hws[IMX6UL_CLK_ENET2_REF_PAD] =imx_obtain_fixed_of_clock(ccm_node, "enet2_ref_pad", 0);
| ^drivers/clk/imx/clk-imx6ul.c:496:39: warning: assignment to 'struct clk_hw *' from 'int' makes pointer from integer without a cast [-Wint-conversion]
496 | hws[IMX6UL_CLK_ENET2_REF_SEL] =
imx_clk_gpr_mux("enet2_ref_sel", "fsl,imx6ul-iomuxc-gpr",
| ^drivers/clk/imx/clk-imx6ul.c:548:9: error: too few arguments to function 'imx_register_uart_clocks'
548 | imx_register_uart_clocks();
| ^~~~~~~~~~~~~~~~~~~~~~~~
In file included from drivers/clk/imx/clk-imx6ul.c:19:
drivers/clk/imx/clk.h:15:6: note: declared here
15 | void imx_register_uart_clocks(unsigned int clk_count);
| ^~~~~~~~~~~~~~~~~~~~~~~~cc1: some warnings being treated as errors
Build log link,
-------------
- https://qa-reports.linaro.org/lkft/linux-stable-rc-linux-6.1.y/build/v6.1.112-799-gc060104c065d/testrun/25432777/suite/build/test/gcc-13-defconfig/log
- https://storage.tuxsuite.com/public/linaro/lkft/builds/2nQwgipt3iHSCkBuMSKQIhwr95q/
metadata:
----
git describe: v6.1.112-799-gc060104c065d
git repo: https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git
git sha: c060104c065dc2884e301155e32dd955e6bb45b5
kernel config:
https://storage.tuxsuite.com/public/linaro/lkft/builds/2nQwgipt3iHSCkBuMSKQIhwr95q/config
build url: https://storage.tuxsuite.com/public/linaro/lkft/builds/2nQwgipt3iHSCkBuMSKQIhwr95q/
toolchain: clang-19 and gcc-13
config: defconfig
arch: arm
Steps to reproduce:
-------
# tuxmake --runtime podman --target-arch arm --toolchain gcc-13 --kconfig defconfig
--
Linaro LKFT
https://lkft.linaro.org
Quoted 10 lines · Greg Kroah-Hartman
This is the start of the stable review cycle for the 6.1.113 release. There are 798 patches in this series, all will be posted as a response to this one. If anyone has any issues with these being applied, please let me know.
Responses should be made by Wed, 16 Oct 2024 14:09:57 +0000. Anything received after that time might be too late.
The whole patch series can be found in one patch at:
Quoted 8 lines
https://www.kernel.org/pub/linux/kernel/v6.x/stable-review/patch-6.1.113-rc1.gzor in the git tree and branch at:
git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.gitlinux-6.1.y
and the diffstat can be found below.thanks,
greg k-h
The S390 build broke on the stable-rc linux-6.1.y branch due to following build warnings / errors.
First seen on v6.1.112-799-gc060104c065d
GOOD: v6.1.112
BAD: v6.1.112-799-gc060104c065d
List of regressions,
* s390, build
- clang-19-allnoconfig
- clang-19-defconfig
- clang-19-tinyconfig
- clang-nightly-allnoconfig
- clang-nightly-defconfig
- clang-nightly-tinyconfig
- gcc-13-allmodconfig
- gcc-13-allnoconfig
- gcc-13-defconfig
- gcc-13-tinyconfig
- gcc-8-allnoconfig
- gcc-8-defconfig-fe40093d
- gcc-8-tinyconfig
The bisection pointing to,
73e9443b9ea8d5a1b9b87c4988acc3daae363832
s390/traps: Handle early warnings gracefully
[ Upstream commit 3c4d0ae0671827f4b536cc2d26f8b9c54584ccc5 ]
Build log:
-------arch/s390/kernel/early.c: In function '__do_early_pgm_check':arch/s390/kernel/early.c:154:30: error: implicit declaration of function 'get_lowcore'; did you mean 'S390_lowcore'? [-Werror=implicit-function-declaration]
154 | struct lowcore *lc = get_lowcore();
| ^~~~~~~~~~~
| S390_lowcorearch/s390/kernel/early.c:154:30: warning: initialization of 'struct lowcore *' from 'int' makes pointer from integer without a cast [-Wint-conversion]
cc1: some warnings being treated as errors
Build log link:
---------
- https://storage.tuxsuite.com/public/linaro/lkft/builds/2nQwhZnhhAGtyIv600SDYdtypnI/
- https://qa-reports.linaro.org/lkft/linux-stable-rc-linux-6.1.y/build/v6.1.112-799-gc060104c065d/testrun/25432580/suite/build/test/gcc-13-defconfig/log
metadata:
----
git describe: v6.1.112-799-gc060104c065d
git repo: https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git
git sha: c060104c065dc2884e301155e32dd955e6bb45b5
kernel config:
https://storage.tuxsuite.com/public/linaro/lkft/builds/2nQwhZnhhAGtyIv600SDYdtypnI/config
build url: https://storage.tuxsuite.com/public/linaro/lkft/builds/2nQwhZnhhAGtyIv600SDYdtypnI/
toolchain: clang-19 and gcc-13
config: defconfig
arch: S390
Steps to reproduce:
-------
# tuxmake --runtime podman --target-arch s390 --toolchain gcc-13 --kconfig defconfig
--
Linaro LKFT
https://lkft.linaro.org
Heiko Carstens
maintainerQuoted 15 lines · Naresh Kamboju
On Mon, 14 Oct 2024 at 20:20, Greg Kroah-Hartman
The bisection pointing to,
73e9443b9ea8d5a1b9b87c4988acc3daae363832
s390/traps: Handle early warnings gracefully[ Upstream commit 3c4d0ae0671827f4b536cc2d26f8b9c54584ccc5 ]Build log:
-------arch/s390/kernel/early.c: In function '__do_early_pgm_check':arch/s390/kernel/early.c:154:30: error: implicit declaration of function 'get_lowcore'; did you mean 'S390_lowcore'? [-Werror=implicit-function-declaration]
Quoted 7 lines
154 | struct lowcore *lc = get_lowcore(); | ^~~~~~~~~~~ | S390_lowcore
arch/s390/kernel/early.c:154:30: warning: initialization of 'struct lowcore *' from 'int' makes pointer from integer without a cast [-Wint-conversion]
cc1: some warnings being treated as errors
Same here, please drop this patch.
Greg Kroah-Hartman
maintainerQuoted 24 lines · Heiko Carstens
Quoted 15 lines · Naresh Kamboju
On Mon, 14 Oct 2024 at 20:20, Greg Kroah-Hartman
The bisection pointing to,
73e9443b9ea8d5a1b9b87c4988acc3daae363832
s390/traps: Handle early warnings gracefully[ Upstream commit 3c4d0ae0671827f4b536cc2d26f8b9c54584ccc5 ]Build log:
-------arch/s390/kernel/early.c: In function '__do_early_pgm_check':arch/s390/kernel/early.c:154:30: error: implicit declaration of function 'get_lowcore'; did you mean 'S390_lowcore'? [-Werror=implicit-function-declaration]Quoted 7 lines
154 | struct lowcore *lc = get_lowcore(); | ^~~~~~~~~~~ | S390_lowcore
arch/s390/kernel/early.c:154:30: warning: initialization of 'struct lowcore *' from 'int' makes pointer from integer without a cast [-Wint-conversion]
cc1: some warnings being treated as errors
Quoted 2 lines
Same here, please drop this patch.
Now dropped from 6.1.y and 6.6.y, thanks!
greg k-h
Quoted 9 lines · Greg Kroah-Hartman
This is the start of the stable review cycle for the 6.1.113 release. There are 798 patches in this series, all will be posted as a response to this one. If anyone has any issues with these being applied, please let me know.
Responses should be made by Wed, 16 Oct 2024 14:09:57 +0000. Anything received after that time might be too late.
The whole patch series can be found in one patch at:
Quoted 9 lines
https://www.kernel.org/pub/linux/kernel/v6.x/stable-review/patch-6.1.113-rc1.gzor in the git tree and branch at:
git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.gitlinux-6.1.y
and the diffstat can be found below.thanks,
greg k-h
Compiled and booted on my test system. No dmesg regressions.
thanks,
-- Shuah