#2299 assigned defect

Filesystem test failures on v850sim

Reported by: Joel Sherrill Owned by: Needs Funding
Priority: low Milestone: Indefinite
Component: unspecified Version: 4.11
Severity: normal Keywords:
Cc: Chris Johns Blocked By:
Blocking:

Description (last modified by Joel Sherrill)

Checking v850sim BSP. Not sure about other variants. Looks like all of the DOSFS and RFS tests fail:

Could be a generic alignment issue or simply the test not really fitting in memory.

Starting as 4.11 milestone until Sebastian says what to do.

(gdb) r
Starting program: /users/joel/rtems-4.11-work/b-v850sim/v850-rtems4.11/c/v850sim/testsuites/fstests/fsdosfsname01/fsdosfsname01.exe
core: 1 byte write to unmapped address 0x22c9c8 at 0x100032

Program received signal SIGSEGV, Segmentation fault.
0x00100032 in start ()
(gdb) bt
#0 0x00100032 in start ()

Change History (13)

comment:1 Changed on 03/11/15 at 21:54:04 by Joel Sherrill

Cc: Chris Johns added
Description: modified (diff)
Summary: DOSFS test failures on v850simFilesystem test failures on v850sim

comment:2 Changed on 03/12/15 at 18:37:48 by Joel Sherrill

I am wondering if this is a broader issue of more tests not working on simulators without a clock tick. After running fsdosfssync01.exe for a LONG time, _Watchdog_Ticks_since_boot was still zero. Does the bdbuf code require a clock tick? Does it poll waiting for something to happen in the background?

In gdb, I see it stuck in the bdbuf tasks.

comment:3 Changed on 03/12/15 at 21:26:38 by Chris Johns

The output path waits a period of time before flushing the data. It stops the output path being hit with a group of small writes generating mostly overhead. What happens if the timer value is set to 0 ?

Is it adding an un-natual constrain on RTEMS, its sub-systems and tests by requiring the code to work without a clock tick ? I think it it is. In this case is the simulator at fault because it or we do not support a tick ?

comment:4 Changed on 03/12/15 at 21:40:48 by Joel Sherrill

This is a case where the simulator does not have a clock tick source. So we just may have more tests which can't be run successfully.

I am not asking for code changes if this is the case, just more tests to be added to the simulator clock idle task ignore test list.

comment:5 Changed on 03/12/15 at 22:17:09 by Joel Sherrill

OK. This is not related to lack of a clock tick ISR source. The gdb arm simulator and arm920 BSP passed all but 3 tests (spfatal26, heapwalk, and block01). This is something else and likely specific to the v850.

comment:6 Changed on 03/12/15 at 23:05:52 by Chris Johns

Are those BSP's which are passing simulating a clock tick using a BSP option and so working ?

comment:7 Changed on 03/12/15 at 23:08:56 by Joel Sherrill

arm920-testsuite.tcfg only has this:

include: testdata/require-tick-isr.tcfg

All the v850 testsuite.tcfg files also have that. So that list doesn't kick the dosfs tests out.

Seems to be something more v850 specific.

comment:8 Changed on 03/12/15 at 23:19:03 by Chris Johns

I would check the BSPOPTS for the passing vs the failing. Some BSPOPTS default to that fast tick hack (which I cannot remember off the top of my head) which is in the generic tick ISR handler code.

comment:9 Changed on 12/23/16 at 14:10:53 by Sebastian Huber

Priority: normallow

comment:10 Changed on 01/23/17 at 07:21:19 by Sebastian Huber

Milestone: 4.11.1Indefinite
Owner: changed from Sebastian Huber to Needs Funding
Status: newassigned

comment:11 Changed on 01/23/17 at 15:28:52 by Joel Sherrill

Component: Generaltesting
Owner: changed from Needs Funding to Sebastian Huber

Hmm... Thanks for bumping these tickets Sebastian.

We may have a solution (or work around) we didn't have when this ticket was filed. Do you know the set of file system tests which require a clock tick?

They should be added to testsuites/testdata/require-tick-isr.tcfg.

I agree that production HW should have a clock tick source but many gdb simulators do not have interrupts and we already account for that in a broad way.

This should not be "FIXED". We should just add the tests to the list.

comment:12 Changed on 01/24/17 at 06:18:17 by Sebastian Huber

Owner: changed from Sebastian Huber to Needs Funding

I don't have time for the v850 at the moment.

comment:13 Changed on 10/10/17 at 06:46:55 by Sebastian Huber

Component: testingunspecified
Note: See TracTickets for help on using tickets.