static bool execute(HandshakeClosure* hs_cl, JavaThread* target);
  static bool execute_direct(HandshakeClosure* hs_cl, JavaThread* target);

There is no documentation as to what the return value indicates, or how to interpret it. Especially for execute_direct where we may need to be able to distinguish thread lifecycle issues from general errors.