[PATCH] libdl/rtl-obj-cache.c: Change format specifiers to fix warnings

This get rid of 4 warnings. Tested on m68k-gen68340(32 bit) and sparc64-niagara(64 bit). Let me know if this is ok. diff --git a/cpukit/libdl/rtl-obj-cache.c b/cpukit/libdl/rtl-obj-cache.c index a4676cf..912689d 100644 --- a/cpukit/libdl/rtl-obj-cache.c +++ b/cpukit/libdl/rtl-obj-cache.c @@

[PATCH] Update:Change format specifiers to fix 4 warnings.

2 of the warnings cropped up on further testing of other BSPs. Sorry, should of been more careful. Also git send-email format. --- cpukit/libdl/rtl-obj-cache.c | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/cpukit/libdl/rtl-obj-cache.c

RE:RE:[PATCH] Update:Change format specifiers to fix 4 warnings.

Great! That looks like a cleaner, more standardized solution. I was just hacking away until the warnings stopped. So can I go ahead and #include or should I wait until is defined before doing any more? Does this work for printk() too? Thanks Joel. ___

RE:RE:RE:Using inttypes macros fixes 4 format warnings

That's great! Thanks for the help Gedare. ___ devel mailing list devel@rtems.org http://lists.rtems.org/mailman/listinfo/devel

RE:[PATCH 04/11] Changes inttype used and adds casts, fixes 6 format warnings

IGNORE THIS ONE. It's a duplicate, that still has warnings. I couldn't figure out how to get rid of it. I'm new to git. ___ devel mailing list devel@rtems.org http://lists.rtems.org/mailman/listinfo/devel

[PATCH 03/11] Using inttype macro, fix 1 format warning

--- cpukit/score/cpu/i386/cpu.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/cpukit/score/cpu/i386/cpu.c b/cpukit/score/cpu/i386/cpu.c index d839f1e..447d8ae 100644 --- a/cpukit/score/cpu/i386/cpu.c +++ b/cpukit/score/cpu/i386/cpu.c @@ -104,7 +104,8 @@ void

[PATCH 02/11] Using inttype macro, fix 1 format warning

--- c/src/libchip/network/dec21140.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/c/src/libchip/network/dec21140.c b/c/src/libchip/network/dec21140.c index 3bfc91d..b76c02a 100644 --- a/c/src/libchip/network/dec21140.c +++ b/c/src/libchip/network/dec21140.c @@ -36,6

[PATCH 04/11] Changes inttype used and adds casts, fixes 6 format warnings

Unfortunately I couldn't figure how to do this without casts. Either 32 or 64 bit was unhappy one way or the other. Type time_t seems to be particularly tricky. --- cpukit/rtems/src/ratemonreportstatistics.c | 36 +++--- 1 file changed, 18 insertions(+), 18 deletions(-)

[PATCH 09/11] Added inttypes, fixes 11 format warnings.

--- c/src/lib/libbsp/powerpc/haleakala/mmu/mmu_405.c | 20 ++-- 1 file changed, 14 insertions(+), 6 deletions(-) diff --git a/c/src/lib/libbsp/powerpc/haleakala/mmu/mmu_405.c b/c/src/lib/libbsp/powerpc/haleakala/mmu/mmu_405.c index f57defb..38a0949 100644 ---

[PATCH 08/11] Added inttype, fix 1 format warning

--- c/src/lib/libbsp/powerpc/mvme5500/pci/pci.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/c/src/lib/libbsp/powerpc/mvme5500/pci/pci.c b/c/src/lib/libbsp/powerpc/mvme5500/pci/pci.c index 8601721..bef0173 100644 --- a/c/src/lib/libbsp/powerpc/mvme5500/pci/pci.c +++

[PATCH 05/11] Using inttype macro, fixes 1 format warning.

--- c/src/lib/libbsp/sparc64/shared/helenos/boot/sparc64/loader/main.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/c/src/lib/libbsp/sparc64/shared/helenos/boot/sparc64/loader/main.c b/c/src/lib/libbsp/sparc64/shared/helenos/boot/sparc64/loader/main.c index

[PATCH 01/11] Using inttype macro, fix 1 format warning

--- c/src/lib/libbsp/powerpc/shared/pci/detect_raven_bridge.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/c/src/lib/libbsp/powerpc/shared/pci/detect_raven_bridge.c b/c/src/lib/libbsp/powerpc/shared/pci/detect_raven_bridge.c index fc2127e..423dc6f 100644 ---

[PATCH 06/11] Inserted inttype macros, fix 11 format warnings.

When I got rid of the casts I was left with char * and couldn't find an inttype to fit it so I left the casts in. --- .../lib/libbsp/powerpc/virtex5/startup/bspstart.c | 25 +++--- 1 file changed, 13 insertions(+), 12 deletions(-) diff --git

