On Sun, 23 Nov 2008, Tejun Heo wrote: > +static int pollwake(wait_queue_t *wait, unsigned mode, int sync, void *key) > +{ > + struct poll_wqueues *pwq = wait->private; > + DECLARE_WAITQUEUE(dummy_wait, pwq->polling_task); > + > + set_mb(pwq->triggered, 1); > + > + /* perform the default wake up operation */ > + return default_wake_function(&dummy_wait, mode, sync, key); > +} Wouldn't it be nicer to un-static try_to_wake_up() (or a wrapper) instead of setting up a fake wait queue just to use default_wake_function(), just to wake up a task? - Davide -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html