From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from fgwmail5.fujitsu.co.jp ([192.51.44.35]) by merlin.infradead.org with esmtps (Exim 4.80.1 #2 (Red Hat Linux)) id 1UHmJf-00059l-Qx for kexec@lists.infradead.org; Tue, 19 Mar 2013 02:30:21 +0000 Received: from m4.gw.fujitsu.co.jp (unknown [10.0.50.74]) by fgwmail5.fujitsu.co.jp (Postfix) with ESMTP id EE18B3EE0C3 for ; Tue, 19 Mar 2013 11:30:15 +0900 (JST) Received: from smail (m4 [127.0.0.1]) by outgoing.m4.gw.fujitsu.co.jp (Postfix) with ESMTP id D11B745DE51 for ; Tue, 19 Mar 2013 11:30:15 +0900 (JST) Received: from s4.gw.fujitsu.co.jp (s4.gw.fujitsu.co.jp [10.0.50.94]) by m4.gw.fujitsu.co.jp (Postfix) with ESMTP id B833445DE4F for ; Tue, 19 Mar 2013 11:30:15 +0900 (JST) Received: from s4.gw.fujitsu.co.jp (localhost.localdomain [127.0.0.1]) by s4.gw.fujitsu.co.jp (Postfix) with ESMTP id A709E1DB803B for ; Tue, 19 Mar 2013 11:30:15 +0900 (JST) Received: from m1001.s.css.fujitsu.com (m1001.s.css.fujitsu.com [10.240.81.139]) by s4.gw.fujitsu.co.jp (Postfix) with ESMTP id 5A171E08004 for ; Tue, 19 Mar 2013 11:30:15 +0900 (JST) From: HATAYAMA Daisuke Subject: [PATCH v3 03/21] vmcore: rearrange program headers without assuming consequtive PT_NOTE entries Date: Sat, 16 Mar 2013 13:01:04 +0900 Message-ID: <20130316040104.15064.57180.stgit@localhost6.localdomain6> In-Reply-To: <20130316040003.15064.62308.stgit@localhost6.localdomain6> References: <20130316040003.15064.62308.stgit@localhost6.localdomain6> MIME-Version: 1.0 List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "kexec" Errors-To: kexec-bounces+dwmw2=infradead.org@lists.infradead.org To: vgoyal@redhat.com, ebiederm@xmission.com, cpw@sgi.com, kumagai-atsushi@mxc.nes.nec.co.jp, lisa.mitchell@hp.com, heiko.carstens@de.ibm.com, akpm@linux-foundation.org Cc: zhangyanfei@cn.fujitsu.com, kexec@lists.infradead.org, linux-kernel@vger.kernel.org Current code assumes all PT_NOTE headers are placed at the beginning of program header table and they are consequtive. But the assumption could be broken by future changes on either kexec-tools or the 1st kernel. This patch removes the assumption and rearranges program headers as the following conditions are satisfied: - PT_NOTE entry is unique at the first entry, - the order of program headers are unchanged during this rearrangement, only their positions are changed in positive direction. - unused part that occurs in the bottom of program headers are filled with 0. Also, this patch adds one exceptional case where the number of PT_NOTE entries is somehow 0. Then, immediately go out of the function. Signed-off-by: HATAYAMA Daisuke --- fs/proc/vmcore.c | 92 +++++++++++++++++++++++++++++++++++++++++++----------- 1 files changed, 74 insertions(+), 18 deletions(-) diff --git a/fs/proc/vmcore.c b/fs/proc/vmcore.c index 94743d2..7d2dc4c 100644 --- a/fs/proc/vmcore.c +++ b/fs/proc/vmcore.c @@ -251,8 +251,7 @@ static u64 __init get_vmcore_size_elf32(char *elfptr) static int __init merge_note_headers_elf64(char *elfptr, size_t *elfsz, struct list_head *vc_list) { - int i, nr_ptnote=0, rc=0; - char *tmp; + int i, j, nr_ptnote=0, i_ptnote, rc=0; Elf64_Ehdr *ehdr_ptr; Elf64_Phdr phdr, *phdr_ptr; Elf64_Nhdr *nhdr_ptr; @@ -301,6 +300,39 @@ static int __init merge_note_headers_elf64(char *elfptr, size_t *elfsz, kfree(notes_section); } + if (nr_ptnote == 0) + goto out; + + phdr_ptr = (Elf64_Phdr *)(elfptr + ehdr_ptr->e_phoff); + + /* Remove unwanted PT_NOTE program headers. */ + + /* - 1st pass shifts non-PT_NOTE entries until the first + PT_NOTE entry. */ + i_ptnote = -1; + for (i = 0; i < ehdr_ptr->e_phnum; ++i) { + if (phdr_ptr[i].p_type == PT_NOTE) { + i_ptnote = i; + break; + } + } + BUG_ON(i_ptnote == -1); /* impossible case since nr_ptnote > 0. */ + memmove(phdr_ptr + 1, phdr_ptr, i_ptnote * sizeof(Elf64_Phdr)); + + /* - 2nd pass moves the remaining non-PT_NOTE entries under + the first PT_NOTE entry. */ + for (i = j = i_ptnote + 1; i < ehdr_ptr->e_phnum; i++) { + if (phdr_ptr[i].p_type != PT_NOTE) { + memmove(phdr_ptr + j, phdr_ptr + i, + sizeof(Elf64_Phdr)); + j++; + } + } + + /* - Finally, fill unused part with 0. */ + memset(phdr_ptr + ehdr_ptr->e_phnum - (nr_ptnote - 1), 0, + (nr_ptnote - 1) * sizeof(Elf64_Phdr)); + /* Prepare merged PT_NOTE program header. */ phdr.p_type = PT_NOTE; phdr.p_flags = 0; @@ -312,18 +344,14 @@ static int __init merge_note_headers_elf64(char *elfptr, size_t *elfsz, phdr.p_align = 0; /* Add merged PT_NOTE program header*/ - tmp = elfptr + ehdr_ptr->e_phoff; - memcpy(tmp, &phdr, sizeof(phdr)); - tmp += sizeof(phdr); + memcpy(phdr_ptr, &phdr, sizeof(Elf64_Phdr)); - /* Remove unwanted PT_NOTE program headers. */ - i = (nr_ptnote - 1) * sizeof(Elf64_Phdr); - *elfsz = *elfsz - i; - memmove(tmp, tmp+i, ((*elfsz)-ehdr_ptr->e_phoff-sizeof(Elf64_Phdr))); + *elfsz = *elfsz - (nr_ptnote - 1) * sizeof(Elf64_Phdr); /* Modify e_phnum to reflect merged headers. */ ehdr_ptr->e_phnum = ehdr_ptr->e_phnum - nr_ptnote + 1; +out: return 0; } @@ -331,8 +359,7 @@ static int __init merge_note_headers_elf64(char *elfptr, size_t *elfsz, static int __init merge_note_headers_elf32(char *elfptr, size_t *elfsz, struct list_head *vc_list) { - int i, nr_ptnote=0, rc=0; - char *tmp; + int i, j, nr_ptnote=0, i_ptnote, rc=0; Elf32_Ehdr *ehdr_ptr; Elf32_Phdr phdr, *phdr_ptr; Elf32_Nhdr *nhdr_ptr; @@ -381,6 +408,39 @@ static int __init merge_note_headers_elf32(char *elfptr, size_t *elfsz, kfree(notes_section); } + if (nr_ptnote == 0) + goto out; + + phdr_ptr = (Elf32_Phdr *)(elfptr + ehdr_ptr->e_phoff); + + /* Remove unwanted PT_NOTE program headers. */ + + /* - 1st pass shifts non-PT_NOTE entries until the first + PT_NOTE entry. */ + i_ptnote = -1; + for (i = 0; i < ehdr_ptr->e_phnum; ++i) { + if (phdr_ptr[i].p_type == PT_NOTE) { + i_ptnote = i; + break; + } + } + BUG_ON(i_ptnote == -1); /* impossible case since nr_ptnote > 0. */ + memmove(phdr_ptr + 1, phdr_ptr, i_ptnote * sizeof(Elf32_Phdr)); + + /* - 2nd pass moves the remaining non-PT_NOTE entries under + the first PT_NOTE entry. */ + for (i = j = i_ptnote + 1; i < ehdr_ptr->e_phnum; i++) { + if (phdr_ptr[i].p_type != PT_NOTE) { + memmove(phdr_ptr + j, phdr_ptr + i, + sizeof(Elf32_Phdr)); + j++; + } + } + + /* - Finally, fill unused part with 0. */ + memset(phdr_ptr + ehdr_ptr->e_phnum - (nr_ptnote - 1), 0, + (nr_ptnote - 1) * sizeof(Elf32_Phdr)); + /* Prepare merged PT_NOTE program header. */ phdr.p_type = PT_NOTE; phdr.p_flags = 0; @@ -392,18 +452,14 @@ static int __init merge_note_headers_elf32(char *elfptr, size_t *elfsz, phdr.p_align = 0; /* Add merged PT_NOTE program header*/ - tmp = elfptr + ehdr_ptr->e_phoff; - memcpy(tmp, &phdr, sizeof(phdr)); - tmp += sizeof(phdr); + memcpy(phdr_ptr, &phdr, sizeof(Elf32_Phdr)); - /* Remove unwanted PT_NOTE program headers. */ - i = (nr_ptnote - 1) * sizeof(Elf32_Phdr); - *elfsz = *elfsz - i; - memmove(tmp, tmp+i, ((*elfsz)-ehdr_ptr->e_phoff-sizeof(Elf32_Phdr))); + *elfsz = *elfsz - (nr_ptnote - 1) * sizeof(Elf32_Phdr); /* Modify e_phnum to reflect merged headers. */ ehdr_ptr->e_phnum = ehdr_ptr->e_phnum - nr_ptnote + 1; +out: return 0; } _______________________________________________ kexec mailing list kexec@lists.infradead.org http://lists.infradead.org/mailman/listinfo/kexec