[LOG] RT 4.0 and NIL 2.0

[crispus]

Thanks for your prompt reply.

One more thing

Code: Select all

void chSysHalt(const char *reason) {

  port_disable();

  /* Halt hook code, usually empty.*/
  CH_CFG_SYSTEM_HALT_HOOK(reason);

  /* Logging the event.*/
  _trace_halt(reason);

  /* Pointing to the passed message.*/
  ch.dbg.panic_msg = reason;

  /* Harmless infinite loop.*/
  while (true) {
  }
}

Can you move the system halt hook just before the infinite loop? In this way, I can stop the execution in the hook, and the event will still be logged.
I don't know if this would be interesting for everyone, but it's worth a try

Thanks again

[Giovanni]

Hi,

Bug fixed and modified chSysHalt() as suggested.

Giovanni

[crispus]

Thank you so much.

[Thargon]

I have a suggestion regarding the C++ wrappers:

Several classes already inherit others (e.g. BaseStaticThread inherits BaseThread inherits ThreadReference), which may lead to the diamond problem when creating new custom classes. To solve this, all classes in the ChibiOS C++ wrapper should use the keyword "virtual" for inheritance (unless virtual is explicitly unsafe in a particular case).

Thomas