diff mbox series

rt-tests: oslat: Allow build for not supported archs

Message ID 20201001184726.82245-1-peterx@redhat.com
State New
Headers show
Series rt-tests: oslat: Allow build for not supported archs | expand

Commit Message

Peter Xu Oct. 1, 2020, 6:47 p.m. UTC
Now rt-tests won't build for archs other than x86/i386/ppc64 after oslat is
merged.  Instead of failing the build, let's make it pass.  However, whenever
oslat is executed, instead of running the real program, dump an error message,
so that people can try to implement the frc() function for it when there's a
real need for the new arch.

Signed-off-by: Peter Xu <peterx@redhat.com>
---

Just in case this got lost - I'm posting as a formal patch.  Please feel free
to comment here directly.
---
 src/oslat/oslat.c | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

Comments

John Kacur Oct. 23, 2020, 2:52 p.m. UTC | #1
On Thu, 1 Oct 2020, Peter Xu wrote:

> Now rt-tests won't build for archs other than x86/i386/ppc64 after oslat is

> merged.  Instead of failing the build, let's make it pass.  However, whenever

> oslat is executed, instead of running the real program, dump an error message,

> so that people can try to implement the frc() function for it when there's a

> real need for the new arch.

> 

> Signed-off-by: Peter Xu <peterx@redhat.com>

> ---

> 

> Just in case this got lost - I'm posting as a formal patch.  Please feel free

> to comment here directly.

> ---

>  src/oslat/oslat.c | 10 +++++++++-

>  1 file changed, 9 insertions(+), 1 deletion(-)

> 

> diff --git a/src/oslat/oslat.c b/src/oslat/oslat.c

> index f1a82f2..a8b6155 100644

> --- a/src/oslat/oslat.c

> +++ b/src/oslat/oslat.c

> @@ -69,7 +69,9 @@ static inline void frc(uint64_t *pval)

>  	__asm__ __volatile__("mfspr %0, 268\n" : "=r" (*pval));

>  }

>  # else

> -#  error Need frc() for this platform.

> +#  define relax()          do { } while (0)

> +#  define frc(x)

> +#  define FRC_MISSING

>  # endif

>  #else

>  # error Need to add support for this compiler.

> @@ -810,6 +812,12 @@ int main(int argc, char *argv[])

>  	int i, n_cores;

>  	cpu_set_t cpu_set;

>  

> +#ifdef FRC_MISSING

> +	printf("This architecture is not yet supported. "

> +	       "Please implement frc() function first for %s.\n", argv[0]);

> +	return 0;

> +#endif

> +

>  	CPU_ZERO(&cpu_set);

>  

>  	g.app_name = argv[0];

> -- 

> 2.26.2

> 

> 

Signed-off-by: John Kacur <jkacur@redhat.com>
diff mbox series

Patch

diff --git a/src/oslat/oslat.c b/src/oslat/oslat.c
index f1a82f2..a8b6155 100644
--- a/src/oslat/oslat.c
+++ b/src/oslat/oslat.c
@@ -69,7 +69,9 @@  static inline void frc(uint64_t *pval)
 	__asm__ __volatile__("mfspr %0, 268\n" : "=r" (*pval));
 }
 # else
-#  error Need frc() for this platform.
+#  define relax()          do { } while (0)
+#  define frc(x)
+#  define FRC_MISSING
 # endif
 #else
 # error Need to add support for this compiler.
@@ -810,6 +812,12 @@  int main(int argc, char *argv[])
 	int i, n_cores;
 	cpu_set_t cpu_set;
 
+#ifdef FRC_MISSING
+	printf("This architecture is not yet supported. "
+	       "Please implement frc() function first for %s.\n", argv[0]);
+	return 0;
+#endif
+
 	CPU_ZERO(&cpu_set);
 
 	g.app_name = argv[0];