Re: [PATCH 4/4] net,rcu: don't assume the size of struct rcu_head

From: Eric Dumazet
Date: Tue Mar 01 2011 - 03:21:05 EST


Le mardi 01 mars 2011 Ã 00:16 -0800, David Miller a Ãcrit :
> From: Lai Jiangshan <laijs@xxxxxxxxxxxxxx>
> Date: Tue, 01 Mar 2011 16:03:44 +0800
>
> >
> > struct dst_entry assumes the size of struct rcu_head as 2 * sizeof(long)
> > and manually adds pads for aligning for "__refcnt".
> >
> > When the size of struct rcu_head is changed, these manual padding
> > is wrong. Use __attribute__((aligned (64))) instead.
> >
> > Signed-off-by: Lai Jiangshan <laijs@xxxxxxxxxxxxxx>
>
> We don't want to use the align if it's going to waste lots of space.
>
> Instead we want to rearrange the structure so that the alignment comes
> more cheaply.

Oh well, I should have read your answer before sending mine :)



--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/