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 BB44C424EF; Mon, 4 Sep 2023 12:11:32 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 956A5402AF; Mon, 4 Sep 2023 12:11:32 +0200 (CEST) Received: from NAM02-BN1-obe.outbound.protection.outlook.com (mail-bn1nam02on2044.outbound.protection.outlook.com [40.107.212.44]) by mails.dpdk.org (Postfix) with ESMTP id 8C46B400EF; Mon, 4 Sep 2023 12:11:30 +0200 (CEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=K4SAWufyvMjA8dWv1SZT1L6EYmUt9N/WBRJOC0f2kAxjvxYBfdBxiscYPXg+AQ95t6g8PH0v6TD+UxByP6m0HLRwXv5ahLHXdPp3Wx4sHL+75JMDZl38M7DOVBcY3QWYgtwbfHwIxRcBIfOif0WqGL9zEm3yz6B4VipmXalQhIF8V2YU0h/PDKRwic1OqhSdFByNS7/+SOixrew4Z0NYbiiWjUQy83BBti6HcvrlwtJTGGhB+rukv5cPu0CZqbwzqk7v2c6oq4tWcr3rR6Pjt0cbCvIfe1bjABeQ01UfjMgu4M97m+aOukSILOh8rO2SmN6AjQUmU9DMW1Em/Fcomg== 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=+jO/kMSmUUlRI+G351r8zXLA9rM2ygjEIzO85fQtwQ4=; b=dj7iY1vFdFs+7tyMVpROvvoHrQXB/bSLYBh6t7c40sLIFK7op6xH2vqYTK8OYjvusHVqt8uVNSugea3aWO0hLSGogUOfx6ulKpU/m6PEJ+XJBLWMe+6YoUZ+B2urwdFI9M9/8k16TidchS6eU5jJEf2h2x6/7N5usZDyEwu1T2L1HHC5irNiC7g7/ePqpdvEJ66yYkyQqcis4nXcTHra/ua3ibst/I9GQ1mrwuAD6piq4dGNHVvm1C79LSYJnSp5/QpU8wwUvKgyXy5RtObG+mFRLmXy3qoNvEiFHvyMRh5JY621pzGfZteYvEvfec3azhrd2f2JmqJyZkJ1VFIh3A== 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=+jO/kMSmUUlRI+G351r8zXLA9rM2ygjEIzO85fQtwQ4=; b=xyXDp8AhMCnJPCjTgZnborFJDx2R2UWTNeFMfrmjh+1dM3TL6Lei1QblFAHAwacg6IVuTDtVExrVWk+wGi6eFlO9IRzH2kVDReGq4QD3d5ZIu5RQ5YyE18hlGjm9SAyy3DPldA9v7cy+FDrxflSKyQh14qXJkUSbr460sELa3tw= 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 PH7PR12MB6935.namprd12.prod.outlook.com (2603:10b6:510:1b9::16) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6745.32; Mon, 4 Sep 2023 10:11:27 +0000 Received: from CH2PR12MB4294.namprd12.prod.outlook.com ([fe80::49e9:2bf6:7f06:bbbd]) by CH2PR12MB4294.namprd12.prod.outlook.com ([fe80::49e9:2bf6:7f06:bbbd%3]) with mapi id 15.20.6745.030; Mon, 4 Sep 2023 10:11:27 +0000 Message-ID: <40e74786-5854-37c2-7f52-1e5d43996b5d@amd.com> Date: Mon, 4 Sep 2023 11:11:20 +0100 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101 Thunderbird/102.15.0 Content-Language: en-US To: Konstantin Ananyev , Bruce Richardson , Stephen Hemminger Cc: "Varghese, Vipin" , "thomas@monjalon.net" , "dev@dpdk.org" , "techboard@dpdk.org" , =?UTF-8?Q?Morten_Br=c3=b8rup?= References: <20230812005720.997-1-vipin.varghese@amd.com> <20230812080025.7626a94d@hermes.local> <20230813085201.719e7a73@hermes.local> From: Ferruh Yigit Subject: Re: [PATCH] usertools: suggest use of hwloc for new cpu In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-ClientProxiedBy: LO3P123CA0033.GBRP123.PROD.OUTLOOK.COM (2603:10a6:600:388::11) To CH2PR12MB4294.namprd12.prod.outlook.com (2603:10b6:610:a9::11) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: CH2PR12MB4294:EE_|PH7PR12MB6935:EE_ X-MS-Office365-Filtering-Correlation-Id: 1cb859e5-9c6d-4fc4-996f-08dbad2f4ce0 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: 3jmNwJQc4vlikreqDI52k0flcMpfKi71mFu8AKFk9sIZPe7qgERQmNVLFG8E+aelZUcc7r88ewnMyTo0v+uq2Vab759aAmc356acAsQJz3oU03wjroKlrRlBy28za/hOIXjm4hT1tXwm066Ehc/XWMUsKXkW7IcjI6aJ6Pr5fyjpvBjiwiic192XvWhtNA7jC1+6wg3EH/xNPxxRYLLMqc761zly7GGBFnjat0PyjsO2f2V8HxRCmITcvzvQvt8HJ9sPc31dQB10DlKIlN/WaycNYe13emVwjlcW3ADvVAc9uBsZGNb2vXpxkBv6LFpIHOYqWOzmZ76UullZtsEJNSI6wOAwNoEFEFdFHukVBKEk+yq1rTbWWF3dSDqGq1dZDSVlNOFwTb/MdeMjgR9Zclm7tEWP1XmuhL7Gsta+4453AeWDh9HG3nO8ImWyt4CbQlZQnZ7VScIUfWNfIMnhr4qV5Py80jEbY2yzxmInT0JqnIvNNj4YLCeCW8fsMd9yHgDsV1vItyFYOa4r1RQqOv44IfVg5yxF3jp4oaRUue6XBnv7Yr0WaTBPhfL/Mg3mNIQcg1eyC39E5QvntQAkj7nJHaJUyuaaahThjxdgUiKLt+fT3nmuHbOBmYMhUUdiquJWGMObwkKbJIH1zo5NGQ== 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:(13230031)(136003)(366004)(376002)(396003)(346002)(39860400002)(186009)(1800799009)(451199024)(6666004)(478600001)(966005)(53546011)(110136005)(54906003)(66946007)(66556008)(66476007)(26005)(2616005)(6486002)(6506007)(6512007)(2906002)(8676002)(4326008)(8936002)(316002)(41300700001)(44832011)(5660300002)(86362001)(31696002)(36756003)(38100700002)(83380400001)(31686004)(66899024)(43740500002)(45980500001); DIR:OUT; SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?ZHVHOXdsaG51UW9PVlN0RVhCUzlJYWFQU0dnNDNHYWZwWkJYSUIzRnVIVGM3?= =?utf-8?B?RFNoT2dFaHBoQks3ZW9EdzNiTS9BZ3FmM2ZJVy9adXlTam14YWo2cHRxMGFl?= =?utf-8?B?VWJDbFdjVkJ3enNLZ2x1dW1YTlNPdG9mVlk1Y09DVld4cENwZWlFR3BjbE93?= =?utf-8?B?V2NVYmdnS2xKUWtId3JXQzlWY0VXSmF3L3Nxb253aVpzc2U5Q3hocVBHVUpZ?= =?utf-8?B?bDdIUHM2eVJDNzV4eWlEVHhTbTJDUWNJZWdyRlMySmhEbzVpQTl0cWl5aTdy?= =?utf-8?B?SU5FS0FISG4xRWtFZmw3a2RNcDA5dTNtVmdabDJLT1ZRaU1rM1lKYVpxODAw?= =?utf-8?B?ZEVnNlcxTnd5QU80NGxJMEdQSmhmN05wVHNLN2gwSEh4ajg1MHV5Vy9RYjZI?= =?utf-8?B?K1Y5MnRGR3ViQ3o1TEJGSDdSSTBqRGtBMFJPWDNraVF4TXFtYjNJQ2hyNUFM?= =?utf-8?B?RWlobEREV1Z3RjRkWFliWUZIeW5MY1RJSzJQVHNKYXZUN0JMRXdJNkRQdjdo?= =?utf-8?B?TytmczQvNFkwN2FITjlLZ2FDaGJ2QkE0RW5YTWNhVmkzcEFJK3d4KzdRRE9M?= =?utf-8?B?ellac0RtdTdtNWRXQUZ6aHZFM2Z6dFRzNkI1U3h3aTltSTZpdnFDdmNNOXZL?= =?utf-8?B?WG9CdVNqNDZiNHBaMTBHTkV5V25NU3lXcXNtUWdGbDJMODcyQ05YQW4vMm1D?= =?utf-8?B?S3kxdEJCbXBlTTR4SjVRcmpQR25mNGhmYlE4NVYrUGVnRWdnVmFlQjhhNWR1?= =?utf-8?B?a2FqSWpUb3ROQktVc2lwTlBMRVRZTFMxUkpKOWxkZlFObWN2NUNQb210eG54?= =?utf-8?B?YW82Qzd0QTljRlh1eWkrWFpXM001OGtSalQ1cEFTOTdOK2NIK0VHOHBLV1Fz?= =?utf-8?B?Y2xiNmlPVmJHYkw0NmwzTnVBd1VTTWxDcHArNkFmeHJDcU8reTNQaGhFeWRR?= =?utf-8?B?RjBnWVdiTStSWmdJUmJTUVZvUFBBWTU5Q1UvcFR5Ny9NLzhPM0I3bXNycnpD?= =?utf-8?B?UVR6cHAxaHIweW9MR25DZmQzOHUvVUJDZDNpcUJtSm4zWll3Ti9lSytZejk5?= =?utf-8?B?MDZRdkw3SWFPZ2FwWkk1a3g1Wjl0UjlCdkVlNVRZQ3MyZ09wWkgxNEg3TUVD?= =?utf-8?B?enFzWU9CQjlYU0tuMnNlYUFVODc5c1BMK3hCbkh0ZFBadTlsWjhCekY5dDZn?= =?utf-8?B?NnVGeWVFN0hNTzFTOGZMb2x6UlZPTUlNSDFSVm8zYTVldWduWGNUY2ZTcHhY?= =?utf-8?B?bUJxMWx2ZXlTRWdHVEJFSmVXd25sTmRYb2Rzam56akFhejd2UWdKakpjMElT?= =?utf-8?B?YlpFdDNPMnB2amRDcDh0VHNCay9rVTBIaG1iU2c0ZGsvYlM2V2REN0cyenZR?= =?utf-8?B?REVkSDFXWE5IcVY4azQ1UnM0ZjU4KzhyNU52QWxsaCtqZ3liQ3UxTFNmTDV2?= =?utf-8?B?OGJnQXhvTm55dHpaS1hUNGJRMGZWUCtITWJxNVJJZ0tyUmtvWnZWQS9FVFJB?= =?utf-8?B?eHhOMnluKzhiUlAzRjFqNUg2SmxpRGlwMmZid2RZbkxiYWFPcGVFeEVveDJa?= =?utf-8?B?TkFmRm11L29VazRjTGVtazlDYkdIRStlNDUzV3p3Z3M0d3JQTzJCZ3RIOUQ3?= =?utf-8?B?RS8wa3RIN3Z5Q2U0OEZQalArU09UWFFpWWFkWXBtWWQyN2ptRzZBOUc4aFBW?= =?utf-8?B?Q3Q3NFBoWnRhZlprVnRKemJyT2NBVjlqa0NZQjB6NVlxcXU2T2dpbkFUVWl6?= =?utf-8?B?bzhDTXhMRmNqeTlveG9yc2VSZ1BHbWFLVTNJT2laeFZ0dUJZM0J2UmJrMkJI?= =?utf-8?B?djJZTFZmMHNDbCtkZktlb2ZDTGFCb3Z0WnY4UG9xM3JUUWxYRlhMQStyWXpD?= =?utf-8?B?WUtIMG1rdHhQdjRtMXhUbEx3UEF4OWFzU3RUR0VhUWltTEp2bDNPWUNSRFZa?= =?utf-8?B?ZFgvZWI4dnQzWWQwaC83dkJOVjEvdy8rRUF3Nm9BTEFzYTJNa2U2ajBNZU80?= =?utf-8?B?UlBPY0JGamNzeWIyM2VLMXVzbWw0bXZCdVd5WTdQV0dYQVVpa041Z3E5eVJv?= =?utf-8?B?MU9NSk5Gekp1blNXSGtqUlcvMjlKc25Xa1lFbHRGb3JtMzl1MnR2RUttSGdx?= =?utf-8?Q?XJWZoAgPk+RM8mOzS1x7Gp1Cg?= X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-Network-Message-Id: 1cb859e5-9c6d-4fc4-996f-08dbad2f4ce0 X-MS-Exchange-CrossTenant-AuthSource: CH2PR12MB4294.namprd12.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 04 Sep 2023 10:11:27.2421 (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: +2m8G6o67n91Eq2XvBpkJ3Q04jOOOn1xQTWIY9X2gbiW1EX+WdK7bAgzw4tBigEv X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH7PR12MB6935 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 8/14/2023 10:25 AM, Konstantin Ananyev wrote: > >> On Sun, Aug 13, 2023 at 08:52:01AM -0700, Stephen Hemminger wrote: >>> On Sun, 13 Aug 2023 02:12:03 +0000 >>> "Varghese, Vipin" wrote: >>> >>>>> >>>>> On Sat, 12 Aug 2023 06:27:20 +0530 >>>>> Vipin Varghese wrote: >>>>> >>>>>> Most modern processor now supports numa by partitioning NUMA based on >>>>>> CPU-IO & Last Level Cache within the same socket. >>>>>> As per the discussion in mailing list, suggesting the make use of >>>>>> hw-loc for such scenarios. >>>>>> >>>>>> Signed-off-by: Vipin Varghese >>>>> >>>>> NAK, no scripting hwloc, it is ugly and creates a dependency that is not listed >>>>> in DPDK packaging. >>>> >>>> There is no calls to hwloc within in thescript. Hence not clear what does ` NAK, no scripting hwloc it is ugly and creates a >> dependency that is not listed in DPDK packaging.`. >>>> >>>> Requesting to cross check why NAK is shared for `print` as suggestion. Hence, I have disagree to this. >>> >>> Sorry, I misinterpreted what the print's were doing. >>> Better off not to list exact flags, the lstopo may change and user may want different >>> format anyway. >>> >>> How about something like this? >>> >>> >>> doc/guides/rel_notes/deprecation.rst | 5 +++++ >>> usertools/cpu_layout.py | 5 +++++ >>> 2 files changed, 10 insertions(+) >>> >>> diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst >>> index 317875c5054b..25a116900dfb 100644 >>> --- a/doc/guides/rel_notes/deprecation.rst >>> +++ b/doc/guides/rel_notes/deprecation.rst >>> @@ -185,3 +185,8 @@ Deprecation Notices >>> will be deprecated and subsequently removed in DPDK 24.11 release. >>> Before this, the new port library API (functions rte_swx_port_*) >>> will gradually transition from experimental to stable status. >>> + >>> +* cpulayout: The CPU layout script is unable to deal with all the possible >>> + complexities of modern CPU topology. Other existing tools offer more >>> + features and do a better job with keeping up with innovations. >>> + Therefore it will be deprecated and removed in a future release. >> >> Does the script really do that bad a job? While I can understand us looking >> to recommend alternatives, I actually find the script in it's current form >> really handy - much more so than working out the exact flags for lstopo >> etc. Since it's not a large maintenance burden, I'd request we keep it >> around - while still recommending lstopo to users. > > +1 > I do use it on regular basis. > It would be a pity if it will be gone. > I also use it time to time and find it useful. But it is not accurate/correct for some AMD platforms (for various NPS (Nodes per Socket) values). So either it needs to be updated/improved or replaced. Vipin sent a patch [1] to update it but it is question how much of this logic belongs to DPDK, or should we rely on external tools dedicated for this purpose. Both are OK, if we can give a decision we can proceed on that direction. [1] https://patchwork.dpdk.org/project/dpdk/patch/20220326073207.489694-1-vipin.varghese@amd.com/