All of lore.kernel.org
 help / color / mirror / Atom feed
From: Akio Takebe <takebe_akio@jp.fujitsu.com>
To: "Graham, Simon" <Simon.Graham@stratus.com>,
	xen-devel <xen-devel@lists.xensource.com>
Cc: Akio Takebe <takebe_akio@jp.fujitsu.com>
Subject: RE: [Patch][RFC] Support "xm dump" (is Re:Re:[Patch]Enable "sysrq c" handler for domU coredump)
Date: Fri, 04 Aug 2006 18:58:47 +0900	[thread overview]
Message-ID: <91C6B7AC943702takebe_akio@jp.fujitsu.com> (raw)
In-Reply-To: <342BAC0A5467384983B586A6B0B3767103471018@EXNA.corp.stratus.com>

Hi, Simon

>I mean that if you want to crash a domain and collect a dump, you can do
>'xm dump' followed by 'xm destroy' or 'xm reboot' -- there's no need to 
>add another command to do the combination.
OK, I understand.

>> >2. I don't see the point of the --noreboot option on 'xm dump' -- I
>> >think this command
>> >   should simply live-dump the specified domain - as above you can
>use
>> >other commands
>> >   to cause the domain to restart afterwards.
>> >
>> Ordinary dump features have atomatically rebooting features.
>> (e.g. diskdump, kdump, and so on)
>> So I think this is necessary.
>
>xm dump foo
>xm reboot foo
OK.

>
>does the job nicely -- why complicate things by adding extra
>options/commands
>
>> 
>> >3. There's no need to pause the domain to dump it - I actually wrote
>a
>> >little utility
>> >   to live dump a guest (based on xenconsoled and attached) and it
>> seems
>> >to work
>> >   quite nicely! This could easily be morphed into the 'xm dump'
>> command
>> >- it just
>> >   didn't occur to me at the time!
>> >
>> Your xendump command is dump feature without pause.
>> In the case without pause, domain's memory is modified while dumping.
>> I think both w/o and w pause are needed.
>
>Hmm... perhaps although I'm not convinced -- I understand that memory
>can change during the dump and therefore there can be some
>inconsistencies in the dump, HOWEVER, pausing the domain doesn't ensure
>all the data structures are consistent either - it pretty much just
>stops the VM wherever is happens to be so the memory can still be
>inconsistent. 
>
>I also think there is an issue with pausing -- unless I am mistaken,
>pause requires code to run in the domain - if the domain is glued up
>this cant run and the pause will hang (and lets face it, the usual
>reason for dumping a domain is because something is wrong; definitely a
>good idea to minimize the amount of work you expect from the domain in
>this case).
>
>FWIW, my opinion is that it isn't necessary to pause.
>
I don't agree it. It is necessary to pause for dumping domU's core,
because debugging is difficut with the unpause core.

But I think we can resolve the issue with the following way.
- If we want to get dump with pause
  1. xm pause foo
  2. xm dump foo
  3. xm reboot foo (if want to reboot)

- If we want to get live dump w/o pause
  1. xm dump foo

BTW, I try to make your xendump.c, but cannot compile it.
Could you post the Makefile?
I'd like to use your xendump.

Best Regards,

Akio Takebe

  reply	other threads:[~2006-08-04  9:58 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-08-03 21:52 [Patch][RFC] Support "xm dump" (is Re: Re:[Patch]Enable "sysrq c" handler for domU coredump) Graham, Simon
2006-08-04  9:58 ` Akio Takebe [this message]
2006-08-04 10:29   ` [Patch][RFC] Support "xm dump" (is " Harry Butterworth
  -- strict thread matches above, loose matches on Subject: below --
2006-08-04 13:58 [Patch][RFC] Support "xm dump" (is " Graham, Simon
2006-08-03 14:45 [Patch][RFC] Support "xm dump" (is Re: Re: [Patch]Enable "sysrq " Graham, Simon
2006-08-03 15:24 ` [Patch][RFC] Support "xm dump" (is Re: " Akio Takebe
2006-08-08 12:31   ` John Levon
2006-08-08 15:00     ` Akio Takebe
2006-08-01  9:26 [Patch] Enable "sysrq c" handler for domU coredump Horms
2006-08-01 10:33 ` Akio Takebe
2006-08-03  0:18   ` [Patch][RFC] Support "xm dump" (is Re: Re: [Patch] Enable "sysrq c" handler for domU coredump) Akio Takebe
2006-08-03  0:26     ` John Levon
2006-08-03  6:09       ` Tristan Gingold
2006-08-03 11:10         ` Akio Takebe
2006-08-03 11:58           ` Tristan Gingold
2006-08-03 12:57             ` Akio Takebe
2006-08-07 18:04           ` John Levon
2006-08-03  1:50     ` Horms
2006-08-03 11:10       ` Akio Takebe

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=91C6B7AC943702takebe_akio@jp.fujitsu.com \
    --to=takebe_akio@jp.fujitsu.com \
    --cc=Simon.Graham@stratus.com \
    --cc=xen-devel@lists.xensource.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.