From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751139AbcGNHt1 (ORCPT ); Thu, 14 Jul 2016 03:49:27 -0400 Received: from eu-smtp-delivery-143.mimecast.com ([207.82.80.143]:27623 "EHLO eu-smtp-delivery-143.mimecast.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750852AbcGNHtT (ORCPT ); Thu, 14 Jul 2016 03:49:19 -0400 From: Zheng Xu To: "agraf@suse.de" , Steve Capper CC: Ard Biesheuvel , Mark Rutland , "mbrugger@suse.com" , "Catalin Marinas" , Will Deacon , "linux-kernel@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , Stuart Monteith Subject: RE: [PATCH] arm64: Add config to limit user space to 47bits Thread-Topic: [PATCH] arm64: Add config to limit user space to 47bits Thread-Index: AQHR3W0OYoogSnRxpUazXxh7tm0Jk6AXee8AgAAE/CCAAATsgIAABu9w Date: Thu, 14 Jul 2016 07:49:14 +0000 Message-ID: References: <1468424567-15925-1-git-send-email-agraf@suse.de> <578668D3.6010406@suse.de> <20160714010812.GA17138@e103986-lin> <837E6456-4718-45D7-8EE7-E28E11DB8BD3@suse.de> In-Reply-To: Accept-Language: zh-CN, en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [101.230.208.200] x-ms-office365-filtering-correlation-id: 81619fb9-4863-48d8-2f1a-08d3abbb5a24 x-microsoft-exchange-diagnostics: 1;VI1PR0801MB1615;20:4a+9zy7WYg904qBcimlDK3A/ostVtdv95oeGiZnbvwVXtMe5lYj/HXGLFMWEvaoTWHqMfoHyiGSdTafPn+7OKRZpwVhwIi8+xVX/OChQ0zOnau25efM7+Mef7PGwD3oQART8wblBB3uVcgCUVEnE09DT/W31UsHEg7ePsH7/Zm0= x-microsoft-antispam: UriScan:;BCL:0;PCL:0;RULEID:;SRVR:VI1PR0801MB1615; x-ld-processed: f34e5979-57d9-4aaa-ad4d-b122a662184d,ExtAddr x-microsoft-antispam-prvs: x-exchange-antispam-report-test: UriScan:(38517522185527)(9452136761055)(258649278758335)(5213294742642); x-exchange-antispam-report-cfa-test: BCL:0;PCL:0;RULEID:(601004)(2401047)(8121501046)(5005006)(10201501046)(3002001)(6055026);SRVR:VI1PR0801MB1615;BCL:0;PCL:0;RULEID:;SRVR:VI1PR0801MB1615; x-forefront-prvs: 00032065B2 x-forefront-antispam-report: SFV:NSPM;SFS:(10009020)(6009001)(7916002)(189002)(199003)(24454002)(13464003)(40434004)(586003)(81166006)(33656002)(81156014)(8676002)(8936002)(9686002)(189998001)(101416001)(86362001)(3846002)(6116002)(102836003)(87936001)(76576001)(11100500001)(4326007)(66066001)(68736007)(2906002)(54356999)(76176999)(93886004)(5003600100003)(7696003)(2501003)(106356001)(77096005)(97736004)(5890100001)(74316002)(5002640100001)(105586002)(92566002)(122556002)(50986999)(3280700002)(305945005)(2900100001)(2950100001)(19580395003)(10400500002)(19580405001)(3660700001)(5001770100001)(106116001)(7736002)(7846002)(15975445007);DIR:OUT;SFP:1101;SCL:1;SRVR:VI1PR0801MB1615;H:VI1PR0801MB1423.eurprd08.prod.outlook.com;FPR:;SPF:None;PTR:InfoNoRecords;MX:1;A:1;LANG:en; spamdiagnosticoutput: 1:99 spamdiagnosticmetadata: NSPM MIME-Version: 1.0 X-OriginatorOrg: arm.com X-MS-Exchange-CrossTenant-originalarrivaltime: 14 Jul 2016 07:49:14.6805 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: f34e5979-57d9-4aaa-ad4d-b122a662184d X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR0801MB1615 X-MC-Unique: a4r_nyDNOmKSmG3JzoqksQ-1 Content-Type: text/plain; charset=UTF-8 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 base64 to 8bit by mail.home.local id u6E7nVrd009077 Sorry, I might misunderstand the issue. I thought there are still issues with master. I saw that you've mentioned there are pointers to .rodata. And I only fixed the heap. So I am just worried if there can be issues with .rodata. If pointers to .rodata are not tagged and used as js objects, it should be fine. Thanks, Zheng -----Original Message----- From: Alexander Graf [mailto:agraf@suse.de] Sent: 14 July 2016 15:14 To: Zheng Xu; Steve Capper Cc: Ard Biesheuvel; Mark Rutland; mbrugger@suse.com; Catalin Marinas; Will Deacon; linux-kernel@vger.kernel.org; linux-arm-kernel@lists.infradead.org; Stuart Monteith Subject: Re: [PATCH] arm64: Add config to limit user space to 47bits On 14.07.16 09:03, Zheng Xu wrote: > LuaJIT also fix the 48VA issue by allocating heap memory below 47 bits. > > For mozjs issue, if there are pointers to .rodata, it can be a problem. Does it happen on master and do we have any case to reproduce the issue so that I can take a look? mozjs is fixed with your patch. I backported it to all of the ancient versions of mozjs, so we're probably good there. Though every distro will have to redo that work in their own trees, because older versions of mozjs are no longer maintained upstream. If you want to be a hero, you can try to port polkitd and gnome to use more recent versions of mozjs ;). Or maybe they can already and we just messed up packaging. The issue I mentioned with "js 1.8.5" is this gem: https://developer.mozilla.org/en-US/docs/Mozilla/Projects/SpiderMonkey/Releases/1.8.5 I don't know the exact history, but I think it predates mozjs. With a bit of sledge hammering your heap allocation patch applies there too, but we still get the rodata references. Alex 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.