public inbox for linux-s390@vger.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox