* [PATCH] arch/powerpc/Kconfig: fix typo in select statement
@ 2015-12-11 7:46 Valentin Rothberg
2015-12-11 8:13 ` Daniel Axtens
0 siblings, 1 reply; 3+ messages in thread
From: Valentin Rothberg @ 2015-12-11 7:46 UTC (permalink / raw)
To: benh, paulus, mpe; +Cc: linuxppc-dev, linux-kernel, Valentin Rothberg
The 'I' in sanitize was missing:
s/ARCH_HAS_UBSAN_SANTIZE_ALL/ARCH_HAS_UBSAN_SANITIZE_ALL/
Fixes: 257e4ee82dbd ("powerpc: enable UBSAN support")
Signed-off-by: Valentin Rothberg <valentinrothberg@gmail.com>
---
Detected with ./scripts/checkkconfigsymbols.py
arch/powerpc/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/powerpc/Kconfig b/arch/powerpc/Kconfig
index b0420a8c458e..4de86d145f17 100644
--- a/arch/powerpc/Kconfig
+++ b/arch/powerpc/Kconfig
@@ -157,7 +157,7 @@ config PPC
select EDAC_ATOMIC_SCRUB
select ARCH_HAS_DMA_SET_COHERENT_MASK
select HAVE_ARCH_SECCOMP_FILTER
- select ARCH_HAS_UBSAN_SANTIZE_ALL
+ select ARCH_HAS_UBSAN_SANITIZE_ALL
config GENERIC_CSUM
def_bool CPU_LITTLE_ENDIAN
--
2.1.4
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] arch/powerpc/Kconfig: fix typo in select statement
2015-12-11 7:46 [PATCH] arch/powerpc/Kconfig: fix typo in select statement Valentin Rothberg
@ 2015-12-11 8:13 ` Daniel Axtens
2015-12-11 9:51 ` Valentin Rothberg
0 siblings, 1 reply; 3+ messages in thread
From: Daniel Axtens @ 2015-12-11 8:13 UTC (permalink / raw)
To: Valentin Rothberg, benh, paulus, mpe
Cc: linuxppc-dev, linux-kernel, Valentin Rothberg, akpm
Thanks Valentin.
Ironically I first noticed the spelling mistake in the original UBSAN
patchset[1], but wanted it to work on powerpc straight away so I went
with the mis-spelled version. It's since been corrected upstream[2]; and
I was going to spin a v2 of my original patch with this fix, especially
since I tagged the original patch as an RFC only. Regardless, thank you:
this should keep -mm and next working in the mean time!
> Fixes: 257e4ee82dbd ("powerpc: enable UBSAN support")
Is this a SHA from linux-next?
Regards,
Daniel
[1]: https://lkml.org/lkml/2015/12/9/1008
[2]: https://lkml.org/lkml/2015/12/10/422
> Signed-off-by: Valentin Rothberg <valentinrothberg@gmail.com>
> ---
> Detected with ./scripts/checkkconfigsymbols.py
>
> arch/powerpc/Kconfig | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/powerpc/Kconfig b/arch/powerpc/Kconfig
> index b0420a8c458e..4de86d145f17 100644
> --- a/arch/powerpc/Kconfig
> +++ b/arch/powerpc/Kconfig
> @@ -157,7 +157,7 @@ config PPC
> select EDAC_ATOMIC_SCRUB
> select ARCH_HAS_DMA_SET_COHERENT_MASK
> select HAVE_ARCH_SECCOMP_FILTER
> - select ARCH_HAS_UBSAN_SANTIZE_ALL
> + select ARCH_HAS_UBSAN_SANITIZE_ALL
>
> config GENERIC_CSUM
> def_bool CPU_LITTLE_ENDIAN
> --
> 2.1.4
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] arch/powerpc/Kconfig: fix typo in select statement
2015-12-11 8:13 ` Daniel Axtens
@ 2015-12-11 9:51 ` Valentin Rothberg
0 siblings, 0 replies; 3+ messages in thread
From: Valentin Rothberg @ 2015-12-11 9:51 UTC (permalink / raw)
To: Daniel Axtens; +Cc: benh, paulus, mpe, linuxppc-dev, linux-kernel, akpm
Hi Daniel,
thank you for the links and the explanation.
On Dec 11 '15 19:13, Daniel Axtens wrote:
> Thanks Valentin.
>
> Ironically I first noticed the spelling mistake in the original UBSAN
> patchset[1], but wanted it to work on powerpc straight away so I went
> with the mis-spelled version. It's since been corrected upstream[2]; and
> I was going to spin a v2 of my original patch with this fix, especially
> since I tagged the original patch as an RFC only. Regardless, thank you:
> this should keep -mm and next working in the mean time!
I haven't seen the fix, sorry for the noise then.
> > Fixes: 257e4ee82dbd ("powerpc: enable UBSAN support")
> Is this a SHA from linux-next?
Yes, it's Linux next. Retrospectively, that was a bad idea :(
Kind regards,
Valentin
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2015-12-11 9:52 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-12-11 7:46 [PATCH] arch/powerpc/Kconfig: fix typo in select statement Valentin Rothberg
2015-12-11 8:13 ` Daniel Axtens
2015-12-11 9:51 ` Valentin Rothberg
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).