X-Git-Url: https://git.kernel.dk/?p=fio.git;a=blobdiff_plain;f=goptions.c;h=671a798fdfeaa6cb83003c465e360ff8e8d429f6;hp=30edd5cdac6e3fd8545063b7682f21afb9576aaa;hb=19921181dc8e92ce5320fa32fcb385df9c20a350;hpb=7fc35ff7f50137503236f67ca9deeca0c11a536f

diff --git a/goptions.c b/goptions.c
index 30edd5cd..671a798f 100644
--- a/goptions.c
+++ b/goptions.c
@@ -16,6 +16,7 @@ struct gopt {
 	unsigned int opt_index;
 	unsigned int opt_type;
 	gulong sig_handler;
+	struct gopt_job_view *gjv;
 };
 
 struct gopt_combo {
@@ -43,6 +44,7 @@ struct gopt_str_val {
 	struct gopt gopt;
 	GtkWidget *spin;
 	GtkWidget *combo;
+	unsigned int maxindex;
 };
 
 #define GOPT_RANGE_SPIN	4
@@ -57,15 +59,22 @@ struct gopt_str_multi {
 	GtkWidget *checks[PARSE_MAX_VP];
 };
 
-static GtkWidget *gopt_widgets[FIO_MAX_OPTS];
-
 struct gopt_frame_widget {
 	GtkWidget *vbox[2];
 	unsigned int nr;
 };
-static struct gopt_frame_widget gopt_g_widgets[__FIO_OPT_G_NR];
 
-static GtkWidget *gopt_get_group_frame(GtkWidget *box, unsigned int groupmask)
+struct gopt_job_view {
+	struct flist_head list;
+	struct gopt_frame_widget g_widgets[__FIO_OPT_G_NR];
+	GtkWidget *widgets[FIO_MAX_OPTS];
+	GtkWidget *vboxes[__FIO_OPT_C_NR];
+};
+
+static GNode *gopt_dep_tree;
+
+static GtkWidget *gopt_get_group_frame(struct gopt_job_view *gjv,
+				       GtkWidget *box, unsigned int groupmask)
 {
 	unsigned int mask, group;
 	struct opt_group *og;
@@ -81,7 +90,7 @@ static GtkWidget *gopt_get_group_frame(GtkWidget *box, unsigned int groupmask)
 		return NULL;
 
 	group = ffz(~groupmask);
-	gfw = &gopt_g_widgets[group];
+	gfw = &gjv->g_widgets[group];
 	if (!gfw->vbox[0]) {
 		frame = gtk_frame_new(og->name);
 		gtk_box_pack_start(GTK_BOX(box), frame, FALSE, FALSE, 3);
@@ -101,27 +110,31 @@ static GtkWidget *gopt_get_group_frame(GtkWidget *box, unsigned int groupmask)
 /*
  * Mark children as invisible, if needed.
  */
-static void gopt_set_children_visible(struct fio_option *parent,
+static void gopt_set_children_visible(struct gopt_job_view *gjv,
+				      struct fio_option *parent,
 				      gboolean visible)
 {
-	struct fio_option *o;
-	int i;
+	GNode *child, *node;
 
-	/*
-	 * This isn't super fast, but it should not be an issue. If it is, we
-	 * can speed it up by caching the lookup at least. Or we can do it
-	 * once, at init time.
-	 */
-	for (i = 0; fio_options[i].name; i++) {
-		o = &fio_options[i];
-		if (!o->parent || !o->hide)
-			continue;
+	if (parent->hide_on_set)
+		visible = !visible;
+
+	node = g_node_find(gopt_dep_tree, G_IN_ORDER, G_TRAVERSE_ALL, parent);
+	child = g_node_first_child(node);
+	while (child) {
+		struct fio_option *o = child->data;
+		struct gopt *g = o->gui_data;
+
+		/*
+		 * Recurse into child, if it also has children
+		 */
+		if (g_node_n_children(child))
+			gopt_set_children_visible(gjv, o, visible);
 
-		if (strcmp(parent->name, o->parent))
-			continue;
+		if (gjv->widgets[g->opt_index])
+			gtk_widget_set_sensitive(gjv->widgets[g->opt_index], visible);
 
-		if (gopt_widgets[i])
-			gtk_widget_set_sensitive(gopt_widgets[i], visible);
+		child = g_node_next_sibling(child);
 	}
 }
 
@@ -134,14 +147,17 @@ static void gopt_str_changed(GtkEntry *entry, gpointer data)
 
 	text = gtk_entry_get_text(GTK_ENTRY(s->entry));
 	set = strcmp(text, "") != 0;
-	gopt_set_children_visible(o, set);
+
+	gopt_set_children_visible(s->gopt.gjv, o, set);
 }
 
-static void gopt_mark_index(struct gopt *gopt, unsigned int idx)
+static void gopt_mark_index(struct gopt_job_view *gjv, struct gopt *gopt,
+			    unsigned int idx)
 {
-	assert(!gopt_widgets[idx]);
+	assert(!gjv->widgets[idx]);
 	gopt->opt_index = idx;
-	gopt_widgets[idx] = gopt->box;
+	gopt->gjv = gjv;
+	gjv->widgets[idx] = gopt->box;
 }
 
 static void gopt_str_destroy(GtkWidget *w, gpointer data)
@@ -152,7 +168,8 @@ static void gopt_str_destroy(GtkWidget *w, gpointer data)
 	gtk_widget_destroy(w);
 }
 
-static struct gopt *gopt_new_str_store(struct fio_option *o, const char *text,
+static struct gopt *gopt_new_str_store(struct gopt_job_view *gjv,
+				       struct fio_option *o, const char *text,
 				       unsigned int idx)
 {
 	struct gopt_str *s;
@@ -168,19 +185,19 @@ static struct gopt *gopt_new_str_store(struct fio_option *o, const char *text,
 		label = gtk_label_new(o->lname);
 
 	s->entry = gtk_entry_new();
-	gopt_mark_index(&s->gopt, idx);
+	gopt_mark_index(gjv, &s->gopt, idx);
 	if (text)
 		gtk_entry_set_text(GTK_ENTRY(s->entry), text);
 	gtk_entry_set_editable(GTK_ENTRY(s->entry), 1);
-	s->gopt.sig_handler = g_signal_connect(GTK_OBJECT(s->entry), "changed", G_CALLBACK(gopt_str_changed), s);
-	g_signal_connect(GTK_OBJECT(s->entry), "destroy", G_CALLBACK(gopt_str_destroy), s);
 
 	if (o->def)
 		gtk_entry_set_text(GTK_ENTRY(s->entry), o->def);
 
+	s->gopt.sig_handler = g_signal_connect(GTK_OBJECT(s->entry), "changed", G_CALLBACK(gopt_str_changed), s);
+	g_signal_connect(GTK_OBJECT(s->entry), "destroy", G_CALLBACK(gopt_str_destroy), s);
+
 	gtk_box_pack_start(GTK_BOX(s->gopt.box), s->entry, FALSE, FALSE, 0);
 	gtk_box_pack_start(GTK_BOX(s->gopt.box), label, FALSE, FALSE, 0);
-	o->gui_data = s;
 	return &s->gopt;
 }
 
@@ -188,8 +205,11 @@ static void gopt_combo_changed(GtkComboBox *box, gpointer data)
 {
 	struct gopt_combo *c = (struct gopt_combo *) data;
 	struct fio_option *o = &fio_options[c->gopt.opt_index];
+	unsigned int index;
+
+	index = gtk_combo_box_get_active(GTK_COMBO_BOX(c->combo));
 
-	printf("combo %s changed\n", o->name);
+	gopt_set_children_visible(c->gopt.gjv, o, index);
 }
 
 static void gopt_combo_destroy(GtkWidget *w, gpointer data)
@@ -200,7 +220,8 @@ static void gopt_combo_destroy(GtkWidget *w, gpointer data)
 	gtk_widget_destroy(w);
 }
 
-static struct gopt_combo *__gopt_new_combo(struct fio_option *o,
+static struct gopt_combo *__gopt_new_combo(struct gopt_job_view *gjv,
+					   struct fio_option *o,
 					   unsigned int idx)
 {
 	struct gopt_combo *c;
@@ -216,30 +237,29 @@ static struct gopt_combo *__gopt_new_combo(struct fio_option *o,
 		label = gtk_label_new(o->lname);
 
 	c->combo = gtk_combo_box_new_text();
-	gopt_mark_index(&c->gopt, idx);
-	c->gopt.sig_handler = g_signal_connect(GTK_OBJECT(c->combo), "changed", G_CALLBACK(gopt_combo_changed), c);
+	gopt_mark_index(gjv, &c->gopt, idx);
 	g_signal_connect(GTK_OBJECT(c->combo), "destroy", G_CALLBACK(gopt_combo_destroy), c);
 
 	gtk_box_pack_start(GTK_BOX(c->gopt.box), c->combo, FALSE, FALSE, 0);
 	gtk_box_pack_start(GTK_BOX(c->gopt.box), label, FALSE, FALSE, 0);
 
-	o->gui_data = c;
 	return c;
 }
 
-static struct gopt *gopt_new_combo_str(struct fio_option *o, const char *text,
+static struct gopt *gopt_new_combo_str(struct gopt_job_view *gjv,
+				       struct fio_option *o, const char *text,
 				       unsigned int idx)
 {
-	struct gopt_combo *combo;
+	struct gopt_combo *c;
 	struct value_pair *vp;
 	int i, active = 0;
 
-	combo = __gopt_new_combo(o, idx);
+	c = __gopt_new_combo(gjv, o, idx);
 
 	i = 0;
 	vp = &o->posval[0];
 	while (vp->ival) {
-		gtk_combo_box_append_text(GTK_COMBO_BOX(combo->combo), vp->ival);
+		gtk_combo_box_append_text(GTK_COMBO_BOX(c->combo), vp->ival);
 		if (o->def && !strcmp(vp->ival, o->def))
 			active = i;
 		if (text && !strcmp(vp->ival, text))
@@ -248,34 +268,38 @@ static struct gopt *gopt_new_combo_str(struct fio_option *o, const char *text,
 		i++;
 	}
 
-	gtk_combo_box_set_active(GTK_COMBO_BOX(combo->combo), active);
-	return &combo->gopt;
+	gtk_combo_box_set_active(GTK_COMBO_BOX(c->combo), active);
+	c->gopt.sig_handler = g_signal_connect(GTK_OBJECT(c->combo), "changed", G_CALLBACK(gopt_combo_changed), c);
+	return &c->gopt;
 }
 
-static struct gopt *gopt_new_combo_int(struct fio_option *o, unsigned int *ip,
+static struct gopt *gopt_new_combo_int(struct gopt_job_view *gjv,
+				       struct fio_option *o, unsigned int *ip,
 				       unsigned int idx)
 {
-	struct gopt_combo *combo;
+	struct gopt_combo *c;
 	struct value_pair *vp;
 	int i, active = 0;
 
-	combo = __gopt_new_combo(o, idx);
+	c = __gopt_new_combo(gjv, o, idx);
 
 	i = 0;
 	vp = &o->posval[0];
 	while (vp->ival) {
-		gtk_combo_box_append_text(GTK_COMBO_BOX(combo->combo), vp->ival);
+		gtk_combo_box_append_text(GTK_COMBO_BOX(c->combo), vp->ival);
 		if (ip && vp->oval == *ip)
 			active = i;
 		vp++;
 		i++;
 	}
 
-	gtk_combo_box_set_active(GTK_COMBO_BOX(combo->combo), active);
-	return &combo->gopt;
+	gtk_combo_box_set_active(GTK_COMBO_BOX(c->combo), active);
+	c->gopt.sig_handler = g_signal_connect(GTK_OBJECT(c->combo), "changed", G_CALLBACK(gopt_combo_changed), c);
+	return &c->gopt;
 }
 
-static struct gopt *gopt_new_str_multi(struct fio_option *o, unsigned int idx)
+static struct gopt *gopt_new_str_multi(struct gopt_job_view *gjv,
+				       struct fio_option *o, unsigned int idx)
 {
 	struct gopt_str_multi *m;
 	struct value_pair *vp;
@@ -285,7 +309,7 @@ static struct gopt *gopt_new_str_multi(struct fio_option *o, unsigned int idx)
 	m = malloc(sizeof(*m));
 	memset(m, 0, sizeof(*m));
 	m->gopt.box = gtk_hbox_new(FALSE, 3);
-	gopt_mark_index(&m->gopt, idx);
+	gopt_mark_index(gjv, &m->gopt, idx);
 
 	if (!o->lname)
 		frame = gtk_frame_new(o->name);
@@ -321,7 +345,8 @@ static void gopt_int_changed(GtkSpinButton *spin, gpointer data)
 	i->lastval = value;
 
 	if (o->inv_opt) {
-		struct gopt_int *i_inv = o->inv_opt->gui_data;
+		struct gopt *b_inv = o->inv_opt->gui_data;
+		struct gopt_int *i_inv = container_of(b_inv, struct gopt_int, gopt);
 		int cur_val;
 
 		assert(o->type == o->inv_opt->type);
@@ -342,7 +367,8 @@ static void gopt_int_destroy(GtkWidget *w, gpointer data)
 	gtk_widget_destroy(w);
 }
 
-static struct gopt_int *__gopt_new_int(struct fio_option *o,
+static struct gopt_int *__gopt_new_int(struct gopt_job_view *gjv,
+				       struct fio_option *o,
 				       unsigned long long *p, unsigned int idx)
 {
 	unsigned long long defval;
@@ -377,7 +403,7 @@ static struct gopt_int *__gopt_new_int(struct fio_option *o,
 		interval = o->interval;
 
 	i->spin = gtk_spin_button_new_with_range(o->minval, maxval, interval);
-	gopt_mark_index(&i->gopt, idx);
+	gopt_mark_index(gjv, &i->gopt, idx);
 	gtk_spin_button_set_update_policy(GTK_SPIN_BUTTON(i->spin), GTK_UPDATE_IF_VALID);
 	gtk_spin_button_set_value(GTK_SPIN_BUTTON(i->spin), defval);
 	i->lastval = defval;
@@ -387,11 +413,11 @@ static struct gopt_int *__gopt_new_int(struct fio_option *o,
 	gtk_box_pack_start(GTK_BOX(i->gopt.box), i->spin, FALSE, FALSE, 0);
 	gtk_box_pack_start(GTK_BOX(i->gopt.box), label, FALSE, FALSE, 0);
 
-	o->gui_data = i;
 	return i;
 }
 
-static struct gopt *gopt_new_int(struct fio_option *o, unsigned int *ip,
+static struct gopt *gopt_new_int(struct gopt_job_view *gjv,
+				 struct fio_option *o, unsigned int *ip,
 				 unsigned int idx)
 {
 	unsigned long long ullp;
@@ -399,19 +425,20 @@ static struct gopt *gopt_new_int(struct fio_option *o, unsigned int *ip,
 
 	if (ip) {
 		ullp = *ip;
-		i = __gopt_new_int(o, &ullp, idx);
+		i = __gopt_new_int(gjv, o, &ullp, idx);
 	} else
-		i = __gopt_new_int(o, NULL, idx);
+		i = __gopt_new_int(gjv, o, NULL, idx);
 
 	return &i->gopt;
 }
 
-static struct gopt *gopt_new_ullong(struct fio_option *o, unsigned long long *p,
+static struct gopt *gopt_new_ullong(struct gopt_job_view *gjv,
+				    struct fio_option *o, unsigned long long *p,
 				    unsigned int idx)
 {
 	struct gopt_int *i;
 
-	i = __gopt_new_int(o, p, idx);
+	i = __gopt_new_int(gjv, o, p, idx);
 	return &i->gopt;
 }
 
@@ -424,7 +451,8 @@ static void gopt_bool_toggled(GtkToggleButton *button, gpointer data)
 	set = gtk_toggle_button_get_active(GTK_TOGGLE_BUTTON(b->check));
 
 	if (o->inv_opt) {
-		struct gopt_bool *b_inv = o->inv_opt->gui_data;
+		struct gopt *g_inv = o->inv_opt->gui_data;
+		struct gopt_bool *b_inv = container_of(g_inv, struct gopt_bool, gopt);
 
 		assert(o->type == o->inv_opt->type);
 
@@ -433,7 +461,7 @@ static void gopt_bool_toggled(GtkToggleButton *button, gpointer data)
 		g_signal_handler_unblock(G_OBJECT(b_inv->check), b_inv->gopt.sig_handler);
 	}
 
-	gopt_set_children_visible(o, set);
+	gopt_set_children_visible(b->gopt.gjv, o, set);
 }
 
 static void gopt_bool_destroy(GtkWidget *w, gpointer data)
@@ -444,7 +472,8 @@ static void gopt_bool_destroy(GtkWidget *w, gpointer data)
 	gtk_widget_destroy(w);
 }
 
-static struct gopt *gopt_new_bool(struct fio_option *o, unsigned int *val,
+static struct gopt *gopt_new_bool(struct gopt_job_view *gjv,
+				  struct fio_option *o, unsigned int *val,
 				  unsigned int idx)
 {
 	struct gopt_bool *b;
@@ -460,7 +489,7 @@ static struct gopt *gopt_new_bool(struct fio_option *o, unsigned int *val,
 		label = gtk_label_new(o->lname);
 
 	b->check = gtk_check_button_new();
-	gopt_mark_index(&b->gopt, idx);
+	gopt_mark_index(gjv, &b->gopt, idx);
 	if (val)
 		defstate = *val;
 	else if (o->def && !strcmp(o->def, "1"))
@@ -475,7 +504,6 @@ static struct gopt *gopt_new_bool(struct fio_option *o, unsigned int *val,
 
 	gtk_box_pack_start(GTK_BOX(b->gopt.box), b->check, FALSE, FALSE, 0);
 	gtk_box_pack_start(GTK_BOX(b->gopt.box), label, FALSE, FALSE, 0);
-	o->gui_data = b;
 	return &b->gopt;
 }
 
@@ -527,7 +555,8 @@ static void gopt_range_destroy(GtkWidget *w, gpointer data)
 	gtk_widget_destroy(w);
 }
 
-static struct gopt *gopt_new_int_range(struct fio_option *o, unsigned int **ip,
+static struct gopt *gopt_new_int_range(struct gopt_job_view *gjv,
+				       struct fio_option *o, unsigned int **ip,
 				       unsigned int idx)
 {
 	struct gopt_range *r;
@@ -539,7 +568,7 @@ static struct gopt *gopt_new_int_range(struct fio_option *o, unsigned int **ip,
 	r = malloc(sizeof(*r));
 	memset(r, 0, sizeof(*r));
 	r->gopt.box = gtk_hbox_new(FALSE, 3);
-	gopt_mark_index(&r->gopt, idx);
+	gopt_mark_index(gjv, &r->gopt, idx);
 	if (!o->lname)
 		label = gtk_label_new(o->name);
 	else
@@ -575,7 +604,6 @@ static struct gopt *gopt_new_int_range(struct fio_option *o, unsigned int **ip,
 
 	gtk_box_pack_start(GTK_BOX(r->gopt.box), label, FALSE, FALSE, 0);
 	g_signal_connect(G_OBJECT(r->gopt.box), "destroy", G_CALLBACK(gopt_range_destroy), r);
-	o->gui_data = r;
 	return &r->gopt;
 }
 
@@ -601,9 +629,12 @@ static void gopt_str_val_spin_wrapped(GtkSpinButton *spin, gpointer data)
 	 * Can't rely on exact value, as fast changes increment >= 1
 	 */
 	if (!val) {
-		val = 1;
 		index = gtk_combo_box_get_active(GTK_COMBO_BOX(g->combo));
-		gtk_combo_box_set_active(GTK_COMBO_BOX(g->combo), ++index);
+		if (index + 1 <= g->maxindex) {
+			val = 1;
+			gtk_combo_box_set_active(GTK_COMBO_BOX(g->combo), ++index);
+		} else
+			val = 1023;
 		gtk_spin_button_set_value(spin, val);
 	} else {
 		index = gtk_combo_box_get_active(GTK_COMBO_BOX(g->combo));
@@ -615,7 +646,8 @@ static void gopt_str_val_spin_wrapped(GtkSpinButton *spin, gpointer data)
 	}
 }
 
-static struct gopt *gopt_new_str_val(struct fio_option *o,
+static struct gopt *gopt_new_str_val(struct gopt_job_view *gjv,
+				     struct fio_option *o,
 				     unsigned long long *p, unsigned int idx)
 {
 	struct gopt_str_val *g;
@@ -630,6 +662,7 @@ static struct gopt *gopt_new_str_val(struct fio_option *o,
 		label = gtk_label_new(o->name);
 	else
 		label = gtk_label_new(o->lname);
+	gopt_mark_index(gjv, &g->gopt, idx);
 
 	g->spin = gtk_spin_button_new_with_range(0.0, 1023.0, 1.0);
 	gtk_spin_button_set_update_policy(GTK_SPIN_BUTTON(g->spin), GTK_UPDATE_IF_VALID);
@@ -644,17 +677,18 @@ static struct gopt *gopt_new_str_val(struct fio_option *o,
 		gtk_combo_box_append_text(GTK_COMBO_BOX(g->combo), postfix[i]);
 		i++;
 	}
+	g->maxindex = i - 1;
 	gtk_combo_box_set_active(GTK_COMBO_BOX(g->combo), 0);
 	gtk_box_pack_start(GTK_BOX(g->gopt.box), g->combo, FALSE, FALSE, 0);
 	gtk_box_pack_start(GTK_BOX(g->gopt.box), label, FALSE, FALSE, 3);
 
 	g_signal_connect(G_OBJECT(g->gopt.box), "destroy", G_CALLBACK(gopt_str_val_destroy), g);
-	o->gui_data = g;
 	return &g->gopt;
 }
 
-static void gopt_add_option(GtkWidget *hbox, struct fio_option *o,
-			    unsigned int opt_index, struct thread_options *to)
+static void gopt_add_option(struct gopt_job_view *gjv, GtkWidget *hbox,
+			    struct fio_option *o, unsigned int opt_index,
+			    struct thread_options *to)
 {
 	struct gopt *go = NULL;
 
@@ -665,7 +699,7 @@ static void gopt_add_option(GtkWidget *hbox, struct fio_option *o,
 		if (o->off1)
 			ullp = td_var(to, o->off1);
 
-		go = gopt_new_str_val(o, ullp, opt_index);
+		go = gopt_new_str_val(gjv, o, ullp, opt_index);
 		break;
 		}
 	case FIO_OPT_STR_VAL_TIME: {
@@ -674,7 +708,7 @@ static void gopt_add_option(GtkWidget *hbox, struct fio_option *o,
 		if (o->off1)
 			ullp = td_var(to, o->off1);
 
-		go = gopt_new_ullong(o, ullp, opt_index);
+		go = gopt_new_ullong(gjv, o, ullp, opt_index);
 		break;
 		}
 	case FIO_OPT_INT: {
@@ -683,7 +717,7 @@ static void gopt_add_option(GtkWidget *hbox, struct fio_option *o,
 		if (o->off1)
 			ip = td_var(to, o->off1);
 
-		go = gopt_new_int(o, ip, opt_index);
+		go = gopt_new_int(gjv, o, ip, opt_index);
 		break;
 		}
 	case FIO_OPT_STR_SET:
@@ -693,7 +727,7 @@ static void gopt_add_option(GtkWidget *hbox, struct fio_option *o,
 		if (o->off1)
 			ip = td_var(to, o->off1);
 
-		go = gopt_new_bool(o, ip, opt_index);
+		go = gopt_new_bool(gjv, o, ip, opt_index);
 		break;
 		}
 	case FIO_OPT_STR: {
@@ -703,10 +737,10 @@ static void gopt_add_option(GtkWidget *hbox, struct fio_option *o,
 			if (o->off1)
 				ip = td_var(to, o->off1);
 
-			go = gopt_new_combo_int(o, ip, opt_index);
+			go = gopt_new_combo_int(gjv, o, ip, opt_index);
 		} else {
 			/* TODO: usually ->cb, or unsigned int pointer */
-			go = gopt_new_str_store(o, NULL, opt_index);
+			go = gopt_new_str_store(gjv, o, NULL, opt_index);
 		}
 
 		break;
@@ -720,15 +754,15 @@ static void gopt_add_option(GtkWidget *hbox, struct fio_option *o,
 		}
 
 		if (!o->posval[0].ival) {
-			go = gopt_new_str_store(o, text, opt_index);
+			go = gopt_new_str_store(gjv, o, text, opt_index);
 			break;
 		}
 
-		go = gopt_new_combo_str(o, text, opt_index);
+		go = gopt_new_combo_str(gjv, o, text, opt_index);
 		break;
 		}
 	case FIO_OPT_STR_MULTI:
-		go = gopt_new_str_multi(o, opt_index);
+		go = gopt_new_str_multi(gjv, o, opt_index);
 		break;
 	case FIO_OPT_RANGE: {
 		unsigned int *ip[4] = { td_var(to, o->off1),
@@ -736,7 +770,7 @@ static void gopt_add_option(GtkWidget *hbox, struct fio_option *o,
 					td_var(to, o->off3),
 					td_var(to, o->off4) };
 
-		go = gopt_new_int_range(o, ip, opt_index);
+		go = gopt_new_int_range(gjv, o, ip, opt_index);
 		break;
 		}
 	/* still need to handle this one */
@@ -756,8 +790,9 @@ static void gopt_add_option(GtkWidget *hbox, struct fio_option *o,
 			gtk_widget_set_tooltip_text(go->box, o->help);
 
 		go->opt_type = o->type;
+		o->gui_data = go;
 
-		dest = gopt_get_group_frame(hbox, o->group);
+		dest = gopt_get_group_frame(gjv, hbox, o->group);
 		if (!dest)
 			gtk_box_pack_start(GTK_BOX(hbox), go->box, FALSE, FALSE, 5);
 		else
@@ -765,7 +800,8 @@ static void gopt_add_option(GtkWidget *hbox, struct fio_option *o,
 	}
 }
 
-static void gopt_add_options(GtkWidget **vboxes, struct thread_options *to)
+static void gopt_add_options(struct gopt_job_view *gjv,
+			     struct thread_options *to)
 {
 	GtkWidget *hbox = NULL;
 	int i;
@@ -779,16 +815,16 @@ static void gopt_add_options(GtkWidget **vboxes, struct thread_options *to)
 		struct opt_group *og;
 
 		while ((og = opt_group_from_mask(&mask)) != NULL) {
-			GtkWidget *vbox = vboxes[ffz(~og->mask)];
+			GtkWidget *vbox = gjv->vboxes[ffz(~og->mask)];
 
 			hbox = gtk_hbox_new(FALSE, 3);
 			gtk_box_pack_start(GTK_BOX(vbox), hbox, FALSE, FALSE, 5);
-			gopt_add_option(hbox, o, i, to);
+			gopt_add_option(gjv, hbox, o, i, to);
 		}
 	}
 }
 
-static GtkWidget *gopt_add_group_tab(GtkWidget *notebook, struct opt_group *og)
+static GtkWidget *gopt_add_tab(GtkWidget *notebook, const char *name)
 {
 	GtkWidget *box, *vbox, *scroll;
 
@@ -800,12 +836,16 @@ static GtkWidget *gopt_add_group_tab(GtkWidget *notebook, struct opt_group *og)
 	box = gtk_hbox_new(FALSE, 0);
 	gtk_box_pack_start(GTK_BOX(vbox), box, FALSE, FALSE, 5);
 	gtk_scrolled_window_add_with_viewport(GTK_SCROLLED_WINDOW(scroll), vbox);
-	gtk_notebook_append_page(GTK_NOTEBOOK(notebook), scroll, gtk_label_new(og->name));
-
+	gtk_notebook_append_page(GTK_NOTEBOOK(notebook), scroll, gtk_label_new(name));
 	return vbox;
 }
 
-static void gopt_add_group_tabs(GtkWidget *notebook, GtkWidget **vbox)
+static GtkWidget *gopt_add_group_tab(GtkWidget *notebook, struct opt_group *og)
+{
+	return gopt_add_tab(notebook, og->name);
+}
+
+static void gopt_add_group_tabs(GtkWidget *notebook, struct gopt_job_view *gjv)
 {
 	struct opt_group *og;
 	unsigned int i;
@@ -817,15 +857,26 @@ static void gopt_add_group_tabs(GtkWidget *notebook, GtkWidget **vbox)
 		og = opt_group_from_mask(&mask);
 		if (!og)
 			break;
-		vbox[i] = gopt_add_group_tab(notebook, og);
+		gjv->vboxes[i] = gopt_add_group_tab(notebook, og);
 		i++;
 	} while (1);
 }
 
-void gopt_get_options_window(GtkWidget *window, struct thread_options *o)
+void gopt_get_options_window(GtkWidget *window, struct gfio_client *gc)
 {
-	GtkWidget *dialog, *notebook;
-	GtkWidget *vboxes[__FIO_OPT_C_NR];
+	GtkWidget *dialog, *notebook, *topnotebook, *vbox;
+	struct gfio_client_options *gco;
+	struct thread_options *o;
+	struct flist_head *entry;
+	struct gopt_job_view *gjv;
+	FLIST_HEAD(gjv_list);
+
+	/*
+	 * Just choose the first item, we need to make each options
+	 * entry the main notebook, with the below options view as
+	 * a sub-notebook
+	 */
+	assert(!flist_empty(&gc->o_list));
 
 	dialog = gtk_dialog_new_with_buttons("Fio options",
 			GTK_WINDOW(window), GTK_DIALOG_DESTROY_WITH_PARENT,
@@ -834,20 +885,83 @@ void gopt_get_options_window(GtkWidget *window, struct thread_options *o)
 
 	gtk_widget_set_size_request(GTK_WIDGET(dialog), 1024, 768);
 
-	notebook = gtk_notebook_new();
-	gtk_notebook_set_scrollable(GTK_NOTEBOOK(notebook), 1);
-	gtk_notebook_popup_enable(GTK_NOTEBOOK(notebook));
-	gtk_box_pack_start(GTK_BOX(GTK_DIALOG(dialog)->vbox), notebook, TRUE, TRUE, 5);
+	topnotebook = gtk_notebook_new();
+	gtk_notebook_set_scrollable(GTK_NOTEBOOK(topnotebook), 1);
+	gtk_notebook_popup_enable(GTK_NOTEBOOK(topnotebook));
+	gtk_box_pack_start(GTK_BOX(GTK_DIALOG(dialog)->vbox), topnotebook, TRUE, TRUE, 5);
+
+	flist_for_each(entry, &gc->o_list) {
+		const char *name;
 
-	gopt_add_group_tabs(notebook, vboxes);
+		gco = flist_entry(entry, struct gfio_client_options, list);
+		o = &gco->o;
+		name = o->name;
+		if (!name || !strlen(name))
+			name = "Default job";
 
-	gopt_add_options(vboxes, o);
+		vbox = gopt_add_tab(topnotebook, name);
+
+		notebook = gtk_notebook_new();
+		gtk_notebook_set_scrollable(GTK_NOTEBOOK(notebook), 1);
+		gtk_notebook_popup_enable(GTK_NOTEBOOK(notebook));
+		gtk_box_pack_start(GTK_BOX(vbox), notebook, TRUE, TRUE, 5);
+
+		gjv = calloc(1, sizeof(*gjv));
+		INIT_FLIST_HEAD(&gjv->list);
+		flist_add_tail(&gjv->list, &gjv_list);
+		gopt_add_group_tabs(notebook, gjv);
+		gopt_add_options(gjv, o);
+	}
 
 	gtk_widget_show_all(dialog);
 
 	gtk_dialog_run(GTK_DIALOG(dialog));
 
 	gtk_widget_destroy(dialog);
-	memset(gopt_widgets, 0, sizeof(gopt_widgets));
-	memset(gopt_g_widgets, 0, sizeof(gopt_g_widgets));
+
+	while (!flist_empty(&gjv_list)) {
+		gjv = flist_entry(gjv_list.next, struct gopt_job_view, list);
+		flist_del(&gjv->list);
+		free(gjv);
+	}
+}
+
+/*
+ * Build n-ary option dependency tree
+ */
+void gopt_init(void)
+{
+	int i;
+
+	gopt_dep_tree = g_node_new(NULL);
+
+	for (i = 0; fio_options[i].name; i++) {
+		struct fio_option *o = &fio_options[i];
+		GNode *node, *nparent;
+
+		/*
+		 * Insert node with either the root parent, or an
+		 * option parent.
+		 */
+		node = g_node_new(o);
+		nparent = gopt_dep_tree;
+		if (o->parent) {
+			struct fio_option *parent;
+
+			parent = fio_option_find(o->parent);
+			nparent = g_node_find(gopt_dep_tree, G_IN_ORDER, G_TRAVERSE_ALL, parent);
+			if (!nparent) {
+				log_err("fio: did not find parent %s for opt %s\n", o->name, o->parent);
+				nparent = gopt_dep_tree;
+			}
+		}
+
+		g_node_insert(nparent, -1, node);
+	}
+}
+
+void gopt_exit(void)
+{
+	g_node_destroy(gopt_dep_tree);
+	gopt_dep_tree = NULL;
 }