[Cluster-devel] Re: [PATCH] DLM: Fix static buffer alignment

2008-01-15 Thread Steven Whitehouse
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 [EMAIL PROTECTED]
 
 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.



Re: [Cluster-devel] Re: [PATCH] DLM: Fix static buffer alignment

2008-01-15 Thread David Teigland
 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...

I'm collecting all dlm patches for 2.6.25 here:

  http://people.redhat.com/teigland/dlm-patches-testing/

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.