[PATCH 07/11] Inserted inttypes, fixes 11 format warnings.

Like virtex5 I couldn't get rid of the casts. --- .../lib/libbsp/powerpc/virtex4/startup/bspstart.c | 25 +++--- 1 file changed, 13 insertions(+), 12 deletions(-) diff --git a/c/src/lib/libbsp/powerpc/virtex4/startup/bspstart.c

[PATCH 10/11] Added inttypes, fixes 6 format warnings

--- c/src/lib/libbsp/m68k/mrm332/console/sci.c | 13 +++-- 1 file changed, 7 insertions(+), 6 deletions(-) diff --git a/c/src/lib/libbsp/m68k/mrm332/console/sci.c b/c/src/lib/libbsp/m68k/mrm332/console/sci.c index 1e92d6d..8e44f22 100644 --- a/c/src/lib/libbsp/m68k/mrm332/console/sci.c

[PATCH 11/11] Change inttype used and add casts, fixes 6 format warnings.

Unfortunately I couldn't figure out how to do this without casts. Either 32 or 64 bit was unhappy one way or the other. Type time_t seems to be particularly tricky. --- cpukit/rtems/src/ratemonreportstatistics.c | 36 +++--- 1 file changed, 18 insertions(+), 18

RE:RE:Using inttypes macros fixes 4 format warnings

Thanks Joel! Plenty more were that came from below. ___ devel mailing list devel@rtems.org http://lists.rtems.org/mailman/listinfo/devel

[PATCH 8/9] powerpc/haleakala/./mmu_405.c: Use inttypes.h constants removes 11 warnings.

--- c/src/lib/libbsp/powerpc/haleakala/mmu/mmu_405.c | 20 ++-- 1 file changed, 14 insertions(+), 6 deletions(-) diff --git a/c/src/lib/libbsp/powerpc/haleakala/mmu/mmu_405.c b/c/src/lib/libbsp/powerpc/haleakala/mmu/mmu_405.c index f57defb..38a0949 100644 ---

[PATCH 6/9] virtex4/startup/bspstart.c: Use %p to fix 11 warnings.

--- .../lib/libbsp/powerpc/virtex4/startup/bspstart.c | 27 +++--- 1 file changed, 14 insertions(+), 13 deletions(-) diff --git a/c/src/lib/libbsp/powerpc/virtex4/startup/bspstart.c b/c/src/lib/libbsp/powerpc/virtex4/startup/bspstart.c index f4a5ced..1c4c531 100644 ---

[PATCH 2/9] libchip/network/dec21140.c: Use inttypes.h to fix 1 warning.

--- c/src/libchip/network/dec21140.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/c/src/libchip/network/dec21140.c b/c/src/libchip/network/dec21140.c index 3bfc91d..9cbf74f 100644 --- a/c/src/libchip/network/dec21140.c +++ b/c/src/libchip/network/dec21140.c @@ -36,6

[PATCH 7/9] powerpc/mvme550/./pci.c: Use inttypes.h constant to fix 1 warning.

--- c/src/lib/libbsp/powerpc/mvme5500/pci/pci.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/c/src/lib/libbsp/powerpc/mvme5500/pci/pci.c b/c/src/lib/libbsp/powerpc/mvme5500/pci/pci.c index 8601721..bef0173 100644 --- a/c/src/lib/libbsp/powerpc/mvme5500/pci/pci.c +++

[PATCH 9/9] m68k/mrm332/./sci.c: Use inttypes.h constants removes 6 warnings.

--- c/src/lib/libbsp/m68k/mrm332/console/sci.c | 13 +++-- 1 file changed, 7 insertions(+), 6 deletions(-) diff --git a/c/src/lib/libbsp/m68k/mrm332/console/sci.c b/c/src/lib/libbsp/m68k/mrm332/console/sci.c index 1e92d6d..8e44f22 100644 --- a/c/src/lib/libbsp/m68k/mrm332/console/sci.c

[PATCH 3/9] cpukit/../i386/cpu.c: Use inttypes.h to fix 1 warning.

--- cpukit/score/cpu/i386/cpu.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/cpukit/score/cpu/i386/cpu.c b/cpukit/score/cpu/i386/cpu.c index d839f1e..447d8ae 100644 --- a/cpukit/score/cpu/i386/cpu.c +++ b/cpukit/score/cpu/i386/cpu.c @@ -104,7 +104,8 @@ void

[PATCH 5/9] virtex5/startup/bspstart.c: Use %p to fix 11 warnings.

--- .../lib/libbsp/powerpc/virtex5/startup/bspstart.c | 27 +++--- 1 file changed, 14 insertions(+), 13 deletions(-) diff --git a/c/src/lib/libbsp/powerpc/virtex5/startup/bspstart.c b/c/src/lib/libbsp/powerpc/virtex5/startup/bspstart.c index 9d610e4..93f99ad 100644 ---

