#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:

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 Mar 11, 2015 at 9:54:04 PM by Joel Sherrill

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

comment:2 Changed on Mar 12, 2015 at 6:37:48 PM 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 Mar 12, 2015 at 9:26:38 PM 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 Mar 12, 2015 at 9:40:48 PM 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 Mar 12, 2015 at 10:17:09 PM 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 Mar 12, 2015 at 11:05:52 PM 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 Mar 12, 2015 at 11:08:56 PM 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 Mar 12, 2015 at 11:19:03 PM 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 Dec 23, 2016 at 2:10:53 PM by Sebastian Huber

Priority: normallow

comment:10 Changed on Jan 23, 2017 at 7:21:19 AM by Sebastian Huber

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

comment:11 Changed on Jan 23, 2017 at 3:28:52 PM 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 Jan 24, 2017 at 6:18:17 AM 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 Oct 10, 2017 at 6:46:55 AM by Sebastian Huber

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