{"id":808517,"url":"http://patchwork.ozlabs.org/api/patches/808517/?format=json","web_url":"http://patchwork.ozlabs.org/project/linuxppc-dev/patch/1504232234-4161-1-git-send-email-wei.guo.simon@gmail.com/","project":{"id":2,"url":"http://patchwork.ozlabs.org/api/projects/2/?format=json","name":"Linux PPC development","link_name":"linuxppc-dev","list_id":"linuxppc-dev.lists.ozlabs.org","list_email":"linuxppc-dev@lists.ozlabs.org","web_url":"https://github.com/linuxppc/wiki/wiki","scm_url":"https://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux.git","webscm_url":"https://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux.git/","list_archive_url":"https://lore.kernel.org/linuxppc-dev/","list_archive_url_format":"https://lore.kernel.org/linuxppc-dev/{}/","commit_url_format":"https://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux.git/commit/?id={}"},"msgid":"<1504232234-4161-1-git-send-email-wei.guo.simon@gmail.com>","list_archive_url":"https://lore.kernel.org/linuxppc-dev/1504232234-4161-1-git-send-email-wei.guo.simon@gmail.com/","date":"2017-09-01T02:17:14","name":"selftests/powerpc: fix build error in powerpc ptrace selftests.","commit_ref":"f36dbfe1a504b85c7b3bf89fdd99991afbaa0f74","pull_url":null,"state":"accepted","archived":false,"hash":"fda260f88f8f51ccfe0b57525a50f2a0ccca6cd9","submitter":{"id":68632,"url":"http://patchwork.ozlabs.org/api/people/68632/?format=json","name":"Simon Guo","email":"wei.guo.simon@gmail.com"},"delegate":null,"mbox":"http://patchwork.ozlabs.org/project/linuxppc-dev/patch/1504232234-4161-1-git-send-email-wei.guo.simon@gmail.com/mbox/","series":[{"id":942,"url":"http://patchwork.ozlabs.org/api/series/942/?format=json","web_url":"http://patchwork.ozlabs.org/project/linuxppc-dev/list/?series=942","date":"2017-09-01T02:17:14","name":"selftests/powerpc: fix build error in powerpc ptrace selftests.","version":1,"mbox":"http://patchwork.ozlabs.org/series/942/mbox/"}],"comments":"http://patchwork.ozlabs.org/api/patches/808517/comments/","check":"pending","checks":"http://patchwork.ozlabs.org/api/patches/808517/checks/","tags":{},"related":[],"headers":{"Return-Path":"<linuxppc-dev-bounces+patchwork-incoming=ozlabs.org@lists.ozlabs.org>","X-Original-To":["patchwork-incoming@ozlabs.org","linuxppc-dev@lists.ozlabs.org"],"Delivered-To":["patchwork-incoming@ozlabs.org","linuxppc-dev@lists.ozlabs.org"],"Received":["from lists.ozlabs.org (lists.ozlabs.org [IPv6:2401:3900:2:1::3])\n\t(using TLSv1.2 with cipher ADH-AES256-GCM-SHA384 (256/256 bits))\n\t(No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3xk2x05qLHz9sPm\n\tfor <patchwork-incoming@ozlabs.org>;\n\tFri,  1 Sep 2017 12:19:00 +1000 (AEST)","from lists.ozlabs.org (lists.ozlabs.org [IPv6:2401:3900:2:1::3])\n\tby lists.ozlabs.org (Postfix) with ESMTP id 3xk2x04GcczDql1\n\tfor <patchwork-incoming@ozlabs.org>;\n\tFri,  1 Sep 2017 12:19:00 +1000 (AEST)","from mail-pf0-x244.google.com (mail-pf0-x244.google.com\n\t[IPv6:2607:f8b0:400e:c00::244])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128\n\tbits)) (No client certificate requested)\n\tby lists.ozlabs.org (Postfix) with ESMTPS id 3xk2vB4ZXjzDqfx\n\tfor <linuxppc-dev@lists.ozlabs.org>;\n\tFri,  1 Sep 2017 12:17:26 +1000 (AEST)","by mail-pf0-x244.google.com with SMTP id r187so798242pfr.5\n\tfor <linuxppc-dev@lists.ozlabs.org>;\n\tThu, 31 Aug 2017 19:17:26 -0700 (PDT)","from simonLocalRHEL7.x64 ([106.38.0.83])\n\tby smtp.gmail.com with ESMTPSA id\n\th185sm1069909pfe.160.2017.08.31.19.17.22\n\t(version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128);\n\tThu, 31 Aug 2017 19:17:23 -0700 (PDT)"],"Authentication-Results":["ozlabs.org;\n\tdkim=fail reason=\"signature verification failed\" (2048-bit key;\n\tunprotected) header.d=gmail.com header.i=@gmail.com\n\theader.b=\"FPMcSVqp\"; dkim-atps=neutral","lists.ozlabs.org;\n\tdkim=fail reason=\"signature verification failed\" (2048-bit key;\n\tunprotected) header.d=gmail.com header.i=@gmail.com\n\theader.b=\"FPMcSVqp\"; dkim-atps=neutral","lists.ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=gmail.com header.i=@gmail.com\n\theader.b=\"FPMcSVqp\"; dkim-atps=neutral"],"DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025;\n\th=from:to:cc:subject:date:message-id:mime-version\n\t:content-transfer-encoding;\n\tbh=raru91LRV/k/F0hwSQzJKP5l+a18Kl9bR8GEuwIwvmQ=;\n\tb=FPMcSVqpk4pD9K1kmrkHNd9YxUmxkBq7eN55/Jr63JDQ1vrnuSwjTVXzns6lym/QT0\n\tfd9yAWiP/KKk6/WZTnkUOahkcZqq+i+h+q9Ld186iwpgltWMhmkekKSosJgetgRuCBoL\n\t3vDcsxCifqYug3XvOPnyIYZvlrO4KPK1pclwvbuXUehJy4jfEifKmofhRBtIPCBeGwwL\n\tO0rxAGfwPIjAKXnGzdebjcwBdzqnUVfIH/hyqPLD+T8orI14/AjNJ9RrC8Lv2wIYXzEf\n\tMe9VA1xt9Z/UN9MF8e7quHkrS53MxMYSEOBD3VAjddIr0dIeLydZg/rOKpANYRWzpn5d\n\t7CRg==","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:from:to:cc:subject:date:message-id:mime-version\n\t:content-transfer-encoding;\n\tbh=raru91LRV/k/F0hwSQzJKP5l+a18Kl9bR8GEuwIwvmQ=;\n\tb=JpyZHyckjhGcaQmmqhCF3RlyeKRE8HjpsCr71HskUtefdluzMG3hT97XDj82J5Yzdf\n\tvzcNYYqpYBHtOmRkfYskzVcDAvUVoYwCVjSl8g67yR/2VElQ7fd73Yc3WRc3WC7+14rM\n\tQyjzlP83BFDVPjRL4tvsptaxaVRsO1y+mbzkN06KysB8P8bbo8TOLm0OKcxzNIMfutWz\n\tbSRztVmarkp0LnxdPZaOa5MBg1TvN4AmQJvh0x8ko9Ebszlq5E3mZEzDVLxNh9WUOfNS\n\t2GR30g92waXLw+A9lWOFvlJRsRSYIa4IynOb6/ySayT0pbChfGFwnp8liA+B9C2hZb6m\n\txVZA==","X-Gm-Message-State":"AHPjjUhiYjCnudvXNniwNLA3MbrdoQk61zzPmiE84CquQxoru2C4ssKs\n\tqm5Nvc5w4TR5/A==","X-Google-Smtp-Source":"ADKCNb5x5lzPqXJRgiCx2INkgrW2isSrt/8IftOlkMyqb3YRsyS37xFFEDiT2L8gDu35JFsFomwLBA==","X-Received":"by 10.84.175.195 with SMTP id t61mr594586plb.138.1504232244550; \n\tThu, 31 Aug 2017 19:17:24 -0700 (PDT)","From":"wei.guo.simon@gmail.com","To":"Anshuman Khandual <khandual@linux.vnet.ibm.com>","Subject":"[PATCH] selftests/powerpc: fix build error in powerpc ptrace\n\tselftests.","Date":"Fri,  1 Sep 2017 10:17:14 +0800","Message-Id":"<1504232234-4161-1-git-send-email-wei.guo.simon@gmail.com>","X-Mailer":"git-send-email 1.8.3.1","MIME-Version":"1.0","Content-Type":"text/plain; charset=UTF-8","Content-Transfer-Encoding":"8bit","X-BeenThere":"linuxppc-dev@lists.ozlabs.org","X-Mailman-Version":"2.1.23","Precedence":"list","List-Id":"Linux on PowerPC Developers Mail List\n\t<linuxppc-dev.lists.ozlabs.org>","List-Unsubscribe":"<https://lists.ozlabs.org/options/linuxppc-dev>,\n\t<mailto:linuxppc-dev-request@lists.ozlabs.org?subject=unsubscribe>","List-Archive":"<http://lists.ozlabs.org/pipermail/linuxppc-dev/>","List-Post":"<mailto:linuxppc-dev@lists.ozlabs.org>","List-Help":"<mailto:linuxppc-dev-request@lists.ozlabs.org?subject=help>","List-Subscribe":"<https://lists.ozlabs.org/listinfo/linuxppc-dev>,\n\t<mailto:linuxppc-dev-request@lists.ozlabs.org?subject=subscribe>","Cc":"linuxppc-dev@lists.ozlabs.org, Simon Guo <wei.guo.simon@gmail.com>","Errors-To":"linuxppc-dev-bounces+patchwork-incoming=ozlabs.org@lists.ozlabs.org","Sender":"\"Linuxppc-dev\"\n\t<linuxppc-dev-bounces+patchwork-incoming=ozlabs.org@lists.ozlabs.org>"},"content":"From: Simon Guo <wei.guo.simon@gmail.com>\n\nGCC 7 will take \"r2\" in clobber list as an error will it will get following\nbuild errors for powerpc ptrace selftests even with -fno-pic option:\n  ptrace-tm-vsx.c: In function ‘tm_vsx’:\n  ptrace-tm-vsx.c:42:2: error: PIC register clobbered by ‘r2’ in ‘asm’\n    asm __volatile__(\n    ^~~\n  make[1]: *** [ptrace-tm-vsx] Error 1\n  ptrace-tm-spd-vsx.c: In function ‘tm_spd_vsx’:\n  ptrace-tm-spd-vsx.c:55:2: error: PIC register clobbered by ‘r2’ in ‘asm’\n    asm __volatile__(\n    ^~~\n  make[1]: *** [ptrace-tm-spd-vsx] Error 1\n  ptrace-tm-spr.c: In function ‘tm_spr’:\n  ptrace-tm-spr.c:46:2: error: PIC register clobbered by ‘r2’ in ‘asm’\n    asm __volatile__(\n    ^~~\n\nThis patch fix the build error by removing \"r2\" out of clobber list.\n\nReported-by: Seth Forshee <seth.forshee@canonical.com>\nSigned-off-by: Simon Guo <wei.guo.simon@gmail.com>\nTested-by: Seth Forshee <seth.forshee@canonical.com>\n---\n tools/testing/selftests/powerpc/ptrace/Makefile            | 2 +-\n tools/testing/selftests/powerpc/ptrace/ptrace-tm-spd-vsx.c | 4 ++--\n tools/testing/selftests/powerpc/ptrace/ptrace-tm-spr.c     | 3 +--\n tools/testing/selftests/powerpc/ptrace/ptrace-tm-vsx.c     | 2 +-\n 4 files changed, 5 insertions(+), 6 deletions(-)","diff":"diff --git a/tools/testing/selftests/powerpc/ptrace/Makefile b/tools/testing/selftests/powerpc/ptrace/Makefile\nindex fe6bc60..c836f6d 100644\n--- a/tools/testing/selftests/powerpc/ptrace/Makefile\n+++ b/tools/testing/selftests/powerpc/ptrace/Makefile\n@@ -6,7 +6,7 @@ include ../../lib.mk\n \n all: $(TEST_PROGS)\n \n-CFLAGS += -m64 -I../../../../../usr/include -I../tm -mhtm\n+CFLAGS += -m64 -I../../../../../usr/include -I../tm -mhtm -fno-pic\n \n $(TEST_PROGS): ../harness.c ../utils.c ../lib/reg.S ptrace.h\n \ndiff --git a/tools/testing/selftests/powerpc/ptrace/ptrace-tm-spd-vsx.c b/tools/testing/selftests/powerpc/ptrace/ptrace-tm-spd-vsx.c\nindex 0df3c23..277dade 100644\n--- a/tools/testing/selftests/powerpc/ptrace/ptrace-tm-spd-vsx.c\n+++ b/tools/testing/selftests/powerpc/ptrace/ptrace-tm-spd-vsx.c\n@@ -79,8 +79,8 @@ void tm_spd_vsx(void)\n \t\t: [res] \"=r\" (result), [texasr] \"=r\" (texasr)\n \t\t: [fp_load] \"r\" (fp_load), [fp_load_ckpt] \"r\" (fp_load_ckpt),\n \t\t[sprn_texasr] \"i\"  (SPRN_TEXASR)\n-\t\t: \"memory\", \"r0\", \"r1\", \"r2\", \"r3\", \"r4\",\n-\t\t\"r8\", \"r9\", \"r10\", \"r11\"\n+\t\t: \"memory\", \"r0\", \"r1\", \"r3\", \"r4\",\n+\t\t\"r7\", \"r8\", \"r9\", \"r10\", \"r11\"\n \t\t);\n \n \tif (result) {\ndiff --git a/tools/testing/selftests/powerpc/ptrace/ptrace-tm-spr.c b/tools/testing/selftests/powerpc/ptrace/ptrace-tm-spr.c\nindex 94e57cb..51427a2 100644\n--- a/tools/testing/selftests/powerpc/ptrace/ptrace-tm-spr.c\n+++ b/tools/testing/selftests/powerpc/ptrace/ptrace-tm-spr.c\n@@ -76,8 +76,7 @@ void tm_spr(void)\n \t\t: [tfhar] \"=r\" (tfhar), [res] \"=r\" (result),\n \t\t[texasr] \"=r\" (texasr), [cptr1] \"=r\" (cptr1)\n \t\t: [sprn_texasr] \"i\"  (SPRN_TEXASR)\n-\t\t: \"memory\", \"r0\", \"r1\", \"r2\", \"r3\", \"r4\",\n-\t\t\"r8\", \"r9\", \"r10\", \"r11\", \"r31\"\n+\t\t: \"memory\", \"r0\", \"r8\", \"r31\"\n \t\t);\n \n \t/* There are 2 32bit instructions before tbegin. */\ndiff --git a/tools/testing/selftests/powerpc/ptrace/ptrace-tm-vsx.c b/tools/testing/selftests/powerpc/ptrace/ptrace-tm-vsx.c\nindex b4081e2..17c23ca 100644\n--- a/tools/testing/selftests/powerpc/ptrace/ptrace-tm-vsx.c\n+++ b/tools/testing/selftests/powerpc/ptrace/ptrace-tm-vsx.c\n@@ -67,7 +67,7 @@ void tm_vsx(void)\n \t\t: [res] \"=r\" (result), [texasr] \"=r\" (texasr)\n \t\t: [fp_load] \"r\" (fp_load), [fp_load_ckpt] \"r\" (fp_load_ckpt),\n \t\t[sprn_texasr] \"i\"  (SPRN_TEXASR), [cptr1] \"r\" (&cptr[1])\n-\t\t: \"memory\", \"r0\", \"r1\", \"r2\", \"r3\", \"r4\",\n+\t\t: \"memory\", \"r0\", \"r1\", \"r3\", \"r4\",\n \t\t\"r7\", \"r8\", \"r9\", \"r10\", \"r11\"\n \t\t);\n \n","prefixes":[]}