[PATCH 1/9] libbsp/powerpc/./pci/detect_raven_bridge.c: Use inttypes.h to fix 1 warning.

--- c/src/lib/libbsp/powerpc/shared/pci/detect_raven_bridge.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/c/src/lib/libbsp/powerpc/shared/pci/detect_raven_bridge.c b/c/src/lib/libbsp/powerpc/shared/pci/detect_raven_bridge.c index fc2127e..423dc6f 100644 ---

RE:RE:Using inttypes macros fixes 4 format warnings

I used the wrong subject heading, that was meant to end up above the last ones. That's all I got. I'll have to get some college assignments out of the way now before I can do any more. I resubmitted them below, the ratemonreportstatistics.c is missing as I'm not sure how to define

[PATCH 1/2] Using inttypes macros fixes 4 format warnings

--- cpukit/libdl/rtl-obj-cache.c | 21 - 1 file changed, 12 insertions(+), 9 deletions(-) diff --git a/cpukit/libdl/rtl-obj-cache.c b/cpukit/libdl/rtl-obj-cache.c index a4676cf..c859f60 100644 --- a/cpukit/libdl/rtl-obj-cache.c +++ b/cpukit/libdl/rtl-obj-cache.c @@ -22,6

[PATCH 2/2] Using inttype macros fixes 39 format warnings

--- .../libcpu/powerpc/mpc5xx/vectors/vectors_init.c | 80 +++--- 1 file changed, 41 insertions(+), 39 deletions(-) diff --git a/c/src/lib/libcpu/powerpc/mpc5xx/vectors/vectors_init.c b/c/src/lib/libcpu/powerpc/mpc5xx/vectors/vectors_init.c index 4a6c5be..ff34a21 100644 ---

Re: Coverage analysis update

On 6 July 2017 at 19:14, Joel Sherrill <j...@rtems.org> wrote: > > > On Thu, Jul 6, 2017 at 11:11 AM, Cillian O'Donnell <cpodonne...@gmail.com> > wrote: >> >> On 6 July 2017 at 15:34, Joel Sherrill <j...@rtems.org> wrote: >> > >>

Trace Block Inconsistent with Coverage Map

I just want to give an update to my understanding of this problem as I'm not sure I was clear enough in the IRC meeting. We ended up talking about nops but I think the problem is something else. So when there is a successful covoar run, it will generate a report but also finish with this message.

Re: Coverage analysis update

hr...@rtems.org >> <mailto:chr...@rtems.org>> wrote: >> >> On 07/07/2017 00:34, Joel Sherrill wrote: >> > On Thu, Jul 6, 2017 at 5:53 AM, Cillian O'Donnell >> <cpodonne...@gmail.com >> <mailto:cpodonne...@gmail.com> >>

Re: Coverage analysis update

On 7 July 2017 at 02:39, Chris Johns <chr...@rtems.org> wrote: > On 07/07/2017 00:34, Joel Sherrill wrote: >> On Thu, Jul 6, 2017 at 5:53 AM, Cillian O'Donnell <cpodonne...@gmail.com >> <mailto:cpodonne...@gmail.com>> wrote: >> >> When a test run

Re: Release candidate tool chain for 4.12

On 27 July 2017 at 23:28, Chris Johns wrote: > On 28/07/2017 00:40, Joel Sherrill wrote: >> On Thu, Jul 27, 2017 at 7:50 AM, Sebastian Huber >> > > >> wrote: >> >> Hallo, >> >> the GCC 7.2

Re: Coverage analysis update

On 6 July 2017 at 15:34, Joel Sherrill <j...@rtems.org> wrote: > > > On Thu, Jul 6, 2017 at 5:53 AM, Cillian O'Donnell <cpodonne...@gmail.com> > wrote: >> >> On 6 July 2017 at 08:05, Chris Johns <chr...@rtems.org> wrote: >> > On 05/07/2017 22:46,

Coverage analysis update

Hi, I now have coverage running successfully with RTEMS Tester and have a coverage report for Leon 3 on the libscore.a set Summary Bytes Analyzed : 43268 Bytes Not Executed : 5756 Percentage Executed : 86.7 Percentage Not Executed : 13.3 Uncovered ranges found : 316

[PATCH] cpukit/../ratemonreportstatistics.c:Use rtems/inttypes removes 12 warnings.

--- cpukit/rtems/src/ratemonreportstatistics.c | 15 --- 1 file changed, 8 insertions(+), 7 deletions(-) diff --git a/cpukit/rtems/src/ratemonreportstatistics.c b/cpukit/rtems/src/ratemonreportstatistics.c index a1bab4a..78477b0 100644 ---

Re: RSB Couverture-Qemu build queries

