Re: [PATCH 1/3] build: Format build items

2023-01-15 Thread Sebastian Huber
On 16.01.23 01:35, Chris Johns wrote: On 13/1/2023 1:54 am, Sebastian Huber wrote: On 12.01.23 15:44, Kinsey Moore wrote: The other two patches look fine to me. The use of dump() that results in this patch does several things: * Removal of whitespace This is fine for whitespace at the base

[PATCH v2] build: Format build items

2023-01-15 Thread Sebastian Huber
Use yaml.dump(data, default_flow_style=False, allow_unicode=True) with a custom representer for integer default values to format all build items. --- spec/build/bsps/aarch64/a53/optloadoff.yml| 2 +- spec/build/bsps/aarch64/a53/optnocachelen.yml | 2 +-

Re: [PATCH v1] move isr_level of SMP_lock_Context into ISR_lock_Context

2023-01-15 Thread Sebastian Huber
On 16.01.23 07:33, Zhu Zhongjie wrote: From: Zhongjie Zhu when Acquires an ISR lock in SMP system, it first call _ISR_Local_disable() to disable ISR, this is the same with in UP system, so we could use the same isr_level in ISR_lock_Context, this change will also save 4 Bytes of

Re: [PATCH] fix compiling warning

2023-01-15 Thread Sebastian Huber
On 13.01.23 10:49, Zhu Zhongjie wrote: diff --git a/cpukit/include/rtems/score/threaddispatch.h b/cpukit/include/rtems/score/threaddispatch.h index 589935823f..ce20378d02 100644 --- a/cpukit/include/rtems/score/threaddispatch.h +++ b/cpukit/include/rtems/score/threaddispatch.h @@ -224,15

[PATCH v1] move isr_level of SMP_lock_Context into ISR_lock_Context

2023-01-15 Thread Zhu Zhongjie
From: Zhongjie Zhu when Acquires an ISR lock in SMP system, it first call _ISR_Local_disable() to disable ISR, this is the same with in UP system, so we could use the same isr_level in ISR_lock_Context, this change will also save 4 Bytes of SMP_lock_Context. ---

Re: [PATCH 1/3] build: Format build items

2023-01-15 Thread Chris Johns
On 13/1/2023 1:54 am, Sebastian Huber wrote: > On 12.01.23 15:44, Kinsey Moore wrote: >> The other two patches look fine to me. The use of dump() that results in this >> patch does several things: >> * Removal of whitespace >> This is fine for whitespace at the base level of indentation.

Re: [PATCH rtems-tools] linkers/rtems-syms: Generate TLS symbols

2023-01-15 Thread Chris Johns
OK Thanks Chris On 13/1/2023 8:12 am, Kinsey Moore wrote: > When generating the symbol table for loadable modules, include TLS > symbols so that the modules can reference them. > --- > linkers/rtems-syms.cpp | 5 + > rtemstoolkit/rld-elf.cpp | 1 + > rtemstoolkit/rld-symbols.cpp |

Re: [PATCH v1 4/4] testsuites/libtest/dl11: Add DL test for TLS

2023-01-15 Thread Chris Johns
On 13/1/2023 12:51 pm, Kinsey Moore wrote: > On Thu, Jan 12, 2023 at 5:11 PM Joel Sherrill > wrote: > > Will this need to be added as an expected fail on other architectures?  > > Just wondering how many bsp test configuration files need touching  > > > I

Re: [PATCH 1/1] RSB: Mitigate too short error reports

2023-01-15 Thread Chris Johns
On 22/12/2022 9:09 pm, Frank Kühndel wrote: > On 12/21/22 00:06, Chris Johns wrote: >> On 21/12/2022 3:44 am, Frank Kuehndel wrote: >>> From: Frank Kühndel >>> >>> Close #4642 >>> --- >>>   source-builder/sb/ereport.py | 4 >>>   1 file changed, 4 insertions(+) >>> >>> diff --git

[PATCH v1] move isr_level of SMP_lock_Context into ISR_lock_Context

2023-01-15 Thread Zhu Zhongjie
From: Zhongjie Zhu when Acquires an ISR lock in SMP system, it first call _ISR_Local_disable() to disable ISR, this is the same with in UP system, so we could use the same isr_level in ISR_lock_Context, this change will also save 4 Bytes of SMP_lock_Context. ---

[PATCH] move isr_level of SMP_lock_Context into ISR_lock_Context

2023-01-15 Thread Zhu Zhongjie
From: Zhongjie Zhu when Acquires an ISR lock in SMP system, it first call _ISR_Local_disable() to disable ISR, this is the same with in UP system, so we could use the same isr_level in ISR_lock_Context, this change will also save 4 Bytes of SMP_lock_Context. ---