linux-kernel - [PATCH 0/8] media: use 'time_left' instead of 'timeout' with wait

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [thread-next>] [day] [month] [year] [list]

Message-ID: <20240603092841.9500-1-wsa+renesas@sang-engineering.com>
Date: Mon,  3 Jun 2024 11:28:31 +0200
From: Wolfram Sang <wsa+renesas@...g-engineering.com>
To: linux-kernel@...r.kernel.org
Cc: Wolfram Sang <wsa+renesas@...g-engineering.com>,
	Alexandre Belloni <alexandre.belloni@...tlin.com>,
	Andrey Utkin <andrey_utkin@...tmail.com>,
	Anton Sviridenko <anton@...p.bluecherry.net>,
	Benoit Parrot <bparrot@...com>,
	Bluecherry Maintainers <maintainers@...echerrydvr.com>,
	Claudiu Beznea <claudiu.beznea@...on.dev>,
	Dmitry Osipenko <digetx@...il.com>,
	Eugen Hristev <eugen.hristev@...labora.com>,
	Fabien Dessenne <fabien.dessenne@...s.st.com>,
	Ismael Luceno <ismael@...ev.co.uk>,
	Jonathan Hunter <jonathanh@...dia.com>,
	Krzysztof Kozlowski <krzk@...nel.org>,
	linux-arm-kernel@...ts.infradead.org,
	linux-media@...r.kernel.org,
	linux-samsung-soc@...r.kernel.org,
	linux-tegra@...r.kernel.org,
	Mauro Carvalho Chehab <mchehab@...nel.org>,
	Michael Tretter <m.tretter@...gutronix.de>,
	Nicolas Ferre <nicolas.ferre@...rochip.com>,
	Sylwester Nawrocki <s.nawrocki@...sung.com>,
	Thierry Reding <thierry.reding@...il.com>
Subject: [PATCH 0/8] media: use 'time_left' instead of 'timeout' with wait_*() functions

There is a confusing pattern in the kernel to use a variable named 'timeout' to
store the result of wait_*() functions causing patterns like:

        timeout = wait_for_completion_timeout(...)
        if (!timeout) return -ETIMEDOUT;

with all kinds of permutations. Use 'time_left' as a variable to make the code
obvious and self explaining.

This is part of a tree-wide series. The rest of the patches can be found here:

git://git.kernel.org/pub/scm/linux/kernel/git/wsa/linux.git i2c/time_left

Because these patches are generated, I audit them before sending. This is why I
will send series step by step. Build bot is happy with these patches, though.
No functional changes intended.

Wolfram Sang (8):
  media: allegro: use 'time_left' variable with
    wait_for_completion_timeout()
  media: atmel-isi: use 'time_left' variable with
    wait_for_completion_timeout()
  media: bdisp: use 'time_left' variable with wait_event_timeout()
  media: fimc-is: use 'time_left' variable with wait_event_timeout()
  media: platform: exynos-gsc: use 'time_left' variable with
    wait_event_timeout()
  media: solo6x10: use 'time_left' variable with
    wait_for_completion_timeout()
  media: tegra-vde: use 'time_left' variable with
    wait_for_completion_interruptible_timeout()
  media: ti: cal: use 'time_left' variable with wait_event_timeout()

 drivers/media/pci/solo6x10/solo6x10-p2m.c        |  8 ++++----
 .../media/platform/allegro-dvt/allegro-core.c    | 16 ++++++++--------
 drivers/media/platform/atmel/atmel-isi.c         |  8 ++++----
 drivers/media/platform/nvidia/tegra-vde/h264.c   | 10 +++++-----
 .../media/platform/samsung/exynos-gsc/gsc-core.c | 10 +++++-----
 .../platform/samsung/exynos4-is/fimc-core.c      | 10 +++++-----
 drivers/media/platform/st/sti/bdisp/bdisp-v4l2.c | 10 +++++-----
 drivers/media/platform/ti/cal/cal.c              |  8 ++++----
 8 files changed, 40 insertions(+), 40 deletions(-)

-- 
2.43.0