lkml.org 
[lkml]   [2010]   [Mar]   [27]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    Date
    From
    Subject[RFC PATCH] create generic alignment api (v2)
    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:

    - offset_align
    - offset_align_floor
    - object_align
    - object_align_floor

    Changelog since v1:

    - Align on the object natural alignment
    (rather than min(arch word alignment, natural alignment))

    The advantage of separating the API in "object alignment" and "offset alignment"
    is that it gives more freedom to play with offset alignment. Very useful to
    implement a tracer ring-buffer alignment. (hint hint)

    Typical users will use "object alignment", but infrastructures like tracers
    which need to perform dynamic alignment will typically use "offset alignment",
    because it allows to align with respect to a base rather than to pass an
    absolute address.

    We use "sizeof(object)" rather than "__alignof__()" object because alignof
    returns "recommended" object alignment for the architecture, which can be
    sub-optimal on some architectures. By ensuring alignment on the object size, we
    are sure to make the right choice.

    Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
    CC: Russell King - ARM Linux <linux@arm.linux.org.uk>
    Cc: Alexander Shishkin <virtuoso@slind.org>,
    CC: linux-arm-kernel@lists.infradead.org
    CC: Imre Deak <imre.deak@nokia.com>
    CC: Jamie Lokier <jamie@shareable.org>
    CC: rostedt@goodmis.org
    CC: mingo@elte.hu
    ---
    include/linux/kernel.h | 35 +++++++++++++++++++++++++++++++++++
    1 file changed, 35 insertions(+)

    Index: linux-2.6-lttng/include/linux/kernel.h
    ===================================================================
    --- linux-2.6-lttng.orig/include/linux/kernel.h 2010-03-27 20:46:07.000000000 -0400
    +++ linux-2.6-lttng/include/linux/kernel.h 2010-03-27 21:17:52.000000000 -0400
    @@ -42,6 +42,41 @@ extern const char linux_proc_banner[];
    #define PTR_ALIGN(p, a) ((typeof(p))ALIGN((unsigned long)(p), (a)))
    #define IS_ALIGNED(x, a) (((x) & ((typeof(x))(a) - 1)) == 0)

    +/**
    + * offset_align - Calculate the offset needed to align an object on its natural
    + * alignment towards higher addresses.
    + * @align_drift: object offset from an "alignment"-aligned address.
    + * @alignment: natural object alignment. Must be non-zero, power of 2.
    + *
    + * Returns the offset that must be added to align towards higher
    + * addresses.
    + */
    +static inline size_t offset_align(size_t align_drift, size_t alignment)
    +{
    + return (alignment - align_drift) & (alignment - 1);
    +}
    +
    +/**
    + * offset_align_floor - Calculate the offset needed to align an object
    + * on its natural alignment towards lower addresses.
    + * @align_drift: object offset from an "alignment"-aligned address.
    + * @alignment: natural object alignment. Must be non-zero, power of 2.
    + *
    + * Returns the offset that must be substracted to align towards lower addresses.
    + */
    +static inline size_t offset_align_floor(size_t align_drift, size_t alignment)
    +{
    + return (align_drift - alignment) & (alignment - 1);
    +}
    +
    +#define object_align(object) \
    + ((typeof(object))((size_t) object + offset_align((size_t) object, \
    + sizeof(object))))
    +
    +#define object_align_floor(object) \
    + ((typeof(object))((size_t) object - offset_align_floor((size_t) object,\
    + sizeof(object))))
    +
    #define ARRAY_SIZE(arr) (sizeof(arr) / sizeof((arr)[0]) + __must_be_array(arr))

    #define FIELD_SIZEOF(t, f) (sizeof(((t*)0)->f))
    --
    Mathieu Desnoyers
    Operating System Efficiency R&D Consultant
    EfficiOS Inc.
    http://www.efficios.com

    \
     
     \ /
      Last update: 2010-03-28 03:31    [W:0.035 / U:29.704 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site