test: disable alarm autotest in FreeBSD

Message ID 1537367950-31678-1-git-send-email-pallantlax.poornima@intel.com (mailing list archive)
State Accepted, archived
Delegated to: Thomas Monjalon
Headers
Series test: disable alarm autotest in FreeBSD |

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/Intel-compilation success Compilation OK

Commit Message

Poornima, PallantlaX Sept. 19, 2018, 2:39 p.m. UTC
  Disabled the alarm_autotest UT in FreeBSD
Interrupts are not supported in FreeBSD.
Alarm API depends on interrupts, so disabled alarm test on FreeBSD.

Signed-off-by: Pallantla Poornima <pallantlax.poornima@intel.com>
---
 test/test/test_alarm.c | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)
  

Comments

Anatoly Burakov Sept. 20, 2018, 4:01 p.m. UTC | #1
On 19-Sep-18 3:39 PM, Pallantla Poornima wrote:
> Disabled the alarm_autotest UT in FreeBSD
> Interrupts are not supported in FreeBSD.
> Alarm API depends on interrupts, so disabled alarm test on FreeBSD.
> 
> Signed-off-by: Pallantla Poornima <pallantlax.poornima@intel.com>
> ---
Reviewed-by: Anatoly Burakov <anatoly.burakov@intel.com>
  
Pattan, Reshma Oct. 17, 2018, 3:13 p.m. UTC | #2
> -----Original Message-----
> From: Poornima, PallantlaX
> Sent: Wednesday, September 19, 2018 3:39 PM
> To: dev@dpdk.org
> Cc: Burakov, Anatoly <anatoly.burakov@intel.com>; Pattan, Reshma
> <reshma.pattan@intel.com>; Poornima, PallantlaX
> <pallantlax.poornima@intel.com>
> Subject: [PATCH] test: disable alarm autotest in FreeBSD
> 
> Disabled the alarm_autotest UT in FreeBSD Interrupts are not supported in
> FreeBSD.
> Alarm API depends on interrupts, so disabled alarm test on FreeBSD.
> 
> Signed-off-by: Pallantla Poornima <pallantlax.poornima@intel.com>
Acked-by: Reshma Pattan <reshma.pattan@intel.com>
  
Thomas Monjalon Oct. 29, 2018, 2:30 a.m. UTC | #3
> > Disabled the alarm_autotest UT in FreeBSD Interrupts are not supported in
> > FreeBSD.
> > Alarm API depends on interrupts, so disabled alarm test on FreeBSD.
> > 
> > Signed-off-by: Pallantla Poornima <pallantlax.poornima@intel.com>
> Acked-by: Reshma Pattan <reshma.pattan@intel.com>

Applied, thanks
  

Patch

diff --git a/test/test/test_alarm.c b/test/test/test_alarm.c
index f566947f2..d1284b379 100644
--- a/test/test/test_alarm.c
+++ b/test/test/test_alarm.c
@@ -178,7 +178,10 @@  static int
 test_alarm(void)
 {
 	int count = 0;
-
+#ifdef RTE_EXEC_ENV_BSDAPP
+	printf("The alarm API is not supported on FreeBSD\n");
+	return 0;
+#endif
 	/* check if the callback will be called */
 	printf("check if the callback will be called\n");
 	flag = 0;