From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from smtp-out1.suse.de (smtp-out1.suse.de [195.135.220.28]) by sourceware.org (Postfix) with ESMTPS id 2136F3858032 for ; Tue, 22 Feb 2022 14:26:32 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org 2136F3858032 Received: from imap2.suse-dmz.suse.de (imap2.suse-dmz.suse.de [192.168.254.74]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (P-521) server-digest SHA512) (No client certificate requested) by smtp-out1.suse.de (Postfix) with ESMTPS id E0D7F21129; Tue, 22 Feb 2022 14:26:30 +0000 (UTC) Received: from imap2.suse-dmz.suse.de (imap2.suse-dmz.suse.de [192.168.254.74]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (P-521) server-digest SHA512) (No client certificate requested) by imap2.suse-dmz.suse.de (Postfix) with ESMTPS id C749413B98; Tue, 22 Feb 2022 14:26:30 +0000 (UTC) Received: from dovecot-director2.suse.de ([192.168.254.65]) by imap2.suse-dmz.suse.de with ESMTPSA id tSdLL5byFGIPAgAAMHmgww (envelope-from ); Tue, 22 Feb 2022 14:26:30 +0000 Content-Type: multipart/mixed; boundary="------------KWcoKzS36N0WmJdrq9pf5w2L" Message-ID: Date: Tue, 22 Feb 2022 15:26:30 +0100 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.4.1 Subject: Re: [Patch] nvptx: Add -mptx=6.0 + -misa=sm_70 Content-Language: en-US To: Tobias Burnus , gcc-patches References: From: Tom de Vries In-Reply-To: X-Spam-Status: No, score=-12.3 required=5.0 tests=BAYES_00, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, DKIM_VALID_EF, GIT_PATCH_0, KAM_NUMSUBJECT, NICE_REPLY_A, SPF_HELO_NONE, SPF_PASS, TXREP, T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.4 X-Spam-Checker-Version: SpamAssassin 3.4.4 (2020-01-24) on server2.sourceware.org X-BeenThere: gcc-patches@gcc.gnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Gcc-patches mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 22 Feb 2022 14:26:33 -0000 This is a multi-part message in MIME format. --------------KWcoKzS36N0WmJdrq9pf5w2L Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit On 2/17/22 18:24, Tobias Burnus wrote: > PTX version (-mptx=) > [patch adds -mptx=6.0 as option] > > * Currently supported internally are 3.1 (CUDA 5.0, used by GCC <= 11), >   6.0 (CUDA 9.0, current GCC 12 default), 6.3 (CUDA 10.0), 7.0 (CUDA 11.0) > * -mptx= supports 3.1, 6.3, 7.0 – but not the internal default 6.0 > I tend not to think in terms of CUDA versions, but supported driver versions. In the end, drivers are used to translate ptx to SASS for execution, CUDA is just used for build time verification (or not, if it's not in the path). And a driver may or may not be supported. F.i. 390.x still may receive updates from nvidia, but there are JIT bugs that we've reported that they've decided not to fix, so from that point of view 390.x is unsupported. > I think it makes sense to expose the 6.0 value to the user and not > only use it internally behind the scenes. As it is already used internally, > the change is tiny but user visible. Sure, I've committed this (with a somewhat shorter commit log). >Thus, it has to stay when we will > bump the default in later GCC versions; on the other hand, if we bump > the default, it might be also a good reason to have it to permit the > user to have a backward compatible PTX output for linking libraries. > FWIW, I think that it's possible to link different versions of ptx isa together (though perhaps there are specific scenarios where that's not possible, I'm not sure). But mixing versions restricts the range of drivers you can use, so it may make sense to just use one version. Thanks, - Tom --------------KWcoKzS36N0WmJdrq9pf5w2L Content-Type: text/x-patch; charset=UTF-8; name="0001-nvptx-Add-mptx-6.0.patch" Content-Disposition: attachment; filename="0001-nvptx-Add-mptx-6.0.patch" Content-Transfer-Encoding: base64 bnZwdHg6IEFkZCAtbXB0eD02LjAKCkN1cnJlbnRseSBzdXBwb3J0ZWQgaW50ZXJuYWxseSBh cmUgMy4xLCA2LjAsIDYuMyBhbmQgNy4wLgoKSG93ZXZlciwgLW1wdHg9IHN1cHBvcnRzIDMu MSwgNi4zLCA3LjAg4oCTIGJ1dCBub3QgdGhlIGludGVybmFsIGRlZmF1bHQgNi4wLgoKQWRk IC1tcHR4PTYuMCBmb3IgY29uc2lzdGVuY3kuCgpUZXN0ZWQgb24gbnZwdHguCgpnY2MvQ2hh bmdlTG9nOgoKCSogY29uZmlnL252cHR4L252cHR4Lm9wdCAobXB0eCk6IEFkZCA2LjAgYWxp YXMgUFRYX1ZFUlNJT05fNl8wLgoJKiBkb2MvaW52b2tlLnRleGkgKC1tcHR4KTogVXBkYXRl IGZvciBuZXcgdmFsdWVzIGFuZCBkZWZhdWx0cy4KCkNvLUF1dGhvcmVkLUJ5OiBUb20gZGUg VnJpZXMgPHRkZXZyaWVzQHN1c2UuZGU+CgotLS0KIGdjYy9jb25maWcvbnZwdHgvbnZwdHgu b3B0IHwgMyArKysKIGdjYy9kb2MvaW52b2tlLnRleGkgICAgICAgIHwgNyArKysrLS0tCiAy IGZpbGVzIGNoYW5nZWQsIDcgaW5zZXJ0aW9ucygrKSwgMyBkZWxldGlvbnMoLSkKCmRpZmYg LS1naXQgYS9nY2MvY29uZmlnL252cHR4L252cHR4Lm9wdCBiL2djYy9jb25maWcvbnZwdHgv bnZwdHgub3B0CmluZGV4IGU1NmVjOTI4OGRhLi45N2UxMjdjYzRmYiAxMDA2NDQKLS0tIGEv Z2NjL2NvbmZpZy9udnB0eC9udnB0eC5vcHQKKysrIGIvZ2NjL2NvbmZpZy9udnB0eC9udnB0 eC5vcHQKQEAgLTgyLDYgKzgyLDkgQEAgS25vd24gUFRYIHZlcnNpb25zIChmb3IgdXNlIHdp dGggdGhlIC1tcHR4PSBvcHRpb24pOgogRW51bVZhbHVlCiBFbnVtKHB0eF92ZXJzaW9uKSBT dHJpbmcoMy4xKSBWYWx1ZShQVFhfVkVSU0lPTl8zXzEpCiAKK0VudW1WYWx1ZQorRW51bShw dHhfdmVyc2lvbikgU3RyaW5nKDYuMCkgVmFsdWUoUFRYX1ZFUlNJT05fNl8wKQorCiBFbnVt VmFsdWUKIEVudW0ocHR4X3ZlcnNpb24pIFN0cmluZyg2LjMpIFZhbHVlKFBUWF9WRVJTSU9O XzZfMykKIApkaWZmIC0tZ2l0IGEvZ2NjL2RvYy9pbnZva2UudGV4aSBiL2djYy9kb2MvaW52 b2tlLnRleGkKaW5kZXggNjM1YzVmNzkyNzguLjU2ZjNhMDFkZTQ0IDEwMDY0NAotLS0gYS9n Y2MvZG9jL2ludm9rZS50ZXhpCisrKyBiL2djYy9kb2MvaW52b2tlLnRleGkKQEAgLTI3Mjg2 LDkgKzI3Mjg2LDEwIEBAIHN0cmluZ3MgbXVzdCBiZSBsb3dlci1jYXNlLiAgVmFsaWQgSVNB IHN0cmluZ3MgaW5jbHVkZSBAc2FtcHtzbV8zMH0gYW5kCiAKIEBpdGVtIC1tcHR4PUB2YXJ7 dmVyc2lvbi1zdHJpbmd9CiBAb3BpbmRleCBtcHR4Ci1HZW5lcmF0ZSBjb2RlIGZvciBnaXZl biB0aGUgc3BlY2lmaWVkIFBUWCB2ZXJzaW9uIChlLmcuQDogQHNhbXB7Ni4zfSkuCi1WYWxp ZCB2ZXJzaW9uIHN0cmluZ3MgaW5jbHVkZSBAc2FtcHszLjF9IGFuZCBAc2FtcHs2LjN9LiAg VGhlIGRlZmF1bHQgUFRYCi12ZXJzaW9uIGlzIDMuMS4KK0dlbmVyYXRlIGNvZGUgZm9yIGdp dmVuIHRoZSBzcGVjaWZpZWQgUFRYIHZlcnNpb24gKGUuZy5AOiBAc2FtcHs3LjB9KS4KK1Zh bGlkIHZlcnNpb24gc3RyaW5ncyBpbmNsdWRlIEBzYW1wezMuMX0sIEBzYW1wezYuMH0sIEBz YW1wezYuM30sIGFuZAorQHNhbXB7Ny4wfS4gIFRoZSBkZWZhdWx0IFBUWCB2ZXJzaW9uIGlz IDYuMCwgdW5sZXNzIGEgaGlnaGVyIG1pbmltYWwKK3ZlcnNpb24gaXMgcmVxdWlyZWQgZm9y IHNwZWNpZmllZCBQVFggSVNBIHZpYSBvcHRpb24gQG9wdGlvbnstbWlzYT19LgogCiBAaXRl bSAtbW1haW5rZXJuZWwKIEBvcGluZGV4IG1tYWlua2VybmVsCg== --------------KWcoKzS36N0WmJdrq9pf5w2L--