gfio: make option hiding actually work
[fio.git] / goptions.c
index 98a05fa1ee510b626bec18f6847dfddd0a456746..d7178eb68aad07a4e27473af9caed4a6e68a3fc8 100644 (file)
@@ -37,12 +37,80 @@ struct gopt_str {
        GtkWidget *entry;
 };
 
+#define GOPT_RANGE_SPIN        4
+
 struct gopt_range {
        struct gopt gopt;
-       GtkWidget *spins[4];
+       GtkWidget *spins[GOPT_RANGE_SPIN];
+};
+
+struct gopt_widget {
+       struct flist_head list;
+       GtkWidget *widget;
 };
 
-static struct gopt *gopt_new_str_store(struct fio_option *o, const char *text)
+static struct flist_head gopt_list[FIO_MAX_OPTS];
+
+static void __gopt_set_children_visible(unsigned int idx, gboolean visible)
+{
+       struct flist_head *entry;
+       struct gopt_widget *gw;
+
+       flist_for_each(entry, &gopt_list[idx]) {
+               gw = flist_entry(entry, struct gopt_widget, list);
+               gtk_widget_set_sensitive(gw->widget, visible);
+       }
+}
+
+/*
+ * Mark children as invisible, if needed.
+ */
+static void gopt_set_children_visible(struct fio_option *parent, gboolean visible)
+{
+       struct fio_option *o;
+       int i;
+
+       /*
+        * 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 (strcmp(parent->name, o->parent))
+                       continue;
+
+               __gopt_set_children_visible(i, visible);
+       }
+}
+
+static void gopt_str_changed(GtkEntry *entry, gpointer data)
+{
+       struct gopt_str *s = (struct gopt_str *) data;
+       struct fio_option *o = &fio_options[s->gopt.opt_index];
+       const gchar *text;
+       int set;
+
+       text = gtk_entry_get_text(GTK_ENTRY(s->entry));
+       set = strcmp(text, "") != 0;
+       gopt_set_children_visible(o, set);
+}
+
+static void gopt_mark_index(struct gopt *gopt, unsigned int idx)
+{
+       struct gopt_widget *gw;
+
+       gopt->opt_index = idx;
+
+       gw = malloc(sizeof(*gw));
+       gw->widget = gopt->box;
+       flist_add_tail(&gw->list, &gopt_list[idx]);
+}
+
+static struct gopt *gopt_new_str_store(struct fio_option *o, const char *text, unsigned int idx)
 {
        struct gopt_str *s;
        GtkWidget *label;
@@ -54,9 +122,11 @@ static struct gopt *gopt_new_str_store(struct fio_option *o, const char *text)
        gtk_box_pack_start(GTK_BOX(s->gopt.box), label, FALSE, FALSE, 0);
 
        s->entry = gtk_entry_new();
+       gopt_mark_index(&s->gopt, idx);
        if (text)
                gtk_entry_set_text(GTK_ENTRY(s->entry), text);
        gtk_entry_set_editable(GTK_ENTRY(s->entry), 1);
+       g_signal_connect(GTK_OBJECT(s->entry), "changed", G_CALLBACK(gopt_str_changed), s);
 
        if (o->def)
                gtk_entry_set_text(GTK_ENTRY(s->entry), o->def);
@@ -65,30 +135,41 @@ static struct gopt *gopt_new_str_store(struct fio_option *o, const char *text)
        return &s->gopt;
 }
 
-static struct gopt_combo *__gopt_new_combo(struct fio_option *o)
+static void gopt_combo_changed(GtkComboBox *box, gpointer data)
 {
-       struct gopt_combo *combo;
+       struct gopt_combo *c = (struct gopt_combo *) data;
+       struct fio_option *o = &fio_options[c->gopt.opt_index];
+
+       printf("combo %s changed\n", o->name);
+}
+
+static struct gopt_combo *__gopt_new_combo(struct fio_option *o, unsigned int idx)
+{
+       struct gopt_combo *c;
        GtkWidget *label;
 
-       combo = malloc(sizeof(*combo));
+       c = malloc(sizeof(*c));
 
-       combo->gopt.box = gtk_hbox_new(FALSE, 3);
+       c->gopt.box = gtk_hbox_new(FALSE, 3);
        label = gtk_label_new(o->name);
-       gtk_box_pack_start(GTK_BOX(combo->gopt.box), label, FALSE, FALSE, 0);
+       gtk_box_pack_start(GTK_BOX(c->gopt.box), label, FALSE, FALSE, 0);
 
-       combo->combo = gtk_combo_box_new_text();
-       gtk_box_pack_start(GTK_BOX(combo->gopt.box), combo->combo, FALSE, FALSE, 0);
+       c->combo = gtk_combo_box_new_text();
+       gopt_mark_index(&c->gopt, idx);
+       gtk_box_pack_start(GTK_BOX(c->gopt.box), c->combo, FALSE, FALSE, 0);
 
-       return combo;
+       g_signal_connect(GTK_OBJECT(c->combo), "changed", G_CALLBACK(gopt_combo_changed), 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 fio_option *o, const char *text, unsigned int idx)
 {
        struct gopt_combo *combo;
        struct value_pair *vp;
        int i, active = 0;
 
-       combo = __gopt_new_combo(o);
+       combo = __gopt_new_combo(o, idx);
 
        i = 0;
        vp = &o->posval[0];
@@ -106,13 +187,13 @@ static struct gopt *gopt_new_combo_str(struct fio_option *o, const char *text)
        return &combo->gopt;
 }
 
-static struct gopt *gopt_new_combo_int(struct fio_option *o, unsigned int *ip)
+static struct gopt *gopt_new_combo_int(struct fio_option *o, unsigned int *ip, unsigned int idx)
 {
        struct gopt_combo *combo;
        struct value_pair *vp;
        int i, active = 0;
 
-       combo = __gopt_new_combo(o);
+       combo = __gopt_new_combo(o, idx);
 
        i = 0;
        vp = &o->posval[0];
@@ -128,11 +209,20 @@ static struct gopt *gopt_new_combo_int(struct fio_option *o, unsigned int *ip)
        return &combo->gopt;
 }
 
-static struct gopt *__gopt_new_int(struct fio_option *o, unsigned long long *p)
+static void gopt_int_changed(GtkSpinButton *spin, gpointer data)
+{
+       struct gopt_int *i = (struct gopt_int *) data;
+       struct fio_option *o = &fio_options[i->gopt.opt_index];
+
+       printf("int %s changed\n", o->name);
+}
+
+static struct gopt_int *__gopt_new_int(struct fio_option *o, unsigned long long *p,
+                                      unsigned int idx)
 {
        unsigned long long defval;
        struct gopt_int *i;
-       guint maxval;
+       guint maxval, interval;
        GtkWidget *label;
 
        i = malloc(sizeof(*i));
@@ -154,32 +244,56 @@ static struct gopt *__gopt_new_int(struct fio_option *o, unsigned long long *p)
                defval = val;
        }
 
-       i->spin = gtk_spin_button_new_with_range(o->minval, maxval, 1.0);
+       interval = 1.0;
+       if (o->interval)
+               interval = o->interval;
+
+       i->spin = gtk_spin_button_new_with_range(o->minval, maxval, interval);
+       gopt_mark_index(&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);
 
        gtk_box_pack_start(GTK_BOX(i->gopt.box), i->spin, FALSE, FALSE, 0);
-       return &i->gopt;
+
+       g_signal_connect(G_OBJECT(i->spin), "value-changed", G_CALLBACK(gopt_int_changed), i);
+
+       return i;
 }
 
-static struct gopt *gopt_new_int(struct fio_option *o, unsigned int *ip)
+static struct gopt *gopt_new_int(struct fio_option *o, unsigned int *ip, unsigned int idx)
 {
        unsigned long long ullp;
+       struct gopt_int *i;
 
        if (ip) {
                ullp = *ip;
-               return __gopt_new_int(o, &ullp);
-       }
+               i = __gopt_new_int(o, &ullp, idx);
+       } else
+               i = __gopt_new_int(o, NULL, idx);
 
-       return __gopt_new_int(o, NULL);
+       return &i->gopt;
 }
 
-static struct gopt *gopt_new_ullong(struct fio_option *o, unsigned long long *p)
+static struct gopt *gopt_new_ullong(struct fio_option *o, unsigned long long *p,
+                                   unsigned int idx)
 {
-       return __gopt_new_int(o, p);
+       struct gopt_int *i;
+
+       i = __gopt_new_int(o, p, idx);
+       return &i->gopt;
 }
 
-static struct gopt *gopt_new_bool(struct fio_option *o, unsigned int *val)
+static void gopt_bool_toggled(GtkToggleButton *button, gpointer data)
+{
+       struct gopt_bool *b = (struct gopt_bool *) data;
+       struct fio_option *o = &fio_options[b->gopt.opt_index];
+       gboolean set;
+
+       set = gtk_toggle_button_get_active(GTK_TOGGLE_BUTTON(b->check));
+       gopt_set_children_visible(o, set);
+}
+
+static struct gopt *gopt_new_bool(struct fio_option *o, unsigned int *val, unsigned int idx)
 {
        struct gopt_bool *b;
        GtkWidget *label;
@@ -191,26 +305,71 @@ static struct gopt *gopt_new_bool(struct fio_option *o, unsigned int *val)
        gtk_box_pack_start(GTK_BOX(b->gopt.box), label, FALSE, FALSE, 0);
 
        b->check = gtk_check_button_new();
+       gopt_mark_index(&b->gopt, idx);
        if (val)
                defstate = *val;
        else if (o->def && !strcmp(o->def, "1"))
                defstate = 1;
 
        gtk_toggle_button_set_active(GTK_TOGGLE_BUTTON(b->check), defstate);
+       g_signal_connect(G_OBJECT(b->check), "toggled", G_CALLBACK(gopt_bool_toggled), b);
 
        gtk_box_pack_start(GTK_BOX(b->gopt.box), b->check, FALSE, FALSE, 0);
        return &b->gopt;
 }
 
-static struct gopt *gopt_new_int_range(struct fio_option *o, unsigned int **ip)
+/*
+ * These are paired 0/1 and 2/3. 0/2 are min values, 1/3 are max values.
+ * If the max is made smaller than min, adjust min down.
+ * If the min is made larger than max, adjust the max.
+ */
+static void range_value_changed(GtkSpinButton *spin, gpointer data)
+{
+       struct gopt_range *r = (struct gopt_range *) data;
+       int changed = -1, i;
+       gint val, mval;
+
+       for (i = 0; i < GOPT_RANGE_SPIN; i++) {
+               if (GTK_SPIN_BUTTON(r->spins[i]) == spin) {
+                       changed = i;
+                       break;
+               }
+       }
+
+       assert(changed != -1);
+
+       /*
+        * Min changed
+        */
+       if (changed == 0 || changed == 2) {
+               GtkWidget *mspin = r->spins[changed + 1];
+
+               val = gtk_spin_button_get_value_as_int(GTK_SPIN_BUTTON(r->spins[changed]));
+               mval = gtk_spin_button_get_value_as_int(GTK_SPIN_BUTTON(mspin));
+               if (val > mval)
+                       gtk_spin_button_set_value(GTK_SPIN_BUTTON(mspin), val);
+       } else {
+               GtkWidget *mspin = r->spins[changed - 1];
+
+               val = gtk_spin_button_get_value_as_int(GTK_SPIN_BUTTON(r->spins[changed]));
+               mval = gtk_spin_button_get_value_as_int(GTK_SPIN_BUTTON(mspin));
+               if (val < mval)
+                       gtk_spin_button_set_value(GTK_SPIN_BUTTON(mspin), val);
+       }
+}
+
+static struct gopt *gopt_new_int_range(struct fio_option *o, unsigned int **ip,
+                                      unsigned int idx)
 {
        struct gopt_range *r;
        gint maxval, defval;
        GtkWidget *label;
+       guint interval;
        int i;
 
        r = malloc(sizeof(*r));
        r->gopt.box = gtk_hbox_new(FALSE, 3);
+       gopt_mark_index(&r->gopt, idx);
        label = gtk_label_new(o->name);
        gtk_box_pack_start(GTK_BOX(r->gopt.box), label, FALSE, FALSE, 0);
 
@@ -226,8 +385,12 @@ static struct gopt *gopt_new_int_range(struct fio_option *o, unsigned int **ip)
                defval = val;
        }
 
-       for (i = 0; i < 4; i++) {
-               r->spins[i] = gtk_spin_button_new_with_range(o->minval, maxval, 512);
+       interval = 1.0;
+       if (o->interval)
+               interval = o->interval;
+
+       for (i = 0; i < GOPT_RANGE_SPIN; i++) {
+               r->spins[i] = gtk_spin_button_new_with_range(o->minval, maxval, interval);
                gtk_spin_button_set_update_policy(GTK_SPIN_BUTTON(r->spins[i]), GTK_UPDATE_IF_VALID);
                if (ip)
                        gtk_spin_button_set_value(GTK_SPIN_BUTTON(r->spins[i]), *ip[i]);
@@ -235,6 +398,7 @@ static struct gopt *gopt_new_int_range(struct fio_option *o, unsigned int **ip)
                        gtk_spin_button_set_value(GTK_SPIN_BUTTON(r->spins[i]), defval);
 
                gtk_box_pack_start(GTK_BOX(r->gopt.box), r->spins[i], FALSE, FALSE, 0);
+               g_signal_connect(G_OBJECT(r->spins[i]), "value-changed", G_CALLBACK(range_value_changed), r);
        }
 
        return &r->gopt;
@@ -253,7 +417,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);
+               go = gopt_new_ullong(o, ullp, opt_index);
                break;
                }
        case FIO_OPT_INT: {
@@ -262,7 +426,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);
+               go = gopt_new_int(o, ip, opt_index);
                break;
                }
        case FIO_OPT_STR_SET:
@@ -272,7 +436,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);
+               go = gopt_new_bool(o, ip, opt_index);
                break;
                }
        case FIO_OPT_STR: {
@@ -281,7 +445,7 @@ 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);
+               go = gopt_new_combo_int(o, ip, opt_index);
                break;
                }
        case FIO_OPT_STR_STORE: {
@@ -293,15 +457,15 @@ static void gopt_add_option(GtkWidget *hbox, struct fio_option *o,
                }
 
                if (!o->posval[0].ival) {
-                       go = gopt_new_str_store(o, text);
+                       go = gopt_new_str_store(o, text, opt_index);
                        break;
                }
 
-               go = gopt_new_combo_str(o, text);
+               go = gopt_new_combo_str(o, text, opt_index);
                break;
                }
        case FIO_OPT_STR_MULTI:
-               go = gopt_new_combo_str(o, NULL);
+               go = gopt_new_combo_str(o, NULL, opt_index);
                break;
        case FIO_OPT_RANGE: {
                unsigned int *ip[4] = { td_var(to, o->off1),
@@ -309,7 +473,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);
+               go = gopt_new_int_range(o, ip, opt_index);
                break;
                }
        /* still need to handle this one */
@@ -327,7 +491,6 @@ static void gopt_add_option(GtkWidget *hbox, struct fio_option *o,
                        gtk_widget_set_tooltip_text(go->box, o->help);
        
                gtk_box_pack_start(GTK_BOX(hbox), go->box, FALSE, FALSE, 5);
-               go->opt_index = opt_index;
                go->opt_type = o->type;
        }
 }
@@ -372,8 +535,12 @@ static GtkWidget *gopt_add_group_tab(GtkWidget *notebook, struct opt_group *og)
 static void gopt_add_group_tabs(GtkWidget *notebook, GtkWidget **vbox)
 {
        struct opt_group *og;
-       unsigned int i = 0;
+       unsigned int i;
+
+       for (i = 0; i < FIO_MAX_OPTS; i++)
+               INIT_FLIST_HEAD(&gopt_list[i]);
 
+       i = 0;
        do {
                unsigned int mask = (1U << i);