> Please try and let me know if it is ok so I can push to master: It worked! Everything's perfect now. Thanks Chris. On 19 June 2017 at 23:19, Jiri Gaisler <j...@gaisler.se> wrote: > > > On 06/17/2017 10:28 AM, Cillian O'Donnell wrote: >> On 16 June 2017 at 22:00, Jiri

RE:rtems-tools build errors after coverage tools patches

Ok sorry I've just realised the const strings split(...) function is obviously something else with the split bit. For the problem line void write_lines (const rld::strings& ss); If I remove 'const' I get ../rtl-host/rld-process.h:175:30: error: ‘rld::strings’ has not been declared void

rtems-tools build errors after coverage tools patches

Hi, I am working on the GSOC coverage analysis tools project. There was some previous work done SOCIS 2014 and SOCIS 2015 . I have just finished going through and reapplying their

Re: rtems-tools build errors after coverage tools patches

as progressed since the patches you > are starting with so expect some breakage, build system changes, etc.. > > That should be enough to find the issue but Chris may have more > insight when he is awake. > > --joel > > On Sat, May 27, 2017 at 5:28 PM, Cillian O'Donnell <cpo

Re: rtems-test runtime errors qemu.cfg

Ah nevermind.. just realised its already printing them out. On 30 May 2017 9:50 pm, "Cillian O'Donnell" <cpodonne...@gmail.com> wrote: > Hi Tanu, > > Thanks for the help, I had been grepping through the files and > backtracking through the definitions. I was just w

Re: rtems-test runtime errors qemu.cfg

/home/cpod/coverage_test On 31 May 2017 at 00:31, Gedare Bloom <ged...@rtems.org> wrote: > Does it work without --coverage flag? > > On Tue, May 30, 2017 at 5:23 PM, Cillian O'Donnell > <cpodonne...@gmail.com> wrote: >> Ah nevermind.. just realised its already printin

[PATCH] rtemstoolkit/rld-process.h: Include rld.h to find 'strings' definition

--- rtemstoolkit/rld-process.h | 1 + 1 file changed, 1 insertion(+) diff --git a/rtemstoolkit/rld-process.h b/rtemstoolkit/rld-process.h index ae89b15..fc9b7bc 100644 --- a/rtemstoolkit/rld-process.h +++ b/rtemstoolkit/rld-process.h @@ -28,6 +28,7 @@ #include #include #include +#include

Coverage tools project direction [GSOC]

Hi, I've just realised that I've been fixing problems with the RTEMS Tester work that has already been done to convert bash scripts to python, but I probably should of first been trying to get the old sim-scripts method working from rtems-testing repo because this will give me more bsps to test

[PATCH 0/2] *** Docs Update ***

*** Just to note the rsb quick-start is all updated except for the output of --list-bsets because my setup only ever finds output for ../bare/config and not ../source-builder/config so I left that bit out just in case. Maybe someone else can update that section to 4.12. *** Cillian O'Donnell

[PATCH 1/2] rtems-docs/user/../quick.rst: Fix typo, hard coded path, code layout.

--- user/start/quick.rst | 15 --- 1 file changed, 8 insertions(+), 7 deletions(-) diff --git a/user/start/quick.rst b/user/start/quick.rst index 47597af..8fc24f1 100644 --- a/user/start/quick.rst +++ b/user/start/quick.rst @@ -15,9 +15,9 @@ You need to have your development host set

[PATCH 2/2] docs/rsb/quick-start.rst: Fix typos, hard coded paths, version

--- rsb/quick-start.rst | 242 ++-- 1 file changed, 121 insertions(+), 121 deletions(-) diff --git a/rsb/quick-start.rst b/rsb/quick-start.rst index f39c0a6..387ab61 100644 --- a/rsb/quick-start.rst +++ b/rsb/quick-start.rst @@ -42,13 +42,13 @@

Re: Coverage tools project direction [GSOC]

me output. > > So sorry about one step at a time. The old scripts were intended to be a > reference for you. That was a working procedure. > > --joel > > On Jun 4, 2017 5:37 AM, "Cillian O'Donnell" <cpodonne...@gmail.com> wrote: >> >> Hi Chris, >>

Re: Coverage tools project direction [GSOC]

qemu.cfg, probably the bsp options in the coverage.mc files, not sure what to use yet. All tests invalid, default to dry-run Thanks, Cillian. On 4 June 2017 at 03:56, Chris Johns <chr...@rtems.org> wrote: > On 3/6/17 10:14 pm, Cillian O'Donnell wrote: >> >> I've just realised

RSB Couverture-Qemu build queries

Hi, I am getting the RSB to build Couverture-Qemu and I just want to check a few things I have done so far. 1. There are about 5 patches applied to the current qemu build. Only one of which applies cleanly to the Couverture build. Do you want me to try and fix these up, comment them out and

Re: RSB Couverture-Qemu build queries

