Re: [PATCHSET] idr: implement idr_alloc() and convert existing users

From: J. Bruce Fields
Date: Thu Mar 21 2013 - 10:06:26 EST


On Mon, Feb 04, 2013 at 09:11:28AM -0800, Tejun Heo wrote:
> On Mon, Feb 04, 2013 at 09:10:31AM -0800, Tejun Heo wrote:
> > Yeah, that should be easily convertable to the new interface. How
> > should we route these changes? Your patch can go through the usual
> > nfs channel and the conversion and deprecation patches can be held off
> > until after -rc1. Does that sound okay?
>
> Ooh, BTW, the cyclic allocation is broken. It's prone to -ENOSPC
> after the first wraparound. There are several cyclic users in the
> kernel and I think it probably would be best to implement cyclic
> support in idr.

Are you looking at this, by the way, or do you have any suggestions?

Wondering if it's something I should try to fix or if I should look into
converting to some other data structure here.

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