linuxppc-dev.lists.ozlabs.org archive mirror
 help / color / mirror / Atom feed
* [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).