From: Alice Ryhl <aliceryhl@google.com>
To: Paul Moore <paul@paul-moore.com>
Cc: Mark Brown <broonie@kernel.org>, Greg KH <greg@kroah.com>,
Arnd Bergmann <arnd@arndb.de>,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
Jahnavi MN <jahnavimn@google.com>, Jann Horn <jannh@google.com>,
Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
Linux Next Mailing List <linux-next@vger.kernel.org>
Subject: Re: linux-next: manual merge of the char-misc tree with the security tree
Date: Tue, 21 Jul 2026 08:18:25 +0000 [thread overview]
Message-ID: <al8rUe1Jh4CMGTpu@google.com> (raw)
In-Reply-To: <CAHC9VhQAYWzzcd6KnbCg+nNAfSHVBetgOnzd5axnL2_6E+wL5g@mail.gmail.com>
On Mon, Jul 20, 2026 at 06:45:23PM -0400, Paul Moore wrote:
> On Mon, Jul 20, 2026 at 10:48 AM Mark Brown <broonie@kernel.org> wrote:
> >
> > Hi all,
> >
> > Today's linux-next merge of the char-misc tree got a conflict in:
> >
> > rust/kernel/task.rs
> >
> > between commits:
> >
> > ef2d3e4635761 ("rust: task: clarify comments on task UID accessors")
> > 15c1f17979712 ("cred: delete task_euid()")
> >
> > from the security tree and commit:
> >
> > 2847d9ab088bb ("rust_binder: Add dynamic debug logging mask")
> >
> > from the char-misc tree.
> >
> > I fixed it up (see below) and can carry the fix as necessary. This
> > is now fixed as far as linux-next is concerned, but any non trivial
> > conflicts should be mentioned to your upstream maintainer when your tree
> > is submitted for merging. You may also want to consider cooperating
> > with the maintainer of the conflicting tree to minimise any particularly
> > complex conflicts.
> >
> > diff --cc rust/kernel/task.rs
> > index c2b3457b700c1,1b290c61714db..0000000000000
> > --- a/rust/kernel/task.rs
> > +++ b/rust/kernel/task.rs
> > @@@ -210,7 -210,14 +210,14 @@@ impl Task
> > unsafe { *ptr::addr_of!((*self.as_ptr()).pid) }
> > }
> >
> > + /// Returns the TGID (Thread Group ID / Process ID) of the given task.
> > + pub fn tgid(&self) -> Pid {
> > + // SAFETY: The tgid of a task never changes after initialization, so reading this field is
> > + // not a data race.
> > + unsafe { *ptr::addr_of!((*self.as_ptr()).tgid) }
> > + }
> > +
> > - /// Returns the UID of the given task.
> > + /// Returns the objective real UID of the given task.
> > #[inline]
> > pub fn uid(&self) -> Kuid {
> > // SAFETY: It's always safe to call `task_uid` on a valid task.
>
> Thanks Mark, that fixup looks reasonable to me.
Looks good to me as well.
Alice
prev parent reply other threads:[~2026-07-21 8:18 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-07-20 14:48 linux-next: manual merge of the char-misc tree with the security tree Mark Brown
2026-07-20 22:45 ` Paul Moore
2026-07-21 8:18 ` Alice Ryhl [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=al8rUe1Jh4CMGTpu@google.com \
--to=aliceryhl@google.com \
--cc=arnd@arndb.de \
--cc=broonie@kernel.org \
--cc=greg@kroah.com \
--cc=gregkh@linuxfoundation.org \
--cc=jahnavimn@google.com \
--cc=jannh@google.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-next@vger.kernel.org \
--cc=paul@paul-moore.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