[PATCH 1/2] makedumpfile: Understand >= v3.11-rc4 dmesg.

WANG Chao chaowang at redhat.com
Wed Nov 27 16:08:52 UTC 2013


This is a backport of commit a01b663 ("[PATCH v2] dump-dmesg: Understand
>= v3.11-rc4 dmesg."):

commit a01b663
Author: Lubomir Rintel <lkundrak at v3.sk>
Date:   Fri Sep 20 15:56:49 2013 +0900

    [PATCH v2] dump-dmesg: Understand >= v3.11-rc4 dmesg.

    Symbol name changed with the following commit:
    62e32ac printk: rename struct log to struct printk_log

    Changes for v2:
      * Only back values for symbol names we did actually read;
      *         either "log" or "printk_log"

makedumpfile --dump-dmesg is broken since VMCOREINFO symbol "log" has
renamed to "printk_log". This patch fixes --dump-dmesg on 3.11 kernel.

Signed-off-by: WANG Chao <chaowang at redhat.com>
---
 ...4-makedumpfile-Understand-v3.11-rc4-dmesg.patch | 197 +++++++++++++++++++++
 kexec-tools.spec                                   |   2 +
 2 files changed, 199 insertions(+)
 create mode 100644 kexec-tools-2.0.4-makedumpfile-Understand-v3.11-rc4-dmesg.patch

diff --git a/kexec-tools-2.0.4-makedumpfile-Understand-v3.11-rc4-dmesg.patch b/kexec-tools-2.0.4-makedumpfile-Understand-v3.11-rc4-dmesg.patch
new file mode 100644
index 0000000..084e7d5
--- /dev/null
+++ b/kexec-tools-2.0.4-makedumpfile-Understand-v3.11-rc4-dmesg.patch
@@ -0,0 +1,197 @@
+From a01b663749c4b221ecd03285fa24a4b31e742004 Mon Sep 17 00:00:00 2001
+From: Lubomir Rintel <lkundrak at v3.sk>
+Date: Fri, 20 Sep 2013 15:56:49 +0900
+Subject: [PATCH] [PATCH v2] dump-dmesg: Understand >= v3.11-rc4 dmesg.
+
+Symbol name changed with the following commit:
+62e32ac printk: rename struct log to struct printk_log
+
+Changes for v2:
+  * Only back values for symbol names we did actually read;
+    either "log" or "printk_log"
+
+Signed-off-by: Lubomir Rintel <lkundrak at v3.sk>
+---
+ makedumpfile.c | 69 +++++++++++++++++++++++++++++++++++++++++++---------------
+ makedumpfile.h |  7 +++---
+ 2 files changed, 55 insertions(+), 21 deletions(-)
+
+diff --git a/makedumpfile-1.5.4/makedumpfile.c b/makedumpfile-1.5.4/makedumpfile.c
+index e01ff50..7bbdcc2 100644
+--- a/makedumpfile-1.5.4/makedumpfile.c
++++ b/makedumpfile-1.5.4/makedumpfile.c
+@@ -1389,10 +1389,23 @@ get_structure_info(void)
+ 	OFFSET_INIT(elf64_phdr.p_paddr, "elf64_phdr", "p_paddr");
+ 	OFFSET_INIT(elf64_phdr.p_memsz, "elf64_phdr", "p_memsz");
+ 
+-	SIZE_INIT(log, "log");
+-	OFFSET_INIT(log.ts_nsec, "log", "ts_nsec");
+-	OFFSET_INIT(log.len, "log", "len");
+-	OFFSET_INIT(log.text_len, "log", "text_len");
++	SIZE_INIT(printk_log, "printk_log");
++	if (SIZE(printk_log) != NOT_FOUND_STRUCTURE) {
++		/*
++		 * In kernel 3.11-rc4 the log structure name was renamed
++		 * to "printk_log".
++		 */
++		info->flag_use_printk_log = TRUE;
++		OFFSET_INIT(printk_log.ts_nsec, "printk_log", "ts_nsec");
++		OFFSET_INIT(printk_log.len, "printk_log", "len");
++		OFFSET_INIT(printk_log.text_len, "printk_log", "text_len");
++	} else {
++		info->flag_use_printk_log = FALSE;
++		SIZE_INIT(printk_log, "log");
++		OFFSET_INIT(printk_log.ts_nsec, "log", "ts_nsec");
++		OFFSET_INIT(printk_log.len, "log", "len");
++		OFFSET_INIT(printk_log.text_len, "log", "text_len");
++	}
+ 
+ 	return TRUE;
+ }
+@@ -1593,7 +1606,10 @@ write_vmcoreinfo_data(void)
+ 	WRITE_STRUCTURE_SIZE("node_memblk_s", node_memblk_s);
+ 	WRITE_STRUCTURE_SIZE("nodemask_t", nodemask_t);
+ 	WRITE_STRUCTURE_SIZE("pageflags", pageflags);
+-	WRITE_STRUCTURE_SIZE("log", log);
++	if (info->flag_use_printk_log)
++		WRITE_STRUCTURE_SIZE("printk_log", printk_log);
++	else
++		WRITE_STRUCTURE_SIZE("log", printk_log);
+ 
+ 	/*
+ 	 * write the member offset of 1st kernel
+@@ -1628,9 +1644,16 @@ write_vmcoreinfo_data(void)
+ 	WRITE_MEMBER_OFFSET("vm_struct.addr", vm_struct.addr);
+ 	WRITE_MEMBER_OFFSET("vmap_area.va_start", vmap_area.va_start);
+ 	WRITE_MEMBER_OFFSET("vmap_area.list", vmap_area.list);
+-	WRITE_MEMBER_OFFSET("log.ts_nsec", log.ts_nsec);
+-	WRITE_MEMBER_OFFSET("log.len", log.len);
+-	WRITE_MEMBER_OFFSET("log.text_len", log.text_len);
++	if (info->flag_use_printk_log) {
++		WRITE_MEMBER_OFFSET("printk_log.ts_nsec", printk_log.ts_nsec);
++		WRITE_MEMBER_OFFSET("printk_log.len", printk_log.len);
++		WRITE_MEMBER_OFFSET("printk_log.text_len", printk_log.text_len);
++	} else {
++		/* Compatibility with pre-3.11-rc4 */
++		WRITE_MEMBER_OFFSET("log.ts_nsec", printk_log.ts_nsec);
++		WRITE_MEMBER_OFFSET("log.len", printk_log.len);
++		WRITE_MEMBER_OFFSET("log.text_len", printk_log.text_len);
++	}
+ 
+ 	if (SYMBOL(node_data) != NOT_FOUND_SYMBOL)
+ 		WRITE_ARRAY_LENGTH("node_data", node_data);
+@@ -1909,7 +1932,6 @@ read_vmcoreinfo(void)
+ 	READ_STRUCTURE_SIZE("node_memblk_s", node_memblk_s);
+ 	READ_STRUCTURE_SIZE("nodemask_t", nodemask_t);
+ 	READ_STRUCTURE_SIZE("pageflags", pageflags);
+-	READ_STRUCTURE_SIZE("log", log);
+ 
+ 	READ_MEMBER_OFFSET("page.flags", page.flags);
+ 	READ_MEMBER_OFFSET("page._count", page._count);
+@@ -1940,9 +1962,20 @@ read_vmcoreinfo(void)
+ 	READ_MEMBER_OFFSET("vm_struct.addr", vm_struct.addr);
+ 	READ_MEMBER_OFFSET("vmap_area.va_start", vmap_area.va_start);
+ 	READ_MEMBER_OFFSET("vmap_area.list", vmap_area.list);
+-	READ_MEMBER_OFFSET("log.ts_nsec", log.ts_nsec);
+-	READ_MEMBER_OFFSET("log.len", log.len);
+-	READ_MEMBER_OFFSET("log.text_len", log.text_len);
++
++	READ_STRUCTURE_SIZE("printk_log", printk_log);
++	if (SIZE(printk_log) != NOT_FOUND_STRUCTURE) {
++		info->flag_use_printk_log = TRUE;
++		READ_MEMBER_OFFSET("printk_log.ts_nsec", printk_log.ts_nsec);
++		READ_MEMBER_OFFSET("printk_log.len", printk_log.len);
++		READ_MEMBER_OFFSET("printk_log.text_len", printk_log.text_len);
++	} else {
++		info->flag_use_printk_log = FALSE;
++		READ_STRUCTURE_SIZE("log", printk_log);
++		READ_MEMBER_OFFSET("log.ts_nsec", printk_log.ts_nsec);
++		READ_MEMBER_OFFSET("log.len", printk_log.len);
++		READ_MEMBER_OFFSET("log.text_len", printk_log.text_len);
++	}
+ 
+ 	READ_ARRAY_LENGTH("node_data", node_data);
+ 	READ_ARRAY_LENGTH("pgdat_list", pgdat_list);
+@@ -3710,13 +3743,13 @@ dump_log_entry(char *logptr, int fp)
+ 	ulonglong nanos;
+ 	ulong rem;
+ 
+-	text_len = USHORT(logptr + OFFSET(log.text_len));
+-	ts_nsec = ULONGLONG(logptr + OFFSET(log.ts_nsec));
++	text_len = USHORT(logptr + OFFSET(printk_log.text_len));
++	ts_nsec = ULONGLONG(logptr + OFFSET(printk_log.ts_nsec));
+ 
+ 	nanos = (ulonglong)ts_nsec / (ulonglong)1000000000;
+ 	rem = (ulonglong)ts_nsec % (ulonglong)1000000000;
+ 
+-	msg = logptr + SIZE(log);
++	msg = logptr + SIZE(printk_log);
+ 
+ 	sprintf(buf, "[%5lld.%06ld] ", nanos, rem/1000);
+ 
+@@ -3754,7 +3787,7 @@ log_from_idx(unsigned int idx, char *logbuf)
+ 	 * the buffer.
+ 	 */
+ 
+-	msglen = USHORT(logptr + OFFSET(log.len));
++	msglen = USHORT(logptr + OFFSET(printk_log.len));
+ 	if (!msglen)
+ 		logptr = logbuf;
+ 
+@@ -3775,9 +3808,9 @@ log_next(unsigned int idx, char *logbuf)
+ 	 * return the one after that.
+ 	 */
+ 
+-	msglen = USHORT(logptr + OFFSET(log.len));
++	msglen = USHORT(logptr + OFFSET(printk_log.len));
+ 	if (!msglen) {
+-		msglen = USHORT(logbuf + OFFSET(log.len));
++		msglen = USHORT(logbuf + OFFSET(printk_log.len));
+ 		return msglen;
+ 	}
+ 
+diff --git a/makedumpfile-1.5.4/makedumpfile.h b/makedumpfile-1.5.4/makedumpfile.h
+index c504bfb..3a7e61a 100644
+--- a/makedumpfile-1.5.4/makedumpfile.h
++++ b/makedumpfile-1.5.4/makedumpfile.h
+@@ -893,6 +893,7 @@ struct DumpInfo {
+ 	int		flag_force;	     /* overwrite existing stuff */
+ 	int		flag_exclude_xen_dom;/* exclude Domain-U from xen-kdump */
+ 	int             flag_dmesg;          /* dump the dmesg log out of the vmcore file */
++	int		flag_use_printk_log; /* did we read printk_log symbol name? */
+ 	int		flag_nospace;	     /* the flag of "No space on device" error */
+ 	unsigned long	vaddr_for_vtop;      /* virtual address for debugging */
+ 	long		page_size;           /* size of page */
+@@ -1176,6 +1177,7 @@ struct size_table {
+ 	long	list_head;
+ 	long	node_memblk_s;
+ 	long	nodemask_t;
++	long	printk_log;
+ 
+ 	/*
+ 	 * for Xen extraction
+@@ -1198,7 +1200,6 @@ struct size_table {
+ 	long	cpumask_t;
+ 	long	kexec_segment;
+ 	long	elf64_hdr;
+-	long	log;
+ 
+ 	long	pageflags;
+ };
+@@ -1337,11 +1338,11 @@ struct offset_table {
+ 		long	p_memsz;
+ 	} elf64_phdr;
+ 
+-	struct log_s {
++	struct printk_log_s {
+ 		long ts_nsec;
+ 		long len;
+ 		long text_len;
+-	} log;
++	} printk_log;
+ 
+ };
+ 
+-- 
+1.8.3.1
+
diff --git a/kexec-tools.spec b/kexec-tools.spec
index 00eb603..6b2a673 100644
--- a/kexec-tools.spec
+++ b/kexec-tools.spec
@@ -89,6 +89,7 @@ Patch608: kexec-tools-2.0.4-makedumpfile-Use-divideup-to-calculate-maximum-requi
 Patch609: kexec-tools-2.0.4-makedumpfile-cache-Allocate-buffers-at-initialization-t.patch
 Patch610: kexec-tools-2.0.4-makedumpfile-cache-Reuse-entry-in-pending-list.patch
 Patch611: kexec-tools-2.0.4-makedumpfile-disable-mmap.patch
+Patch612: kexec-tools-2.0.4-makedumpfile-Understand-v3.11-rc4-dmesg.patch
 
 %description
 kexec-tools provides /sbin/kexec binary that facilitates a new
@@ -131,6 +132,7 @@ tar -z -x -v -f %{SOURCE19}
 %patch001 -p1
 %patch002 -p1
 %patch003 -p1
+%patch612 -p1
 
 
 tar -z -x -v -f %{SOURCE13}
-- 
1.8.3.1



More information about the kexec mailing list