On 6/2/25 19:31, Ilpo Järvinen wrote:
- mutex_lock(&tty_mutex);
Use guard() so you don't need to change the returns and rollback path.
Thanks, I didn't know about this new kind of helper.
I'll leave it up to the TTY maintainers - if they don't express a preference for guard(), then I deem this code simple enough to leave it as-is, because I don't have any experience with guard(), and in fact, until 5 minutes ago, I didn't know at all that GCC cleanup attributes even exist.
Interestingly, Documentation/process/maintainer-netdev.rst documents a preference against guard(). I wonder why, but that's for another day.
Do you have an idea on how to solve the circular lock that the kernel test robot found for v1 of this patch?
https://lore.kernel.org/linux-serial/202505281412.8c836cb7-lkp@intel.com/
Max