Added splat taskq task ordering test case.
This test case verifies the correct behavior of taskq_wait_id(). In particular it ensure the the following two cases are handled properly: 1) Task ids larger than the waited for task id can run and complete as long as there is an available worker thread. 2) All task ids lower than the waited one must complete before unblocking even if the waited task id itself has completed.
This commit is contained in:
Brian Behlendorf
parent
82387586af
commit
5562e5d105
|
|
@ -45,16 +45,30 @@
|
|||
#define SPLAT_TASKQ_TEST4_NAME "wait"
|
||||
#define SPLAT_TASKQ_TEST4_DESC "Multiple task waiting"
|
||||
|
||||
#define SPLAT_TASKQ_TEST5_ID 0x0205
|
||||
#define SPLAT_TASKQ_TEST5_NAME "order"
|
||||
#define SPLAT_TASKQ_TEST5_DESC "Correct task ordering"
|
||||
|
||||
#define SPLAT_TASKQ_ORDER_MAX 8
|
||||
|
||||
typedef struct splat_taskq_arg {
|
||||
int flag;
|
||||
int id;
|
||||
atomic_t count;
|
||||
int order[SPLAT_TASKQ_ORDER_MAX];
|
||||
spinlock_t lock;
|
||||
struct file *file;
|
||||
const char *name;
|
||||
} splat_taskq_arg_t;
|
||||
|
||||
/* Validation Test 1 - Create a taskq, queue a task, wait until
|
||||
* task completes, ensure task ran properly, cleanup taskq,
|
||||
typedef struct splat_taskq_id {
|
||||
int id;
|
||||
splat_taskq_arg_t *arg;
|
||||
} splat_taskq_id_t;
|
||||
|
||||
/*
|
||||
* Create a taskq, queue a task, wait until task completes, ensure
|
||||
* task ran properly, cleanup taskq.
|
||||
*/
|
||||
static void
|
||||
splat_taskq_test13_func(void *arg)
|
||||
|
|
@ -112,9 +126,11 @@ splat_taskq_test1(struct file *file, void *arg)
|
|||
return (tq_arg.flag) ? 0 : -EINVAL;
|
||||
}
|
||||
|
||||
/* Validation Test 2 - Create multiple taskq's, each with multiple tasks,
|
||||
* wait until all tasks complete, ensure all tasks ran properly and in the
|
||||
* the correct order, cleanup taskq's
|
||||
/*
|
||||
* Create multiple taskq's, each with multiple tasks, wait until
|
||||
* all tasks complete, ensure all tasks ran properly and in the
|
||||
* correct order. Run order must be the same as the order submitted
|
||||
* because we only have 1 thread per taskq. Finally cleanup the taskq.
|
||||
*/
|
||||
static void
|
||||
splat_taskq_test2_func1(void *arg)
|
||||
|
|
@ -145,7 +161,7 @@ splat_taskq_test2_func2(void *arg)
|
|||
}
|
||||
|
||||
#define TEST2_TASKQS 8
|
||||
#define TEST2_THREADS_PER_TASKQ 4
|
||||
#define TEST2_THREADS_PER_TASKQ 1
|
||||
|
||||
static int
|
||||
splat_taskq_test2(struct file *file, void *arg) {
|
||||
|
|
@ -238,8 +254,9 @@ splat_taskq_test2(struct file *file, void *arg) {
|
|||
return rc;
|
||||
}
|
||||
|
||||
/* Validation Test 3 - Use the global system task queue with a single
|
||||
* task, * wait until task completes, ensure task ran properly.
|
||||
/*
|
||||
* Use the global system task queue with a single task, wait until task
|
||||
* completes, ensure task ran properly.
|
||||
*/
|
||||
static int
|
||||
splat_taskq_test3(struct file *file, void *arg)
|
||||
|
|
@ -270,6 +287,12 @@ splat_taskq_test3(struct file *file, void *arg)
|
|||
return (tq_arg.flag) ? 0 : -EINVAL;
|
||||
}
|
||||
|
||||
/*
|
||||
* Create a taskq and dispatch a large number of tasks to the queue.
|
||||
* Then use taskq_wait() to block until all the tasks complete, then
|
||||
* cross check that all the tasks ran by checking tg_arg->count which
|
||||
* is incremented in the task function. Finally cleanup the taskq.
|
||||
*/
|
||||
static void
|
||||
splat_taskq_test4_func(void *arg)
|
||||
{
|
||||
|
|
@ -337,6 +360,181 @@ out:
|
|||
return rc;
|
||||
}
|
||||
|
||||
/*
|
||||
* Create a taskq and dispatch a specific sequence of tasks carefully
|
||||
* crafted to validate the order in which tasks are processed. When
|
||||
* there are multiple worker threads each thread will process the
|
||||
* next pending task as soon as it completes its current task. This
|
||||
* means that tasks do not strictly complete in order in which they
|
||||
* were dispatched (increasing task id). This is fine but we need to
|
||||
* verify that taskq_wait_id() blocks until the passed task id and all
|
||||
* lower task ids complete. We do this by dispatching the following
|
||||
* specific sequence of tasks each of which block for N time units.
|
||||
* We then use taskq_wait_id() to unblock at specific task id and
|
||||
* verify the only the expected task ids have completed and in the
|
||||
* correct order. The two cases of interest are:
|
||||
*
|
||||
* 1) Task ids larger than the waited for task id can run and
|
||||
* complete as long as there is an available worker thread.
|
||||
* 2) All task ids lower than the waited one must complete before
|
||||
* unblocking even if the waited task id itself has completed.
|
||||
*
|
||||
* The following table shows each task id and how they will be
|
||||
* scheduled. Each rows represent one time unit and each column
|
||||
* one of the three worker threads. The places taskq_wait_id()
|
||||
* must unblock for a specific id are identified as well as the
|
||||
* task ids which must have completed and their order.
|
||||
*
|
||||
* +-----+ <--- taskq_wait_id(tq, 8) unblocks
|
||||
* | | Required Completion Order: 1,2,4,5,3
|
||||
* +-----+ |
|
||||
* | | |
|
||||
* | | +-----+
|
||||
* | | | 8 |
|
||||
* | | +-----+ <--- taskq_wait_id(tq, 3) unblocks
|
||||
* | | 7 | | Required Completion Order: 1,2,4,5,3,8,6,7
|
||||
* | +-----+ |
|
||||
* | 6 | | |
|
||||
* +-----+ | |
|
||||
* | | 5 | |
|
||||
* | +-----+ |
|
||||
* | 4 | | |
|
||||
* +-----+ | |
|
||||
* | 1 | 2 | 3 |
|
||||
* +-----+-----+-----+
|
||||
*
|
||||
*/
|
||||
static void
|
||||
splat_taskq_test5_func(void *arg)
|
||||
{
|
||||
splat_taskq_id_t *tq_id = (splat_taskq_id_t *)arg;
|
||||
splat_taskq_arg_t *tq_arg = tq_id->arg;
|
||||
int factor;
|
||||
|
||||
/* Delays determined by above table */
|
||||
switch (tq_id->id) {
|
||||
default: factor = 0; break;
|
||||
case 1: case 8: factor = 1; break;
|
||||
case 2: case 4: case 5: factor = 2; break;
|
||||
case 6: case 7: factor = 4; break;
|
||||
case 3: factor = 5; break;
|
||||
}
|
||||
|
||||
msleep(factor * 100);
|
||||
splat_vprint(tq_arg->file, tq_arg->name,
|
||||
"Taskqid %d complete for taskq '%s'\n",
|
||||
tq_id->id, tq_arg->name);
|
||||
|
||||
spin_lock(&tq_arg->lock);
|
||||
tq_arg->order[tq_arg->flag] = tq_id->id;
|
||||
tq_arg->flag++;
|
||||
spin_unlock(&tq_arg->lock);
|
||||
}
|
||||
|
||||
static int
|
||||
splat_taskq_test5_order(splat_taskq_arg_t *tq_arg, int *order)
|
||||
{
|
||||
int i, j;
|
||||
|
||||
for (i = 0; i < SPLAT_TASKQ_ORDER_MAX; i++) {
|
||||
if (tq_arg->order[i] != order[i]) {
|
||||
splat_vprint(tq_arg->file, SPLAT_TASKQ_TEST5_NAME,
|
||||
"Taskq '%s' incorrect completion "
|
||||
"order\n", tq_arg->name);
|
||||
splat_vprint(tq_arg->file, SPLAT_TASKQ_TEST5_NAME,
|
||||
"%s", "Expected { ");
|
||||
|
||||
for (j = 0; j < SPLAT_TASKQ_ORDER_MAX; j++)
|
||||
splat_print(tq_arg->file, "%d ", order[j]);
|
||||
|
||||
splat_print(tq_arg->file, "%s", "}\n");
|
||||
splat_vprint(tq_arg->file, SPLAT_TASKQ_TEST5_NAME,
|
||||
"%s", "Got { ");
|
||||
|
||||
for (j = 0; j < SPLAT_TASKQ_ORDER_MAX; j++)
|
||||
splat_print(tq_arg->file, "%d ",
|
||||
tq_arg->order[j]);
|
||||
|
||||
splat_print(tq_arg->file, "%s", "}\n");
|
||||
return -EILSEQ;
|
||||
}
|
||||
}
|
||||
|
||||
splat_vprint(tq_arg->file, SPLAT_TASKQ_TEST5_NAME,
|
||||
"Taskq '%s' validated correct completion order\n",
|
||||
tq_arg->name);
|
||||
|
||||
return 0;
|
||||
}
|
||||
|
||||
static int
|
||||
splat_taskq_test5(struct file *file, void *arg)
|
||||
{
|
||||
taskq_t *tq;
|
||||
taskqid_t id;
|
||||
splat_taskq_id_t tq_id[SPLAT_TASKQ_ORDER_MAX];
|
||||
splat_taskq_arg_t tq_arg;
|
||||
int order1[SPLAT_TASKQ_ORDER_MAX] = { 1,2,4,5,3,0,0,0 };
|
||||
int order2[SPLAT_TASKQ_ORDER_MAX] = { 1,2,4,5,3,8,6,7 };
|
||||
int i, rc = 0;
|
||||
|
||||
splat_vprint(file, SPLAT_TASKQ_TEST5_NAME, "Taskq '%s' creating\n",
|
||||
SPLAT_TASKQ_TEST5_NAME);
|
||||
if ((tq = taskq_create(SPLAT_TASKQ_TEST5_NAME, 3, maxclsyspri,
|
||||
50, INT_MAX, TASKQ_PREPOPULATE)) == NULL) {
|
||||
splat_vprint(file, SPLAT_TASKQ_TEST5_NAME,
|
||||
"Taskq '%s' create failed\n",
|
||||
SPLAT_TASKQ_TEST5_NAME);
|
||||
return -EINVAL;
|
||||
}
|
||||
|
||||
tq_arg.flag = 0;
|
||||
memset(&tq_arg.order, 0, sizeof(int) * SPLAT_TASKQ_ORDER_MAX);
|
||||
spin_lock_init(&tq_arg.lock);
|
||||
tq_arg.file = file;
|
||||
tq_arg.name = SPLAT_TASKQ_TEST5_NAME;
|
||||
|
||||
for (i = 0; i < SPLAT_TASKQ_ORDER_MAX; i++) {
|
||||
tq_id[i].id = i + 1;
|
||||
tq_id[i].arg = &tq_arg;
|
||||
|
||||
if ((id = taskq_dispatch(tq, splat_taskq_test5_func,
|
||||
&tq_id[i], TQ_SLEEP)) == 0) {
|
||||
splat_vprint(file, SPLAT_TASKQ_TEST5_NAME,
|
||||
"Taskq '%s' function '%s' dispatch failed\n",
|
||||
tq_arg.name, sym2str(splat_taskq_test5_func));
|
||||
rc = -EINVAL;
|
||||
goto out;
|
||||
}
|
||||
|
||||
if (tq_id[i].id != id) {
|
||||
splat_vprint(file, SPLAT_TASKQ_TEST5_NAME,
|
||||
"Taskq '%s' expected taskqid %d got %d\n",
|
||||
tq_arg.name, (int)tq_id[i].id, (int)id);
|
||||
rc = -EINVAL;
|
||||
goto out;
|
||||
}
|
||||
}
|
||||
|
||||
splat_vprint(file, SPLAT_TASKQ_TEST5_NAME, "Taskq '%s' "
|
||||
"waiting for taskqid %d completion\n", tq_arg.name, 3);
|
||||
taskq_wait_id(tq, 3);
|
||||
if ((rc = splat_taskq_test5_order(&tq_arg, order1)))
|
||||
goto out;
|
||||
|
||||
splat_vprint(file, SPLAT_TASKQ_TEST5_NAME, "Taskq '%s' "
|
||||
"waiting for taskqid %d completion\n", tq_arg.name, 8);
|
||||
taskq_wait_id(tq, 8);
|
||||
rc = splat_taskq_test5_order(&tq_arg, order2);
|
||||
|
||||
out:
|
||||
splat_vprint(file, SPLAT_TASKQ_TEST5_NAME,
|
||||
"Taskq '%s' destroying\n", tq_arg.name);
|
||||
taskq_destroy(tq);
|
||||
|
||||
return rc;
|
||||
}
|
||||
|
||||
splat_subsystem_t *
|
||||
splat_taskq_init(void)
|
||||
{
|
||||
|
|
@ -362,6 +560,8 @@ splat_taskq_init(void)
|
|||
SPLAT_TASKQ_TEST3_ID, splat_taskq_test3);
|
||||
SPLAT_TEST_INIT(sub, SPLAT_TASKQ_TEST4_NAME, SPLAT_TASKQ_TEST4_DESC,
|
||||
SPLAT_TASKQ_TEST4_ID, splat_taskq_test4);
|
||||
SPLAT_TEST_INIT(sub, SPLAT_TASKQ_TEST5_NAME, SPLAT_TASKQ_TEST5_DESC,
|
||||
SPLAT_TASKQ_TEST5_ID, splat_taskq_test5);
|
||||
|
||||
return sub;
|
||||
}
|
||||
|
|
@ -370,6 +570,7 @@ void
|
|||
splat_taskq_fini(splat_subsystem_t *sub)
|
||||
{
|
||||
ASSERT(sub);
|
||||
SPLAT_TEST_FINI(sub, SPLAT_TASKQ_TEST5_ID);
|
||||
SPLAT_TEST_FINI(sub, SPLAT_TASKQ_TEST4_ID);
|
||||
SPLAT_TEST_FINI(sub, SPLAT_TASKQ_TEST3_ID);
|
||||
SPLAT_TEST_FINI(sub, SPLAT_TASKQ_TEST2_ID);
|
||||
|
|
|
|||
Loading…
Reference in New Issue