* Sashiko reviews for the LSM mailing list
@ 2026-06-15 15:37 Mickaël Salaün
2026-06-15 20:13 ` Paul Moore
0 siblings, 1 reply; 2+ messages in thread
From: Mickaël Salaün @ 2026-06-15 15:37 UTC (permalink / raw)
To: linux-security-module; +Cc: sashiko, Roman Gushchin, Günther Noack
Hi,
I've been reading Sashiko's (AI bot) reviews wrt Landlock patches, and
most of them were valuable. It found issues (security or not), but it
requires to go to https://sashiko.dev to find them, which is too easy to
forget, and requires additional work from maintainers to copy or point
to these reviews. I sent a PR (currently in draft) to enable email
replies from Sashiko to the Linux Security Module mailing list (most
patches are already reviewed anyway):
https://github.com/sashiko-dev/sashiko/pull/278
Making such reviews broadly available can improve the quality of patches
we receive without much noise, helping for all LSM-related code. We can
fine tune some email-related settings if needed.
If there are any concern or question, this is the right time to start a
discussion.
Regards,
Mickaël
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: Sashiko reviews for the LSM mailing list
2026-06-15 15:37 Sashiko reviews for the LSM mailing list Mickaël Salaün
@ 2026-06-15 20:13 ` Paul Moore
0 siblings, 0 replies; 2+ messages in thread
From: Paul Moore @ 2026-06-15 20:13 UTC (permalink / raw)
To: Mickaël Salaün
Cc: linux-security-module, sashiko, Roman Gushchin,
Günther Noack
On Mon, Jun 15, 2026 at 11:41 AM Mickaël Salaün <mic@digikod.net> wrote:
>
> Hi,
>
> I've been reading Sashiko's (AI bot) reviews wrt Landlock patches, and
> most of them were valuable. It found issues (security or not), but it
> requires to go to https://sashiko.dev to find them, which is too easy to
> forget, and requires additional work from maintainers to copy or point
> to these reviews. I sent a PR (currently in draft) to enable email
> replies from Sashiko to the Linux Security Module mailing list (most
> patches are already reviewed anyway):
> https://github.com/sashiko-dev/sashiko/pull/278
>
> Making such reviews broadly available can improve the quality of patches
> we receive without much noise, helping for all LSM-related code. We can
> fine tune some email-related settings if needed.
>
> If there are any concern or question, this is the right time to start a
> discussion.
I recently enabled Sashiko for the SELinux list to trial it there
first, with the goal of eventually bringing this topic up for the rest
of the LSM folks on the LSM list.
While I think Sashiko's review comments are generally okay, you should
have contacted the LSM mailing list folks *before* submitting a PR
that would cause an automated bot to send email to the LSM list (this
applies to all automated emails, not just LLM reviews). Please hold
the PR until you have given people a chance to comment on the issue.
Personally, I'm okay with it.
--
paul-moore.com
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2026-06-15 20:14 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-06-15 15:37 Sashiko reviews for the LSM mailing list Mickaël Salaün
2026-06-15 20:13 ` Paul Moore
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox