linux-kernel - RE: [patch V3 28/33] PCI/MSI: Provide IMS (Interrupt Message Store) support

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <BN9PR11MB5276A75A242ACA5B8ADA55478C3A2@BN9PR11MB5276.namprd11.prod.outlook.com>
Date: Fri, 29 Mar 2024 01:41:27 +0000
From: "Tian, Kevin" <kevin.tian@...el.com>
To: Bjorn Helgaas <helgaas@...nel.org>, Thomas Gleixner <tglx@...utronix.de>
CC: LKML <linux-kernel@...r.kernel.org>, "x86@...nel.org" <x86@...nel.org>,
	Joerg Roedel <joro@...tes.org>, Will Deacon <will@...nel.org>,
	"linux-pci@...r.kernel.org" <linux-pci@...r.kernel.org>, Bjorn Helgaas
	<bhelgaas@...gle.com>, Lorenzo Pieralisi <lorenzo.pieralisi@....com>, "Marc
 Zyngier" <maz@...nel.org>, Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
	Jason Gunthorpe <jgg@...lanox.com>, "Jiang, Dave" <dave.jiang@...el.com>,
	Alex Williamson <alex.williamson@...hat.com>, "Williams, Dan J"
	<dan.j.williams@...el.com>, Logan Gunthorpe <logang@...tatee.com>, "Raj,
 Ashok" <ashok.raj@...el.com>, Jon Mason <jdmason@...zu.us>, Allen Hubbe
	<allenbh@...il.com>
Subject: RE: [patch V3 28/33] PCI/MSI: Provide IMS (Interrupt Message Store)
 support

> From: Bjorn Helgaas <helgaas@...nel.org>
> Sent: Thursday, March 28, 2024 12:33 AM
> 
> On Fri, Nov 25, 2022 at 12:26:29AM +0100, Thomas Gleixner wrote:
> > IMS (Interrupt Message Store) is a new specification which allows
> > implementation specific storage of MSI messages contrary to the
> > strict standard specified MSI and MSI-X message stores.
> > ...
> 
> > + * pci_create_ims_domain - Create a secondary IMS domain for a PCI
> device
> 
> > + * Return: True on success, false otherwise
> 
> > +bool pci_create_ims_domain(struct pci_dev *pdev, const struct
> msi_domain_template *template,
> > +			   unsigned int hwsize, void *data)
> 
> pci_create_ims_domain() is exported for use by modules, but AFAICT,
> there is no in-tree user of this yet.
> 
> I assume one is coming, but if there isn't one on the near horizon,
> we could/should remove this for now.
> 

There won't be one in the near term. So I agree it's a good idea to
remove it. Anyway this can be easily added back when the real
user comes.