From: sashiko-bot@kernel.org
To: "Matthias Goergens" <matthias.goergens@gmail.com>
Cc: "Oliver Upton" <oupton@kernel.org>,
kvmarm@lists.linux.dev, "Marc Zyngier" <maz@kernel.org>
Subject: Re: [RFC PATCH] Documentation/process: Add a maintainer entry profile for KVM/arm64
Date: Fri, 25 Sep 2026 08:58:45 +0000 [thread overview]
Message-ID: <20260925085846.501EE1F000FF@smtp.kernel.org> (raw)
In-Reply-To: <20260925085621.562448-1-matthias.goergens@gmail.com>
Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
- [Low] docs: unresolved draft placeholders in maintainer profile
--
commit dc2a38b3be99b9dc53f4e7a7537844707bf4e7ca
Author: Matthias Goergens <matthias.goergens@gmail.com>
Subject: Documentation/process: Add a maintainer entry profile for KVM/arm64
This commit introduces a new maintainer profile for the KVM/arm64 subsystem.
It details the development trees, patch bases, review cadence, and testing
requirements for contributors.
Link: https://lore.kernel.org/all/875wzu4vuv.wl-maz@kernel.org/
> diff --git a/Documentation/process/maintainer-kvm-arm64.rst b/Documentation/process/maintainer-kvm-arm64.rst
> new file mode 100644
> index 0000000000000..deba2f812cef3
> --- /dev/null
> +++ b/Documentation/process/maintainer-kvm-arm64.rst
[ ... ]
> +``fixes``
> + Fixes for the release currently in its -rc phase. linux-next merges this
> + branch too.
> +
> +[?: ``master`` still points at a 2020 commit. Once it follows -rc1, should
> +this document mention it?]
[Severity: Low]
This isn't a bug, but should this draft placeholder regarding the master branch
be resolved and removed before the documentation is merged?
> +
> +Changes leave the tree as signed tags, which are pulled into the main KVM tree
> +(``git://git.kernel.org/pub/scm/virt/kvm/kvm.git``) and from there reach Linus
> +Torvalds.
> +
> +Unlike the KVM x86 tree, KVM/arm64 has no standing topic branches for
> +contributors to base their work on. Where a series also touches code
> +maintained elsewhere, most often the arm64 architecture code, the maintainers
> +may set up a shared stable branch, based on an -rc tag, that both trees merge.
> +Say in the cover letter which parts of a series touch other subsystems.
> +[?: Is this an accurate description of topic branches and of how
> +cross-tree changes are handled?]
[Severity: Low]
Should this question about topic branches and cross-tree changes be answered
and the text finalized rather than keeping the placeholder in the document?
> +
> +Base for patches
> +~~~~~~~~~~~~~~~~
[ ... ]
> +Key Cycle Dates
> +---------------
> +
> +Fixes for the current release are queued on ``fixes`` and sent throughout
> +the -rc phase. Changes for the next merge window are queued on ``next``.
> +
> +[?: What are the cut-offs: the last -rc for submitting new features, and the
> +last -rc at which the maintainers decide what goes into the next merge
> +window?]
[Severity: Low]
Should this placeholder regarding merge window cut-offs be updated with the
actual deadlines before the patch is accepted?
--
Sashiko AI review · https://sashiko.dev/#/patchset/20260925085621.562448-1-matthias.goergens@gmail.com?part=1
next prev parent reply other threads:[~2026-09-25 8:58 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-09-24 3:35 [PATCH] MAINTAINERS: name the kvmarm/kvmarm next branch Matthias Goergens
2026-09-24 6:59 ` Marc Zyngier
2026-09-24 8:30 ` Matthias Goergens
2026-09-24 13:33 ` Marc Zyngier
2026-09-25 8:56 ` [RFC PATCH] Documentation/process: Add a maintainer entry profile for KVM/arm64 Matthias Goergens
2026-09-25 8:58 ` sashiko-bot [this message]
2026-09-27 10:22 ` Marc Zyngier
2026-10-05 12:10 ` Oliver Upton
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=20260925085846.501EE1F000FF@smtp.kernel.org \
--to=sashiko-bot@kernel.org \
--cc=kvmarm@lists.linux.dev \
--cc=matthias.goergens@gmail.com \
--cc=maz@kernel.org \
--cc=oupton@kernel.org \
--cc=sashiko-reviews@lists.linux.dev \
/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.