Linux Test Project
 help / color / mirror / Atom feed
From: Wake Liu via ltp <ltp@lists.linux.it>
To: Li Wang <li.wang@linux.dev>
Cc: Wake Liu <wakel@google.com>,
	ltp@lists.linux.it, linuxtestproject.agent@gmail.com
Subject: Re: [LTP] lib: Use backoff polling to wait for loop device nodes
Date: Mon, 10 Aug 2026 04:16:49 +0000	[thread overview]
Message-ID: <20260810041649.1714200-1-wakel@google.com> (raw)
In-Reply-To: <anlLyuZW-8dHISLB@linux.dev>

Hi Li,

Thanks for the review.

I agree that LTP_TIMEOUT_MUL=3000 is an extreme case and unlikely to happen in practice.

However, since we already sent v7 which fixes this overflow cleanly (using 'unsigned long long' in the macro to prevent the wrap-around), and the AI reviewer has already passed it (Verdict - Reviewed), I think it's worth having the more robust version in v7.

Please take a look at v7.

Thanks,
Wake

-- 
Mailing list info: https://lists.linux.it/listinfo/ltp

  reply	other threads:[~2026-08-10  4:17 UTC|newest]

Thread overview: 28+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-08-03 15:27 [LTP] [PATCH] lib: Use exponential-backoff polling to wait for loop device nodes Wake Liu via ltp
2026-08-03 16:28 ` [LTP] " linuxtestproject.agent
2026-08-04  1:59 ` [LTP] [PATCH v2] lib: Use backoff " Wake Liu via ltp
2026-08-04  3:58   ` Li Wang
2026-08-04 15:24     ` [LTP] [PATCH v3] " Wake Liu via ltp
2026-08-05  2:24       ` Li Wang
2026-08-05  2:54         ` [LTP] [PATCH v4] " Wake Liu via ltp
2026-08-05  4:09           ` [LTP] " linuxtestproject.agent
2026-08-05  6:53             ` Li Wang
2026-08-05  6:35           ` [LTP] [PATCH v4] " Li Wang
2026-08-05  6:52           ` Andrea Cervesato via ltp
2026-08-05  6:55           ` Andrea Cervesato via ltp
2026-08-05  7:10             ` Li Wang
2026-08-07 10:04               ` [LTP] [PATCH v5] " Wake Liu via ltp
2026-08-07 11:05                 ` [LTP] " linuxtestproject.agent
2026-08-07 23:49                   ` [LTP] [PATCH v6] " Wake Liu via ltp
2026-08-08  1:32                     ` [LTP] " linuxtestproject.agent
2026-08-10  3:55                       ` Li Wang
2026-08-10  4:16                         ` Wake Liu via ltp [this message]
2026-08-10  2:58                     ` [LTP] [PATCH v7 0/2] Fix overflow and use backoff polling for loop devices Wake Liu via ltp
2026-08-10  2:58                       ` [LTP] [PATCH v7 1/2] lib: Fix overflow in backoff polling and timeout multiplication Wake Liu via ltp
2026-08-10  3:56                         ` [LTP] " linuxtestproject.agent
2026-08-10  9:17                         ` [LTP] [PATCH v7 1/2] " Andrea Cervesato via ltp
2026-08-10 10:02                         ` Li Wang
2026-08-10  2:58                       ` [LTP] [PATCH v7 2/2] lib: Use backoff polling to wait for loop device nodes Wake Liu via ltp
2026-08-10  9:17                         ` Andrea Cervesato via ltp
2026-08-10 10:10                     ` [LTP] [PATCH v6] " Li Wang
2026-08-10 12:30                       ` Wake Liu via ltp

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=20260810041649.1714200-1-wakel@google.com \
    --to=ltp@lists.linux.it \
    --cc=li.wang@linux.dev \
    --cc=linuxtestproject.agent@gmail.com \
    --cc=wakel@google.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