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

[Christoph Hellwig <hch@xxxxxxxxxxxxx>]

> @@ -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.