From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-16.5 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI, MENTIONS_GIT_HOSTING,NICE_REPLY_A,SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS, UNPARSEABLE_RELAY,USER_AGENT_SANE_1 autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id AB07AC433DF for ; Fri, 14 Aug 2020 16:33:39 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 71C582074D for ; Fri, 14 Aug 2020 16:33:39 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726268AbgHNQdi (ORCPT ); Fri, 14 Aug 2020 12:33:38 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:56994 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726196AbgHNQdi (ORCPT ); Fri, 14 Aug 2020 12:33:38 -0400 Received: from bhuna.collabora.co.uk (bhuna.collabora.co.uk [IPv6:2a00:1098:0:82:1000:25:2eeb:e3e3]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 82B85C061384; Fri, 14 Aug 2020 09:33:37 -0700 (PDT) Received: from [127.0.0.1] (localhost [127.0.0.1]) (Authenticated sender: gtucker) with ESMTPSA id 8FCE229A26B Subject: Re: [PATCH 06/13] ARM: dts: omap4-panda-common: Add CMA pools and enable IPU & DSP To: Suman Anna , Tony Lindgren Cc: Tero Kristo , devicetree@vger.kernel.org, linux-omap@vger.kernel.org, linux-arm-kernel@lists.infradead.org, kernelci-results@groups.io References: <20200709231954.1973-1-s-anna@ti.com> <20200709231954.1973-7-s-anna@ti.com> <3e44fc46-07ac-6103-5c4d-8c7389453b87@collabora.com> From: Guillaume Tucker Message-ID: <94dbd2ee-bbac-2374-93bc-15ed67ba0e01@collabora.com> Date: Fri, 14 Aug 2020 17:33:31 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.11.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit Sender: devicetree-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: devicetree@vger.kernel.org On 14/08/2020 16:26, Suman Anna wrote: > On 8/14/20 9:32 AM, Guillaume Tucker wrote: >> On 12/08/2020 06:31, Guillaume Tucker wrote: >>> On 10/07/2020 00:19, Suman Anna wrote: >>>> The CMA reserved memory nodes have been added for the IPU and DSP >>>> remoteproc devices on all the OMAP4-based Panda boards. These nodes >>>> are assigned to the respective rproc device nodes, and both the >>>> IPU and DSP remote processors are enabled for all these boards. >>>> >>>> The current CMA pools and sizes are defined statically for each device. >>>> The starting addresses are fixed to meet current dependencies on the >>>> remote processor firmwares, and will go away when the remote-side >>>> code has been improved to gather this information runtime during >>>> its initialization. >>>> >>>> An associated pair of the rproc node and its CMA node can be disabled >>>> later on if there is no use-case defined to use that remote processor. >>>> >>>> Signed-off-by: Suman Anna >>>> --- >>>> arch/arm/boot/dts/omap4-panda-common.dtsi | 30 +++++++++++++++++++++++ >>>> 1 file changed, 30 insertions(+) >>>> >>>> diff --git a/arch/arm/boot/dts/omap4-panda-common.dtsi b/arch/arm/boot/dts/omap4-panda-common.dtsi >>>> index 55ea8b6189af..ef79028fc95f 100644 >>>> --- a/arch/arm/boot/dts/omap4-panda-common.dtsi >>>> +++ b/arch/arm/boot/dts/omap4-panda-common.dtsi >>>> @@ -12,6 +12,26 @@ memory@80000000 { >>>> reg = <0x80000000 0x40000000>; /* 1 GB */ >>>> }; >>>> >>>> + reserved-memory { >>>> + #address-cells = <1>; >>>> + #size-cells = <1>; >>>> + ranges; >>>> + >>>> + dsp_memory_region: dsp-memory@98000000 { >>>> + compatible = "shared-dma-pool"; >>>> + reg = <0x98000000 0x800000>; >>>> + reusable; >>>> + status = "okay"; >>>> + }; >>>> + >>>> + ipu_memory_region: ipu-memory@98800000 { >>>> + compatible = "shared-dma-pool"; >>>> + reg = <0x98800000 0x7000000>; >>>> + reusable; >>>> + status = "okay"; >>>> + }; >>>> + }; >>>> + >>>> chosen { >>>> stdout-path = &uart3; >>>> }; >>>> @@ -571,3 +591,13 @@ hdmi_out: endpoint { >>>> }; >>>> }; >>>> }; >>>> + >>>> +&dsp { >>>> + status = "okay"; >>>> + memory-region = <&dsp_memory_region>; >>>> +}; >>>> + >>>> +&ipu { >>>> + status = "okay"; >>>> + memory-region = <&ipu_memory_region>; >>>> +}; >>>> >>> >>> This appears to be causing some BUG alert messages: >>> >>> BUG: Bad page state in process swapper/0 pfn:9c801 >>> >>> as reported on kernelci.org: >>> >>> https://kernelci.org/test/case/id/5f326c6661360154c452c1c9/ >>> >>> I've run a bisection and it landed on this commit. If you fix it >>> with another patch, please add: >>> >>> Reported-by: "kernelci.org bot" >> >> >> This was bisected again automatically on mainline, see the report >> below. Is anyone available to take a look, or could the patch be >> reverted? > > Thanks Guillaume for the report. I will take a look at this today. It is strange > that the bisect is pointing to this commit as reserving a CMA pool and assigning > it to a device should be fairly normal usage. Is the issue seen only on OMAP4 > Pandaboard and not any of the other OMAP5 uEVM or DRA7xx/AM57xx EVMS? Thanks for taking a look. In the extended OMAP family, aside from the Panda only the BeagleBone Black and BeagleBoard xM are being tested on kernelci.org and they don't show this problem: https://kernelci.org/soc/omap2/job/next/kernel/next-20200814/plan/baseline/ Thanks, Guillaume >> On 14/08/2020 15:22, KernelCI bot wrote: >>> * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * >>> * This automated bisection report was sent to you on the basis * >>> * that you may be involved with the breaking commit it has * >>> * found. No manual investigation has been done to verify it, * >>> * and the root cause of the problem may be somewhere else. * >>> * * >>> * If you do send a fix, please include this trailer: * >>> * Reported-by: "kernelci.org bot" * >>> * * >>> * Hope this helps! * >>> * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * >>> >>> mainline/master bisection: baseline.dmesg.alert on panda >>> >>> Summary: >>> Start: a1d21081a60d Merge git://git.kernel.org/pub/scm/linux/kernel/git/netdev/net >>> Plain log: https://storage.kernelci.org/mainline/master/v5.8-13249-ga1d21081a60d/arm/multi_v7_defconfig+CONFIG_SMP=n/gcc-8/lab-collabora/baseline-omap4-panda.txt >>> HTML log: https://storage.kernelci.org/mainline/master/v5.8-13249-ga1d21081a60d/arm/multi_v7_defconfig+CONFIG_SMP=n/gcc-8/lab-collabora/baseline-omap4-panda.html >>> Result: b4778e787fe9 ARM: dts: omap4-panda-common: Add CMA pools and enable IPU & DSP >>> >>> Checks: >>> revert: PASS >>> verify: PASS >>> >>> Parameters: >>> Tree: mainline >>> URL: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git >>> Branch: master >>> Target: panda >>> CPU arch: arm >>> Lab: lab-collabora >>> Compiler: gcc-8 >>> Config: multi_v7_defconfig+CONFIG_SMP=n >>> Test case: baseline.dmesg.alert >>> >>> Breaking commit found: >>> >>> ------------------------------------------------------------------------------- >>> commit b4778e787fe9e82dcbff8150ebfbe6fea0b6c4e1 >>> Author: Suman Anna >>> Date: Thu Jul 9 18:19:47 2020 -0500 >>> >>> ARM: dts: omap4-panda-common: Add CMA pools and enable IPU & DSP >>> >>> The CMA reserved memory nodes have been added for the IPU and DSP >>> remoteproc devices on all the OMAP4-based Panda boards. These nodes >>> are assigned to the respective rproc device nodes, and both the >>> IPU and DSP remote processors are enabled for all these boards. >>> >>> The current CMA pools and sizes are defined statically for each device. >>> The starting addresses are fixed to meet current dependencies on the >>> remote processor firmwares, and will go away when the remote-side >>> code has been improved to gather this information runtime during >>> its initialization. >>> >>> An associated pair of the rproc node and its CMA node can be disabled >>> later on if there is no use-case defined to use that remote processor. >>> >>> Signed-off-by: Suman Anna >>> Signed-off-by: Tony Lindgren >>> >>> diff --git a/arch/arm/boot/dts/omap4-panda-common.dtsi b/arch/arm/boot/dts/omap4-panda-common.dtsi >>> index 55ea8b6189af..ef79028fc95f 100644 >>> --- a/arch/arm/boot/dts/omap4-panda-common.dtsi >>> +++ b/arch/arm/boot/dts/omap4-panda-common.dtsi >>> @@ -12,6 +12,26 @@ >>> reg = <0x80000000 0x40000000>; /* 1 GB */ >>> }; >>> >>> + reserved-memory { >>> + #address-cells = <1>; >>> + #size-cells = <1>; >>> + ranges; >>> + >>> + dsp_memory_region: dsp-memory@98000000 { >>> + compatible = "shared-dma-pool"; >>> + reg = <0x98000000 0x800000>; >>> + reusable; >>> + status = "okay"; >>> + }; >>> + >>> + ipu_memory_region: ipu-memory@98800000 { >>> + compatible = "shared-dma-pool"; >>> + reg = <0x98800000 0x7000000>; >>> + reusable; >>> + status = "okay"; >>> + }; >>> + }; >>> + >>> chosen { >>> stdout-path = &uart3; >>> }; >>> @@ -571,3 +591,13 @@ >>> }; >>> }; >>> }; >>> + >>> +&dsp { >>> + status = "okay"; >>> + memory-region = <&dsp_memory_region>; >>> +}; >>> + >>> +&ipu { >>> + status = "okay"; >>> + memory-region = <&ipu_memory_region>; >>> +}; >>> ------------------------------------------------------------------------------- >>> >>> >>> Git bisection log: >>> >>> ------------------------------------------------------------------------------- >>> git bisect start >>> # good: [e4cbce4d131753eca271d9d67f58c6377f27ad21] Merge tag 'sched-core-2020-08-03' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip >>> git bisect good e4cbce4d131753eca271d9d67f58c6377f27ad21 >>> # bad: [a1d21081a60dfb7fddf4a38b66d9cef603b317a9] Merge git://git.kernel.org/pub/scm/linux/kernel/git/netdev/net >>> git bisect bad a1d21081a60dfb7fddf4a38b66d9cef603b317a9 >>> # bad: [47ec5303d73ea344e84f46660fff693c57641386] Merge git://git.kernel.org/pub/scm/linux/kernel/git/netdev/net-next >>> git bisect bad 47ec5303d73ea344e84f46660fff693c57641386 >>> # bad: [e4a7b2dc35d9582c253cf5e6d6c3605aabc7284d] Merge tag 'leds-5.9-rc1' of git://git.kernel.org/pub/scm/linux/kernel/git/pavel/linux-leds >>> git bisect bad e4a7b2dc35d9582c253cf5e6d6c3605aabc7284d >>> # bad: [74858abbb1032222f922487fd1a24513bbed80f9] Merge tag 'cap-checkpoint-restore-v5.9' of git://git.kernel.org/pub/scm/linux/kernel/git/brauner/linux >>> git bisect bad 74858abbb1032222f922487fd1a24513bbed80f9 >>> # bad: [2f3fbfdaf77f3ac417d0511fac221f76af79f6fc] Merge tag 'arm-dt-5.9' of git://git.kernel.org/pub/scm/linux/kernel/git/soc/soc >>> git bisect bad 2f3fbfdaf77f3ac417d0511fac221f76af79f6fc >>> # bad: [c6e2e454baef6080ef89c2b6488e708d5fa0f052] Merge tag 'qcom-arm64-for-5.9' of git://git.kernel.org/pub/scm/linux/kernel/git/qcom/linux into arm/dt >>> git bisect bad c6e2e454baef6080ef89c2b6488e708d5fa0f052 >>> # bad: [3502e079c6bcff95f5c34eecb5c1d9ad1379ae0d] Merge tag 'tegra-for-5.9-arm-dt' of git://git.kernel.org/pub/scm/linux/kernel/git/tegra/linux into arm/dt >>> git bisect bad 3502e079c6bcff95f5c34eecb5c1d9ad1379ae0d >>> # bad: [39a85f6d91a1a827985ce44a346a99f68167d0ee] Merge tag 'v5.8-next-dts64' of git://git.kernel.org/pub/scm/linux/kernel/git/matthias.bgg/linux into arm/dt >>> git bisect bad 39a85f6d91a1a827985ce44a346a99f68167d0ee >>> # good: [dfe2a4cf8e2f4c1f53877aa6cb38eda102a14681] Merge tag 'uniphier-dt64-v5.9' of git://git.kernel.org/pub/scm/linux/kernel/git/masahiroy/linux-uniphier into arm/dt >>> git bisect good dfe2a4cf8e2f4c1f53877aa6cb38eda102a14681 >>> # bad: [75f66813e081d2bd718d931ee50334c12a9e4492] Replace HTTP links with HTTPS ones: OMAP DEVICE TREE SUPPORT >>> git bisect bad 75f66813e081d2bd718d931ee50334c12a9e4492 >>> # good: [9ae60ac13fc847d7175587290a1a9aa2aac091b0] ARM: dts: omap4: Update the DSP node >>> git bisect good 9ae60ac13fc847d7175587290a1a9aa2aac091b0 >>> # bad: [3026ce47498dfdc92966d8d66f10afabf7190c46] ARM: dts: omap5: Add DSP and IPU nodes >>> git bisect bad 3026ce47498dfdc92966d8d66f10afabf7190c46 >>> # good: [691eb1805fcfc1a2ede06aec6a4d85d312961146] ARM: dts: omap4: Add aliases for rproc nodes >>> git bisect good 691eb1805fcfc1a2ede06aec6a4d85d312961146 >>> # bad: [7f7d771c00bf65d18a3e30e983b4061a418efbf4] ARM: dts: omap4-panda-common:: Add system timers to DSP and IPU >>> git bisect bad 7f7d771c00bf65d18a3e30e983b4061a418efbf4 >>> # bad: [b4778e787fe9e82dcbff8150ebfbe6fea0b6c4e1] ARM: dts: omap4-panda-common: Add CMA pools and enable IPU & DSP >>> git bisect bad b4778e787fe9e82dcbff8150ebfbe6fea0b6c4e1 >>> # first bad commit: [b4778e787fe9e82dcbff8150ebfbe6fea0b6c4e1] ARM: dts: omap4-panda-common: Add CMA pools and enable IPU & DSP >>> ------------------------------------------------------------------------------- >> >> > From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-16.5 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH, MAILING_LIST_MULTI,MENTIONS_GIT_HOSTING,NICE_REPLY_A,SIGNED_OFF_BY, SPF_HELO_NONE,SPF_PASS,UNPARSEABLE_RELAY,URIBL_BLOCKED,USER_AGENT_SANE_1 autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id C8680C433DF for ; Fri, 14 Aug 2020 16:35:09 +0000 (UTC) Received: from merlin.infradead.org (merlin.infradead.org [205.233.59.134]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 882D82074D for ; Fri, 14 Aug 2020 16:35:09 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="GrfFmCAL" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 882D82074D Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=collabora.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=merlin.20170209; h=Sender:Content-Transfer-Encoding: Content-Type:Cc:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:Date:Message-ID:From: References:To:Subject:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=tNyVBBUsKX0cABkcFkFFzjJV/KlkKuLrfbTToQQYgPA=; b=GrfFmCALR/kp3UdWFVfhDWrkT Z1xhF8CGeOcis6knVOnKNYrofw0PeVirmJYCIeODwXaz8UEmj8sTeW3O+KR93ieBn7/9agD7/YEdK LvBdX29GUuaXJHU9gj+z8R7H7eMCAxPTnVkGLLkrEFKA0YS0pN5z1yFABVRWcmW/CHGXjLD5dMRHS eNql+01ZO8oyb4gEDU8dHG3YjyRDm4pGNq08qYZ0zhVEl3HJLogav41dcoyoUTlBIXmuNg43Pro2G BJU9ditdtfRroNRi0PqBrSj4mJLLzMbG558us6ahDAiA2xhe3MvIGzBTqZiEQy7+dDJ2zhrkY3wLA VdH4gjuTQ==; Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1k6ceD-0005RF-Ms; Fri, 14 Aug 2020 16:33:41 +0000 Received: from bhuna.collabora.co.uk ([46.235.227.227]) by merlin.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1k6ceA-0005QS-7v for linux-arm-kernel@lists.infradead.org; Fri, 14 Aug 2020 16:33:39 +0000 Received: from [127.0.0.1] (localhost [127.0.0.1]) (Authenticated sender: gtucker) with ESMTPSA id 8FCE229A26B Subject: Re: [PATCH 06/13] ARM: dts: omap4-panda-common: Add CMA pools and enable IPU & DSP To: Suman Anna , Tony Lindgren References: <20200709231954.1973-1-s-anna@ti.com> <20200709231954.1973-7-s-anna@ti.com> <3e44fc46-07ac-6103-5c4d-8c7389453b87@collabora.com> From: Guillaume Tucker Message-ID: <94dbd2ee-bbac-2374-93bc-15ed67ba0e01@collabora.com> Date: Fri, 14 Aug 2020 17:33:31 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.11.0 MIME-Version: 1.0 In-Reply-To: Content-Language: en-US X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20200814_123338_559602_4F866D96 X-CRM114-Status: GOOD ( 22.60 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Tero Kristo , devicetree@vger.kernel.org, linux-omap@vger.kernel.org, kernelci-results@groups.io, linux-arm-kernel@lists.infradead.org Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 14/08/2020 16:26, Suman Anna wrote: > On 8/14/20 9:32 AM, Guillaume Tucker wrote: >> On 12/08/2020 06:31, Guillaume Tucker wrote: >>> On 10/07/2020 00:19, Suman Anna wrote: >>>> The CMA reserved memory nodes have been added for the IPU and DSP >>>> remoteproc devices on all the OMAP4-based Panda boards. These nodes >>>> are assigned to the respective rproc device nodes, and both the >>>> IPU and DSP remote processors are enabled for all these boards. >>>> >>>> The current CMA pools and sizes are defined statically for each device. >>>> The starting addresses are fixed to meet current dependencies on the >>>> remote processor firmwares, and will go away when the remote-side >>>> code has been improved to gather this information runtime during >>>> its initialization. >>>> >>>> An associated pair of the rproc node and its CMA node can be disabled >>>> later on if there is no use-case defined to use that remote processor. >>>> >>>> Signed-off-by: Suman Anna >>>> --- >>>> arch/arm/boot/dts/omap4-panda-common.dtsi | 30 +++++++++++++++++++++++ >>>> 1 file changed, 30 insertions(+) >>>> >>>> diff --git a/arch/arm/boot/dts/omap4-panda-common.dtsi b/arch/arm/boot/dts/omap4-panda-common.dtsi >>>> index 55ea8b6189af..ef79028fc95f 100644 >>>> --- a/arch/arm/boot/dts/omap4-panda-common.dtsi >>>> +++ b/arch/arm/boot/dts/omap4-panda-common.dtsi >>>> @@ -12,6 +12,26 @@ memory@80000000 { >>>> reg = <0x80000000 0x40000000>; /* 1 GB */ >>>> }; >>>> >>>> + reserved-memory { >>>> + #address-cells = <1>; >>>> + #size-cells = <1>; >>>> + ranges; >>>> + >>>> + dsp_memory_region: dsp-memory@98000000 { >>>> + compatible = "shared-dma-pool"; >>>> + reg = <0x98000000 0x800000>; >>>> + reusable; >>>> + status = "okay"; >>>> + }; >>>> + >>>> + ipu_memory_region: ipu-memory@98800000 { >>>> + compatible = "shared-dma-pool"; >>>> + reg = <0x98800000 0x7000000>; >>>> + reusable; >>>> + status = "okay"; >>>> + }; >>>> + }; >>>> + >>>> chosen { >>>> stdout-path = &uart3; >>>> }; >>>> @@ -571,3 +591,13 @@ hdmi_out: endpoint { >>>> }; >>>> }; >>>> }; >>>> + >>>> +&dsp { >>>> + status = "okay"; >>>> + memory-region = <&dsp_memory_region>; >>>> +}; >>>> + >>>> +&ipu { >>>> + status = "okay"; >>>> + memory-region = <&ipu_memory_region>; >>>> +}; >>>> >>> >>> This appears to be causing some BUG alert messages: >>> >>> BUG: Bad page state in process swapper/0 pfn:9c801 >>> >>> as reported on kernelci.org: >>> >>> https://kernelci.org/test/case/id/5f326c6661360154c452c1c9/ >>> >>> I've run a bisection and it landed on this commit. If you fix it >>> with another patch, please add: >>> >>> Reported-by: "kernelci.org bot" >> >> >> This was bisected again automatically on mainline, see the report >> below. Is anyone available to take a look, or could the patch be >> reverted? > > Thanks Guillaume for the report. I will take a look at this today. It is strange > that the bisect is pointing to this commit as reserving a CMA pool and assigning > it to a device should be fairly normal usage. Is the issue seen only on OMAP4 > Pandaboard and not any of the other OMAP5 uEVM or DRA7xx/AM57xx EVMS? Thanks for taking a look. In the extended OMAP family, aside from the Panda only the BeagleBone Black and BeagleBoard xM are being tested on kernelci.org and they don't show this problem: https://kernelci.org/soc/omap2/job/next/kernel/next-20200814/plan/baseline/ Thanks, Guillaume >> On 14/08/2020 15:22, KernelCI bot wrote: >>> * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * >>> * This automated bisection report was sent to you on the basis * >>> * that you may be involved with the breaking commit it has * >>> * found. No manual investigation has been done to verify it, * >>> * and the root cause of the problem may be somewhere else. * >>> * * >>> * If you do send a fix, please include this trailer: * >>> * Reported-by: "kernelci.org bot" * >>> * * >>> * Hope this helps! * >>> * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * >>> >>> mainline/master bisection: baseline.dmesg.alert on panda >>> >>> Summary: >>> Start: a1d21081a60d Merge git://git.kernel.org/pub/scm/linux/kernel/git/netdev/net >>> Plain log: https://storage.kernelci.org/mainline/master/v5.8-13249-ga1d21081a60d/arm/multi_v7_defconfig+CONFIG_SMP=n/gcc-8/lab-collabora/baseline-omap4-panda.txt >>> HTML log: https://storage.kernelci.org/mainline/master/v5.8-13249-ga1d21081a60d/arm/multi_v7_defconfig+CONFIG_SMP=n/gcc-8/lab-collabora/baseline-omap4-panda.html >>> Result: b4778e787fe9 ARM: dts: omap4-panda-common: Add CMA pools and enable IPU & DSP >>> >>> Checks: >>> revert: PASS >>> verify: PASS >>> >>> Parameters: >>> Tree: mainline >>> URL: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git >>> Branch: master >>> Target: panda >>> CPU arch: arm >>> Lab: lab-collabora >>> Compiler: gcc-8 >>> Config: multi_v7_defconfig+CONFIG_SMP=n >>> Test case: baseline.dmesg.alert >>> >>> Breaking commit found: >>> >>> ------------------------------------------------------------------------------- >>> commit b4778e787fe9e82dcbff8150ebfbe6fea0b6c4e1 >>> Author: Suman Anna >>> Date: Thu Jul 9 18:19:47 2020 -0500 >>> >>> ARM: dts: omap4-panda-common: Add CMA pools and enable IPU & DSP >>> >>> The CMA reserved memory nodes have been added for the IPU and DSP >>> remoteproc devices on all the OMAP4-based Panda boards. These nodes >>> are assigned to the respective rproc device nodes, and both the >>> IPU and DSP remote processors are enabled for all these boards. >>> >>> The current CMA pools and sizes are defined statically for each device. >>> The starting addresses are fixed to meet current dependencies on the >>> remote processor firmwares, and will go away when the remote-side >>> code has been improved to gather this information runtime during >>> its initialization. >>> >>> An associated pair of the rproc node and its CMA node can be disabled >>> later on if there is no use-case defined to use that remote processor. >>> >>> Signed-off-by: Suman Anna >>> Signed-off-by: Tony Lindgren >>> >>> diff --git a/arch/arm/boot/dts/omap4-panda-common.dtsi b/arch/arm/boot/dts/omap4-panda-common.dtsi >>> index 55ea8b6189af..ef79028fc95f 100644 >>> --- a/arch/arm/boot/dts/omap4-panda-common.dtsi >>> +++ b/arch/arm/boot/dts/omap4-panda-common.dtsi >>> @@ -12,6 +12,26 @@ >>> reg = <0x80000000 0x40000000>; /* 1 GB */ >>> }; >>> >>> + reserved-memory { >>> + #address-cells = <1>; >>> + #size-cells = <1>; >>> + ranges; >>> + >>> + dsp_memory_region: dsp-memory@98000000 { >>> + compatible = "shared-dma-pool"; >>> + reg = <0x98000000 0x800000>; >>> + reusable; >>> + status = "okay"; >>> + }; >>> + >>> + ipu_memory_region: ipu-memory@98800000 { >>> + compatible = "shared-dma-pool"; >>> + reg = <0x98800000 0x7000000>; >>> + reusable; >>> + status = "okay"; >>> + }; >>> + }; >>> + >>> chosen { >>> stdout-path = &uart3; >>> }; >>> @@ -571,3 +591,13 @@ >>> }; >>> }; >>> }; >>> + >>> +&dsp { >>> + status = "okay"; >>> + memory-region = <&dsp_memory_region>; >>> +}; >>> + >>> +&ipu { >>> + status = "okay"; >>> + memory-region = <&ipu_memory_region>; >>> +}; >>> ------------------------------------------------------------------------------- >>> >>> >>> Git bisection log: >>> >>> ------------------------------------------------------------------------------- >>> git bisect start >>> # good: [e4cbce4d131753eca271d9d67f58c6377f27ad21] Merge tag 'sched-core-2020-08-03' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip >>> git bisect good e4cbce4d131753eca271d9d67f58c6377f27ad21 >>> # bad: [a1d21081a60dfb7fddf4a38b66d9cef603b317a9] Merge git://git.kernel.org/pub/scm/linux/kernel/git/netdev/net >>> git bisect bad a1d21081a60dfb7fddf4a38b66d9cef603b317a9 >>> # bad: [47ec5303d73ea344e84f46660fff693c57641386] Merge git://git.kernel.org/pub/scm/linux/kernel/git/netdev/net-next >>> git bisect bad 47ec5303d73ea344e84f46660fff693c57641386 >>> # bad: [e4a7b2dc35d9582c253cf5e6d6c3605aabc7284d] Merge tag 'leds-5.9-rc1' of git://git.kernel.org/pub/scm/linux/kernel/git/pavel/linux-leds >>> git bisect bad e4a7b2dc35d9582c253cf5e6d6c3605aabc7284d >>> # bad: [74858abbb1032222f922487fd1a24513bbed80f9] Merge tag 'cap-checkpoint-restore-v5.9' of git://git.kernel.org/pub/scm/linux/kernel/git/brauner/linux >>> git bisect bad 74858abbb1032222f922487fd1a24513bbed80f9 >>> # bad: [2f3fbfdaf77f3ac417d0511fac221f76af79f6fc] Merge tag 'arm-dt-5.9' of git://git.kernel.org/pub/scm/linux/kernel/git/soc/soc >>> git bisect bad 2f3fbfdaf77f3ac417d0511fac221f76af79f6fc >>> # bad: [c6e2e454baef6080ef89c2b6488e708d5fa0f052] Merge tag 'qcom-arm64-for-5.9' of git://git.kernel.org/pub/scm/linux/kernel/git/qcom/linux into arm/dt >>> git bisect bad c6e2e454baef6080ef89c2b6488e708d5fa0f052 >>> # bad: [3502e079c6bcff95f5c34eecb5c1d9ad1379ae0d] Merge tag 'tegra-for-5.9-arm-dt' of git://git.kernel.org/pub/scm/linux/kernel/git/tegra/linux into arm/dt >>> git bisect bad 3502e079c6bcff95f5c34eecb5c1d9ad1379ae0d >>> # bad: [39a85f6d91a1a827985ce44a346a99f68167d0ee] Merge tag 'v5.8-next-dts64' of git://git.kernel.org/pub/scm/linux/kernel/git/matthias.bgg/linux into arm/dt >>> git bisect bad 39a85f6d91a1a827985ce44a346a99f68167d0ee >>> # good: [dfe2a4cf8e2f4c1f53877aa6cb38eda102a14681] Merge tag 'uniphier-dt64-v5.9' of git://git.kernel.org/pub/scm/linux/kernel/git/masahiroy/linux-uniphier into arm/dt >>> git bisect good dfe2a4cf8e2f4c1f53877aa6cb38eda102a14681 >>> # bad: [75f66813e081d2bd718d931ee50334c12a9e4492] Replace HTTP links with HTTPS ones: OMAP DEVICE TREE SUPPORT >>> git bisect bad 75f66813e081d2bd718d931ee50334c12a9e4492 >>> # good: [9ae60ac13fc847d7175587290a1a9aa2aac091b0] ARM: dts: omap4: Update the DSP node >>> git bisect good 9ae60ac13fc847d7175587290a1a9aa2aac091b0 >>> # bad: [3026ce47498dfdc92966d8d66f10afabf7190c46] ARM: dts: omap5: Add DSP and IPU nodes >>> git bisect bad 3026ce47498dfdc92966d8d66f10afabf7190c46 >>> # good: [691eb1805fcfc1a2ede06aec6a4d85d312961146] ARM: dts: omap4: Add aliases for rproc nodes >>> git bisect good 691eb1805fcfc1a2ede06aec6a4d85d312961146 >>> # bad: [7f7d771c00bf65d18a3e30e983b4061a418efbf4] ARM: dts: omap4-panda-common:: Add system timers to DSP and IPU >>> git bisect bad 7f7d771c00bf65d18a3e30e983b4061a418efbf4 >>> # bad: [b4778e787fe9e82dcbff8150ebfbe6fea0b6c4e1] ARM: dts: omap4-panda-common: Add CMA pools and enable IPU & DSP >>> git bisect bad b4778e787fe9e82dcbff8150ebfbe6fea0b6c4e1 >>> # first bad commit: [b4778e787fe9e82dcbff8150ebfbe6fea0b6c4e1] ARM: dts: omap4-panda-common: Add CMA pools and enable IPU & DSP >>> ------------------------------------------------------------------------------- >> >> > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel