From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jiri Pirko Subject: Re: [patch net-next 1/3] idr: Add new APIs to support unsigned long Date: Tue, 29 Aug 2017 09:57:11 +0200 Message-ID: <20170829075711.GE1977@nanopsycho.orion> References: <1503902477-39829-1-git-send-email-chrism@mellanox.com> <1503902477-39829-2-git-send-email-chrism@mellanox.com> <87y3q27sn7.fsf@stressinduktion.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Hannes Frederic Sowa , "netdev@vger.kernel.org" , "jhs@mojatatu.com" , "xiyou.wangcong@gmail.com" , "davem@davemloft.net" , "mawilcox@microsoft.com" To: Chris Mi Return-path: Received: from mail-wm0-f67.google.com ([74.125.82.67]:36028 "EHLO mail-wm0-f67.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750779AbdH2H5O (ORCPT ); Tue, 29 Aug 2017 03:57:14 -0400 Received: by mail-wm0-f67.google.com with SMTP id j72so2840168wmi.3 for ; Tue, 29 Aug 2017 00:57:14 -0700 (PDT) Content-Disposition: inline In-Reply-To: Sender: netdev-owner@vger.kernel.org List-ID: Tue, Aug 29, 2017 at 09:34:47AM CEST, chrism@mellanox.com wrote: >Hi, > >> -----Original Message----- >> From: Hannes Frederic Sowa [mailto:hannes@stressinduktion.org] >> Sent: Tuesday, August 29, 2017 3:14 PM >> To: Chris Mi >> Cc: netdev@vger.kernel.org; jhs@mojatatu.com; >> xiyou.wangcong@gmail.com; jiri@resnulli.us; davem@davemloft.net; >> mawilcox@microsoft.com >> Subject: Re: [patch net-next 1/3] idr: Add new APIs to support unsigned long >> >> Hello, >> >> Chris Mi writes: >> >> > The following new APIs are added: >> > >> > int idr_alloc_ext(struct idr *idr, void *ptr, unsigned long *index, >> > unsigned long start, unsigned long end, gfp_t gfp); >> > static inline void *idr_remove_ext(struct idr *idr, unsigned long id); >> > static inline void *idr_find_ext(const struct idr *idr, unsigned long >> > id); void *idr_replace_ext(struct idr *idr, void *ptr, unsigned long >> > id); void *idr_get_next_ext(struct idr *idr, unsigned long *nextid); >> > >> > Signed-off-by: Chris Mi >> > Signed-off-by: Jiri Pirko >> > --- >> > include/linux/idr.h | 16 ++++++++++ >> > include/linux/radix-tree.h | 3 ++ >> > lib/idr.c | 56 +++++++++++++++++++++++++++++++++++ >> > lib/radix-tree.c | 73 >> ++++++++++++++++++++++++++++++++++++++++++++++ >> > 4 files changed, 148 insertions(+) >> > >> >> [...] >> >> > +int idr_alloc_ext(struct idr *idr, void *ptr, unsigned long *index, >> > + unsigned long start, unsigned long end, gfp_t gfp) { >> > + void __rcu **slot; >> > + struct radix_tree_iter iter; >> > + >> > + if (WARN_ON_ONCE(radix_tree_is_internal_node(ptr))) >> > + return -EINVAL; >> > + >> > + radix_tree_iter_init(&iter, start); >> > + slot = idr_get_free_ext(&idr->idr_rt, &iter, gfp, end); >> > + if (IS_ERR(slot)) >> > + return PTR_ERR(slot); >> > + >> > + radix_tree_iter_replace(&idr->idr_rt, &iter, slot, ptr); >> > + radix_tree_iter_tag_clear(&idr->idr_rt, &iter, IDR_FREE); >> > + >> > + if (index) >> > + *index = iter.index; >> > + return 0; >> > +} >> > +EXPORT_SYMBOL_GPL(idr_alloc_ext); >> >> Can you express idr_alloc in terms of idr_alloc_ext? Same for most of the >> other functions (it seems that signed int was used as return value to indicate >> error cases, thus it should be easy to map those). >In idr_alloc(), we have the following check: > > if (WARN_ON_ONCE(start < 0)) > return -EINVAL; > >But in idr_alloc_ext(), since we are using unsigned long, we needn't such check. You can just check and call idr_alloc_ext then to do the actual work.