Re: [libvirt] [PATCH] docs: virsh: Added note for the dump command

[Martin Kletzander]

On Mon, Jun 20, 2016 at 01:34:28PM +0200, jsuch...@redhat.com wrote:

From: Jaroslav Suchanek 



Feel free to fix your git send-email settings if you used that to send
the patch, or otherwise fix your MUA or whatever you used.


Crash dump in a old kvmdump format is being obsolete and cannot be loaded and


s/a old/an old/ or just remove the 'a'.


processed by crash utility since its version 6.1.0. A --memory-only option is
required in order to produce valid ELF file which can be later processed by the
crash utility. A new note is added to the dump command description.
---
tools/virsh.pod | 5 +
1 file changed, 5 insertions(+)

diff --git a/tools/virsh.pod b/tools/virsh.pod
index 98fed76..8e3ba69 100644
--- a/tools/virsh.pod
+++ b/tools/virsh.pod
@@ -1355,6 +1355,11 @@ B command. I<--verbose> displays the progress of 
dump.
NOTE: Some hypervisors may require the user to manually ensure proper
permissions on file and path specified by argument I.

+NOTE: Crash dump in a old kvmdump format is being obsolete and cannot be loaded


Same here.  Since that is trivial, I'll fix that and push the patch.
Congratulations on your first libvirt patch!


+and processed by crash utility since its version 6.1.0. A --memory-only option
+is required in order to produce valid ELF file which can be later processed by
+the crash utility.
+
=item B I [I<--inactive>] [I<--security-info>]
[I<--update-cpu>] [I<--migratable>]

--
1.8.3.1

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list


signature.asc
Description: Digital signature
--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list

[libvirt] [PATCH] docs: virsh: Added note for the dump command

[jsuchane]

From: Jaroslav Suchanek 

Crash dump in a old kvmdump format is being obsolete and cannot be loaded and
processed by crash utility since its version 6.1.0. A --memory-only option is
required in order to produce valid ELF file which can be later processed by the
crash utility. A new note is added to the dump command description.
---
 tools/virsh.pod | 5 +
 1 file changed, 5 insertions(+)

diff --git a/tools/virsh.pod b/tools/virsh.pod
index 98fed76..8e3ba69 100644
--- a/tools/virsh.pod
+++ b/tools/virsh.pod
@@ -1355,6 +1355,11 @@ B command. I<--verbose> displays the progress of 
dump.
 NOTE: Some hypervisors may require the user to manually ensure proper
 permissions on file and path specified by argument I.

+NOTE: Crash dump in a old kvmdump format is being obsolete and cannot be loaded
+and processed by crash utility since its version 6.1.0. A --memory-only option
+is required in order to produce valid ELF file which can be later processed by
+the crash utility.
+
 =item B I [I<--inactive>] [I<--security-info>]
 [I<--update-cpu>] [I<--migratable>]

-- 
1.8.3.1

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list