On Mon, Apr 06, 2020 at 10:09:55PM +0800, Yafang Shao wrote: > What about ida_alloc_range() ? Should we correct it as well ? > > > * Return: The allocated ID, or %-ENOMEM if memory could not be allocated, > * or %-ENOSPC if there are no free IDs. > */ > int ida_alloc_range(struct ida *ida, unsigned int min, unsigned int max, > gfp_t gfp) > > If there're no free IDs, ida_alloc_range() also returns ENOSPC. Do you want to check all the callers of ida_.*alloc()?