10:19 <cyndis> digetx: fixed the iommu issue and needing to disable firewall, haven't seen the firewall failures so far though

11:44 <cyndis> here's my latest status https://github.com/cyndis/linux/commits/work/tegradrm-v7-staging

11:44 <cyndis> I can't disable the IOMMU, if I disable CONFIG_TEGRA_IOMMU_SMMU for some reason init crashes immediately and the kernel panics

11:45 <cyndis> (even without my patches)

14:41 <digetx_> cyndis: thank you, the staging uapi works now using the v7, but new uapi still has the fw errors https://dpaste.com/48EM4RPW6.txt (T20), the "mapped addresses" are wrong (m->iova == m->iova_end)

14:43 <cyndis> is this with smmu enabled or disabled?

14:43 <cyndis> or both

14:43 <cyndis> I guess bo->size must be zero

14:48 <digetx> T20 has GART, but it's not supported by upstream yet (inactive), so no IOMMU for T20 using yours v7

14:50 <cyndis> yep, I guess no-iommu path doesn't set bo->size, and I had only tested with IOMMU path so far, and on newer chips where I tested without IOMMU the firewall is otherwise disabled

14:52 <cyndis> digetx: can you test http://cyndis.org/~cyndis/size_diff.txt?

14:57 <digetx> cyndis: it works, thank you

14:58 <cyndis> great, thanks for testing

14:58 <digetx> I will test opentegra next and then start to review

14:58 <cyndis> thanks

15:03 <digetx> cyndis: the init crash is not good, please try to enable kasan in kernel config

15:47 <cyndis> of course, with kasan enabled, the bug disappears.. :)

16:01 <p2-mate> heisenbug :)