On 7/3/20 8:34 AM, Klaus Jensen wrote: > From: Klaus Jensen <k.jen...@samsung.com> > > Reject the nsid broadcast value (0xffffffff) and 0xfffffffe in the > Active Namespace ID list.
Can we have a definition instead of this 0xfffffffe magic value please? > > Signed-off-by: Klaus Jensen <k.jen...@samsung.com> > --- > hw/block/nvme.c | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/hw/block/nvme.c b/hw/block/nvme.c > index 65c2fa3ac1f4..0dac7a41ddae 100644 > --- a/hw/block/nvme.c > +++ b/hw/block/nvme.c > @@ -956,6 +956,10 @@ static uint16_t nvme_identify_nslist(NvmeCtrl *n, > NvmeIdentify *c) > > trace_pci_nvme_identify_nslist(min_nsid); > > + if (min_nsid == 0xfffffffe || min_nsid == NVME_NSID_BROADCAST) { > + return NVME_INVALID_NSID | NVME_DNR; > + } > + > list = g_malloc0(data_len); > for (i = 0; i < n->num_namespaces; i++) { > if (i < min_nsid) { >