From: Andrea Cervesato via ltp <ltp@lists.linux.it>
To: linuxtestproject.agent@gmail.com
Cc: ltp@lists.linux.it
Subject: Re: [LTP] tse_swap: Add fallback definitions for swapon
Date: Fri, 04 Sep 2026 07:16:51 +0000 [thread overview]
Message-ID: <6a9a7064.18b7d65e.17573c.2e31@mx.google.com> (raw)
In-Reply-To: <20260903160137.8963-1-linuxtestproject.agent@gmail.com>
> Hi Andrea,
>
> On Thu, Sep 3, 2026, Andrea Cervesato wrote:
> > [PATCH 1/3] tse_swap: Add fallback definitions for swapon
>
> --- [PATCH 2/3] ---
>
> > static struct tst_test test = {
> > .needs_root = 1,
> > .mount_device = 1,
>
> SWAP_FLAG_DISCARD_ONCE and SWAP_FLAG_DISCARD_PAGES were added in Linux
> 3.11. Add `.min_kver = "3.11"`; older kernels reject these bits and the
> test reports TFAIL instead of TCONF.
>
> --- [PATCH 3/3] ---
>
> > {
> > .err_desc = "Invalid swapflags (undefined flag bits)",
> > .exp_errno = EINVAL,
> > .path = SWAP_FILE,
> > .flags = 0x80000,
> > },
>
> Invalid swap flags started returning EINVAL in Linux 3.4. Add
> `.min_kver = "3.4"` so the test skips older kernels, which may accept the
> flags and produce spurious failures.
>
> Verdict - Needs revision
>
> ---
> Note:
>
> The agent can sometimes produce false positives although often its
> findings are genuine. If you find issues with the review, please
> comment this email or ignore the suggestions.
>
> Regards,
> LTP AI Reviewer
both can be ignored, we don't support those kernels. I will add a
rule to the Agent so we remove this once for all.
--
Andrea Cervesato
SUSE QE Automation Engineer Linux
andrea.cervesato@suse.com
--
Mailing list info: https://lists.linux.it/listinfo/ltp
next prev parent reply other threads:[~2026-09-04 7:17 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-09-03 13:04 [LTP] [PATCH v4 0/3] Increase coverage for swapon syscall Andrea Cervesato
2026-09-03 13:04 ` [LTP] [PATCH v4 1/3] tse_swap: Add fallback definitions for swapon Andrea Cervesato
2026-09-03 16:01 ` [LTP] " linuxtestproject.agent
2026-09-04 7:16 ` Andrea Cervesato via ltp [this message]
2026-09-10 11:47 ` [LTP] [PATCH v4 1/3] " Cyril Hrubis
2026-09-03 13:04 ` [LTP] [PATCH v4 2/3] swapon04: Add test for discard flags Andrea Cervesato
2026-09-10 14:21 ` Cyril Hrubis
2026-09-03 13:04 ` [LTP] [PATCH v4 3/3] swapon02: Add test cases for invalid swapflags Andrea Cervesato
2026-09-10 14:29 ` Cyril Hrubis
2026-09-11 6:33 ` [LTP] [PATCH v4 0/3] Increase coverage for swapon syscall Andrea Cervesato via ltp
-- strict thread matches above, loose matches on Subject: below --
2026-08-28 11:10 [LTP] [PATCH v3 1/3] tse_swap: Add fallback definitions for swapon Andrea Cervesato
2026-08-28 11:30 ` [LTP] " linuxtestproject.agent
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=6a9a7064.18b7d65e.17573c.2e31@mx.google.com \
--to=ltp@lists.linux.it \
--cc=andrea.cervesato@suse.com \
--cc=linuxtestproject.agent@gmail.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 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.