All of lore.kernel.org
 help / color / mirror / Atom feed
From: Dave Anderson <anderson@redhat.com>
To: Jay Lan <jlan@sgi.com>
Cc: Ken'ichi Ohmichi <oomichi@mxs.nes.nec.co.jp>,
	Bernhard Walle <bwalle@suse.de>,
	kexec@lists.infradead.org
Subject: Re: [PATCH] [makedumpfile] Follow debuginfo link of vmlinux file
Date: Thu, 24 May 2007 14:31:06 -0400	[thread overview]
Message-ID: <4655D9EA.8020601@redhat.com> (raw)
In-Reply-To: <4655C9B3.7060903@sgi.com>

Jay Lan wrote:
> Ken'ichi Ohmichi wrote:
> 
>>Hi Bernhard,
>>
>>2007/05/01 15:39:12 +0200, Bernhard Walle <bwalle@suse.de> wrote:
>>
>>>This patch implements support for following the debug information link in the
>>>.gnu_debuginfo section if the specified binary contains one. That makes it
>>>possible to call makedumpfile with -x /boot/vmlinux-<version> on distributions
>>>that ship extra -debuginfo packages for the kernel.
>>
>>Isn't it good to specify the debuginfo file directly instead of your patch ?
>>I confirmed that makedumpfile without your patch can run by specifying the
>>debuginfo file directly.
>>
>>makedumpfile needs only .debug_info section and doesn't need other sections,
>>so it needs only the debuginfo file. I will add the following comment to the
>>manual for "Debugging Information in Separate Files". Please let me know your
>>opinion.
>>
>>-------------------------------------------------------------------------------
>>       -x VMLINUX
>>              Specify the first kernel’s VMLINUX  with  debug  information  to
>>              analyze the first kernel’s memory usage.
>>+             If there is a file containing debug information and separated from
>>+             VMLINUX, the file should be specified instead of VMLINUX.
> 
> 
> This behavior seems to be different from that of crash. If a vmlinux
> does not contain debug_info and a separate vmlinux-<version>.debug is
> present, the 'crash' would try to look up for the accompanying debug
> file.
> 
> The way 'crash' looks up for the accompanying debug file seems
> ambiguous and not documented. The purpose of makedumpfile is to create
> an output file for 'crash' to consume. I hope 'crash' and 'makedumpfile'
> can use same semantics to specify the vmlinux and its accompanying
> debug file.

The crash utility hides the details of there being a separate debug
file in the same way the gdb does when working with a binary executable
that has a separate debuginfo file.  It tries *not* to be ambiguous,
i.e., the point is to stay true to the "crash vmlinux vmcore" model.

Not only that, crash needs stuff such as the kernel's static data,
which is located only in the stripped vmlinux file.  That's apparently
not the case with makedumpfile.

That being said, does any other distro besides RHEL3 actually use
the split vmlinux/vmlinux.debug format?  Thankfully Red Hat came
to its senses in RHEL4 and beyond...

Dave


> 
> Thanks,
>  - jay
> 
> 
>>              The  page  size of the first kernel and the second kernel should
>>              match.
>>              Example:
>>              # makedumpfile -d 31 -x vmlinux /proc/vmcore dumpfile
>>-------------------------------------------------------------------------------
>>
>>
>>Thanks
>>Ken'ichi Ohmichi
>>
>>_______________________________________________
>>kexec mailing list
>>kexec@lists.infradead.org
>>http://lists.infradead.org/mailman/listinfo/kexec
> 
> 



_______________________________________________
kexec mailing list
kexec@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/kexec

  reply	other threads:[~2007-05-24 18:27 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-05-01 13:39 [PATCH] [makedumpfile] Follow debuginfo link of vmlinux file Bernhard Walle
2007-05-24  0:35 ` Ken'ichi Ohmichi
2007-05-24  6:54   ` Bernhard Walle
2007-05-24  7:09     ` Bernhard Walle
2007-05-24 17:21   ` Jay Lan
2007-05-24 18:31     ` Dave Anderson [this message]
2007-05-24 19:21       ` Bernhard Walle
2007-05-24 19:44         ` Jay Lan
2007-05-24 19:54           ` Bernhard Walle
2007-05-29 11:43     ` Ken'ichi Ohmichi
2007-05-29 17:44       ` Jay Lan
2007-05-29 18:42         ` Bernhard Walle

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=4655D9EA.8020601@redhat.com \
    --to=anderson@redhat.com \
    --cc=bwalle@suse.de \
    --cc=jlan@sgi.com \
    --cc=kexec@lists.infradead.org \
    --cc=oomichi@mxs.nes.nec.co.jp \
    /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.