Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 15 May 2015 22:13:11 +0000 (UTC)
From:      Garrett Cooper <ngie@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-stable@freebsd.org, svn-src-stable-9@freebsd.org
Subject:   svn commit: r282992 - in stable/9/tools/regression/aio/kqueue: . lio
Message-ID:  <201505152213.t4FMDBA4017329@svn.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: ngie
Date: Fri May 15 22:13:11 2015
New Revision: 282992
URL: https://svnweb.freebsd.org/changeset/base/282992

Log:
  MFC r280893,r280894,r280895:
  
  r280893:
  
  Cleanup and do minor refactoring before converting testcases to ATF
  
  - Convert errx(-1, ..) to errx(1, ..)
  - Move the aio(4) checks to a single function (aio_available); use modfind(2)
    instead of depending on SIGSYS (doesn't work when aio(4) support is missing,
    not documented in the aio syscall manpages).
  - Use aio_available liberally in the testcase functions
  - Use mkstemp(3) + unlink(2) instead of mktemp(3)
  - Fix some -Wunused warnings
  - Bump WARNS to 6
  
  Submitted by: mjohnston [*]
  Sponsored by: EMC / Isilon Storage Division
  
  r280894:
  
  Minor cleanup before converting to ATF testcases
  
  - Remove blank (tab-only) lines.
  - Fix -Wunused warnings.
  - Bump up to WARNS= 6
  
  r280895:
  
  - Fix -Wsign issue
  - Bump up to WARNS=6

Modified:
  stable/9/tools/regression/aio/kqueue/Makefile
  stable/9/tools/regression/aio/kqueue/aio_kqueue.c
  stable/9/tools/regression/aio/kqueue/lio/Makefile
  stable/9/tools/regression/aio/kqueue/lio/lio_kqueue.c
Directory Properties:
  stable/9/   (props changed)
  stable/9/tools/   (props changed)
  stable/9/tools/regression/   (props changed)
  stable/9/tools/regression/aio/aiotest/   (props changed)

Modified: stable/9/tools/regression/aio/kqueue/Makefile
==============================================================================
--- stable/9/tools/regression/aio/kqueue/Makefile	Fri May 15 21:33:19 2015	(r282991)
+++ stable/9/tools/regression/aio/kqueue/Makefile	Fri May 15 22:13:11 2015	(r282992)
@@ -3,4 +3,8 @@
 PROG=	aio_kqueue
 NO_MAN=
 
+WARNS?=	6
+
+SUBDIR+=	lio
+
 .include <bsd.prog.mk>

Modified: stable/9/tools/regression/aio/kqueue/aio_kqueue.c
==============================================================================
--- stable/9/tools/regression/aio/kqueue/aio_kqueue.c	Fri May 15 21:33:19 2015	(r282991)
+++ stable/9/tools/regression/aio/kqueue/aio_kqueue.c	Fri May 15 22:13:11 2015	(r282992)
@@ -177,7 +177,7 @@ main (int argc, char *argv[])
 #endif
 			if (result != sizeof(buffer)) {
 				printf("FAIL: run %d, operation %d, result %d "
-				    " (errno=%d) should be %d\n", run, pending,
+				    " (errno=%d) should be %zu\n", run, pending,
 				    result, errno, sizeof(buffer));
 				failed++;
 			} else

Modified: stable/9/tools/regression/aio/kqueue/lio/Makefile
==============================================================================
--- stable/9/tools/regression/aio/kqueue/lio/Makefile	Fri May 15 21:33:19 2015	(r282991)
+++ stable/9/tools/regression/aio/kqueue/lio/Makefile	Fri May 15 22:13:11 2015	(r282992)
@@ -3,4 +3,6 @@
 PROG=	lio_kqueue
 NO_MAN=
 
+WARNS?=	6
+
 .include <bsd.prog.mk>

Modified: stable/9/tools/regression/aio/kqueue/lio/lio_kqueue.c
==============================================================================
--- stable/9/tools/regression/aio/kqueue/lio/lio_kqueue.c	Fri May 15 21:33:19 2015	(r282991)
+++ stable/9/tools/regression/aio/kqueue/lio/lio_kqueue.c	Fri May 15 22:13:11 2015	(r282992)
@@ -37,13 +37,15 @@
  * 	http://www.ambrisko.com/doug/listio_kqueue/listio_kqueue.patch
  */
 
