Re: [PATCH] prio_tree iterator cleanup.

From: Andrew Morton
Date: Mon Jul 26 2004 - 18:38:37 EST


Rajesh Venkatasubramanian <vrajesh@xxxxxxxxx> wrote:
>
> I have gone through all the 3 prio_tree patches you have sent
> over the last few weeks. Your 2nd (kill vma_prio_tree_init) and
> 3rd (iterator cleanup) patches look straight-forward.
>
> I agree that vm_set.head linking is ugly. Your 1st patch
> attempts to simplify it and removes few lines of code. However,
> your 1st patch is intrusive and I am not convinced that it
> improves the code very much.
>
> If you can post your 2nd and 3rd patches independent of your
> first patch, that will be great. Please covert vma_prio_tree_next
> in arch/* directories if you plan to post a new version of your
> 3rd patch.
>
> Overall I am inclined to leave the prio_tree code as it is.
> However, your 2nd and 3rd patches are quiet straight-forward
> and I am okay with them.

Thanks, Rajesh.

Oleg, I didn't apply any patches in this area. Please reissue any
prio-tree patches from scratch. Thanks.

-
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/