Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 2f4ae01d authored by Joerg Roedel's avatar Joerg Roedel Committed by Patrick Daly
Browse files

iommu: Move report_iommu_fault() to iommu.c 



The function is in no fast-path, there is no need for it to
be static inline in a header file. This also removes the
need to include iommu trace-points in iommu.h.

Signed-off-by: default avatarJoerg Roedel <jroedel@suse.de>
Git-commit: 207c6e36f122ebb1164d611c9f34f128313f47d5
Git-repo: git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git


Change-Id: I73967268e951747c3d66e19da895beb44ff5569f
Signed-off-by: default avatarPatrick Daly <pdaly@codeaurora.org>
parent b578e4b9

drivers/iommu/iommu.c

+42 −1
Original line number Diff line number Diff line
@@ -1519,8 +1519,49 @@ void iommu_domain_window_disable(struct iommu_domain *domain, u32 wnd_nr) 
}

EXPORT_SYMBOL_GPL(iommu_domain_window_disable);



struct dentry *iommu_debugfs_top;

/**

 * report_iommu_fault() - report about an IOMMU fault to the IOMMU framework

 * @domain: the iommu domain where the fault has happened

 * @dev: the device where the fault has happened

 * @iova: the faulting address

 * @flags: mmu fault flags (e.g. IOMMU_FAULT_READ/IOMMU_FAULT_WRITE/...)

 *

 * This function should be called by the low-level IOMMU implementations

 * whenever IOMMU faults happen, to allow high-level users, that are

 * interested in such events, to know about them.

 *

 * This event may be useful for several possible use cases:

 * - mere logging of the event

 * - dynamic TLB/PTE loading

 * - if restarting of the faulting device is required

 *

 * Returns 0 on success and an appropriate error code otherwise (if dynamic

 * PTE/TLB loading will one day be supported, implementations will be able

 * to tell whether it succeeded or not according to this return value).

 *

 * Specifically, -ENOSYS is returned if a fault handler isn't installed

 * (though fault handlers can also return -ENOSYS, in case they want to

 * elicit the default behavior of the IOMMU drivers).

 */

int report_iommu_fault(struct iommu_domain *domain, struct device *dev,

		       unsigned long iova, int flags)

{

	int ret = -ENOSYS;



	/*

	 * if upper layers showed interest and installed a fault handler,

	 * invoke it.

	 */

	if (domain->handler)

		ret = domain->handler(domain, dev, iova, flags,

						domain->handler_token);



	trace_io_page_fault(dev, iova, flags);

	return ret;

}

EXPORT_SYMBOL_GPL(report_iommu_fault);



struct dentry *iommu_debugfs_top;

static int __init iommu_init(void)

{

	iommu_group_kset = kset_create_and_add("iommu_groups",

include/linux/iommu.h

+2 −44
Original line number Diff line number Diff line
@@ -341,51 +341,9 @@ extern void iommu_domain_window_disable(struct iommu_domain *domain, u32 wnd_nr) 


extern uint64_t iommu_iova_to_pte(struct iommu_domain *domain,

	    dma_addr_t iova);

/**

 * report_iommu_fault() - report about an IOMMU fault to the IOMMU framework

 * @domain: the iommu domain where the fault has happened

 * @dev: the device where the fault has happened

 * @iova: the faulting address

 * @flags: mmu fault flags (e.g. IOMMU_FAULT_READ/IOMMU_FAULT_WRITE/...)

 *

 * This function should be called by the low-level IOMMU implementations

 * whenever IOMMU faults happen, to allow high-level users, that are

 * interested in such events, to know about them.

 *

 * This event may be useful for several possible use cases:

 * - mere logging of the event

 * - dynamic TLB/PTE loading

 * - if restarting of the faulting device is required

 *

 * Returns 0 on success and an appropriate error code otherwise (if dynamic

 * PTE/TLB loading will one day be supported, implementations will be able

 * to tell whether it succeeded or not according to this return value).

 *

 * Specifically, -ENOSYS is returned if a fault handler isn't installed

 * (though fault handlers can also return -ENOSYS, in case they want to

 * elicit the default behavior of the IOMMU drivers).



 * Client fault handler returns -EBUSY to signal to the IOMMU driver

 * that the client will take responsibility for any further fault

 * handling, including clearing fault status registers or retrying

 * the faulting transaction.

 */

static inline int report_iommu_fault(struct iommu_domain *domain,

		struct device *dev, unsigned long iova, int flags)

{

	int ret = -ENOSYS;



	/*

	 * if upper layers showed interest and installed a fault handler,

	 * invoke it.

	 */

	if (domain->handler)

		ret = domain->handler(domain, dev, iova, flags,

						domain->handler_token);



	trace_io_page_fault(dev, iova, flags);

	return ret;

}

extern int report_iommu_fault(struct iommu_domain *domain, struct device *dev,

			      unsigned long iova, int flags);



static inline size_t iommu_map_sg(struct iommu_domain *domain,

				  unsigned long iova, struct scatterlist *sg,