From: Alexander Graf <graf@amazon.com>
To: Sasha Levin <sashal@kernel.org>
Cc: "Thomas Gleixner" <tglx@kernel.org>,
x86@kernel.org, linux-kernel@vger.kernel.org,
linux-doc@vger.kernel.org, "Clemens Ladisch" <clemens@ladisch.de>,
"Arnd Bergmann" <arnd@arndb.de>,
"Greg Kroah-Hartman" <gregkh@linuxfoundation.org>,
"Dave Hansen" <dave.hansen@linux.intel.com>,
"Borislav Petkov" <bp@alien8.de>,
"Ingo Molnar" <mingo@redhat.com>,
"Jonathan Corbet" <corbet@lwn.net>,
"Paolo Bonzini" <pbonzini@redhat.com>,
"Pasha Tatashin" <pasha.tatashin@soleen.com>,
nh-open-source@amazon.com,
"Nicolas Saenz Julienne" <nsaenz@amazon.es>,
"Hendrik Borghorst" <hborghor@amazon.de>,
"Filippo Sironi" <sironi@amazon.de>,
"David Woodhouse" <dwmw@amazon.co.uk>,
"Jan Schönherr" <jschoenh@amazon.de>,
ricardo.neri-calderon@linux.intel.com
Subject: Re: [PATCH 2/2] hpet: Add HPET-based NMI watchdog support
Date: Tue, 3 Feb 2026 18:19:16 +0100 [thread overview]
Message-ID: <1d74c3cc-d125-49cb-af05-43ca7d15f3f8@amazon.com> (raw)
In-Reply-To: <aYIhJWiBtiCF4UCr@laps>
On 03.02.26 17:24, Sasha Levin wrote:
> On Tue, Feb 03, 2026 at 01:36:30PM +0100, Alexander Graf wrote:
>>
>> On 03.02.26 11:32, Thomas Gleixner wrote:
>>> On Mon, Feb 02 2026 at 17:48, Alexander Graf wrote:
>>>> (Disclaimer: Some of this code was written with the help of Kiro,
>>>> an AI
>>>> coding assistant)
>>> You could have sent your change log through AI too so it conforms with
>>> the change log rules ...
>>
>>
>> Maybe we should introduce an AGENTS.md file in Linux that tells the AI
>> tool to do that automatically? These tools usually don't read README
>> files. :)
>>
>> Looks like - similar to the HPET watchdog - that never concluded though:
>>
>> https://lore.kernel.org/lkml/20250813203647.06e49600@gandalf.local.home/
>>
>> Sasha, are you going to resend your @README commit with a single
>> AGENTS.md? FWIW that is pretty much what everything standardized on by
>> now.
>
> Out of curiosity, can you test your coding assistant on a tree with
> https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/commit/Documentation?id=78d979db6cef557c171d6059cbce06c3db89c7ee
>
> applied on top?
>
> From my previous testing, the coding assistants I tried it with went
> to the
> README and DTRT. If that's not the case I'm happy to respin the
> AGENTS.md idea,
> even if it just explicitly points to the README.
Kiro does not seem to read README automatically. I spun up kiro-cli and
gave it this prompt: "Write "Hello World" before invoking the init
process. Then create a descriptive git commit for the change.". No
Assisted-by: tag, so it did not properly read the README.
I tried the same with an AGENTS.md file present that contains "@README"
and it gave me effectively the same result. Same for a symlink from
AGENTS.md to README.
I think it just never really jumped to the conclusion that it should
read further than just the AGENTS.md file and also ingest the rst,
effectively ignoring the section's instructions. Or maybe it actually
reads the .rst and ignores its contents? At least it does read it
according to strace, even without an AGENTS.md file.
Let me file a bug report with Kiro.
Alex
Amazon Web Services Development Center Germany GmbH
Tamara-Danz-Str. 13
10243 Berlin
Geschaeftsfuehrung: Christof Hellmis, Andreas Stieger
Eingetragen am Amtsgericht Charlottenburg unter HRB 257764 B
Sitz: Berlin
Ust-ID: DE 365 538 597
next prev parent reply other threads:[~2026-02-03 17:19 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-02-02 17:48 [PATCH 0/2] Add HPET NMI Watchdog support Alexander Graf
2026-02-02 17:48 ` [PATCH 1/2] x86/ioapic: Add NMI delivery configuration helper Alexander Graf
2026-02-03 10:08 ` Thomas Gleixner
2026-02-03 10:44 ` Alexander Graf
2026-02-03 10:45 ` David Woodhouse
2026-02-02 17:48 ` [PATCH 2/2] hpet: Add HPET-based NMI watchdog support Alexander Graf
2026-02-03 10:32 ` Thomas Gleixner
2026-02-03 12:36 ` Alexander Graf
2026-02-03 15:28 ` Thomas Gleixner
2026-02-03 19:44 ` Ricardo Neri
2026-02-03 20:49 ` Thomas Gleixner
2026-02-04 5:05 ` Ricardo Neri
2026-02-03 16:24 ` Sasha Levin
2026-02-03 17:19 ` Alexander Graf [this message]
2026-02-03 17:43 ` David Woodhouse
2026-02-03 20:46 ` Thomas Gleixner
2026-02-03 23:13 ` David Woodhouse
2026-02-04 10:34 ` Thomas Gleixner
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=1d74c3cc-d125-49cb-af05-43ca7d15f3f8@amazon.com \
--to=graf@amazon.com \
--cc=arnd@arndb.de \
--cc=bp@alien8.de \
--cc=clemens@ladisch.de \
--cc=corbet@lwn.net \
--cc=dave.hansen@linux.intel.com \
--cc=dwmw@amazon.co.uk \
--cc=gregkh@linuxfoundation.org \
--cc=hborghor@amazon.de \
--cc=jschoenh@amazon.de \
--cc=linux-doc@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mingo@redhat.com \
--cc=nh-open-source@amazon.com \
--cc=nsaenz@amazon.es \
--cc=pasha.tatashin@soleen.com \
--cc=pbonzini@redhat.com \
--cc=ricardo.neri-calderon@linux.intel.com \
--cc=sashal@kernel.org \
--cc=sironi@amazon.de \
--cc=tglx@kernel.org \
--cc=x86@kernel.org \
/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
all inboxes | Powered by JetHome®