Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Date:   Sun, 20 Dec 2020 16:25:35 +0100
From:   Christian Brauner <christian.brauner@ubuntu.com>
To:     Leesoo Ahn <dev@ooseel.net>
Cc:     linux-kernel@vger.kernel.org, Leesoo Ahn <lsahn@ooseel.net>,
        "Eric W. Biederman" <ebiederm@xmission.com>,
        Oleg Nesterov <oleg@redhat.com>, Jens Axboe <axboe@kernel.dk>,
        Peter Collingbourne <pcc@google.com>,
        Zhiqiang Liu <liuzhiqiang26@huawei.com>
Subject: Re: [PATCH] signal: Don't init struct kernel_siginfo fields to zero
 again
Message-ID: <20201220152535.k74foxjgejedslgx@wittgenstein>
References: <20201220074555.12655-1-lsahn@ooseel.net>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
In-Reply-To: <20201220074555.12655-1-lsahn@ooseel.net>
Precedence: bulk

On Sun, Dec 20, 2020 at 04:45:54PM +0900, Leesoo Ahn wrote:
> clear_siginfo() is responsible for clearing struct kernel_siginfo object.
> It's obvious that manually initializing those fields is needless as
> a commit[1] explains why the function introduced and its guarantee that
> all bits in the struct are cleared after it.
> 
> [1]: commit 8c5dbf2ae00b ("signal: Introduce clear_siginfo")
> 
> Signed-off-by: Leesoo Ahn <lsahn@ooseel.net>
> ---

Acked-by: Christian Brauner <christian.brauner@ubuntu.com>

I have a __user annotation fix in my tree from Jann that I plan to send
soon so I'll just stick this on top of it if noone minds.

Christian