c++ - Is there anything wrong with passing "this" as an argument to ISR registration?

Question

I am writing a firmware for an esp32 in C++ and I found it to be awesome to pass an object reference as parameter when registering an ISR because I can simply delegate the work from the ISR to the specific object (a resource in this case), managing the resource properties and functionality directly from the class. However, I'm not exactly sure what to look out for when using this technique.

1. Is there anything inherently wrong with it?
2. If not, what should one be aware of when doing it?

question from:https://stackoverflow.com/questions/65843029/is-there-anything-wrong-with-passing-this-as-an-argument-to-isr-registration

Answer 1

Is there anything wrong with passing “this” as an argument to ISR registration?

No.

Is there anything inherently wrong with it?

No.

If not, what should one be aware of when doing it?

Object lifetime. Compiler optimization. Resource sharing. Locking. Atomic access. Specific hardware semantics. Additionally, esp32 is a two core platform. The list could go on and there are many books about programming on esp32, about programming in parallel environments and sharing resources between main part of code and interrupt routines. One should be aware of all such programming related topics and also about good code conventions, naming and C++ language.