Signed-off-by: Jens Axboe <axboe@fb.com>
static FLIST_HEAD(engine_list);
static FLIST_HEAD(engine_list);
-static int check_engine_ops(struct ioengine_ops *ops)
+static bool check_engine_ops(struct ioengine_ops *ops)
{
if (ops->version != FIO_IOOPS_VERSION) {
log_err("bad ioops version %d (want %d)\n", ops->version,
FIO_IOOPS_VERSION);
{
if (ops->version != FIO_IOOPS_VERSION) {
log_err("bad ioops version %d (want %d)\n", ops->version,
FIO_IOOPS_VERSION);
}
if (!ops->queue) {
log_err("%s: no queue handler\n", ops->name);
}
if (!ops->queue) {
log_err("%s: no queue handler\n", ops->name);
}
/*
* sync engines only need a ->queue()
*/
if (ops->flags & FIO_SYNCIO)
}
/*
* sync engines only need a ->queue()
*/
if (ops->flags & FIO_SYNCIO)
- if (!ops->event) {
- log_err("%s: no event handler\n", ops->name);
- return 1;
- }
- if (!ops->getevents) {
- log_err("%s: no getevents handler\n", ops->name);
- return 1;
+ if (!ops->event || !ops->getevents) {
+ log_err("%s: no event/getevents handler\n", ops->name);
+ return true;
}
void unregister_ioengine(struct ioengine_ops *ops)
}
void unregister_ioengine(struct ioengine_ops *ops)