On Mon, Apr 04, 2022 at 08:44:20PM +0530, Kanchan Joshi wrote: > Another stuff that got left out from the previous series :-( > Using this field for a bit of sanity checking at the moment. Like this in nvme: > > + if (ioucmd->cmd_len != sizeof(struct nvme_passthru_cmd64)) > + return -EINVAL; > + cptr = (struct nvme_passthru_cmd64 *)ioucmd->cmd; Do we actually need that sanity checking? Each command should have a known length bound by the SQE size, right?
