On 2020-02-21 02:46, Jack Wang wrote: > +enum rtrs_imm_type { > + RTRS_IO_REQ_IMM = 0, /* client to server */ > + RTRS_IO_RSP_IMM = 1, /* server to client */ > + RTRS_IO_RSP_W_INV_IMM = 2, /* server to client */ > + > + RTRS_HB_MSG_IMM = 8, > + RTRS_HB_ACK_IMM = 9, > + > + RTRS_LAST_IMM, > +}; This is the first time in this header file that the abbreviation "hb" is used. Please add a comment that explains what this abbreviation stands for. Thanks, Bart.