Joel, I've been talking with Fabien Chouteau from the Couverture project, I'll reach out to him with the patches and cc you and Chris and thanks Gedare for the info. > > On 06/16/2017 03:13 PM, Cillian O'Donnell wrote: >> Hi, >> >> I am getting the RSB to build Couverture-Qemu

Couverture-Qemu Testing Results

Hi, I'm testing Couverture-Qemu against the RSB-Qemu to check if it is working correctly. So far I have good results for ARM bsps xilinx_zynq_a9_qemu and realview_pbx_a9_qemu and Sparc Leon3. They show identical results for the samples and almost identical results for the full testsuite. They

Re: RSB Couverture-Qemu build queries

On 18 June 2017 at 03:23, Chris Johns <chr...@rtems.org> wrote: > On 17/6/17 6:38 am, Joel Sherrill wrote: >> >> >> On Fri, Jun 16, 2017 at 2:13 PM, Cillian O'Donnell <cpodonne...@gmail.com >> <mailto:cpodonne...@gmail.com>> wrote: >> >>

[PATCH] docs/rsb/configuration.rst: Fix typos, grammar.

--- rsb/configuration.rst | 56 +-- 1 file changed, 28 insertions(+), 28 deletions(-) diff --git a/rsb/configuration.rst b/rsb/configuration.rst index 62fc073..b2745a5 100644 --- a/rsb/configuration.rst +++ b/rsb/configuration.rst @@ -115,18

Re: Couverture-Qemu Testing Results

ok similar? Are there other qemu bsps? > > I guess this means it is almost time to look at switching the rsb > > On Jun 10, 2017 1:25 PM, "Cillian O'Donnell" <cpodonne...@gmail.com> wrote: > > Hi, > > I'm testing Couverture-Qemu against the RSB-Qemu t

Re: Couverture-Qemu Testing Results

ng. Ok, so I should try and fix the above 2, file a ticket for the rest and then move on to the RSB work? On 11 June 2017 at 18:12, Joel Sherrill <j...@rtems.org> wrote: > > > On Jun 11, 2017 4:13 AM, "Cillian O'Donnell" <cpodonne...@gmail.com> wrote: > >> Does

rtems-test runtime errors qemu.cfg

Hi, I'm trying to run rtems-test for pc386 with the added --coverage tag for couverture-qemu. All tests default to dry-run because of errors in qemu.cfg error: qemu.cfg:81: execute failed: qemu-system-i386 -m 128 -boot b -hda

Re: rtems-test runtime errors qemu.cfg

ong process to step into each line. > I wanted to ask what command did you use exactly with rtems-test? I > couldn't find a recipe for qemu simulation for pc386 in > rtems-tools.git/tester/rtems/testing/bsps. > > Regards, > Tanu Hari Dixit. > > On Wed, May 31, 2017 at 12:42 AM,

[PATCH] covoar: Add function defs to fix Windows build errors.

Closes #3129 --- tester/covoar/DesiredSymbols.cc | 2 +- tester/covoar/ReportsBase.cc| 4 ++-- tester/covoar/TraceConverter.cc | 4 tester/covoar/covoar.cc | 4 4 files changed, 11 insertions(+), 3 deletions(-) diff --git a/tester/covoar/DesiredSymbols.cc

[PATCH] *** Covoar Windows Build Failure. ***

*** The build is successfull if I run ./waf -v however the build fails on a normal ./waf command as it can't link to -lelf 4-mingw32/bin/ld.exe: cannot find -lelf Has anyone seen this behaviour before were it only builds in verbose mode?*** Cillian O'Donnell (1): covoar: Add function defs

Re: source-builder-documentation

Here's the one your looking for https://docs.rtems.org/branches/master/rsb/index.html All docs are at: https://docs.rtems.org/ On 11 October 2017 at 09:09, emanuel stiebler wrote: > Did it move? > > both links on the wiki > > https://devel.rtems.org/wiki/Developer/Tools/RSB > >

Re: source-builder-documentation

> --joel > > > On Oct 11, 2017 3:16 AM, "Cillian O'Donnell" <cpodonne...@gmail.com> wrote: > > Here's the one your looking for > https://docs.rtems.org/branches/master/rsb/index.html > > All docs are at: > https://docs.rtems.org/ > > On 11 Oct

[PATCH 7/9] covoar: Use rld tempfile and add signals to clean up in event of crash.

Use rld tempfile for temporary files and add fatal signal handling to clean them up in the event of a crash. --- tester/covoar/ObjdumpProcessor.cc | 168 +--- tester/covoar/ObjdumpProcessor.h| 30 --- tester/covoar/TraceConverter.cc | 60

[PATCH 0/9] *** Covoar Improvements ***

