From: mathieu.desnoyers@efficios.com (Mathieu Desnoyers)
To: linux-arm-kernel@lists.infradead.org
Subject: [RFC PATCH] create generic alignment api (v5)
Date: Sat, 1 May 2010 15:45:45 -0400 [thread overview]
Message-ID: <20100501194545.GA17271@Krystal> (raw)
In-Reply-To: <20100501192915.GD27062@shisha.kicks-ass.net>
* Alexander Shishkin (virtuoso at slind.org) wrote:
> On Sat, May 01, 2010 at 03:10:45 -0400, Mathieu Desnoyers wrote:
> > * Alexander Shishkin (virtuoso at 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.
Yep, I'm ok with this change.
Thanks,
Mathieu
>
> Regards,
> --
> Alex
--
Mathieu Desnoyers
Operating System Efficiency R&D Consultant
EfficiOS Inc.
http://www.efficios.com
WARNING: multiple messages have this Message-ID (diff)
From: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
To: Imre Deak <imre.deak@nokia.com>,
Russell King - ARM Linux <linux@arm.linux.org.uk>,
"linux-arm-kernel@lists.infradead.org"
<linux-arm-kernel@lists.infradead.org>,
Jamie Lokier <jamie@shareable.org>,
"rostedt@goodmis.org" <rostedt@goodmis.org>,
"mingo@elte.hu" <mingo@elte.hu>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: [RFC PATCH] create generic alignment api (v5)
Date: Sat, 1 May 2010 15:45:45 -0400 [thread overview]
Message-ID: <20100501194545.GA17271@Krystal> (raw)
In-Reply-To: <20100501192915.GD27062@shisha.kicks-ass.net>
* Alexander Shishkin (virtuoso@slind.org) wrote:
> 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.
Yep, I'm ok with this change.
Thanks,
Mathieu
>
> Regards,
> --
> Alex
--
Mathieu Desnoyers
Operating System Efficiency R&D Consultant
EfficiOS Inc.
http://www.efficios.com
next prev parent reply other threads:[~2010-05-01 19:45 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-03-28 1:22 [RFC PATCH] create generic alignment api (v2) Mathieu Desnoyers
2010-03-28 1:22 ` Mathieu Desnoyers
2010-03-28 15:40 ` Imre Deak
2010-03-28 15:40 ` Imre Deak
2010-03-28 18:29 ` Mathieu Desnoyers
2010-03-28 18:29 ` Mathieu Desnoyers
2010-03-29 0:09 ` [RFC PATCH] create generic alignment api (v5) Mathieu Desnoyers
2010-03-29 0:09 ` Mathieu Desnoyers
2010-05-01 18:35 ` Alexander Shishkin
2010-05-01 19:10 ` Mathieu Desnoyers
2010-05-01 19:10 ` Mathieu Desnoyers
2010-05-01 19:29 ` Alexander Shishkin
2010-05-01 19:45 ` Mathieu Desnoyers [this message]
2010-05-01 19:45 ` Mathieu Desnoyers
2010-05-01 19:23 ` [PATCH 1/2] create generic alignment api (v6) Alexander Shishkin
2010-05-01 19:24 ` [PATCH 2/2] [RFCv3] arm: add half-word __xchg Alexander Shishkin
2010-05-03 17:27 ` Mathieu Desnoyers
[not found] ` <20100505080155.GF27062@shisha.kicks-ass.net>
2010-05-08 23:30 ` Mathieu Desnoyers
2010-05-06 18:12 ` [PATCH 1/2] create generic alignment api (v6) Mathieu Desnoyers
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=20100501194545.GA17271@Krystal \
--to=mathieu.desnoyers@efficios.com \
--cc=linux-arm-kernel@lists.infradead.org \
/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.