MIME-Version: 1.0
References: <20230126154118.2393850-1-dvyukov@google.com> <20230220144347.267585-1-dvyukov@google.com>
 <20230222151908.GA20057@redhat.com>
In-Reply-To: <20230222151908.GA20057@redhat.com>
From:   Dmitry Vyukov <dvyukov@google.com>
Date:   Tue, 14 Mar 2023 09:25:21 +0100
Message-ID: <CACT4Y+Yvp7qj_420fk6UiPNB+dsRzRssPf4K=fMSNBoLB1oBiA@mail.gmail.com>
Subject: Re: [PATCH v5] posix-timers: Prefer delivery of signals to the
 current thread
To:     Oleg Nesterov <oleg@redhat.com>
Cc:     tglx@linutronix.de, linux-kernel@vger.kernel.org,
        "Eric W . Biederman" <ebiederm@xmission.com>,
        Frederic Weisbecker <frederic@kernel.org>,
        Marco Elver <elver@google.com>
Content-Type: text/plain; charset="UTF-8"
Precedence: bulk

On Wed, 22 Feb 2023 at 16:19, Oleg Nesterov <oleg@redhat.com> wrote:
>
> On 02/20, Dmitry Vyukov wrote:
> >
> >  kernel/signal.c                               | 23 +++++-
> >  tools/testing/selftests/timers/posix_timers.c | 73 +++++++++++++++++++
> >  2 files changed, 93 insertions(+), 3 deletions(-)
>
> For the change in kernel/signal.c
>
> Reviewed-by: Oleg Nesterov <oleg@redhat.com>

Hi Thomas,

Do you have any remaining concerns about this patch?
Should I drop the test to not confuse people? Then it will be just a
few lines in signal.c.