From: Frank Wunderlich <frank-w@public-files.de>
To: Yong Wu <yong.wu@mediatek.com>
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, yong.wu@mediatek.com,
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: Aw: [PATCH v6 00/11] Clean up "mediatek,larb"
Date: Wed, 14 Jul 2021 10:59:18 +0200 [thread overview]
Message-ID: <trinity-7d9ebdc9-4849-4d93-bfb5-429dcb4ee449-1626253158870@3c-app-gmx-bs01> (raw)
In-Reply-To: <20210714025626.5528-1-yong.wu@mediatek.com>
Hi,
sorry this (or the 2 depency-series) cause a NULL Pointer deref in iommu_group_remove_device on mt7623/bpi-r2
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
_______________________________________________
Linux-mediatek mailing list
Linux-mediatek@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-mediatek
next prev parent reply other threads:[~2021-07-14 8:59 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 ` 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 ` Frank Wunderlich [this message]
2021-07-14 11:18 ` Aw: " Yong Wu
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=trinity-7d9ebdc9-4849-4d93-bfb5-429dcb4ee449-1626253158870@3c-app-gmx-bs01 \
--to=frank-w@public-files.de \
--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=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=yong.wu@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).