From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from esa4.mentor.iphmx.com (esa4.mentor.iphmx.com [68.232.137.252]) by sourceware.org (Postfix) with ESMTPS id CF7553858D20 for ; Tue, 11 Jul 2023 14:17:45 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org CF7553858D20 Authentication-Results: sourceware.org; dmarc=none (p=none dis=none) header.from=codesourcery.com Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=mentor.com X-IronPort-AV: E=Sophos;i="6.01,196,1684828800"; d="diff'?scan'208";a="11534834" Received: from orw-gwy-01-in.mentorg.com ([192.94.38.165]) by esa4.mentor.iphmx.com with ESMTP; 11 Jul 2023 06:17:43 -0800 IronPort-SDR: xH6i4w72Cb5bf6tAcjCFIiZs/vvO9paBv0FhAeAaihUS4r216UL2JuAuxtuXRrw0PQP2wIfH81 UIV09Lzjj2wyQWLp5gD+GJs0xYa60PictBzaNxArIYzOsYFYsCMPt+nkCBSn/LIX9QlMQ4Box5 4+YYqQaAvL1QO8YBO0tMUAAoe4d0UIeQVdYSPGQLsaYTWcIvp3Cg3EUBlqJXOf11Q0Vi2lhqMk aNGwpc6YUAB45XypeZW++UKhAchnAGoV6t9a+zih79PC3+oPN4RpSVBTVs7YpNPP7Az8IpqkQ8 Z6Q= Content-Type: multipart/mixed; boundary="------------mo69vF8320UuFYhbVs0uuyXI" Message-ID: <80c17edd-60ce-66d0-fc11-7834be230798@codesourcery.com> Date: Tue, 11 Jul 2023 16:17:38 +0200 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.13.0 Subject: Re: [Patch] libgomp: Update OpenMP memory allocation doc, fix omp_high_bw_mem_space Content-Language: en-US From: Tobias Burnus To: Jakub Jelinek , gcc-patches References: <59099ed8-3e38-9532-f8ba-e776c356c3a0@codesourcery.com> In-Reply-To: <59099ed8-3e38-9532-f8ba-e776c356c3a0@codesourcery.com> X-Originating-IP: [137.202.0.90] X-ClientProxiedBy: svr-ies-mbx-13.mgc.mentorg.com (139.181.222.13) To svr-ies-mbx-12.mgc.mentorg.com (139.181.222.12) X-Spam-Status: No, score=-11.4 required=5.0 tests=BAYES_00,GIT_PATCH_0,HEADER_FROM_DIFFERENT_DOMAINS,KAM_DMARC_STATUS,NICE_REPLY_A,SPF_HELO_PASS,SPF_PASS,TXREP,T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org List-Id: --------------mo69vF8320UuFYhbVs0uuyXI Content-Type: text/plain; charset="UTF-8"; format=flowed Content-Transfer-Encoding: quoted-printable I have now committed this (mostly .texi) patch as Rev. r14-2434-g8c2fc744a25ec4 Changes to my previously posted version: Fixed a typo in .texi and in the changelog, tweaked the wording for {nearest} to sound better and to provide more details. Tobias On 11.07.23 00:07, Tobias Burnus wrote: > I noted that all memory spaces are supported, some by falling > back to the default ("malloc") - except for omp_high_bw_mem_space > (unless the memkind lib is available). > > I think it makes more sense to fallback to 'malloc' also for > omp_high_bw_mem_space. > > Additionally, I updated the documentation to more explicitly state > what the current implementation is. ----------------- Siemens Electronic Design Automation GmbH; Anschrift: Arnulfstra=C3=9Fe 201= , 80634 M=C3=BCnchen; Gesellschaft mit beschr=C3=A4nkter Haftung; Gesch=C3= =A4ftsf=C3=BChrer: Thomas Heurung, Frank Th=C3=BCrauf; Sitz der Gesellschaf= t: M=C3=BCnchen; Registergericht M=C3=BCnchen, HRB 106955 --------------mo69vF8320UuFYhbVs0uuyXI Content-Type: text/x-patch; charset="UTF-8"; name="committed.diff" Content-Disposition: attachment; filename="committed.diff" Content-Transfer-Encoding: base64 Y29tbWl0IDhjMmZjNzQ0YTI1ZWM0MjNhYjFhMzE3YmY0ZTdkMjQzMTVjNDAwMjQKQXV0aG9y OiBUb2JpYXMgQnVybnVzIDx0b2JpYXNAY29kZXNvdXJjZXJ5LmNvbT4KRGF0ZTogICBUdWUg SnVsIDExIDE2OjExOjM1IDIwMjMgKzAyMDAKCiAgICBsaWJnb21wOiBVcGRhdGUgT3Blbk1Q IG1lbW9yeSBhbGxvY2F0aW9uIGRvYywgZml4IG9tcF9oaWdoX2J3X21lbV9zcGFjZQogICAg CiAgICBsaWJnb21wLwogICAgCiAgICAgICAgICAgICogYWxsb2NhdG9yLmMgKG9tcF9pbml0 X2FsbG9jYXRvcik6IFVzZSBtYWxsb2MgZm9yCiAgICAgICAgICAgIG9tcF9oaWdoX2J3X21l bV9zcGFjZSB3aGVuIHRoZSBtZW1raW5kIGxpYiBpcyB1bmF2YWlsYWJsZQogICAgICAgICAg ICBpbnN0ZWFkIG9mIHJldHVybmluZyBvbXBfbnVsbF9hbGxvY2F0b3IuCiAgICAgICAgICAg ICogbGliZ29tcC50ZXhpIChPcGVuTVAgNS4wKTogRml4IHR5cG8uCiAgICAgICAgICAgIChN ZW1vcnkgYWxsb2NhdGlvbiB3aXRoIGxpYm1lbWtpbmQpOiBEb2N1bWVudCBpbXBsZW1lbnRh dGlvbgogICAgICAgICAgICBpbiBtb3JlIGRldGFpbC4KLS0tCiBsaWJnb21wL2FsbG9jYXRv ci5jICB8ICAyICstCiBsaWJnb21wL2xpYmdvbXAudGV4aSB8IDMwICsrKysrKysrKysrKysr KysrKysrKysrKysrKystLQogMiBmaWxlcyBjaGFuZ2VkLCAyOSBpbnNlcnRpb25zKCspLCAz IGRlbGV0aW9ucygtKQoKZGlmZiAtLWdpdCBhL2xpYmdvbXAvYWxsb2NhdG9yLmMgYi9saWJn b21wL2FsbG9jYXRvci5jCmluZGV4IGM0OTkzMWNiYWQ0Li4yNWMwZjE1MDMwMiAxMDA2NDQK LS0tIGEvbGliZ29tcC9hbGxvY2F0b3IuYworKysgYi9saWJnb21wL2FsbG9jYXRvci5jCkBA IC0zMDEsNyArMzAxLDcgQEAgb21wX2luaXRfYWxsb2NhdG9yIChvbXBfbWVtc3BhY2VfaGFu ZGxlX3QgbWVtc3BhY2UsIGludCBudHJhaXRzLAogCSAgYnJlYWs7CiAJfQogI2VuZGlmCi0g ICAgICByZXR1cm4gb21wX251bGxfYWxsb2NhdG9yOworICAgICAgYnJlYWs7CiAgICAgY2Fz ZSBvbXBfbGFyZ2VfY2FwX21lbV9zcGFjZToKICNpZmRlZiBMSUJHT01QX1VTRV9NRU1LSU5E CiAgICAgICBtZW1raW5kX2RhdGEgPSBnb21wX2dldF9tZW1raW5kICgpOwpkaWZmIC0tZ2l0 IGEvbGliZ29tcC9saWJnb21wLnRleGkgYi9saWJnb21wL2xpYmdvbXAudGV4aQppbmRleCA3 ZDI3Y2M1MGRmNS4uZDFhNWU2NzMyOWEgMTAwNjQ0Ci0tLSBhL2xpYmdvbXAvbGliZ29tcC50 ZXhpCisrKyBiL2xpYmdvbXAvbGliZ29tcC50ZXhpCkBAIC0xOTIsNyArMTkyLDcgQEAgVGhl IE9wZW5NUCA0LjUgc3BlY2lmaWNhdGlvbiBpcyBmdWxseSBzdXBwb3J0ZWQuCiAgICAgICBl bnYgdmFyaWFibGUgQHRhYiBZIEB0YWIKIEBpdGVtIE5lc3RlZC1wYXJhbGxlbCBjaGFuZ2Vz IHRvIEB2YXJ7bWF4LWFjdGl2ZS1sZXZlbHMtdmFyfSBJQ1YgQHRhYiBZIEB0YWIKIEBpdGVt IEBjb2Rle3JlcXVpcmVzfSBkaXJlY3RpdmUgQHRhYiBQCi0gICAgICBAdGFiIGNvbXBsZXRl IGJ1dCBubyBub24taG9zdCBkZXZpY2VzIHByb3ZpZGVzIEBjb2Rle3VuaWZpZWRfc2hhcmVk X21lbW9yeX0KKyAgICAgIEB0YWIgY29tcGxldGUgYnV0IG5vIG5vbi1ob3N0IGRldmljZSBw cm92aWRlcyBAY29kZXt1bmlmaWVkX3NoYXJlZF9tZW1vcnl9CiBAaXRlbSBAY29kZXt0ZWFt c30gY29uc3RydWN0IG91dHNpZGUgYW4gZW5jbG9zaW5nIHRhcmdldCByZWdpb24gQHRhYiBZ IEB0YWIKIEBpdGVtIE5vbi1yZWN0YW5ndWxhciBsb29wIG5lc3RzIEB0YWIgUCBAdGFiIEZ1 bGwgc3VwcG9ydCBmb3IgQy9DKyssIHBhcnRpYWwgZm9yIEZvcnRyYW4KIEBpdGVtIEBjb2Rl eyE9fSBhcyByZWxhdGlvbmFsLW9wIGluIGNhbm9uaWNhbCBsb29wIGZvcm0gZm9yIEMvQysr IEB0YWIgWSBAdGFiCkBAIC00NjM0LDYgKzQ2MzQsMTcgQEAgc21hbGxlciBudW1iZXIuICBP biBub24taG9zdCBkZXZpY2VzLCB0aGUgdmFsdWUgb2YgdGhlCiBAbm9kZSBNZW1vcnkgYWxs b2NhdGlvbiB3aXRoIGxpYm1lbWtpbmQKIEBzZWN0aW9uIE1lbW9yeSBhbGxvY2F0aW9uIHdp dGggbGlibWVta2luZAogCitGb3IgdGhlIG1lbW9yeSBzcGFjZXMsIHRoZSBmb2xsb3dpbmcg YXBwbGllczoKK0BpdGVtaXplCitAaXRlbSBAY29kZXtvbXBfZGVmYXVsdF9tZW1fc3BhY2V9 IGlzIHN1cHBvcnRlZAorQGl0ZW0gQGNvZGV7b21wX2NvbnN0X21lbV9zcGFjZX0gbWFwcyB0 byBAY29kZXtvbXBfZGVmYXVsdF9tZW1fc3BhY2V9CitAaXRlbSBAY29kZXtvbXBfbG93X2xh dF9tZW1fc3BhY2V9IG1hcHMgdG8gQGNvZGV7b21wX2RlZmF1bHRfbWVtX3NwYWNlfQorQGl0 ZW0gQGNvZGV7b21wX2xhcmdlX2NhcF9tZW1fc3BhY2V9IG1hcHMgdG8gQGNvZGV7b21wX2Rl ZmF1bHRfbWVtX3NwYWNlfSwKKyAgICAgIHVubGVzcyB0aGUgbWVta2luZCBsaWJyYXJ5IGlz IGF2YWlsYWJsZQorQGl0ZW0gQGNvZGV7b21wX2hpZ2hfYndfbWVtX3NwYWNlfSBtYXBzIHRv IEBjb2Rle29tcF9kZWZhdWx0X21lbV9zcGFjZX0sCisgICAgICB1bmxlc3MgdGhlIG1lbWtp bmQgbGlicmFyeSBpcyBhdmFpbGFibGUKK0BlbmQgaXRlbWl6ZQorCiBPbiBMaW51eCBzeXN0 ZW1zLCB3aGVyZSB0aGUgQHVyZWZ7aHR0cHM6Ly9naXRodWIuY29tL21lbWtpbmQvbWVta2lu ZCwgbWVta2luZAogbGlicmFyeX0gKEBjb2Rle2xpYm1lbWtpbmQuc28uMH0pIGlzIGF2YWls YWJsZSBhdCBydW50aW1lLCBpdCBpcyB1c2VkIHdoZW4KIGNyZWF0aW5nIG1lbW9yeSBhbGxv Y2F0b3JzIHJlcXVlc3RpbmcKQEAgLTQ2NDEsOSArNDY1MiwyNCBAQCBjcmVhdGluZyBtZW1v cnkgYWxsb2NhdG9ycyByZXF1ZXN0aW5nCiBAaXRlbWl6ZQogQGl0ZW0gdGhlIG1lbW9yeSBz cGFjZSBAY29kZXtvbXBfaGlnaF9id19tZW1fc3BhY2V9CiBAaXRlbSB0aGUgbWVtb3J5IHNw YWNlIEBjb2Rle29tcF9sYXJnZV9jYXBfbWVtX3NwYWNlfQotQGl0ZW0gdGhlIHBhcnRpdGlv biB0cmFpdCBAY29kZXtvbXBfYXR2X2ludGVybGVhdmVkfQorQGl0ZW0gdGhlIHBhcnRpdGlv biB0cmFpdCBAY29kZXtvbXBfYXR2X2ludGVybGVhdmVkfTsgbm90ZSB0aGF0IGZvcgorICAg ICAgQGNvZGV7b21wX2xhcmdlX2NhcF9tZW1fc3BhY2V9IHRoZSBhbGxvY2F0aW9uIHdpbGwg bm90IGJlIGludGVybGVhdmVkCiBAZW5kIGl0ZW1pemUKIAorQWRkaXRpb25hbCBub3RlczoK K0BpdGVtaXplCitAaXRlbSBUaGUgQGNvZGV7cGlubmVkfSB0cmFpdCBpcyB1bnN1cHBvcnRl ZC4KK0BpdGVtIEZvciB0aGUgQGNvZGV7cGFydGl0aW9ufSB0cmFpdCwgdGhlIHBhcnRpdGlv biBwYXJ0IHNpemUgd2lsbCBiZSB0aGUgc2FtZQorICAgICAgYXMgdGhlIHJlcXVlc3RlZCBz aXplIChpLmUuIEBjb2Rle2ludGVybGVhdmVkfSBvciBAY29kZXtibG9ja2VkfSBoYXMgbm8K KyAgICAgIGVmZmVjdCksIGV4Y2VwdCBmb3IgQGNvZGV7aW50ZXJsZWF2ZWR9IHdoZW4gdGhl IG1lbWtpbmQgbGlicmFyeSBpcworICAgICAgYXZhaWxhYmxlLiAgRnVydGhlcm1vcmUsIGZv ciBAY29kZXtuZWFyZXN0fSB0aGUgbWVtb3J5IG1pZ2h0IG5vdCBiZQorICAgICAgb24gdGhl IHNhbWUgTlVNQSBub2RlIGFzIHRocmVhZCB0aGF0IGFsbG9jYXRlZCB0aGUgbWVtb3J5OyBv biBMaW51eCwKKyAgICAgIHRoaXMgaXMgaW4gcGFydGljdWxhciB0aGUgY2FzZSB3aGVuIHRo ZSBtZW1vcnkgcGxhY2VtZW50IHBvbGljeSBpcworICAgICAgc2V0IHRvIHByZWZlcnJlZC4K K0BpdGVtIFRoZSBAY29kZXthY2Nlc3N9IHRyYWl0IGhhcyBubyBlZmZlY3Qgc3VjaCB0aGF0 IG1lbW9yeSBpcyBhbHdheXMKKyAgICAgIGFjY2Vzc2libGUgYnkgYWxsIHRocmVhZHMuCitA aXRlbSBUaGUgQGNvZGV7c3luY19oaW50fSB0cmFpdCBoYXMgbm8gZWZmZWN0LgorQGVuZCBp dGVtaXplCiAKIEBjIC0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0t LS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLQogQGMgT2ZmbG9hZC1UYXJnZXQgU3BlY2lm aWNzCg== --------------mo69vF8320UuFYhbVs0uuyXI--