On 6/10/21 19:14, Wu Bo wrote: >> case NVME_ID_CNS_CTRL: >> - return nvmet_execute_identify_ctrl(req); >> + switch (req->cmd->identify.csi) { >> + case NVME_CSI_NVM: >> + return nvmet_execute_identify_ctrl(req); >> + } > Is it missing to add the default branch here ? > maybe for the consistency