ros_canopen icon indicating copy to clipboard operation
ros_canopen copied to clipboard

Refine fatal error configuration.

Open windelbouwman opened this issue 3 years ago • 3 comments

This refines selection of fatal en non-fatal errors.

There was no way to select BUSOFF as non fatal. My use case would be to select BUSOFF as non fatal and configure the can device with a restart-ms.

windelbouwman avatar Mar 01 '21 12:03 windelbouwman

There was no way to select BUSOFF as non fatal.

It was written this way on purpose ;)

My use case would be to select BUSOFF as non fatal and configure the can device with a restart-ms

Does this really work with this fix? (I did not have a chance to test it)

mathias-luedtke avatar Mar 01 '21 13:03 mathias-luedtke

It was written this way on purpose ;)

Why would the BUS_OFF error be a special case? I think it's fair to treat it as the other errors. What will happen with BUS_OFF, is that the output queue will fill up, and eventually the send asio call will fail / block.

windelbouwman avatar Mar 01 '21 19:03 windelbouwman

What will happen with BUS_OFF, is that the output queue will fill up, and eventually the send asio call will fail / block.

In our usecase, the output queue will fill up immediately and then the kernel driver would close the socket..

Why would the BUS_OFF error be a special case? I think it's fair to treat it as the other errors

Agreed. Again, I did not test this special case.

mathias-luedtke avatar Mar 01 '21 19:03 mathias-luedtke