Signed-off-by: Jens Axboe <axboe@kernel.dk>
-static void remove_client(struct fio_client *client)
+void fio_put_client(struct fio_client *client)
- assert(client->refs);
-
if (--client->refs)
return;
if (--client->refs)
return;
+ free(client->hostname);
+ if (client->argv)
+ free(client->argv);
+ if (client->name)
+ free(client->name);
+
+ free(client);
+}
+
+static void remove_client(struct fio_client *client)
+{
+ assert(client->refs);
+
dprint(FD_NET, "client: removed <%s>\n", client->hostname);
dprint(FD_NET, "client: removed <%s>\n", client->hostname);
- flist_del(&client->list);
+
+ if (!flist_empty(&client->list))
+ flist_del_init(&client->list);
fio_client_remove_hash(client);
fio_client_remove_hash(client);
fio_client_dec_jobs_eta(client->eta_in_flight, client->ops->eta);
}
fio_client_dec_jobs_eta(client->eta_in_flight, client->ops->eta);
}
- free(client->hostname);
- if (client->argv)
- free(client->argv);
- if (client->name)
- free(client->name);
-
- free(client);
nr_clients--;
sum_stat_clients--;
nr_clients--;
sum_stat_clients--;
-void fio_put_client(struct fio_client *client)
-{
- remove_client(client);
+ fio_put_client(client);
}
struct fio_client *fio_get_client(struct fio_client *client)
}
struct fio_client *fio_get_client(struct fio_client *client)
int ret;
ret = __fio_client_send_ini(client, filename);
int ret;
ret = __fio_client_send_ini(client, filename);
- if (ret) {
- remove_client(client);
- return ret;
- }
+ if (!ret)
+ client->sent_job = 1;
flist_for_each_safe(entry, tmp, &client_list) {
client = flist_entry(entry, struct fio_client, list);
flist_for_each_safe(entry, tmp, &client_list) {
client = flist_entry(entry, struct fio_client, list);
- fio_client_send_ini(client, filename);
+ if (fio_client_send_ini(client, filename))
+ remove_client(client);