All of lore.kernel.org
 help / color / mirror / Atom feed
From: Wolfram Sang <wsa@the-dreams.de>
To: Kent Overstreet <kmo@daterainc.com>
Cc: Tatyana Nikolova <Tatyana.E.Nikolova@intel.com>,
	Linus Walleij <linus.walleij@linaro.org>,
	Paul Mackerras <paulus@samba.org>, Christoph Hellwig <hch@lst.de>,
	Alon Levy <alevy@redhat.com>,
	Davidlohr Bueso <davidlohr.bueso@hp.com>,
	Ohad Ben-Cohen <ohad@wizery.com>,
	"John W. Linville" <linville@tuxdriver.com>,
	Inki Dae <inki.dae@samsung.com>,
	Alex Williamson <alex.williamson@redhat.com>,
	David Teigland <teigland@redhat.com>,
	Joel Becker <jlbec@evilplan.org>,
	Tim Gardner <tim.gardner@canonical.com>,
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	Seung-Woo Kim <sw0312.kim@samsung.com>,
	linux-kernel@vger.kernel.org, Li Zefan <lizefan@huawei.com>,
	Tejun Heo <tj@kernel.org>, Guo Chao <yan@linux.vnet.ibm.com>,
	akpm@linux-foundation.org, Andy Grover <agrover@redhat.com>,
	Dmitry
Subject: Re: [PATCH 07/10] idr: Rename idr_alloc() -> idr_alloc_range()
Date: Wed, 7 Aug 2013 21:04:53 +0200	[thread overview]
Message-ID: <20130807190453.GL2978@katana> (raw)
In-Reply-To: <1375897579-8202-7-git-send-email-kmo@daterainc.com>


