From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1758484AbcDHSYp (ORCPT ); Fri, 8 Apr 2016 14:24:45 -0400 Received: from mail-io0-f196.google.com ([209.85.223.196]:35117 "EHLO mail-io0-f196.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753056AbcDHSYm (ORCPT ); Fri, 8 Apr 2016 14:24:42 -0400 MIME-Version: 1.0 Reply-To: cw00.choi@samsung.com In-Reply-To: References: <1460091646-28701-1-git-send-email-cw00.choi@samsung.com> Date: Sat, 9 Apr 2016 03:24:40 +0900 Message-ID: Subject: Re: [PATCH 0/7] PM / devfreq: Add NoCP devfreq-event and support busfreq on exyno5422-odroidxu3 From: Chanwoo Choi To: Anand Moon Cc: "myungjoo.ham@samsung.com" , Kyungmin Park , =?UTF-8?Q?Krzysztof_Koz=C5=82owski?= , Kukjin Kim , Sylwester Nawrocki , Tomasz Figa , "Rafael J. Wysocki" , Rob Herring , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , Russell King , Markus Reichl , Tobias Jakobi , "inki.dae@samsung.com" , Linux Kernel , Linux PM list , "linux-samsung-soc@vger.kernel.org" , linux-arm-kernel , devicetree Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Anand, On Sat, Apr 9, 2016 at 3:11 AM, Anand Moon wrote: > Hi Chanwoo, > > On 8 April 2016 at 10:30, Chanwoo Choi wrote: >> This patchset support the AMBA bus frequency scaling on Exynos5422-based >> Odroid-XU3 board. But, this series only support the bus frequency scaling >> for INT (Internal) block using VDD_INT power line. >> >> Also, to support the bus frequency scaling for Exynos542x SoC, >> Exynos542x SoC has the specific 'NoC (Network on Chip) Probe' device >> to measure the transfered data traffic on NoC (Network on Chip) >> instead of PPMU (Platform Performance Monitoring Unit). NoC Probe device >> provide the utilization for INT block of Exynos542x SoC. >> >> The generic exynos-bus frequency driver uses the 'NoC Probe' devfreq-event >> device (drivers/devfreq/event/exynos-nocp.c) without any modification. >> Just add the phandle of 'NoC Probe' dt node to bus dt node. >> >> Depend on: >> This patchset depends on patch[1] which support the generic exynos-bus >> frequency driver. >> >> [1] https://lkml.org/lkml/2016/4/8/14 >> - [PATCH v8 00/20] PM / devferq: Add generic exynos bus frequency driver and new passive governor >> >> Chanwoo Choi (7): >> PM / devfreq: event: Add new Exynos NoC probe driver >> PM / devfreq: exynos: Add the detailed correlation for Exynos5422 bus >> ARM: dts: Add NoC Probe dt node for Exynos542x SoC >> dt-bindings: clock: Add the clock id for ACLK clock of Exynos542x SoC >> clk: samsung: exynos542x: Add the clock id for ACLK >> ARM: dts: Add bus nodes using VDD_INT for Exynos542x SoC >> ARM: dts: Add support of Bus frequency using VDD_INT for exynos5422-odroidxu3 >> >> .../bindings/devfreq/event/exynos-nocp.txt | 86 +++++ >> .../devicetree/bindings/devfreq/exynos-bus.txt | 19 + >> arch/arm/boot/dts/exynos5420.dtsi | 407 +++++++++++++++++++++ >> arch/arm/boot/dts/exynos5422-odroidxu3-common.dtsi | 99 +++++ >> drivers/clk/samsung/clk-exynos5420.c | 85 +++-- >> drivers/devfreq/event/Kconfig | 8 + >> drivers/devfreq/event/Makefile | 2 + >> drivers/devfreq/event/exynos-nocp.c | 247 +++++++++++++ >> drivers/devfreq/event/exynos-nocp.h | 78 ++++ >> include/dt-bindings/clock/exynos5420.h | 24 +- >> 10 files changed, 1024 insertions(+), 31 deletions(-) >> create mode 100644 Documentation/devicetree/bindings/devfreq/event/exynos-nocp.txt >> create mode 100644 drivers/devfreq/event/exynos-nocp.c >> create mode 100644 drivers/devfreq/event/exynos-nocp.h >> >> -- >> 1.9.1 >> > > I am observing following deadlock. Both on Odroid U3 and Odroid XU4. Thanks for your test. I'll test it again and fix it. Best Regards, Chanwoo Choi > > [ 7.718372] [ INFO: possible recursive locking detected ] > [ 7.723747] 4.6.0-rc2-xu4ml #4 Not tainted > [ 7.727817] --------------------------------------------- > [ 7.733190] kworker/u16:1/136 is trying to acquire lock: > [ 7.738476] (&devfreq->lock){+.+.+.}, at: [] > update_devfreq_passive+0x2c/0x88 > [ 7.746534] > [ 7.746534] but task is already holding lock: > [ 7.752339] (&devfreq->lock){+.+.+.}, at: [] > devfreq_monitor+0x1c/0x78 > [ 7.759791] > [ 7.759791] other info that might help us debug this: > [ 7.766291] Possible unsafe locking scenario: > [ 7.766291] > [ 7.766307] usb 5-1: new high-speed USB device number 2 using xhci-hcd > [ 7.778682] CPU0 > [ 7.781107] ---- > [ 7.783533] lock(&devfreq->lock); > [ 7.786999] lock(&devfreq->lock); > [ 7.790467] > [ 7.790467] *** DEADLOCK *** > [ 7.790467] > [ 7.796359] May be due to missing lock nesting notation > [ 7.796359] > [ 7.803120] 4 locks held by kworker/u16:1/136: > [ 7.807537] #0: ("%s"("devfreq_wq")){.+.+..}, at: [] > process_one_work+0x13c/0x454 > [ 7.816029] #1: ((&(&devfreq->work)->work)){+.+...}, at: > [] process_one_work+0x13c/0x454 > [ 7.825128] #2: (&devfreq->lock){+.+.+.}, at: [] > devfreq_monitor+0x1c/0x78 > [ 7.833013] #3: (&nh->srcu){......}, at: [] > __srcu_notifier_call_chain+0x54/0xe0 > [ 7.841419] > [ 7.841419] stack backtrace: > [ 7.845758] CPU: 1 PID: 136 Comm: kworker/u16:1 Not tainted > 4.6.0-rc2-xu4ml #4 > [ 7.852946] Hardware name: SAMSUNG EXYNOS (Flattened Device Tree) > [ 7.859017] Workqueue: devfreq_wq devfreq_monitor > [ 7.863717] [] (unwind_backtrace) from [] > (show_stack+0x10/0x14) > [ 7.871417] [] (show_stack) from [] > (dump_stack+0xa4/0xd0) > [ 7.878613] [] (dump_stack) from [] > (__lock_acquire+0x1ce8/0x207c) > [ 7.886492] [] (__lock_acquire) from [] > (lock_acquire+0x90/0xd4) > [ 7.892726] usb 5-1: New USB device found, idVendor=0bda, idProduct=8153 > [ 7.892737] usb 5-1: New USB device strings: Mfr=1, Product=2, SerialNumber=6 > [ 7.892745] usb 5-1: Product: USB 10/100/1000 LAN > [ 7.892753] usb 5-1: Manufacturer: Realtek > [ 7.892761] usb 5-1: SerialNumber: 000001000000 > [ 7.921248] [] (lock_acquire) from [] > (mutex_lock_nested+0x78/0x4ec) > [ 7.929301] [] (mutex_lock_nested) from [] > (update_devfreq_passive+0x2c/0x88) > [ 7.938138] [] (update_devfreq_passive) from [] > (devfreq_passive_notifier_call+0x48/0x50) > [ 7.948016] [] (devfreq_passive_notifier_call) from > [] (notifier_call_chain+0x54/0xa4) > [ 7.957633] [] (notifier_call_chain) from [] > (__srcu_notifier_call_chain+0x98/0xe0) > [ 7.966991] [] (__srcu_notifier_call_chain) from > [] (srcu_notifier_call_chain+0x18/0x20) > [ 7.976783] [] (srcu_notifier_call_chain) from > [] (update_devfreq+0xe4/0x18c) > [ 7.985622] [] (update_devfreq) from [] > (devfreq_monitor+0x24/0x78) > [ 7.993595] [] (devfreq_monitor) from [] > (process_one_work+0x1ac/0x454) > [ 8.001913] [] (process_one_work) from [] > (worker_thread+0x50/0x508) > [ 8.009976] [] (worker_thread) from [] > (kthread+0xf8/0x110) > [ 8.017254] [] (kthread) from [] > (ret_from_fork+0x14/0x24) > > Best Regards > -Anand Moon > -- > To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html