Skip to content

Commit

Permalink
genirq/msi: Make descriptor allocation device domain aware
Browse files Browse the repository at this point in the history
Change the descriptor allocation and insertion functions to take a domain
id to prepare for the upcoming multi MSI domain per device support.

Signed-off-by: Thomas Gleixner <[email protected]>
Reviewed-by: Kevin Tian <[email protected]>
Acked-by: Marc Zyngier <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
  • Loading branch information
KAGA-KOKO committed Dec 5, 2022
1 parent 1c89396 commit fc8ab38
Show file tree
Hide file tree
Showing 2 changed files with 27 additions and 9 deletions.
16 changes: 15 additions & 1 deletion include/linux/msi.h
Original file line number Diff line number Diff line change
Expand Up @@ -286,7 +286,21 @@ static inline void msi_desc_set_iommu_cookie(struct msi_desc *desc,
}
#endif

int msi_insert_msi_desc(struct device *dev, struct msi_desc *init_desc);
int msi_domain_insert_msi_desc(struct device *dev, unsigned int domid,
struct msi_desc *init_desc);
/**
* msi_insert_msi_desc - Allocate and initialize a MSI descriptor in the
* default irqdomain and insert it at @init_desc->msi_index
* @dev: Pointer to the device for which the descriptor is allocated
* @init_desc: Pointer to an MSI descriptor to initialize the new descriptor
*
* Return: 0 on success or an appropriate failure code.
*/
static inline int msi_insert_msi_desc(struct device *dev, struct msi_desc *init_desc)
{
return msi_domain_insert_msi_desc(dev, MSI_DEFAULT_DOMAIN, init_desc);
}

void msi_free_msi_descs_range(struct device *dev, unsigned int first_index, unsigned int last_index);

/**
Expand Down
20 changes: 12 additions & 8 deletions kernel/irq/msi.c
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ static inline int msi_sysfs_create_group(struct device *dev);
* Return: pointer to allocated &msi_desc on success or %NULL on failure
*/
static struct msi_desc *msi_alloc_desc(struct device *dev, int nvec,
const struct irq_affinity_desc *affinity)
const struct irq_affinity_desc *affinity)
{
struct msi_desc *desc = kzalloc(sizeof(*desc), GFP_KERNEL);

Expand All @@ -64,9 +64,10 @@ static void msi_free_desc(struct msi_desc *desc)
kfree(desc);
}

static int msi_insert_desc(struct msi_device_data *md, struct msi_desc *desc, unsigned int index)
static int msi_insert_desc(struct msi_device_data *md, struct msi_desc *desc,
unsigned int domid, unsigned int index)
{
struct xarray *xa = &md->__domains[MSI_DEFAULT_DOMAIN].store;
struct xarray *xa = &md->__domains[domid].store;
int ret;

desc->msi_index = index;
Expand All @@ -77,15 +78,17 @@ static int msi_insert_desc(struct msi_device_data *md, struct msi_desc *desc, un
}

/**
* msi_insert_msi_desc - Allocate and initialize a MSI descriptor and
* insert it at @init_desc->msi_index
* msi_domain_insert_msi_desc - Allocate and initialize a MSI descriptor and
* insert it at @init_desc->msi_index
*
* @dev: Pointer to the device for which the descriptor is allocated
* @domid: The id of the interrupt domain to which the desriptor is added
* @init_desc: Pointer to an MSI descriptor to initialize the new descriptor
*
* Return: 0 on success or an appropriate failure code.
*/
int msi_insert_msi_desc(struct device *dev, struct msi_desc *init_desc)
int msi_domain_insert_msi_desc(struct device *dev, unsigned int domid,
struct msi_desc *init_desc)
{
struct msi_desc *desc;

Expand All @@ -97,7 +100,8 @@ int msi_insert_msi_desc(struct device *dev, struct msi_desc *init_desc)

/* Copy type specific data to the new descriptor. */
desc->pci = init_desc->pci;
return msi_insert_desc(dev->msi.data, desc, init_desc->msi_index);

return msi_insert_desc(dev->msi.data, desc, domid, init_desc->msi_index);
}

/**
Expand All @@ -120,7 +124,7 @@ static int msi_add_simple_msi_descs(struct device *dev, unsigned int index, unsi
desc = msi_alloc_desc(dev, 1, NULL);
if (!desc)
goto fail_mem;
ret = msi_insert_desc(dev->msi.data, desc, idx);
ret = msi_insert_desc(dev->msi.data, desc, MSI_DEFAULT_DOMAIN, idx);
if (ret)
goto fail;
}
Expand Down

0 comments on commit fc8ab38

Please sign in to comment.