[PATCH v2] nvme: fix nvme_pr_* status code parsing

Keith Busch kbusch at kernel.org
Fri May 31 13:53:24 PDT 2024


On Fri, May 31, 2024 at 10:16:50AM +0200, Christoph Hellwig wrote:
> On Thu, May 30, 2024 at 02:16:46PM +0800, Weiwen Hu wrote:
> > +	switch (nvme_sc & 0x7ff) {
> 
> Can you use the opportunity to add a NVME_SC_MASK symbolic
> name for the magic 0x7ff?  We should have done this earlier
> with already 5 occurrences, but better now than never.
> 
> Otherwise this looks good to me.

Yeah, I suggested sending the minimal patch first just for the
backport-ability.



More information about the Linux-nvme mailing list