From: Yong Wu <yong.wu@mediatek.com>
To: Frank Wunderlich <frank-w@public-files.de>
Cc: Matthias Brugger <matthias.bgg@gmail.com>,
Joerg Roedel <joro@8bytes.org>, Rob Herring <robh+dt@kernel.org>,
Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>,
David Airlie <airlied@linux.ie>,
"Mauro Carvalho Chehab" <mchehab@kernel.org>,
Evan Green <evgreen@chromium.org>,
Robin Murphy <robin.murphy@arm.com>,
Tomasz Figa <tfiga@chromium.org>,
"Will Deacon" <will.deacon@arm.com>,
<linux-mediatek@lists.infradead.org>,
<srv_heupstream@mediatek.com>, <devicetree@vger.kernel.org>,
<linux-kernel@vger.kernel.org>,
<linux-arm-kernel@lists.infradead.org>,
<iommu@lists.linux-foundation.org>, <youlin.pei@mediatek.com>,
Nicolas Boichat <drinkcat@chromium.org>,
Matthias Kaehlcke <mka@chromium.org>, <anan.sun@mediatek.com>,
<ming-fan.chen@mediatek.com>, <yi.kuo@mediatek.com>,
<acourbot@chromium.org>, <linux-media@vger.kernel.org>,
<dri-devel@lists.freedesktop.org>,
Daniel Vetter <daniel@ffwll.ch>,
Chun-Kuang Hu <chunkuang.hu@kernel.org>,
Philipp Zabel <p.zabel@pengutronix.de>,
Xia Jiang <xia.jiang@mediatek.com>,
Tiffany Lin <tiffany.lin@mediatek.com>,
Dafna Hirschfeld <dafna.hirschfeld@collabora.com>,
Hsin-Yi Wang <hsinyi@chromium.org>,
"Eizan Miyamoto" <eizan@chromium.org>,
<anthony.huang@mediatek.com>
Subject: Re: Aw: [PATCH v6 00/11] Clean up "mediatek,larb"
Date: Wed, 14 Jul 2021 19:18:16 +0800 [thread overview]
Message-ID: <1626261496.14352.16.camel@mhfsdcap03> (raw)
In-Reply-To: <trinity-7d9ebdc9-4849-4d93-bfb5-429dcb4ee449-1626253158870@3c-app-gmx-bs01>
On Wed, 2021-07-14 at 10:59 +0200, Frank Wunderlich wrote:
> Hi,
>
> sorry this (or the 2 depency-series) cause a NULL Pointer deref in iommu_group_remove_device on mt7623/bpi-r2
Hi Frank,
Thanks for your report. mt7623 use mtk_iommu_v1.c.
I will try to reproduce this locally.
>
> i wonder why on bootup a cleanup is run, but have no hint about this.
>
> since "dts: mtk-mdp: remove mediatek, vpu property from primary MDP device" all is good, i guess problem comes up while removing larb with DT
>
> this is backtrace
>
> [ 6.274465] PC is at iommu_group_remove_device+0x28/0x148
> [ 6.279877] LR is at iommu_release_device+0x4c/0x70
>
> [ 6.674347] Backtrace:
> [ 6.676797] [<c0c9c37c>] (iommu_group_remove_device) from [<c06bf028>] (iomm)
> [ 6.686221] r7:00000000 r6:c06bf04c r5:c0d7a1ac r4:c21fc010
> [ 6.691883] [<c06befdc>] (iommu_release_device) from [<c06bf064>] (remove_io)
> [ 6.700689] r5:00000000 r4:00000000
> [ 6.704265] [<c06bf04c>] (remove_iommu_group) from [<c0733434>] (bus_for_eac)
> [ 6.712725] [<c07333ac>] (bus_for_each_dev) from [<c06bf658>] (bus_set_iommu)
> [ 6.720753] r6:c331f440 r5:c1406f58 r4:ffffffea
> [ 6.725370] [<c06bf5a0>] (bus_set_iommu) from [<c06c1e88>] (mtk_iommu_probe+)
> [ 6.733484] r7:c32db0b8 r6:c21f9c00 r5:c331f1c0 r4:00000000
> [ 6.739145] [<c06c1bfc>] (mtk_iommu_probe) from [<c0738c14>] (platform_probe)
> [ 6.747176] r10:c21f9c10 r9:c2496f54 r8:c14623b8 r7:c14623b8 r6:c1405b90 r50
> [ 6.755012] r4:00000000
> [ 6.757544] [<c0738ba8>] (platform_probe) from [<c0735968>] (really_probe.pa)
> [ 6.766006] r7:c14623b8 r6:c1405b90 r5:00000000 r4:c21f9c10
> [ 6.771667] [<c07358a0>] (really_probe.part.0) from [<c0735cec>] (really_pro)
> [ 6.779866] r7:c21f9c10 r6:c2549e74 r5:c1405b90 r4:c21f9c10
> [ 6.785527] [<c0735ca4>] (really_probe) from [<c0735de0>] (__driver_probe_de)
> [ 6.793984] r5:c1405b90 r4:c21f9c10
> [ 6.797560] [<c0735d30>] (__driver_probe_device) from [<c0735fa0>] (driver_p)
> [ 6.806543] r9:c2496f54 r8:00000008 r7:c21f9c10 r6:c2549e74 r5:c14c6ec8 r4:4
> [ 6.814291] [<c0735f5c>] (driver_probe_device) from [<c0736410>] (__device_a)
> [ 6.823448] r9:c2496f54 r8:00000000 r7:c21f9c10 r6:c2549e74 r5:c1405b90 r4:1
> [ 6.831196] [<c073635c>] (__device_attach_driver) from [<c0733540>] (bus_for)
> [ 6.840007] r7:c14623b8 r6:c073635c r5:c2549e74 r4:00000000
> [ 6.845669] [<c07334ac>] (bus_for_each_drv) from [<c07357e8>] (__device_atta)
> [ 6.854044] r6:00000001 r5:c21f9c54 r4:c21f9c10
> [ 6.858662] [<c07356e4>] (__device_attach) from [<c073662c>] (device_initial)
> [ 6.867207] r6:c21f9c10 r5:c1406f58 r4:c1406ca0
> [ 6.871825] [<c0736610>] (device_initial_probe) from [<c07346dc>] (bus_probe)
> [ 6.880454] [<c0734648>] (bus_probe_device) from [<c0734cc8>] (deferred_prob)
>
>
> bisect shows this commit as breaking:
>
> Author: Yong Wu <yong.wu@mediatek.com>
> Date: Wed Jul 14 10:56:17 2021 +0800
>
> iommu/mediatek: Add probe_defer for smi-larb
>
> Prepare for adding device_link.
>
> regards Frank
next prev parent reply other threads:[~2021-07-14 11:18 UTC|newest]
Thread overview: 28+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-07-14 2:56 [PATCH v6 00/11] Clean up "mediatek,larb" Yong Wu
2021-07-14 2:56 ` [PATCH v6 01/11] dt-binding: mediatek: Get rid of mediatek,larb for multimedia HW Yong Wu
2021-07-14 8:13 ` [PATCH v6 01/11] dt-binding: mediatek: Get rid of mediatek, larb " Dafna Hirschfeld
2021-07-14 8:49 ` Dafna Hirschfeld
2021-07-14 2:56 ` [PATCH v6 02/11] iommu/mediatek: Add probe_defer for smi-larb Yong Wu
2021-07-14 2:56 ` [PATCH v6 03/11] iommu/mediatek: Add device_link between the consumer and the larb devices Yong Wu
2021-07-14 8:26 ` Dafna Hirschfeld
2021-07-14 11:16 ` Yong Wu
2021-07-14 2:56 ` [PATCH v6 04/11] media: mtk-jpeg: Get rid of mtk_smi_larb_get/put Yong Wu
2021-07-14 8:28 ` Dafna Hirschfeld
2021-07-14 2:56 ` [PATCH v6 05/11] media: mtk-mdp: " Yong Wu
2021-07-14 8:29 ` Dafna Hirschfeld
2021-07-14 2:56 ` [PATCH v6 06/11] drm/mediatek: Add pm runtime support for ovl and rdma Yong Wu
2021-07-14 8:44 ` Dafna Hirschfeld
2021-07-14 11:17 ` Yong Wu
2021-07-14 2:56 ` [PATCH v6 07/11] drm/mediatek: Get rid of mtk_smi_larb_get/put Yong Wu
2021-07-14 8:30 ` Dafna Hirschfeld
2021-07-14 2:56 ` [PATCH v6 08/11] media: mtk-vcodec: " Yong Wu
2021-07-14 8:31 ` Dafna Hirschfeld
2021-07-14 2:56 ` [PATCH v6 09/11] memory: mtk-smi: " Yong Wu
2021-07-14 8:32 ` Dafna Hirschfeld
2021-07-14 2:56 ` [PATCH v6 10/11] arm: dts: mediatek: Get rid of mediatek,larb for MM nodes Yong Wu
2021-07-14 2:56 ` [PATCH v6 11/11] arm64: " Yong Wu
2021-07-14 8:56 ` [PATCH v6 00/11] Clean up "mediatek,larb" Dafna Hirschfeld
2021-07-14 11:18 ` Yong Wu
2021-07-14 8:59 ` Aw: " Frank Wunderlich
2021-07-14 11:18 ` Yong Wu [this message]
2021-07-14 11:23 ` Aw: " Frank Wunderlich
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1626261496.14352.16.camel@mhfsdcap03 \
--to=yong.wu@mediatek.com \
--cc=acourbot@chromium.org \
--cc=airlied@linux.ie \
--cc=anan.sun@mediatek.com \
--cc=anthony.huang@mediatek.com \
--cc=chunkuang.hu@kernel.org \
--cc=dafna.hirschfeld@collabora.com \
--cc=daniel@ffwll.ch \
--cc=devicetree@vger.kernel.org \
--cc=dri-devel@lists.freedesktop.org \
--cc=drinkcat@chromium.org \
--cc=eizan@chromium.org \
--cc=evgreen@chromium.org \
--cc=frank-w@public-files.de \
--cc=hsinyi@chromium.org \
--cc=iommu@lists.linux-foundation.org \
--cc=joro@8bytes.org \
--cc=krzysztof.kozlowski@canonical.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-media@vger.kernel.org \
--cc=linux-mediatek@lists.infradead.org \
--cc=matthias.bgg@gmail.com \
--cc=mchehab@kernel.org \
--cc=ming-fan.chen@mediatek.com \
--cc=mka@chromium.org \
--cc=p.zabel@pengutronix.de \
--cc=robh+dt@kernel.org \
--cc=robin.murphy@arm.com \
--cc=srv_heupstream@mediatek.com \
--cc=tfiga@chromium.org \
--cc=tiffany.lin@mediatek.com \
--cc=will.deacon@arm.com \
--cc=xia.jiang@mediatek.com \
--cc=yi.kuo@mediatek.com \
--cc=youlin.pei@mediatek.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).