linux-kernel - Re: [PATCH] kthread: Rename user_mode_thread() to kmuser

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAAhV-H7qk8wSFt=ZMUSUt4cAmH+-5AsGqszXNW9w+u6Y8vDN+A@mail.gmail.com>
Date:   Tue, 18 Jul 2023 20:43:40 +0800
From:   Huacai Chen <chenhuacai@...nel.org>
To:     Luis Chamberlain <mcgrof@...nel.org>
Cc:     Huacai Chen <chenhuacai@...ngson.cn>,
        Andrew Morton <akpm@...ux-foundation.org>,
        "Eric W . Biederman" <ebiederm@...ssion.com>,
        Kees Cook <keescook@...omium.org>, linux-mm@...ck.org,
        linux-kernel@...r.kernel.org, Thomas Gleixner <tglx@...utronix.de>
Subject: Re: [PATCH] kthread: Rename user_mode_thread() to kmuser_thread()

Hi, Luis,

On Sat, Jul 1, 2023 at 7:25 AM Luis Chamberlain <mcgrof@...nel.org> wrote:
>
> On Sun, Jun 25, 2023 at 04:55:33PM +0800, Huacai Chen wrote:
> > Friendly ping?
>
> You want to cc the folks who Nacked your patch. Until then, this
> probably can't go further.
Thank you very much. Eric and Andrew are already in the CC list, so
add Thomas now.

My brain is a little old-fashioned so I insisted that "a thread
without mm_struct should be a kernel thread" in the previous patch.
Unfortunately this makes Eric and Thomas unhappy, I'm very sorry for
that.

During the discussion of the previous patch I know I made some
mistakes about some basic concepts, but I also found the name
"user_mode_thread()" is somewhat confusing. I think rename it to
kmuser_thread() is better, because:
1, it identify init and umh as user threads;
2, it points out that init and umh are special user threads that run
in kernel mode before loading a user program.

Sorry for my rudeness again.

Huacai
>
>   Luis