From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from EUR04-HE1-obe.outbound.protection.outlook.com (mail-eopbgr70044.outbound.protection.outlook.com [40.107.7.44]) by sourceware.org (Postfix) with ESMTPS id B27553856243 for ; Mon, 31 Oct 2022 09:28:32 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org B27553856243 Authentication-Results: sourceware.org; dmarc=pass (p=none dis=none) header.from=arm.com Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=arm.com ARC-Seal: i=2; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=pass; b=n2pIjiUF+YIUMwJTZnbPniI1THBniPR+31zyyEmK6XSRONLA5tNz6a35OHcazKTMPKqUHPyVCFirHsiH7iUWsV/HIELvMNwq6zxVkzaTasS/BBdrduwsQV4NGHb4jFZLSNuw8idS0OTeiDaaPAh+xb47HFZUsnjBOSF4bTVVndo8hO39DcyVpRCwDivLrhICxcoWuaXmaU/EYhg1Yp3RqsA6LohICu3xfJbjiWiLCWzgLRkSd+mV2FAcuw5dzmt6aNgjz/4quTeDkbJd+Gt7CtbOmXL0Zgct7bO7yJkPDVbvMgFSZimRSeRV8EZAjHitiVQtYvPgJYcHe9mjK5H78w== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=HM+poAP/OYq8NGnbsjWPPYoXVU6IbEYeAx4qjSdaq1Q=; b=jzYmERfyn7Y5m5oXyQFEuMbBZ4p8inB5uatKnD/NVI2a90yMrViI7nR8eAN9ldmYHR9YIJtvseihZZ3ajA/xPBfnyemWXhDJ7k08qflzzc2XZbKyiYb/kwet388OU2LCXO1dJTdfw4i+J0oWG9dtK75Jj1oAh+/hRWvT8sCwJZCAP4eZpKYDFE+JqLtIBuaQFOTPaMkSv/2selrmiUs5d9QunsGKvwDT5EwM72RIbnXhM6yyTuwyw0B0bwNnvvVYa8YPGSj0UVV6LZW1YtWbC1v9WQY9Nzy0k/galrR0dVyHvBtdtvdQSt8c2yZDlXeBrrh5nJPv2xbM1B81gHX8Rw== ARC-Authentication-Results: i=2; mx.microsoft.com 1; spf=pass (sender ip is 63.35.35.123) smtp.rcpttodomain=sourceware.org smtp.mailfrom=arm.com; dmarc=pass (p=none sp=none pct=100) action=none header.from=arm.com; dkim=pass (signature was verified) header.d=armh.onmicrosoft.com; arc=pass (0 oda=1 ltdi=1 spf=[1,1,smtp.mailfrom=arm.com] dkim=[1,1,header.d=arm.com] dmarc=[1,1,header.from=arm.com]) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=armh.onmicrosoft.com; s=selector2-armh-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=HM+poAP/OYq8NGnbsjWPPYoXVU6IbEYeAx4qjSdaq1Q=; b=EKcsRW4Fcr37pzdV0kQ3P+zwOYawyPY6tTYmd4Dfn9JXliOfIngbLQnAQY/jLp2ZmdJxBMpTVqL/+obgilHweUwCX+NL4iuigp4IcCz9Q/sLZpAl0ZlMqkkCbW2w6PKscazaJ/iZTZ551ppsTui3FiZYb9ozgF/+by06lO4VF/A= Received: from AS9PR04CA0175.eurprd04.prod.outlook.com (2603:10a6:20b:530::26) by DB9PR08MB6667.eurprd08.prod.outlook.com (2603:10a6:10:2af::8) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5769.19; Mon, 31 Oct 2022 09:28:29 +0000 Received: from AM7EUR03FT049.eop-EUR03.prod.protection.outlook.com (2603:10a6:20b:530:cafe::ee) by AS9PR04CA0175.outlook.office365.com (2603:10a6:20b:530::26) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5769.14 via Frontend Transport; Mon, 31 Oct 2022 09:28:29 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 63.35.35.123) smtp.mailfrom=arm.com; dkim=pass (signature was verified) header.d=armh.onmicrosoft.com;dmarc=pass action=none header.from=arm.com; Received-SPF: Pass (protection.outlook.com: domain of arm.com designates 63.35.35.123 as permitted sender) receiver=protection.outlook.com; client-ip=63.35.35.123; helo=64aa7808-outbound-1.mta.getcheckrecipient.com; pr=C Received: from 64aa7808-outbound-1.mta.getcheckrecipient.com (63.35.35.123) by AM7EUR03FT049.mail.protection.outlook.com (100.127.140.234) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5769.14 via Frontend Transport; Mon, 31 Oct 2022 09:28:29 +0000 Received: ("Tessian outbound 58faf9791229:v130"); Mon, 31 Oct 2022 09:28:29 +0000 X-CheckRecipientChecked: true X-CR-MTA-CID: 1a2b8d6037227d0b X-CR-MTA-TID: 64aa7808 Received: from 43ebe4bed580.1 by 64aa7808-outbound-1.mta.getcheckrecipient.com id 778382DE-2B8F-4261-9FC7-EB3A95D22CED.1; Mon, 31 Oct 2022 09:28:21 +0000 Received: from EUR04-DB3-obe.outbound.protection.outlook.com by 64aa7808-outbound-1.mta.getcheckrecipient.com with ESMTPS id 43ebe4bed580.1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384); Mon, 31 Oct 2022 09:28:21 +0000 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=DPfo6TN8gOdiNI7gB42dKwkoDrT5shKtKq/SEZSzkRlpvX4TQ6aFhasJEtBh8gJKkI/UNyzJAKQNHLnaDiekaSb2h/qvQYyoGZniQeNrIAWWEiMh1SyipXE0AEF3s2GwibCufw1JrafnYNfDhpJVe1mQHvwUPsz61bMFWz7ohyQeyf7oy6Ous4nm77aoJDhq4ETAl3Be4Dsoi93zv29weN0bywY1F5D8Gk6a+GvYLm+sRn9WNuQqHu7cNU9QhnTBk2cxiN+VUtN5W4Dv81E9pCjMGPYBeNVzgPCgqAYcJeFSSWiwCY4mZJC+Iw4x1QC46ged8+bMBvUUkfRaugrymg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=HM+poAP/OYq8NGnbsjWPPYoXVU6IbEYeAx4qjSdaq1Q=; b=NQyN/bto1byUSbAVF2TcBLDoL5244RlPD7+iR9Bo/q4xf7KXXcp+wjsToeJ8cIDj6iKZc1bo+Dee/07GV6UhUA5K0rOyLwEpqT30zcsVShziT/tJ94bpozSN+IuOyhDQBVyCot2CpD7/ptp0eweMMwuBT21kydFruh9bcLfMbGQbDrq0XOIxi3AtR02NCNgSwenErM00NqyArF8iCF2vbdvujHlzzQtmAnpDQGFqE6PbhSIfRPaDvyN9z1kpEhvHX6chTku6W+tp3YJ8HE4LGWSNO1E/WsvaWoL74xp50RLQ9qShc+z+AXVGt5aPh1odlC0QV0Irroh2H7FKNN+XYw== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=arm.com; dmarc=pass action=none header.from=arm.com; dkim=pass header.d=arm.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=armh.onmicrosoft.com; s=selector2-armh-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=HM+poAP/OYq8NGnbsjWPPYoXVU6IbEYeAx4qjSdaq1Q=; b=EKcsRW4Fcr37pzdV0kQ3P+zwOYawyPY6tTYmd4Dfn9JXliOfIngbLQnAQY/jLp2ZmdJxBMpTVqL/+obgilHweUwCX+NL4iuigp4IcCz9Q/sLZpAl0ZlMqkkCbW2w6PKscazaJ/iZTZ551ppsTui3FiZYb9ozgF/+by06lO4VF/A= Authentication-Results-Original: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=arm.com; Received: from VI1PR08MB3919.eurprd08.prod.outlook.com (2603:10a6:803:c4::31) by DU0PR08MB8954.eurprd08.prod.outlook.com (2603:10a6:10:465::21) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5769.19; Mon, 31 Oct 2022 09:28:16 +0000 Received: from VI1PR08MB3919.eurprd08.prod.outlook.com ([fe80::6ba4:2f18:7531:943b]) by VI1PR08MB3919.eurprd08.prod.outlook.com ([fe80::6ba4:2f18:7531:943b%7]) with mapi id 15.20.5769.016; Mon, 31 Oct 2022 09:28:16 +0000 Message-ID: Date: Mon, 31 Oct 2022 09:28:08 +0000 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.2.2 Subject: Re: Any concrete plans after the GDB BoF? Content-Language: en-US To: Simon Marchi , "gdb@sourceware.org" , John Baldwin Cc: Mark Wielaard References: <83485199-965e-7ff5-1dc8-d027b74b56f7@arm.com> From: Luis Machado In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-ClientProxiedBy: SN1PR12CA0054.namprd12.prod.outlook.com (2603:10b6:802:20::25) To VI1PR08MB3919.eurprd08.prod.outlook.com (2603:10a6:803:c4::31) MIME-Version: 1.0 X-MS-TrafficTypeDiagnostic: VI1PR08MB3919:EE_|DU0PR08MB8954:EE_|AM7EUR03FT049:EE_|DB9PR08MB6667:EE_ X-MS-Office365-Filtering-Correlation-Id: 0f8e982a-0edd-4d6c-f694-08dabb224570 x-checkrecipientrouted: true NoDisclaimer: true X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam-Untrusted: BCL:0; X-Microsoft-Antispam-Message-Info-Original: nb0eH0+U3/oyJdM3rg3QiPfLGz/OLbfOuCdeX2Nvv9G3gc8r7vhnjKJiB3NUNx71eVkJZQ70uifns9jMplPfT71o9OJtk+KWpLsZ5j7bWspNQp9ZS28zcBw2/UaofTPp/3X61YzCWFtQIJKgvybbGMuWT3Xz6D1k3u6b1KQTmZSFGPShNqiPvX3enuCqbvGSomhrr8XGNX4+nAT32Z4Wz70kUY4Sn8shKT0rGiZ//gyEJLlri8QZPa81UT5FVgvew/azHnu6Vg0OAGQfqtArbeR44m+wwshkQi7/zDYrNnG8si7qOtO59udbKgvKcAhGGlNzKbWNUPn/SoTSVJCLdsyeTnTmdJ1mAtny3pGzqQX8WqZMtVPrZp2NoT1G08liX7gdT60kKyEBEZbK0/2lb9nPwgkh5mvMKKCK3IUtZdfbMkxy5q7qlaD7ZGu9Zzfa1npj6D71hEj9sgzYPCVjEg8n3J5A1Vpi5QGbmKzG81HaDQ54aAPgVp59ZHefc74GttA5DKVyLp+YaW2oQUStGgYxWziDTrRr0pFD8+5FZ3Tnx1VjjbvcjqLzsjxXO3QEE1XVKqptJdf6CLvJPUGU6iq1PSQ3c3vbX7PeqQgilE5swXHipIWWzZv/ledZhKqpHnvpmkJvp6eUjAXZjfKZtnGu7phkLvGs7EQtmcyDzzS0BE4nsE1UCH6ycEqPZdXahDFJoCMVFXid8KlHmoROFMSaaMyoWndbKc9FUv7ykkpfIoOWNeazs2syKvXczDzqu0GnQjSV70MJZTN55gB9J0nsBMpxJVUV8m+Q1UEZYfQ= X-Forefront-Antispam-Report-Untrusted: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:VI1PR08MB3919.eurprd08.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230022)(4636009)(136003)(39860400002)(376002)(346002)(396003)(366004)(451199015)(36756003)(31696002)(86362001)(31686004)(38100700002)(83380400001)(4001150100001)(2906002)(44832011)(478600001)(6666004)(6512007)(26005)(53546011)(2616005)(186003)(6506007)(316002)(41300700001)(6486002)(110136005)(66946007)(66556008)(66476007)(8676002)(4326008)(5660300002)(8936002)(43740500002)(45980500001);DIR:OUT;SFP:1101; X-MS-Exchange-Transport-CrossTenantHeadersStamped: DU0PR08MB8954 Original-Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=arm.com; X-EOPAttributedMessage: 0 X-MS-Exchange-Transport-CrossTenantHeadersStripped: AM7EUR03FT049.eop-EUR03.prod.protection.outlook.com X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id-Prvs: a3330aed-fade-40fd-e035-08dabb223d04 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: Rt6V8u2HRZbc5KFqhlSeTnhDCqTzkJNB70t15bfjIl6B+CWwZj6trFyj6CmSm/ZTrYpQbBgS1WY9QEm4TLVNZmW0haBnPkwgJUn8eveXBX48a7am7hU23K50Kr4DLBs/w7svuBTgaPhGm4vdYIx+7ym2lYEMixL1RGALeCqOIeQX9kBLQCH/CPtBjVeKOZe/RT38ggxuxH57lVHTblvgE5xYWOD010dyDcXwCQzj0sLCv9SJ3Cdcryn5LSzhjcv+qGTQ9iemmTLLg4YISbww0DXrTKv5Pg9DLPNEx1dv35IRaJNdGV48TN/1TSVonTJc7bxDtXmJCNwx97cg5nM3VKg4QemaQF5suZY5DEQu0R1rj4G+hDAD+/Pb+P8jm6W9qCL1ShHJrKKyIx8gnsnaTCJqAlOaaLJ49VM064mQVf4OIb25wEAObuwCnHViN0GIhzH6GlxW8HXt3dmPxhx4f6bTQL9p/VMeYbdSBtEthG/UXQUVOQNje3jmK+YPAjSwtUS5iHzOQqAwNksWL6NQ51CO+rB4gRnjE3r3OnkjPVubynDTR9dfy3TA5ly3/K6lmbNN1KWS+szlMpPxo2nUWRdAVOjRPWshpy4v9S33LIRKaCbbCJBMte1FbiMDh29ReuA+ODpxKh9Buv3babBwU7ZwvSkrcgTn0tmuoJr6fYXGoqdk7BXRtSGUhglQQu4RW/w7fVLYYc6dJGF4skf9qrIa3b9ixZQYL6gxShL1K8xkGziPFwHeB0pTelbAxhwe1CjaMdkMbsqlImGMCptgMuqNvn5SakWOPN4GBGtITmQ00abaNvWAB8nXUR7zWiMU X-Forefront-Antispam-Report: CIP:63.35.35.123;CTRY:IE;LANG:en;SCL:1;SRV:;IPV:CAL;SFV:NSPM;H:64aa7808-outbound-1.mta.getcheckrecipient.com;PTR:ec2-63-35-35-123.eu-west-1.compute.amazonaws.com;CAT:NONE;SFS:(13230022)(4636009)(396003)(39860400002)(136003)(376002)(346002)(451199015)(46966006)(36840700001)(40470700004)(4326008)(8676002)(316002)(70586007)(70206006)(110136005)(41300700001)(5660300002)(31686004)(44832011)(26005)(6512007)(2906002)(4001150100001)(6506007)(478600001)(40460700003)(356005)(53546011)(81166007)(107886003)(336012)(6486002)(6666004)(186003)(2616005)(82310400005)(83380400001)(36860700001)(36756003)(8936002)(31696002)(40480700001)(47076005)(86362001)(82740400003)(43740500002);DIR:OUT;SFP:1101; X-OriginatorOrg: arm.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 31 Oct 2022 09:28:29.4034 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 0f8e982a-0edd-4d6c-f694-08dabb224570 X-MS-Exchange-CrossTenant-Id: f34e5979-57d9-4aaa-ad4d-b122a662184d X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=f34e5979-57d9-4aaa-ad4d-b122a662184d;Ip=[63.35.35.123];Helo=[64aa7808-outbound-1.mta.getcheckrecipient.com] X-MS-Exchange-CrossTenant-AuthSource: AM7EUR03FT049.eop-EUR03.prod.protection.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB9PR08MB6667 X-Spam-Status: No, score=-6.3 required=5.0 tests=BAYES_00,DKIM_SIGNED,DKIM_VALID,FORGED_SPF_HELO,KAM_DMARC_NONE,NICE_REPLY_A,RCVD_IN_DNSWL_NONE,RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_NONE,TXREP,UNPARSEABLE_RELAY autolearn=no autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org List-Id: Hi Simon, On 10/28/22 17:16, Simon Marchi wrote: > On 2022-10-27 06 h 47, Luis Machado via Gdb wrote: >> Hi, >> >> Having suggested a few topics for the GDB BoF (I noticed they were discussed, to some extent), are there >> any concrete plans from the GDB global maintainers (leadership? I don't know how to call it) to address >> some of those concerns? >> >> Simon was kind enough to cleanup the patchworks instance, though that is not yet fully integrated into >> something we can easily use to do tests/CI. I see the number of unreviewed patches is growing again. >> >> For example, it is not easy to pick a patch to review. You need to locate the entry in your inbox so you >> can reply to it. > > I do not know of a way to trigger CI tests from Patchwork, that would > perhaps be a question for Mark (added in CC). > > On a personal note, coming back from the Cauldron, I set myself a goal > to do more reviews as part of my daily work. I'm trying to do around 1 > hour a day of upstream reviews, and to choose what to review, I use > patchwork, sorting patches by oldest date. I check if the patch I'm > looking at has already been reviewed, merged, or superseded by a new > version, and if so I update its status. Rinse and repeat until I find a > patch that needs reviewing. Otherwise, just looking at my inbox's > gdb-patches folder with thousands of unread messages, I don't know what > to start with. Just by myself, I certainly won't get through the whole > list of patches pending review, but I think it is a somewhat fair > algorithm. So in that regard, patchwork is useful for me. > > I wanted to send an announcement on the list to say "hey, patchwork has > been cleaned, let's use it!", but I have been procrastinating since I > came back. I think those of us usually chatting on IRC are aware that you restarted it, so thanks for doing that. With that said, John Baldwin exposed some valid points. I also find the Patchwork workflow and interface odd and hard to work with. I can't simply pick up a random patch and easily review it like I did with Gerrit, for example. I need to go out of my way to find it there, look for the mailing list entry etc. I feel this goes a bit against enabling non-maintainers to do code reviews. The current workflow, though it works nicely for some, is quite limited and very prone to letting patches be forgotten at the end of the list. There are better ways to get this done these days. The PING mechanism, for example, is a burden. It is more manual work that you need to remember to do. On the other hand, if patches are archived in a good way in some system, it is just a matter of someone spotting it in a list and reviewing it. For instance, someone may have 5 minutes to spare. This person might go and look for a smaller patch to review, make comments inline and go off to do something else. In summary, even though glibc uses patchworks, it might not be the case it is the best tool for the GDB community. We seem to be short on reviewers (maintainers and non-maintainers). Enabling more non-maintainers to do reviews seems like a positive move towards a more efficient development process upstream. Some people admittedly don't like gerrit, but the tool has a lot of benefits, plus it integrates very nicely with Jenkins. And we need to have continuous testing back for GDB development, otherwise we risk having targets getting silently broken. It is reasonable to say one can't guarantee things won't break based solely on code reviews. > >> On formatting, have we considered the benefit of using clang-format for GDB, therefore potentially saving lots of time >> in reviews not having to worry about formatting? > > This often comes up, I am all for it. We need someone to write up a > proposal of how this would work (a bit like Bruno did for the > attribution tags). I might get to it, but I don't promise anything. I can do it. I know some of us tried it already. Tom Tromey seems to have done it as well. I think this is another step towards getting the contribution burden off of contributors. Formatting should not be something one needs to spend time with. One space x two spaces, 80 columns x 100 columns are certainly not as important as code that does what needs to be done and improves GDB overall. Also, there are lots of different code styles out there. It is not unusual to have GDB contributors doing other work on a project with different formatting standards. Having to remember formatting nits is not very pleasant nor efficient it seems. > > Simon