Messages in this thread | | | Date | Fri, 13 Jan 2023 12:28:45 +0000 | From | Jonathan Cameron <> | Subject | Re: [PATCH v6 2/8] cxl/mem: Wire up event interrupts |
| |
On Mon, 09 Jan 2023 11:42:21 -0800 ira.weiny@intel.com wrote:
> From: Davidlohr Bueso <dave@stgolabs.net> > > Currently the only CXL features targeted for irq support require their > message numbers to be within the first 16 entries. The device may > however support less than 16 entries depending on the support it > provides. > > Attempt to allocate these 16 irq vectors. If the device supports less > then the PCI infrastructure will allocate that number. Upon successful > allocation, users can plug in their respective isr at any point > thereafter. > > CXL device events are signaled via interrupts. Each event log may have > a different interrupt message number. These message numbers are > reported in the Get Event Interrupt Policy mailbox command. > > Add interrupt support for event logs. Interrupts are allocated as > shared interrupts. Therefore, all or some event logs can share the same > message number. > > In addition all logs are queried on any interrupt in order of the most > to least severe based on the status register. > > Finally place all event configuration logic into cxl_event_config(). > Previously the logic was a simple 'read all' on start up. But > interrupts must be configured prior to any reads to ensure no events are > missed. A single event configuration function results in a cleaner over > all implementation. > > Cc: Bjorn Helgaas <helgaas@kernel.org> > Cc: Jonathan Cameron <Jonathan.Cameron@huawei.com> > Co-developed-by: Ira Weiny <ira.weiny@intel.com> > Signed-off-by: Ira Weiny <ira.weiny@intel.com> > Signed-off-by: Davidlohr Bueso <dave@stgolabs.net>
> > --- > Changes from V5: > Dan > Addressed missed comments on V4 > https://lore.kernel.org/all/6398dd6ed80fa_b05d1294fc@dwillia2-xfh.jf.intel.com.notmuch/
You also added pci_set_master() :)
Which means I can now give Reviewed-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
I only noticed because I couldn't remember why I'd not given a tag to v4.
Jonathan
| |