*** As the full coverage analysis tools integrated with RTEMS Tester is not ready yet, I've cherry picked these standalone fixes to covoar. This includes work from myself and the previous 2 students that worked on the project in 2014 and 2015. *** Cillian O'Donnell (5): rtemstoolkit/host.py

[PATCH 9/9] covoar: Remove config file and test critical options are valid.

--- tester/covoar/covoar.cc | 164 +++- tester/covoar/wscript | 1 - 2 files changed, 63 insertions(+), 102 deletions(-) diff --git a/tester/covoar/covoar.cc b/tester/covoar/covoar.cc index 005cb8e..d362255 100644 --- a/tester/covoar/covoar.cc +++

[PATCH 5/9] covoar/ObjdumpProcessor.cc: Remove jump tables from the end of symbols.

Some tests objdumps have jump tables added to the end of them causing the same symbols to have different sizes when processed. This removes them and their associated errors. --- tester/covoar/ObjdumpProcessor.cc | 37 - 1 file changed, 36 insertions(+), 1

[PATCH 1/9] rtemstoolkit/host.py: Change option to search parent directory in Python3.

This works in Python2.7 and Python3.5. Python2 __import__ searches relative and absolute paths by default. Python3 searches only absolute paths as default. The option change searches 1 parent directory relative to the calling function and has the same meaning in Python2 and Python3. ---

[PATCH 4/9] covoar: Add information to improve diagnostics.

From: Hermann Felbinger --- tester/covoar/CoverageMap.cc | 7 --- tester/covoar/CoverageMap.h | 5 +++-- tester/covoar/CoverageMapBase.cc | 6 -- tester/covoar/CoverageMapBase.h | 11 +-- tester/covoar/DesiredSymbols.cc | 25

[PATCH 2/9] covoar/qemu-traces.h: Change to match current couverture-qemu file.

From: Fabien Chouteau --- tester/covoar/qemu-traces.h | 76 +++-- 1 file changed, 39 insertions(+), 37 deletions(-) diff --git a/tester/covoar/qemu-traces.h b/tester/covoar/qemu-traces.h index 7e37840..1b3d118 100644 ---

[PATCH 6/9] covoar/wscript: Add paths to rtemstoolkit to build.

From: Krzysztof Miesowicz <krzysztof.miesow...@gmail.com> Co-Author: Cillian O'Donnell <cpodonne...@gmail.com> --- tester/covoar/wscript | 18 +- 1 file changed, 13 insertions(+), 5 deletions(-) diff --git a/tester/covoar/wscript b/tester/covoar/wscript index 6818

[PATCH 8/9] covoar/CoverageReaderQEMU.cc: Remove trace block matching check.

This removes the 'Trace block inconsistent with coverage map' check as it was deemed to be too restrictive and not neccessary. --- tester/covoar/CoverageReaderQEMU.cc | 15 --- 1 file changed, 15 deletions(-) diff --git a/tester/covoar/CoverageReaderQEMU.cc

[PATCH 4/9] covoar: Add information to improve diagnostics.

From: Hermann Felbinger --- tester/covoar/CoverageMap.cc | 7 --- tester/covoar/CoverageMap.h | 5 +++-- tester/covoar/CoverageMapBase.cc | 6 -- tester/covoar/CoverageMapBase.h | 11 +-- tester/covoar/DesiredSymbols.cc | 25

[PATCH 5/9] covoar/ObjdumpProcessor.cc: Remove jump tables from the end of symbols.

Some tests objdumps have jump tables added to the end of them causing the same symbols to have different sizes when processed. This removes them and their associated errors. --- tester/covoar/ObjdumpProcessor.cc | 37 - 1 file changed, 36 insertions(+), 1

[PATCH 9/9] covoar: Remove config file and test critical options are valid.

--- tester/covoar/covoar.cc | 164 +++- tester/covoar/wscript | 1 - 2 files changed, 63 insertions(+), 102 deletions(-) diff --git a/tester/covoar/covoar.cc b/tester/covoar/covoar.cc index 005cb8e..5b55720 100644 --- a/tester/covoar/covoar.cc +++

[PATCH 0/9] *** Covoar Improvements v2 ***

implemented.*** Cillian O'Donnell (5): rtemstoolkit/host.py: Change option to search parent directory in Python3. covoar/ObjdumpProcessor.cc: Remove jump tables from the end of symbols. covoar: Use rld tempfile and add signals to clean up in event of crash. covoar

[PATCH 8/9] covoar/CoverageReaderQEMU.cc: Remove trace block matching check.

This removes the 'Trace block inconsistent with coverage map' check as it was deemed to be too restrictive and not neccessary. --- tester/covoar/CoverageReaderQEMU.cc | 15 --- 1 file changed, 15 deletions(-) diff --git a/tester/covoar/CoverageReaderQEMU.cc

[PATCH 6/9] covoar/wscript: Add paths to rtemstoolkit to build.

