RDMA/ucma: Simplify ucma_get_event()
Use wait_event_interruptible() instead of a more complicated open-coded equivalent. Signed-off-by: Sean Hefty <sean.hefty@intel.com>
This commit is contained in:
parent
30c00986f3
commit
d92f76448c
@ -306,25 +306,17 @@ static ssize_t ucma_get_event(struct ucma_file *file, const char __user *inbuf,
|
|||||||
|
|
||||||
mutex_lock(&file->mut);
|
mutex_lock(&file->mut);
|
||||||
while (list_empty(&file->event_list)) {
|
while (list_empty(&file->event_list)) {
|
||||||
if (file->filp->f_flags & O_NONBLOCK) {
|
|
||||||
ret = -EAGAIN;
|
|
||||||
break;
|
|
||||||
}
|
|
||||||
|
|
||||||
if (signal_pending(current)) {
|
|
||||||
ret = -ERESTARTSYS;
|
|
||||||
break;
|
|
||||||
}
|
|
||||||
|
|
||||||
prepare_to_wait(&file->poll_wait, &wait, TASK_INTERRUPTIBLE);
|
|
||||||
mutex_unlock(&file->mut);
|
mutex_unlock(&file->mut);
|
||||||
schedule();
|
|
||||||
mutex_lock(&file->mut);
|
|
||||||
finish_wait(&file->poll_wait, &wait);
|
|
||||||
}
|
|
||||||
|
|
||||||
if (ret)
|
if (file->filp->f_flags & O_NONBLOCK)
|
||||||
goto done;
|
return -EAGAIN;
|
||||||
|
|
||||||
|
if (wait_event_interruptible(file->poll_wait,
|
||||||
|
!list_empty(&file->event_list)))
|
||||||
|
return -ERESTARTSYS;
|
||||||
|
|
||||||
|
mutex_lock(&file->mut);
|
||||||
|
}
|
||||||
|
|
||||||
uevent = list_entry(file->event_list.next, struct ucma_event, list);
|
uevent = list_entry(file->event_list.next, struct ucma_event, list);
|
||||||
|
|
||||||
|
Loading…
x
Reference in New Issue
Block a user