From patchwork Mon Mar 27 08:40:50 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Wei Chen X-Patchwork-Id: 9645601 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 5E504602D6 for ; Mon, 27 Mar 2017 08:44:10 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 54F4E28307 for ; Mon, 27 Mar 2017 08:44:10 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 4982728334; Mon, 27 Mar 2017 08:44:10 +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=-4.1 required=2.0 tests=BAYES_00,DKIM_SIGNED, RCVD_IN_DNSWL_MED,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 915BC28307 for ; Mon, 27 Mar 2017 08:44:09 +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 1csQDH-00034i-5s; Mon, 27 Mar 2017 08:41:19 +0000 Received: from mail6.bemta6.messagelabs.com ([193.109.254.103]) by lists.xenproject.org with esmtp (Exim 4.84_2) (envelope-from ) id 1csQDG-00034c-7b for xen-devel@lists.xen.org; Mon, 27 Mar 2017 08:41:18 +0000 Received: from [193.109.254.147] by server-9.bemta-6.messagelabs.com id 18/0C-14382-D20D8D85; Mon, 27 Mar 2017 08:41:17 +0000 X-Brightmail-Tracker: H4sIAAAAAAAAA1VSa0hTYRj2OzvbjuKx4+blVbRoEoU0c2GpFFh /oj+J0B+Jos7stC22Oc6ZYUTkRKUszTSTVLxToSFkDAMveSEtb6lBaCp4w2umoZU4lc7ZmWbf r+f9nud53+f7eAmJwiYLJJhkK8OaaaNK5oHrj2Gx6qP9Q/HhtTWHo6p+VuJn0PkPDQ4sDl2SG szaxORrUn1d63vc0hiRnF5fgFLQfGgm8iBwKkUCzY/HUSZyJxRUPgbLs1ECoaA6MBjueoMLBa IGEdhbq5DINCOwzX6UiEUOBnmdGZhYlCIodPRJhWYy6iBkPbHhAvahgqEms1guiCRUCYKn26+ cIiWVABuTJRIB49Qh+J3b7cQkFQOlmb8wAQO1H4b7HrjuveHT82lnUwkF0Do3JxGTB0PBaC8S 9QHwdbifH0bwOAS60mKFuUCV4NAyPY+JRZkUbI63ctFwAWZe213DDFBb0yQTRU1yqE1rcjlmc KhYnHQ6vCgF1I83uhxG2NjelIr4FAw8K5WJuIf/wI59otkug7GKbJchCB6upstFIlsOqdVdKA eFFu55X+Ge95UhrBod4Rj2FsOqNdFhWtag01tNtMGo1oSfDDMxHEfrGCOt5cISEk11iF8HN/6 8Q+sv4tpQAIGpfMmV7qF4hZc28fptPc3pr7JJRoZrQ0EEoQLS0cdz3iyjY5JvGIz8Tu3QQHiq fEjyM0+TnIU2cQadSHUhNVFc1PAdU+DmRDMT6E9GCyJKEOmTzLstdjZzEAUHKknEh1J4WhjWZ LD+zy8gfwKplGStkMTTYLbuTlrgQ2B8iPZSZwgr/Y8KTEHZVd6LqyMxW7MO3YyWvaLRZM9NjH RGBqvvWdx69HlbESXtlS3LJyYSfNeUB+zHO4rKHWNBZx8pY76cNr8sTymq+FHvF3l5LeuPW2z S0lav5mbZ5uhU+zdLXLs9pDNtYHbKlr9yf2ZjeSzyjuain3dRRmTqXdxdvTSQy1VZ086tq3BO T2tCJSxH/wU+0QZklAMAAA== X-Env-Sender: Wei.Chen@arm.com X-Msg-Ref: server-8.tower-27.messagelabs.com!1490604076!83563795!1 X-Originating-IP: [104.47.2.78] X-SpamReason: No, hits=0.5 required=7.0 tests=BODY_RANDOM_LONG X-StarScan-Received: X-StarScan-Version: 9.2.3; banners=-,-,- X-VirusChecked: Checked Received: (qmail 17551 invoked from network); 27 Mar 2017 08:41:16 -0000 Received: from mail-db5eur01on0078.outbound.protection.outlook.com (HELO EUR01-DB5-obe.outbound.protection.outlook.com) (104.47.2.78) by server-8.tower-27.messagelabs.com with AES256-SHA256 encrypted SMTP; 27 Mar 2017 08:41:16 -0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=armh.onmicrosoft.com; s=selector1-arm-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version; bh=VetOF/ZSZY/JoIyowJBO3fGI+cKZg6aksZdKCKAOxNg=; b=dFfD6+yNPFAcKllDPl+2Nu+B1FoCBQO3bB+jlfG3UmJPInI7vXweh36hK+8Kz3GcYVrKmr5J7DurKoTjiALgdfslDQdKNiH+774nY1yQP0OfG71k9Cciqhaq4p4XM+n8sf/IDZIuYqR3siwD+P/vZRW/qn6xk31qP6xmtaZBaVE= Received: from VI1PR0801CA0064.eurprd08.prod.outlook.com (10.168.60.160) by DB4PR08MB0222.eurprd08.prod.outlook.com (10.161.250.28) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P384) id 15.1.977.11; Mon, 27 Mar 2017 08:41:15 +0000 Received: from AM5EUR03FT048.eop-EUR03.prod.protection.outlook.com (2a01:111:f400:7e08::207) by VI1PR0801CA0064.outlook.office365.com (2603:10a6:800:4d::32) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.991.14 via Frontend Transport; Mon, 27 Mar 2017 08:41:14 +0000 Received-SPF: Pass (protection.outlook.com: domain of arm.com designates 217.140.96.140 as permitted sender) receiver=protection.outlook.com; client-ip=217.140.96.140; helo=nebula.arm.com; Received: from nebula.arm.com (217.140.96.140) by AM5EUR03FT048.mail.protection.outlook.com (10.152.17.177) with Microsoft SMTP Server (version=TLS1_0, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA_P384) id 15.1.977.7 via Frontend Transport; Mon, 27 Mar 2017 08:41:14 +0000 Received: from P300.p300.shanghai.arm.com (10.1.2.79) by mail.arm.com (10.1.105.66) with Microsoft SMTP Server id 14.3.294.0; Mon, 27 Mar 2017 09:40:53 +0100 From: Wei Chen To: Date: Mon, 27 Mar 2017 16:40:50 +0800 Message-ID: <1490604050-5955-1-git-send-email-Wei.Chen@arm.com> X-Mailer: git-send-email 2.7.4 MIME-Version: 1.0 X-EOPAttributedMessage: 0 X-MS-Office365-Filtering-HT: Tenant X-Forefront-Antispam-Report: CIP:217.140.96.140; IPV:CAL; SCL:-1; CTRY:GB; EFV:NLI; SFV:NSPM; SFS:(10009020)(6009001)(39840400002)(39450400003)(39410400002)(2980300002)(438002)(199003)(189002)(4326008)(5660300001)(47776003)(36756003)(50226002)(2351001)(2906002)(356003)(54906002)(6306002)(48376002)(77096006)(50466002)(189998001)(305945005)(8676002)(104016004)(38730400002)(110136004)(106466001)(86362001)(50986999)(5003940100001)(8936002)(6916009); DIR:OUT; SFP:1101; SCL:1; SRVR:DB4PR08MB0222; H:nebula.arm.com; FPR:; SPF:Pass; MLV:sfv; A:1; MX:1; LANG:en; X-Microsoft-Exchange-Diagnostics: 1; AM5EUR03FT048; 1:X3glE9eBXNSCw2e/BLrFsoyN+A35GrmlHjvE1tUYVFS2TDmNHff+sIavdI+rwX6CGjc/2L9qgQZtHRaAOhf6cOTqgERdwlmM5PfycGJWSHmSVJCgvOPVEPMDUm9jm/77bpxpcFuOSeKZdC7uJykBtsewWOugmT0yRUNVdm7rojpyNflRYQA2/ju8ivrPsIDz2uwv+5Ke/Zm5Q715MRp4RuT0mpAIPaO9KrjDYtT6tJ732yx5EJ8BIUJMPq2rQYM53zVxCDyfqzFyg5JDjm3YwFKbzWB/W03vbd+1D5rGeBPPkcRVTIN49BNv61FfstavaxlDZVkjyspIZOuwJu4bGuhjxLSYjOzKUspP++4ALWes5sB2Zq94z0cvNne4aeJp/FGDrzkzcLC92j8xUgNWlNRWLIQqgvSmJ3aTvEQ1KOrv5NBt0B6iLCioCGs+/Uf8Ias8GQHWaRF53bvDgZD1BcHktiC0xE6hZ7syihQAMq4TEY7Z/7lij2AEjzRlccx+6iBi3OO6SqTtjIcTgfhcSa5iq7gqqoa37RLaQ7loZgxdol6qK/i757mpRYaQgqmaCyuZ5tAWIQhnP0JL5JBF1383t/RY+5TZPdQNv9YYN6Q= X-MS-Office365-Filtering-Correlation-Id: b0ca0ef0-d118-4b5f-eed8-08d474ed0749 X-Microsoft-Antispam: UriScan:; BCL:0; PCL:0; RULEID:(22001)(8251501002)(2017030254075); SRVR:DB4PR08MB0222; X-Microsoft-Exchange-Diagnostics: 1; DB4PR08MB0222; 3:4y+WDWIZWyD2caL/5c4WriNB8KATaq9K+saBh8uH/bvilakp90C6gfSgYFjoK/Zlz9752f1r6fyu3ohjG4HptNGfRg3pg1QvyZxL3rafbFwBWQAbN08maZ3i830cHmbPyaVymLIe/UVVT0WVVdDyRp/0yAhIsKwiExwTD3fjCRcrCtYgYtJFHx+j+XOx9U51ktWa5rBjgqHO0+zHn++W/ksbQmgONhl9JhXVV7+rgZv47FcC/MYIME0b6a5XF6qlZweZkI21FzUPtacmYHBEYsXh8yaZZtwcvzE9y+YvCHqeVdhOq63QaCI/b+npX5y12BiwhwxvMThLFS9ULuRr+twKIGEvPjzgzCHwz0GRceocJ9wlfAoAGw1bFDQ/i00UeTWrfYVFucuQ/GJs+nLRHEC5nRmHTEAlrrYR5zK0vZ0=; 25:JEi2a4zLAN/egmyQoI1QgQq3yAMDz49CNzw2WFN34cGwt25rmltoYnZ//FG6sD19e8d24FrpeeK5qa3flB6f8Ke71sSqFn1tkdvHjLCSmzjBv436lgL/Ez6an6z1mPIMJDjIRvC0D0iRP6z0IuaFFINv/kuon7QI49r5zz7a9kmTjU8zmJHR6UqtjQYtZzW0mrd91D651PcTaadLSOF0yOFKZ1pssg6Az+vYtXv9QcoDL1BR1iEIvKzSQn1+U2A9YLkP58/u++01hiS9epJdOh2/hTW+0OtiphDQubc0QVfD1FccXNdwttatzORXNhRG/i4qN6mclneqJDIlxSi9/ZeStjeWclbr4zcV6X8lA+kMwXtPvv7vsfkO8KD8pT08luVLt+gE9Alq+DiTeQIbg5k7feGRGHGksVtG34DAucLEYrgfXyDIT0Qpj9md/I9/BkLITVxctTafJEfyLli8ZQ== X-Microsoft-Exchange-Diagnostics: 1; DB4PR08MB0222; 31:bhV+mBABK9jx1os5Zlsd19El4hU5u66yUTMLjLempw0CA6Swm41+hWgQKB9BwlCV1VhLpva1NeABjZP7PG+JWwcONrBzrNbTIwAE9j4buO0+fibGtxA11pJzO7bdhX1AgAsvvlrqLy00nC51nxcg4y6IPofllGUTMqOqz7eTyR4vVURl1/1x/OjK9Q6kq6M4mdT8oYg0nJCs6TNqrrcQZ6i6pkrJgcE6KDvsmNZCEfDDCCJtpjTvB6zaGOa7PqWTrqBb8fEurcIRGuxc1R5ZFdJDwtYwv4FlZsvYErU/4sE=; 20:2Q5NL8RNpBqB1WNg9ehDgI9Ize3lM2KyH40kuqYIcoTttfZF9EvU/LXd9NNHxSdjm/7d+q+GMWfoc8nIwgDzN1AX5yFj5JE2wkjL3KZ5778CkJ6zJUK+fvt8ldcOUYc4sVXDPBYTcx2LL6O/VDyQkE0y82955jmZkAkZZMFk19/KiIvDYixwgZPHdkKqE8jCxum1dTzFxaMRCduQIZWIgedPqwyR7ZvKVjgFr+0Rd9l4mle94WaA98ru/GR1PftV NoDisclaimer: True X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:(180628864354917); X-Exchange-Antispam-Report-CFA-Test: BCL:0; PCL:0; RULEID:(6040375)(601004)(2401047)(8121501046)(5005006)(13013025)(13024025)(13023025)(13020025)(10201501046)(3002001)(6055026)(6041248)(20161123562025)(20161123555025)(20161123560025)(20161123558025)(20161123564025)(6072148); SRVR:DB4PR08MB0222; BCL:0; PCL:0; RULEID:; SRVR:DB4PR08MB0222; X-Microsoft-Exchange-Diagnostics: 1; DB4PR08MB0222; 4:fYVSCmJGUOFlTrEGVr+cqQyECuGZIPIFu2k1vbbA1uSON7weCKBb3pAARgSKuZs6s6Zk3YK17PwpMi6zU1lESfnuCV0HTPRODXSNCEhMMtiA5r/+RZlffGFdKjQo/Hxvk01SUUseywt9YFLP3Y0fECa3d1JjnOwEvm/pVGNIvP/lbZjOdo4hX/wKJG/xD2SN0X9VQtCR+oF5MWdrrf+El4Ewm6pUXcPXdPjvzP7F7zFfmmstvAAaaDjY5i6x4mDc/Wy6kubXYL17YObhbANGS9SVzfwIzMmBoDY5m3YBOzuw3rhlXPfDdhsKZMiQ5Hhfcm6BSOwn8MlooPnBAsBjth74jWSYNbhv5ZHAtzAiOM4fSHQCsA9tIMInIfOwInqo4PFOGwlZM4ZZcbLcgW/dT6g1CL4RGtcKe7HsN0/T4UkhVXDyX6PxMAniDRn7nPkFMJ2kLS/dxnbOGlJEOasCw19Xo9ECqewE9z5ahjEtSjJOVDXq2n8r0bON2roPDWNFVAkObVxS1gasEn2ldyGEzSa3wa9+ocIQNDopr/SabHm/DIyfVUhjwV+55TFWOBPKD7IZIDoow8v9xuud363bRIXJ+z0nmDBeuYW6x43/JgeUJ8oCFLhLfkoNJTUUmNCDs/b+WlLDqUtmePimWB2pwEGPdoGz42d2st0r3t/GUO1v0QiAG1SnnerVwDtlFoTlRhrxGqjugqkhzqO+zmKYSyVd2AHNebmkXmwFVC4hpaJt7Ziboi27xdkd1jRhTFpt X-Forefront-PRVS: 02596AB7DA X-Microsoft-Exchange-Diagnostics: 1; DB4PR08MB0222; 23:YHXN3ws/K1oQQR1CypWCNC50yb5ikjOsl3DNyvwEHpQj6k+5E6/94L+A5zom31msuXptxZyg3zxEv0gPuvy+0793ZihvofPKkvkjvlMHJn+tY/zRDKzwS5yMgR0+0dogdpXD2nshEcJrpS+Q7TlVwViOghHLdIdj7n+fiUMsWSYjRkTp1C98xJWA73vuqkFmR6J1To3JGCZEqbUqpniUBBV4GXJvB9z7edKcbDhEWnHLtGOnOMzLW8mOZJ8sIXDOs+zYSYc/5pdO7fTZepT3pJ65F6qRdlrR/MDKoIb6axmdsL1zv4Dwd0EPCcT3baSOVqQq48RsV7Ktt8B8acQYC6eo+gNV3JUYAOeRVOYyjMFMi4PUhH3qqlc9Ft0rUHSrxQDKICiI7+wp8l9NaqaXSYeV0G7+2/bnHiqI7psByHJraNA9DS6wf7ZTcOBohLnBFHq4Rx0UEZz1BgrrDv+29/2BwwfSmhxx01kCOxZDx7EFbhHDAyJkJJIjO1su+xHjCh+venVrVmkWqcaKw/rV65ERK0y39CIt/LC1mjG6KEiswfsG6eeyPWXLaVc9jVk3Nsa6tUffqKKRIUdBWUbRnkrOobncoyqNzFjF4Y0UpD0X64uLDnd5uYp4cVJ5/H13tu8bUKSHhqpcFmGFv557kSALb87Ft8LHwVSliE7XLJtlt9cmbwEDW4ysMw3g8NcTikAXpVusJ4h7OU8sOowWQyeT8SJCn/+eghJny7s/eeNFvBLpos5QWFmcJUxk0ibgwle0xSqG9fLnHfqULZBwu95SeDOlO07BiXhtGjkue+8bTmZsVtp7FfvLHGCSdByTQzu4bWf+f2PD0kmzfzbA8w== X-Microsoft-Exchange-Diagnostics: 1; DB4PR08MB0222; 6:pcoSb9erl65BdB1S+YGqzly3fKfo5r+OIcMZkGLTo7R66KrisYR4YzNGMAArcdLiFTbR/kH3dz9JTiU0327IXiELEYgxNHnHuWj0zX5yHmUQyQ/nKvKx7tb/EOITgvcsy67o+ZyFqbRi25hFpxAmohyJsl2hbZGGM31wGGPDWNMdAuB/igeQ+PuUBsr/GhwTj5CChr9pk6wRBfbEOSjyrzqzXyixS0TNfAfElpNjKJ9lPfpHBhponTlV+8lN2PAfITuivAlXHX2miseleqcHO9mlbujEjRnLjaLpOKoNyIH5TJWMjFVZd9ITCj/B8y3VHO0egGlBbG35BhdFjVBKIaR+NfAQIh/It2aworBrabO09oXH6mPvnuHGll2wnUTpVS7FEFXOfSNoJDsZGMoxxtiM8dy5d0y2mFC/ZIyVE/8=; 5:HMRCP/v45FbW0jMoVY/2G/BKTjQMGLqszUBAB6LXb+kHbZ/HF+goKUQePpbrgyOfiFqE13anKezVVEajoCD5GFcZMmRreopkXMgKJgiiQFs5oYgiQITKJ+N1mjQ4d2u3eJSqy0E/LQ+UXUvHB8WwFA==; 24:Q3x9SBeHnfSqCKxiC+jAVE9UVzyJvTvmd/c9qjO7fUirvoy1XsS+qcUyWVP+zdeKsBbNEZfljDoMBHEE8vvLUs4JWmvTmwkhO9P5Nwo8G6Y= SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-Microsoft-Exchange-Diagnostics: 1; DB4PR08MB0222; 7:me2WTSQCtp3/nhg4Qb3hDnHB5hjymT1PQZQGJhq1TUi/hoF0SwYDHfh4WLAi2rT53Jz/iX4CoKljgZkC2GRroXgrTdWn6UlyI1Wm/SxiV1uGa2e7xUmuoDab5eAkOfCwIx01/NNvx08aA5wOjyrDAOWutSY8bhKI1K2DFQQWmg5+9lkqiUo/42of/pae4TP0HVXDTl5aOz7FofFwR0W8pFWcHtbL82Ub7PQsd3kZOxEJvPdDPsmVPjo9wWCSovkqfTOWxgi8vhSaaZKZ+kg9DvDlFZafymf9lSujVdojCY61IFOR2TQFPB3VQ+D9iZi0tmpufzqzoTMr+PhubCoRTA== X-OriginatorOrg: arm.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 27 Mar 2017 08:41:14.1939 (UTC) X-MS-Exchange-CrossTenant-Id: f34e5979-57d9-4aaa-ad4d-b122a662184d X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=f34e5979-57d9-4aaa-ad4d-b122a662184d; Ip=[217.140.96.140]; Helo=[nebula.arm.com] X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB4PR08MB0222 Cc: sstabellini@kernel.org, wei.chen@arm.com, steve.capper@arm.com, Kaly.Xin@arm.com, julien.grall@arm.com, nd@arm.com Subject: [Xen-devel] [PATCH v3] xen/arm: alternative: Register re-mapped Xen area as a temporary virtual region 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: , Errors-To: xen-devel-bounces@lists.xen.org Sender: "Xen-devel" X-Virus-Scanned: ClamAV using ClamSMTP While I was using the alternative patching in the SErrors patch series [1]. I used a branch instruction as alternative instruction. ALTERNATIVE("nop", "b skip_check", SKIP_CHECK_PENDING_VSERROR) Unfortunately, I got a system panic message with this code: (XEN) build-id: f64081d86e7e88504b7d00e1486f25751c004e39 (XEN) alternatives: Patching with alt table 100b9480 -> 100b9498 (XEN) Xen BUG at alternative.c:61 (XEN) ----[ Xen-4.9-unstable arm32 debug=y Tainted: C ]---- (XEN) CPU: 0 (XEN) PC: 00252b68 alternative.c#__apply_alternatives+0x128/0x1d4 (XEN) CPSR: 800000da MODE:Hypervisor (XEN) R0: 00000000 R1: 00000000 R2: 100b9490 R3: 100b949c (XEN) R4: eafeff84 R5: 00000000 R6: 100b949c R7: 10079290 (XEN) R8: 100792ac R9: 00000001 R10:100b948c R11:002cfe04 R12:002932c0 (XEN) HYP: SP: 002cfdc4 LR: 00239128 (XEN) (XEN) VTCR_EL2: 80003558 (XEN) VTTBR_EL2: 0000000000000000 (XEN) (XEN) SCTLR_EL2: 30cd187f (XEN) HCR_EL2: 000000000038663f (XEN) TTBR0_EL2: 00000000bff09000 (XEN) (XEN) ESR_EL2: 00000000 (XEN) HPFAR_EL2: 0000000000000000 (XEN) HDFAR: 00000000 (XEN) HIFAR: 00000000 (XEN) (XEN) Xen stack trace from sp=002cfdc4: (XEN) 00000000 00294328 002e0004 00000001 10079290 002cfe14 100b9490 00000000 (XEN) 10010000 10122700 00200000 002cfe1c 00000080 00252c14 00000000 002cfe64 (XEN) 00252dd8 00000007 00000000 000bfe00 100b9480 100b9498 002cfe1c 002cfe1c (XEN) 10010000 10122700 00000000 00000000 00000000 00000000 00000000 00000000 (XEN) 00000000 00000000 00000000 002ddf30 00000000 003113e8 0030f018 002cfe9c (XEN) 00238914 00000002 00000000 00000000 00000000 0028b000 00000002 00293800 (XEN) 00000002 0030f238 00000002 00290640 00000001 002cfea4 002a2840 002cff54 (XEN) 002a65fc 11112131 10011142 00000000 0028d194 00000000 00000000 00000000 (XEN) bdffb000 80000000 00000000 c0000000 00000000 00000002 00000000 c0000000 (XEN) 002b8060 00002000 002b8040 00000000 c0000000 bc000000 00000000 c0000000 (XEN) 00000000 be000000 00000000 00112701 00000000 bff12701 00000000 00000000 (XEN) 00000000 00000000 00000000 00000000 00000018 00000000 00000001 00000000 (XEN) 9fece000 80200000 80000000 00400000 00200550 00000000 00000000 00000000 (XEN) 00000000 00000000 00000000 00000000 00000000 00000000 00000000 00000000 (XEN) 00000000 00000000 00000000 00000000 00000000 00000000 00000000 00000000 (XEN) 00000000 00000000 00000000 00000000 00000000 00000000 00000000 00000000 (XEN) 00000000 00000000 00000000 00000000 00000000 00000000 00000000 00000000 (XEN) 00000000 00000000 00000000 00000000 00000000 00000000 00000000 (XEN) Xen call trace: (XEN) [<00252b68>] alternative.c#__apply_alternatives+0x128/0x1d4 (PC) (XEN) [<00239128>] is_active_kernel_text+0x10/0x28 (LR) (XEN) [<00252dd8>] alternative.c#__apply_alternatives_multi_stop+0x1c4/0x204 (XEN) [<00238914>] stop_machine_run+0x1e8/0x254 (XEN) [<002a2840>] apply_alternatives_all+0x38/0x54 (XEN) [<002a65fc>] start_xen+0xcf4/0xf88 (XEN) [<00200550>] arm32/head.o#paging+0x94/0xd8 (XEN) (XEN) (XEN) **************************************** (XEN) Panic on CPU 0: (XEN) Xen BUG at alternative.c:61 (XEN) **************************************** This panic was triggered by the BUG(); in branch_insn_requires_update. That's because in this case the alternative patching needs to update the offset of the branch instruction. But the new target address of the branch instruction could not pass the check of is_active_kernel_text(); The reason is that: When Xen is booting, it will call apply_alternatives_all to do patching with alternative tables. In this progress, we should update the offset of branch instructions if required. This means we should modify the Xen text section. But Xen text section is marked as read-only and we configure the hardware to not allow a region to be writable and executable at the same time. So we re-map Xen in a temporary area for writing. In this case, the calculation of the new target address of the branch instruction is based on this re-mapped area. The new target address will point to a value in the re-mapped area. But we haven't registered this area as an active kernel text. So the check of is_active_kernel_text will always return false. We have to register the re-mapped Xen area as a virtual region temporarily to solve this problem. 1. https://lists.xenproject.org/archives/html/xen-devel/2017-03/msg01939.html Signed-off-by: Wei Chen Reviewed-by: Konrad Rzeszutek Wilk Reviewed-by: Julien Grall --- Notes: This bug will affect the staging, staging-4.8 and stable-4.8 source trees. --- v2->v3 1. Fix typos. 2. Explain this bug only happening when booting Xen in commit message. --- xen/arch/arm/alternative.c | 18 +++++++++++++++++- 1 file changed, 17 insertions(+), 1 deletion(-) diff --git a/xen/arch/arm/alternative.c b/xen/arch/arm/alternative.c index 1d10f51..96859fc 100644 --- a/xen/arch/arm/alternative.c +++ b/xen/arch/arm/alternative.c @@ -24,6 +24,7 @@ #include #include #include +#include #include #include #include @@ -154,8 +155,12 @@ static int __apply_alternatives_multi_stop(void *unused) int ret; struct alt_region region; mfn_t xen_mfn = _mfn(virt_to_mfn(_start)); - unsigned int xen_order = get_order_from_bytes(_end - _start); + unsigned int xen_size = _end - _start; + unsigned int xen_order = get_order_from_bytes(xen_size); void *xenmap; + struct virtual_region patch_region = { + .list = LIST_HEAD_INIT(patch_region.list), + }; BUG_ON(patched); @@ -169,6 +174,15 @@ static int __apply_alternatives_multi_stop(void *unused) BUG_ON(!xenmap); /* + * If we generate a new branch instruction, the target will be + * calculated in this re-mapped Xen region. So we have to register + * this re-mapped Xen region as a virtual region temporarily. + */ + patch_region.start = xenmap; + patch_region.end = xenmap + xen_size; + register_virtual_region(&patch_region); + + /* * Find the virtual address of the alternative region in the new * mapping. * alt_instr contains relative offset, so the function @@ -182,6 +196,8 @@ static int __apply_alternatives_multi_stop(void *unused) /* The patching is not expected to fail during boot. */ BUG_ON(ret != 0); + unregister_virtual_region(&patch_region); + vunmap(xenmap); /* Barriers provided by the cache flushing */