Thread (5 messages) flat view 5 messages, 2 authors, 2014-10-01
DORMANTno replies

[PATCH v4 2/2] iommu/arm-smmu: add support for iova_to_phys through ATS1PR

From: arnd@arndb.de (Arnd Bergmann)
Date: 2014-10-01 08:27:27
Also in: linux-iommu

On Tuesday 30 September 2014 18:28:13 Mitchel Humpherys wrote:
+       if (readl_poll_timeout_atomic(cb_base + ARM_SMMU_CB_ATSR, tmp,
+                               !(tmp & ATSR_ACTIVE), 50, 100)) {
This looks really bad.

You are doing up to 50 100us delays, each of which can be much longer,
so you can do up to 10ms total delay with interrupts disabled.

Don't do that.

	Arnd
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help