All of lore.kernel.org
 help / color / mirror / Atom feed
From: Joel Becker <Joel.Becker@oracle.com>
To: ocfs2-devel@oss.oracle.com
Subject: [Ocfs2-devel] ocfs2: initialize stack_user lvbptr
Date: Mon, 10 Nov 2008 14:55:07 -0800	[thread overview]
Message-ID: <20081110225507.GC12445@mail.oracle.com> (raw)
In-Reply-To: <20081110222457.GE17894@redhat.com>

On Mon, Nov 10, 2008 at 04:24:57PM -0600, David Teigland wrote:
> commit 3d48397962cdad2aa32df75cfdae980af9bce8cc
> Author: David Teigland <teigland@redhat.com>
> Date:   Mon Nov 10 16:16:07 2008 -0600
> 
>     ocfs2: initialize stack_user lvbptr
>     
>     The locking_state dump, ocfs2_dlm_seq_show, reads the lvb on locks
>     where it has not yet been initialized by a lock call.
>     
>     Signed-off-by: David Teigland <teigland@redhat.com>

Acked-by: Joel Becker <joel.becker@oracle.com>

> 
> diff --git a/fs/ocfs2/stack_user.c b/fs/ocfs2/stack_user.c
> index faec2d8..9b76d41 100644
> --- a/fs/ocfs2/stack_user.c
> +++ b/fs/ocfs2/stack_user.c
> @@ -740,6 +740,9 @@ static int user_dlm_lock_status(union ocfs2_dlm_lksb *lksb)
>  
>  static void *user_dlm_lvb(union ocfs2_dlm_lksb *lksb)
>  {
> +	if (!lksb->lksb_fsdlm.sb_lvbptr)
> +		lksb->lksb_fsdlm.sb_lvbptr = (char *)lksb +
> +					     sizeof(struct dlm_lksb);
>  	return (void *)(lksb->lksb_fsdlm.sb_lvbptr);
>  }
>  

-- 

"What does it say about a society's priorities when the time you
 spend in meetings on Monday is greater than the total number of
 hours you spent sleeping over the weekend?"
	- Nat Friedman

Joel Becker
Principal Software Developer
Oracle
E-mail: joel.becker at oracle.com
Phone: (650) 506-8127

       reply	other threads:[~2008-11-10 22:55 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20081110222457.GE17894@redhat.com>
2008-11-10 22:55 ` Joel Becker [this message]
2008-11-10 23:44   ` [Ocfs2-devel] ocfs2: initialize stack_user lvbptr Mark Fasheh

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20081110225507.GC12445@mail.oracle.com \
    --to=joel.becker@oracle.com \
    --cc=ocfs2-devel@oss.oracle.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.