Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   Thomas Gleixner <tglx@linutronix.de>
To:     Prakash Sangappa <prakash.sangappa@oracle.com>
Cc:     "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "akpm@linux-foundation.org" <akpm@linux-foundation.org>,
        "peterz@infradead.org" <peterz@infradead.org>
Subject: Re: [PATCH] ipc: Update semtimedop() to use hrtimer
In-Reply-To: <B8775ABA-892E-40FE-BA0D-A72B635D33A9@oracle.com>
References: <1650333099-27214-1-git-send-email-prakash.sangappa@oracle.com>
 <87k0baw59j.ffs@tglx> <B8775ABA-892E-40FE-BA0D-A72B635D33A9@oracle.com>
Date:   Thu, 28 Apr 2022 09:47:06 +0200
Message-ID: <878rrpwsxx.ffs@tglx>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: quoted-printable
Precedence: bulk

On Wed, Apr 27 2022 at 23:42, Prakash Sangappa wrote:
>> On Apr 27, 2022, at 3:06 PM, Thomas Gleixner <tglx@linutronix.de> wrote:
>> Converting to ktime first makes this cheaper:
>>=20
>>                expires =3D ktime_get() + timespec64_to_ns(timeout);
>
> Since user provided timespec is added to current time, shouldn=E2=80=99t =
it check for overflow?
>
> So, perhaps=20
>
> 		expires =3D ktime_add_safe(ktime_get(), timespec64_to_ns(timeout));

Of course. This was just for illustration and I assumed you figure it
out, which you did. :)

Thanks,

        tglx