Re: [PATCH 1/2] radix-tree: Implement function radix_tree_range_tag_if_tagged

Jan Kara
2010-06-21T22:44:09Z

On Mon 21-06-10 22:09:34, Nick Piggin wrote:
> On Fri, Jun 18, 2010 at 03:18:24PM -0700, Andrew Morton wrote:
> > On Wed, 16 Jun 2010 18:33:50 +0200
> > Jan Kara <jack@suse.cz> wrote:
> >
> > > Implement function for setting one tag if another tag is set
> > > for each item in given range.
> > >
> >
> > These two patches look OK to me.
> >
> > fwiw I have a userspace test harness for radix-tree.c:
> > http://userweb.kernel.org/~akpm/stuff/rtth.tar.gz. Nick used it for a
> > while and updated it somewhat, but it's probably rather bitrotted and
> > surely needs to be taught how to test the post-2006 additions.
> >
>
> Main thing I did was add RCU support (pretty dumb RCU but it found
> a couple of bugs), and add some more tests. I'll try to find it...
Please send them my way if you can find them. I'll gladly run those tests
(and extend them to check also my new function).

Honza

--
Jan Kara <jack@suse.cz>
SUSE Labs, CR