From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 21934 invoked by alias); 22 Aug 2017 14:56:13 -0000 Mailing-List: contact gdb-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-owner@sourceware.org Received: (qmail 21894 invoked by uid 89); 22 Aug 2017 14:56:12 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-1.5 required=5.0 tests=AWL,BAYES_00,MIME_BASE64_BLANKS,RCVD_IN_DNSWL_NONE,RCVD_IN_SORBS_SPAM autolearn=no version=3.3.2 spammy=Anyone, H*r:ip*192.168.0.101, sourceware1, dje X-HELO: mail-wr0-f178.google.com Received: from mail-wr0-f178.google.com (HELO mail-wr0-f178.google.com) (209.85.128.178) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Tue, 22 Aug 2017 14:56:10 +0000 Received: by mail-wr0-f178.google.com with SMTP id p14so59371697wrg.1 for ; Tue, 22 Aug 2017 07:56:09 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:subject:message-id:date:user-agent :mime-version:content-transfer-encoding; bh=pxoPQLfQwbU9rB7aJnmsJcQRPS4bec7p6UzPVdSIMOM=; b=BWvNr4pOY9ja70yDt4L+XHq+LwbYG3JTjG4Z7MBrpcuah2ZwAMyhZPdLsHEcBisDwO Ozm4+WZOJ19jBOM9MirhWiWDmfJoRpZuVP7dH+v0Lx0P/gNq3pi4xm9gsFecbnI5sKEl p+3YEVCHW8HyPvohrfizHNQDXV8hxG6oR35EzGMLLrb2OR+sav35xidAiC4BO+9Z9poz T6GV6/KxLc8vunJTWNghSa9zXAZuBIfOS4c72/ms6+vv/Q7irLZpMZ3NcxAogxX2eGrE AgqPFCXRjYldGRxUAzp3RZcW1FITzsVRkiqPAYHKTnTmRSIb9op5LeWsStngl51cx/vS Lh8w== X-Gm-Message-State: AHYfb5gQ62NPeVXzXLJWEatkg57aSBrcanSJB5XVBoGPSUJkCyVictUy og+cwWk5RBU7psd9 X-Received: by 10.223.128.42 with SMTP id 39mr610622wrk.247.1503413768025; Tue, 22 Aug 2017 07:56:08 -0700 (PDT) Received: from [192.168.0.101] (bl17-148-124.dsl.telepac.pt. [188.82.148.124]) by smtp.gmail.com with ESMTPSA id b47sm12354792wra.64.2017.08.22.07.56.06 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Tue, 22 Aug 2017 07:56:07 -0700 (PDT) From: Pedro Alves To: "binutils@sourceware.org" , "gdb@sourceware.org" , Joel Brobecker Subject: can't push to binutils-gdb, git-hooks rejecting commits Message-ID: Date: Tue, 22 Aug 2017 14:56:00 -0000 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.4.0 MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: base64 X-SW-Source: 2017-08/txt/msg00046.txt.bz2 SGksDQoNCkknbSB0cnlpbmcgdG8gcHVzaCBhIHBhdGNoIHRvIG1hc3Rlciwg YnV0IGl0J3MgZ2V0dGluZyByZWplY3RlZCBieSB0aGUNCnNlcnZlci1zaWRl IGdpdCBob29rcy4gIEFueW9uZSBlbHNlIHNlZWluZyB0aGUgc2FtZT8gIFNl ZSBiZWxvdy4NCg0KUmVwZWF0aW5nIHRoYXQgYmlnICJnaXQgcmV2LWxpc3Qi IGNvbW1hbmQgYmVsb3cgbG9jYWxseSBzaG93czoNCg0KICBmYXRhbDogYmFk IHJldmlzaW9uICdecmVmcy9oZWFkcy91c2Vycy9oamwvcHIxNzcwOScNCg0K VGhhdCBicmFuY2ggZG9lc24ndCBhcHBlYXIgdG8gZXhpc3QgYW55bW9yZS4g ICBTb3VuZHMgbGlrZSB0aGUgaG9va3MNCm5lZWQgdG8gcnVuICJnaXQgcmVt b3RlIHBydW5lIG9yaWdpbiIgb24gdGhlaXIgY2xvbmU/ICBEb2VzIHRoYXQg bWFrZQ0Kc2Vuc2U/DQoNCkpvZWwsIGFueSBpZGVhcz8NCg0KJCBnaXQgcHVz aCBzc2g6Ly9wYWx2ZXNAc291cmNld2FyZS5vcmcvZ2l0L2JpbnV0aWxzLWdk Yi5naXQgSEVBRDptYXN0ZXINCkNvdW50aW5nIG9iamVjdHM6IDYsIGRvbmUu DQpEZWx0YSBjb21wcmVzc2lvbiB1c2luZyB1cCB0byA4IHRocmVhZHMuDQpD b21wcmVzc2luZyBvYmplY3RzOiAxMDAlICg2LzYpLCBkb25lLg0KV3JpdGlu ZyBvYmplY3RzOiAxMDAlICg2LzYpLCAxLjE5IEtpQiB8IDAgYnl0ZXMvcywg ZG9uZS4NClRvdGFsIDYgKGRlbHRhIDUpLCByZXVzZWQgMCAoZGVsdGEgMCkN CnJlbW90ZTogVHJhY2ViYWNrIChtb3N0IHJlY2VudCBjYWxsIGxhc3QpOg0K cmVtb3RlOiAgIEZpbGUgImhvb2tzL3VwZGF0ZS5weSIsIGxpbmUgNjksIGlu IDxtb2R1bGU+DQpyZW1vdGU6ICAgICBjaGVja191cGRhdGUoYXJncy5yZWZf bmFtZSwgYXJncy5vbGRfcmV2LCBhcmdzLm5ld19yZXYpDQpyZW1vdGU6ICAg RmlsZSAiaG9va3MvdXBkYXRlLnB5IiwgbGluZSA2MiwgaW4gY2hlY2tfdXBk YXRlDQpyZW1vdGU6ICAgICB1cGRhdGVfY2xzLnZhbGlkYXRlKCkNCnJlbW90 ZTogICBGaWxlICIvc291cmNld2FyZTEvcHJvamVjdHMvc3JjLWhvbWUvZ2l0 LWhvb2tzL2hvb2tzL3VwZGF0ZXMvX19pbml0X18ucHkiLCBsaW5lIDEyMywg aW4gdmFsaWRhdGUNCnJlbW90ZTogICAgIHNlbGYuX19jaGVja19tYXhfY29t bWl0X2VtYWlscygpDQpyZW1vdGU6ICAgRmlsZSAiL3NvdXJjZXdhcmUxL3By b2plY3RzL3NyYy1ob21lL2dpdC1ob29rcy9ob29rcy91cGRhdGVzL19faW5p dF9fLnB5IiwgbGluZSA2NTMsIGluIF9fY2hlY2tfbWF4X2NvbW1pdF9lbWFp bHMNCnJlbW90ZTogICAgIHNlbGYuX19zZXRfc2VuZF9lbWFpbF9wX2F0dHIo c2VsZi5hZGRlZF9jb21taXRzKQ0KcmVtb3RlOiAgIEZpbGUgIi9zb3VyY2V3 YXJlMS9wcm9qZWN0cy9zcmMtaG9tZS9naXQtaG9va3MvaG9va3MvdXBkYXRl cy9fX2luaXRfXy5weSIsIGxpbmUgNzEzLCBpbiBfX3NldF9zZW5kX2VtYWls X3BfYXR0cg0KcmVtb3RlOiAgICAgX3NwbGl0X2xpbmVzPVRydWUpDQpyZW1v dGU6ICAgRmlsZSAiL3NvdXJjZXdhcmUxL3Byb2plY3RzL3NyYy1ob21lL2dp dC1ob29rcy9ob29rcy9naXQucHkiLCBsaW5lIDEzMiwgaW4gZg0KcmVtb3Rl OiAgICAgcmV0dXJuIGdpdF9ydW4oY29tbWFuZCwgKmFyZ3MsICoqa3dhcmdz KQ0KcmVtb3RlOiAgIEZpbGUgIi9zb3VyY2V3YXJlMS9wcm9qZWN0cy9zcmMt aG9tZS9naXQtaG9va3MvaG9va3MvZ2l0LnB5IiwgbGluZSAxMDQsIGluIGdp dF9ydW4NCnJlbW90ZTogICAgIG91dHB1dCkNCnJlbW90ZTogZ2l0LkNhbGxl ZFByb2Nlc3NFcnJvcjogQ29tbWFuZCAnZ2l0IHJldi1saXN0IDBkOTk5YTZl ZjBmOThiMjI0MzBkNzA5NTE0MDg4Njk4NjRjOTc5ZTAgXnJlZnMvaGVhZHMv dXNlcnMvaGpsL3ByMTc3MDkgXnJlZnMvaGVhZHMvdXNlcnMvaGpsL2NvbXBy ZXNzZWQgXnJlZnMvaGVhZHMvdXNlcnMvaWJtL2JpbnV0aWxzLTJfMjUgXnJl ZnMvaGVhZHMvdXNlcnMvaWJtL2JpbnV0aWxzLTJfMjQgXnJlZnMvaGVhZHMv dXNlcnMvcGFsdmVzL2N4eC1lbGltaW5hdGUtY2xlYW51cHMgXnJlZnMvaGVh ZHMvdXNlcnMvaGpsL21weC9wbHQyIF5yZWZzL2hlYWRzL3VzZXJzL2hqbC9i cmFuY2ggXnJlZnMvaGVhZHMvdXNlcnMvYXJuZXovbGszIF5yZWZzL2hlYWRz L3VzZXJzL2hqbC9keW5vYmogXnJlZnMvaGVhZHMvdXNlcnMvYXBpbnNraS9n ZGItaWxwMzItZ2Ric2VydmVyIF5yZWZzL2hlYWRzL3VzZXJzL2Njb3V0YW50 L3R3by1sZXZlbC1saW5lLTE1MDMzMSBecmVmcy9oZWFkcy91c2Vycy9oamwv dmVyc2lvbiBecmVmcy9oZWFkcy91c2Vycy9iaGVja2VsL2ZvcnRyYW4tdmxh LXN0cmluZ3MgXnJlZnMvaGVhZHMvdXNlcnMvcGFsdmVzL3RocmVhZF9hcHBs eS12MiBecmVmcy9oZWFkcy91c2Vycy9hcm5lei9sayBecmVmcy9oZWFkcy91 c2Vycy9wYWx2ZXMvd2hhdGlzIF5yZWZzL2hlYWRzL3VzZXJzL3BhbHZlcy9j eHgtdGFyZ2V0X29wcyBecmVmcy9oZWFkcy91c2Vycy9oamwvb3V0cHV0IF5y ZWZzL2hlYWRzL3VzZXJzL2hqbC9zZWNvbmRhcnkvbWFzdGVyIF5yZWZzL2hl YWRzL3VzZXJzL2hqbC9tcHgvcGx0ZXh0OCBecmVmcy9oZWFkcy91c2Vycy9o amwvcHIxMjYzOSBecmVmcy9oZWFkcy91c2Vycy9wYWx2ZXMvY29yZS1wcnN0 YXR1cy1yZXdvcmsgXnJlZnMvaGVhZHMvdXNlcnMvaGpsL3ByMTg1OTkgXnJl ZnMvaGVhZHMvdXNlcnMvaGpsL2dvdCBecmVmcy9oZWFkcy91c2Vycy9td2sv czM5MC1yZWxyby1vbGQgXnJlZnMvaGVhZHMvdXNlcnMvaGpsL3ByMTgwMjgg XnJlZnMvaGVhZHMvdXNlcnMvcGFsdmVzL2N4eC1nZGJfZXhjZXB0aW9ucyBe cmVmcy9oZWFkcy91c2Vycy9zaW1hcmsvdXNlci1zZWxlY3Rpb24tcmZjIF5y ZWZzL2hlYWRzL3VzZXJzL2hqbC9saW5rX2Fzc2lnbm1lbnRzIF5yZWZzL2hl YWRzL3VzZXJzL2hsL3ByMjE2NzkgXnJlZnMvaGVhZHMvdXNlcnMvcGluc2tp YS9nZGItYWFyY2g2NC1pbHAzMiBecmVmcy9oZWFkcy91c2Vycy9ydGgveDg2 LXNlZyBecmVmcy9oZWFkcy91c2Vycy9oamwvZ29sZC10ZXN0IF5yZWZzL2hl YWRzL3VzZXJzL3NpbWFyay9wdGlkLWluLWZldGNoLXN0b3JlLXJlZ3MgXnJl ZnMvaGVhZHMvdXNlcnMvYmhlY2tlbC9mb3J0cmFuLW9vcC1leHRlbmQtdHlw ZSBecmVmcy9oZWFkcy91c2Vycy9wYWx2ZXMvZHdhcmYyX3Blcl9vYmpmaWxl IF5yZWZzL2hlYWRzL3VzZXJzL3BhbHZlcy9maXgtZG91YmxlLWZyZWUgXnJl ZnMvaGVhZHMvdXNlcnMvaGpsL25vaXNhIF5yZWZzL2hlYWRzL3VzZXJzL2Nj b3V0YW50L2V4cGVyaW1lbnRhbC1yZWxyIF5yZWZzL2hlYWRzL3VzZXJzL3Bh bHZlcy9pbnRlcnJ1cHQtd2hpbGUtc3RlcC1vdmVyLXYxIF5yZWZzL2hlYWRz L3VzZXJzL213ay9zMzkwLXJlbHJvLXJlbCBecmVmcy9oZWFkcy91c2Vycy9w YWx2ZXMvaml0LXNwZWVkdXAgXnJlZnMvaGVhZHMvdXNlcnMvQVJNL2VtYmVk ZGVkLWdkYi03LjEwLWJyYW5jaCBecmVmcy9oZWFkcy91c2Vycy9oamwvbXB4 L3BsdDMyIF5yZWZzL2hlYWRzL3VzZXJzL2hqbC9tcHgvcGx0ZXh0MTYgXnJl ZnMvaGVhZHMvdXNlcnMvYW9saXZhL1NGTiBecmVmcy9oZWFkcy91c2Vycy9h bGFoYXkwMS90YXJnZXRyZWdjYWNoZSBecmVmcy9oZWFkcy91c2Vycy9wYWx2 ZXMvZHdhcmY1LXJlZ3Jlc3Npb24tZm9yLWphbiBecmVmcy9oZWFkcy91c2Vy cy9oamwvY29tcHJlc3MgXnJlZnMvaGVhZHMvdXNlcnMvaGpsL2x0by1taXhl ZC9tYXN0ZXIgXnJlZnMvaGVhZHMvdXNlcnMvaGpsL3ByMjE4ODQgXnJlZnMv aGVhZHMvdXNlcnMvaGpsL2xpbnV4L21hc3RlciBecmVmcy9oZWFkcy91c2Vy cy9jY291dGFudC90d28tbGV2ZWwtbGluZS10YWJsZXMgXnJlZnMvaGVhZHMv dXNlcnMvcGFsdmVzL2xpbmVfaGVhZGVyX2xlYWsgXnJlZnMvaGVhZHMvdXNl cnMvcWl5YW8vdGFyZ2V0LWRlc2MtMiBecmVmcy9oZWFkcy91c2Vycy9xaXlh by90YXJnZXQtZGVzYy0zIF5yZWZzL2hlYWRzL3VzZXJzL0FSTS9zdmUgXnJl ZnMvaGVhZHMvdXNlcnMvaGpsL3ByMjAyNzYgXnJlZnMvaGVhZHMvdXNlcnMv cGFsdmVzL2N4eC1lbnVtLWZsYWdzIF5yZWZzL2hlYWRzL3VzZXJzL2hqbC9w cjE3ODc4IF5yZWZzL2hlYWRzL3VzZXJzL3BwYWxrYS9yZWFkbGluZS03LjAt dXBkYXRlIF5yZWZzL2hlYWRzL3VzZXJzL2hqbC9jZXQvbWFzdGVyIF5yZWZz L2hlYWRzL3VzZXJzL3BhbHZlcy9hbWQ2NC10ZGVzYyBecmVmcy9oZWFkcy91 c2Vycy9iaGVja2VsL2ZvcnRyYW4tZW50cnktcG9pbnQtbmVzdGVkLXN1YnJv dXRpbmVzIF5yZWZzL2hlYWRzL3VzZXJzL2hqbC9wcjE5ODE4IF5yZWZzL2hl YWRzL3VzZXJzL3BhbHZlcy9Xd3JpdGUtc3RyaW5ncy12MS41IF5yZWZzL2hl YWRzL3VzZXJzL2hqbC9ncG9mZiBecmVmcy9oZWFkcy91c2Vycy9wcGFsa2Ev cmVhZGxpbmUtNi4zLXVwZGF0ZSBecmVmcy9oZWFkcy91c2Vycy9oamwvc2hh cmFibGUgXnJlZnMvaGVhZHMvdXNlcnMvcGFsdmVzL2ZpeC1vYmpjLXRlc3Rz IF5yZWZzL2hlYWRzL3VzZXJzL2hqbC9wcm9wZXJ0eSBecmVmcy9oZWFkcy91 c2Vycy9rZWl0aHMvaW50ZWxsaWdlbnQtYnJlYWtwb2ludF9yZV9zZXQgXnJl ZnMvaGVhZHMvdXNlcnMvbXdrL3MzOTAtcmVscm8gXnJlZnMvaGVhZHMvdXNl cnMvaGpsL3RscyBecmVmcy9oZWFkcy91c2Vycy9oamwvZ2RiLTguMC1icmFu Y2ggXnJlZnMvaGVhZHMvdXNlcnMvaGpsL2JpbnV0aWxzLTJfMjUtYnJhbmNo IF5yZWZzL2hlYWRzL3VzZXJzL3BhbHZlcy9nZGJzZXJ2ZXItZXh0cmFjdC11 bnNpZ25lZC1pbnRlZ2VyIF5yZWZzL2hlYWRzL3VzZXJzL3BtdWxkb29uL2Mr K2NvbXBpbGUgXnJlZnMvaGVhZHMvdXNlcnMvQVJNL2VtYmVkZGVkLWJpbnV0 aWxzLTJfMjYtYnJhbmNoIF5yZWZzL2hlYWRzL3VzZXJzL2dlbnRvby8yLjI3 IF5yZWZzL2hlYWRzL3VzZXJzL2dlbnRvby8yLjI4IF5yZWZzL2hlYWRzL3Vz ZXJzL2hqbC9yZWxheC1wbHQgXnJlZnMvaGVhZHMvdXNlcnMvaGpsL3ByMjE5 NjQgXnJlZnMvaGVhZHMvdXNlcnMvaGpsL3ByMjE1NjIgXnJlZnMvaGVhZHMv dXNlcnMvZGplL2dvb2dsZS03LjEwIF5yZWZzL2hlYWRzL3VzZXJzL3BhbHZl cy9jeHgtYnJlYWtwb2ludC1pbXByb3ZlbWVudHMgXnJlZnMvaGVhZHMvdXNl cnMva2VpdGhzL2MrK2NvbXBpbGUtc3VibWl0IF5yZWZzL2hlYWRzL3VzZXJz L2hqbC9tbWFwL3dyaXRlIF5yZWZzL2hlYWRzL3VzZXJzL3BhbHZlcy9maXgt dmFyLXVwZGF0ZS1mb3ItcmVnaXN0ZXJzIF5yZWZzL2hlYWRzL3VzZXJzL2hq bC9jYWNoZSBecmVmcy9oZWFkcy91c2Vycy9jY291dGFudC9nb2xkLXJlY29y ZCBecmVmcy9oZWFkcy91c2Vycy9oamwvcmVsb2NzIF5yZWZzL2hlYWRzL3Vz ZXJzL21tZXR6Z2VyL3JlY29yZC1nb3RvLW1pIF5yZWZzL2hlYWRzL3VzZXJz L3BhbHZlcy9jeHgtZHdhcmYycmVhZCBecmVmcy9oZWFkcy91c2Vycy9oamwv aW5kYnIgXnJlZnMvaGVhZHMvdXNlcnMvcGFsdmVzL2ludGVycnVwdC13aGls ZS1zdGVwLW92ZXIgXnJlZnMvaGVhZHMvdXNlcnMvaGpsL29sZC1sdG8tbWl4 ZWQgXnJlZnMvaGVhZHMvdXNlcnMvaGpsL3ByMTk4MjcvYmludXRpbHMtMl8y Ni1icmFuY2ggXnJlZnMvaGVhZHMvdXNlcnMvaGpsL3ByMTk1NTMvYmludXRp bHMtMl8yNi1icmFuY2ggXnJlZnMvaGVhZHMvdXNlcnMvaGpsL3ByMjE5MDMg XnJlZnMvaGVhZHMvdXNlcnMvaGpsL2xpbnV4L3RyYWNrIF5yZWZzL2hlYWRz L3VzZXJzL3BhbHZlcy9mdW5jdGlvbl92aWV3IF5yZWZzL2hlYWRzL3VzZXJz L2hqbC9saW51eC9hcHBsaWVkIF5yZWZzL2hlYWRzL3VzZXJzL2hqbC9tcHgv bWFzdGVyIF5yZWZzL2hlYWRzL3VzZXJzL3BhbHZlcy9jcC1saW5lc3BlYy1m b3Ita2VpdGhzIF5yZWZzL2hlYWRzL3VzZXJzL2hqbC9wcjIwODgyIF5yZWZz L2hlYWRzL3VzZXJzL2JoZWNrZWwvZm9ydHJhbi1zdHJpZGVzIF5yZWZzL2hl YWRzL3VzZXJzL2hqbC9wbHQgXnJlZnMvaGVhZHMvdXNlcnMvcGtvbmluZy9t dWx0aS10YXJnZXQgXjMyOWQ1ZTdlNTZmODUyZGNmOTk1NjY4M2JlNDM0NWMx NjVkMjc3ZGInIHJldHVybmVkIG5vbi16ZXJvIGV4aXQgc3RhdHVzIDEyOA0K cmVtb3RlOiBlcnJvcjogaG9vayBkZWNsaW5lZCB0byB1cGRhdGUgcmVmcy9o ZWFkcy9tYXN0ZXINClRvIHNzaDovL3BhbHZlc0Bzb3VyY2V3YXJlLm9yZy9n aXQvYmludXRpbHMtZ2RiLmdpdA0KICEgW3JlbW90ZSByZWplY3RlZF0gSEVB RCAtPiBtYXN0ZXIgKGhvb2sgZGVjbGluZWQpDQplcnJvcjogZmFpbGVkIHRv IHB1c2ggc29tZSByZWZzIHRvICdzc2g6Ly9wYWx2ZXNAc291cmNld2FyZS5v cmcvZ2l0L2JpbnV0aWxzLWdkYi5naXQnDQoNClRoYW5rcywNClBlZHJvIEFs dmVzDQo= >From gdb-return-45898-listarch-gdb=sources.redhat.com@sourceware.org Tue Aug 22 15:03:29 2017 Return-Path: Delivered-To: listarch-gdb@sources.redhat.com Received: (qmail 4340 invoked by alias); 22 Aug 2017 15:03:28 -0000 Mailing-List: contact gdb-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-owner@sourceware.org Delivered-To: mailing list gdb@sourceware.org Received: (qmail 2146 invoked by uid 89); 22 Aug 2017 15:03:27 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-3.4 required=5.0 tests=AWL,BAYES_00,RCVD_IN_DNSWL_NONE,RCVD_IN_SORBS_SPAM,SPF_PASS autolearn=no version=3.3.2 spammy= X-HELO: mail-wr0-f170.google.com Received: from mail-wr0-f170.google.com (HELO mail-wr0-f170.google.com) (209.85.128.170) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Tue, 22 Aug 2017 15:03:24 +0000 Received: by mail-wr0-f170.google.com with SMTP id p14so59571362wrg.1 for ; Tue, 22 Aug 2017 08:03:24 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:references:user-agent:from:to:cc:subject :in-reply-to:date:message-id:mime-version:content-transfer-encoding; bh=ljeNmQNCghE76ux0euyTKQfdzPZZSkqrjDjQyXVu5WQ=; b=WWooLgBPfLIxSLIJfikniqFBOq1CFsWsXcwNkGW+IlBvXUAS3jGzWFFtHhX+rCdWTm j1vcFXL0ASNIUD7CqUHyTEQnKaRgQx9KXUJ1/bafXxH3WZh3QdNKmFfLB1dc5gOn711Y O7pyF/eHfjryRFb1wUFh0Mrqg7/aS3ZoQOUeUA9EXJ3gzdoXS1VhRQN954I26uJdqUFs 9TH+op5N39e3I14OAlY7k9yLUOFOa9peX5aoq5b5GVE9ck+MxuTVW+9Hq4SijjQzxAdh Ih6lWkvyNTAqdyfju8bvy7r+pPydlSoBERS+U3bJs2kQCQKqf01qd2VxYfiR8cpTG8Ux cxjA== X-Gm-Message-State: AHYfb5hgsE+kZK3UsALcbvuHTcIa/CSKlegrdhixewkP14WMCAVyeMlU JUpL98uKyKMm9vRB X-Received: by 10.28.37.2 with SMTP id l2mr636321wml.112.1503414202480; Tue, 22 Aug 2017 08:03:22 -0700 (PDT) Received: from zen.linaro.local ([81.128.185.34]) by smtp.gmail.com with ESMTPSA id u127sm7099433wmd.15.2017.08.22.08.03.21 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Tue, 22 Aug 2017 08:03:21 -0700 (PDT) Received: from zen (localhost [127.0.0.1]) by zen.linaro.local (Postfix) with ESMTPS id F20163E00E2; Tue, 22 Aug 2017 16:03:20 +0100 (BST) References: <1502280338-23002-1-git-send-email-Dave.Martin@arm.com> <1502280338-23002-10-git-send-email-Dave.Martin@arm.com> <87y3qb52ez.fsf@linaro.org> <20170822111705.GT6321@e103592.cambridge.arm.com> <87tw0z4sk2.fsf@linaro.org> <20170822142135.GU6321@e103592.cambridge.arm.com> User-agent: mu4e 0.9.19; emacs 25.2.50.3 From: Alex =?utf-8?Q?Benn=C3=A9e?= To: Dave Martin Cc: linux-arch@vger.kernel.org, libc-alpha@sourceware.org, Ard Biesheuvel , Szabolcs Nagy , gdb@sourceware.org, Yao Qi , Will Deacon , Richard Sandiford , Alan Hayward , Catalin Marinas , kvmarm@lists.cs.columbia.edu, linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH 09/27] arm64/sve: Signal frame and context structure definition In-reply-to: <20170822142135.GU6321@e103592.cambridge.arm.com> Date: Tue, 22 Aug 2017 15:03:00 -0000 Message-ID: <87shgj4pc7.fsf@linaro.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit X-IsSubscribed: yes X-SW-Source: 2017-08/txt/msg00047.txt.bz2 Content-length: 4341 Dave Martin writes: > On Tue, Aug 22, 2017 at 02:53:49PM +0100, Alex Bennée wrote: >> >> Dave Martin writes: >> >> > On Tue, Aug 22, 2017 at 11:22:44AM +0100, Alex Bennée wrote: >> >> >> >> Dave Martin writes: > > [...] > >> >> > +/* >> >> > + * The SVE architecture leaves space for future expansion of the >> >> > + * vector length beyond its initial architectural limit of 2048 bits >> >> > + * (16 quadwords). >> >> > + */ >> >> > +#define SVE_VQ_MIN 1 >> >> > +#define SVE_VQ_MAX 0x200 >> >> > + >> >> > +#define SVE_VL_MIN (SVE_VQ_MIN * 0x10) >> >> > +#define SVE_VL_MAX (SVE_VQ_MAX * 0x10) >> >> > + >> >> > +#define SVE_NUM_ZREGS 32 >> >> > +#define SVE_NUM_PREGS 16 >> >> > + >> >> > +#define sve_vl_valid(vl) \ >> >> > + ((vl) % 0x10 == 0 && (vl) >= SVE_VL_MIN && (vl) <= SVE_VL_MAX) >> >> > +#define sve_vq_from_vl(vl) ((vl) / 0x10) >> >> > +#define sve_vl_from_vq(vq) ((vq) * 0x10) >> >> >> >> I got a little confused first time through over what VQ and VL where. >> >> Maybe it would make sense to expand a little more from first principles? >> >> >> >> /* >> >> * The SVE architecture defines vector registers as a multiple of 128 >> >> * bit quadwords. The current architectural limit is 2048 bits (16 >> >> * quadwords) but there is room for future expansion beyond that. >> >> */ >> > >> > This comes up in several places and so I didn't want to comment it >> > repeatedly everywhere. >> > >> > Instead, I wrote up something in section 2 (Vector length terminology) >> > of Documentation/arm64/sve.txt -- see patch 25. Can you take a look and >> > see whether that's adequate? >> >> Ahh, I hadn't got to that yet. I'm unsure to the order the kernel likes >> to put things but I like to put design documents at the front of the > > I don't have a strong opinion on that -- I had preferred not to add a > document describing stuff that doesn't exist at the time of commit. > I could commit a stub document at the start of the series, and then > commit the real document later. > > Either way, it seemed overkill. > > Perhaps I should have drawn more attention to the documentation in the > cover letter, and encouraged reviewers to look at it first. My > experience is that people don't often read cover letters... > > Now the series is posted, I'm minded to keep the order as-is, unless you > think it's a big issue. > > Adding a reference to the document seems a reasonable thing to do, > so I could add that. > >> patch queue as they are useful primers and saves you having to patch a: >> >> modified arch/arm64/include/uapi/asm/sigcontext.h >> @@ -132,19 +132,24 @@ struct sve_context { >> /* >> * The SVE architecture leaves space for future expansion of the >> * vector length beyond its initial architectural limit of 2048 bits >> - * (16 quadwords). >> + * (16 quadwords). See Documentation/arm64/sve.txt for a summary of >> + * the terminology of Vector Quads (VQ) and Vector Lengths (VL). >> */ >> + >> +#define SVE_VQ_BITS 128 /* 128 bits in one quadword */ >> +#define SVE_VQ_BYTES (SVE_VQ_BITS / 8) >> + > > I was trying to keep extraneous #defines to a minimum, since this is a > uapi header, and people may depend on anything defined here. > > I think SVE_VQ_BYTES is reasonable to have, and this allows us to > rewrite a few hard-coded 0x10s and 16s symbolically which is probably a > good idea -- I'll add this. > > SVE_VQ_BITS looks redundant to me though. It wouldn't be used for any > purpose other than defining SVE_VQ_BYTES. Yeah I was more concerned with getting rid of the magic 0x10's than showing exactly how many bits something is. > >> #define SVE_VQ_MIN 1 >> #define SVE_VQ_MAX 0x200 >> >> -#define SVE_VL_MIN (SVE_VQ_MIN * 0x10) >> -#define SVE_VL_MAX (SVE_VQ_MAX * 0x10) >> +#define SVE_VL_MIN (SVE_VQ_MIN * SVE_VQ_BYTES) >> +#define SVE_VL_MAX (SVE_VQ_MAX * SVE_VQ_BYTES) >> >> #define SVE_NUM_ZREGS 32 >> #define SVE_NUM_PREGS 16 >> >> #define sve_vl_valid(vl) \ >> - ((vl) % 0x10 == 0 && (vl) >= SVE_VL_MIN && (vl) <= SVE_VL_MAX) >> + ((vl) % SVE_VQ_BYTES == 0 && (vl) >= SVE_VL_MIN && (vl) <= SVE_VL_MAX) >> #define sve_vq_from_vl(vl) ((vl) / 0x10) >> #define sve_vl_from_vq(vq) ((vq) * 0x10) > > [...] > > Cheers > ---Dave -- Alex Bennée