[-- Attachment #1.1: Type: text/plain, Size: 333 bytes --]

On Wed, Aug 07, 2013 at 10:46:16AM -0700, Kent Overstreet wrote:
> Then also add an idr_alloc() wrapper, without the start/end arguments -
> for consistency with the ida interfaces.
> 
> Signed-off-by: Kent Overstreet <kmo@daterainc.com>

For my taste, the motivation of the change could be described in a bit
more detail.


[-- Attachment #1.2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

[-- Attachment #2: Type: text/plain, Size: 0 bytes --]



WARNING: multiple messages have this Message-ID (diff)
From: Wolfram Sang <wsa@the-dreams.de>
To: Kent Overstreet <kmo@daterainc.com>
Cc: "Linus Walleij" <linus.walleij@linaro.org>,
	"Paul Mackerras" <paulus@samba.org>,
	"Christoph Hellwig" <hch@lst.de>,
	"James E.J. Bottomley" <JBottomley@parallels.com>,
	"Alon Levy" <alevy@redhat.com>,
	"Davidlohr Bueso" <davidlohr.bueso@hp.com>,
	"Ohad Ben-Cohen" <ohad@wizery.com>,
	"John W. Linville" <linville@tuxdriver.com>,
	"Inki Dae" <inki.dae@samsung.com>,
	"Alex Williamson" <alex.williamson@redhat.com>,
	"David Teigland" <teigland@redhat.com>,
	"Joel Becker" <jlbec@evilplan.org>,
	"Tim Gardner" <tim.gardner@canonical.com>,
	"Greg Kroah-Hartman" <gregkh@linuxfoundation.org>,
	"Seung-Woo Kim" <sw0312.kim@samsung.com>,
	linux-kernel@vger.kernel.org, "Li Zefan" <lizefan@huawei.com>,
	"Tejun Heo" <tj@kernel.org>, "Guo Chao" <yan@linux.vnet.ibm.com>,
	akpm@linux-foundation.org, "Andy Grover" <agrover@redhat.com>,
	"Dmitry Torokhov" <dtor@vmware.com>,
	"David Airlie" <airlied@linux.ie>,
	"Anil Gurumurthy" <agurumur@brocade.com>,
	dm-devel@redhat.com, "Alasdair Kergon" <agk@redhat.com>,
	"Lars Ellenberg" <drbd-dev@lists.linbit.com>,
	"Thomas Hellstrom" <thellstrom@vmware.com>,
	"Vinod Koul" <vinod.koul@intel.com>,
	"Eduardo Valentin" <eduardo.valentin@ti.com>,
	"Dave Airlie" <airlied@redhat.com>, "Chris Ball" <cjb@laptop.org>,
	"Hal Rosenstock" <hal.rosenstock@gmail.com>,
	"Rik van Riel" <riel@redhat.com>,
	"Peter Zijlstra" <a.p.zijlstra@chello.nl>,
	"Mark Fasheh" <mfasheh@suse.com>,
	"Mike Marciniszyn" <infinipath@intel.com>,
	"Steve Wise" <swise@chelsio.com>,
	"Maarten Lankhorst" <maarten.lankhorst@canonical.com>,
	"Anton Vorontsov" <cbou@mail.ru>,
	"Kyungmin Park" <kyungmin.park@samsung.com>,
	"Dan Williams" <djbw@fb.com>,
	"James Smart" <james.smart@emulex.com>,
	"Jason Wang" <jasowang@redhat.com>,
	"Trond Myklebust" <Trond.Myklebust@netapp.com>,
	"Omar Ramirez Luna" <omar.ramirez@copitl.com>,
	"Sean Hefty" <sean.hefty@intel.com>,
	"Christine Caulfield" <ccaulfie@redhat.com>,
	"Doug Gilbert" <dgilbert@interlog.com>,
	"Grant Likely" <grant.likely@linaro.org>,
	"Sasha Levin" <sasha.levin@oracle.com>,
	"Paul E. McKenney" <paulmck@linux.vnet.ibm.com>,
	"Dan Magenheimer" <dan.magenheimer@oracle.com>,
	"Alex Dubov" <oakad@yahoo.com>,
	"Haggai Eran" <haggaie@mellanox.com>,
	"Rodolfo Giometti" <giometti@enneenne.com>,
	"Tatyana Nikolova" <Tatyana.E.Nikolova@intel.com>,
	"Eric Dumazet" <edumazet@google.com>,
	"FUJITA Tomonori" <fujita.tomonori@lab.ntt.co.jp>,
	"Stefan Richter" <stefanr@s5r6.in-berlin.de>,
	"Arnd Bergmann" <arnd@arndb.de>,
	"David S. Miller" <davem@davemloft.net>,
	"Michael S. Tsirkin" <mst@redhat.com>,
	"Daniel Vetter" <daniel.vetter@ffwll.ch>,
	"Kai Mäkisara" <Kai.Makisara@kolumbus.fi>,
	"David Howells" <dhowells@redhat.com>,
	"Hans J. Koch" <hjk@hansjkoch.de>,
	"Michel Lespinasse" <walken@google.com>,
	"Chas Williams" <chas@cmf.nrl.navy.mil>,
	"Joonyoung Shim" <jy0922.shim@samsung.com>,
	"Ingo Molnar" <mingo@redhat.com>,
	"Brian Paul" <brianp@vmware.com>,
	"Arnaldo Carvalho de Melo" <acme@ghostprotocols.net>,
	"Zhang Rui" <rui.zhang@intel.com>,
	"Roland Dreier" <roland@kernel.org>,
	"Erez Shitrit" <erezsh@mellanox.co.il>,
	"Vijaya Mohan Guvva" <vmohan@brocade.com>,
	"Christoph Raisch" <raisch@de.ibm.com>,
	"Hoang-Nam Nguyen" <hnguyen@de.ibm.com>,
	"Samuel Ortiz" <sameo@linux.intel.com>,
	"Jens Axboe" <axboe@kernel.dk>,
	"Masanari Iida" <standby24x7@gmail.com>,
	"Willem Riede" <osst@riede.org>,
	"Johannes Berg" <johannes@sipsolutions.net>,
	"David Woodhouse" <dwmw2@infradead.org>,
	"Guennadi Liakhovetski" <g.liakhovetski@gmx.de>,
	"Al Viro" <viro@zeniv.linux.org.uk>
Subject: Re: [Drbd-dev] [PATCH 07/10] idr: Rename idr_alloc() -> idr_alloc_range()
Date: Wed, 7 Aug 2013 21:04:53 +0200	[thread overview]
Message-ID: <20130807190453.GL2978@katana> (raw)
In-Reply-To: <1375897579-8202-7-git-send-email-kmo@daterainc.com>

[-- Attachment #1: Type: text/plain, Size: 333 bytes --]

On Wed, Aug 07, 2013 at 10:46:16AM -0700, Kent Overstreet wrote:
> Then also add an idr_alloc() wrapper, without the start/end arguments -
> for consistency with the ida interfaces.
> 
> Signed-off-by: Kent Overstreet <kmo@daterainc.com>

For my taste, the motivation of the change could be described in a bit
more detail.


[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

  reply	other threads:[~2013-08-07 19:04 UTC|newest]

Thread overview: 54+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-08-07 17:34 IDA/IDR rewrite, percpu ida Kent Overstreet
2013-08-07 17:34 ` [PATCH 03/10] idr: Rewrite ida Kent Overstreet
2013-08-07 20:22   ` Tejun Heo
2013-08-07 20:51     ` [PATCH] idr: Document ida tree sections Kent Overstreet
2013-08-09 14:57       ` Tejun Heo
2013-08-13 22:13         ` Kent Overstreet
2013-08-13 22:19           ` Tejun Heo
2013-08-13 22:27             ` Kent Overstreet
2013-08-13 22:44               ` Tejun Heo
2013-08-13 22:59                 ` Kent Overstreet
2013-08-13 23:22                   ` Tejun Heo
2013-08-13 23:51                     ` Kent Overstreet
2013-08-13 23:59                       ` Tejun Heo
2013-08-15  0:04                         ` Kent Overstreet
2013-08-15  0:22                           ` Tejun Heo
2013-08-13 22:33         ` Kent Overstreet
2013-08-07 17:34 ` [PATCH 04/10] idr: Percpu ida Kent Overstreet
2013-08-07 17:56   ` Christoph Lameter
2013-08-07 18:33     ` Kent Overstreet
2013-08-07 19:40       ` Christoph Lameter
2013-08-07 19:57         ` [PATCH] idr: Use this_cpu_ptr() for percpu_ida Kent Overstreet
2013-08-08 14:32           ` Christoph Lameter
2013-08-20 21:19             ` Nicholas A. Bellinger
2013-08-20 21:29               ` Andrew Morton
2013-08-21  2:01                 ` Kent Overstreet
2013-08-21  2:07                   ` Tejun Heo
2013-08-21  2:31                     ` Kent Overstreet
2013-08-21 11:59                       ` Tejun Heo
2013-08-21 21:09                         ` Kent Overstreet
2013-08-21 21:16                           ` Tejun Heo
2013-08-21 21:24                             ` Kent Overstreet
2013-08-21 21:31                               ` Tejun Heo
2013-08-21 14:32               ` Christoph Lameter
2013-08-21 17:49                 ` Nicholas A. Bellinger
2013-08-21 20:49                 ` Andrew Morton
2013-08-22 16:44                   ` Christoph Lameter
2013-08-22 16:56                     ` Jens Axboe
2013-08-07 17:46 ` [PATCH 05/10] idr: Kill old deprecated idr interfaces Kent Overstreet
2013-08-07 17:46 ` [PATCH 06/10] idr: Rename idr_get_next() -> idr_find_next() Kent Overstreet
2013-08-07 17:46 ` [PATCH 07/10] idr: Rename idr_alloc() -> idr_alloc_range() Kent Overstreet
2013-08-07 17:46   ` [Drbd-dev] " Kent Overstreet
2013-08-07 19:04   ` Wolfram Sang [this message]
2013-08-07 19:04     ` Wolfram Sang
2013-08-07 17:46 ` [PATCH 08/10] idr: Reimplement idr on top of ida/radix trees Kent Overstreet
     [not found] ` <1375896905-6074-1-git-send-email-kmo-PEzghdH756F8UrSeD/g0lQ@public.gmane.org>
2013-08-07 17:46   ` [PATCH 09/10] idr: Remove unneeded idr locking, idr_preload() usage Kent Overstreet
2013-08-07 17:46     ` Kent Overstreet
2013-08-07 17:46 ` [Cluster-devel] [PATCH 10/10] idr: Rework idr_preload() Kent Overstreet
2013-08-07 17:46   ` Kent Overstreet
  -- strict thread matches above, loose matches on Subject: below --
2013-07-06  5:08 [PATCH v3] lib/idr.c rewrite, percpu ida/tag allocator Kent Overstreet
2013-07-06  5:08 ` [PATCH 07/10] idr: Rename idr_alloc() -> idr_alloc_range() Kent Overstreet
2013-07-06  5:26   ` Dan Williams
2013-06-19  0:02 [PATCH v2] lib/idr.c rewrite, percpu ida/tag allocator Kent Overstreet
     [not found] ` <1371600150-23557-1-git-send-email-koverstreet-hpIqsD4AKlfQT0dZR+AlfA@public.gmane.org>
2013-06-19  0:02   ` [PATCH 07/10] idr: Rename idr_alloc() -> idr_alloc_range() Kent Overstreet
2013-06-19 16:44     ` Alex Williamson
2013-06-19 17:28     ` Davidlohr Bueso
2013-06-21  3:13     ` Vinod Koul

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20130807190453.GL2978@katana \
    --to=wsa@the-dreams.de \
    --cc=Tatyana.E.Nikolova@intel.com \
    --cc=agrover@redhat.com \
    --cc=akpm@linux-foundation.org \
    --cc=alevy@redhat.com \
    --cc=alex.williamson@redhat.com \
    --cc=davidlohr.bueso@hp.com \
    --cc=dm-devel@redhat.com \
    --cc=gregkh@linuxfoundation.org \
    --cc=hch@lst.de \
    --cc=inki.dae@samsung.com \
    --cc=jlbec@evilplan.org \
    --cc=kmo@daterainc.com \
    --cc=linus.walleij@linaro.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linville@tuxdriver.com \
    --cc=lizefan@huawei.com \
    --cc=ohad@wizery.com \
    --cc=paulus@samba.org \
    --cc=sw0312.kim@samsung.com \
    --cc=teigland@redhat.com \
    --cc=tim.gardner@canonical.com \
    --cc=tj@kernel.org \
    --cc=yan@linux.vnet.ibm.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.