--
I'm going to make him an offer he can't refuse.
01-15-2008, 12:33 PM
Steven Whitehouse
DLM: Fix static buffer alignment
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.
01-15-2008, 05:50 PM
"Fabio M. Di Nitto"
DLM: Fix static buffer alignment
Ok this is weird.. not the first time that this is happening between us..
tho i did change MUA that has a slightly more sane way of adding stuff...
anyway I will prepare a git tree you can pull from..
at this point it will just save us pain for the future.
Thanks
Fabio
On Tue, 15 Jan 2008, Steven Whitehouse wrote:
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>
Ok this is weird.. not the first time that this is happening between us.. tho
i did change MUA that has a slightly more sane way of adding stuff... anyway
I will prepare a git tree you can pull from..
at this point it will just save us pain for the future.
Thanks
Fabio
On Tue, 15 Jan 2008, Steven Whitehouse wrote:
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>
{
- 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.
--
I'm going to make him an offer he can't refuse.
--
I'm going to make him an offer he can't refuse.
01-15-2008, 09:45 PM
David Teigland
DLM: Fix static buffer alignment
> Hi Steven,
>
> you can try to pull from here:
>
> git pull git://git.fugedabout.it/people/fabbione/gfs2-2.6-nmw.git
>
> [DLM] Fix endian issue when transmitting or receiving LOCK_REPLY
> [DLM] align static buffer
>
> gitweb:
> http://git.fugedabout.it/?p=people/fabbione/gfs2-2.6-nmw.git;a=summary
>
> Let me know if there are any problems...
It includes the dlm patches already in the gfs2nmw git tree, plus the new
patches you've sent the past couple days. I'm working on getting them
into my own public git tree. If that doesn't happen soon enough for
2.6.25 I'll send the lot to Steve.
DLM: Fix static buffer alignment
