Re: [LTP] [PATCH v2] openposix: update invalid clock id to accomodate new auxiliary clock ids

[Avinesh Kumar <akumar@suse.de>]

On Tuesday, October 7, 2025 4:32:06 PM CEST Avinesh Kumar wrote:
> Starting kernel v6.17, there are 8 more auxiliary clock ids reserved on
> systems with CONFIG_POSIX_AUX_CLOCK=y [1] and moreover posix systems can
> define and support any number of clocks beyond the mandatory ones.
> 
> [1] https://lore.kernel.org/lkml/20250519083025.905800695@linutronix.de/
> 
> Signed-off-by: Avinesh Kumar <akumar@suse.de>
> ---
>  .../conformance/interfaces/clock_getres/6-2.c                | 5 ++---
>  .../conformance/interfaces/clock_gettime/8-2.c               | 4 ++--
>  .../conformance/interfaces/clock_settime/17-2.c              | 4 ++--
>  3 files changed, 6 insertions(+), 7 deletions(-)
> 
> diff --git a/testcases/open_posix_testsuite/conformance/interfaces/clock_getres/6-2.c b/testcases/open_posix_testsuite/conformance/interfaces/clock_getres/6-2.c
> index c44809012..af45e27c9 100644
> --- a/testcases/open_posix_testsuite/conformance/interfaces/clock_getres/6-2.c
> +++ b/testcases/open_posix_testsuite/conformance/interfaces/clock_getres/6-2.c
> @@ -17,8 +17,7 @@
>   *   unassigned value = -1073743192 (ex. of what gcc will set to)
>   *   unassigned value = 1073743192 (ex. of what gcc will set to)
>   *   -1
> - *   17 (currently not = to any clock)
> - *
> + *   50 (hopefully big enough value not to be a valid clock id)
>   */
>  #include <stdio.h>
>  #include <time.h>
> @@ -33,7 +32,7 @@ int main(void)
>  	struct timespec res;
>  	int invalid_tests[NUMINVALIDTESTS] = {
>  		INT32_MIN, INT32_MAX, 2147483647, -2147483647, -1073743192,
> -		1073743192, -1, 17
> +		1073743192, -1, 50
>  	};
>  	int i;
>  	int failure = 0;
> diff --git a/testcases/open_posix_testsuite/conformance/interfaces/clock_gettime/8-2.c b/testcases/open_posix_testsuite/conformance/interfaces/clock_gettime/8-2.c
> index ed4cd4078..92263ce87 100644
> --- a/testcases/open_posix_testsuite/conformance/interfaces/clock_gettime/8-2.c
> +++ b/testcases/open_posix_testsuite/conformance/interfaces/clock_gettime/8-2.c
> @@ -15,7 +15,7 @@
>   *   unassigned value = -1073743192 (ex. of what gcc will set to)
>   *   unassigned value = 1073743192 (ex. of what gcc will set to)
>   *   -1
> - *   17 (currently not = to any clock)
> +  *   50 (hopefully big enough value not to be a valid clock id)
I see I added extra space in this one, please fix while merging if patch is fine.

Thanks,
Avinesh



-- 
Mailing list info: https://lists.linux.it/listinfo/ltp