linux-kernel - Re: [PATCH v2] clk: qcom: smd: Add support for MSM8998 rpm clocks

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <8b3702e8-3779-db5c-1777-94a6d97c1080@codeaurora.org>
Date:   Fri, 7 Dec 2018 08:13:16 -0700
From:   Jeffrey Hugo <jhugo@...eaurora.org>
To:     Marc Gonzalez <marc.w.gonzalez@...e.fr>,
        Michael Turquette <mturquette@...libre.com>,
        Stephen Boyd <sboyd@...nel.org>
Cc:     Bjorn Andersson <bjorn.andersson@...aro.org>,
        linux-clk <linux-clk@...r.kernel.org>,
        MSM <linux-arm-msm@...r.kernel.org>,
        LKML <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v2] clk: qcom: smd: Add support for MSM8998 rpm clocks

On 12/7/2018 7:23 AM, Marc Gonzalez wrote:
> On 06/12/2018 23:11, Jeffrey Hugo wrote:
> 
>> Add rpm smd clocks, PMIC and bus clocks which are required on MSM8998
>> for clients to vote on.
>>
>> Signed-off-by: Jeffrey Hugo <jhugo@...eaurora.org>
>> ---
>> v2
>> -fix compatible ordering nits per Stephen
>>
>>   .../devicetree/bindings/clock/qcom,rpmcc.txt       |  1 +
>>   drivers/clk/qcom/clk-smd-rpm.c                     | 62 ++++++++++++++++++++++
>>   include/dt-bindings/clock/qcom,rpmcc.h             |  6 +++
>>   3 files changed, 69 insertions(+)
> 
> Hmmm, my board seems to dislike this patch... it locks up for a while,
> then reboots.
> 
> [    6.957289] ufshcd-qcom 1da4000.ufshc: ufshcd_query_flag: Sending flag query for idn 1 failed, err = -11
> [    6.961233] ufshcd-qcom 1da4000.ufshc: ufshcd_query_flag_retry: failed with error -11, retries 2
> [    6.970782] ufshcd-qcom 1da4000.ufshc: ufshcd_query_flag_retry: query attribute, opcode 6, idn 1, failed with error -11 after 3 retires
> [    6.979576] ufshcd-qcom 1da4000.ufshc: ufshcd_complete_dev_init setting fDeviceInit flag failed with error -11
> [   23.432798] rcu: INFO: rcu_preempt detected stalls on CPUs/tasks:
> [   23.432884] rcu:     7-...0: (1 GPs behind) idle=6a2/1/0x4000000000000000 softirq=92/93 fqs=2626
> [   23.437927] rcu:     (detected by 0, t=5252 jiffies, g=-819, q=112)
> [   23.446425] Task dump for CPU 7:
> [   23.452567] swapper/0       R  running task        0     1      0 0x0000002a
> [   23.455806] Call trace:
> [   23.462872]  __switch_to+0x94/0xe0
> [   23.465009]  _regmap_write+0x58/0xb0
> [   23.468452]  _regmap_update_bits+0xf0/0x110
> [   23.472186]  regmap_update_bits_base+0x60/0x90
> [   23.476110]  clk_disable_regmap+0x34/0x40
> [   23.480614]  clk_branch_toggle+0x108/0x1b0
> [   23.484681]  clk_branch2_disable+0x18/0x20
> [   23.488692]  clk_disable_unused_subtree+0xc4/0xe0
> [   23.492761]  clk_disable_unused+0x3c/0x130
> [   23.497535]  do_one_initcall+0x5c/0x180
> [   23.501557]  kernel_init_freeable+0x198/0x244
> [   23.505276]  kernel_init+0x10/0x110
> [   23.509768]  ret_from_fork+0x10/0x20
> 
> /*** REBOOT ***/
> Format: Log Type - Time(microsec) - Message - Optional Info
> Log Type: B - Since Boot(Power On Reset),  D - Delta,  S - Statistic
> 
> I need to check the ufs_reset pin and phy init seq before I can investigate
> this issue, but I wanted to send a report ASAP.
> 

Hmm.  I run with clk_ignore_unused.  I'm guessing that something was 
defined that should be used eventually, but isn't used now, and so its 
getting turned off when it probably shouldn't be.

I'll try to repro on my end.

-- 
Jeffrey Hugo
Qualcomm Datacenter Technologies as an affiliate of Qualcomm 
Technologies, Inc.
Qualcomm Technologies, Inc. is a member of the
Code Aurora Forum, a Linux Foundation Collaborative Project.