From patchwork Tue Sep 12 00:37:15 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Konrad Rzeszutek Wilk X-Patchwork-Id: 9948221 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id 0A2106038F for ; Tue, 12 Sep 2017 00:40:08 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id EF23F28CE9 for ; Tue, 12 Sep 2017 00:40:07 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id E3D7828D31; Tue, 12 Sep 2017 00:40:07 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-3.6 required=2.0 tests=BAYES_00,DKIM_SIGNED, RCVD_IN_DNSWL_MED,RCVD_IN_SORBS_SPAM,T_DKIM_INVALID autolearn=ham version=3.3.1 Received: from lists.xenproject.org (lists.xenproject.org [192.237.175.120]) (using TLSv1.2 with cipher AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by mail.wl.linuxfoundation.org (Postfix) with ESMTPS id 2161B28D43 for ; Tue, 12 Sep 2017 00:40:02 +0000 (UTC) Received: from localhost ([127.0.0.1] helo=lists.xenproject.org) by lists.xenproject.org with esmtp (Exim 4.84_2) (envelope-from ) id 1drZDB-0000f6-Ak; Tue, 12 Sep 2017 00:37:57 +0000 Received: from mail6.bemta3.messagelabs.com ([195.245.230.39]) by lists.xenproject.org with esmtp (Exim 4.84_2) (envelope-from ) id 1drZDA-0000eX-7p for xen-devel@lists.xenproject.org; Tue, 12 Sep 2017 00:37:56 +0000 Received: from [85.158.137.68] by server-8.bemta-3.messagelabs.com id 04/28-02224-36C27B95; Tue, 12 Sep 2017 00:37:55 +0000 X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFrrIIsWRWlGSWpSXmKPExsVyMfTOId0kne2 RBt9aVC2+b5nM5MDocfjDFZYAxijWzLyk/IoE1oy+OU/YCt7bVfyfdpKlgbHfrIuRi0NIYCaj xJeODYwgDovABxaJx08usIA4EgLTWCWeHXzI2sXICeTESRw+8RfKTpPYNesRI4RdLfH583IwW 0hASWLL5MeMEGMPMkm8bGoBaxAW0JOY/O02WBGbgL7E07XXmLsYOYCa3SQ+XeMCqRcRaGeUmH 3lF1g9s4ChROvbo2wQvdkSB3e8ButlEVCVWHxvGTOIzStgKdHx/DE7xBHyEhN7p4HVcApYSfz +MY8V4iCgmgVXmCYwCi9gZFjFqFGcWlSWWqRrZK6XVJSZnlGSm5iZo2toYKyXm1pcnJiempOY VKyXnJ+7iREYpPUMDIw7GFv2+h1ilORgUhLl/S28PVKILyk/pTIjsTgjvqg0J7X4EKMMB4eSB K+hNlBOsCg1PbUiLTMHGC8waQkOHiUR3iwtoDRvcUFibnFmOkTqFKMlx4U7l/4wcRzYcwtIdt y8+4dJiCUvPy9VSpz3N0iDAEhDRmke3DhYTF9ilJUS5mVkYGAQ4ilILcrNLEGVf8UozsGoJMy rDnIVT2ZeCdzWV0AHMQEdxHNpC8hBJYkIKakGRtV/c6J3vXRMjPN+b711l9QchiK3jG/mCkoZ 3P4Xt731myMS9spGvtZ9UcDVSd+zltrI3ZLW6U4yDLKIUBL895VP4VV+gmvJ5Ljcc58qqrLYn u053hZhVJAtfbg16uwxFrf/D4NbFLevTDcVvCQjv+7w5d6bsy1iWBKnHZoyIbH9+c575mcTlF iKMxINtZiLihMBvYLk4eQCAAA= X-Env-Sender: ketuzsezr@gmail.com X-Msg-Ref: server-4.tower-31.messagelabs.com!1505176673!56212891!1 X-Originating-IP: [209.85.220.194] X-SpamReason: No, hits=0.0 required=7.0 tests= X-StarScan-Received: X-StarScan-Version: 9.4.45; banners=-,-,- X-VirusChecked: Checked Received: (qmail 61876 invoked from network); 12 Sep 2017 00:37:54 -0000 Received: from mail-qk0-f194.google.com (HELO mail-qk0-f194.google.com) (209.85.220.194) by server-4.tower-31.messagelabs.com with AES128-GCM-SHA256 encrypted SMTP; 12 Sep 2017 00:37:54 -0000 Received: by mail-qk0-f194.google.com with SMTP id g128so6397311qke.0 for ; Mon, 11 Sep 2017 17:37:54 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=sender:from:to:cc:subject:date:message-id:in-reply-to:references; bh=xgcwW5YAa/yIxWfEkfzGIRm74kNogN20z7Wdob9Hco4=; b=kkoSlQDKesFYawq88g55PPODVMNNSmNP1895cBaObPIS4osSUDEnh/WT7k+MpfxzPW f7dG+DAynPzgRHnSXtAYyKcl9Ok+Ox2WGalWmgGEHyLzwC3EM1fLLasrETCTZS31iRJx hYN/uWylo3+Pa0B2d3uBThQcIkBvsQWQNU0V1JIon6gvMm7ubkGBnb3fmJad2dQp5bVf M+akRs5CRgrzmv451GhBuUAcjGLJP3MV5YQWkpn7CeQrES/k17zLKZVegkiNB91ocTiU HnW1Ri2KJbEsxnjhxboUVlEbu7vvLZksUGcB/PXF30a3vbpqkfd3TNuXdrcwFBCPhzGn 7bhQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:sender:from:to:cc:subject:date:message-id :in-reply-to:references; bh=xgcwW5YAa/yIxWfEkfzGIRm74kNogN20z7Wdob9Hco4=; b=oPVG3Qz5y+6WK1JkEi7l6+AyIY0008BV5Q42tK4X/K5gLrzJNyqjDQVH7Yvy+JZQQz MvgeAfcEGufDFtKrJ85dLRMQprMwVuHjMoS5TrqVGv9j08Iz5P2tbT1ajQhwLOcwYZ2D udhCTvwlf/7HYxZcn/S0zUfwpfHwwNzWMV+UHfoRVScDnpbh0WQ8e7iMkVmUYnzxpFgA Kue92OOO4GrSpOZULsGa/NhYzUUZq9FS+iQaLV7nvDbJNVgIRv5xnlEYkRvB8kee2WkT tbYLzks3IQKjeeFZ4teOAtuvkYTlCXb16q6OcGA1+bCR9Uy03phMSm5fNIsG0d2PONh9 +AKw== X-Gm-Message-State: AHPjjUhfKcLS8xfhVuc6vVY9QpOY/mfr94RYJueYHX9y8F+7TRp9EWaa 2S9ps/JiYHEDiXkB X-Google-Smtp-Source: AOwi7QAtjRs7OgF0oE74DHeBBDsZaia8k8eD24Ew5xSiYOzoZcFbz2vBLcGvJaeHLHmnnB2F2We5kA== X-Received: by 10.55.99.139 with SMTP id x133mr18390505qkb.263.1505176672779; Mon, 11 Sep 2017 17:37:52 -0700 (PDT) Received: from localhost.localdomain (209-6-200-48.s4398.c3-0.smr-ubr2.sbo-smr.ma.cable.rcncustomer.com. [209.6.200.48]) by smtp.gmail.com with ESMTPSA id z75sm6771034qkb.71.2017.09.11.17.37.51 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Mon, 11 Sep 2017 17:37:52 -0700 (PDT) From: Konrad Rzeszutek Wilk X-Google-Original-From: Konrad Rzeszutek Wilk To: xen-devel@lists.xenproject.org, ross.lagerwall@citrix.com, konrad.wilk@oracle.com, julien.grall@arm.com, sstabellini@kernel.org Date: Mon, 11 Sep 2017 20:37:15 -0400 Message-Id: <20170912003726.368-7-konrad.wilk@oracle.com> X-Mailer: git-send-email 2.13.3 In-Reply-To: <20170912003726.368-1-konrad.wilk@oracle.com> References: <20170912003726.368-1-konrad.wilk@oracle.com> Cc: andrew.cooper3@citrix.com, jbeulich@suse.com Subject: [Xen-devel] [PATCH v3 06/17] xen/livepatch/x86/arm32: Force .livepatch.depends section to be uint32_t aligned. X-BeenThere: xen-devel@lists.xen.org X-Mailman-Version: 2.1.18 Precedence: list List-Id: Xen developer discussion List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , MIME-Version: 1.0 Errors-To: xen-devel-bounces@lists.xen.org Sender: "Xen-devel" X-Virus-Scanned: ClamAV using ClamSMTP By default when using objcopy we lose the alignment when we copy it from xen-syms - with the result that alignment (on ARM32 for example) can be 1: [Nr] Name Type Addr Off Size ES Flg Lk Inf Al .. [ 6] .livepatch.depend PROGBITS 00000000 000093 000024 00 A 0 0 1 That, combined with wacky offset means it will be loaded in memory with the wrong alignment: (XEN) livepatch.c:425: livepatch: xen_bye_world: Loaded .livepatch.depends at 000a08043 And later we get: (XEN) livepatch.c:501: livepatch: xen_bye_world: .livepatch.depends is not aligned properly! This fix forces all the test-cases to be built with a .livepatch.depends structure containing the build-id extracted from the hypervisor (except the xen_bye_world test-case). We use the 'mkhex' tool instead of 'xxd' as the end result is an 'unsigned' instead of 'char' type array - which naturally forces the alignment to be of four. Also the 'mkhex' tools allows us to pass the section name as parameter. The end result is much better alignment: [ 7] .livepatch.depend PROGBITS 00000000 000094 000024 00 A 0 0 4 Note that thanks to 'unsigned int .. __note_depends' the symbol becomes global: $ readelf --symbols *.livepatch | grep depen 23: 0000000000000000 36 OBJECT GLOBAL HIDDEN 6 note_depends 49: 0000000000000000 36 OBJECT GLOBAL HIDDEN 17 note_depends 16: 0000000000000000 36 OBJECT GLOBAL HIDDEN 3 note_depends 21: 0000000000000000 36 OBJECT GLOBAL HIDDEN 6 note_depends See patch titled: "livepatch/arm/x86: Strip note_depends symbol from test-cases." which fixes this. Signed-off-by: Konrad Rzeszutek Wilk --- v2: First version --- docs/misc/livepatch.markdown | 2 ++ xen/test/livepatch/Makefile | 56 +++++++++++++++------------------- xen/test/livepatch/xen_bye_world.c | 1 + xen/test/livepatch/xen_hello_world.c | 1 + xen/test/livepatch/xen_nop.c | 1 + xen/test/livepatch/xen_replace_world.c | 1 + 6 files changed, 31 insertions(+), 31 deletions(-) diff --git a/docs/misc/livepatch.markdown b/docs/misc/livepatch.markdown index 505dc37cda..922a64436f 100644 --- a/docs/misc/livepatch.markdown +++ b/docs/misc/livepatch.markdown @@ -430,6 +430,8 @@ checksum, MD5 checksum or any unique value. The size of these structures varies with the --build-id linker option. +On ARM32 this section must by four-byte aligned. + ## Hypercalls We will employ the sub operations of the system management hypercall (sysctl). diff --git a/xen/test/livepatch/Makefile b/xen/test/livepatch/Makefile index 6831383db1..89ad89dfd5 100644 --- a/xen/test/livepatch/Makefile +++ b/xen/test/livepatch/Makefile @@ -1,15 +1,7 @@ include $(XEN_ROOT)/Config.mk -ifeq ($(XEN_TARGET_ARCH),x86_64) -OBJCOPY_MAGIC := -I binary -O elf64-x86-64 -B i386:x86-64 -endif -ifeq ($(XEN_TARGET_ARCH),arm64) -OBJCOPY_MAGIC := -I binary -O elf64-littleaarch64 -B aarch64 -endif -ifeq ($(XEN_TARGET_ARCH),arm32) -OBJCOPY_MAGIC := -I binary -O elf32-littlearm -B arm -endif - +NOTE_SYMBOL = "note_depends" +NOTE_DEPENDS = "const __section(\".livepatch.depends\") $(NOTE_SYMBOL)" CODE_ADDR=$(shell nm --defined $(1) | grep $(2) | awk '{print "0x"$$1}') CODE_SZ=$(shell nm --defined -S $(1) | grep $(2) | awk '{ print "0x"$$2}') @@ -38,7 +30,7 @@ uninstall: .PHONY: clean clean:: - rm -f *.o .*.o.d *.livepatch config.h + rm -f *.o .*.o.d *.livepatch config.h livepatch_depends.h hello_world_livepatch_depends.h *.bin # # To compute these values we need the binary files: xen-syms @@ -56,10 +48,10 @@ config.h: xen_hello_world_func.o echo "#define MINOR_VERSION_ADDR $(MINOR_VERSION_ADDR)"; \ echo "#define OLD_CODE_SZ $(OLD_CODE_SZ)") > $@ -xen_hello_world.o: config.h +xen_hello_world.o: config.h livepatch_depends.h .PHONY: $(LIVEPATCH) -$(LIVEPATCH): xen_hello_world_func.o xen_hello_world.o note.o +$(LIVEPATCH): xen_hello_world_func.o xen_hello_world.o $(LD) $(LDFLAGS) $(build_id_linker) -r -o $(LIVEPATCH) $^ # @@ -71,40 +63,42 @@ $(LIVEPATCH): xen_hello_world_func.o xen_hello_world.o note.o # not be built (it is for EFI builds), and that we do not have # the note.o.bin to muck with (as it gets deleted) # -.PHONY: note.o -note.o: - $(OBJCOPY) -O binary --only-section=.note.gnu.build-id $(BASEDIR)/xen-syms $@.bin - $(OBJCOPY) $(OBJCOPY_MAGIC) \ - --rename-section=.data=.livepatch.depends,alloc,load,readonly,data,contents -S $@.bin $@ - rm -f $@.bin +.PHONY: note.bin +note.bin: + $(OBJCOPY) -O binary --only-section=.note.gnu.build-id $(BASEDIR)/xen-syms $@ + +.PHONY: livepatch_depends.h +livepatch_depends.h: note.bin + $(shell (../../../tools/firmware/hvmloader/mkhex $(NOTE_DEPENDS) $^ > $@)) # # Extract the build-id of the xen_hello_world.livepatch # (which xen_bye_world will depend on). # -.PHONY: hello_world_note.o -hello_world_note.o: $(LIVEPATCH) - $(OBJCOPY) -O binary --only-section=.note.gnu.build-id $(LIVEPATCH) $@.bin - $(OBJCOPY) $(OBJCOPY_MAGIC) \ - --rename-section=.data=.livepatch.depends,alloc,load,readonly,data,contents -S $@.bin $@ - rm -f $@.bin +.PHONY: hello_world_note.bin +hello_world_note.bin: $(LIVEPATCH) + $(OBJCOPY) -O binary --only-section=.note.gnu.build-id $(LIVEPATCH) $@ + +.PHONY: hello_world_livepatch_depends.h +hello_world_livepatch_depends.h: hello_world_note.bin + $(shell (../../../tools/firmware/hvmloader/mkhex $(NOTE_DEPENDS) $^ > $@)) -xen_bye_world.o: config.h +xen_bye_world.o: config.h hello_world_livepatch_depends.h .PHONY: $(LIVEPATCH_BYE) -$(LIVEPATCH_BYE): xen_bye_world_func.o xen_bye_world.o hello_world_note.o +$(LIVEPATCH_BYE): xen_bye_world_func.o xen_bye_world.o $(LD) $(LDFLAGS) $(build_id_linker) -r -o $(LIVEPATCH_BYE) $^ -xen_replace_world.o: config.h +xen_replace_world.o: config.h livepatch_depends.h .PHONY: $(LIVEPATCH_REPLACE) -$(LIVEPATCH_REPLACE): xen_replace_world_func.o xen_replace_world.o note.o +$(LIVEPATCH_REPLACE): xen_replace_world_func.o xen_replace_world.o $(LD) $(LDFLAGS) $(build_id_linker) -r -o $(LIVEPATCH_REPLACE) $^ -xen_nop.o: config.h +xen_nop.o: config.h livepatch_depends.h .PHONY: $(LIVEPATCH_NOP) -$(LIVEPATCH_NOP): xen_nop.o note.o +$(LIVEPATCH_NOP): xen_nop.o $(LD) $(LDFLAGS) $(build_id_linker) -r -o $(LIVEPATCH_NOP) $^ .PHONY: livepatch diff --git a/xen/test/livepatch/xen_bye_world.c b/xen/test/livepatch/xen_bye_world.c index 2700f0eedd..935e76ca8b 100644 --- a/xen/test/livepatch/xen_bye_world.c +++ b/xen/test/livepatch/xen_bye_world.c @@ -10,6 +10,7 @@ #include #include +#include "hello_world_livepatch_depends.h" static const char bye_world_patch_this_fnc[] = "xen_extra_version"; extern const char *xen_bye_world(void); diff --git a/xen/test/livepatch/xen_hello_world.c b/xen/test/livepatch/xen_hello_world.c index 02f3f85dc0..988a3b14f4 100644 --- a/xen/test/livepatch/xen_hello_world.c +++ b/xen/test/livepatch/xen_hello_world.c @@ -11,6 +11,7 @@ #include #include +#include "livepatch_depends.h" static const char hello_world_patch_this_fnc[] = "xen_extra_version"; extern const char *xen_hello_world(void); diff --git a/xen/test/livepatch/xen_nop.c b/xen/test/livepatch/xen_nop.c index a224b7c670..8d0c8f5097 100644 --- a/xen/test/livepatch/xen_nop.c +++ b/xen/test/livepatch/xen_nop.c @@ -7,6 +7,7 @@ #include #include +#include "livepatch_depends.h" /* * All of the .new_size and .old_addr are based on assumptions that the diff --git a/xen/test/livepatch/xen_replace_world.c b/xen/test/livepatch/xen_replace_world.c index 78a8f528b3..a653cc4268 100644 --- a/xen/test/livepatch/xen_replace_world.c +++ b/xen/test/livepatch/xen_replace_world.c @@ -9,6 +9,7 @@ #include #include +#include "livepatch_depends.h" static const char xen_replace_world_name[] = "xen_extra_version"; extern const char *xen_replace_world(void);