From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757445Ab0EATaq (ORCPT ); Sat, 1 May 2010 15:30:46 -0400 Received: from filtteri6.pp.htv.fi ([213.243.153.189]:37636 "EHLO filtteri6.pp.htv.fi" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757063Ab0EATao (ORCPT ); Sat, 1 May 2010 15:30:44 -0400 Date: Sat, 1 May 2010 22:29:15 +0300 From: Alexander Shishkin To: Mathieu Desnoyers Cc: Imre Deak , Russell King - ARM Linux , "linux-arm-kernel@lists.infradead.org" , Jamie Lokier , "rostedt@goodmis.org" , "mingo@elte.hu" , "linux-kernel@vger.kernel.org" Subject: Re: [RFC PATCH] create generic alignment api (v5) Message-ID: <20100501192915.GD27062@shisha.kicks-ass.net> Mail-Followup-To: Mathieu Desnoyers , Imre Deak , Russell King - ARM Linux , "linux-arm-kernel@lists.infradead.org" , Jamie Lokier , "rostedt@goodmis.org" , "mingo@elte.hu" , "linux-kernel@vger.kernel.org" References: <20100328012247.GA17763@Krystal> <20100328154029.GA21061@localhost> <20100329000900.GB27379@Krystal> <20100501183544.GC27062@shisha.kicks-ass.net> <20100501191045.GA16710@Krystal> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20100501191045.GA16710@Krystal> User-Agent: Mutt/1.5.20 (2009-06-14) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Sat, May 01, 2010 at 03:10:45 -0400, Mathieu Desnoyers wrote: > * Alexander Shishkin (virtuoso@slind.org) wrote: > > On Sun, Mar 28, 2010 at 08:09:00 -0400, Mathieu Desnoyers wrote: > > > Rather than re-doing the "alignment on a type size" trick all over again at > > > different levels, import the "ltt_align" from LTTng into kernel.h and make this > > > available to everyone. Renaming to: > > > > > > - object_align() > > > - object_align_floor() > > > - offset_align() > > > - offset_align_floor() > > > > Do you plan to have this integrated to the mainline? I'd like to use one of > > these in my next version of the __xchg patch. So, do you need any help or > > feedback or whatnot with this one? > > Given that no one seems to oppose to this patch, I think I'll just re-submit it > as-is for real this time. Sorry, a short explanation: it has to be a separate header file in order to be of use to me, since I need those in system.h, which gets included implicitly through bitops.h earlier in kernel.h. So I took the liberty to hack up a v6 of this patch. Regards, -- Alex