All of lore.kernel.org
 help / color / mirror / Atom feed
From: Philipp Rudo <prudo@linux.ibm.com>
To: Petr Tesarik <ptesarik@suse.cz>
Cc: linux-s390@vger.kernel.org, Michael Holzheu <holzheu@linux.vnet.ibm.com>
Subject: Re: Is OS_INFO_VMCOREINFO unimplemented?
Date: Mon, 19 Oct 2020 12:18:11 +0200	[thread overview]
Message-ID: <20201019121811.2fd4c598@ibm> (raw)
In-Reply-To: <20201016172419.3abfdeda@ezekiel.suse.cz>

Hi Petr,

On Fri, 16 Oct 2020 17:24:19 +0200
Petr Tesarik <ptesarik@suse.cz> wrote:

> Hi Philipp,
> 
> On Fri, 16 Oct 2020 16:11:25 +0200
> Philipp Rudo <prudo@linux.ibm.com> wrote:
> 
> > Hi Petr,
> > 
> > sorry for the late reply.  
> 
> No problem. ;-)
> 
> > It's an interface for non-Linux systems for the stand-alone kdump.
> > 
> > But that's all I'm sure of. I'm afraid only Michael knows the full history
> > behind the implementation. Unfortunately he left IBM ~2 years ago so this piece
> > of knowledge is lost...
> > 
> > My theory is that originally it was planned to use this mechanism for the
> > "normal" kdump as well. But for kdump common code "corrupts" the vmcoreinfo by
> > adding the CRASHTIME shortly before kexec'ing the crash kernel. So the crash
> > kernel would refuse to load the os_info anyway and thus it is never set.  
> 
> Sure, the checksum would have to be recalculated after setting CRASHTIME. But that's perfectly possible.

True. But it's not working out of the box and there's a working workaround. So
let's live with the workaround and implement the proper solution at a later
date. You know, the usual stuff... 

> > Hope this helps you at least a little  
> 
> Yes, to some extent. The reason I asked was that I also implemented parsing of OS_INFO_VMCOREINFO in libkdumpfile a few years ago, but it has no test coverage. So, I looked around a bit and to my surprise all dump files contained a NULL pointer there, which looked somewhat suspicious.
> 
> Anyway, if nobody knows for certain, then my plan is to add the necessary code to the Linux kernel. Patch coming soon on the mailing list. ;-)

Not to discourage you, but my long term goal was to remove the mechanism.
Anyhow I'm willing to revisit this plan. You definitely got me curious :)

Thanks
Philipp


> Thanks,
> Petr T
> 
> > Philipp
> > 
> > 
> > On Tue, 13 Oct 2020 14:53:03 +0200
> > Petr Tesarik <ptesarik@suse.cz> wrote:
> >   
> > > Hi all,
> > > 
> > > I've been looking into kernel crash dump analysis for some time now,
> > > and I've noticed that none of my sample dumps for z/Architecture sets
> > > OS_INFO_VMCOREINFO.
> > > 
> > > Commit 4857d4bbe9821c8d732cb84455e18e12b3d79add suggests that the
> > > "os_info" structure should contain vmcoreinfo, but it seems that the
> > > corresponding fields are always zero, and a quick grep for
> > > OS_INFO_VMCOREINFO finds only code that tries to read this entry in the
> > > panic kernel, but no code that would initialize it in the old (crashed)
> > > kernel.
> > > 
> > > In short, the panic kernel always prints an informational message that
> > > entry 0 is not available and falls back to get_vmcoreinfo_old().
> > > 
> > > Is this a bug, or is this interface used by a non-Linux operating
> > > system that I'm not aware of?
> > > 
> > > TIA,
> > > Petr Tesarik
> > > SUSE HW Enablement    
> 

      reply	other threads:[~2020-10-19 10:18 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-10-13 12:53 Is OS_INFO_VMCOREINFO unimplemented? Petr Tesarik
2020-10-16 14:11 ` Philipp Rudo
2020-10-16 15:24   ` Petr Tesarik
2020-10-19 10:18     ` Philipp Rudo [this message]

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=20201019121811.2fd4c598@ibm \
    --to=prudo@linux.ibm.com \
    --cc=holzheu@linux.vnet.ibm.com \
    --cc=linux-s390@vger.kernel.org \
    --cc=ptesarik@suse.cz \
    /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.