From: Jeffrey Hugo <jeffrey.l.hugo@gmail.com> To: Sai Prakash Ranjan <saiprakash.ranjan@codeaurora.org> Cc: Mark Rutland <mark.rutland@arm.com>, Rajendra Nayak <rnayak@codeaurora.org>, Mathieu Poirier <mathieu.poirier@linaro.org>, Suzuki K Poulose <suzuki.poulose@arm.com>, Alexander Shishkin <alexander.shishkin@linux.intel.com>, MSM <linux-arm-msm@vger.kernel.org>, Marc Gonzalez <marc.w.gonzalez@free.fr>, lkml <linux-kernel@vger.kernel.org>, Bjorn Andersson <bjorn.andersson@linaro.org>, David Brown <david.brown@linaro.org>, Andy Gross <agross@kernel.org>, Sibi Sankar <sibis@codeaurora.org>, Leo Yan <leo.yan@linaro.org>, linux-arm-kernel@lists.infradead.org Subject: Re: [PATCHv9 2/3] arm64: dts: qcom: msm8998: Add Coresight support Date: Tue, 1 Oct 2019 11:14:03 -0600 [thread overview] Message-ID: <CAOCk7NohO67qeYCnrjy4P0KN9nLUiamphHRvj-bFP++K7khPOw@mail.gmail.com> (raw) In-Reply-To: <16212a577339204e901cf4eefa5e82f1@codeaurora.org> On Tue, Oct 1, 2019 at 11:04 AM Sai Prakash Ranjan <saiprakash.ranjan@codeaurora.org> wrote: > > On 2019-10-01 09:13, Jeffrey Hugo wrote: > > Sai, > > > > This patch breaks boot on the 835 laptops. However, I haven't seen > > the same issue on the MTP. I wonder, is coresight expected to work > > with production fused devices? I wonder if thats the difference > > between the laptop and MTP that is causing the issue. > > > > Let me know what I can do to help debug. > > > > I did test on MSM8998 MTP and didn't face any issue. I am guessing this > is the same issue which you reported regarding cpuidle? Coresight ETM Yes, its the same issue. Right now, I need both patches reverted to boot. > and cpuidle do not work well together since ETMs share the same power > domain as CPU and they might get turned off when CPU enters idle states. > Can you try with cpuidle.off=1 cmdline or just remove idle states from > DT to confirm? If this is the issue, then we can try the below patch > from Andrew Murray for ETM save and restore: > > https://patchwork.kernel.org/patch/11097893/ Is there still value in testing this if the idle states are removed, yet the coresight nodes still cause issues? Funny enough, I'm using the arm64 defconfig which doesn't seem to select CONFIG_CORESIGHT, so I'm not even sure what would be binding to the DT devices... > > It is not merged yet. They would appreciate your tested by ;) > > Thanks, > Sai
WARNING: multiple messages have this Message-ID (diff)
From: Jeffrey Hugo <jeffrey.l.hugo@gmail.com> To: Sai Prakash Ranjan <saiprakash.ranjan@codeaurora.org> Cc: Mark Rutland <mark.rutland@arm.com>, Rajendra Nayak <rnayak@codeaurora.org>, Mathieu Poirier <mathieu.poirier@linaro.org>, Suzuki K Poulose <suzuki.poulose@arm.com>, Alexander Shishkin <alexander.shishkin@linux.intel.com>, MSM <linux-arm-msm@vger.kernel.org>, Marc Gonzalez <marc.w.gonzalez@free.fr>, lkml <linux-kernel@vger.kernel.org>, Bjorn Andersson <bjorn.andersson@linaro.org>, David Brown <david.brown@linaro.org>, Andy Gross <agross@kernel.org>, Sibi Sankar <sibis@codeaurora.org>, Leo Yan <leo.yan@linaro.org>, linux-arm-kernel@lists.infradead.org Subject: Re: [PATCHv9 2/3] arm64: dts: qcom: msm8998: Add Coresight support Date: Tue, 1 Oct 2019 11:14:03 -0600 [thread overview] Message-ID: <CAOCk7NohO67qeYCnrjy4P0KN9nLUiamphHRvj-bFP++K7khPOw@mail.gmail.com> (raw) In-Reply-To: <16212a577339204e901cf4eefa5e82f1@codeaurora.org> On Tue, Oct 1, 2019 at 11:04 AM Sai Prakash Ranjan <saiprakash.ranjan@codeaurora.org> wrote: > > On 2019-10-01 09:13, Jeffrey Hugo wrote: > > Sai, > > > > This patch breaks boot on the 835 laptops. However, I haven't seen > > the same issue on the MTP. I wonder, is coresight expected to work > > with production fused devices? I wonder if thats the difference > > between the laptop and MTP that is causing the issue. > > > > Let me know what I can do to help debug. > > > > I did test on MSM8998 MTP and didn't face any issue. I am guessing this > is the same issue which you reported regarding cpuidle? Coresight ETM Yes, its the same issue. Right now, I need both patches reverted to boot. > and cpuidle do not work well together since ETMs share the same power > domain as CPU and they might get turned off when CPU enters idle states. > Can you try with cpuidle.off=1 cmdline or just remove idle states from > DT to confirm? If this is the issue, then we can try the below patch > from Andrew Murray for ETM save and restore: > > https://patchwork.kernel.org/patch/11097893/ Is there still value in testing this if the idle states are removed, yet the coresight nodes still cause issues? Funny enough, I'm using the arm64 defconfig which doesn't seem to select CONFIG_CORESIGHT, so I'm not even sure what would be binding to the DT devices... > > It is not merged yet. They would appreciate your tested by ;) > > Thanks, > Sai _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2019-10-01 17:14 UTC|newest] Thread overview: 44+ messages / expand[flat|nested] mbox.gz Atom feed top 2019-07-31 5:57 [PATCHv9 0/3] Add coresight support for SDM845, MSM8998 and MSM8996 Sai Prakash Ranjan 2019-07-31 5:57 ` Sai Prakash Ranjan 2019-07-31 5:58 ` [PATCHv9 1/3] arm64: dts: qcom: sdm845: Add Coresight support Sai Prakash Ranjan 2019-07-31 5:58 ` Sai Prakash Ranjan 2019-07-31 6:05 ` Sai Prakash Ranjan 2019-07-31 6:05 ` Sai Prakash Ranjan 2019-08-07 10:08 ` Sai Prakash Ranjan 2019-08-07 10:08 ` Sai Prakash Ranjan 2019-08-07 10:12 ` Suzuki K Poulose 2019-08-07 10:12 ` Suzuki K Poulose 2019-08-07 11:16 ` Sai Prakash Ranjan 2019-08-07 11:16 ` Sai Prakash Ranjan 2019-07-31 5:58 ` [PATCHv9 2/3] arm64: dts: qcom: msm8998: " Sai Prakash Ranjan 2019-07-31 5:58 ` Sai Prakash Ranjan 2019-10-01 16:13 ` Jeffrey Hugo 2019-10-01 16:13 ` Jeffrey Hugo 2019-10-01 17:04 ` Sai Prakash Ranjan 2019-10-01 17:04 ` Sai Prakash Ranjan 2019-10-01 17:14 ` Jeffrey Hugo [this message] 2019-10-01 17:14 ` Jeffrey Hugo 2019-10-01 17:31 ` Suzuki K Poulose 2019-10-01 17:31 ` Suzuki K Poulose 2019-10-01 17:52 ` Sai Prakash Ranjan 2019-10-01 17:52 ` Sai Prakash Ranjan 2019-10-01 18:01 ` Jeffrey Hugo 2019-10-01 18:01 ` Jeffrey Hugo 2019-10-01 18:05 ` Sai Prakash Ranjan 2019-10-01 18:05 ` Sai Prakash Ranjan 2019-10-02 15:03 ` Mathieu Poirier 2019-10-02 15:03 ` Mathieu Poirier 2019-10-02 15:21 ` Marc Gonzalez 2019-10-02 15:21 ` Marc Gonzalez 2019-10-02 15:34 ` Mathieu Poirier 2019-10-02 15:34 ` Mathieu Poirier 2019-10-02 15:49 ` Jeffrey Hugo 2019-10-02 15:49 ` Jeffrey Hugo 2019-10-03 10:20 ` Daniel Thompson 2019-10-03 10:20 ` Daniel Thompson 2019-10-03 10:52 ` Suzuki K Poulose 2019-10-03 10:52 ` Suzuki K Poulose 2019-10-03 12:52 ` Daniel Thompson 2019-10-03 12:52 ` Daniel Thompson 2019-07-31 5:58 ` [PATCHv9 3/3] arm64: dts: qcom: msm8996: " Sai Prakash Ranjan 2019-07-31 5:58 ` Sai Prakash Ranjan
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=CAOCk7NohO67qeYCnrjy4P0KN9nLUiamphHRvj-bFP++K7khPOw@mail.gmail.com \ --to=jeffrey.l.hugo@gmail.com \ --cc=agross@kernel.org \ --cc=alexander.shishkin@linux.intel.com \ --cc=bjorn.andersson@linaro.org \ --cc=david.brown@linaro.org \ --cc=leo.yan@linaro.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-arm-msm@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=marc.w.gonzalez@free.fr \ --cc=mark.rutland@arm.com \ --cc=mathieu.poirier@linaro.org \ --cc=rnayak@codeaurora.org \ --cc=saiprakash.ranjan@codeaurora.org \ --cc=sibis@codeaurora.org \ --cc=suzuki.poulose@arm.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: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.