From mboxrd@z Thu Jan  1 00:00:00 1970
From: Steven Whitehouse <swhiteho@redhat.com>
Date: Tue, 15 Jan 2008 13:33:02 +0000
Subject: [Cluster-devel] Re: [PATCH] DLM: Fix static buffer alignment
In-Reply-To: <Pine.LNX.4.64.0801150544550.6826@trider-g7>
References: <Pine.LNX.4.64.0801150544550.6826@trider-g7>
Message-ID: <1200403982.22038.139.camel@quoit>
List-Id: <cluster-devel.redhat.com>
To: cluster-devel.redhat.com
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit

Hi,

There is something not quite right about this patch... it doesn't seem
to apply and I suspect whitespace, but I can't see what the problem is
at the moment,

Steve.

On Tue, 2008-01-15 at 05:51 +0100, Fabio M. Di Nitto wrote:
> Hi Steven,
> 
> gcc does not guarantee that a static buffer is 64bit aligned. This change 
> allows sparc64 to mount and use gfs2.
> 
> Signed-off-by: Fabio M. Di Nitto <fabbione@ubuntu.com>
> 
> diff --git a/fs/dlm/midcomms.c b/fs/dlm/midcomms.c
> index f8c69dd..da653b5 100644
> --- a/fs/dlm/midcomms.c
> +++ b/fs/dlm/midcomms.c
> @@ -58,7 +58,7 @@ static void copy_from_cb(void *dst, const void *base, unsigned offset,
>   int dlm_process_incoming_buffer(int nodeid, const void *base,
>   				unsigned offset, unsigned len, unsigned limit)
>   {
> -	unsigned char __tmp[DLM_INBUF_LEN];
> +	unsigned char __tmp[DLM_INBUF_LEN] __attribute__((aligned(64)));
>   	struct dlm_header *msg = (struct dlm_header *) __tmp;
>   	int ret = 0;
>   	int err = 0;
> 
> 
> --
> I'm going to make him an offer he can't refuse.