2022-11-21 15:49:13

by Thomas Gleixner

[permalink] [raw]
Subject: [patch V2 03/33] genirq/msi: Provide data structs for per device domains

Provide struct msi_domain_template which contains a bundle of struct
irq_chip, struct msi_domain_ops and struct msi_domain_info and a name
field.

This template is used by MSI device domain implementations to provide the
domain specific functionality, feature bits etc.

When a MSI domain is created the template is duplicated in the core code
so that it can be modified per instance. That means templates can be
marked const at the MSI device domain code.

The template is a bundle to avoid several allocations and duplications
of the involved structures.

The name field is used to construct the final domain and chip name via:

$PREFIX-$NAME-$DEVNAME

where prefix is the optional prefix of the MSI parent domain, $NAME is the
provided name in template::chip and the device name so that the domain
is properly identified. On x86 this results for PCI/MSI in:

PCI-MSI-0000:3d:00.1 or IR-PCI-MSIX-0000:3d:00.1

depending on the domain type and the availability of remapping.

Signed-off-by: Thomas Gleixner <[email protected]>
---
include/linux/msi.h | 16 +++++++++++++++-
1 file changed, 15 insertions(+), 1 deletion(-)

--- a/include/linux/msi.h
+++ b/include/linux/msi.h
@@ -24,6 +24,7 @@
#include <linux/xarray.h>
#include <linux/mutex.h>
#include <linux/list.h>
+#include <linux/irq.h>
#include <linux/bits.h>

#include <asm/msi.h>
@@ -74,7 +75,6 @@ struct msi_msg {

extern int pci_msi_ignore_mask;
/* Helper functions */
-struct irq_data;
struct msi_desc;
struct pci_dev;
struct platform_msi_priv_data;
@@ -434,6 +434,20 @@ struct msi_domain_info {
void *data;
};

+/**
+ * struct msi_domain_template - Template for MSI device domains
+ * @name: Storage for the resulting name. Filled in by the core.
+ * @chip: Interrupt chip for this domain
+ * @ops: MSI domain ops
+ * @info: MSI domain info data
+ */
+struct msi_domain_template {
+ char name[48];
+ struct irq_chip chip;
+ struct msi_domain_ops ops;
+ struct msi_domain_info info;
+};
+
/*
* Flags for msi_domain_info
*



2022-11-23 08:41:14

by Tian, Kevin

[permalink] [raw]
Subject: RE: [patch V2 03/33] genirq/msi: Provide data structs for per device domains

> From: Thomas Gleixner <[email protected]>
> Sent: Monday, November 21, 2022 10:38 PM
>
> Provide struct msi_domain_template which contains a bundle of struct
> irq_chip, struct msi_domain_ops and struct msi_domain_info and a name
> field.
>
> This template is used by MSI device domain implementations to provide the
> domain specific functionality, feature bits etc.
>
> When a MSI domain is created the template is duplicated in the core code
> so that it can be modified per instance. That means templates can be
> marked const at the MSI device domain code.
>
> The template is a bundle to avoid several allocations and duplications
> of the involved structures.
>
> The name field is used to construct the final domain and chip name via:
>
> $PREFIX-$NAME-$DEVNAME

$PREFIX$NAME-$DEVNAME

2022-11-23 12:41:12

by Thomas Gleixner

[permalink] [raw]
Subject: RE: [patch V2 03/33] genirq/msi: Provide data structs for per device domains

On Wed, Nov 23 2022 at 07:57, Kevin Tian wrote:
>> From: Thomas Gleixner <[email protected]>
>> Sent: Monday, November 21, 2022 10:38 PM
>>
>> Provide struct msi_domain_template which contains a bundle of struct
>> irq_chip, struct msi_domain_ops and struct msi_domain_info and a name
>> field.
>>
>> This template is used by MSI device domain implementations to provide the
>> domain specific functionality, feature bits etc.
>>
>> When a MSI domain is created the template is duplicated in the core code
>> so that it can be modified per instance. That means templates can be
>> marked const at the MSI device domain code.
>>
>> The template is a bundle to avoid several allocations and duplications
>> of the involved structures.
>>
>> The name field is used to construct the final domain and chip name via:
>>
>> $PREFIX-$NAME-$DEVNAME
>
> $PREFIX$NAME-$DEVNAME

Indeed.