All of lore.kernel.org
 help / color / mirror / Atom feed
From: Li Wang via ltp <ltp@lists.linux.it>
To: Petr Vorel <pvorel@suse.cz>
Cc: Sebastian Chlad <schlad@suse.de>,
	ltp@lists.linux.it, Martin Doucha <martin.doucha@suse.com>
Subject: Re: [LTP] LTP old API conversion
Date: Wed, 18 Mar 2026 13:36:02 +0800	[thread overview]
Message-ID: <abo5wgYIYzo9_yo0@redhat.com> (raw)
In-Reply-To: <20260317223920.GA370623@pevik>

On Tue, Mar 17, 2026 at 11:39:20PM +0100, Petr Vorel wrote:
> > Hi Petr,
> 
> > > I'd vote for limiting patchsets to max tests in a single directory.
> > > Why? Smaller patchset is easier to review. And if just some of the commits are
> > > accepted then fewer commits need to be rebased.
> 
> > Do you mean one patch-set for multiple (unrelated) tests (for instance syscalls
> > folder) [1], or one patch-set per testing suite [2] ?
> 
> I meant [2], i.e. tests from certain folder (usually tests related to single syscall).
> But it's just a suggestion, feel free to do it differently if it's significantly
> simpler.

Some tests may relocate their folder after the refacoring.

Regardless of which method we ultimately adopt, I recommend that
the AI send no more than 10 patches per week [1].

Furthermore, new patches should be sent only after the previous
patchset has been fully processed.

[1] Given that the total number of legacy tests is only 242,
the cleanup work will not take much time.

-- 
Regards,
Li Wang


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

  reply	other threads:[~2026-03-18  5:36 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-03-17 10:42 [LTP] LTP old API conversion Andrea Cervesato via ltp
2026-03-17 11:58 ` Petr Vorel
2026-03-17 12:14   ` Andrea Cervesato via ltp
2026-03-17 22:39     ` Petr Vorel
2026-03-18  5:36       ` Li Wang via ltp [this message]
2026-03-17 15:32 ` Cyril Hrubis
2026-03-17 15:49   ` Andrea Cervesato via ltp
2026-03-18 13:04     ` Andrea Cervesato via ltp
2026-03-18  5:18   ` Li Wang 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=abo5wgYIYzo9_yo0@redhat.com \
    --to=ltp@lists.linux.it \
    --cc=liwang@redhat.com \
    --cc=martin.doucha@suse.com \
    --cc=pvorel@suse.cz \
    --cc=schlad@suse.de \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.