On Tue, 4 Nov 2008, Jiri Slaby wrote:

quoted

in this code:

		mutex_lock(&list->read_mutex);

		if (list->head == list->tail) {
			add_wait_queue(&list->hidraw->wait, &wait);
			set_current_state(TASK_INTERRUPTIBLE);

it would make more sense to lock the mutex interruptable so all
tasks sleep the same way. Is this intentional?

Well, we have the test for pending signals just below that code, right?

-- 
Jiri Kosina
SUSE Labs