Skip to content

Commit

Permalink
Prevent reclaim in send_traverse_thread()
Browse files Browse the repository at this point in the history
As is the case with traverse_prefetch_thread(), the deep stacks caused
by traversal require disabling reclaim in the send traverse thread.

Also, do the same for receive_writer_thread() in which similar problems
have been observed.

Signed-off-by: Tim Chase <[email protected]>
Signed-off-by: Brian Behlendorf <[email protected]>
Closes #4912
Closes #4998
  • Loading branch information
dweeezil authored and behlendorf committed Aug 22, 2016
1 parent 23827a4 commit 3e635ac
Showing 1 changed file with 5 additions and 0 deletions.
5 changes: 5 additions & 0 deletions module/zfs/dmu_send.c
Original file line number Diff line number Diff line change
Expand Up @@ -573,6 +573,7 @@ send_traverse_thread(void *arg)
struct send_thread_arg *st_arg = arg;
int err;
struct send_block_record *data;
fstrans_cookie_t cookie = spl_fstrans_mark();

if (st_arg->ds != NULL) {
err = traverse_dataset_resume(st_arg->ds,
Expand All @@ -585,6 +586,7 @@ send_traverse_thread(void *arg)
data = kmem_zalloc(sizeof (*data), KM_SLEEP);
data->eos_marker = B_TRUE;
bqueue_enqueue(&st_arg->q, data, 1);
spl_fstrans_unmark(cookie);
}

/*
Expand Down Expand Up @@ -2737,6 +2739,8 @@ receive_writer_thread(void *arg)
{
struct receive_writer_arg *rwa = arg;
struct receive_record_arg *rrd;
fstrans_cookie_t cookie = spl_fstrans_mark();

for (rrd = bqueue_dequeue(&rwa->q); !rrd->eos_marker;
rrd = bqueue_dequeue(&rwa->q)) {
/*
Expand All @@ -2761,6 +2765,7 @@ receive_writer_thread(void *arg)
rwa->done = B_TRUE;
cv_signal(&rwa->cv);
mutex_exit(&rwa->mutex);
spl_fstrans_unmark(cookie);
}

static int
Expand Down

0 comments on commit 3e635ac

Please sign in to comment.