[PATCH 1/2] nvmet_fc: add defer_req callback for deferment of cmd buffer return

Christoph Hellwig hch at infradead.org
Thu Aug 10 02:06:04 PDT 2017


> @@ -463,9 +472,9 @@ static struct nvmet_fc_fcp_iod *
>  nvmet_fc_alloc_fcp_iod(struct nvmet_fc_tgt_queue *queue)
>  {
>  	static struct nvmet_fc_fcp_iod *fod;

This isn't new, but is this really supposed to be a static variable,
that is all instances of this code sharing it use the same?

After a short code inspection this looks like a nasty bug to me.



More information about the Linux-nvme mailing list