svn commit: samba r23061 - in branches/SAMBA_4_0/source/torture/raw: .

[tridge at samba.org]

Author: tridge
Date: 2007-05-22 04:53:05 +0000 (Tue, 22 May 2007)
New Revision: 23061

WebSVN: http://websvn.samba.org/cgi-bin/viewcvs.cgi?view=rev&root=samba&rev=23061

Log:

keep the RAW-BENCH-OPEN test balanced

Modified:
   branches/SAMBA_4_0/source/torture/raw/openbench.c


Changeset:
Modified: branches/SAMBA_4_0/source/torture/raw/openbench.c
===================================================================
--- branches/SAMBA_4_0/source/torture/raw/openbench.c	2007-05-22 04:13:17 UTC (rev 23060)
+++ branches/SAMBA_4_0/source/torture/raw/openbench.c	2007-05-22 04:53:05 UTC (rev 23061)
@@ -40,17 +40,20 @@
 static char **fnames;
 static int num_connected;
 
+enum open_stage {OPEN_INITIAL, OPEN_OPEN, OPEN_CLOSE};
+
 struct benchopen_state {
 	TALLOC_CTX *mem_ctx;
 	struct event_context *ev;
 	struct smbcli_state *cli;
 	struct smbcli_tree *tree;
+	enum open_stage stage;
 	int client_num;
+	int old_fnum;
 	int fnum;
 	int file_num;
 	int count;
 	int lastcount;
-	BOOL waiting_open, waiting_close;
 	union smb_open open_parms;
 	union smb_close close_parms;
 	struct smbcli_request *req_open;
@@ -65,6 +68,7 @@
 };
 
 static void next_open(struct benchopen_state *state);
+static void next_operation(struct benchopen_state *state);
 static void reopen_connection(struct event_context *ev, struct timed_event *te, 
 			      struct timeval t, void *private_data);
 
@@ -93,7 +97,9 @@
 	DEBUG(0,("reconnect to %s finished (%u connected)\n", state->dest_host,
 		 num_connected));
 
-	next_open(state);
+	state->stage = OPEN_INITIAL;
+	state->fnum = -1;
+	next_operation(state);
 }
 
 	
@@ -127,8 +133,6 @@
 	talloc_free(state->tree);
 	state->tree = NULL;
 	state->fnum = -1;
-	state->waiting_open = False;
-	state->waiting_close = False;
 
 	ctx = smb_composite_connect_send(io, state->mem_ctx, state->ev);
 	if (ctx == NULL) {
@@ -165,20 +169,18 @@
 	state->req_open = smb_raw_open_send(state->tree, &state->open_parms);
 	state->req_open->async.fn = open_completed;
 	state->req_open->async.private = state;
-	state->waiting_open = True;
+}
 
-	if (state->fnum == -1) {
-		return;
-	}
 
+static void next_close(struct benchopen_state *state)
+{
 	state->close_parms.close.level = RAW_CLOSE_CLOSE;
-	state->close_parms.close.in.file.fnum = state->fnum;
+	state->close_parms.close.in.file.fnum = state->old_fnum;
 	state->close_parms.close.in.write_time = 0;
 
 	state->req_close = smb_raw_close_send(state->tree, &state->close_parms);
 	state->req_close->async.fn = close_completed;
 	state->req_close->async.private = state;
-	state->waiting_close = True;
 }
 
 /*
@@ -190,6 +192,8 @@
 	TALLOC_CTX *tmp_ctx = talloc_new(state->mem_ctx);
 	NTSTATUS status;
 
+	state->old_fnum = state->fnum;
+
 	status = smb_raw_open_recv(req, tmp_ctx, &state->open_parms);
 
 	talloc_free(tmp_ctx);
@@ -225,11 +229,8 @@
 	}
 
 	state->fnum = state->open_parms.ntcreatex.out.file.fnum;
-	state->waiting_open = False;
 
-	if (!state->waiting_close) {
-		next_open(state);
-	}
+	next_operation(state);
 }	
 
 /*
@@ -262,14 +263,27 @@
 		return;
 	}
 
-	state->waiting_close = False;
+	next_operation(state);
+}	
 
-	if (!state->waiting_open) {
+static void next_operation(struct benchopen_state *state)
+{
+	switch (state->stage) {
+	case OPEN_INITIAL:
 		next_open(state);
+		state->stage = OPEN_OPEN;
+		break;
+	case OPEN_OPEN:
+		next_open(state);
+		state->stage = OPEN_CLOSE;
+		break;
+	case OPEN_CLOSE:
+		next_close(state);
+		state->stage = OPEN_OPEN;
+		break;
 	}
-}	
+}
 
-
 static void report_rate(struct event_context *ev, struct timed_event *te, 
 			struct timeval t, void *private_data)
 {
@@ -311,6 +325,7 @@
 		state[i].mem_ctx = talloc_new(state);
 		state[i].client_num = i;
 		state[i].ev = ev;
+		state[i].stage = OPEN_INITIAL;
 		if (!torture_open_connection_ev(&state[i].cli, i, ev)) {
 			return False;
 		}
@@ -339,7 +354,7 @@
 	for (i=0;i<nprocs;i++) {
 		state[i].fnum = -1;
 		state[i].file_num = i;		
-		next_open(&state[i]);
+		next_operation(&state[i]);
 	}
 
 	tv = timeval_current();