gfio: add View -> Results menu item

We store the incoming results, so that we can re-display them if
asked to. The menu item is sensitized appropriately depending
on whether results are available or not.

Signed-off-by: Jens Axboe <axboe@kernel.dk>

server: add missing IOLOG command to text list

Signed-off-by: Jens Axboe <axboe@kernel.dk>

gfio: add completion percentile bar graph

Mostly just to display the bar graph for now, we'll change this to
be the latency buckets instead.

Signed-off-by: Jens Axboe <axboe@kernel.dk>

gfio: only parse white color once

Signed-off-by: Stephen M. Cameron <stephenmcameron@gmail.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>

gfio: include name of iolog in gdio_client_iolog()

Signed-off-by: Jens Axboe <axboe@kernel.dk>

client/server: zlib error code fixup

Signed-off-by: Jens Axboe <axboe@kernel.dk>

server: add vectored fio_send_cmd_ext_pdu()

And use it for the multi segment continue type sending that
the iolog handler uses.

Signed-off-by: Jens Axboe <axboe@kernel.dk>

gfio: debug dump on iolog receive

Signed-off-by: Jens Axboe <axboe@kernel.dk>

server: add fio_sendv_data()

Signed-off-by: Jens Axboe <axboe@kernel.dk>

gfio: remove stale comment, hiding menu items DOES work now

Signed-off-by: Jens Axboe <axboe@kernel.dk>

gfio: start of results window menu support

Signed-off-by: Jens Axboe <axboe@kernel.dk>

gfio: fix double alloc of action_group

Signed-off-by: Jens Axboe <axboe@kernel.dk>

gfio: patch around widget destroy crash

Signed-off-by: Jens Axboe <axboe@kernel.dk>

server: memset job_add pdu before filling and sending

Signed-off-by: Jens Axboe <axboe@kernel.dk>

server: convert iolog in place instead of copy allocating it

Signed-off-by: Jens Axboe <axboe@kernel.dk>

gfio: Add support for sending logs over the network

This is for things like bw log, latency logs, etc. We add
a new depedency, zlib, for this. The logs can be quite big and
the network is not necessarily super fast, so we compress them
using zlib.

Signed-off-by: Jens Axboe <axboe@kernel.dk>

iolog: note type of log

Signed-off-by: Jens Axboe <axboe@kernel.dk>

client/server: pass type of client to the backend

Signed-off-by: Jens Axboe <axboe@kernel.dk>

client: unify cmd handling ops

Signed-off-by: Jens Axboe <axboe@kernel.dk>

graph: move graphs a bit to the left, now we don't use Y axis labels

Signed-off-by: Jens Axboe <axboe@kernel.dk>

gfio: terminate and destroy clients before calling gtk_main_quit()

Signed-off-by: Jens Axboe <axboe@kernel.dk>

gfio: graphing updates

- Don't draw tickmark labels for an empty graph
- Remove Y-axis label. It was a bit awkward and hard to read.
- Ensure that we always put proper units on the Y-axis graph.
- Ensure that we always include the top most Y-axis tickmark
  and make that the ceiling, not the highest Y value. It's hard
  to visually understand a graph that is drawing somewhere above
  the last labeled tickmark.
- Fix rounding errors in graph.c

Signed-off-by: Jens Axboe <axboe@kernel.dk>

gfio: fix complaint about unsupported type for recent files

Signed-off-by: Jens Axboe <axboe@kernel.dk>

gfio: fix leak of ge->job_files

Signed-off-by: Jens Axboe <axboe@kernel.dk>

gfio: update TODO

We have recent file support now.

Signed-off-by: Jens Axboe <axboe@kernel.dk>

gfio: Add Drag'n drop support

And unify the file-open-in-a-tab functionality.

Signed-off-by: Jens Axboe <axboe@kernel.dk>

Rename t0001 regression job to have .fio file extension

Signed-off-by: Jens Axboe <axboe@kernel.dk>

Merge branch 'master' into gfio

Conflicts:
client.c
fio.h

Signed-off-by: Jens Axboe <axboe@kernel.dk>

Add regression test for 82af2a7c

Signed-off-by: Jens Axboe <axboe@kernel.dk>

Fix failure to verify in mixed read/write workload with backlog

If you run a workload like this:

