Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:7 as permitted sender) client-ip=2620:137:e000::3:7;
Date:   Tue, 26 Sep 2023 23:52:34 +0200
From:   Oleg Nesterov <oleg@redhat.com>
To:     Li kunyu <kunyu@nfschina.com>
Cc:     peterz@infradead.org, mcgrof@kernel.org,
        mathieu.desnoyers@efficios.com, elver@google.com,
        viro@zeniv.linux.org.uk, dvyukov@google.com,
        vincent.whitchurch@axis.com, michael.christie@oracle.com,
        linux-kernel@vger.kernel.org,
        Andrew Morton <akpm@linux-foundation.org>
Subject: Re: [PATCH] kernel/signal: =?utf-8?Q?Remov?=
 =?utf-8?B?ZSB1bm5lY2Vzc2FyeSDigJhOVUxM4oCZ?= values from ucounts
Message-ID: <20230926215233.GA26968@redhat.com>
References: <20230926022410.4280-1-kunyu@nfschina.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20230926022410.4280-1-kunyu@nfschina.com>
User-Agent: Mutt/1.5.24 (2015-08-30)
Precedence: bulk

On 09/26, Li kunyu wrote:
>
> --- a/kernel/signal.c
> +++ b/kernel/signal.c
> @@ -415,7 +415,7 @@ __sigqueue_alloc(int sig, struct task_struct *t, gfp_t gfp_flags,
>  		 int override_rlimit, const unsigned int sigqueue_flags)
>  {
>  	struct sigqueue *q = NULL;
> -	struct ucounts *ucounts = NULL;
> +	struct ucounts *ucounts;
>  	long sigpending;

this looks obviously correct, so

Acked-by: Oleg Nesterov <oleg@redhat.com>


but you know, this change is so trivial, perhaps you should
resend this minor cleanup to kernel-janitors@vger.kernel.org

but let me add Andrew, perhaps he will take this patch...

Oleg.