From: Julian Braha <julianbraha@gmail.com>
To: Alexander Gordeev <agordeev@linux.ibm.com>
Cc: gor@linux.ibm.com, hca@linux.ibm.com, iii@linux.ibm.com,
meted@linux.ibm.com, borntraeger@linux.ibm.com,
svens@linux.ibm.com, akpm@linux-foundation.org,
linux-s390@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [RESEND PATCH] s390: fix dead defaults for S390_MODULES_SANITY_TEST and S390_UNWIND_SELFTEST
Date: Fri, 8 May 2026 13:35:34 +0100 [thread overview]
Message-ID: <27bc3de0-7006-4ced-a53c-9fb0d3e760f9@gmail.com> (raw)
In-Reply-To: <5bd572e5-78c6-4f6e-ba95-4e79a8ef8341-agordeev@linux.ibm.com>
On 5/8/26 12:22, Alexander Gordeev wrote:
> I could not recreate it. Could you please provide the exact steps?
Sure, here are the exact commands I ran to reproduce for
S390_UNWIND_SELFTEST. Note that I'm on x86-64 and cross-compiling for
s390:
1. Start from a minimal configuration:
'make ARCH=s390 CROSS_COMPILE=s390x-unknown-linux-gnu- allnoconfig'
2. Enter menuconfig:
'make ARCH=s390 CROSS_COMPILE=s390x-unknown-linux-gnu- menuconfig'
3. Set CONFIG_KUNIT=y
4. Set CONFIG_KUNIT_ALL_TESTS=y
5. Check '.config', and see that S390_UNWIND_SELFTEST is still disabled
The steps to reproduce for S390_MODULES_SANITY_TEST are the same, but
you also need to enable MODULES because of the additional dependency.
> I do appreciate your effort to bring a new tool! But unless it is part
> of
> the toolchain it shold not be referred to in a commit message as far as
> I am concerned. IOW, once it is in the tree, it is fine to mention it:
Thanks :)
Separately, I checked with Greg, and he says that:
1. it is a requirement to acknowledge tool usage, but
2. this patch (and others like it) shouldn't have the Fixes tag.
So I will resubmit this patch without "Fixes", but still acknowledging
the tool, see also:
https://lore.kernel.org/all/2026050728-construct-tingle-0852@gregkh/
- Julian Braha
prev parent reply other threads:[~2026-05-08 12:35 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-04-14 21:56 [RESEND PATCH] s390: fix dead defaults for S390_MODULES_SANITY_TEST and S390_UNWIND_SELFTEST Julian Braha
2026-04-29 14:42 ` Alexander Gordeev
2026-04-29 21:39 ` Julian Braha
2026-05-06 12:37 ` Alexander Gordeev
2026-05-06 16:51 ` Julian Braha
2026-05-08 11:22 ` Alexander Gordeev
2026-05-08 12:35 ` Julian Braha [this message]
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=27bc3de0-7006-4ced-a53c-9fb0d3e760f9@gmail.com \
--to=julianbraha@gmail.com \
--cc=agordeev@linux.ibm.com \
--cc=akpm@linux-foundation.org \
--cc=borntraeger@linux.ibm.com \
--cc=gor@linux.ibm.com \
--cc=hca@linux.ibm.com \
--cc=iii@linux.ibm.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-s390@vger.kernel.org \
--cc=meted@linux.ibm.com \
--cc=svens@linux.ibm.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