From mboxrd@z Thu Jan 1 00:00:00 1970 From: shuah Subject: Re: [PATCH v5 1/3] powerpc: Fix vDSO clock_getres() Date: Thu, 30 May 2019 08:28:27 -0600 Message-ID: References: <20190528120446.48911-1-vincenzo.frascino@arm.com> <20190528120446.48911-2-vincenzo.frascino@arm.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20190528120446.48911-2-vincenzo.frascino@arm.com> Content-Language: en-US Sender: stable-owner@vger.kernel.org To: Vincenzo Frascino , linux-arch@vger.kernel.org, linuxppc-dev@lists.ozlabs.org, linux-s390@vger.kernel.org, linux-kselftest@vger.kernel.org Cc: Christophe Leroy , Benjamin Herrenschmidt , Paul Mackerras , Michael Ellerman , Martin Schwidefsky , Heiko Carstens , Thomas Gleixner , Arnd Bergmann , stable@vger.kernel.org, shuah , Shuah Khan List-Id: linux-arch.vger.kernel.org On 5/28/19 6:04 AM, Vincenzo Frascino wrote: > clock_getres in the vDSO library has to preserve the same behaviour > of posix_get_hrtimer_res(). > > In particular, posix_get_hrtimer_res() does: > sec = 0; > ns = hrtimer_resolution; > and hrtimer_resolution depends on the enablement of the high > resolution timers that can happen either at compile or at run time. > > Fix the powerpc vdso implementation of clock_getres keeping a copy of > hrtimer_resolution in vdso data and using that directly. > > Fixes: a7f290dad32e ("[PATCH] powerpc: Merge vdso's and add vdso support > to 32 bits kernel") > Cc: stable@vger.kernel.org > Cc: Benjamin Herrenschmidt > Cc: Paul Mackerras > Cc: Michael Ellerman > Signed-off-by: Vincenzo Frascino > Reviewed-by: Christophe Leroy > --- > > Note: This patch is independent from the others in this series, hence it > can be merged singularly by the powerpc maintainers. > Acked-by: Shuah Khan thanks, -- Shuah From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail.kernel.org ([198.145.29.99]:42692 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725870AbfE3O2l (ORCPT ); Thu, 30 May 2019 10:28:41 -0400 Subject: Re: [PATCH v5 1/3] powerpc: Fix vDSO clock_getres() References: <20190528120446.48911-1-vincenzo.frascino@arm.com> <20190528120446.48911-2-vincenzo.frascino@arm.com> From: shuah Message-ID: Date: Thu, 30 May 2019 08:28:27 -0600 MIME-Version: 1.0 In-Reply-To: <20190528120446.48911-2-vincenzo.frascino@arm.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit Sender: linux-arch-owner@vger.kernel.org List-ID: To: Vincenzo Frascino , linux-arch@vger.kernel.org, linuxppc-dev@lists.ozlabs.org, linux-s390@vger.kernel.org, linux-kselftest@vger.kernel.org Cc: Christophe Leroy , Benjamin Herrenschmidt , Paul Mackerras , Michael Ellerman , Martin Schwidefsky , Heiko Carstens , Thomas Gleixner , Arnd Bergmann , stable@vger.kernel.org, shuah , Shuah Khan Message-ID: <20190530142827.0Q7qVgTlhCgUvo1BPIx68mCHgMPI5gQ2zTCxGmv4EeE@z> On 5/28/19 6:04 AM, Vincenzo Frascino wrote: > clock_getres in the vDSO library has to preserve the same behaviour > of posix_get_hrtimer_res(). > > In particular, posix_get_hrtimer_res() does: > sec = 0; > ns = hrtimer_resolution; > and hrtimer_resolution depends on the enablement of the high > resolution timers that can happen either at compile or at run time. > > Fix the powerpc vdso implementation of clock_getres keeping a copy of > hrtimer_resolution in vdso data and using that directly. > > Fixes: a7f290dad32e ("[PATCH] powerpc: Merge vdso's and add vdso support > to 32 bits kernel") > Cc: stable@vger.kernel.org > Cc: Benjamin Herrenschmidt > Cc: Paul Mackerras > Cc: Michael Ellerman > Signed-off-by: Vincenzo Frascino > Reviewed-by: Christophe Leroy > --- > > Note: This patch is independent from the others in this series, hence it > can be merged singularly by the powerpc maintainers. > Acked-by: Shuah Khan thanks, -- Shuah