From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752674AbdBMLQn (ORCPT ); Mon, 13 Feb 2017 06:16:43 -0500 Received: from mail-db5eur01on0126.outbound.protection.outlook.com ([104.47.2.126]:34784 "EHLO EUR01-DB5-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1752172AbdBMLQl (ORCPT ); Mon, 13 Feb 2017 06:16:41 -0500 Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=dsafonov@virtuozzo.com; Subject: Re: [PATCHv4 1/5] x86/mm: split arch_mmap_rnd() on compat/native versions To: Thomas Gleixner , Dmitry Safonov <0x7f454c46@gmail.com> References: <20170130120432.6716-1-dsafonov@virtuozzo.com> <20170130120432.6716-2-dsafonov@virtuozzo.com> <20170209135525.qlwrmlo7njk3fsaq@pd.tnic> CC: Borislav Petkov , open list , Ingo Molnar , "H. Peter Anvin" , Andy Lutomirski , Borislav Petkov , X86 ML , From: Dmitry Safonov Message-ID: <3a275384-0c4a-5334-f31b-5fadda9d8ee9@virtuozzo.com> Date: Mon, 13 Feb 2017 14:12:55 +0300 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.7.1 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset="windows-1252"; format=flowed Content-Transfer-Encoding: 7bit X-Originating-IP: [195.214.232.6] X-ClientProxiedBy: HE1PR0802CA0020.eurprd08.prod.outlook.com (10.172.123.158) To HE1PR0801MB1738.eurprd08.prod.outlook.com (10.168.149.150) X-MS-Office365-Filtering-Correlation-Id: 3e235a25-b90b-432f-1baf-08d45401c752 X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:(22001);SRVR:HE1PR0801MB1738; X-Microsoft-Exchange-Diagnostics: 1;HE1PR0801MB1738;3:MlGDfwj8DaCQpefWoyCVwA6XZQlavUGBYPMvq2P2M5bU/TljcQbVjv1wL6MT5YabPvJwKV0/o1VVwEGgabo4P9YZsB/N+PVOTqhIAjZWaWjIvq19RXyV8p7eW9mqV0JATmZSVW4K/0lnHYvym1ejnXdKNK7kdJbkwDYlcqMrcv7tlT7Ne7/R2xsmGcIJAbd8Se19ktzjkwGubB9MHbxZRegRspNiSc+K2ec3rtqNhHqILu3Jw6APlCNrZkglJUxsOzR9EPJUX7s8AqwtP3F6pw==;25:A+nP0Odk8Xa9Jc0VuzD5GpczYnUYTRagKgHiPNuPhOHGpYSDSjAuBm9cf0L3+1Lwzqo+20nzj4/Ao+Fqd4iazKkSmHwtbZ0wCoKn4YsQLJAxiojmGbZKoHBcsA2Q/Fe6F98Tpl2GKQsR2w52GqHSj0wkzWWUNcxofSFayry5dKYbC2OXy7/e/pti0KCN0i2pfIROAp2Uy6m91t9Znm9SAhzh2DkmfHk+7WBudepsS2IQ7LtlT+/iUA33iQHnwBjxPBx0hMtwYDCRRdkVtjCjSANIupTQJde2hOOC6KWawBB1QlUrZpg+03ogZTjZxTuTxzk/ZwkUJRKr4t2mPmP33CIYpqUdZNOe4zHP/83Q8+5wdyZPs29HSjmiQc6Z7fSJob61118ZqJPG8e1VR2z0hJBqk2OcgHIXnV2QJIOdxra5jsJ+ctEQZ9j0slW7lXYkdpwTn/QhvQTiEP45k75H8g== X-Microsoft-Exchange-Diagnostics: 1;HE1PR0801MB1738;31:QyQNqkXeY7MKDEOypxDByLynXCaZJpMDcIaYfl+UH/moDtOYNRx85q8j+Mjo5YPRP+qOjUdLpsssZeWzkmdhZ+z7mWhA76eWaVqOqwX0Qg4gfRFHxj2PBpVMyRGGOQjiysVaGIopOoPfQr6g4hIyUJ8ntzTEj1GWIctREKQnMa+U1ukpZNc9saI723XOrLTW5yXSkP4FVibE1ho85Pb2JvCgJeqjv/aWwL/zTs6lcpc=;20:o4vJ1+PR50ucB56mfJnYBFr2df6q80sK+gpd8O5HKVQ/GRhjo0ikzPjBzTdQdJfRNFEOGLiMPjlKsCgSDosQyuduNYqPmXfNKsBZ2zLeRQuWklSgL+nNyW9e+tY32/XQsESHlDC09b24Hu+OvM55gPEZ1wt3cCq9ty2/uEvtxCC1gGGZ8yS2fPja4RZKpe3oOvtHrsg/sQbtP2F4ySZfhuokeSDoW/ECIk1XQi5sngccLvMFVbl7o+8up3hj7JwzpP8+Bf54KwV2UYBuONQluMWM5/yUYXFHfvs2mik1M5aYXBgysW/BAA8vc8F9qzNTf46HkaE8TNjjgEpK7yFtPWP40XwsHQhcJA6PN4VjShd8FoVIUOgoxCZ+eUYlnLgS6KGdGX5N7rh/Mpn7peQUzILG6Y99Z0yDZksnAo49K+g= X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:; X-Exchange-Antispam-Report-CFA-Test: BCL:0;PCL:0;RULEID:(6040375)(601004)(2401047)(5005006)(8121501046)(10201501046)(3002001)(6041248)(20161123564025)(20161123555025)(20161123558025)(20161123560025)(20161123562025)(6072148);SRVR:HE1PR0801MB1738;BCL:0;PCL:0;RULEID:;SRVR:HE1PR0801MB1738; X-Microsoft-Exchange-Diagnostics: 1;HE1PR0801MB1738;4:FcRNmIWAol0Oi78Gn7V7FA/HpoUlmZItG7CcUmsnQaMZQqN+MdDhZcv1ntWrHN24jn5pybBZP3msRtFqscdEUV6wkxSiPqtuV4eOLsKtbRkIrP4roVNIvaQ6f1ajMeuoOHNOO79fxq8jq1W5NPCBGCHvw4nHijw5LzNTDQWnVQyfY5erX+iFsw3nxuSKGQptor+oVu2hHgjdvw2XQzp6BoLmnWHaoP3USekrrNOYnzihyi18q2j70PorDK22wCGeVn2rMrCDoQVngQNEF/KcJT9Xiw3Knr8eqftp4fFFdNZFpBgenZloNNECNwt1bN8JfkArFlNM7w8sotQAQaBELyML1LUr7kDrzSbOK2H//PeB1WJC3UzYF+1hK2uad7ThAiJ0ac8TTqJk0/ahW2KVMFFKGWfe668zD4Q9dt7LuXOFDCU+T0531837do2rkp5CMq3FCU5MsE2dp39A9kVEGCAvmQcT78iPJTDaJghDNMqGbc83O8eT7HHztutskN7WZbEh2dXBxKG7zEz47etROx+Qe1bg6QJPHdbNbmh8TqRcJlEnm3FaHi68hZSDP/ir+HlB2JO25bfEOrWW0pehkeDuvS5qKlYWEgEtudnZapU= X-Forefront-PRVS: 02176E2458 X-Forefront-Antispam-Report: SFV:NSPM;SFS:(10019020)(4630300001)(6049001)(6009001)(7916002)(39450400003)(377424004)(199003)(377454003)(189002)(24454002)(6246003)(53546003)(93886004)(83506001)(5660300001)(106356001)(33646002)(53936002)(305945005)(68736007)(42186005)(2906002)(4326007)(23746002)(105586002)(7736002)(3846002)(6116002)(54356999)(76176999)(50986999)(31686004)(81166006)(81156014)(8676002)(101416001)(7416002)(47776003)(86362001)(77096006)(39060400001)(6486002)(90366009)(36756003)(2950100002)(50466002)(6666003)(229853002)(54906002)(38730400002)(31696002)(230700001)(189998001)(25786008)(64126003)(92566002)(65956001)(97736004)(65826007)(66066001)(65806001)(4001350100001);DIR:OUT;SFP:1102;SCL:1;SRVR:HE1PR0801MB1738;H:[172.16.25.13];FPR:;SPF:None;PTR:InfoNoRecords;A:1;MX:1;LANG:en; X-Microsoft-Exchange-Diagnostics: =?Windows-1252?Q?1;HE1PR0801MB1738;23:z3HvIOR/3Lpdu7NTXewd8kpHQoQ0Vk+6bAQ?= =?Windows-1252?Q?yX0IMDEkoD1XYVzJDsYOAtI13NEmPcOdyLaZAECZST1ahbsWshOsWwr+?= =?Windows-1252?Q?Yciu2RjH3bfrfOAqN1ETWETIG3FO6SIzm8wwyMtjr7Hd2wfaFn/E/JS1?= =?Windows-1252?Q?e6DHA3OdIt7jj2KGudiZTP02T7VXzaL3Lwk+qfMoS33hnaMGiodYZ8c/?= =?Windows-1252?Q?pwugvPxqfSO4i5r9hum6XLHkTFJM3eObVEekZMfR+vkZulENk1GN9aqe?= =?Windows-1252?Q?c99+I/nx1C2wdR/Do0/RpXMTFmQvAEgw5HKe+sElakE4OZ6TnTfb9b7N?= =?Windows-1252?Q?fYwV1rE/fNPk4ySaJ/az7U0B1oNudYZgRnMU0aZA3kuMEQIL4mv7nzGS?= =?Windows-1252?Q?uvUCtt8McyW3G+dsQVSAcwp3tS64vjTmswuqElhBopNk1sunsfXDi2pA?= =?Windows-1252?Q?WNBm+RIWP/VuPd96dkrDOexXM99bSYqjNkPY3M3RmYSw9+hv/pQ9GKiz?= =?Windows-1252?Q?OQ/Iop+k2osceRCfGcx202ewc9eVbhoOft2HIpBYJtKwmU//1aeo4aNy?= =?Windows-1252?Q?a6OMcC+1THfk+EVhwsv9goZvuB6TaALST03jN5rdiE+Pbhq5CyoGVERW?= =?Windows-1252?Q?fpOizYuc0AWh1CIeYlmmvFcFW5jWFzGZE1HXOBdyK19DEvlZj1in2KY5?= =?Windows-1252?Q?QBo5vmSu4jb+I9shCLux+IRtGWAhjkqBa3bvR5/hAQobJWugDDqtzUEX?= =?Windows-1252?Q?n4fcZCiUhAN/3nz10+psR5bPZPVxPzAAE+J1pIVtmSP72LpIaIEj3Mfc?= =?Windows-1252?Q?dsGoUMjdvMWTeSfCIJR2nhST7hODbc0gCF/jsqCcbI6HGaqlPpGJ5LYc?= =?Windows-1252?Q?nCf3zpBDOSXhN8qy+4ixSwHild2pambnRDOft1ybL11U7XUAWCUro2BD?= =?Windows-1252?Q?94lmje7gzActJGtuPy7hL4kGQ9UFMRQ6IyvL+0wkqPAJy6oG2R5sE+uR?= =?Windows-1252?Q?AHIHBwmz2cpPSjW8FkX3rtU5pYN7Cssn68qu0ySjCIaLaGjVYfM9pmyQ?= =?Windows-1252?Q?fZJk+FVHuwhvGfSUmv9yU6od42GhGHs59Lrl1XjwZWCBfASa9wt1Alcq?= =?Windows-1252?Q?Cj+ESbRNlyuAJKG68YBdNaOUkA1/FgLarXpwk4ezHAFwoGupoopehL4m?= =?Windows-1252?Q?/DtGolbZab2fWsxXEC0uZBPVowWx8pQTvRDJo4uyEhq9LVnyJtdN1Yc8?= =?Windows-1252?Q?cfK9GEgnoucT89HJScsjqg2wzi15b8ZkCA1uMMl/3wnEg/OYKPkOns6x?= =?Windows-1252?Q?Q0N3HwSLLxqiI4DKsiUvfl6zbVTQ4CTb+af6Avd4WqD/mPqKmKy2bi08?= =?Windows-1252?Q?gFAAqNE0/vdhqyJBpz74RJlcTXH/U3HgN07QKHbDtJ0qKQy3xcboMxAc?= =?Windows-1252?Q?QtdMvp4aE2RzklRWlAQIvUJ37Bhn+V9QVbHoEWN2izzenASW5L69iITs?= =?Windows-1252?Q?Vlqxybb9c8y/CZl8v7Bm68s9IlrxtyZMYCMavVWZuyCIYUCFEfyH371u?= =?Windows-1252?Q?nkiO7M+ag+QVGCXCmnDpKjtzj8cIph6UiGsbSdd5PLZs4VB3i2IbP6ZV?= =?Windows-1252?Q?J3w=3D=3D?= X-Microsoft-Exchange-Diagnostics: 1;HE1PR0801MB1738;6:3J5G6suMBLtfPVeIk9utxPMl1xV3VGS+DlFSoaqxaWPd4c3+K4AfDEDFUjID7MNEEZfDmUXUzQseEVTU0CW/ovqPMXkVPO5jv1lIMmtzFYa7mtFp7jE3gYqDnivgPgsA8ezx+79ofDbax0mfBE3JblOPEuSH7teUhq3V8yxs+dX+vfA4grvc6lxSDXpI0kATrFwktI4EwfmP43c1974yTT1BmdAVfyQSKjooRqdmvxEI/vOKrH6KuAcfqXTbBZU5cDycaC4Yuv0dLmF5e/LNl14cTECmVIA/G3Vk/8IHjAQ3O+bTkbBTqfgoJyIuzA8NruEGPXFtMTRn0sgW2XVSFp/5aJywqlcRqD7zBqf7Ey2mXtZg21BFPfPLE9pwfb7rITTn7fxVfNy/HSaRrfMPDw==;5:R5uxu4X9LA6K1NzhJLunp6tR/TorWLFXUAJBOPpanodrc9QW7I/6Bfe250VWmZUHQpXJLA2ZYH5nB0SnM87bT37Szi6SFaB4kNXP9wwmmSH0Y6mNl9fREglVvzoeWgdZVxDW0yKVzlWOvj6SbQMdUA==;24:uaKTLDq1yZO9sBD7MfZ35MSTob+DLTsEy4OPdR/+nB3SxzqA5rcKE6sDsoIDzTSQINIgao07S9BY/vgyhIxybSddIA8wPJUZJ1AjETKtBSQ= SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-Microsoft-Exchange-Diagnostics: 1;HE1PR0801MB1738;7:JD6idE06xWWLNhldpNs5y0X/AB1JvSWZqWKR+q0mclsmmCSxmpQxEWXKpEPM79DW9FEZoikmeeHDvyaxwCWU7GalXj4i1Y4C7jWjeTjbCf1yw9vuIqeqllRqVe3UqrTX7VihXaPA5bkLlopAW4PS4WzGkBapwqK0ez6btNnza8FkW+Lh3XseuxkSUpYewAXbLPNPmrJHf+1PntfuZibiXDuauIzSBldfMZY7n7WQFkpkLa6bpZDvlqTV9U+GT5FAAfCCGtuHDX3jAFA/W+awrc7HdO8HyoqhQx8AH+YVi+3UB2fJ5HZ9ZPqAFjGJIg/57cSUqHWj2zqL2OuYf+qRm/C5R5ARW9IalQRAwj5r1KIjeYnxWc6sfM2R7Qvu7txybMb8rdkn67AfkKwl2oSl8mARqghHjY9s0dsqcbytcSuwq8bLYAXhZXj6l+0O4n8N6br8Hb5CugBRrccT1/4gQMgs0qNx3bh1EEZJFd7UPMj/G0UifakskyqE2M4srK4PisJJEF+TU6tVipB3zFWirQ==;20:jUwbps7TdqSVTx24t3EO75UswoG72KxdgntIcykClvjdmmclqOblAiW8UlQ5ku8suJWVNIzb87kehnDNK5qjkGAhPPVdjxSLmubxzusk9CGGDL454JLPFjJwkmQk91aO11n9VELEegn1orHzNgrXmwFWrbo40Dx208IhwWz/Mbe48qdI61m2ebtJC/sWfxBRkYTqi9gek3VDEwEq38edfJ5OvmtFKbow1W9l5TIjfRtoBAmrx3AP1pCDwOifesuH X-OriginatorOrg: virtuozzo.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 13 Feb 2017 11:16:37.5454 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-Transport-CrossTenantHeadersStamped: HE1PR0801MB1738 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 02/11/2017 11:23 AM, Thomas Gleixner wrote: > On Sat, 11 Feb 2017, Dmitry Safonov wrote: > >> 2017-02-10 23:10 GMT+03:00 Thomas Gleixner : >>> On Thu, 9 Feb 2017, Borislav Petkov wrote: >>>> I can't say that I'm thrilled about the ifdeffery this is adding. >>>> >>>> But I can't think of a cleaner approach at a quick glance, though - >>>> that's generic and arch-specific code intertwined muck. Sad face. >>> >>> It's trivial enough to do .... >>> >>> Thanks, >>> >>> tglx >>> >>> --- >>> arch/x86/mm/mmap.c | 22 ++++++++++------------ >>> 1 file changed, 10 insertions(+), 12 deletions(-) >>> >>> --- a/arch/x86/mm/mmap.c >>> +++ b/arch/x86/mm/mmap.c >>> @@ -55,6 +55,10 @@ static unsigned long stack_maxrandom_siz >>> #define MIN_GAP (128*1024*1024UL + stack_maxrandom_size()) >>> #define MAX_GAP (TASK_SIZE/6*5) >>> >>> +#ifndef CONFIG_COMPAT >>> +# define mmap_rnd_compat_bits mmap_rnd_bits >>> +#endif >>> + >> >> >From my POV, I can't say that it's clearer to shadow mmap_compat_bits >> like that then to have two functions with native/compat names. >> But if you insist, I'll resend patches set with your version. > > You can make that > > #ifdef CONFIG_64BIT > # define mmap32_rnd_bits mmap_compat_rnd_bits > # define mmap64_rnd_bits mmap_rnd_bits > #else > # define mmap32_rnd_bits mmap_rnd_bits > # define mmap64_rnd_bits mmap_rnd_bits > #endif > > and use that. That's still way more readable than the unholy ifdef mess. Ok, will send this version in v5. Ping me if you mind me using your SOB for this patch. > > Thanks, > > tglx > -- Dmitry