Re: crypt01 execution time

2014-11-27 Thread Chris Johns
On 27/11/2014 12:48 am, Joel Sherrill wrote: On 11/26/2014 01:12 AM, Sebastian Huber wrote: On 25/11/14 23:25, Joel Sherrill wrote: How long is this test supposed to run? It takes 4:42 using sis on my computer which is a 2.9 Ghz i7 . SIS is a slow simulator. On Qemu it runs much faster. Is

Re: [rtems commit] sync.c: Add asserts to document and check assumptions

2014-11-27 Thread Joel Sherrill
I am reverting this back to an appropriate comment and (void) on the return values. fsync_h on the IMFS is set to the default handler which is rtems_filesystem_default_fsync_or_fdatasync(). It is a one line method that returns EINVAL. I can either throw away the _Assert's or make them more compli

sis, psim,anf jmr3904 test results

2014-11-27 Thread Joel Sherrill
Hi I thought I would post these for commentary. sparc/sis === Passed: 511 Failed: 3 Timeouts: 7 Invalid:0 - Total:521 Failures: heapwalk.exe dl02.exe psx13.exe Timeouts: *** Interactive termios.exe, pppd.exe, capture.exe,

Re: Where should _fini and _init be defined?

2014-11-27 Thread Joel Sherrill
On 11/27/2014 10:56 AM, Hesham Moustafa wrote: Hi all, I had a compilation error when compiling RTEMS for Epiphany which is undefined reference to _init, and _fini. I got over it by defining dummy related functions at cpu.c, and hello world is working with it. I know this is not the right way, s

Where should _fini and _init be defined?

2014-11-27 Thread Hesham Moustafa
Hi all, I had a compilation error when compiling RTEMS for Epiphany which is undefined reference to _init, and _fini. I got over it by defining dummy related functions at cpu.c, and hello world is working with it. I know this is not the right way, so I'd like to ask here what's the correct solutio

Re: [PATCH] sync.c: Add asserts to document and check assumptions

2014-11-27 Thread Joel Sherrill
On 11/27/2014 09:43 AM, Sebastian Huber wrote: On 27/11/14 16:31, Joel Sherrill wrote: --- cpukit/libcsupport/src/sync.c | 11 +-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/cpukit/libcsupport/src/sync.c b/cpukit/libcsupport/src/sync.c index 653a177..e449cab 10064

Re: [PATCH] bdbuf: Fix race condition with sync active flag

2014-11-27 Thread Gedare Bloom
Looks OK. Need to apply to releases? On Thu, Nov 27, 2014 at 8:43 AM, Sebastian Huber wrote: > Bug report by Oleg Kravtsov: > > In rtems_bdbuf_swapout_processing() function there is the following > lines: > > if (bdbuf_cache.sync_active && !transfered_buffers) > { > > rtems_id sync_requester;

Re: [PATCH] sync.c: Add asserts to document and check assumptions

2014-11-27 Thread Sebastian Huber
On 27/11/14 16:31, Joel Sherrill wrote: --- cpukit/libcsupport/src/sync.c | 11 +-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/cpukit/libcsupport/src/sync.c b/cpukit/libcsupport/src/sync.c index 653a177..e449cab 100644 --- a/cpukit/libcsupport/src/sync.c +++ b/cpuki

Re: [PATCH 1/2] libcsupport: Delete malloc statistics

2014-11-27 Thread Gedare Bloom
Any particular reason for dropping memalign_calls? Any other stats get dropped? On Thu, Nov 27, 2014 at 7:26 AM, Sebastian Huber wrote: > Use the heap handler statistics instead. Add heap walk option to MALLOC > shell command. > > close #1367 > --- > cpukit/libcsupport/Makefile.am

Re: [PATCH 2/2] score: Return heap stats via _Heap_Get_information

2014-11-27 Thread Gedare Bloom
OK. A general comment it it would be nice to unify RTEMS various "statistics" into a common format for easier printing and transferring off-board. -gedare On Thu, Nov 27, 2014 at 7:26 AM, Sebastian Huber wrote: > Print out heap statistics via the MALLOC and WKSPACE shell commands. > --- > cpuki

Re: [rtems commit] tools/build/*.c: Clean up issues reported by CodeSonar

2014-11-27 Thread Joel Sherrill
On 11/27/2014 02:26 AM, Sebastian Huber wrote: On 26/11/14 14:55, Joel Sherril wrote: Module:rtems Branch:master Commit:b1643030121f5ea336670cc9474f4a4a0b2a21eb Changeset:http://git.rtems.org/rtems/commit/?id=b1643030121f5ea336670cc9474f4a4a0b2a21eb Author:Josh Oguin Date:

[PATCH] sync.c: Add asserts to document and check assumptions

2014-11-27 Thread Joel Sherrill
--- cpukit/libcsupport/src/sync.c | 11 +-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/cpukit/libcsupport/src/sync.c b/cpukit/libcsupport/src/sync.c index 653a177..e449cab 100644 --- a/cpukit/libcsupport/src/sync.c +++ b/cpukit/libcsupport/src/sync.c @@ -29,6 +29,7 @@ in

[PATCH] bdbuf: Fix race condition with sync active flag

2014-11-27 Thread Sebastian Huber
Bug report by Oleg Kravtsov: In rtems_bdbuf_swapout_processing() function there is the following lines: if (bdbuf_cache.sync_active && !transfered_buffers) { rtems_id sync_requester; rtems_bdbuf_lock_cache (); ... } Here access to bdbuf_cache.sync_active is not protected with anyth

[PATCH 1/2] libcsupport: Delete malloc statistics

2014-11-27 Thread Sebastian Huber
Use the heap handler statistics instead. Add heap walk option to MALLOC shell command. close #1367 --- cpukit/libcsupport/Makefile.am | 7 +- cpukit/libcsupport/include/rtems/malloc.h | 63 - cpukit/libcsupport/src/calloc.c| 5 --

[PATCH 2/2] score: Return heap stats via _Heap_Get_information

2014-11-27 Thread Sebastian Huber
Print out heap statistics via the MALLOC and WKSPACE shell commands. --- cpukit/libmisc/shell/internal.h | 7 -- cpukit/libmisc/shell/main_mallocinfo.c | 1 + cpukit/libmisc/shell/main_wkspaceinfo.c | 3 ++- cpukit/libmisc/shell/print_heapinfo.c | 39 +++--

Re: [rtems commit] tools/build/*.c: Clean up issues reported by CodeSonar

2014-11-27 Thread Sebastian Huber
On 26/11/14 14:55, Joel Sherril wrote: Module:rtems Branch:master Commit:b1643030121f5ea336670cc9474f4a4a0b2a21eb Changeset:http://git.rtems.org/rtems/commit/?id=b1643030121f5ea336670cc9474f4a4a0b2a21eb Author:Josh Oguin Date: Tue Nov 25 15:55:49 2014 -0600 tools/build/*.c: