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

From: J. Bruce Fields
Date: Mon Feb 04 2013 - 13:15:46 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.

Ugh. OK, well it'll take some work to wrap that around, so it's
probably not urgent for nfsd.

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