From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 59521 invoked by alias); 9 Dec 2016 18:31:11 -0000 Mailing-List: contact libc-alpha-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: libc-alpha-owner@sourceware.org Received: (qmail 59479 invoked by uid 89); 9 Dec 2016 18:31:10 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-1.4 required=5.0 tests=AWL,BAYES_00,RCVD_IN_DNSWL_NONE,RCVD_IN_SORBS_SPAM,SPF_HELO_PASS autolearn=no version=3.3.2 spammy=the, The, H*r:sk:mail-co, H*r:sk:NAM03-C X-HELO: NAM03-CO1-obe.outbound.protection.outlook.com Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=Steve.Ellcey@cavium.com; Message-ID: <1481308253.29811.77.camel@caviumnetworks.com> Subject: Re: [PATCH v3 00/17] Consolidate Linux sysvipc implementation From: Steve Ellcey To: Adhemerval Zanella , Date: Fri, 09 Dec 2016 18:31:00 -0000 In-Reply-To: References: Content-Type: text/plain; charset="UTF-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-ClientProxiedBy: MWHPR01CA0009.prod.exchangelabs.com (10.168.201.147) To DM2PR0701MB1067.namprd07.prod.outlook.com (10.160.26.20) X-MS-Office365-Filtering-Correlation-Id: 1e93ea47-9260-460b-142e-08d42061845d X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:(22001);SRVR:DM2PR0701MB1067; X-Microsoft-Exchange-Diagnostics: 1;DM2PR0701MB1067;3:6GB7qNGiasFe58XIRuAvVLzm3zReby98vVF4AU2Umi5avai94dUj5F10GseRQhFtINwbJF6wFDrTblgLwWIIuULiDV5xRY5B/8c3IzEGZGJM27RYXNnRxCqpFb7w0wTnMstNtRE7LTNpxVDToDXcI9sshyzgEg0Ud1mMvfI8TxpRwCLEUJx3L4/8YepSoNPwWopOx/5HwWWCxBn6d9axyFS++oa+GO2mNPEQwd8OAcXjIWNs281+0Yj16EFcrOsE/MftY0ncxTxNzuTGF5y1wQ==;25:69CSrb4DzpIs9a96194SMfpVP3y0vncTwgdKW2/K/dmGt14247dYZiEIfW/qhI8R54/huHIQooX/UVKsL1bpaiuxPZnu60CYWtPrgjJPGcD/GgYXV+VeZ1+tpAaHEwioYmczmKOry8oKwpRLiLWxug+VEFvjSjvXYbDRjU9keYpNQkgb++LmfWfYB3eK3bveWWAI/upgFkQDdQtmM4VXTfUyEb9k1IUGpCXDwWhmYrNpFnwoWSOWyW4Ek8d4N2kdBKh059NWZuIJeYNuJYmvaqthPEF+h4fXXTKLEL+y+2lxzAxkmQxwVUSnZaEpHVaItrkrAoWwifIeb6YocOBQ38Nn2HAXvXE8hLdMII2SDDlk22yU7IQBIlSGHSmrGt74NYuZL4xXORyV2DZgQvqhKLHklwMt3VUrrfnM2FIvOwdpETsJ+V83UfcxSHVbo9r9ArrXJI8UVjDl7thiNnH/5A== X-Microsoft-Exchange-Diagnostics: 1;DM2PR0701MB1067;31:8ikzT+xmVo8rYodfqLZEv1FuN7YVR0++mYj4P0O32hucorlHajmrQvVQn+hEmUztwhMBW8RUVcMeS68nCqLEZt53VlT36DhutEui6CnQVKDhDwiyfSHGLBfMDsFAPuBNhjEA3tYKj82k9Pv6O1R4qld1+629ketI3RvjKwGD+s4PPaQyyWdrgmckIDUEj/IRBhN1tD4GeLpY717U5Z4n7zVv6pYnMSwpqDQl2hpMh+Rg/Dc4aiWTM8QblDDzu/kxCyAhhXgJK9ipdnGh7sOkutHKdlxGePVGZhkZa/fC2jY=;20:fjjk8JuNTrk6MvmOJHQsopjg6hVjBtzz1TOj1ke+UBzHbqS9nBxrkuLtMKJaQ8XG/RJ9Etq8E/x9ZyuskdpZEBemhV9OBQuUW40V11cZk/SjoewQAjFK6ERVE42BUkLtm1qxX9y/xexCWkqHwPSQ5crnPYDBN8yzAAHcY33CWUCMSZYC9F+qsYNGrrnFCBOxuHtg3OFSKtM+tANaZOBNVczKAEG78hz4b4Y2Y4r/xW8PZMShViRDGFipq5wEPdBk7jEYWW5YfW7U5+Bb7PMRVGK+JHeC2iFPUzmaI0bmLby6c+zLz9zAykyaWMkO4zcecVtPbn91WZB++bIFjrjOCuGjt20Bs8qEm8X19TD0oEg/RCwyjlkmZ6U1iwMMc9KfOmtXlDxmKiRnbKn2btSoWTenA4QNaD2I0IBwg/wAsDgRMtC16WIbPaTUgAz4xeW1AGBSk0eZIrhOCcyUe409NYaInidmwso9hw50EPHSkNt4GY1UGM4zNMbP/ovbMbu0Rm3qTR403Ub3mREk8KcUWHg/kF3bFdYa24BGTvRo5DcyN4jF2HXng6IH456gtnwDz+LzoeuccMsAZ+tQpgWXTY33qRDndM1aKdp/nbgfUE0= X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:; X-Exchange-Antispam-Report-CFA-Test: BCL:0;PCL:0;RULEID:(6040375)(601004)(2401047)(8121501046)(5005006)(10201501046)(3002001)(6041248)(20161123555025)(20161123560025)(20161123562025)(20161123564025)(6072148);SRVR:DM2PR0701MB1067;BCL:0;PCL:0;RULEID:;SRVR:DM2PR0701MB1067; X-Microsoft-Exchange-Diagnostics: 1;DM2PR0701MB1067;4:mIVMBMjN4y0k7dkKQTErPsPvPacKnoxf7l5ZbDHGPsnFag3m+s//T4Cx+wc276EnVCuz8eP3f/e4uTde00gHYi8UR5KMEvqCuZEqEk39UqyIIoOz8bD29wLO/ge0Wx2Mjg7WbegVXKXkkHA/qT0pB48Fq1EYwb+fy6+sxshpreQ8gpxdISzb8WDWFcjpwdBux2CFYuH/7J91Y2WFx+PZVBsbFoFEYvg1MnpzdlY0nvuon4bvg2TCrCBh24wY1CuH1Es9BIIgqo9+CIefkGwUNKo1F0K7dEuHj7PayHlp1pnrULLjPt8bJbZkPuMwenK1dSvoo3SkU3GC5z0nMHot+HHUqGYgwaC2I/DMBbLs7hV2K07MWXHbd1DfVvfX3ceQK+YqWvb6r/BlS0Dn7RRT8lZsYRB6ayQwLy23SflZeJX9dIpi0oGTts7vQc+/ycWBmA5/tJf8VYFVrYsHSg4dAFSAbbIYohpXiLfd5SoOtdN4opwYdvCtLSHMU4bK9IAlGciCiORmAWbDUihGpI3d+kF5fzw3Zi+N+Qb//4v5pNKH1sF5v/TLKCcgtuDqHkPGKvR//Bp6uKwxOJHl1jPdQg== X-Forefront-PRVS: 015114592F X-Forefront-Antispam-Report: SFV:NSPM;SFS:(10009020)(4630300001)(6009001)(7916002)(39450400003)(39410400002)(39850400002)(39840400002)(189002)(199003)(24454002)(377424004)(6506006)(6512006)(2870700001)(5001770100001)(97736004)(305945005)(53416004)(6486002)(38730400001)(7736002)(69596002)(101416001)(229853002)(106356001)(76176999)(2906002)(6116002)(33646002)(50466002)(3846002)(42882006)(2950100002)(50986999)(8676002)(6666003)(50226002)(66066001)(92566002)(47776003)(81156014)(81166006)(68736007)(5660300001)(107886002)(189998001)(36756003)(5820100001)(103116003)(105586002)(23676002)(42186005)(99106002);DIR:OUT;SFP:1101;SCL:1;SRVR:DM2PR0701MB1067;H:sellcey-lt.caveonetworks.com;FPR:;SPF:None;PTR:InfoNoRecords;MX:1;A:1;LANG:en; Received-SPF: None (protection.outlook.com: cavium.com does not designate permitted sender hosts) X-Microsoft-Exchange-Diagnostics: =?utf-8?B?MTtETTJQUjA3MDFNQjEwNjc7MjM6NnlHUVE0dUFsSXFnT2o5ZlNyYUhjYyts?= =?utf-8?B?TUNnaEQySmJHbGNSaXJ5OUt0NEtpQzNFblpEang4UDEvRGVyT0pSU0RDQW93?= =?utf-8?B?dVlPVmJKZnp1bERpaU0vbHhQUmtPeWV4SUFQZ1JrczlzMjh2RG9mVnF2NFRT?= =?utf-8?B?NTRCUHFoTEJ4MTI5OFVHd3FZSGlIYk40QURCY3dzdm9HQ3ZER0NGZnJPMm02?= =?utf-8?B?KytLakFQVTAxdlpOQk9TdXJjKzlIRXI3V2lmU0tiazJGTFNPVk52M2VNQnRr?= =?utf-8?B?NDkxY2Y0b2NDQkJvTEtLL3dKMmoyeUlpQlkyZEtUOFgyR1g2U0FabHhkZHo1?= =?utf-8?B?MHVjN3NKU29vdWtJQ1JOdmZLL0Q4NWl0b1dUTTVZcnBCUElhU3F1SndqZ0Qz?= =?utf-8?B?V1hGTDV5QVpRMlR5cmYwVVJ5L3pJTGFuaXJyY0svdGR6VW9td254RkxXbDY0?= =?utf-8?B?dG9MaGw4YTE0cXhUejR5eElFK1Z0QkJxelJYV3MvM0hPN1NPd2xnYjB2RDdp?= =?utf-8?B?TW0xOE9yaTFTS1BybGNtNGR3eURiZndMQVh4Y0pkZ3RnU3NwWmRLWDh0OFcx?= =?utf-8?B?bEI2Z05LTGVPM3M3cmhNVjNtMGt6SVlHU1NCcFU4cFp2a3BsOUxDcGw5b09R?= =?utf-8?B?aU4vOVlqVmV5cUE4NVQ1MU1CNzBzRHhhMXdEdE91bHErc0lZZDdFeXpTa2R0?= =?utf-8?B?d0tUbUlIUzRWV3ZKaW41ZXNObGIrS0RoK3o2NTZpZlpKUDZMS3JmZWlCK2Zj?= =?utf-8?B?NjdzYVBXZ0VXV29oQlpLNEQwRlN2ZjJ6Zjlma2hOS1prU0gxWDl6TGxXcklu?= =?utf-8?B?SDhXS3o1czBjdXpMK0hBZUJtRmo2KzkraFp1N2FDS0h0MWhzR3BRQ08wZlVV?= =?utf-8?B?aDB1enMyQnhpYzFsNWQ1OXhUNEJrTUFZSjJGSjhFc3ZxZEZtTVJjckFSMng5?= =?utf-8?B?OGt6WXNxTnBIRnAvMytyZ3NHall3M2RhYkdUVnd6VVg0RTZRUFpIWnExVmho?= =?utf-8?B?NWUvS05xb2xwZWY2VzdxQWRtTjRZcUYwZGxaVEVUNFExWElkOEJWWW1oeG1u?= =?utf-8?B?MElKM3Y3Z1MwbW5NekxVdXM1QVhsMjB2d2NxcHZ6Zy9GUjcyM3d6N0szYnRD?= =?utf-8?B?VEdOcGJoZ08xMk56bFdPcXZ1Sm5CNTVvVWgwbWVibU04d3ptczIybjFVYXBU?= =?utf-8?B?ajdaaWl4bHlkTURjUDN6RTBob1IvYzdQdzhWTDc3dmF3bFp0bmUyS05kR3gr?= =?utf-8?B?aEhYb3JQREF3c1FVM1JJUGJGdFlpQlVXaGtHRlJFYzg4c2pNWWxsTmpUOGd1?= =?utf-8?B?Y1YwQ3JQeDFXRFcvVERySVBtdmlHY05HNExzTHJjL0RleWoxUjhRcWVYcVJV?= =?utf-8?B?YzJXTXZjNlJDNnkrYnVxeDZqMFVUSjRnWWxSa0lNNzIrWHNpL3lHUHhpT1dS?= =?utf-8?B?OEhlUW9zdW9uZTlrZXRieHNwOFdrdzhZYW5YUC82c21uRVlMWUJKWFFtTG94?= =?utf-8?B?YWpTZzJKN1ZIaG1XUlBxVEFzMVllbTBYNTNianV6R0NEMGphcS9scE5pUUk1?= =?utf-8?B?ZGIwLytwaEJHS0l0SU8wdVUzOVkxVUgvVElaczdlUW5PejFtcWFweW9HeXRT?= =?utf-8?B?TEMvSU1QT2JocmNtZDY5NHlDSm01T09wdmNQcFFXWmZpTHdvVkhQc0pidHZG?= =?utf-8?B?TjV2MlJmNXVXVGJoY0dQWjY1ZThPVzVGRlRRYnQ4bndHZ0cwOWZEdEVNcDJI?= =?utf-8?B?OTNjRmNzck5MN2hHMXlLYXRRPT0=?= X-Microsoft-Exchange-Diagnostics: 1;DM2PR0701MB1067;6:SW1/r165rMK7NKFFv3RfAxACXPD6hDypLxiwbz4dTKPFP7XseWwc6Hy71/JQVbdtihqJdM85wvzzJUyslSzEFNA99f8Fhxb+7Z6fmeNcKSwOYSkMQXAaxxeHieFmN4Sk0WRDshtDGK4VcMCSwvNhjl7z6AddAtK6EqFZbHnwoj/rh2ha7wGkaNbC/XGRcu71gXpjLqKLO/Zh3X9mNe3VtJlBEN0ZJOfKND9JSTYaoDdjRyj2V1NjnaH1AlMKNDyrHrDU1Fe9udLuF4CBj9VtthAUfcYQBHRmyEdHq5+yffMZwqFG+7Sgx/RyyDmB603LGwEjPHc1/Roojz8FsOaIXQCSeJgBiM1mPZdtR7Qxbg4OPnn4kMYwUppiK8g+gFvvEPLbmCjzN9i2Dr9lHFv/WsoKIS1HwyN0MVie+s+GQt0=;5:LNR2sLr5l70bTXOOPgjGQ0aN0xF2hlAzCGxIZDgXqjJsb/7fS0+DY//tGejd/iwbP2DzZeRKWtKFFtbv8oaVyBCdIPUWrShrVYMZ2eDK0QzjKsRRJjroXTaDnhYcY/byIdd4DHJu6II3/umob7kPrw==;24:keGzNX721c/NBy9QUf8zcekbfG/bXf8YZ2hX6IiAhG82e+BxRl/kpU3vJq5VOVY7IzfCJXsiOf++06CWvRdHGf/rKarc7t5vq46kyiWdgZ0= SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-Microsoft-Exchange-Diagnostics: 1;DM2PR0701MB1067;7:20Pgo4xiiuIExwwnukz/1PiGRcgJcYmmQgu93HSjX7f2Bazeu96VQsqXtI3w6y6ydd8VzrdJgBllvcFE+XQIE0RX9qwX0IFUz5q1kfSoDA5u8+sIY0koF456bmMioNOR/otDm/VNTWLuB4UQppTw8993YNrmo2ooTqdR7bLnswxHfSqXyPkwn0xK7Ccyeg6Vvmu3UvTvGu5QQU8E0l3mxMpKMDnhikuK/ZRtfSekTUSr5ozgsa/klrBRZ9o+2AfwseOdNWBg8j4IUbkULBZRm7XeoDzN7QIWMw1N3gIymPqyDhja6tUJSRZTgoICK4g/bHNoGig4MtoQGeAJ1+twxzOieEa7Bc1JHOtMTqqI8n4AlJL4OAs8oYiP+SHedl+zylMc9ysljUG5pE36GHfBSAT8auA5CAlEcIn29LZMQJWEwAkl9NZpq7CwjeBtnHuQRLQm27AaoBWH/qIAvjiLRg== X-OriginatorOrg: caviumnetworks.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 09 Dec 2016 18:30:56.7889 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM2PR0701MB1067 X-SW-Source: 2016-12/txt/msg00324.txt.bz2 On Wed, 2016-12-07 at 17:13 -0200, Adhemerval Zanella wrote: > Ping x2. > > > On 08/11/2016 18:29, Adhemerval Zanella wrote: > > > > > > Changes from previous version: > > > > > >   - Refactor ipc_priv.h header to have working definition for all > > >     architectures.  The header basically contains Linux specific > > >     ABI definitions for the architecture to correctly use default > > >     implementation, including old ipc_perm definition, calling > > >     convention, and __IPC_64 value. > > > > > >   - Set all architectures to use the default implementation.  It > > >     leaded to some code changes to accomodate all the > > > architectures > > >     calling convention (with ipc_priv.h change) and some fixes > > >     on new default implementation. > > > > > >   - Change mips64 implementation way to use the default one. > > > > > >   - Some tests changes (name typos). > > > > > > -- Adhemerval, It looks like you may have to tweek your change to test-skeleton.c in this patch.  The FAIL_EXIT macro was moved from test-skeleton.c to support/check.h so I assume your new FAIL_UNSUPPORTED macro should be moved there as well. I have been doing aarch64 and x86 builds here with your patch and have not had any problems except with my new aarch64 ILP32 code.  There I had to change the definition of __IPC_64 in the new header file sysdeps/unix/sysv/linux/aarch64/ipc_priv.h from: #define __IPC_64     0x0 to: #ifdef __LP64__ # define __IPC_64     0x0 #else /* __ILP32  */ # define __IPC_64     0x100 #endif I don't know if that is something that you want to incorporate into your patch or if I should leave it as part of my overall aarch64 ILP32 patch.   Right now it is hard to include it in my patch since that header file isn't in the official sources yet (being a new file in your patch). Hopefully someone will approve your patch soon and then I can just build my patch on ToT instead of having to also include your patch. I don't see any other problems with your patch myself but I am not that familiar with this part of glibc so I would like to see someone else review/approve this patch before you check it in but I would also like to see it checked in soon so that I can move ahead with my aarch64 ILP32 patches before we get too close to the GLIBC 2.25 cutoff date. Steve Ellcey sellcey@caviumnetworks.com