From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from EUR01-VE1-obe.outbound.protection.outlook.com (mail-eopbgr140041.outbound.protection.outlook.com [40.107.14.41]) by sourceware.org (Postfix) with ESMTPS id 98B59385DC13 for ; Tue, 27 Sep 2022 08:39:18 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org 98B59385DC13 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=lSl5ZAd5HHvTflF4coQBdnbj8leoHdWLTq5P3iRXCYvixTCHEr0xn0Q/bd5sd19l4AXg9hYMM8+GIIh/mQ5NqSQDfkZ+bpBqeJQZubo/I6Ays8/emJO/MDUI+koLh5dizFYQ+J4wEnIo86sqasVS43h7+z7ZbcgdO/4dP9VPhwgOqprmB2dLt13yqWXzXYWl/GHEWOnyB34p5PBFhhMPJy6FeGyFj9ate8U6BY1vHmxbLCXDtokfeO53Q0p4+j7O30Qllxt+0kNFBIf4WWW8sZlrF7M5xRRQpugNIf0OX0nIrbX8AGIHG+r57Yjf8pNgV9v2Daz/ttNILpRabrviqg== 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=I8c0xU3iByIRUxy5vDjGFrKr8tSN2vlgfFi9X/ugn9g=; b=lwAJVqno3hYRfhJcEo+VMuMYUbpg015MUV6CI9vH+zhbYuDoBQNzPY34nwhKfQtBUaC7oUJrsOIpNKZKt29ZJ3eKjVwCcp874ymJzZDWav3BhSVDeOKlB4S4q2bLpz+YYmHyzR3TWetZ8nQxOrkOk7Lmp8gy9syRyCRB8PJuuVJmsgOoH5dQtlbOk4/AyBqz/mtB6Pm6nxRadT4itvgoPNdu9/F1fXPnADf3vdQzB1f5RnDKcdNqDCqsOrhGtzj04ihk2u4ft9EuwsXDcuIR7TPa5u4CpmLqNbglrll2e3fD4cXxBE1Lf9igUG8LWCX8EKzQIHbwFcRIX/l3loJYSQ== 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=I8c0xU3iByIRUxy5vDjGFrKr8tSN2vlgfFi9X/ugn9g=; b=WWf1ji0nHVdT+Ak76SN1wTcfJuL+qirJ48lkVzq7jqUo3VNzrwQWQS1GDpslvqVds60ihHlC2vClQBMQJeubABO+u1RMKq7AP4hByB3i/ed9LcOcEc7Dt9YN9IqLAfr8MyEb/FO1AtYWBRa72scV1SR2KdXWTNH0Wi7htrlY/ts= Received: from DB6P18901CA0003.EURP189.PROD.OUTLOOK.COM (2603:10a6:4:16::13) by PA4PR08MB5919.eurprd08.prod.outlook.com (2603:10a6:102:e0::17) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5654.26; Tue, 27 Sep 2022 08:39:14 +0000 Received: from DBAEUR03FT022.eop-EUR03.prod.protection.outlook.com (2603:10a6:4:16:cafe::e6) by DB6P18901CA0003.outlook.office365.com (2603:10a6:4:16::13) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5654.25 via Frontend Transport; Tue, 27 Sep 2022 08:39:14 +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 DBAEUR03FT022.mail.protection.outlook.com (100.127.142.217) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5654.14 via Frontend Transport; Tue, 27 Sep 2022 08:39:14 +0000 Received: ("Tessian outbound ee41cdb23966:v124"); Tue, 27 Sep 2022 08:39:13 +0000 X-CheckRecipientChecked: true X-CR-MTA-CID: ee0a24810e9d9f8e X-CR-MTA-TID: 64aa7808 Received: from 8296ce51782f.1 by 64aa7808-outbound-1.mta.getcheckrecipient.com id 78015FF0-20C8-421E-B53D-67480F5FED84.1; Tue, 27 Sep 2022 08:39:07 +0000 Received: from EUR05-DB8-obe.outbound.protection.outlook.com by 64aa7808-outbound-1.mta.getcheckrecipient.com with ESMTPS id 8296ce51782f.1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384); Tue, 27 Sep 2022 08:39:07 +0000 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=P3fcaOn3CBja+4AQjf6GKD59l/5nGypbWlS+29CPnf1Ps0Lg4OWaUUuqhAsYP53hQA8C1JxLRadEHiGfUQu2DiIBN4HzlHYkM4MdNZwuliK9RPbrkoTTEY8JZ57LutUDqRz0Gy/9AF6zF6DwbNQQJuPNegZ/Qw4pB7TduCMTfkrVqyiEyzO6MigzyBzv2u6VM9QhMGv2PaCtsrhT0WeazsXMHP6kjJC0MLPe0XI4OcZQEWXRuXlv48SRy7ziG0GBjlEEYbwon08/T6CDQamPrl3hJCJJ4jDfjC1m5t129iBr/ez9sJMdAFVc+17lRCXtJsapP+qgKAnI80yCBbRMWA== 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=I8c0xU3iByIRUxy5vDjGFrKr8tSN2vlgfFi9X/ugn9g=; b=RjY2HAPopo6dpmI6QVN/+nuF14NoJO8Fk82ylpY0MV5cmPtkJGtiyDD+vd5X2pS3E1lfa9YIGfX1XhmNPbmLj+N0gzxyJRBoSBt4BtqbLfErC8opcfI8sBRK0J5Ypom6fGoXmXbRW4N5P1lu+wx/lITyeZ98KyzT6T1cKI/HJyt1Y3cWxI9t57p65YDM8fVhqk3QfbnoIQS+kbyxbo0Mbbd1MFwKxxblDHFndGoWK9CpkTfwPDKWxaFt2dbOpkQ+1JRVCVZ5I7TXJSsB/bZ6VQAy8AxiRQxzPi6rHz7DGIxF+EUjJnl5QUFkolzniQX9yliNcf9VOQRSf7PQsUxdfQ== 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=I8c0xU3iByIRUxy5vDjGFrKr8tSN2vlgfFi9X/ugn9g=; b=WWf1ji0nHVdT+Ak76SN1wTcfJuL+qirJ48lkVzq7jqUo3VNzrwQWQS1GDpslvqVds60ihHlC2vClQBMQJeubABO+u1RMKq7AP4hByB3i/ed9LcOcEc7Dt9YN9IqLAfr8MyEb/FO1AtYWBRa72scV1SR2KdXWTNH0Wi7htrlY/ts= 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 AS2PR08MB9522.eurprd08.prod.outlook.com (2603:10a6:20b:60e::9) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5654.26; Tue, 27 Sep 2022 08:39:04 +0000 Received: from VI1PR08MB3919.eurprd08.prod.outlook.com ([fe80::c5f9:a25b:a5f2:6094]) by VI1PR08MB3919.eurprd08.prod.outlook.com ([fe80::c5f9:a25b:a5f2:6094%5]) with mapi id 15.20.5654.026; Tue, 27 Sep 2022 08:39:04 +0000 Message-ID: <674788ed-f691-447c-206d-4a4e15347d4b@arm.com> Date: Tue, 27 Sep 2022 09:39:02 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.11.0 Subject: Re: Proposal: Add review tags to patch review workflow. Content-Language: en-US To: Joel Brobecker , Simon Marchi via Gdb References: <453759b1-1ddf-1aff-a033-6183b84a4a4d@simark.ca> From: Luis Machado In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-ClientProxiedBy: LO2P265CA0334.GBRP265.PROD.OUTLOOK.COM (2603:10a6:600:a4::34) To VI1PR08MB3919.eurprd08.prod.outlook.com (2603:10a6:803:c4::31) MIME-Version: 1.0 X-MS-TrafficTypeDiagnostic: VI1PR08MB3919:EE_|AS2PR08MB9522:EE_|DBAEUR03FT022:EE_|PA4PR08MB5919:EE_ X-MS-Office365-Filtering-Correlation-Id: 2c3e462d-96c9-4c05-a7f4-08daa063c1d3 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: J7GeZukV1rdL16URWu58tQ6TQaqhbx4VTGoTBjpet4eKLAq24kQ4CQFYXJtK4Cz2ODq85GbAQgBcpVCSYpAcNXLhnSIwpeR2B374iUL6g3iqGO72w3wqm/RGddGl5tsGuJSIDf6PQMLTaCDJg1qZo0bc/ozCMRKL4eoW7PGFDAiuyKbd4g7LzeixRMre/Ar8L2Z/yH1JlI33fsr+c8nFAICRNb7KMt7jGFRs1QKI+FoZ8rZCB70XpaUQQ/L566peRU2nLrkB7TKjUwklkY6jGC7rTzlh249vrMRl250dPV8v876s1QlkH6lCqxbMJZ9/2rgNCAkZEPVIMS3umfLiMoDPSVY8YAL8VG4gsCgFAIkOZ5a+ZVrXq87fH9GhzKaLOI+X+zOq2DQilpTtp/EaFZcPyCixcNsz71EO6lC8myoEEVq3+3hwdHefp8SOonFq0pTsqFKeLOboHdPUdLQr5wQLgX8IIxgDAcgmaZrGhhdFT5n8q2gXk2KmGWykPn3PI+H6eMsC7y48vnzcn/a6us0WaQSRZGAz+k1yKY3uPCkEA9rLTeDI41JFJ1Oa6cHvCuD8dxxrF409mVMnResWSEWGbLQVKxNqEqISNmUnyfiItjuvL192fFn2KGltprsLylpMx9ibqioQlUEUxeRU7vnU1szRP/B9MuGehMMi+CyGwoS4sZ0arHzwk7vW4N4q+7gyHuZzhDJp46dhpHrhixCcCEl2r9h6uFv6+SHzP6VAT9QuPyyLy/cXAMDJlyOwXGx3Cp4pT+wi5QvbnJvzn3CzaZbLkA6+87J3JslKJagUD2qLr/GolvkVe1cwbLId 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)(376002)(39860400002)(396003)(136003)(346002)(366004)(451199015)(38100700002)(41300700001)(6506007)(478600001)(26005)(6512007)(186003)(36756003)(44832011)(316002)(31696002)(86362001)(2616005)(83380400001)(8936002)(53546011)(110136005)(5660300002)(31686004)(966005)(6486002)(8676002)(2906002)(66556008)(66946007)(66476007)(43740500002)(45980500001);DIR:OUT;SFP:1101; X-MS-Exchange-Transport-CrossTenantHeadersStamped: AS2PR08MB9522 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: DBAEUR03FT022.eop-EUR03.prod.protection.outlook.com X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id-Prvs: ba493404-8f30-4b9e-edd2-08daa063bbf8 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: GWcNwcnEo9scNtmr2LZOsg6mQmdrjZ3cVGbEou2CmVmEjvjFyim9G3hrXqzn9kXSvLYGpX6NYHoB+UU34beVwXITT3quyxEbBAbdU5WJ9BHzrSe0Xhnx9rOelgbkzFgb2EhKA3Jyvuhj+NoOFYxT4V/P0ikgyy/JmcXQThm8DqsXLVaLB+BrFKtNuiZ6PYLPvinEx6MFg54kF/b0wKpB+Pi7rJSvg7oTKKJmlAx+yBuhTbytpe2lw4ExQU02FkVhYFmj8b2kvnOEkZkaSNzoWZovLePgHfpDSZ18mDCWPIh09RSoiDNGoHxsQog+CCvT8ER6YhMoK+gqQiJnnNXhYwInV+XzgcJKXCI9N3aV6V/6DQa48Dx/pmpXNx0ApW4gk1NOhL0GAnulIoYdweSMBEdi59R+0DJkUv7rjepmB6xAW6R5MIvXb3DGkIpWPTgQFOaM0CF5SrHU5D8ONAPXR4nFPFkqUwT6SckY5ACQniV7EbbPGKUHSDrakYHfFThDxUcYuG5h2KiMQ22mWOc/NP4tqTq2olSddCZUr3KVjCP/eGnJcJSV7LepRvw08Qb3thIHcuVVjZk/LLwO5goMImCiOzNXnSCEt88eJLt5+vHxYXMYjAFD3224TQO8Anuu0Fu/u+AtpqinYum4hzTj3XLwaxFHduVSepURq2QM4Wn6F3P9lSrRIA/7S75nvz6RsbJS8VT4pcWGceYe0iQQ0tOT9u1qXYywgUxG6o96Hwjb5ht1c8Y5lhZbRppU7bcJCXlT5Ym69rEmaJgA54qQLpw9d2+Tfsbu7zstGxoYGN0o+RvQm4sLr1cCVHbS70NuM+XaILzzkjFW8WIWTQ5ffA== 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)(346002)(376002)(136003)(39860400002)(451199015)(40470700004)(46966006)(36840700001)(40480700001)(83380400001)(336012)(186003)(47076005)(40460700003)(70586007)(81166007)(36860700001)(82310400005)(82740400003)(356005)(8676002)(5660300002)(316002)(2616005)(44832011)(2906002)(41300700001)(8936002)(6506007)(53546011)(6512007)(26005)(110136005)(70206006)(6486002)(478600001)(36756003)(966005)(31686004)(31696002)(86362001)(43740500002);DIR:OUT;SFP:1101; X-OriginatorOrg: arm.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 27 Sep 2022 08:39:14.0270 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 2c3e462d-96c9-4c05-a7f4-08daa063c1d3 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: DBAEUR03FT022.eop-EUR03.prod.protection.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: PA4PR08MB5919 X-Spam-Status: No, score=-7.5 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=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: Hi Joel, Thanks for bringing this up. On 9/26/22 17:42, Joel Brobecker via Gdb wrote: > Just thinking out loud... > >> I completely agree with the proposal. I really like the fact that it >> makes communication less ambiguous. Following some process (or changing >> the process) can feel a bit heavy for long-timers, but I think it makes >> things much clearer for newcomers. > > Speaking of ambiguous, one thing that we used to do well in the past > but then kind of got worse was the subject prefix we used to use > to indicate the status of a patch. In particular, we used to reserve > certain keywords for that in the subject (e.g. "RFA" vs "PATCH", or > "OB" for obvious, etc). We lost that part, not sure exactly when, > but I suspect sometime when we transitionned to Git. > > Something else also that I have been feeling the last year or two > is that I'm not sure people now explicitly confirm to the list > when a patch is pushed. I think that's been happening, yes. But the IRC bot mentions commits explicitly, and developers tend to see updates in the git repo when they update the sources. With that said, in general the frequent GDB contributors tend to be quite busy (with GDB or other things), so I'm inclined to say it is positive to have less steps to take care of to push a change. For example, ChangeLog's were a big time sink, and we managed to get rid of that rule. I think that was very positive. We still have other potential improvements waiting to be discussed, like auto-formatting of code with some tool like clang-format. Time spent correcting formatting is not very useful. > > The reason I mention this is to show that perhaps we're getting back > to the fact that our email reviewing system is still email-based. > One way to address the various limitations is by adding more > processes, as suggested here. This has the good property of being > fairly cheap to discuss and implement, at the cost of a small > added overhead. I don't have a strong opinion about it, either > for or against (and given the amount of time I have to contribute > anyway, I don't think I should have a say). > > With that said, I have a feeling that switching to a system designed > to manage patch submissions and reviews, no matter imperfect, is going > to solve a lot of the limitations of the current email-based system. > So that's another option worth reviewing from time to time, I think. > I understand that selecting, deploying and trying new review systems > requires a fair amount of effort. But having seen the benefits of > using several different such systems, I am convinced that the gains > will be very much worth whatever the drawbacks of that system might be. That's a fairly good point, and I agree. We tried a patch reviewing system (gerrit) before. For me, at the time, it was obvious that the number of reviews increased significantly. It was just easier to do reviews that way. If you had 5 minutes, you could scan for a small change and give some feedback. The list of patches to-be-reviewed was never forgotten. But back then we didn't want to risk alienating global maintainers that didn't like gerrit or liked the e-mail system better, so we dropped that effort and put nothing back in its place. It feels to me patch reviewing by non-contributors lost some of the traction it had gained with gerrit. It is not a secret that some of us contributors would like to see improvements in this area, hence my suggestion to address patch reviewing/more maintainers/CI-based testing as topics for the GDB BoF. But at the end of the day, it's up to the global maintainers to make a decision on this topic, or to let contributors know they are open to adopting improvements. So, in summary, I see the proposal to add tags as a way to improve a patch reviewing system that is not being capable of keeping up with demand. I doubt we would need such tagging if we had a proper reviewing system in place (be it gerrit, patchworks or any other). > >> Assuming we will go through with this proposal, it will need to be >> documented on the wiki so we can easily refer people to the procedure. >> Probably the ContributionChecklist page? >> >> https://sourceware.org/gdb/wiki/ContributionChecklist >> >> Will you be able to take care of this when needed (do you have write >> access to the wiki)? >> >> In the mean time, message to others: please let us know if you agree >> with this, it's difficult to know we have the support of the community >> if everybody silently agrees! >