From mboxrd@z Thu Jan 1 00:00:00 1970 From: Steven Whitehouse Date: Mon, 08 Nov 2010 10:26:16 +0000 Subject: [Cluster-devel] [PATCH 3/7] fs/gfs2/glock.c: Convert sprintf_symbol to %pS In-Reply-To: <1288998760-11775-4-git-send-email-joe@perches.com> References: <1288998760-11775-1-git-send-email-joe@perches.com> <1288998760-11775-4-git-send-email-joe@perches.com> Message-ID: <1289211976.2408.1.camel@dolmen> List-Id: To: cluster-devel.redhat.com MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Hi, On Fri, 2010-11-05 at 16:12 -0700, Joe Perches wrote: > Signed-off-by: Joe Perches > --- Seems like a good idea: Acked-by: Steven Whitehouse Steve. > fs/gfs2/glock.c | 15 +++++++-------- > 1 files changed, 7 insertions(+), 8 deletions(-) > > diff --git a/fs/gfs2/glock.c b/fs/gfs2/glock.c > index 8777885..08dd4a2 100644 > --- a/fs/gfs2/glock.c > +++ b/fs/gfs2/glock.c > @@ -1627,18 +1627,17 @@ static const char *hflags2str(char *buf, unsigned flags, unsigned long iflags) > static int dump_holder(struct seq_file *seq, const struct gfs2_holder *gh) > { > struct task_struct *gh_owner = NULL; > - char buffer[KSYM_SYMBOL_LEN]; > char flags_buf[32]; > > - sprint_symbol(buffer, gh->gh_ip); > if (gh->gh_owner_pid) > gh_owner = pid_task(gh->gh_owner_pid, PIDTYPE_PID); > - gfs2_print_dbg(seq, " H: s:%s f:%s e:%d p:%ld [%s] %s\n", > - state2str(gh->gh_state), > - hflags2str(flags_buf, gh->gh_flags, gh->gh_iflags), > - gh->gh_error, > - gh->gh_owner_pid ? (long)pid_nr(gh->gh_owner_pid) : -1, > - gh_owner ? gh_owner->comm : "(ended)", buffer); > + gfs2_print_dbg(seq, " H: s:%s f:%s e:%d p:%ld [%s] %pS\n", > + state2str(gh->gh_state), > + hflags2str(flags_buf, gh->gh_flags, gh->gh_iflags), > + gh->gh_error, > + gh->gh_owner_pid ? (long)pid_nr(gh->gh_owner_pid) : -1, > + gh_owner ? gh_owner->comm : "(ended)", > + (void *)gh->gh_ip); > return 0; > } >