From: Krzysztof Miesowicz <krzysztof.miesow...@gmail.com> Co-Author: Cillian O'Donnell <cpodonne...@gmail.com> --- tester/covoar/wscript | 18 +- 1 file changed, 13 insertions(+), 5 deletions(-) diff --git a/tester/covoar/wscript b/tester/covoar/wscript index 6818

[PATCH 7/9] covoar: Use rld tempfile and add signals to clean up in event of crash.

Use rld tempfile for temporary files and add fatal signal handling to clean them up in the event of a crash. --- tester/covoar/ObjdumpProcessor.cc | 168 +--- tester/covoar/ObjdumpProcessor.h| 30 --- tester/covoar/TraceConverter.cc | 60

[PATCH 1/9] rtemstoolkit/host.py: Change option to search parent directory in Python3.

This works in Python2.7 and Python3.5. Python2 __import__ searches relative and absolute paths by default. Python3 searches only absolute paths as default. The option change searches 1 parent directory relative to the calling function and has the same meaning in Python2 and Python3. ---

Re: [PATCH] covoar.cc: Correct build path checks for multiple executables.

On Sun, 13 May 2018, 22:15 Vijay Kumar Banerjee, <vijaykumar9...@gmail.com> wrote: > On 14 May 2018 at 02:15, Cillian O'Donnell <cpodonne...@gmail.com> wrote: > >> --- >> tester/covoar/covoar.cc | 10 +++--- >> 1 file changed, 3 insertions(+), 7 deletions

[PATCH] covoar: Fix build path checks for multiple executables.

--- tester/covoar/covoar.cc | 10 +++--- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/tester/covoar/covoar.cc b/tester/covoar/covoar.cc index 5c87402..c6b0589 100644 --- a/tester/covoar/covoar.cc +++ b/tester/covoar/covoar.cc @@ -75,7 +75,7 @@ static void

Re: [PATCH] covoar.cc: Correct build path checks for multiple executables.

On Mon, 14 May 2018, 16:46 Joel Sherrill, <j...@rtems.org> wrote: > I'll commit this once there is a log message. :) > You want the word 'Fix' is it?... :) > > On Sun, May 13, 2018 at 3:45 PM, Cillian O'Donnell <cpodonne...@gmail.com> > wrote: > >>

Re: [PATCH] covoar.cc: Correct build path checks for multiple executables.

On Mon, 14 May 2018, 09:50 Vijay Kumar Banerjee, <vijaykumar9...@gmail.com> wrote: > On 14 May 2018 at 12:10, Cillian O'Donnell <cpodonne...@gmail.com> wrote: > >> >> >> On Sun, 13 May 2018, 22:15 Vijay Kumar Banerjee, < >> vijaykumar9...@gmail.com>

Re: Fwd: [PATCH 2/4] covoar: Use DWARF to map addresses to source files and lines.

../tester/covoar/covoar.cc:330 On 7 May 2018 at 20:03, Joel Sherrill <j...@rtems.org> wrote: > > > On Mon, May 7, 2018 at 1:59 PM, Cillian O'Donnell <cpodonne...@gmail.com> > wrote: > >> >> >> On 7 May 2018 at 18:56, Joel Sherrill <j...@rtems.org>

Re: Fwd: [PATCH 2/4] covoar: Use DWARF to map addresses to source files and lines.

ck=full to see details of leaked memory ==20928== ==20928== For counts of detected and suppressed errors, rerun with: -v ==20928== ERROR SUMMARY: 3 errors from 3 contexts (suppressed: 0 from 0) Segmentation fault On 7 May 2018 at 20:16, Cillian O'Donnell <cpodonne...@gmail.com> wrote: >

Re: [PATCH 2/2] rtems-tools/../testing: Add configuration files for coverage analysis.

overage. > > The .css file is a support file for the output. > > Not sure where the else fall but these seems like a handful of small > patches > rather than one larger patch. > > --joel > > On Wed, Apr 25, 2018 at 3:33 PM, Cillian O'Donnell <cpodonne...@g

Re: [PATCH 1/2] covoar: Add symbol set reader and ELF data parser to covoar.

On Fri, 27 Apr 2018, 04:48 Chris Johns, wrote: > On 27/04/2018 08:51, Joel Sherrill wrote: > > > >// Create the set of desired symbols. > >SymbolsToAnalyze = new Coverage::DesiredSymbols(); > > - SymbolsToAnalyze->load( symbolsFile ); > > > > > > I am

Re: [PATCH 0/2] *** Covoar Outstanding Changes ***

On 26 April 2018 at 08:50, Chris Johns <chr...@rtems.org> wrote: > On 26/04/2018 16:11, Cillian O'Donnell wrote: > > On Thu, 26 Apr 2018, 03:27 Chris Johns, <chr...@rtems.org > > <mailto:chr...@rtems.org>> wrote: > > > >> Also, are there any

