Re: [PATCH v0 1/6] nvme-fabrics: introduce connect_sync option

From: Christoph Hellwig
Date: Tue Feb 20 2024 - 01:51:51 EST


On Fri, Feb 16, 2024 at 05:44:02PM +0100, Daniel Wagner wrote:
> James answered this point on my attempt to make this synchronous:
>
> https://lore.kernel.org/linux-nvme/0605ac36-16d5-2026-d3c6-62d346db6dfb@xxxxxxxxx/

That needs to go into the commit log. And I call complete BS on that
to be honest.

> Ideally, we could agree on behavior which is the same for all
> transports.

sync and async opt in it is. I'm still pissed FC did this differently
without any proper discussion of the tradeoffs.