Re: [PATCH 3/3] Replace hash-of-lists (cmd_hash_list) with a list (cmd_list)

[FUJITA Tomonori <fujita.tomonori@xxxxxxxxxxxxx>]

On Tue, 27 Sep 2011 12:21:32 -0700
Andy Grover <agrover@xxxxxxxxxx> wrote:

> It doesn't appear hashing the commands on an it_nexus is a benefit.
> Both it_nexus_destroy and abort_task_set iterate across all lists
> in the hash, which is equivalent to iterating through the list. Insertion
> and removal from the list remains O(1).
> 
> Signed-off-by: Andy Grover <agrover@xxxxxxxxxx>
> ---
>  usr/hash.h   |   58 ----------------------------------------------------------
>  usr/target.c |   35 ++++++++++++++---------------------
>  usr/target.h |    7 +------
>  3 files changed, 15 insertions(+), 85 deletions(-)
>  delete mode 100644 usr/hash.h

Applied, thanks. I even want to kill cmd_list (not sure if we can).

--
To unsubscribe from this list: send the line "unsubscribe stgt" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html