From: John Youn <John.Youn@synopsys.com>
To: Marek Szyprowski <m.szyprowski@samsung.com>,
"linux-usb@vger.kernel.org" <linux-usb@vger.kernel.org>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
"linux-samsung-soc@vger.kernel.org"
<linux-samsung-soc@vger.kernel.org>
Cc: Robert Baldyga <r.baldyga@samsung.com>,
John Youn <John.Youn@synopsys.com>,
Krzysztof Kozlowski <k.kozlowski@samsung.com>,
Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>
Subject: Re: [PATCH v4 4/4] usb: dwc2: refactor common low-level hw code to platform.c
Date: Mon, 5 Oct 2015 22:26:19 +0000 [thread overview]
Message-ID: <2B3535C5ECE8B5419E3ECBE30077290901DC3876E9@US01WEMBX2.internal.synopsys.com> (raw)
In-Reply-To: 1443771918-19075-1-git-send-email-m.szyprowski@samsung.com
On 10/2/2015 12:45 AM, Marek Szyprowski wrote:
> DWC2 module on some platforms needs three additional hardware
> resources: phy controller, clock and power supply. All of them must be
> enabled/activated to properly initialize and operate. This was initially
> handled in s3c-hsotg driver, which has been converted to 'gadget' part
> of dwc2 driver. Unfortunately, not all of this code got moved to common
> platform code, what resulted in accessing DWC2 registers without
> enabling low-level hardware resources. This fails for example on Exynos
> SoCs. This patch moves all the code for managing those resources to
> common platform.c file and provides convenient wrappers for controlling
> them.
>
> Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>
> ---
> Changelog:
> v4:
> - fixed broken conditional compilation and adjusted comments in dwc2_hsotg
> structure documentation
>
> v3:
> - rebased onto latest 'testing/next' from Felipe Balbi (includes
> s3c_hsotg -> dwc2 rename)
>
> v2:
> - moved setting of ll_hw_enabled flag to enable/disable functions,
> as suggested by John Youn
> - moved setting of phy width to dwc2_lowlevel_init function
> ---
> drivers/usb/dwc2/core.h | 24 +++--
> drivers/usb/dwc2/gadget.c | 193 ++++--------------------------------
> drivers/usb/dwc2/platform.c | 234 +++++++++++++++++++++++++++++++++++++-------
> 3 files changed, 228 insertions(+), 223 deletions(-)
>
Hi Marek,
I still see lockdep warnings.
Any ideas about these?
[ 1618.179611] ======================================================
[ 1618.179612] [ INFO: possible circular locking dependency detected ]
[ 1618.179613] 4.3.0-rc3-snps-00125-g744fd93 #28 Not tainted
[ 1618.179614] -------------------------------------------------------
[ 1618.179615] modprobe/2658 is trying to acquire lock:
[ 1618.179616] (&hsotg->init_mutex){+.+.+.}, at: [<ffffffffc043aa3c>] dwc2_hsotg_udc_start+0x5c/0x200 [dwc2]
[ 1618.179622]
[ 1618.179622] but task is already holding lock:
[ 1618.179623] (udc_lock){+.+.+.}, at: [<ffffffffc0374b8a>] usb_gadget_probe_driver+0x3a/0xd0 [udc_core]
[ 1618.179627]
[ 1618.179627] which lock already depends on the new lock.
[ 1618.179627]
[ 1618.179628]
[ 1618.179628] the existing dependency chain (in reverse order) is:
[ 1618.179629]
[ 1618.179629] -> #1 (udc_lock){+.+.+.}:
[ 1618.179631] [<ffffffff810da56d>] lock_acquire+0xdd/0x1f0
[ 1618.179635] [<ffffffff818658a6>] mutex_lock_nested+0x76/0x3e0
[ 1618.179638] [<ffffffffc0374da7>] usb_add_gadget_udc_release+0x187/0x240 [udc_core]
[ 1618.179640] [<ffffffffc0374e70>] usb_add_gadget_udc+0x10/0x20 [udc_core]
[ 1618.179642] [<ffffffffc043b30c>] dwc2_gadget_init+0x47c/0x580 [dwc2]
[ 1618.179645] [<ffffffffc042d2f2>] dwc2_driver_probe+0x422/0x4b0 [dwc2]
[ 1618.179648] [<ffffffff8153fe94>] platform_drv_probe+0x34/0x90
[ 1618.179650] [<ffffffff8153db54>] driver_probe_device+0x224/0x480
[ 1618.179652] [<ffffffff8153deb1>] __device_attach_driver+0x71/0xa0
[ 1618.179654] [<ffffffff8153b78d>] bus_for_each_drv+0x5d/0x90
[ 1618.179655] [<ffffffff8153d83f>] __device_attach+0xbf/0x140
[ 1618.179657] [<ffffffff8153df23>] device_initial_probe+0x13/0x20
[ 1618.179658] [<ffffffff8153cb03>] bus_probe_device+0xa3/0xb0
[ 1618.179660] [<ffffffff8153a76d>] device_add+0x40d/0x690
[ 1618.179661] [<ffffffff8153fb81>] platform_device_add+0x111/0x270
[ 1618.179663] [<ffffffffc0394128>] dwc2_pci_probe+0xe8/0x1d2 [dwc2_pci]
[ 1618.179665] [<ffffffff81446085>] local_pci_probe+0x45/0xa0
[ 1618.179668] [<ffffffff81447451>] pci_device_probe+0xe1/0x130
[ 1618.179669] [<ffffffff8153db54>] driver_probe_device+0x224/0x480
[ 1618.179671] [<ffffffff8153de38>] __driver_attach+0x88/0x90
[ 1618.179672] [<ffffffff8153b6d6>] bus_for_each_dev+0x66/0xa0
[ 1618.179674] [<ffffffff8153d31e>] driver_attach+0x1e/0x20
[ 1618.179675] [<ffffffff8153ce8e>] bus_add_driver+0x1ee/0x280
[ 1618.179677] [<ffffffff8153e930>] driver_register+0x60/0xe0
[ 1618.179678] [<ffffffff81445a60>] __pci_register_driver+0x60/0x70
[ 1618.179680] [<ffffffffc000601e>] 0xffffffffc000601e
[ 1618.179681] [<ffffffff81002123>] do_one_initcall+0xb3/0x200
[ 1618.179684] [<ffffffff811a8a41>] do_init_module+0x5f/0x1e7
[ 1618.179687] [<ffffffff81120e48>] load_module+0x21a8/0x2840
[ 1618.179689] [<ffffffff8112170a>] SyS_finit_module+0x9a/0xc0
[ 1618.179691] [<ffffffff81869c36>] entry_SYSCALL_64_fastpath+0x16/0x7a
[ 1618.179693]
[ 1618.179693] -> #0 (&hsotg->init_mutex){+.+.+.}:
[ 1618.179695] [<ffffffff810d97f5>] __lock_acquire+0x1d35/0x1db0
[ 1618.179697] [<ffffffff810da56d>] lock_acquire+0xdd/0x1f0
[ 1618.179698] [<ffffffff818658a6>] mutex_lock_nested+0x76/0x3e0
[ 1618.179700] [<ffffffffc043aa3c>] dwc2_hsotg_udc_start+0x5c/0x200 [dwc2]
[ 1618.179703] [<ffffffffc0374a34>] udc_bind_to_driver+0xa4/0x100 [udc_core]
[ 1618.179705] [<ffffffffc0374bca>] usb_gadget_probe_driver+0x7a/0xd0 [udc_core]
[ 1618.179707] [<ffffffffc059a674>] usb_composite_probe+0xa4/0xc0 [libcomposite]
[ 1618.179709] [<ffffffffc0474010>] msg_init+0x10/0x1000 [g_mass_storage]
[ 1618.179711] [<ffffffff81002123>] do_one_initcall+0xb3/0x200
[ 1618.179713] [<ffffffff811a8a41>] do_init_module+0x5f/0x1e7
[ 1618.179714] [<ffffffff81120e48>] load_module+0x21a8/0x2840
[ 1618.179716] [<ffffffff8112170a>] SyS_finit_module+0x9a/0xc0
[ 1618.179717] [<ffffffff81869c36>] entry_SYSCALL_64_fastpath+0x16/0x7a
[ 1618.179719]
[ 1618.179719] other info that might help us debug this:
[ 1618.179719]
[ 1618.179720] Possible unsafe locking scenario:
[ 1618.179720]
[ 1618.179721] CPU0 CPU1
[ 1618.179722] ---- ----
[ 1618.179722] lock(udc_lock);
[ 1618.179723] lock(&hsotg->init_mutex);
[ 1618.179725] lock(udc_lock);
[ 1618.179726] lock(&hsotg->init_mutex);
[ 1618.179727]
[ 1618.179727] *** DEADLOCK ***
[ 1618.179727]
[ 1618.179728] 1 lock held by modprobe/2658:
[ 1618.179729] #0: (udc_lock){+.+.+.}, at: [<ffffffffc0374b8a>] usb_gadget_probe_driver+0x3a/0xd0 [udc_core]
[ 1618.179732]
[ 1618.179732] stack backtrace:
[ 1618.179734] CPU: 3 PID: 2658 Comm: modprobe Not tainted 4.3.0-rc3-snps-00125-g744fd93 #28
[ 1618.179735] Hardware name: ASUS All Series/H97M-PLUS, BIOS 2305 10/09/2014
[ 1618.179736] ffffffff82aa4b20 ffff880215a57a20 ffffffff813f3289 ffffffff82aa4b20
[ 1618.179738] ffff880215a57a60 ffffffff810d609c ffff880215a57ac0 ffff880213560808
[ 1618.179740] 0000000000000001 0000000000000000 ffff880213560840 ffff880213560000
[ 1618.179742] Call Trace:
[ 1618.179744] [<ffffffff813f3289>] dump_stack+0x4b/0x72
[ 1618.179746] [<ffffffff810d609c>] print_circular_bug+0x1ec/0x260
[ 1618.179748] [<ffffffff810d97f5>] __lock_acquire+0x1d35/0x1db0
[ 1618.179749] [<ffffffff81539400>] ? dev_vprintk_emit+0x90/0x240
[ 1618.179752] [<ffffffffc0000100>] ? pps_cdev_release+0x10/0x30 [pps_core]
[ 1618.179754] [<ffffffff810da56d>] lock_acquire+0xdd/0x1f0
[ 1618.179756] [<ffffffffc043aa3c>] ? dwc2_hsotg_udc_start+0x5c/0x200 [dwc2]
[ 1618.179758] [<ffffffff818658a6>] mutex_lock_nested+0x76/0x3e0
[ 1618.179760] [<ffffffffc043aa3c>] ? dwc2_hsotg_udc_start+0x5c/0x200 [dwc2]
[ 1618.179762] [<ffffffff8153984c>] ? __dev_printk+0x3c/0x80
[ 1618.179764] [<ffffffffc043aa3c>] ? dwc2_hsotg_udc_start+0x5c/0x200 [dwc2]
[ 1618.179767] [<ffffffffc043aa3c>] dwc2_hsotg_udc_start+0x5c/0x200 [dwc2]
[ 1618.179768] [<ffffffffc0474000>] ? 0xffffffffc0474000
[ 1618.179770] [<ffffffffc0374a34>] udc_bind_to_driver+0xa4/0x100 [udc_core]
[ 1618.179772] [<ffffffffc0374bca>] usb_gadget_probe_driver+0x7a/0xd0 [udc_core]
[ 1618.179774] [<ffffffffc059a674>] usb_composite_probe+0xa4/0xc0 [libcomposite]
[ 1618.179776] [<ffffffffc0474010>] msg_init+0x10/0x1000 [g_mass_storage]
[ 1618.179777] [<ffffffff81002123>] do_one_initcall+0xb3/0x200
[ 1618.179779] [<ffffffff810f881d>] ? rcu_read_lock_sched_held+0x6d/0x80
[ 1618.179782] [<ffffffff8120c064>] ? kmem_cache_alloc_trace+0x1f4/0x2d0
[ 1618.179783] [<ffffffff811a8a41>] do_init_module+0x5f/0x1e7
[ 1618.179785] [<ffffffff81120e48>] load_module+0x21a8/0x2840
[ 1618.179786] [<ffffffff8111d0e0>] ? __symbol_put+0x40/0x40
[ 1618.179788] [<ffffffff8112170a>] SyS_finit_module+0x9a/0xc0
[ 1618.179790] [<ffffffff81869c36>] entry_SYSCALL_64_fastpath+0x16/0x7a
Regards,
John
next prev parent reply other threads:[~2015-10-05 22:26 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-09-21 10:16 [PATCH v3 0/4] Exynos4412-based Trats2 USB gadget (DWC2) fixes Marek Szyprowski
2015-09-21 10:16 ` [PATCH v3 1/4] usb: dwc2: remove double call to dwc2_hsotg_of_probe Marek Szyprowski
2015-09-21 10:16 ` [PATCH v3 2/4] usb: dwc2: remove non-functional clock gating Marek Szyprowski
2015-09-21 10:16 ` [PATCH v3 3/4] usb: dwc2: fix unbalanced phy control Marek Szyprowski
2015-09-21 10:16 ` [PATCH v3 4/4] usb: dwc2: refactor common low-level hw code to platform.c Marek Szyprowski
2015-10-01 15:50 ` Felipe Balbi
2015-10-01 15:59 ` Felipe Balbi
2015-10-02 7:45 ` [PATCH v4 " Marek Szyprowski
2015-10-05 22:26 ` John Youn [this message]
2015-10-05 23:27 ` Felipe Balbi
2015-10-06 8:55 ` [PATCH v5 1/2] usb: dwc2: remove no longer needed init_mutex Marek Szyprowski
2015-10-06 8:55 ` [PATCH v5 2/2] usb: dwc2: refactor common low-level hw code to platform.c Marek Szyprowski
2015-10-07 2:37 ` John Youn
2015-10-14 6:52 ` [PATCH v6 1/2] usb: dwc2: remove no longer needed init_mutex Marek Szyprowski
2015-10-14 6:52 ` [PATCH v6 2/2] usb: dwc2: refactor common low-level hw code to platform.c Marek Szyprowski
2015-10-06 8:55 ` [PATCH v4 4/4] " Marek Szyprowski
2015-10-01 21:04 ` [PATCH v3 " John Youn
2015-10-01 22:03 ` Felipe Balbi
2015-10-01 22:21 ` John Youn
2015-10-01 22:31 ` Felipe Balbi
2015-10-02 7:47 ` Marek Szyprowski
2015-09-28 18:21 ` [PATCH v3 0/4] Exynos4412-based Trats2 USB gadget (DWC2) fixes John Youn
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=2B3535C5ECE8B5419E3ECBE30077290901DC3876E9@US01WEMBX2.internal.synopsys.com \
--to=john.youn@synopsys.com \
--cc=b.zolnierkie@samsung.com \
--cc=k.kozlowski@samsung.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-samsung-soc@vger.kernel.org \
--cc=linux-usb@vger.kernel.org \
--cc=m.szyprowski@samsung.com \
--cc=r.baldyga@samsung.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).