fio --rw=randrw --bs=4k --direct=1 --ioengine=libaio --iodepth=32
--verify=meta --verify_backlog=1024 --verify_fatal=1 --name=ver-test
--filename=foo --size=1G --verify_pattern=0xaaa

Fio ends up never actually verifying the written blocks. This happens
because as we generate an entry to be verified, the backend checks
whether this is a read/write mixed workload. It then thinks that the
READ is just that, a normal READ, when in fact it could be coming
from our verify list.

Mark such a read as coming from our verify list, so that we know if
it's a "normal" read or one generated to verify previously written
data.

Signed-off-by: Jens Axboe <axboe@kernel.dk>

gfio: add recently opened files to File menu

Signed-off-by: Jens Axboe <axboe@kernel.dk>

gfio: change Job->Connect menu item string

Change it from Connect <-> Disconnect, same as how we handle
the button.

Signed-off-by: Jens Axboe <axboe@kernel.dk>

Add gfio TODO file

Not complete, but it's a start.

Signed-off-by: Jens Axboe <axboe@kernel.dk>

gfio: remove widgets from struct gui/gui_entry that we don't really use

Lots of these are only used for initial setup of the page, other
entries are the ones we reuse later. So kill the entries to avoid
confusing them post init.

Signed-off-by: Jens Axboe <axboe@kernel.dk>

Client reference fixes

Signed-off-by: Jens Axboe <axboe@kernel.dk>

gfio: add support for graph tooltips

Will show the actual value logged at that point.

Signed-off-by: Jens Axboe <axboe@kernel.dk>

gfio: allow close with ctrl-w if there are no clients

Signed-off-by: Jens Axboe <axboe@kernel.dk>

Merge branch 'gfio' of ssh://brick.kernel.dk/data/git/fio into gfio

gfio: fix hiding of menu entries

It helps a lot to use the proper casing...

Signed-off-by: Jens Axboe <axboe@kernel.dk>

gfio: fix crash when pushing 'start job' and no client connected

We should not really need this fix, but until the menu item
sensitivity is fixed, they are visible and we should avoid
the crash.

Signed-off-by: Jens Axboe <axboe@kernel.dk>

Makefile: properly clean all objects on make clean

Signed-off-by: Jens Axboe <axboe@kernel.dk>

gfio: fix wrong pointer being passed into file_open()

We passed the gui_entry, it wants the gui. Fixes segfault
on clicking "Connect", when attempting to auto-load the
file open dialog.

Signed-off-by: Jens Axboe <axboe@kernel.dk>

gfio: prep for recent files

Signed-off-by: Jens Axboe <axboe@kernel.dk>

gfio: improve Job menu

Still need to figure out how to make certain items in the
job menu sensitive/insensitive, apparently I can't lookup
the item widgets (only the menu widget).

Signed-off-by: Jens Axboe <axboe@kernel.dk>

gfio: add support for making job menu visible or not

Signed-off-by: Jens Axboe <axboe@kernel.dk>

gfio: add close-tab functionality

And start Job menu.

Signed-off-by: Jens Axboe <axboe@kernel.dk>

gfio: prevent multitext crash on invalid update

Signed-off-by: Jens Axboe <axboe@kernel.dk>

Group options into categories

Each option can belong to several categories. This is going to
help us make some sort of sane GUI representation of options.

Signed-off-by: Jens Axboe <axboe@kernel.dk>

gfio: pack per-job info in a multitext structure

Just like the name entry, we can have differing io engines, depth,
etc. Allow these to be stored and set when the user selects the
appropriate entry in the combo box.

Signed-off-by: Jens Axboe <axboe@kernel.dk>

Only send job options once

If we have numjobs=x set with x higher than 1, the CLI client
prints the first and last job. But for the GUI, we just want
the one notification of an added job.

This fixes the case where we seem to have loaded two jobs,
but it is just one.

Signed-off-by: Jens Axboe <axboe@kernel.dk>

gfio: add extra space to the graphs at the edges

This is to make it easier to read and also keeps the lines from
going beyond the edges of the graph.

Signed-off-by: Stephen M. Cameron <stephenmcameron@gmail.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>

gfio: fix tickmark shortening for zero special case

"0" tickmark can effectively be shortened by dividing by
any number as 0 / anything is still 0, but the code was
considering "0" to be unshortenable.

