From: "luo.liu.linux" <luo.liu.linux@163.com>
To: "Sakari Ailus" <sakari.ailus@linux.intel.com>
Cc: mchehab@kernel.org, linux-media@vger.kernel.org,
linux-kernel@vger.kernel.org
Subject: Re:Re: Re: Re: Re: [PATCH] media:v4l2-async:debugfs for registered subdevices
Date: Tue, 17 Mar 2026 19:14:43 +0800 (CST) [thread overview]
Message-ID: <4155da3e.9361.19cfb815615.Coremail.luo.liu.linux@163.com> (raw)
In-Reply-To: <abkPC3sHVUdGuww1@kekkonen.localdomain>
Hi Sakari,
The existing pending_async_subdevices interface provides excellent visibility into the notifier_list (the 'waiter' side).
To achieve full symmetry and complete debuggability, we should also expose the subdev_list (the 'provider' side).These two views solve different problems:
1 Notifier List: Diagnoses why binding is stalled (missing sub-devices).
2 Subdev List: Diagnoses state inconsistencies (e.g., sub-devices present but unmatched) and verifies resource cleanup upon unbind.
From practical experience, lacking visibility into subdev_list makes it difficult to distinguish between a sub-device probe failure and an async matching failure.
Adding this interface would provide a holistic view of the async engine's state, which has proven essential for rapid issue localization in complex driver stacks.
Kind regards,
Luo
At 2026-03-17 16:21:31, "Sakari Ailus" <sakari.ailus@linux.intel.com> wrote:
>Hi Luo,
>
>On Tue, Mar 17, 2026 at 11:37:52AM +0800, luo.liu.linux wrote:
>>
>>
>> Hi Sakari,
>>
>> You are absolutely right. For an experienced kernel developer like
>> yourself, tools like KASAN and CONFIG_DEBUG_LIST are second nature and
>> incredibly effective for pinpointing such issues. I truly admire your
>> expertise in leveraging these advanced debugging mechanisms.
>>
>> However, I think it is important to consider the reality for many
>> junior driver developers (myself included). We often lack the deep
>> intuition and extensive experience required to wield these powerful tools
>> effectively in every scenario. More often than not, we still rely on
>> primitive methods: struggling to reproduce intermittent crashes,
>> scattering printk logs everywhere, and manually tracing execution paths.
>> This process is extremely time-consuming and often yields no clear
>> conclusions for "silent" resource leaks.
>>
>> While I am actively working to improve my skills and learn to use
>> these advanced tools more proficiently. I remain convinced that providing
>> such a simple, intuitive interface offers a necessary supplement by
>> serving as a low-barrier entry point for developers.
>>
>> I hope this perspective clarifies why I believe this small change
>> can bring a bit of convenience to a broader range of driver developers.
>
>Just enable KASAN and list debugging in the future. New interfaces like
>this won't improve things at large.
>
>--
>Kind regards,
>
>Sakari Ailus
next prev parent reply other threads:[~2026-03-17 11:14 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-03-13 7:58 luo.liu
2026-03-13 10:28 ` Sakari Ailus
2026-03-13 11:21 ` luo.liu.linux
2026-03-13 11:42 ` Sakari Ailus
2026-03-13 13:50 ` luo.liu.linux
2026-03-16 16:48 ` Sakari Ailus
2026-03-17 3:37 ` luo.liu.linux
2026-03-17 8:21 ` Sakari Ailus
2026-03-17 11:14 ` luo.liu.linux [this message]
2026-03-19 20:30 ` Laurent Pinchart
2026-03-20 6:52 ` luo.liu.linux
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=4155da3e.9361.19cfb815615.Coremail.luo.liu.linux@163.com \
--to=luo.liu.linux@163.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-media@vger.kernel.org \
--cc=mchehab@kernel.org \
--cc=sakari.ailus@linux.intel.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
Powered by JetHome