Re: [patch 05/14] syslets: core code

From: Davide Libenzi
Date: Fri Feb 16 2007 - 19:52:26 EST


On Thu, 15 Feb 2007, Ingo Molnar wrote:

> + spin_lock(&ah->lock);
> +
> + new_async_thread = pick_ready_cachemiss_thread(ah);
> + if (!new_async_thread)
> + goto out_unlock;
> +
> + async_ready = t->async_ready;
> + WARN_ON(!async_ready);
> + t->async_ready = NULL;
> +
> + new_task = new_async_thread->task;
> +
> + move_user_context(new_task, t);

Since I lost the email where you were describing the todo list, I'll hook
into this one.
I haven't seen you mentioning it, but at least TID and signal handling
need to be swapped to. So it'd be better to have a generic
move_user_context() that does system independent work, and an
arch_move_user_context() that takes care of system dependent stuff (that
is called by move_user_context()).



- Davide


-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/