Signed-off-by: Stephen M. Cameron <stephenmcameron@gmail.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>

graph: drop more than 1 entry, if we are more than 1 above the limit

If we dynamically reduce the data entry limit, it's not enough to
drop one entry when we add one. In that case, drop two everytime
we add one. Then we'll eventually get smoothly down to the specified
limit.

Signed-off-by: Jens Axboe <axboe@kernel.dk>

gfio: use current value of graph limit as the default

Signed-off-by: Jens Axboe <axboe@kernel.dk>

gfio: encapsulate x- and y-offsets into graph library

Don't need to carry around drawing area dimensions alongside
the graphs, the size and xy-offsets of the graphs can be set
directly in the drawing area configure callback.

Signed-off-by: Stephen M. Cameron <stephenmcameron@gmail.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>

gfio: add comments to interface of graph mini library

Signed-off-by: Stephen M. Cameron <stephenmcameron@gmail.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>

gfio: use graph axis unit change notification callbacks

Signed-off-by: Stephen M. Cameron <stephenmcameron@gmail.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>

gfio: add graph axis unit change notification callbacks

This enables code that uses the graph functions to change the axis
titles on the fly to display unit information which matches how the
tick labels are shortened.

Signed-off-by: Stephen M. Cameron <scameron@beardog.cce.hp.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>

gfio: fix discrepancies between graph header and implementation

Header defined a couple functions as returning void, but they really return int.

Signed-off-by: Stephen M. Cameron <stephenmcameron@gmail.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>

gfio: do not mix K M and G when shortening axis labels

That is, do not scale axis labels by differing amounts
like 900K, 1M, 1100K, instead: 900k, 1000k, 1100k.

Signed-off-by: Stephen M. Cameron <stephenmcameron@gmail.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>

gfio: move debug preferences last

Signed-off-by: Jens Axboe <axboe@kernel.dk>

gfio: add eta msec interval preference

Signed-off-by: Jens Axboe <axboe@kernel.dk>

client: make ETA interval poll period configurage in fio_client_ops

Signed-off-by: Jens Axboe <axboe@kernel.dk>

gfio: add preference entry for number of values to keep in graph

It's not really seconds, but pretty close since ETA interval is
close to that now. Will make it really seconds later.

Signed-off-by: Jens Axboe <axboe@kernel.dk>

Mark buffer_compress_chunk as child of buffer_compress_percentage

Signed-off-by: Jens Axboe <axboe@kernel.dk>

Add buffer_compress_chunk option

Instead of doing bs/2 of incompressible and zeroed data for a 50%
compression setting, use the smaller chunk size specified by
this option.

Signed-off-by: Jens Axboe <axboe@kernel.dk>

Honor refill_buffers even for verifies

Signed-off-by: Jens Axboe <axboe@kernel.dk>

Compression buffer fill fix

We need to advance after the zero fill as well.

Reported-by: David Sterba <dsterba@suse.cz>
Signed-off-by: Jens Axboe <axboe@kernel.dk>

client: send probe from fio_client_connect()

We only do it from fio_clients_connect() now, better move it
into the lower per-client handler. This gets probes working
with gfio again.

Signed-off-by: Jens Axboe <axboe@kernel.dk>

gfio: better handle multiple threads/processes in a job

Make the name entry a combo, before we were just overwriting
so it would contain the last entry noted.

Signed-off-by: Jens Axboe <axboe@kernel.dk>

gfio: make results notebook scrollable too

And also adds menus to both results and job notebook.

Signed-off-by: Jens Axboe <axboe@kernel.dk>

gfio: make notebooks scrollable if there are too many

Signed-off-by: Jens Axboe <axboe@kernel.dk>

gfio: hold a reference to the fio_client structure

Signed-off-by: Jens Axboe <axboe@kernel.dk>

gfio: more error reporting

Signed-off-by: Jens Axboe <axboe@kernel.dk>

server: return real error from fio_send_data()

Signed-off-by: Jens Axboe <axboe@kernel.dk>

client: return more errors

Signed-off-by: Jens Axboe <axboe@kernel.dk>

client: return errno on connection failure

Signed-off-by: Jens Axboe <axboe@kernel.dk>

gfio: handle clients individually

