G1ParScanThreadState::do_copy_to_survivor_space() has an unnecessary change against upstream:

 if (old_mark.is_marked()) {
    // Already forwarded by somebody else, return forwardee.
    return old->forwardee(old_mark);
  }

All callers of this method already do the same, so this only adds code and possible affects performance for no reason.