From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755288AbdBGRTj (ORCPT ); Tue, 7 Feb 2017 12:19:39 -0500 Received: from mail-ve1eur01on0086.outbound.protection.outlook.com ([104.47.1.86]:55917 "EHLO EUR01-VE1-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1754211AbdBGRTh (ORCPT ); Tue, 7 Feb 2017 12:19:37 -0500 Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=Punit.Agrawal@arm.com; Subject: Re: [PATCH] arm64: hwpoison: add VM_FAULT_HWPOISON[_LARGE] handling To: "Baicar, Tyler" References: <1485987318-9145-1-git-send-email-tbaicar@codeaurora.org> <877f57qmeq.fsf@e105922-lin.cambridge.arm.com> CC: , , , , , , , , , From: Punit Agrawal Message-ID: <52890e50-8488-13b0-4517-4ee5f7cd3ee4@arm.com> Date: Tue, 7 Feb 2017 17:19:28 +0000 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Icedove/45.6.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset="windows-1252"; format=flowed X-Originating-IP: [217.140.96.140] X-ClientProxiedBy: HE1PR0202CA0042.eurprd02.prod.outlook.com (10.171.89.156) To AM4PR0802MB2356.eurprd08.prod.outlook.com (10.172.218.140) X-MS-Office365-Filtering-Correlation-Id: bf8344f5-b116-484f-374d-08d44f7d7c49 X-MS-Office365-Filtering-HT: Tenant X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:(22001)(48565401081);SRVR:AM4PR0802MB2356; X-Microsoft-Exchange-Diagnostics: 1;AM4PR0802MB2356;3:wotTkcLrBamlpf3YzDBhuwKfKdrGFBwdErmP/Y4z6hHNRtqMe3PgdIEfG3v+W7pHSOHbFlq4tUnmVoh4ROmsAs8zbn1zMiMNpXKLslxGc9dJ0nVPpM4EITOlVuI78fvDruopDPYIhP93b8fMajchK0+3jqtzcAR0RcFq9zrh62Jcko2L3wlE5Grurd4QO9ADZJWKPwZYmUQPbtCCe9hUKCU0dEQ/K40doetm0lCEanoNWHtu2lv0qpYaNAnrpKMybI4Ta0osNADAU6xvYjCAiXW2JWButj7oNDXGjYXDzE8=;25:RCEgSgruAOguSHVaN+Dna+QqY2QR2kJkM9xoOsxAXMyCOkWYORDzdaN6HQwlqwqXRG8QDBLKXV+qG0hneIvs+7yt8wX3OI9637VcDpS6IwHFK1RKHZPzrXvvb/ey6M+ooW4pnPCh7mNvwSnJr90fhdVRqixicHQLu8QAys07k/CQmwKqBrXPcJRxFFELFrTfqvTg6UpsFV1LlPcUL7Rv37sFqmjkjKCLa0eMSVFPoRe1OkFj+z8DRMiqxBKIGyncvz/2v4P+wH24laDUSHKese0SEkdIDvFe1M46o0AQYofnyHFUaMjMZU8CMckPjXXrCb9J5qZuDfOacStLbo/26n40Fy4eVLXzhAjK9NSOHdrEe1cOZOoRH1zR07I6FJpJeuZmuRHiG5B7L2lqWp75BrPT8wOvUpD+EKvhkxhr/3MEveJ1dlGy83zLkpRc30p5T8H/uzvrID/8v8SPt5sCpQ== X-Microsoft-Exchange-Diagnostics: 1;AM4PR0802MB2356;31:P4wfgiIpfLUnx3EJ4AqmjjpgxaI4GyyScPwUkHpO+hYAKROyZBcpk5k+djwaY96BIrWS9Oc/Gwjsgeb2D4/QMY1sKv7W9VNVoPalWW3+ypMSNpTurFLDQu/5/cM6rouMMoKFx+gQXH/Ck0Y5YeYSpJDZbfW58vlDpXlyHYDXsHSSWBuShUWgmaGFpstSQu9rPpHaruxxCWUAs+Nh5FjCUTFuv0gTuNXuTTXd2PtEDbWTaBKHXYdxN5lPVy0dYSHi;20:8MAaEo9WcfqRa95hV3TbXxNXythlzSkOU5VwUIctWtDApW8tYugnY4hPbuQgpiP6YxfP8vHtilyEjsv4z4c5mypBDe9ELoOxIE64DGZYT3wZx4qlx6QQR29QQ9goG4Dpzo1r9L/E8z3qoIojvuMOU3Rox09IIgXtY4dIjOFHABey5jicm3LTANAQpmjdMwLowUjtGw9zwidaO4faKHgbmWKUbgwdR+P6+Nbu5/Q260ZSCsi7B+G54pmPvuqsZ/tlB+esX4MZlLE/bumF/xPKLqWZyzIH8FTIrJcLzo7YXPXwuyDNBvAvYYTUnIFRgjEccfNq/0O+vLUXEc1dGDaj5HvVdpwDvn3HCF3avwuwcsOg39AQnUJdJtPuCbceGmzmP28465mU1Cjs4sNOFKz+oE+CdYJZPgTDRq2e5QW++W5HIqDE2x57HxXLoG9mswsv8GJIzFGoA6DtQ3XAVD8zv45u1Qm8PVO8kIrH9djMCannqzMdi163pZgtCcXPREJm X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:; X-Exchange-Antispam-Report-CFA-Test: BCL:0;PCL:0;RULEID:(6040375)(601004)(2401047)(2017020603029)(5005006)(8121501046)(20170203043)(3002001)(10201501046)(6055026)(6041248)(20161123555025)(20161123562025)(20161123560025)(20161123558025)(20161123564025)(6072148);SRVR:AM4PR0802MB2356;BCL:0;PCL:0;RULEID:;SRVR:AM4PR0802MB2356; X-Microsoft-Exchange-Diagnostics: 1;AM4PR0802MB2356;4:czolCRqThjOS0ZwF4b0o+s8H0qZT05S+HxnesyVJ05OAsRdmf1wEvKpkQNuGL/WI9Fd3XvbJeCEW16hnJltOuOj5iDCacpSNyRtYlDm7sPjTQxZQl2mmX74QPkH66y3iKwZmV+rJ5QqkJdUNHacwZDBz1NanFUepwMs4bGehcv9oJVFAswKWUvGxV5Fv3L9WV32nIdOEQ9lCgMoBKpDNSmgjF8drlY4wMykah3yZlt4Ubgnizr8RepaU7Bl1iks1jEG+8N24gRVG6KnyeIsSrv1uQQCJqIeYwBn1GvqwPfNw9JBy+CdBLllAFRvOq9Bn9uo25LV3lYxTJovOp/iccBLOjhOMfM9HOOIMDVwVxWP6H7oAIKF8WgTbnN6sIQTtzo5eIFYFPhx4sKX4kqD75QPbKYFgKxSrYoy4KEzTIPNlnUE9xlHDnMjewGBMhOyWJvJsGKy4gVhsXxaQ34zHWOD7EEepUgEjuVASmcHXfb82fMhwUUEDtuJqHUBFxv8ck/9Q7L2YlYymNKkKF76SUYQZNd7+Nnx3NaxaiEelCQpyyVlTGpHrjHIYyxdVfhfwWy/sgBxKz6D4OQHWlcEA2op1oVYeUeKXIOfn7aidudHlGzpycZwoPTsBiAUpZfkZhOtlT9xxStjj074lK2KNGOAAk/l0wF21J3XTawNvKg1nSu2YnEh5l31G9uekP651 X-Forefront-PRVS: 0211965D06 X-Forefront-Antispam-Report: SFV:NSPM;SFS:(10009020)(4630300001)(6049001)(6009001)(7916002)(39850400002)(39450400003)(39840400002)(39860400002)(39410400002)(377454003)(40434004)(24454002)(199003)(189002)(2950100002)(77096006)(23746002)(50466002)(6486002)(92566002)(83506001)(4001350100001)(5890100001)(36756003)(189998001)(31686004)(229853002)(90366009)(54356999)(110136004)(39060400001)(38730400002)(97736004)(33646002)(76176999)(25786008)(50986999)(42186005)(7736002)(305945005)(6916009)(31696002)(101416001)(86362001)(54906002)(8676002)(53936002)(2906002)(68736007)(8746002)(105586002)(81156014)(64126003)(6666003)(66066001)(6246003)(81166006)(47776003)(6116002)(4326007)(3846002)(106356001)(5660300001)(53546003);DIR:OUT;SFP:1101;SCL:1;SRVR:AM4PR0802MB2356;H:[10.1.194.26];FPR:;SPF:None;PTR:InfoNoRecords;A:1;MX:1;LANG:en; X-Microsoft-Exchange-Diagnostics: =?Windows-1252?Q?1;AM4PR0802MB2356;23:dhmhqqSj0vugT6Pa0fm8TlUanhZ+FEzPfJG?= =?Windows-1252?Q?PA0Ob/Qq0kcY2+r8OnMtW0gx3t+l2pT4Luxtbx5CZNdruzZNKAb+152t?= =?Windows-1252?Q?x5oEj5qefNnCBikk927CFblm3RDKtR8AUEPMjJNFnqcz2TxSxxglo3FN?= =?Windows-1252?Q?fRaoWX0btqoZrE3H4E3ZJKVpuvZk+xSjjQsIZHr5VQbcVxhaU/jON0tR?= =?Windows-1252?Q?3cvbh5EsTvTEyFbajV57RkG6ReUwMKs9w/jFCoxCsOBMK3w1p5ZKrrEt?= =?Windows-1252?Q?48GYCGlJHNJKf65uXi0+3Q6rHICoecXn6Sg078xR5NXOOMXHBIS8rH/B?= =?Windows-1252?Q?6bpP+V+8wecww8pnLEPf3u/l6Vvv5pgGELKewYveaEj3DXjxK66JY3XR?= =?Windows-1252?Q?esVgfAx9xc9SnhwJdpV6RJUToKxq/sJPGM7CkuRykhLih+kFzfwLwtFW?= =?Windows-1252?Q?5VvvjJGuN6Xt6jemHUUgia8mO0BU1pQOgcLXxUp77ZeEx05BEm7J70qq?= =?Windows-1252?Q?h3pUmbvlP4/OQM+DJBkFSThWgjT8IotiKkYpsUrH3yaFeBIwyn9ELsVJ?= =?Windows-1252?Q?ZeejlPILfctF5KyYUYwYMLUIamKiRhRkCkdP/WHHcpND75Bzh6VYpNTa?= =?Windows-1252?Q?YzKGpEdILInL6GNu0vgqhfnOFrLQd1uvaWuKbiO1SORCMx42j2a1CSCD?= =?Windows-1252?Q?AwIuLnWXcfXL3e6QCCijh1PwbLAleJIR+0VOB2580m1rPiGYng6TKKoJ?= =?Windows-1252?Q?3fMNsjdRnSw6nCzBLwzrdyPYBSMK1qOT0Eqa+uZrq0O6xlJCugw7Ju0A?= =?Windows-1252?Q?uhYHcgVVXW61NsCzkIGDqc3Kqpkaoh5hHevYulTrhC/2etjiPhUPScdX?= =?Windows-1252?Q?ROYWoDlCXsyZxfqKAYaJxQOSnY4B3WvXEY9piCLEUlRZD9LLNweu8nu9?= =?Windows-1252?Q?nxjDfAODgcJclWyOyfTcH630lQiGH7Fb38Ob27+M9bg7yhJLjXwbjM6x?= =?Windows-1252?Q?Tt0mccsZTRkosOtKzMVt7jv44LWh/qkuyCML1uWD+3V4zC+mwtXMAO46?= =?Windows-1252?Q?9dhYMalnql0id+W2U5tlqwt/Ul2gxnGBZXhKuuDCL+XxfxsmXdSmolFX?= =?Windows-1252?Q?vmdaNPrTSh75oUqzSE9Rf5RTfxauE5sEMRzT4H8oKlRljMY4H0mlBGg8?= =?Windows-1252?Q?I2pFICva/1H7EQIR/0QBce8t6m7jDQYLNif0n7cXtzKeId1iC8WgHhUd?= =?Windows-1252?Q?DFSLg+FiFkyU3URLAdb/AsPs5EDU9gtJfnz5ypVp15D3lcZeLqW3zgAk?= =?Windows-1252?Q?is3wJq4tGp3tDsnCa6jHAY98nXxdq0OuvaHbR+dHII6krNjLMqimQOcz?= =?Windows-1252?Q?2hZXvP7PlakPWLFxlEXVeUNl6GvUO428w49aZKHgul6sgoKyIgNlSvbv?= =?Windows-1252?Q?vZAauzRZzCvyHfol6MRr6h0r0JacJ5I243d4dd739rbiBm2W+FsIfTKa?= =?Windows-1252?Q?gg2ClM472qZfz8hSdwNiQg1l3hYwAdEz1uCuZAVX/dmOSn1xUnhbp8WS?= =?Windows-1252?Q?mLqpo30rZz9cICJdKf3IkeGt/VSHbX2Bmz9YMwi9/TzEsuusAnCRha4V?= =?Windows-1252?Q?sElLIfYn5kO7MfhITzTbRb/CPT9ssmDLXUV9E7/bFCPmC2BDPO5pH3NO?= =?Windows-1252?Q?YUsYRdXHbsQ=3D=3D?= X-Microsoft-Exchange-Diagnostics: 1;AM4PR0802MB2356;6:Y+wIZ3+uqp2SMPYj5XXOpjBTM1TMU5VQGHiZ7vz3AYMQ2MNUQbkB+3kptBKuQoDTar+T/uDpUffGLJCIb0cPALHkj3EEgfPqIg0mJsMIi9mMEa/Lj4jeFs1ZOqP0u2T6WfEz6Ekbn4m53usewGMuHMRDWgels8ghxu0Jn4kvSGTmQXQJT5sPTw4ZrfY8wHU1fEJAb6RfbyWHUKPrbBcoSr9o4qW+kZ81l+ZhhqZYFZzG0xedO4J15jThsDsS+zxaS76X45CJ91SPKISq8BI7MOdUYnwPkJ98SO3E30ZlLBEW83yc8WluBnQV4r7GiSjCpBDBrtSACYYoy4Yj2CuNjuz73cl7vOpLQo6lRcuVd7Ze9ZUCy6oOEcnbMD1EghWv3yvAikB/6iCzUdsf5fJ1dnvzeuaBbwVBmfO8sHk7tXk=;5:2G1dNsBFmT+79FWtxCIqctDFfbKQ2xfRwUhqLC+WwOv9hhyePs5Wq9D646gzD1ImtrbgtKVrPcE5/QcLrSvA/nVxpQXfoj9NX0ZV7bikuaZgIjAryzshe3AfyW53zSX2mGIE9mJOzWf4HvNEjCbo9eZ8B/ZtE8lkbKZDbMqvQCo=;24:NQIK32qbrWlUBVrm8YrU3VCOzKvcJ35khWLfiGFwziJ3FBJ8MqzYymYs7m3Sj7oFv5ECS1WgjO/SuH4xHJ2M++IbHz0XeMYfiUKPcVOn3ho= SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-Microsoft-Exchange-Diagnostics: 1;AM4PR0802MB2356;7:fFhkKa5fG0PpD0C8bIt4n0gt+eB7nROULgYoz40o4RFtp648vQC3RGkb3BMuIjC8K8len0bXHUeHVWFhFwST4JZcoRCMOfG5gjkLORryme/BPsLVus3c0WsKxs1/PNY6pqFL2ncstp0aZLa3YeNOy0xopl1ZvDqfYL12x0CyEsno5WcBkUaK2So5DX2F9CD5Q8/PAXWz3OOwiVDUr1wuHKo7Kzu9bV10Q9rQx9jkdkDQRZ3bIN6przxNFDAfjn9Z0Pl5CkGMcdv1hebnWi/L5I0hlrI8hKjv/m1tojm9qK4BRST6IQbgK3z5pPc48N9mxpMKEDUs6KyOrj2/q7vb1sUXt5POwEIrhL1WdgzibVQygoHv8+PCwYS8oX2VekqOyqbhqm+rnVSeo85JFvb3ILiwjhh9qZpd1J7tRCq29f58+SMw66cpUIldNkqwwIEmpajvEEb45hghStzARpHOEbB2vs9Lz3AgxJ5IRrCJ0fGEg7rSJUyzK+Eu8VXx2YBTFdwEHUm5gGI2b7+4Ys4/8Q== X-OriginatorOrg: arm.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 07 Feb 2017 17:19:32.6149 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM4PR0802MB2356 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from quoted-printable to 8bit by mail.home.local id v17HJjjj028002 On 06/02/17 22:21, Baicar, Tyler wrote: > Hello Punit, > > > On 2/3/2017 9:17 AM, Punit Agrawal wrote: >> Tyler Baicar writes: >> >>> From: "Jonathan (Zhixiong) Zhang" >>> >>> Add VM_FAULT_HWPOISON[_LARGE] handling to the arm64 page fault >>> handler. Handling of VM_FAULT_HWPOISON[_LARGE] is very similar >>> to VM_FAULT_OOM, the only difference is that a different si_code >>> (BUS_MCEERR_AR) is passed to user space and si_addr_lsb field is >>> initialized. >>> >>> Signed-off-by: Jonathan (Zhixiong) Zhang >>> Signed-off-by: Tyler Baicar >>> --- >>> arch/arm64/mm/fault.c | 31 +++++++++++++++++++++++++++---- >>> 1 file changed, 27 insertions(+), 4 deletions(-) >>> >>> diff --git a/arch/arm64/mm/fault.c b/arch/arm64/mm/fault.c >> [...] >> >>> @@ -426,7 +439,17 @@ static int __kprobes do_page_fault(unsigned long addr, unsigned int esr, >>> */ >>> sig = SIGBUS; >>> code = BUS_ADRERR; >>> - } else { >>> + } >>> +#ifdef CONFIG_MEMORY_FAILURE >>> + else if (fault & (VM_FAULT_HWPOISON|VM_FAULT_HWPOISON_LARGE)) { >> Please add spaces around '|'. > Will do! >> >>> + pr_err( >>> + "Killing %s:%d due to hardware memory corruption fault at %lx\n", >>> + tsk->comm, tsk->pid, addr); >> The message is misleading as we're not really killing a task but >> delivering a signal (SIGBUS) which might not always lead to the receiver >> being killed. >> >> But considering that we don't print any message for the other faults, >> I'd prefer that we drop this pr_err. > Yes, I'll drop the pr_err. >>> + sig = SIGBUS; >>> + code = BUS_MCEERR_AR; >>> + } >>> +#endif >> Although to get a HWPOISON fault CONFIG_MEMORY_FAILURE is needed, the >> handling seems safe even when it is not enabled. Can the ifdeffery be >> dropped? > Yes, I can drop the ifdef. The handling would be fine either way. >> >> Also, I was wondering how this code was tested? Did you by any chance >> try using hwpoison inject debugfs interface? > This was originally tested using proprietary error injection that we have. > > I just tried the hwpoison inject interface and it didn't result in > hitting this code path. > > [ 70.747697] Injecting memory failure at pfn 0x400340 > > [ 70.748547] Memory failure: 0x400340: Unknown page state > > [ 70.752911] Memory failure: 0x400340: unknown page still referenced > by 1 users > > [ 70.760167] Memory failure: 0x400340: recovery action for unknown > page: Failed > > > I've never used hwpoison inject though, so maybe I'm doing something > wrong :) No worries. Writing the pfn an executable is loaded at /sys/kernel/debug/hwpoison/corrupt-pfn triggered the code for me. On my system the program dies after printing "Bus error" - probably what the default handler from glibc is configured to do in this situation. :) > > Thanks, > Tyler > > -- > Qualcomm Datacenter Technologies, Inc. as an affiliate of Qualcomm Technologies, Inc. > Qualcomm Technologies, Inc. is a member of the Code Aurora Forum, > a Linux Foundation Collaborative Project. > IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.