Now we have per-client support, don't use any of the "do FOO for
all clients" functions.

Signed-off-by: Jens Axboe <axboe@kernel.dk>

gfio: only start one client net handler

Signed-off-by: Jens Axboe <axboe@kernel.dk>

gfio: start of support for notebooked jobs

We split struct gui into a main window and per-client part,
having a separate style of layout for each.

This needs a lot of work still, but it gets the point across.

Signed-off-by: Jens Axboe <axboe@kernel.dk>

gfio: ignore group stats for now

Signed-off-by: Jens Axboe <axboe@kernel.dk>

Add basic cconv test

Signed-off-by: Jens Axboe <axboe@kernel.dk>

Separate the act of adding and starting a job

CMD_JOB or CMD_JOB_LINE only adds the necessary jobs, which will
automatically return the job option info to the client. Let the
client do what it wants with the options, then have the client
send a CMD_RUN command to actually start the jobs.

Next we'll have a command for passing back edited options, so that
the client can update existing options.

This adds a new button to gfio, to note this split of add and run
as well.

Signed-off-by: Jens Axboe <axboe@kernel.dk>

example: bump netio example to 100g transfer

Signed-off-by: Jens Axboe <axboe@kernel.dk>

server: rename CMD_RUN to CMD_SERVER_START

Signed-off-by: Jens Axboe <axboe@kernel.dk>

client/server: pass back packed thread_options in add_job()

Now the client can see the parsed options. As it turns out,
we need to parse locally since the client and server can be
vastly different operating systems and architectures. Pass
back the options, allowing the client to change them before
starting the job.

Signed-off-by: Jens Axboe <axboe@kernel.dk>

gfio: Add y-axis labels

"Kbytes /sec" for bandwidth, and "IOs / sec" for IOPS.

Signed-off-by: Stephen M. Cameron <stephenmcameron@gmail.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>

gfio: move graph area a bit to the right

To make more room for y-axis title text.

Signed-off-by: Stephen M. Cameron <stephenmcameron@gmail.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>

gfio: fix a bug in the y-axis label shortening algorithm

Signed-off-by: Stephen M. Cameron <stephenmcameron@gmail.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>

Add thread_options_pack structure

First step towards having an on-disk and on-wire independent
options structure, with the goal being that we parse options
locally (hence allowing them to be edited sanely) and send them
to the server in one nice package.

Additionally, this is a step towards being able to have a new
fio/gfio "project" file format, which includes all the info a
given client needs. This includes job options, where to connect,
etc.

Signed-off-by: Jens Axboe <axboe@kernel.dk>

Abstract out the thread_options structure

We need to prepare to have an on-disk/net version of the same.

Signed-off-by: Jens Axboe <axboe@kernel.dk>

client/server: store/convert/send jobs_eta->nr_threads too

We still need to handle jobs_eta->run_str[] as well...

Signed-off-by: Jens Axboe <axboe@kernel.dk>

gfio: improve stop handling

Signed-off-by: Jens Axboe <axboe@kernel.dk>

gfio: make the end results window a scrolled one

Signed-off-by: Jens Axboe <axboe@kernel.dk>

gfio: don't use gtk_dialog_get_content_area()

It's not available in gtk 2.10, which we'd like to work with.

Signed-off-by: Jens Axboe <axboe@kernel.dk>

gfio: add missing disk utilization stat to end results

Signed-off-by: Jens Axboe <axboe@kernel.dk>

gfio: allow graph to stretch or shrink as window expands or contracts

Signed-off-by: Stephen M. Cameron <stephenmcameron@gmail.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>

gfio: right justify y-axis labels

Signed-off-by: Jens Axboe <axboe@kernel.dk>

gfio: fix problem with graph finding data range.

In finding minimum value, it used zero as an initializer
but if the graph had no data below say, 1000, it would still
find zero as the minimum, which was incorrect.

Signed-off-by: Stephen M. Cameron <scameron@beardog.cce.hp.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>

gfio: get rid of invisible data

At first the invisible data seemed like an elegant way to get
reasonable graph ticks when no data was really present, but it
proved to be an ugly hack that caused other problems later,e.g.
when you want to cap other data so the xy graph scrolls, the
invisible data kept it from scrolling, etc.

Signed-off-by: Stephen M. Cameron <stephenmcameron@gmail.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>