[LTP] [PATCH v2 2/2] syscalls/getcpu:Add libc sched_getcpu() detection &&fix compiler errors

Cyril Hrubis chrubis@suse.cz
Mon May 13 10:55:20 CEST 2019


Hi!
> > sched_getcpu() isn't defined on some old distros, so we can add dection
> > and fix compiler error.
> 
> > Signed-off-by: Yang Xu <xuyang2018.jy@cn.fujitsu.com>
> Acked-by: Petr Vorel <pvorel@suse.cz>
> 
> @Metan: I'd be for pushing this before new release.

Acked, please go ahead.

> ...
> >  static inline int get_cpu(unsigned *cpu_id,
> >  			  unsigned *node_id LTP_ATTRIBUTE_UNUSED,
> >  			  void *cache_struct LTP_ATTRIBUTE_UNUSED)
> >  {
> > -#if defined(__i386__)
> > -	return syscall(__NR_getcpu, cpu_id, node_id, cache_struct);
> > +#ifndef HAVE_SCHED_GETCPU
> > +	return tst_syscall(__NR_getcpu, cpu_id, node_id, cache_struct);
> >  #else
> >  	*cpu_id = sched_getcpu();
> >  #endif
> 
> BTW: when rewriting into new C API (whenever it happens) it'd be nice to test
> both syscall and libc sched_getcpu() (if available) via recently added test_variants [1].
> 
> Kind regards,
> Petr
> 
> [1] https://github.com/linux-test-project/ltp/wiki/Test-Writing-Guidelines#2229-testing-similar-syscalls-in-one-test


Sure, but let's aim for that after the release has been finished.

-- 
Cyril Hrubis
chrubis@suse.cz


More information about the ltp mailing list