* arch/arm64/kernel/mte.c:121:15: error: 'mte_enable_kernel_sync' undeclared here (not in a function); did you mean 'mte_enable_kernel'? @ 2021-02-02 10:09 ` Anders Roxell 0 siblings, 0 replies; 6+ messages in thread From: Anders Roxell @ 2021-02-02 10:09 UTC (permalink / raw) To: Stephen Rothwell, Andrey Konovalov Cc: Linux-Next Mailing List, lkft-triage, Catalin Marinas, Will Deacon, Linux ARM Hi, I've seen this failure on tag next-20210202: arch/arm64/kernel/mte.c:121:15: error: 'mte_enable_kernel_sync' undeclared here (not in a function); did you mean 'mte_enable_kernel'? I think it may be a merge conflict that didn't get resolved correctly? The below change fixed the issue: diff --git a/arch/arm64/kernel/mte.c b/arch/arm64/kernel/mte.c index 275b5d0f38b8..8f5bd1293496 100644 --- a/arch/arm64/kernel/mte.c +++ b/arch/arm64/kernel/mte.c @@ -118,7 +118,7 @@ void mte_set_report_once(bool state) { WRITE_ONCE(report_fault_once, state); } -EXPORT_SYMBOL(mte_enable_kernel_sync); +EXPORT_SYMBOL(mte_enable_kernel); EXPORT_SYMBOL(mte_set_report_once); bool mte_report_once(void) Cheers, Anders ^ permalink raw reply related [flat|nested] 6+ messages in thread
* arch/arm64/kernel/mte.c:121:15: error: 'mte_enable_kernel_sync' undeclared here (not in a function); did you mean 'mte_enable_kernel'? @ 2021-02-02 10:09 ` Anders Roxell 0 siblings, 0 replies; 6+ messages in thread From: Anders Roxell @ 2021-02-02 10:09 UTC (permalink / raw) To: Stephen Rothwell, Andrey Konovalov Cc: Catalin Marinas, Linux-Next Mailing List, Will Deacon, Linux ARM, lkft-triage Hi, I've seen this failure on tag next-20210202: arch/arm64/kernel/mte.c:121:15: error: 'mte_enable_kernel_sync' undeclared here (not in a function); did you mean 'mte_enable_kernel'? I think it may be a merge conflict that didn't get resolved correctly? The below change fixed the issue: diff --git a/arch/arm64/kernel/mte.c b/arch/arm64/kernel/mte.c index 275b5d0f38b8..8f5bd1293496 100644 --- a/arch/arm64/kernel/mte.c +++ b/arch/arm64/kernel/mte.c @@ -118,7 +118,7 @@ void mte_set_report_once(bool state) { WRITE_ONCE(report_fault_once, state); } -EXPORT_SYMBOL(mte_enable_kernel_sync); +EXPORT_SYMBOL(mte_enable_kernel); EXPORT_SYMBOL(mte_set_report_once); bool mte_report_once(void) Cheers, Anders _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: arch/arm64/kernel/mte.c:121:15: error: 'mte_enable_kernel_sync' undeclared here (not in a function); did you mean 'mte_enable_kernel'? 2021-02-02 10:09 ` Anders Roxell @ 2021-02-02 13:46 ` Andrey Konovalov -1 siblings, 0 replies; 6+ messages in thread From: Andrey Konovalov @ 2021-02-02 13:46 UTC (permalink / raw) To: Anders Roxell, Andrew Morton Cc: Stephen Rothwell, Linux-Next Mailing List, lkft-triage, Catalin Marinas, Will Deacon, Linux ARM On Tue, Feb 2, 2021 at 11:09 AM Anders Roxell <anders.roxell@linaro.org> wrote: > > Hi, > > I've seen this failure on tag next-20210202: > arch/arm64/kernel/mte.c:121:15: error: 'mte_enable_kernel_sync' > undeclared here (not in a function); did you mean 'mte_enable_kernel'? > > I think it may be a merge conflict that didn't get resolved correctly? Yes, that patch was supposed to go on top of another one (which actually renames mte_enable_kernel to mte_enable_kernel_sync), but the latter wasn't picked up into mm. > The below change fixed the issue: > > diff --git a/arch/arm64/kernel/mte.c b/arch/arm64/kernel/mte.c > index 275b5d0f38b8..8f5bd1293496 100644 > --- a/arch/arm64/kernel/mte.c > +++ b/arch/arm64/kernel/mte.c > @@ -118,7 +118,7 @@ void mte_set_report_once(bool state) > { > WRITE_ONCE(report_fault_once, state); > } > -EXPORT_SYMBOL(mte_enable_kernel_sync); > +EXPORT_SYMBOL(mte_enable_kernel); > EXPORT_SYMBOL(mte_set_report_once); > > bool mte_report_once(void) The changed export also needs to be moved next to mte_enable_kernel(). Thanks! ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: arch/arm64/kernel/mte.c:121:15: error: 'mte_enable_kernel_sync' undeclared here (not in a function); did you mean 'mte_enable_kernel'? @ 2021-02-02 13:46 ` Andrey Konovalov 0 siblings, 0 replies; 6+ messages in thread From: Andrey Konovalov @ 2021-02-02 13:46 UTC (permalink / raw) To: Anders Roxell, Andrew Morton Cc: Stephen Rothwell, Catalin Marinas, Will Deacon, lkft-triage, Linux-Next Mailing List, Linux ARM On Tue, Feb 2, 2021 at 11:09 AM Anders Roxell <anders.roxell@linaro.org> wrote: > > Hi, > > I've seen this failure on tag next-20210202: > arch/arm64/kernel/mte.c:121:15: error: 'mte_enable_kernel_sync' > undeclared here (not in a function); did you mean 'mte_enable_kernel'? > > I think it may be a merge conflict that didn't get resolved correctly? Yes, that patch was supposed to go on top of another one (which actually renames mte_enable_kernel to mte_enable_kernel_sync), but the latter wasn't picked up into mm. > The below change fixed the issue: > > diff --git a/arch/arm64/kernel/mte.c b/arch/arm64/kernel/mte.c > index 275b5d0f38b8..8f5bd1293496 100644 > --- a/arch/arm64/kernel/mte.c > +++ b/arch/arm64/kernel/mte.c > @@ -118,7 +118,7 @@ void mte_set_report_once(bool state) > { > WRITE_ONCE(report_fault_once, state); > } > -EXPORT_SYMBOL(mte_enable_kernel_sync); > +EXPORT_SYMBOL(mte_enable_kernel); > EXPORT_SYMBOL(mte_set_report_once); > > bool mte_report_once(void) The changed export also needs to be moved next to mte_enable_kernel(). Thanks! _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: arch/arm64/kernel/mte.c:121:15: error: 'mte_enable_kernel_sync' undeclared here (not in a function); did you mean 'mte_enable_kernel'? 2021-02-02 13:46 ` Andrey Konovalov @ 2021-02-02 20:29 ` Stephen Rothwell -1 siblings, 0 replies; 6+ messages in thread From: Stephen Rothwell @ 2021-02-02 20:29 UTC (permalink / raw) To: Andrey Konovalov Cc: Anders Roxell, Andrew Morton, Linux-Next Mailing List, lkft-triage, Catalin Marinas, Will Deacon, Linux ARM [-- Attachment #1: Type: text/plain, Size: 2203 bytes --] Hi all, On Tue, 2 Feb 2021 14:46:24 +0100 Andrey Konovalov <andreyknvl@google.com> wrote: > > On Tue, Feb 2, 2021 at 11:09 AM Anders Roxell <anders.roxell@linaro.org> wrote: > > > > I've seen this failure on tag next-20210202: > > arch/arm64/kernel/mte.c:121:15: error: 'mte_enable_kernel_sync' > > undeclared here (not in a function); did you mean 'mte_enable_kernel'? > > > > I think it may be a merge conflict that didn't get resolved correctly? > > Yes, that patch was supposed to go on top of another one (which > actually renames mte_enable_kernel to mte_enable_kernel_sync), but the > latter wasn't picked up into mm. > > > The below change fixed the issue: > > > > diff --git a/arch/arm64/kernel/mte.c b/arch/arm64/kernel/mte.c > > index 275b5d0f38b8..8f5bd1293496 100644 > > --- a/arch/arm64/kernel/mte.c > > +++ b/arch/arm64/kernel/mte.c > > @@ -118,7 +118,7 @@ void mte_set_report_once(bool state) > > { > > WRITE_ONCE(report_fault_once, state); > > } > > -EXPORT_SYMBOL(mte_enable_kernel_sync); > > +EXPORT_SYMBOL(mte_enable_kernel); > > EXPORT_SYMBOL(mte_set_report_once); > > > > bool mte_report_once(void) > > The changed export also needs to be moved next to mte_enable_kernel(). > > Thanks! I have applied the following to linux-next today: From: Stephen Rothwell <sfr@canb.auug.org.au> Date: Wed, 3 Feb 2021 07:22:35 +1100 Subject: [PATCH] arm64-kasan-export-mte-symbols-for-kasan-tests-fix Signed-off-by: Stephen Rothwell <sfr@canb.auug.org.au> --- arch/arm64/kernel/mte.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm64/kernel/mte.c b/arch/arm64/kernel/mte.c index 275b5d0f38b8..8c7e32054172 100644 --- a/arch/arm64/kernel/mte.c +++ b/arch/arm64/kernel/mte.c @@ -113,12 +113,12 @@ void mte_enable_kernel(void) sysreg_clear_set(sctlr_el1, SCTLR_ELx_TCF_MASK, SCTLR_ELx_TCF_SYNC); isb(); } +EXPORT_SYMBOL(mte_enable_kernel); void mte_set_report_once(bool state) { WRITE_ONCE(report_fault_once, state); } -EXPORT_SYMBOL(mte_enable_kernel_sync); EXPORT_SYMBOL(mte_set_report_once); bool mte_report_once(void) -- 2.30.0 -- Cheers, Stephen Rothwell [-- Attachment #2: OpenPGP digital signature --] [-- Type: application/pgp-signature, Size: 488 bytes --] ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: arch/arm64/kernel/mte.c:121:15: error: 'mte_enable_kernel_sync' undeclared here (not in a function); did you mean 'mte_enable_kernel'? @ 2021-02-02 20:29 ` Stephen Rothwell 0 siblings, 0 replies; 6+ messages in thread From: Stephen Rothwell @ 2021-02-02 20:29 UTC (permalink / raw) To: Andrey Konovalov Cc: Anders Roxell, Catalin Marinas, Will Deacon, lkft-triage, Linux-Next Mailing List, Andrew Morton, Linux ARM [-- Attachment #1.1: Type: text/plain, Size: 2203 bytes --] Hi all, On Tue, 2 Feb 2021 14:46:24 +0100 Andrey Konovalov <andreyknvl@google.com> wrote: > > On Tue, Feb 2, 2021 at 11:09 AM Anders Roxell <anders.roxell@linaro.org> wrote: > > > > I've seen this failure on tag next-20210202: > > arch/arm64/kernel/mte.c:121:15: error: 'mte_enable_kernel_sync' > > undeclared here (not in a function); did you mean 'mte_enable_kernel'? > > > > I think it may be a merge conflict that didn't get resolved correctly? > > Yes, that patch was supposed to go on top of another one (which > actually renames mte_enable_kernel to mte_enable_kernel_sync), but the > latter wasn't picked up into mm. > > > The below change fixed the issue: > > > > diff --git a/arch/arm64/kernel/mte.c b/arch/arm64/kernel/mte.c > > index 275b5d0f38b8..8f5bd1293496 100644 > > --- a/arch/arm64/kernel/mte.c > > +++ b/arch/arm64/kernel/mte.c > > @@ -118,7 +118,7 @@ void mte_set_report_once(bool state) > > { > > WRITE_ONCE(report_fault_once, state); > > } > > -EXPORT_SYMBOL(mte_enable_kernel_sync); > > +EXPORT_SYMBOL(mte_enable_kernel); > > EXPORT_SYMBOL(mte_set_report_once); > > > > bool mte_report_once(void) > > The changed export also needs to be moved next to mte_enable_kernel(). > > Thanks! I have applied the following to linux-next today: From: Stephen Rothwell <sfr@canb.auug.org.au> Date: Wed, 3 Feb 2021 07:22:35 +1100 Subject: [PATCH] arm64-kasan-export-mte-symbols-for-kasan-tests-fix Signed-off-by: Stephen Rothwell <sfr@canb.auug.org.au> --- arch/arm64/kernel/mte.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm64/kernel/mte.c b/arch/arm64/kernel/mte.c index 275b5d0f38b8..8c7e32054172 100644 --- a/arch/arm64/kernel/mte.c +++ b/arch/arm64/kernel/mte.c @@ -113,12 +113,12 @@ void mte_enable_kernel(void) sysreg_clear_set(sctlr_el1, SCTLR_ELx_TCF_MASK, SCTLR_ELx_TCF_SYNC); isb(); } +EXPORT_SYMBOL(mte_enable_kernel); void mte_set_report_once(bool state) { WRITE_ONCE(report_fault_once, state); } -EXPORT_SYMBOL(mte_enable_kernel_sync); EXPORT_SYMBOL(mte_set_report_once); bool mte_report_once(void) -- 2.30.0 -- Cheers, Stephen Rothwell [-- Attachment #1.2: OpenPGP digital signature --] [-- Type: application/pgp-signature, Size: 488 bytes --] [-- Attachment #2: Type: text/plain, Size: 176 bytes --] _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply related [flat|nested] 6+ messages in thread
end of thread, other threads:[~2021-02-02 20:31 UTC | newest] Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2021-02-02 10:09 arch/arm64/kernel/mte.c:121:15: error: 'mte_enable_kernel_sync' undeclared here (not in a function); did you mean 'mte_enable_kernel'? Anders Roxell 2021-02-02 10:09 ` Anders Roxell 2021-02-02 13:46 ` Andrey Konovalov 2021-02-02 13:46 ` Andrey Konovalov 2021-02-02 20:29 ` Stephen Rothwell 2021-02-02 20:29 ` Stephen Rothwell
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.