From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 831BB42D09; Tue, 20 Jun 2023 16:10:36 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 72912410F6; Tue, 20 Jun 2023 16:10:36 +0200 (CEST) Received: from NAM11-DM6-obe.outbound.protection.outlook.com (mail-dm6nam11on2040.outbound.protection.outlook.com [40.107.223.40]) by mails.dpdk.org (Postfix) with ESMTP id 49AFF4068E; Tue, 20 Jun 2023 16:10:35 +0200 (CEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=c/D/t/ZLnNWdd/RJtaH+nO4d1Z1QBedS5FQ35qcdhnvUbaTHaAe/P8jXwOpl1cH3UNIcU2+O0FDMrSCrGSkHnPb4YIevD23/MkbQJsAliqq1RP/Z2QD7BFj6ueYqRIeJRDTbsd6nSTYYie3hbqmaFAHlAbTYJBucBqwLvJylq231rd/qTMRmtQMqsGDvl8YYJsOdbn3Ii+l3cljenaiCGVf+viAmWrO2Iijw52RLek0RV+QDoCWEK1CTRX5drgyyD/j9Ft6zT5oRWzxt5lGQZy46/Z4BH/MBcV0+gHStub6WN7tjSk0kiEyetUJ73WgogCDEaDtHvzVRT2wJ23lAYA== 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=zqGX0ZmauxZr8kfPhU71h+K2BFnMp3lAgtU0/ihjGq4=; b=VnOOx9AFob08ZG/BaTsF3gowp34bG2P0WjDhJ6dPxA/5fzUei23Mg00bDtbMjBz4AFtyClDZoU9qVLeoVOcaptUxtieWK83Cr0NcwS6zvSKtDgDeR6pwSSQWoQYxfZXJvcfdEYnzJz4yOZlM1LAlS5NBPkF78dxlD8deZOoThpSHf6qVFR4B6k8AWxocCZCRP3nSBzkREOQT/mvX30pksOPGshhpFTd9XGQE8fNXSuzgDgPGNuIYDH4VLD83YEAYMrWB49tqVWH+VKKVBHka09xIoaqp4k2tbrSTUIQSj5KsfbKP86gXPbPlV8PyISOmgWjKWiKGrk1Mpb0rGdKwuA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=amd.com; dmarc=pass action=none header.from=amd.com; dkim=pass header.d=amd.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=amd.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=zqGX0ZmauxZr8kfPhU71h+K2BFnMp3lAgtU0/ihjGq4=; b=O205QwaiCvLPtK8X6wqS5FAhGGyv90EmOHVxW8KNHZsu7pLFMlMSHb3qy+DQoXkj7MNys2y4KII7AgRI3gGb1O0QRSPPFo9r8lU876kGyDKo/f731olMqQmib/CEIiLSQLKc3EFd30UFqgnq0cnBbM1FW1TwPm+Ebe1l0fUN1XM= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=amd.com; Received: from CH2PR12MB4294.namprd12.prod.outlook.com (2603:10b6:610:a9::11) by CH3PR12MB9146.namprd12.prod.outlook.com (2603:10b6:610:19c::18) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6521.21; Tue, 20 Jun 2023 14:10:33 +0000 Received: from CH2PR12MB4294.namprd12.prod.outlook.com ([fe80::16e3:326c:5c2a:be42]) by CH2PR12MB4294.namprd12.prod.outlook.com ([fe80::16e3:326c:5c2a:be42%3]) with mapi id 15.20.6500.036; Tue, 20 Jun 2023 14:10:33 +0000 Message-ID: <0cda2d4a-1deb-9024-9578-de98082c6ffc@amd.com> Date: Tue, 20 Jun 2023 15:10:26 +0100 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101 Thunderbird/102.12.0 Content-Language: en-US To: "humin (Q)" , =?UTF-8?Q?Niklas_S=c3=b6derlund?= Cc: Chaoyong He , dev@dpdk.org, oss-drivers@corigine.com, Zerun Fu , stable@dpdk.org, Peng Zhang , Long Wu , Thomas Monjalon , David Marchand References: <20230616071558.1278520-1-chaoyong.he@corigine.com> <20230616072034.1278546-1-chaoyong.he@corigine.com> <3237ae77-bd68-cffa-f1f6-537ff871496e@amd.com> <5eceb17a-adf4-11e3-4274-0d6e111b091d@huawei.com> From: Ferruh Yigit Subject: Re: [PATCH v3] net/bonding: fix bond startup failure when NUMA is -1 In-Reply-To: <5eceb17a-adf4-11e3-4274-0d6e111b091d@huawei.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-ClientProxiedBy: LO4P123CA0667.GBRP123.PROD.OUTLOOK.COM (2603:10a6:600:316::20) To CH2PR12MB4294.namprd12.prod.outlook.com (2603:10b6:610:a9::11) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: CH2PR12MB4294:EE_|CH3PR12MB9146:EE_ X-MS-Office365-Filtering-Correlation-Id: f18e3def-2091-4f4f-0286-08db71981cc4 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: 5Bs4Lq3xIteq8M2a73W/oyJt7EPpKFNk+3jRqzNuqgA59mDKrPipnkqa/pSQ33r6MPSDMH3DG6jqIA1H1z9L1QZKMiYi4vDSRPCLT/Nxb/FRS04j70fBcOUP8oln1rPZNbcXhuagoP6IoUp3krNZKleY95xEjN8WZ6sexZYrzJJems2QUC5qNceiGaUP4g090zBJLUP5Bz+WPUB7kJC5DUFUh8HN3ydmFyTX5jkoneQA17DT5A9oNWFmpObGUt/Tf9cDSX6QJvB4du/x6wkMouknNZgWZuTbAB+vs3VuRPxZGn9fXRd+8nd2BpSttIPLxuPFkaI18WdJWVJXuMGFeze00zY9k7zZCZ/4pzpqtDwW3QFqJC1fVL3rQ0Rw9BqYv/YX8SHhzS2KIlx/TPCB+2LkKQZWxbwGRBi8xLBVX6VVU8EPVmVvg514pg8NCdrR6OgvJ1v40U348+n4XdvAUbrXLKw8UMZYQb7wLK1/23LGiugsAVyn0GSUlVD8+v3PAua/Hhbazs8omWALgZu/rxMosbe/o2DfooEudTvueBmYftsmoS/LCSigz+k7oXbIFx5CG7hJTDf3i835YoiKK16a3D2e1fCiMmVaqEf3a26XWKUw1ejn+e/DSjYV02LYdvDn75qVR4+zx4ykh9duOA== X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:CH2PR12MB4294.namprd12.prod.outlook.com; PTR:; CAT:NONE; SFS:(13230028)(4636009)(376002)(366004)(346002)(396003)(39860400002)(136003)(451199021)(66574015)(6486002)(478600001)(2616005)(110136005)(6666004)(83380400001)(54906003)(6512007)(26005)(6506007)(53546011)(186003)(2906002)(44832011)(7416002)(5660300002)(36756003)(38100700002)(4326008)(66946007)(31696002)(316002)(86362001)(8936002)(8676002)(41300700001)(66476007)(66556008)(31686004)(45980500001)(43740500002); DIR:OUT; SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?andGbmF4M016Q2JBRTIyM0puNVp1Snhrd0hLdVNPWFhwUVdTMjRxREdJamZC?= =?utf-8?B?ajNBa1hhQzN2U1NiVUlDckhVNjdzMmVMOHN1a3NYN2VVTVlIVlU3SDR3aUhw?= =?utf-8?B?Um1wRk54UkoyRWc1Z0ZGZUZ6N2FBTFM3ZFFmZitoNGpZQjJvWmNld0hrTnBM?= =?utf-8?B?M1hROWltVUNMU0ljTCtqc3RIaERDdGx1ckR5amo1VFJmTjI3YnBpTWl6YVVC?= =?utf-8?B?K0MvNk1NNDZxZXZwTnkzcmczc09BWm9WakdaaFlRUG8wcjNUeW9hbkEvYmFW?= =?utf-8?B?TU4yV2RPS2JEOTI5NCtsQnhoQ0M1Zjd2SG40OUhieHZXU2J4S05MWmNrUzVn?= =?utf-8?B?ZmVGNmYwbU81SlpzSjdXdGJ2RW9ZWk9UdXNCQUhrb2twanZsdmZpZnFLUkQ4?= =?utf-8?B?UlZjZ0NtaCtxNjdRSTNWYzdPOUlRL25BWFh4SnV2LzZQSVpJS2oxbS9KcTJE?= =?utf-8?B?MStjcmRtNmEvbHd2LzVDTmR6YU5BbWwwWlZmRjJKalRoZXdKRURwaTRiRnFD?= =?utf-8?B?ZVpSMi93SkpFY2M2TjNhYmd0QnVnMlJSWlUyN29xS1kwU2JtKzNDRlJ2aG1u?= =?utf-8?B?c0xsSWovdVc3YUVtZk1YTmZRUGd1dGw0RzM0TlYxSE1zci8vMGpSclFhRUVL?= =?utf-8?B?TjJlcXdBTlBoQWJzVzM5YlQ3UENwWFQrS1ZjdDhFelBBakJLd1J2MU9PNTJm?= =?utf-8?B?cUJXOFUwdFVzWHlHcG00OTdYem9wMTdJak5RbkZ5Y2l0a0ZUdExlQVY3VmVQ?= =?utf-8?B?TThYVGlTRnkwWVlwcnJuamNQY3dHK3JQMU5tMjQwQ2N0a3JXTEZCVy9PbSta?= =?utf-8?B?NWI0eWRlTlNPdC9KZnlTVGlmc25rQmJ5d0FuOTdGZ1BldVRlb2VBcnlXaW1R?= =?utf-8?B?bEpTdGlsL0dIbmhPUVFFSmlVVnc1TjZkUnhLSm5kamtPdlhoeHZRN2s3a2dY?= =?utf-8?B?R1EzQk9wZU43dGlyeDkyQVloNDd4MTJJSlRSSzFGb3hIL2JXRUg2Qm8yOFRu?= =?utf-8?B?L2NwTVN6dkF0ZHRhaFBTRWlZM1p5ZzdTZFF5YVdDa3JsRlhXaEgzTDRpK1Z1?= =?utf-8?B?c05IcmtTcWpuSEZzVGtOR2V5T2llVUtMdU8vR2NWK2M0QU00eGZ6aVhmZS96?= =?utf-8?B?MnlCMmxRemplbjg1V3MvYVYvUEVndDV6eWV0bnBHSGdLWG05c1ZRUGt6MEVV?= =?utf-8?B?VlVwZXZJNUU2NHVpejIvR3F2b20wajFSbC9seWJBbmpCVVUwTlExVm8wWHVK?= =?utf-8?B?MW1xZmlnTTNiWmJLMytzZ2Uxd3Z3LzVqb2lzRTMzbGpjbjdRbWVDN2t0d1I0?= =?utf-8?B?YThLai9IVXk1a01ISmVleUgxbFFmYXk1V01xZmRrWURUUXNuSnBVby9mQ2Nv?= =?utf-8?B?TWlFK3hEbHlyMTcva3pWRFp0VCsxdFpRMnJtK2dNMHRiSlh1aDRxc1hkYkEz?= =?utf-8?B?b3I4L2dldVdwVTFKMVd2Z2tGSUFWK2JWWUlTWGFqVVB5NldGMXQ1SEJCeldp?= =?utf-8?B?R2JaYWorakpmakovVWY0ZXJxMmxJbExsbitvREFoWWl1RWJ4clFsU1NrcmQy?= =?utf-8?B?S016enovdWhuaUdqbHBndjlhemxpMEdrNnlzbWw1RzVWcHhjTEZXTHJXOXRP?= =?utf-8?B?WDdndmtjbnY0eElJMDRDWituSkloVG5vQTIxd3hpRjZGNHRXd1FEMXo0RDZr?= =?utf-8?B?UlczbXNiaDRxOUZlNFdUMTIvM29SSWMvclRtY3RDSTBnK3pCdmdkdWJtZ1kr?= =?utf-8?B?eElOUlRmWVc5SVhGY1piVTVNR1pNYVRFc2tnMDN3MGFJM1poVDFBUGdEYVE4?= =?utf-8?B?VE16YzlYUnZVcjhuWmpXMmJiYktQbXgybDJIMHMxZ1g0UDJMWExlMkxDeVpL?= =?utf-8?B?WHpWK3ZMQmNmdzFmTFF6Vm9abGhoVUkreWFjV25iQlB2MXFxSWdCM1dpbGdy?= =?utf-8?B?TEVUeW9wN0VBdXJRWnUrQzJraUpEY3BRclZuaVdaV1RoVTFjejFCRVNJVjNk?= =?utf-8?B?RURwN1QvNGxOSzQ0Mmw1akVWakVGMytJdVpZRGpBWXRkSklsQUxRR0MxV3Nz?= =?utf-8?B?R2l2ckcvdjhDblFJMmhnLzA0UWhWbi9DTnlDQW9lV1dpZEV0ZFVaWUhCaHlP?= =?utf-8?Q?2I1JTHXFwPp4Irmas/wrEy5e2?= X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-Network-Message-Id: f18e3def-2091-4f4f-0286-08db71981cc4 X-MS-Exchange-CrossTenant-AuthSource: CH2PR12MB4294.namprd12.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 20 Jun 2023 14:10:33.5857 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: rl2u8ScU4wHc0hIy9ViOeR3XAuWpcHD4cF5aIFVMjNrZmDnoT0sSI3xOT/e4w+9F X-MS-Exchange-Transport-CrossTenantHeadersStamped: CH3PR12MB9146 X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org On 6/20/2023 2:15 PM, humin (Q) wrote: > Hi, Niklas, Ferruh, > > 在 2023/6/20 19:03, Niklas Söderlund 写道: >> Hi Connor and Ferruh, >> >> On 2023-06-19 09:57:17 +0100, Ferruh Yigit wrote: >>> On 6/16/2023 1:00 PM, humin (Q) wrote: >>>> Hi, >>>> >>>> 在 2023/6/16 15:20, Chaoyong He 写道: >>>>> From: Zerun Fu >>>>> >>>>> After the mainline Linux kernel commit >>>>> "fe205d984e7730f4d21f6f8ebc60f0698404ac31" (ACPI: Remove side effect >>>>> of partly creating a node in acpi_map_pxm_to_online_node) by >>>>> Jonathan Cameron. When the system does not support NUMA architecture, >>>>> the "socket_id" is expected to be -1. The valid "socket_id" in >>>>> BOND PMD is greater than or equal to zero. So it will cause an error >>>>> when DPDK checks the validity of the "socket_id" when starting the >>>>> bond. This commit can fix this bug. >>>>> >>>>> Fixes: f294e04851fd ("net/bonding: fix socket ID check") >>>>> Cc: stable@dpdk.org >>>>> >>>>> Signed-off-by: Zerun Fu >>>>> Reviewed-by: Peng Zhang >>>>> Reviewed-by: Chaoyong He >>>>> Reviewed-by: Long Wu >>>> No need add your colleagues unless they "reviwed-by" through >>>> email-list. >>>> >>> Hi Connor, >>> >>> This is done time to time, if code is already internally reviewed, send >>> review/ack tags within the patch, to reduce noise in the mail list. >> This is the reason why patches from us usually have 1 or 2 R-b tags when >> we post to the list. We have an internal review and CI pipeline we run >> patches thru to reduce the noise at the list and to not waste upstream >> review resources. >> >> We follow the DPDK workflow internally before we submit patches to the >> public mailing list. I hope we can continue to do so and add R-b tags, >> as they represent real effort by the developers. > > Actually,  this is an interesting story. > The reason why I said so is I met same circumstances a few years ago. > Then I sent one patch with R-b tags which added internally by my > colleagues. > > Someone from mail-list said this was not right.  From then on, every time I > send patches, I will delete R-b tags and send to mail-list. > For a driver, vendor and maintainers are experts and probably internal reviews are good enough. Most of the times details are not interesting to the rest of the community. But for code that impact multiple vendors, like libraries, it is good to have public reviews to share reasoning and updates and record them for future. Sometimes for this code that impact multiple vendor, if the author and maintainer are from same company, we receive a patch with maintainer's ack with it. This raises questions on how much it is reviewed, how many internal version it went through 1 or 11, or what was the design decision reasoning etc.. If these concerns felt somehow, explicit acks from maintainer requested time to time. Or for me, some obvious mistakes in patch, with maintainer's ack already embedded in it raises similar concerns and I request explicit ack. But technically there is nothing preventing ack to be embedded into patch itself.