On Mon, Jan 26, 2026 at 07:04:38PM -0300, Fabiano Rosas wrote: > Queued, let me know if you want to change patch 2.
Thanks. I have no strong preference. Considering that cpr-exec's execve() is not designed to fail at all at least in production, and FAILED after DONE is utterly confusing.. maybe we don't need to mention it in a doc everyone would read? I actually instead hope nobody noticed it's even possible.. I believe that code was there only because when Steve worked on it he wanted to trap when he typed something wrong by accident when testing cpr-exec. It's a feature too special and mustn't used wrong or VM data can definitely get lost. From that POV, maybe some day later we can use an g_assert_not_reached() to replace that whole blob, including the notify. -- Peter Xu
