Project

General

Profile

User story #115

MARTe-EX-F-1.3.4: Framework developers shall be capable of waiting on any type of semaphore with a infinite timeout (i.e. "forever").

Added by André Neto almost 10 years ago. Updated over 9 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Ivan Herrero
Target version:
0.2
Start date:
Due date:
% Done:

0%

Estimated time:
2.00 h
Git branch (link):
Git merge to develop (link):
SVN commit (link/?p=rev):

Description

Implement requirement MARTe-EX-F-1.3.4 based on the original MARTe implementation

Note: Framework developers might be interested to wait for a shared resource until this is available, no matter how long this might take.

Source code files modified

  • Source/Core/L0Portability/MutexSem.cpp
  • Source/Core/L0Portability/MutexSem.h
  • Source/Core/L0Portability/OperatingSystem/Linux/MutexSemOS.h
  • Source/Core/L0Portability/OperatingSystem/Windows/MutexSemOS.h
  • Source/Core/L0Portability/SemCore.h
  • Source/Core/L0Portability/EventSem.cpp
  • Source/Core/L0Portability/EventSem.h
  • Source/Core/L0Portability/OperatingSystem/Linux/EventSemOS.h
  • Source/Core/L0Portability/OperatingSystem/Windows/EventSemOS.h
  • Source/Core/L0Portability/FastPollingMutexSem.h
  • Test/Core/L0Portability/MutexSemTest.cpp
  • Test/Core/L0Portability/MutexSemTest.h
  • Test/Core/L0Portability/SemCoreTest.cpp
  • Test/Core/L0Portability/SemCoreTest.h
  • Test/Core/L0Portability/EventSemTest.cpp
  • Test/Core/L0Portability/EventSemTest.h
  • Test/Core/L0Portability/FastPollingMutexSemTest.cpp
  • Test/Core/L0Portability/FastPollingMutexSemTest.h

Requirements review

N/A

Architecture & design review

N/A

Code and documentation review

Date of the review: 25/06/2015

Person who did the review: Ivan Herrero

Result of review: PASS

List of non-conformities: N/A

Unit test review

Date of the review: 14/07/2015

Person who did the review: Ivan Herrero

Version of unit tests: 0.2

Result of coverage tests review: PASS

Result of functional tests review: PASS

Result of review: PASS

List of non-conformities: N/A

Integration test review

N/A

Acceptance test review

N/A

Note: SemCoreTest class does not have unit test, because it has not implementation and it is not used by any other class of L0.

Associated revisions

Revision d10f7c3c (diff)
Added by Giuseppe Ferrò almost 10 years ago

Added MutexSem implementation.
#115 #117 #114

Revision 40b0fa40 (diff)
Added by Giuseppe Ferrò almost 10 years ago

Added EventSem implementation
#117 #115 #118

Revision 0eb7c1c6 (diff)
Added by Giuseppe Ferrò almost 10 years ago

Added FastPollingMutex implementation
#115 #117 #119

Revision 16238ec2 (diff)
Added by Giuseppe Ferrò almost 10 years ago

Fixed semaphore files accordingly with revision.
#115 #117 #119 #118 #114

Revision cd81fb26 (diff)
Added by Giuseppe Ferrò almost 10 years ago

Revision 4c0c6e8a (diff)
Added by Giuseppe Ferrò almost 10 years ago

Fixed issues accordingly with revision.
#115

Revision 192c81cf (diff)
Added by Giuseppe Ferrò almost 10 years ago

Added MutexSem unit tests.
Renamed previous tests as integration tests.
#116 #115 #117 #114

Revision 367679ea (diff)
Added by Giuseppe Ferrò almost 10 years ago

Added EventSem tests.
Renamed previous tests as integration tests.
#116 #115 #117 #118

Revision 08f24f50 (diff)
Added by Giuseppe Ferrò almost 10 years ago

Added FastPollingMutexSem tests.
Renamed previous tests as integration tests.
#116 #115 #117 #119

Revision c36415cb (diff)
Added by André Neto almost 10 years ago

Updated the FastPollingMutexSem against the agreed unit testing
strategy. #115 and #116

Revision 79091e85 (diff)
Added by Ivan Herrero over 9 years ago

Correct issues detected on unit testing review for #115

  • Add a unit test for EventSem::IsClosed() on EventSemTest test case.
  • Correct date format on file's documentation for EventSemTest.h.
  • Add a unit test for MutexSem::MutexSem() on MutexSemTest test case.
  • Add a unit test for MutexSem::IsClosed() on MutexSemTest test case.
  • Remove unit tests TestFastLock, TestFastUnLock, TestFastTryLock, and
    TestFastLockErrorCode on MutexSemTest test case, because target class
    MutexSem does not have these methods.
  • Rename FastPollingMutexTest test case to FastPollingMutexSemTest (this
    rename includes renaming of the file name for its container files and
    updating of references on cpp and makefiles).

