On 2020/11/28 9:13, Chaitanya Kulkarni wrote:
> On 11/26/20 00:40, Damien Le Moal wrote:
>>> diff --git a/drivers/nvme/target/admin-cmd.c b/drivers/nvme/target/admin-cmd.c
>>> index e7d2b96cda6b..cd368cbe3855 100644
>>> --- a/drivers/nvme/target/admin-cmd.c
>>> +++ b/drivers/nvme/target/admin-cmd.c
>>> @@ -648,6 +648,10 @@ static void nvmet_execute_identify(struct nvmet_req *req)
>>> switch (req->cmd->identify.cns) {
>>> case NVME_ID_CNS_NS:
>>> return nvmet_execute_identify_ns(req);
>>> + case NVME_ID_CNS_CS_NS:
>>> + if (req->cmd->identify.csi == NVME_CSI_ZNS)
>>> + return nvmet_execute_identify_cns_cs_ns(req);
>>> + break;
>>> case NVME_ID_CNS_CTRL:
>>> return nvmet_execute_identify_ctrl(req);
>>> case NVME_ID_CNS_CS_CTRL:
>>>
>> Same patch as patch 5 ? Bug ?
>
> Yes, but right now there are no ns-type than ZNS that we support.
> Can you please explain I think I didn't understand what bug you are
> referring to.
Patch 5 and 6 in the series you posted look identical to me. It looks like the
same patch was sent twice with a different number. Unless I missed some subtle
difference between them ?
--
Damien Le Moal
Western Digital Research
