From mboxrd@z Thu Jan 1 00:00:00 1970 From: Konrad Rzeszutek Wilk Subject: Re: [PATCH] switch rangeset's lock to rwlock Date: Fri, 19 Sep 2014 12:33:07 -0400 Message-ID: <20140919163307.GB9006@laptop.dumpdata.com> References: <5413094B020000780003480E@mail.emea.novell.com> <20140918104316.GD65000@deinos.phlegethon.org> <541AE8FC02000078000362D1@mail.emea.novell.com> <20140918130258.GB81607@deinos.phlegethon.org> <541AFB130200007800036360@mail.emea.novell.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mail6.bemta4.messagelabs.com ([85.158.143.247]) by lists.xen.org with esmtp (Exim 4.72) (envelope-from ) id 1XV17T-0000Dj-Lv for xen-devel@lists.xenproject.org; Fri, 19 Sep 2014 16:33:15 +0000 Content-Disposition: inline In-Reply-To: <541AFB130200007800036360@mail.emea.novell.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Jan Beulich Cc: Ian Campbell , Ian Jackson , Keir Fraser , Tim Deegan , xen-devel List-Id: xen-devel@lists.xenproject.org On Thu, Sep 18, 2014 at 02:32:35PM +0100, Jan Beulich wrote: > >>> On 18.09.14 at 15:02, wrote: > > At 13:15 +0100 on 18 Sep (1411042524), Jan Beulich wrote: > >> >>> On 18.09.14 at 12:43, wrote: > >> > At 13:55 +0100 on 12 Sep (1410526507), Jan Beulich wrote: > >> >> As a general library routine, it should behave as efficiently as > >> >> possible, even if at present no significant contention is known here. > >> >> > >> >> Signed-off-by: Jan Beulich > >> >> --- > >> >> With the widened use of rangesets I'd like to re-suggest this change > >> >> which I had posted already a couple of years back. > >> > > >> > Is this addressing an actual (measured) problem or just seems like a > >> > good idea? If the latter maybe keep it until after 4.5? > >> > >> The latter. And yes, I have no problem keeping it until after 4.5, > >> it's just that the multi-ioreq-server's extended use of rangesets > >> (as said elsewhere) would seem to make this a reasonable fit for > >> 4.5. > > > > Well, I think it's a question for the release manager, then. :) > > Konrad? I am comfortable with it. Responded with an Reviewed-by on the patch itself and Ack for Xen 4.5.