Re: [PATCH v3 01/16] nvmet: report ioccsz and iorcsz for disc ctrl

From: Max Gurtovoy
Date: Tue Dec 19 2023 - 19:50:55 EST




On 19/12/2023 6:27, Christoph Hellwig wrote:
On Mon, Dec 18, 2023 at 04:30:49PM +0100, Daniel Wagner wrote:
+ /*
+ * Max command capsule size is sqe + in-capsule data size.
+ * Disable in-capsule data for Metadata capable controllers.

A why on the disable would be useful here - the fact that it is disabled
is pretty obvious from the code.


We have another thread on this patch that I think is wrong since the discovery controller shouldn't set these values according to the spec. It is explicitly say that ioccsz and iorcsz are reserved for discovery controllers.

I've sent a proposal to fix the initiator/host code.