ESP crash when using interrupts + webserver about hlw8012 HOT 7 CLOSED

Changes included in version 1.0.1 of the library

from hlw8012.

Hi
Sorry, been away for a while. Very useful information. Will update the library and my code.
Thank you so much!

from hlw8012.

Original comment by Alexander Christian (Bitbucket: alexander_christian, GitHub: Unknown):

Still working fine. Webpage refreshes every 5sec and displays interrupt-measured values. Now running for ~45min... So I assume: Finally solved.
Please consider putting the attribute to your lib and mention the use of the attribute in your documentation.

from hlw8012.

Original comment by Alexander Christian (Bitbucket: alexander_christian, GitHub: Unknown):

Issue solved for me.

Solution, according to esp8266/Arduino#3337 (comment):

Put ICACHE_RAM_ATTR attribute to library's interrupt handler functions, as well as to interrupt-wrapper functions in sketch:

So, in HLW8012.cpp:

#!arduino

void ICACHE_RAM_ATTR  HLW8012::cf_interrupt() {
    unsigned long now = micros();
    _power_pulse_width = now - _last_cf_interrupt;
    _last_cf_interrupt = now;
}

void ICACHE_RAM_ATTR  HLW8012::cf1_interrupt() {

    unsigned long now = micros();
    unsigned long pulse_width;

    if ((now - _first_cf1_interrupt) > _pulse_timeout) {

        if (_last_cf1_interrupt == _first_cf1_interrupt) {
......

and in your sketch:

#!arduino

// redirect interrupt to HLW library
void ICACHE_RAM_ATTR  hlw8012_cf1_interrupt() {
  _hlw8012.cf1_interrupt();
}
void ICACHE_RAM_ATTR  hlw8012_cf_interrupt() {
  _hlw8012.cf_interrupt();
}

No crash since ~15min... Before it crashed within seconds...

from hlw8012.

Original comment by Alexander Christian (Bitbucket: alexander_christian, GitHub: Unknown):

looks like the ESP does not crash if I simply disable the "crashing" interrupt-handler:

#!arduino

  // attachInterrupt(GPIO_HLW8012_CF1, hlw8012_cf1_interrupt, CHANGE);
  attachInterrupt(GPIO_HLW8012_CF, hlw8012_cf_interrupt, CHANGE);

very weird... the only difference is then, that

• one interrupt less
• one digitalWrite()
• a bit less if/else logic

Any ideas?

from hlw8012.

Original comment by Alexander Christian (Bitbucket: alexander_christian, GitHub: Unknown):

Also created an issue on esp8266 arduino core: esp8266/Arduino#3337

from hlw8012.

Original comment by Alexander Christian (Bitbucket: alexander_christian, GitHub: Unknown):

Looks like it's esp8266 arduino core related?! --> esp8266/Arduino#1020
But the suggested solution (disabling interrupts before actually handling interrupt code in HLW lib and enabling it again after) does not work/solve the problem.... Still facing crashes.

from hlw8012.