-#include <aio.h>
-#include <fcntl.h>
-#include <stdio.h>
-#include <errno.h>
 #include <sys/types.h>
 #include <sys/event.h>
 #include <sys/time.h>
+#include <aio.h>
+#include <fcntl.h>
+#include <errno.h>
+#include <stdio.h>
+#include <stdlib.h>
+#include <string.h>
 #include <unistd.h>
 
 #define PATH_TEMPLATE   "/tmp/aio.XXXXXXXXXX"
@@ -52,9 +54,10 @@
 #define MAX LIO_MAX * 16
 #define MAX_RUNS 300
 
+int
 main(int argc, char *argv[]){
 	int fd;
-	struct aiocb *iocb[MAX], *kq_iocb;
+	struct aiocb *iocb[MAX];
 	struct aiocb **lio[LIO_MAX], **lio_element, **kq_lio;
 	int i, result, run, error, j, k;
 	char buffer[32768];
@@ -65,7 +68,7 @@ main(int argc, char *argv[]){
 	time_t time1, time2;
 	char *file, pathname[sizeof(PATH_TEMPLATE)-1];
 	int tmp_file = 0, failed = 0;
-	
+
 	if (kq < 0) {
 		perror("No kqeueue\n");
 		exit(1);
@@ -89,7 +92,7 @@ main(int argc, char *argv[]){
 #ifdef DEBUG
 	printf("Hello kq %d fd %d\n", kq, fd);
 #endif
-	
+
 	for (run = 0; run < MAX_RUNS; run++){
 #ifdef DEBUG
 		printf("Run %d\n", run);
@@ -137,13 +140,13 @@ main(int argc, char *argv[]){
 			printf("write %d is at %p\n", j, lio[j]);
 #endif
 		}
-		
+
 		for(i = 0; i < LIO_MAX; i++) {
 			for(j = LIO_MAX - 1; j >=0; j--) {
 				if (lio[j])
 					break;
 			}
-			
+
 			for(;;) {
 				bzero(&ke, sizeof(ke));
 				bzero(&kq_returned, sizeof(ke));
@@ -169,18 +172,18 @@ main(int argc, char *argv[]){
 				       kq_returned.udata, 
 				       lio[j]);
 #endif
-				
+
 				if(kq_lio)
 					break;
 #ifdef DEBUG
 				printf("Try again\n");
 #endif
-			}			
-			
+			}
+
 #ifdef DEBUG
 			printf("lio %p\n", lio);
 #endif
-			
+
 			for (j = 0; j < LIO_MAX; j++) {
 				if (lio[j] == kq_lio) {
 					break;
@@ -201,12 +204,12 @@ main(int argc, char *argv[]){
 				printf("PASS: run %d, operation %d result %d \n", run, LIO_MAX - i -1, result);
 			}
 			for(k = 0; k < MAX / LIO_MAX; k++){
-				result = aio_return(kq_lio[k]);			
+				result = aio_return(kq_lio[k]);
 #ifdef DEBUG
 				printf("Return Resulto for %d %d is %d\n", j, k, result);
 #endif
 				if (result != sizeof(buffer)) {
-					printf("FAIL: run %d, operation %d sub-opt %d  result %d (errno=%d) should be %d\n",
+					printf("FAIL: run %d, operation %d sub-opt %d  result %d (errno=%d) should be %zu\n",
 					   run, LIO_MAX - i -1, k, result, errno, sizeof(buffer));
 				} else {
 					printf("PASS: run %d, operation %d sub-opt %d  result %d\n",
@@ -216,13 +219,13 @@ main(int argc, char *argv[]){
 #ifdef DEBUG
 			printf("\n");
 #endif
-			
+
 			for(k = 0; k < MAX / LIO_MAX; k++) {
 				free(lio[j][k]);
 			}
 			free(lio[j]);
 			lio[j] = NULL;
-		}	
+		}
 	}
 #ifdef DEBUG
 	printf("Done\n");



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201505152213.t4FMDBA4017329>