All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] ubsan: cosmetic fix to Kconfig text
@ 2016-02-06  0:05 Yang Shi
  2016-02-08  7:48 ` Andrey Ryabinin
  0 siblings, 1 reply; 2+ messages in thread
From: Yang Shi @ 2016-02-06  0:05 UTC (permalink / raw)
  To: aryabinin; +Cc: linux-kernel, linaro-kernel, yang.shi

When enabling UBSAN_SANITIZE_ALL, the kernel image size gets increased
significantly (~3x). So, it sounds better to have some note in Kconfig.

And, fixed a typo.

Signed-off-by: Yang Shi <yang.shi@linaro.org>
---
 lib/Kconfig.ubsan | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/lib/Kconfig.ubsan b/lib/Kconfig.ubsan
index 49518fb..e07c1ba 100644
--- a/lib/Kconfig.ubsan
+++ b/lib/Kconfig.ubsan
@@ -18,6 +18,8 @@ config UBSAN_SANITIZE_ALL
 	  This option activates instrumentation for the entire kernel.
 	  If you don't enable this option, you have to explicitly specify
 	  UBSAN_SANITIZE := y for the files/directories you want to check for UB.
+	  Enabling this option will get kernel image size increased
+	  significantly.
 
 config UBSAN_ALIGNMENT
 	bool "Enable checking of pointers alignment"
@@ -25,5 +27,5 @@ config UBSAN_ALIGNMENT
 	default y if !HAVE_EFFICIENT_UNALIGNED_ACCESS
 	help
 	  This option enables detection of unaligned memory accesses.
-	  Enabling this option on architectures that support unalligned
+	  Enabling this option on architectures that support unaligned
 	  accesses may produce a lot of false positives.
-- 
2.0.2

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] ubsan: cosmetic fix to Kconfig text
  2016-02-06  0:05 [PATCH] ubsan: cosmetic fix to Kconfig text Yang Shi
@ 2016-02-08  7:48 ` Andrey Ryabinin
  0 siblings, 0 replies; 2+ messages in thread
From: Andrey Ryabinin @ 2016-02-08  7:48 UTC (permalink / raw)
  To: Yang Shi; +Cc: linux-kernel, linaro-kernel, Andrew Morton

On 02/06/2016 03:05 AM, Yang Shi wrote:
> When enabling UBSAN_SANITIZE_ALL, the kernel image size gets increased
> significantly (~3x). So, it sounds better to have some note in Kconfig.
> 
> And, fixed a typo.
> 
> Signed-off-by: Yang Shi <yang.shi@linaro.org>
> ---

Acked-by: Andrey Ryabinin <aryabinin@virtuozzo.com>

>  lib/Kconfig.ubsan | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/lib/Kconfig.ubsan b/lib/Kconfig.ubsan
> index 49518fb..e07c1ba 100644
> --- a/lib/Kconfig.ubsan
> +++ b/lib/Kconfig.ubsan
> @@ -18,6 +18,8 @@ config UBSAN_SANITIZE_ALL
>  	  This option activates instrumentation for the entire kernel.
>  	  If you don't enable this option, you have to explicitly specify
>  	  UBSAN_SANITIZE := y for the files/directories you want to check for UB.
> +	  Enabling this option will get kernel image size increased
> +	  significantly.
>  
>  config UBSAN_ALIGNMENT
>  	bool "Enable checking of pointers alignment"
> @@ -25,5 +27,5 @@ config UBSAN_ALIGNMENT
>  	default y if !HAVE_EFFICIENT_UNALIGNED_ACCESS
>  	help
>  	  This option enables detection of unaligned memory accesses.
> -	  Enabling this option on architectures that support unalligned
> +	  Enabling this option on architectures that support unaligned
>  	  accesses may produce a lot of false positives.
> 

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2016-02-08  7:47 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-02-06  0:05 [PATCH] ubsan: cosmetic fix to Kconfig text Yang Shi
2016-02-08  7:48 ` Andrey Ryabinin

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.