Throw an exception when running wait_for_edge() in a new thread about jetsongpio HOT 7 CLOSED

@ShimmyShaman Can you check this issue please?

from jetsongpio.

As long as the two channels are different channels the usage is okay. There should've been more error detail (at least to the console) for why the edge file cannot be opened (submitted pull request addresses this).

Try again with that, and at least find out why.

If not then I won't get my hands on my Jetson device for another 2-3 days to check it out.

from jetsongpio.

Yeap, right on, a file wasn't being closed when it should've been. Good error report.

Added pull request. The files get closed now and the records don't pile up in /proc/$pid/fd

from jetsongpio.

Thanks! Now it runs fine.

from jetsongpio.

Thanks for the reply. I added the std::perror() and tried again.
I got an extra message:

sysfs/edge open: Too many open files

And I checked the /proc/$pid/fd. There are hundreds of records like:

/sys/devices/2200000.gpio/gpiochip0/gpio/gpio428/value
and
/sys/devices/2200000.gpio/gpiochip0/gpio/gpio445/value

from jetsongpio.

I found it is irrelevant to multi-threading. If I run it independently on a single channel for a long time, I will get the same error.
The complete code is as below.

#include <chrono>
#include <thread>
#include <signal.h>
#include <sys/time.h>
#include <JetsonGPIO.h>

static bool end_this_program = false;
void signalHandler(int s) { end_this_program = true; }
void delay(int s) { std::this_thread::sleep_for(std::chrono::seconds(s)); }


int main()
{
    signal(SIGINT, signalHandler);
    int pwm_input = 11; 
    GPIO::setmode(GPIO::BOARD);
    GPIO::setup(pwm_input, GPIO::IN);
    struct timeval start, end;
    int high;

    while (!end_this_program) 
    {
        printf("start to listen...\n");
        GPIO::wait_for_edge(pwm_input, GPIO::Edge::RISING);
        gettimeofday(&start,NULL);
        GPIO::wait_for_edge(pwm_input, GPIO::Edge::FALLING);
        gettimeofday(&end,NULL);
        high = end.tv_usec - start.tv_usec;
        printf("high voltage time: %d us\n", high);
        if(high>1500 && high<=2000)
        {
            printf("received save flag\n");
        }
    }

    GPIO::cleanup();
    return 0;
}

from jetsongpio.

@frostcza Thank you for confirming and the error report.
@ShimmyShaman Thank you for your help.

Merged the pull request. Closed the issue.

from jetsongpio.