Received-SPF: pass (google.com: domain of linux-kernel+bounces-29701-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223;
Date: Wed, 17 Jan 2024 20:06:25 -0800
From: Davidlohr Bueso <dave@stgolabs.net>
To: Ira Weiny <ira.weiny@intel.com>
Cc: Jonathan Cameron <jonathan.cameron@huawei.com>, 
	Dave Jiang <dave.jiang@intel.com>, Alison Schofield <alison.schofield@intel.com>, 
	Vishal Verma <vishal.l.verma@intel.com>, Dan Williams <dan.j.williams@intel.com>, 
	linux-cxl@vger.kernel.org, linux-kernel@vger.kernel.org, 
	Dave Larsen <davelarsen58@gmail.com>, Fan Ni <fan.ni@samsung.com>
Subject: Re: [PATCH v2] cxl/pci: Skip irq features if MSI/MSI-X are not
 supported
Message-ID: <t6wkohozmtchuzzabjigr66tx6576nni54ig7lu2orlvqwmt5o@r52mxzei7uxs>
References: <20240117-dont-fail-irq-v2-1-f33f26b0e365@intel.com>
Precedence: bulk
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii; format=flowed
Content-Disposition: inline
In-Reply-To: <20240117-dont-fail-irq-v2-1-f33f26b0e365@intel.com>
User-Agent: NeoMutt/20231221

On Wed, 17 Jan 2024, Ira Weiny wrote:

>CXL 3.1 Section 3.1.1 states:
>
>	"A Function on a CXL device must not generate INTx messages if
>	that Function participates in CXL.cache protocol or CXL.mem
>	protocols."
>
>The generic CXL memory driver only supports devices which use the
>CXL.mem protocol.  The current driver attempts to allocate MSI/MSI-X
>vectors in anticipation of their need for mailbox interrupts or event
>processing.  However, the above requirement does not require a device to
>support interrupts, only that they use MSI/MSI-X.  For example, a device
>may disable mailbox interrupts and either be configured for firmware
>first or skip event processing and function.
>
>Dave Larsen reported that the following Intel / Agilex card does not
>support interrupts on function 0.
>
>	CXL: Intel Corporation Device 0ddb (rev 01) (prog-if 10 [CXL Memory Device (CXL 2.x)])
>
>Rather than fail device probe if interrupts are not supported; flag that
>irqs are not enabled and avoid features which require interrupts.
>Emit messages appropriate for the situation to aid in debugging should
>device behavior be unexpected due to a failure to allocate vectors.
>
>Note that it is possible for a device to have host based event
>processing through polling.  However, the driver does not support
>polling and it is not anticipated to be generally required.  Leave that
>functionality to a future patch if such a device comes along.
>
>Reported-by: Dave Larsen <davelarsen58@gmail.com>
>Reviewed-by: Dave Jiang <dave.jiang@intel.com>
>Reviewed-by: Fan Ni <fan.ni@samsung.com>
>Signed-off-by: Ira Weiny <ira.weiny@intel.com>

Yes, I suspected this might come up at some point.

Reviewed-and-tested-by: Davidlohr Bueso <dave@stgolabs.net>