History

#1 Updated by Riccardo Vitelli almost 10 years ago

  • Target version changed from Backlog to 0.1
  • Start date deleted (29.05.2015)
  • Estimated time set to 2.00 h

#2 Updated by Riccardo Vitelli almost 10 years ago

  • Status changed from New to Arch: Rev

#3 Updated by Riccardo Vitelli almost 10 years ago

  • Assignee set to André Neto

#4 Updated by André Neto almost 10 years ago

  • Description updated (diff)

#5 Updated by Ivan Herrero almost 10 years ago

  • Status changed from Arch: Rev to Code: Impl

#6 Updated by Ivan Herrero almost 10 years ago

  • Assignee changed from André Neto to Giuseppe Ferro

#7 Updated by Ivan Herrero almost 10 years ago

  • Assignee changed from Giuseppe Ferro to Ivan Herrero

#8 Updated by Ivan Herrero almost 10 years ago

  • Status changed from Code: Impl to Code: Rev

#9 Updated by Ivan Herrero almost 10 years ago

  • Description updated (diff)

#10 Updated by Ivan Herrero almost 10 years ago

  • Description updated (diff)
  • Assignee deleted (Ivan Herrero)

Code and documentation review made. Code and documentation corrections pending.

#11 Updated by Ivan Herrero almost 10 years ago

  • Status changed from Code: Rev to Code: Impl

#12 Updated by Riccardo Vitelli almost 10 years ago

  • Assignee set to Giuseppe Ferro

#13 Updated by Riccardo Vitelli almost 10 years ago

  • Status changed from Code: Impl to Code: Rev

#14 Updated by Riccardo Vitelli almost 10 years ago

  • Assignee changed from Giuseppe Ferro to Ivan Herrero

#15 Updated by Ivan Herrero almost 10 years ago

  • Description updated (diff)
  • Assignee deleted (Ivan Herrero)

Code and documentation review made on revision b99ba9f. Code and documentation corrections pending.

#16 Updated by André Neto almost 10 years ago

  • Status changed from Code: Rev to Code: Impl

#17 Updated by André Neto almost 10 years ago

  • Assignee set to Giuseppe Ferro

#18 Updated by Riccardo Vitelli almost 10 years ago

  • Status changed from Code: Impl to Code: Rev

#19 Updated by Riccardo Vitelli almost 10 years ago

  • Assignee deleted (Giuseppe Ferro)

#20 Updated by Ivan Herrero almost 10 years ago

  • Description updated (diff)

All corrections done.

#21 Updated by Ivan Herrero almost 10 years ago

  • Status changed from Code: Rev to Code: Impl

#22 Updated by Ivan Herrero almost 10 years ago

  • Status changed from Code: Impl to Unit: Impl

#23 Updated by Giuseppe Ferro almost 10 years ago

  • Assignee set to Giuseppe Ferro

#24 Updated by Riccardo Vitelli almost 10 years ago

  • Target version changed from 0.1 to 0.2

#25 Updated by Riccardo Vitelli over 9 years ago

  • Assignee deleted (Giuseppe Ferro)

#26 Updated by Riccardo Vitelli over 9 years ago

  • Status changed from Unit: Impl to Unit: Rev

#27 Updated by Ivan Herrero over 9 years ago

  • Description updated (diff)

#28 Updated by Ivan Herrero over 9 years ago

  • Description updated (diff)

#29 Updated by Ivan Herrero over 9 years ago

  • Assignee set to Ivan Herrero

#30 Updated by Ivan Herrero over 9 years ago

  • Description updated (diff)

New issues discovered but fixed by myself on commit 79091e8:

  • Add a unit test for EventSem::IsClosed() on EventSemTest test case.
  • Correct date format on file's documentation for EventSemTest.h.
  • Add a unit test for MutexSem::MutexSem() on MutexSemTest test case.
  • Add a unit test for MutexSem::IsClosed() on MutexSemTest test case.
  • Remove unit tests TestFastLock, TestFastUnLock, TestFastTryLock, and TestFastLockErrorCode on MutexSemTest test case, because target class MutexSem does not have these methods.
  • Rename FastPollingMutexTest test case to FastPollingMutexSemTest (this rename includes renaming of the file name for its container files and updating of references on cpp and makefiles).

#31 Updated by Ivan Herrero over 9 years ago

  • Description updated (diff)

#32 Updated by Riccardo Vitelli over 9 years ago

  • Status changed from Unit: Rev to Closed

Also available in: Atom PDF