On Tue, Sep 17, 2013 at 02:19:49PM +0100, Will Deacon wrote:
The following changes since commit 272b98c6455f00884f0350f775c5342358ebb73f:
Linux 3.12-rc1 (2013-09-16 16:17:51 -0400)
are available in the git repository at:
git://git.kernel.org/pub/scm/linux/kernel/git/will/linux.git
On Fri, Sep 13, 2013 at 01:10:27PM -0600, Stephen Warren wrote:
I believe that Joerg Roedel is at least the path through which
drivers/iommu changes should be merged. Add a MAINTAINERS entry to
make this clear, so that he's Cd'd on all relevant patches. This is
relevant for non-AMD/Intel
On Fri, Aug 16, 2013 at 11:20:16AM -0600, Shuah Khan wrote:
+DEFINE_EVENT(iommu_fault, report_fault,
+
+ TP_PROTO(struct device *dev, unsigned long iova, int flags),
+
+ TP_ARGS(dev, iova, flags)
+);
I am not entirely happy with the naming. It is better to name the class
iommu_error
On Tue, Sep 17, 2013 at 04:38:29PM +0800, ZhenHua wrote:
Hi Guys,
Though DMAR_ICS_REG is not used yet, I think this patch is
necessary. So please take a look at it.
You are right, my Spec says the same. It doesn't matter much since the
register seems to be unused in the VT-d driver. I
On 09/24/2013 04:48 AM, Joerg Roedel wrote:
On Fri, Aug 16, 2013 at 11:20:16AM -0600, Shuah Khan wrote:
+DEFINE_EVENT(iommu_fault, report_fault,
+
+ TP_PROTO(struct device *dev, unsigned long iova, int flags),
+
+ TP_ARGS(dev, iova, flags)
+);
I am not entirely happy with the
On Wed, Sep 18, 2013 at 03:09:01PM +0900, Takao Indoh wrote:
+ /*
+ * In the case of kdump, ioremap is needed because root-entry table
+ * exists in first kernel's memory area which is not mapped in second
+ * kernel
+ */
+ root = (struct root_entry
On Tue, 2013-09-24 at 15:16 +0200, Joerg Roedel wrote:
I am not convinced that this is the right approach. If a device wasn't
translated by VT-d in the old kernel doesn't mean it will not be
translated in the new kernel. How about unconditionally resetting all
PCI busses and/or functions
Each device is put into its own protection domain (if possible).
For configuration with one or just a view masters per SMMU that
is easy to achieve.
In case of many devices per SMMU (e.g. MMU-500 with it's distributed
translation support) isolation of each device might not be possible --
... otherwise it is impossible for the low level iommu driver to
figure out which pte flags should be used.
In __map_sg_chunk we can derive the flags from dma_data_direction.
In __iommu_create_mapping we should treat the memory like
DMA_BIDIRECTIONAL and pass both IOMMU_READ and IOMMU_WRITE to
Signed-off-by: Andreas Herrmann andreas.herrm...@calxeda.com
---
drivers/iommu/arm-smmu.c |9 +
1 file changed, 9 insertions(+)
diff --git a/drivers/iommu/arm-smmu.c b/drivers/iommu/arm-smmu.c
index 251564e..a499146 100644
--- a/drivers/iommu/arm-smmu.c
+++ b/drivers/iommu/arm-smmu.c
arm-smmu= arm-smmu driver option
off Disable arm-smmu driver (ie. ignore available SMMUs)
force_isolation
Try to attach each master device on every SMMU to a
separate iommu_domain.
Default is that driver detects SMMUs but no translation
Currently it is derived from smmu resource size. If the resource size
is wrongly specified (e.g. too large) this leads to a miscalculation
and can cause undefined behaviour when context bank registers are
modified.
Signed-off-by: Andreas Herrmann andreas.herrm...@calxeda.com
---
This should ensure that arm-smmu is initialized before other drivers
start handling devices that propably need smmu support.
Also remove module_exit function as we most likely never want to
unload this driver.
Signed-off-by: Andreas Herrmann andreas.herrm...@calxeda.com
---
Hi,
Following patches fix misc issues, that I've seen when using arm-smmu
driver with MMU-400.
I also suggest to add an option to automatically create domains for
each master of detected SMMUs. (Similar to what is available for other
iommu driver(s).)
Comments welcome.
Regards,
Andreas
PS:
On Tue, Sep 24, 2013 at 04:14:30PM +0100, Andreas Herrmann wrote:
iommu/arm-smmu: Remove bogus semicolon from if conditions
Those prevented proper registration of arm_smmu_ops.
Signed-off-by: Andreas Herrmann andreas.herrm...@calxeda.com
---
drivers/iommu/arm-smmu.c |4 ++--
1 file
iommu/arm-smmu: Remove bogus semicolon from if conditions
Those prevented proper registration of arm_smmu_ops.
Signed-off-by: Andreas Herrmann andreas.herrm...@calxeda.com
---
drivers/iommu/arm-smmu.c |4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
Will,
It seems that I've
On Tue, Sep 24, 2013 at 04:06:54PM +0100, Andreas Herrmann wrote:
Hi,
Hi Andreas,
Following patches fix misc issues, that I've seen when using arm-smmu
driver with MMU-400.
Thanks for the patches! I'll respond to each one in turn.
Will
___
iommu
On Tue, Sep 24, 2013 at 04:06:56PM +0100, Andreas Herrmann wrote:
Currently it is derived from smmu resource size. If the resource size
is wrongly specified (e.g. too large) this leads to a miscalculation
and can cause undefined behaviour when context bank registers are
modified.
On Tue, Sep 24, 2013 at 04:06:58PM +0100, Andreas Herrmann wrote:
With the right (or wrong;-) definition of v1 SMMU node in DTB it is
possible to trigger a division by zero in arm_smmu_init_domain_context
(if number of context irqs is 0):
if (smmu-version == 1) {
On Tue, Sep 24, 2013 at 04:06:57PM +0100, Andreas Herrmann wrote:
... otherwise it is impossible for the low level iommu driver to
figure out which pte flags should be used.
In __map_sg_chunk we can derive the flags from dma_data_direction.
In __iommu_create_mapping we should treat the
On Tue, Sep 24, 2013 at 04:07:01PM +0100, Andreas Herrmann wrote:
Signed-off-by: Andreas Herrmann andreas.herrm...@calxeda.com
---
drivers/iommu/arm-smmu.c |9 +
1 file changed, 9 insertions(+)
diff --git a/drivers/iommu/arm-smmu.c b/drivers/iommu/arm-smmu.c
index
On Tue, Sep 24, 2013 at 11:36:25AM -0400, Will Deacon wrote:
On Tue, Sep 24, 2013 at 04:06:57PM +0100, Andreas Herrmann wrote:
... otherwise it is impossible for the low level iommu driver to
figure out which pte flags should be used.
In __map_sg_chunk we can derive the flags from
On Tue, Sep 24, 2013 at 11:34:57AM -0400, Will Deacon wrote:
On Tue, Sep 24, 2013 at 04:06:56PM +0100, Andreas Herrmann wrote:
Currently it is derived from smmu resource size. If the resource size
is wrongly specified (e.g. too large) this leads to a miscalculation
and can cause undefined
On Tue, Sep 24, 2013 at 11:42:52AM -0400, Will Deacon wrote:
On Tue, Sep 24, 2013 at 04:07:01PM +0100, Andreas Herrmann wrote:
Signed-off-by: Andreas Herrmann andreas.herrm...@calxeda.com
---
drivers/iommu/arm-smmu.c |9 +
1 file changed, 9 insertions(+)
diff --git
This patch set adds iommu_error class event to iommu trace. iommu_error class
event can be enabled to trigger when an iommu error occurs. This trace event
is intended to be called to report the error information. Trace information
includes driver name, device name, iova, and flags.
Change iommu driver call io_page_fault trace event. This iommu_error class
event can be enabled to trigger when an iommu error occurs. Trace information
includes driver name, device name, iova, and flags.
Testing:
Added trace calls to iommu_prepare_identity_map() for testing some of the
iommu_error class event can be enabled to trigger when an iommu
error occurs. This trace event is intended to be called to report the
error information. Trace information includes driver name, device name,
iova, and flags.
iommu_error:io_page_fault
Signed-off-by: Shuah Khan shuah...@samsung.com
On Thu, Sep 19, 2013 at 12:59:17PM +0530, Bharat Bhushan wrote:
This patch adds interface to get following information
- Number of MSI regions (which is number of MSI banks for powerpc).
- Get the region address range: Physical page which have the
address/addresses used for generating
28 matches
Mail list logo