Re: [PATCH] covoar: Add symbol set reader and ELF data parser to covoar.

On 28 April 2018 at 08:08, Chris Johns <chr...@rtems.org> wrote: > From: Cillian O'Donnell <cpodonne...@gmail.com> > > Add ability to organize symbol sets of libraries in INI file > and then read them with covoar and load the symbols directly from the > libraries. >

[PATCH 1/3] source-builder/config: Add couverture-qemu general config & build instructions.

--- source-builder/config/couverture-qemu-2-1.cfg | 104 ++ 1 file changed, 104 insertions(+) create mode 100644 source-builder/config/couverture-qemu-2-1.cfg diff --git a/source-builder/config/couverture-qemu-2-1.cfg b/source-builder/config/couverture-qemu-2-1.cfg new

[PATCH 3/3] bare/config/devel: Add couverture-qemu build dependencies.

--- bare/config/devel/couverture-qemu.bset | 25 + 1 file changed, 25 insertions(+) create mode 100644 bare/config/devel/couverture-qemu.bset diff --git a/bare/config/devel/couverture-qemu.bset b/bare/config/devel/couverture-qemu.bset new file mode 100644 index

[PATCH 2/3] bare/config/devel: Add couverture-qemu source location and patches needed.

--- bare/config/devel/couverture-qemu-git-1.cfg | 32 + 1 file changed, 32 insertions(+) create mode 100644 bare/config/devel/couverture-qemu-git-1.cfg diff --git a/bare/config/devel/couverture-qemu-git-1.cfg b/bare/config/devel/couverture-qemu-git-1.cfg new file

Re: [PATCH 0/2] *** Covoar Outstanding Changes ***

On 28 April 2018 at 01:25, Chris Johns <chr...@rtems.org> wrote: > > > On 28/4/18 7:41 am, Cillian O'Donnell wrote: > > > > > > On 26 April 2018 at 08:50, Chris Johns <chr...@rtems.org > > <mailto:chr...@rtems.org>> wrote: > > > >

Re: V2 Add DWARF support to the rtemstoolkit and put it to use.

On Thu, 10 May 2018, 03:43 Chris Johns, <chr...@rtems.org> wrote: > On 10/5/18 8:47 am, Cillian O'Donnell wrote: > > Just running covoar there, we seem to be running into some problems at > the > > source line lookup stage > > Thank you for testing this. I assu

Re: V2 Add DWARF support to the rtemstoolkit and put it to use.

On Thu, 10 May 2018, 11:48 Chris Johns, <chr...@rtems.org> wrote: > On 10/5/18 2:43 pm, Chris Johns wrote: > > On 10/5/18 8:47 am, Cillian O'Donnell wrote: > >> > >> > >> GDB > >> > >> > >&

Re: Fwd: [PATCH 2/4] covoar: Use DWARF to map addresses to source files and lines.

On Thu, 10 May 2018, 16:01 Vijay Kumar Banerjee, <vijaykumar9...@gmail.com> wrote: > > > On 8 May 2018 at 00:33, Cillian O'Donnell <cpodonne...@gmail.com> wrote: > >> Here's the branch with work in progress for the tester. Have a look when >> you

Re: V2 Add DWARF support to the rtemstoolkit and put it to use.

Just running covoar there, we seem to be running into some problems at the source line lookup stage cpod@cpod ~/development/rtems/leon3 $ covoar -S /home/cpod/development/rtems/test/rtems-tools/tester/rtems/testing/coverage/score-symbols.ini -O /home/cpod/coverage_test/test/score

Re: [PATCH] covoar: fixing the extension mismatch in trace file

On 13 May 2018 at 13:35, Vijay Kumar Banerjee <vijaykumar9...@gmail.com> wrote: > > On Sun, 13 May 2018, 17:32 Cillian O'Donnell, <cpodonne...@gmail.com> > wrote: > >> I understand but that's a separate issue. I can do a full coverage run >> from t

Re: [PATCH] covoar: fixing the extension mismatch in trace file

collaborative development, you'll have to get used to it :)... On 13 May 2018 at 12:39, Vijay Kumar Banerjee <vijaykumar9...@gmail.com> wrote: > > > On Sun, 13 May 2018, 17:02 Cillian O'Donnell, <cpodonne...@gmail.com> > wrote: > >> But you see the thing is that i

Re: [PATCH] covoar: fixing the extension mismatch in trace file

=leon3-qemu-cov --coverage $HOME/development/rtems/leon3/sparc-rtems5/c/leon3/testsuites/samples On 13 May 2018 at 13:58, Vijay Kumar Banerjee <vijaykumar9...@gmail.com> wrote: > > > On Sun, 13 May 2018, 18:24 Cillian O'Donnell, <cpodonne...@gmail.com> > wrote: > >&

  1   2   >