lgtm. you might need to add more info into patch header.

On Fri, Mar 11, 2022 at 1:05 AM Sundeep KOKKONDA
<[email protected]> wrote:
>
> Signed-off-by: Sundeep KOKKONDA <[email protected]>
> ---
>  meta/recipes-devtools/gcc/gcc-11.2.inc         |  1 +
>  .../gcc/0042-Fix-thread-stack-size-init.patch  | 18 ++++++++++++++++++
>  2 files changed, 19 insertions(+)
>  create mode 100644 
> meta/recipes-devtools/gcc/gcc/0042-Fix-thread-stack-size-init.patch
>
> diff --git a/meta/recipes-devtools/gcc/gcc-11.2.inc 
> b/meta/recipes-devtools/gcc/gcc-11.2.inc
> index 2394c86e29..0c6c54888c 100644
> --- a/meta/recipes-devtools/gcc/gcc-11.2.inc
> +++ b/meta/recipes-devtools/gcc/gcc-11.2.inc
> @@ -68,6 +68,7 @@ SRC_URI = "\
>             file://0003-CVE-2021-42574.patch \
>             file://0004-CVE-2021-42574.patch \
>             file://0001-CVE-2021-46195.patch \
> +          file://0042-Fix-thread-stack-size-init.patch \
>  "
>  SRC_URI[sha256sum] = 
> "d08edc536b54c372a1010ff6619dd274c0f1603aa49212ba20f7aa2cda36fa8b"
>
> diff --git 
> a/meta/recipes-devtools/gcc/gcc/0042-Fix-thread-stack-size-init.patch 
> b/meta/recipes-devtools/gcc/gcc/0042-Fix-thread-stack-size-init.patch
> new file mode 100644
> index 0000000000..ab463f252a
> --- /dev/null
> +++ b/meta/recipes-devtools/gcc/gcc/0042-Fix-thread-stack-size-init.patch
> @@ -0,0 +1,18 @@
> +Upstream-Status: Backport 
> [https://gcc.gnu.org/git/?p=gcc.git;a=commit;h=90e46074e6b3561ae7d8ebd205127f286cc0c6b6]
> +---
> +--- a/libsanitizer/sanitizer_common/sanitizer_posix_libcdep.cpp        
> 2022-03-08 03:04:25.871012986 -0800
> ++++ b/libsanitizer/sanitizer_common/sanitizer_posix_libcdep.cpp        
> 2022-03-10 23:28:20.131064433 -0800
> +@@ -166,9 +166,10 @@
> + #if !SANITIZER_GO
> + // TODO(glider): different tools may require different altstack size.
> + static uptr GetAltStackSize() {
> +-  // SIGSTKSZ is not enough.
> +-  static const uptr kAltStackSize = SIGSTKSZ * 4;
> +-  return kAltStackSize;
> ++  // Note: since GLIBC_2.31, SIGSTKSZ may be a function call, so this may be
> ++  // more costly that you think. However GetAltStackSize is only call 2-3 
> times
> ++  // per thread so don't cache the evaluation.
> ++  return SIGSTKSZ * 4;
> + }
> +
> + void SetAlternateSignalStack() {
> --
> 2.25.1
>
>
> 
>
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#163070): 
https://lists.openembedded.org/g/openembedded-core/message/163070
Mute This Topic: https://lists.openembedded.org/mt/89706923/21656
Group Owner: [email protected]
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to