From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f48.google.com (mail-wm1-f48.google.com [209.85.128.48]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id E9E722F9996 for ; Tue, 26 Aug 2025 09:18:17 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.48 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1756199901; cv=none; b=hg1VLX43OS8X2for6hxbZ4KDeHHy5grploSWjH5xwb/r75IvQfHfhNhYZ5wqawuGuwpd0QD3VCFVSwVv6RswabJKRdr18LfMQno6ltIi4+DiLcb7OwFxBfew6MeyyjKeIprMiqJScgA1rdXigh8T7Z8HDVG91K3tegN3nZDkRwI= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1756199901; c=relaxed/simple; bh=TkwVE2JtukWSJCieI0x/e2MinM15H3GeZu1WuQJWONE=; h=Mime-Version:Content-Type:Date:Message-Id:Cc:Subject:From:To: References:In-Reply-To; b=X4d8PI+OnEv/XN+S9M+/7HTh/sFIf9C2TzNictAmTkEpjL+SNWGh0Gx74LZC8UMYOlFgXf/skh6TDtUIN16uuyv63YnDsxqaBswtl6qZ4zJ352tcTg9SiVvqH0NUF13tskbfY+2HxVE3AyKjawvepcZfnOuwFgw9HyGDq9w0+Wk= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org; spf=pass smtp.mailfrom=linaro.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b=vKFu5IyX; arc=none smtp.client-ip=209.85.128.48 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linaro.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="vKFu5IyX" Received: by mail-wm1-f48.google.com with SMTP id 5b1f17b1804b1-45a1b066b5eso28290005e9.1 for ; Tue, 26 Aug 2025 02:18:17 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1756199896; x=1756804696; darn=vger.kernel.org; h=in-reply-to:references:to:from:subject:cc:message-id:date :content-transfer-encoding:mime-version:from:to:cc:subject:date :message-id:reply-to; bh=9y82ZcMTvoPNY3nU1FX8UKDNck7AnFMwEFoUX52rQcM=; b=vKFu5IyXgmCUHoNdfRVzChw8Knkq9L4J9tGkkg4vAQ3EjWBrPuIsaXTy5djOTSowX/ pLOXVqEojG0DVbPa2m7OG4IEw4zdhCGiFkA4OZ/iH2QrULXRgPLLKVjWqlJU5piIL9B5 R81c9NU5bgUiEmxfssH1Es8iH7iTWmKD+gp1RzagC4cKj32g660GBA8ysTwG37X9Jltp pLf8vNhHKbwi+MkRpcJ15fXPXBnINJrTN4DI3B/KeempkcmGOatClOVYhNwwavvdcf/g 7GCzUv2474392wlK5YaGS/5Dykg7cJNLPRFMw79InSbfvRjZTfsHjSEI7CanZXlHegxU RoQQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1756199896; x=1756804696; h=in-reply-to:references:to:from:subject:cc:message-id:date :content-transfer-encoding:mime-version:x-gm-message-state:from:to :cc:subject:date:message-id:reply-to; bh=9y82ZcMTvoPNY3nU1FX8UKDNck7AnFMwEFoUX52rQcM=; b=E5IStyHkRx8l/qZRYP7qhCcsCwWmY9URcghTjzDe28FDlMMgUr7rg6GCYb50boccDP PEn4ns9lVZaK8KOEzsZKPv3KA+0VsgBv/rhYmR0i3u95vd6DNogTAb5CQcCEeIkqjho3 J556Uwv1ph5D/j0u0Qv/VcNhB9lfN4osLEH3aSosnALQe8fRnqdetYJhRGXQyxUkSZrV TEB2zhcGv8MteXgoZ52sKd8LQb4C0VHK3+hHO2qGyGhaqo2HJBMhbm6nL1UHWbyTJJIA /qHjZss7XFZt8hMWHKE0cqGMsXmY1QD9ZCpRzyOahEeIgq2u3Usj96tkw+rYvW85/NMH noQQ== X-Forwarded-Encrypted: i=1; AJvYcCUb0BGDFOLeFTUta6sqbgNO/DJBuI/upDoidhBhn6KqJP7pFUy95wRjWuOgLUaG4CqdTsYV6McORb/u@vger.kernel.org X-Gm-Message-State: AOJu0YwzIapVEe16dfuOEM6gYshp2NVomxuzqd4umNPCFkIaBI9mVsY5 Q6tuXdiJtTvxWx4kjcHID/tJM8AkZHBr0BDhTYn4lInWLf+lcRt6QWw0aa+x4PUrBQs= X-Gm-Gg: ASbGnctwjUNV4sWOXZZdRYiLkp2tA1tirXgKHn9OqbskawLteTj4PjUufFqL9qV2xLM vHMNdOHnllBOceV7d/NG0Y5Z3PgTVNOnFXX5wV1JrQPlL0Z49cUvPL0WDjqf+8pXLApE2QjVM2r Z1GJ/PRGk9SuOL6Em6cnoEdqrP2nXnG+XDbVsOasY161c8jhTk+xBMWgT150nxdIEot33FhD1wv 65eDqJIBfsATNZJdIyz6V3mOn2mby+l8lVLmiHiFY0KgLKTjxo44cUUdcxa627Sj4TtAvwBAen+ DLTwV5/9xqzlCcsn0xRCM8nMg5cTURDH6+nIAZTMfs97e81dl5+/lYLGggiDR4bnUWY4sESP3Df TW47q6DbWBHBiO1BzKzkU7EY1vBQ= X-Google-Smtp-Source: AGHT+IHaeN71tfpJgG6c+hKwU5zaiIqic2SJiq3pkNi5jCxctYL6arGbSvZYy9jIMTOStKjZ+u5h9g== X-Received: by 2002:a05:600c:a09:b0:459:dfde:332e with SMTP id 5b1f17b1804b1-45b517c2ea6mr85576555e9.29.1756199896087; Tue, 26 Aug 2025 02:18:16 -0700 (PDT) Received: from localhost ([2a02:c7c:7259:a00:e633:2c7e:2d3c:f5ec]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-45b5748798fsm141608355e9.13.2025.08.26.02.18.15 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Tue, 26 Aug 2025 02:18:15 -0700 (PDT) Precedence: bulk X-Mailing-List: devicetree@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: Mime-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset=UTF-8 Date: Tue, 26 Aug 2025 10:18:13 +0100 Message-Id: Cc: "Krzysztof Kozlowski" , "Greg Kroah-Hartman" , , , , , , , , , , "Jiri Slaby" , "Rob Herring" , "Krzysztof Kozlowski" , "Conor Dooley" , , , , Subject: Re: [PATCH v7 7/8] serial: qcom-geni: Enable PM runtime for serial driver From: "Alexey Klimov" To: "Praveen Talari" , "Bjorn Andersson" , "Konrad Dybcio" , , X-Mailer: aerc 0.20.0 References: <20250721174532.14022-1-quic_ptalari@quicinc.com> <20250721174532.14022-8-quic_ptalari@quicinc.com> <577d05d4-789b-4556-a2d2-d0ad15b2c213@quicinc.com> In-Reply-To: Hi Praveen, Hi Praveen, On Tue Aug 19, 2025 at 8:16 AM BST, Krzysztof Kozlowski wrote: > On 19/08/2025 08:50, Praveen Talari wrote: >> Hi Alexey. >>=20 >> Thank you for your patience, Well, any update on this? >> On 8/12/2025 3:35 PM, Alexey Klimov wrote: >>> (c/c Neil and Srini) >>> >>> On Mon Jul 21, 2025 at 6:45 PM BST, Praveen Talari wrote: >>>> The GENI serial driver currently handles power resource management >>>> through calls to the statically defined geni_serial_resources_on() and >>>> geni_serial_resources_off() functions. This approach reduces modularit= y >>>> and limits support for platforms with diverse power management >>>> mechanisms, including resource managed by firmware. >>>> >>>> Improve modularity and enable better integration with platform-specifi= c >>>> power management, introduce support for runtime PM. Use >>>> pm_runtime_resume_and_get() and pm_runtime_put_sync() within the >>>> qcom_geni_serial_pm() callback to control resource power state >>>> transitions based on UART power state changes. >>>> >>>> Reviewed-by: Bryan O'Donoghue >>>> Signed-off-by: Praveen Talari >>> >>> >>> This breaks at least RB1 (QRB2210), maybe others. >>> Currently broken on -master and on linux-next. >>> >>> Upon login prompt random parts of kernel seems to be off/failed and >>> debugging led to udev being stuck: >>> >>> [ 85.369834] INFO: task kworker/u16:0:12 blocked for more than 42 sec= onds. >>> [ 85.376699] Not tainted 6.17.0-rc1-00004-g53e760d89498 #9 >>> [ 85.382660] "echo 0 > /proc/sys/kernel/hung_task_timeout_secs" disab= les this message. >>> [ 85.390547] task:kworker/u16:0 state:D stack:0 pid:12 tgid:= 12 ppid:2 task_flags:0x4208060 flags:0x00000010 >>> [ 85.401748] Workqueue: async async_run_entry_fn >>> [ 85.406349] Call trace: >>> [ 85.408828] __switch_to+0xe8/0x1a0 (T) >>> [ 85.412724] __schedule+0x290/0x7c0 >>> [ 85.416275] schedule+0x34/0x118 >>> [ 85.419554] rpm_resume+0x14c/0x66c >>> [ 85.423111] rpm_resume+0x2a4/0x66c >>> [ 85.426647] rpm_resume+0x2a4/0x66c >>> [ 85.430188] rpm_resume+0x2a4/0x66c >>> [ 85.433722] __pm_runtime_resume+0x50/0x9c >>> [ 85.437869] __driver_probe_device+0x58/0x120 >>> [ 85.442287] driver_probe_device+0x3c/0x154 >>> [ 85.446523] __driver_attach_async_helper+0x4c/0xc0 >>> [ 85.451446] async_run_entry_fn+0x34/0xe0 >>> [ 85.455504] process_one_work+0x148/0x290 >>> [ 85.459565] worker_thread+0x2c4/0x3e0 >>> [ 85.463368] kthread+0x118/0x1c0 >>> [ 85.466651] ret_from_fork+0x10/0x20 >>> [ 85.470337] INFO: task irq/92-4a8c000.:71 blocked for more than 42 s= econds. >>> [ 85.477351] Not tainted 6.17.0-rc1-00004-g53e760d89498 #9 >>> [ 85.483323] "echo 0 > /proc/sys/kernel/hung_task_timeout_secs" disab= les this message. >>> [ 85.491195] task:irq/92-4a8c000. state:D stack:0 pid:71 tgid:= 71 ppid:2 task_flags:0x208040 flags:0x00000010 >>> [ 85.502290] Call trace: >>> [ 85.504786] __switch_to+0xe8/0x1a0 (T) >>> [ 85.508687] __schedule+0x290/0x7c0 >>> [ 85.512231] schedule+0x34/0x118 >>> [ 85.515504] __synchronize_irq+0x60/0xa0 >>> [ 85.519483] disable_irq+0x3c/0x4c >>> [ 85.522929] msm_pinmux_set_mux+0x3a8/0x44c >>> [ 85.527167] pinmux_enable_setting+0x1c4/0x28c >>> [ 85.531665] pinctrl_commit_state+0xa0/0x260 >>> [ 85.535989] pinctrl_pm_select_default_state+0x4c/0xa0 >>> [ 85.541182] geni_se_resources_on+0xd0/0x15c >>> [ 85.545522] geni_serial_resource_state+0x8c/0xbc >>> [ 85.550282] qcom_geni_serial_runtime_resume+0x24/0x3c >>> [ 85.555470] pm_generic_runtime_resume+0x2c/0x44 >>> [ 85.560139] __rpm_callback+0x48/0x1e0 >>> [ 85.563949] rpm_callback+0x74/0x80 >>> [ 85.567494] rpm_resume+0x39c/0x66c >>> [ 85.571040] __pm_runtime_resume+0x50/0x9c >>> [ 85.575193] handle_threaded_wake_irq+0x30/0x80 >>> [ 85.579771] irq_thread_fn+0x2c/0xb0 >>> [ 85.583443] irq_thread+0x16c/0x278 >>> [ 85.587003] kthread+0x118/0x1c0 >>> [ 85.590283] ret_from_fork+0x10/0x20 >>> [ 85.593943] INFO: task (udev-worker):228 blocked for more than 42 se= conds. >>> [ 85.600873] Not tainted 6.17.0-rc1-00004-g53e760d89498 #9 >>> [ 85.606846] "echo 0 > /proc/sys/kernel/hung_task_timeout_secs" disab= les this message. >>> [ 85.614717] task:(udev-worker) state:D stack:0 pid:228 tgid:= 228 ppid:222 task_flags:0x400140 flags:0x00000818 >>> [ 85.625823] Call trace: >>> [ 85.628316] __switch_to+0xe8/0x1a0 (T) >>> [ 85.632217] __schedule+0x290/0x7c0 >>> [ 85.635765] schedule+0x34/0x118 >>> [ 85.639044] async_synchronize_cookie_domain.part.0+0x50/0xa4 >>> [ 85.644854] async_synchronize_full+0x78/0xa0 >>> [ 85.649270] do_init_module+0x190/0x23c >>> [ 85.653154] load_module+0x1708/0x1ca0 >>> [ 85.656952] init_module_from_file+0x74/0xa0 >>> [ 85.661273] __arm64_sys_finit_module+0x130/0x2f8 >>> [ 85.666023] invoke_syscall+0x48/0x104 >>> [ 85.669842] el0_svc_common.constprop.0+0xc0/0xe0 >>> [ 85.674604] do_el0_svc+0x1c/0x28 >>> [ 85.677973] el0_svc+0x2c/0x84 >>> [ 85.681078] el0t_64_sync_handler+0xa0/0xe4 >>> [ 85.685316] el0t_64_sync+0x198/0x19c >>> [ 85.689032] INFO: task (udev-worker):229 blocked for more than 42 se= conds. >>> >>> >>> Usually wifi, all remoteprocs and anything that depends on lpass/pinctr= l fail to probe. >>=20 >> May i know what is testcase which you are running on target? > > Boot the board? > >> what is target? > > It is written in original report. Did you even read it? > >> Which usecase is this issue occurring in? > > Boot? FWIW, what said above by Krzysztof is correct, there is no usecase, just bo= oting the board. Best regards, Alexey